Skip to content

Allow loading external content during review#6086

Draft
padenot wants to merge 11 commits into
mozilla:masterfrom
padenot:load-skills-reorganized
Draft

Allow loading external content during review#6086
padenot wants to merge 11 commits into
mozilla:masterfrom
padenot:load-skills-reorganized

Conversation

@padenot
Copy link
Copy Markdown
Collaborator

@padenot padenot commented May 27, 2026

No description provided.

@padenot padenot marked this pull request as draft May 27, 2026 17:30
@padenot padenot force-pushed the load-skills-reorganized branch 3 times, most recently from 0048d1b to fac1b71 Compare May 28, 2026 17:23
padenot added 9 commits June 1, 2026 16:36
Adds the moz.tools MCP server to the Anthropic client so the agent can
call mozilla__ tools during review. URL is overridable via
BUGBUG_MOZ_MCP_URL.
LocalPatch wraps a raw unified diff string as a first-class Patch so it
can flow through the review pipeline without a Phabricator backing.
It accepts an optional commit_message and parses from it: the patch
title (first line), description (remaining lines), bug ID (Bug NNN),
and Differential Revision URL, and date if present in the format-patch
header.

On the MCP server side, patch_review now accepts optional diff= and
commit_message= parameters alongside patch_url=, and a new
patch_review_tool exposes the same logic as an @mcp.tool(). A shared
_patch_review_impl handles both cases, using a lighter CodeReviewTool
instance (no DB, no suggestion filtering) for local diffs.
LOCAL_SYSTEM_PROMPT_TEMPLATE uses searchfox-cli and local agent tools
instead of the agent-specific expand_context and searchfox tools.
@padenot padenot force-pushed the load-skills-reorganized branch 2 times, most recently from 2d67a7c to 8c4fe6d Compare June 1, 2026 16:22
@padenot padenot force-pushed the load-skills-reorganized branch from 8c4fe6d to cab6ec7 Compare June 1, 2026 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant