feat(#258): add file_path parameter to note_publish_article#259
Merged
Conversation
Add file_path parameter to note_publish_article tool to extract tags from Markdown frontmatter when publishing existing drafts. This allows users to preserve tags from the original Markdown file during publish. - When article_id is specified with file_path (and no tags), tags are extracted from the Markdown file's YAML frontmatter - Explicit tags parameter takes precedence over file_path - file_path is ignored for new article creation (title/body mode) - Returns appropriate error message if file_path doesn't exist Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Add file_path to expected properties in note_publish_article schema test. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Add ValueError handling when parsing Markdown file for tags extraction. This prevents unhandled exceptions when the file has no title. Also add test case for invalid Markdown file scenario. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
file_pathparameter tonote_publish_articletool to extract tags from Markdown frontmatter when publishing existing draftsarticle_idis specified withfile_path(and no explicittags), tags are extracted from the Markdown file's YAML frontmattertagsparameter takes precedence overfile_pathfile_pathis ignored for new article creation (title/body mode)file_pathdoesn't existTest plan
file_pathprovides tags whentagsnot specifiedtagsparameter takes precedence overfile_pathfile_pathwithout tags publishes without tagsfile_pathnot found returns errorfile_pathignored for new article creationCloses #258
🤖 Generated with Claude Code