Skip to content

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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

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