Skip to content

refactor(@angular/cli): use main element extraction for MCP doc search #30959

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Aug 20, 2025

Conversation

clydin
Copy link
Member

@clydin clydin commented Aug 19, 2025

The angular.dev documenation now uses main elements within each page to indicate the actual content. To reduce the amount of HTML passed back in the MCP search_documentation tool, only the main element is now used instead of the entire body element.

The angular.dev documenation now uses `main` elements within each page to
indicate the actual content. To reduce the amount of HTML passed back in
the MCP `search_documentation` tool, only the `main` element is now used
instead of the entire `body` element.
@clydin clydin added action: review The PR is still awaiting reviews from at least one requested reviewer target: rc This PR is targeted for the next release-candidate labels Aug 19, 2025
@clydin clydin requested a review from alan-agius4 August 19, 2025 14:47
@alan-agius4 alan-agius4 added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Aug 20, 2025
@alan-agius4 alan-agius4 merged commit f88fd74 into angular:main Aug 20, 2025
34 checks passed
@alan-agius4
Copy link
Collaborator

This PR was merged into the repository. The changes were merged into the following branches:

alan-agius4 added a commit to alan-agius4/dev-infra that referenced this pull request Aug 20, 2025
This commit adds the unique commit SHA next to each branch listed in the pull request merge message. This provides a more specific reference to the exact commit that was merged.

Example: angular/angular-cli#30959 (comment)
alan-agius4 added a commit to alan-agius4/dev-infra that referenced this pull request Aug 20, 2025
This commit adds the unique commit SHA next to each branch listed in the pull request merge message. This provides a more specific reference to the exact commit that was merged.

Example: angular/angular-cli#30959 (comment)
alan-agius4 added a commit to alan-agius4/dev-infra that referenced this pull request Aug 20, 2025
This commit adds the unique commit SHA next to each branch listed in the pull request merge message. This provides a more specific reference to the exact commit that was merged.

Example: angular/angular-cli#30959 (comment)
alan-agius4 added a commit to alan-agius4/dev-infra that referenced this pull request Aug 20, 2025
This commit adds the unique commit SHA next to each branch listed in the pull request merge message. This provides a more specific reference to the exact commit that was merged.

Example: angular/angular-cli#30959 (comment)
alan-agius4 added a commit to alan-agius4/dev-infra that referenced this pull request Aug 20, 2025
This commit adds the unique commit SHA next to each branch listed in the pull request merge message. This provides a more specific reference to the exact commit that was merged.

Example: angular/angular-cli#30959 (comment)
@clydin clydin deleted the mcp/doc-search-main-content branch August 20, 2025 14:23
josephperrott pushed a commit to angular/dev-infra that referenced this pull request Aug 20, 2025
This commit adds the unique commit SHA next to each branch listed in the pull request merge message. This provides a more specific reference to the exact commit that was merged.

Example: angular/angular-cli#30959 (comment)

PR Close #2979
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
action: merge The PR is ready for merge by the caretaker area: @angular/cli target: rc This PR is targeted for the next release-candidate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants