Skip to content

refactor(@angular/cli): add markdown-dir support to find_examples MCP tool#31799

Closed
clydin wants to merge 1 commit intoangular:mainfrom
clydin:mcp/example-markdown-dirs
Closed

refactor(@angular/cli): add markdown-dir support to find_examples MCP tool#31799
clydin wants to merge 1 commit intoangular:mainfrom
clydin:mcp/example-markdown-dirs

Conversation

@clydin
Copy link
Member

@clydin clydin commented Nov 13, 2025

This commit enhances the find_examples tool within the MCP server to load examples directly from directories of structured markdown files, in addition to the existing SQLite format.

A new format type, markdown-dir, is now supported in the angular.examples metadata in a package's package.json. When this format is detected, the tool will dynamically build an in-memory example database by parsing the markdown files from the specified directory.

… tool

This commit enhances the `find_examples` tool within the MCP server to load examples directly from directories of structured markdown files, in addition to the existing SQLite format.

A new `format` type, `markdown-dir`, is now supported in the `angular.examples` metadata in a package's `package.json`. When this format is detected, the tool will dynamically build an in-memory example database by parsing the markdown files from the specified directory.

This change includes:
-  Safety checks to limit the number and size of markdown files processed.
- A backward-compatible format versioning system for the markdown front matter to ensure future compatibility.
@clydin clydin force-pushed the mcp/example-markdown-dirs branch from ad3dd16 to 4f6b100 Compare November 13, 2025 23:54
@clydin clydin added target: rc This PR is targeted for the next release-candidate target: minor This PR is targeted for the next minor release action: review The PR is still awaiting reviews from at least one requested reviewer and removed target: rc This PR is targeted for the next release-candidate labels Nov 13, 2025
Copy link
Contributor

@hybrist hybrist left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some of this logic is getting pretty complex - do we have plans to add more tests to this?

@clydin clydin added action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Nov 15, 2025
@clydin clydin closed this Nov 18, 2025
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Dec 19, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews area: @angular/cli target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants