Skip to main content
Glama

Angular PR Review MCP

Reusable MCP server for automated pull-request review workflows across multiple Angular repositories.

This package exposes three MCP tools:

  • get_pr_review_template

  • generate_pr_diff

  • read_pr_diff

  • save_pr_review

It runs in the target repository context (the current working directory), so one published package can be reused by any Angular app.

Requirements

  • Node.js 20+

  • Git installed and available in PATH

  • VS Code with Copilot Chat MCP support

Related MCP server: GitHub Code Reviewer

Local Development

npm install
npm run build
npm run dev

Publish To npm

If publishing publicly:

npm login
npm publish --access public

If publishing under a scoped package (recommended for account/org ownership):

  1. Change the name field in package.json to your scope, for example @your-scope/angular-pr-review-mcp.

  2. Publish:

npm login
npm publish --access public

Create Separate GitHub Repository

From this folder, create and push to a dedicated repository in your account:

git init
git branch -M main
git remote add origin https://github.com/<your-username>/angular-pr-review-mcp.git
git add .
git commit -m "Initial reusable Angular PR review MCP package"
git push -u origin main

Use From Any Angular Repository

Add or update .vscode/mcp.json in each Angular repository:

{
  "servers": {
    "angular-pr-review": {
      "type": "stdio",
      "command": "npx",
      "args": ["-y", "angular-pr-review-mcp"],
      "cwd": "${workspaceFolder}",
      "env": {
        "PR_REVIEW_BASE_BRANCH": "main",
        "PR_REVIEW_OUTPUT_DIR": "pr_review"
      }
    }
  }
}

For a scoped package, replace angular-pr-review-mcp with your published name.

Environment Variables

  • PR_REVIEW_BASE_BRANCH: target base branch for comparison (default main)

  • PR_REVIEW_OUTPUT_DIR: review artifacts directory (default pr_review under repo root)

  • PR_REVIEW_REPO_ROOT: override repository root (default current working directory)

  • PR_REVIEW_SERVER_NAME: MCP server name (default angular-pr-review)

Tool Workflow

  1. generate_pr_diff with commit SHA.

  2. read_pr_diff to retrieve diff content.

  3. save_pr_review to persist markdown review comments.

Recommended flow:

  1. get_pr_review_template first, to load the local review rubric and avoid re-deriving rules each run.

  2. generate_pr_diff and read_pr_diff for evidence collection.

  3. save_pr_review for the final report.

Generated files are written into the configured output folder in each consuming repository.

Local Template Artifact

The repository includes pr_review_prompt_template.md so teams can version review guidance in source control. The get_pr_review_template tool returns the same policy text (with optional commit token replacement), which helps keep reviews deterministic and reduces token consumption.

Available Tools

4 tools
generate_pr_diffB

Generate a PR review diff for a validated Git commit hash and return the resulting diff file path.

ParametersJSON Schema
NameRequiredDescriptionDefault
commitHashYes

TDQS

B3.3/5.0
Behavior3/5

Does the description disclose side effects, auth requirements, rate limits, or destructive behavior?

With no annotations provided, the description carries the full burden of behavioral disclosure. It does reveal a key side effect: the tool creates a diff file and returns its path. However, it is vague about what 'validated' means and does not explain if any resources are modified or side effects occur beyond file creation. It adds some transparency but leaves room for interpretation.

Agents need to know what a tool does to the world before calling it. Descriptions should go beyond structured annotations to explain consequences.

Conciseness5/5

Is the description appropriately sized, front-loaded, and free of redundancy?

The description is a single, tight sentence of 18 words. It fronts the main verb 'Generate', includes all essential elements (action, object, input, output), and wastes no words. It is a model of concise writing.

Shorter descriptions cost fewer tokens and are easier for agents to parse. Every sentence should earn its place.

Completeness3/5

Given the tool's complexity, does the description cover enough for an agent to succeed on first attempt?

For a tool with one parameter and no output schema, the description covers the basic action and return value. However, it does not clarify how this tool fits into the broader workflow with siblings like 'read_pr_diff' and 'save_pr_review', nor does it mention failure behaviors or the nature of the output path. It is adequate but not comprehensive for an agent to invoke it with full confidence in all scenarios.

Complex tools with many parameters or behaviors need more documentation. Simple tools need less. This dimension scales expectations accordingly.

Parameters3/5

Does the description clarify parameter syntax, constraints, interactions, or defaults beyond what the schema provides?

The input schema describes the 'commitHash' parameter with a regex pattern and length limits, but the description adds the semantic that it should be a valid Git commit hash. This is useful but minimal, especially since schema coverage is 0% and the parameter name already hints at its content. The description does not elaborate on format or edge cases, so it only partially compensates for the lack of schema documentation.

Input schemas describe structure but not intent. Descriptions should explain non-obvious parameter relationships and valid value ranges.

Purpose4/5

Does the description clearly state what the tool does and how it differs from similar tools?

The description clearly states the action ('Generate a PR review diff'), the input ('for a validated Git commit hash'), and the output ('return the resulting diff file path'). It is specific enough to understand the tool's purpose, though it does not explicitly differentiate from the sibling 'read_pr_diff' or mention alternatives, missing the top score.

Agents choose between tools based on descriptions. A clear purpose with a specific verb and resource helps agents select the right tool.

Usage Guidelines2/5

Does the description explain when to use this tool, when not to, or what alternatives exist?

The description gives no guidance on when to use this tool versus the siblings (e.g., 'use this to create a diff before reading it'). There is no indication of prerequisites beyond the hash being 'validated', and no mention of alternatives or conditions. The usage context must be inferred entirely from the name and description.

Agents often have multiple tools that could apply. Explicit usage guidance like "use X instead of Y when Z" prevents misuse.

get_pr_review_templateB

Return the local Angular PR review template used to keep review rules consistent and token-efficient.

ParametersJSON Schema
NameRequiredDescriptionDefault
commitHashNo

TDQS

B3.1/5.0
Behavior3/5

Does the description disclose side effects, auth requirements, rate limits, or destructive behavior?

No annotations are present, so the description alone must convey the operation's nature. It fails to explicitly state this is a non-destructive read, though 'return' is a safe verb. The description is accurate but doesn't go far in revealing behavior beyond the function name.

Agents need to know what a tool does to the world before calling it. Descriptions should go beyond structured annotations to explain consequences.

Conciseness5/5

Is the description appropriately sized, front-loaded, and free of redundancy?

A single, clear sentence that efficiently communicates the tool's purpose without wasted words. The intended characteristic ('token-efficient') is implicit and effective.

Shorter descriptions cost fewer tokens and are easier for agents to parse. Every sentence should earn its place.

Completeness3/5

Given the tool's complexity, does the description cover enough for an agent to succeed on first attempt?

For a simple read tool with one optional parameter and no output schema, this description is largely sufficient. The main gaps are the lack of detail about the commitHash parameter and absence of explicit non-destructive language, but these are minor for a tool of this complexity.

Complex tools with many parameters or behaviors need more documentation. Simple tools need less. This dimension scales expectations accordingly.

Parameters2/5

Does the description clarify parameter syntax, constraints, interactions, or defaults beyond what the schema provides?

The description gives no explanation of the commitHash parameter. While an optional commitHash might reasonably be inferred to be a Git commit hash, the format, purpose (likely to fetch the template at that commit), and impact are left entirely to the agent. More guidance would be valuable.

Input schemas describe structure but not intent. Descriptions should explain non-obvious parameter relationships and valid value ranges.

Purpose4/5

Does the description clearly state what the tool does and how it differs from similar tools?

Description clearly identifies the tool's function: returning the local Angular PR review template. Although it doesn't mention how it differs from sibling tools, the verb 'return' and the specific resource (Angular PR review template) make the purpose unambiguous.

Agents choose between tools based on descriptions. A clear purpose with a specific verb and resource helps agents select the right tool.

Usage Guidelines2/5

Does the description explain when to use this tool, when not to, or what alternatives exist?

The description implies this is for users needing the review template, and the phrase 'used to keep review rules consistent' gives some context, but there's no explicit guidance on when to use this tool instead of the siblings. The description labels the template as a 'local' resource, which hints at a distinguishing factor, but it doesn't spell out when an agent should choose this tool over the others.

Agents often have multiple tools that could apply. Explicit usage guidance like "use X instead of Y when Z" prevents misuse.

read_pr_diffA

Read the most recently generated PR diff file, or a specific diff by commit hash/file path.

ParametersJSON Schema
NameRequiredDescriptionDefault
filePathNo
commitHashNo

TDQS

A4.4/5.0
Behavior4/5

Does the description disclose side effects, auth requirements, rate limits, or destructive behavior?

With no annotations provided, the description carries the burden of expressing behavioral traits. It explicitly frames the operation as 'Read' and 'read a PR diff,' which strongly indicates a non-mutating operation. It also explains the target's selection behavior (most recent by default or specific diff via commit/file path). It doesn't cover edge cases like missing diff files or parameter conflicts, but the core behavioral intent is clear and supported.

Agents need to know what a tool does to the world before calling it. Descriptions should go beyond structured annotations to explain consequences.

Conciseness5/5

Is the description appropriately sized, front-loaded, and free of redundancy?

The description is a single sentence that front-loads the primary behavior ('Read the most recently generated PR diff file') and then appends the optional two selection modes. There is no redundant wording or restatement of the tool name, and every word contributes to the tool's meaning.

Shorter descriptions cost fewer tokens and are easier for agents to parse. Every sentence should earn its place.

Completeness4/5

Given the tool's complexity, does the description cover enough for an agent to succeed on first attempt?

This is a simple two-parameter, read-only tool with no required parameters, no output schema, and no annotations. The description covers the main invocation paths that an agent needs: read latest by omitting parameters, or read a specific diff by supplying a commit hash or file path. It omits details like error handling when no diff exists or combined parameter behavior, but those are minor for this straightforward tool.

Complex tools with many parameters or behaviors need more documentation. Simple tools need less. This dimension scales expectations accordingly.

Parameters4/5

Does the description clarify parameter syntax, constraints, interactions, or defaults beyond what the schema provides?

Schema description coverage is 0%, so the description must add meaning to the parameters on its own. It does: 'commit hash/file path' directly maps to the commitHash and filePath properties, and clarifies that these modes override the default most-recent behavior. It doesn't explain the exact path semantics or how the two parameters interact if both are provided, but it gives enough information to correctly choose an optional parameter.

Input schemas describe structure but not intent. Descriptions should explain non-obvious parameter relationships and valid value ranges.

Purpose5/5

Does the description clearly state what the tool does and how it differs from similar tools?

The description uses a specific verb ('Read') and a clear resource ('PR diff file'), and immediately distinguishes between the default behavior (most recently generated) and the alternate modes (specific commit hash or file path). This clearly separates it from siblings like generate_pr_diff, get_pr_review_template, and save_pr_review.

Agents choose between tools based on descriptions. A clear purpose with a specific verb and resource helps agents select the right tool.

Usage Guidelines4/5

Does the description explain when to use this tool, when not to, or what alternatives exist?

The description states the main use cases: read the latest generated diff or read a specific diff by commit hash or filePath. It implies this tool should be used after a diff has been generated, which gives useful context. It does not explicitly name alternatives, but the sibling tools are self-evidently different, so only minor additional guidance would be needed.

Agents often have multiple tools that could apply. Explicit usage guidance like "use X instead of Y when Z" prevents misuse.

save_pr_reviewB

Save a completed review as a markdown report inside the configured output folder.

ParametersJSON Schema
NameRequiredDescriptionDefault
contentYes
commitHashYes

TDQS

B3.3/5.0
Behavior3/5

Does the description disclose side effects, auth requirements, rate limits, or destructive behavior?

With no annotations present, the description is the sole source of behavioral information. It discloses that the tool writes a markdown file to a folder, which is a useful side‑effect detail, but it omits any mention of overwrite behavior, file naming, error handling, or whether the action is reversible. For a write operation, this is a moderate disclosure.

Agents need to know what a tool does to the world before calling it. Descriptions should go beyond structured annotations to explain consequences.

Conciseness5/5

Is the description appropriately sized, front-loaded, and free of redundancy?

The description is a single, grammatically clean sentence that conveys the essential information without any fluff. It is appropriately front-loaded with the action and includes only the necessary details about format and destination.

Shorter descriptions cost fewer tokens and are easier for agents to parse. Every sentence should earn its place.

Completeness3/5

Given the tool's complexity, does the description cover enough for an agent to succeed on first attempt?

For a tool with just two simple scalar parameters and no nested objects, the description covers the high-level behavior adequately but falls short on parameter-level detail and its role in the broader workflow with sibling tools. The mention of a 'configured output folder' assumes background knowledge that is not elaborated, yet the overall simplicity of the tool keeps the description from being grossly incomplete.

Complex tools with many parameters or behaviors need more documentation. Simple tools need less. This dimension scales expectations accordingly.

Parameters2/5

Does the description clarify parameter syntax, constraints, interactions, or defaults beyond what the schema provides?

Since the schema has 0% description coverage, the description bears the responsibility of clarifying parameters, but it fails to explain either 'content' or 'commitHash'. While 'content' is somewhat self-evident as the review text, 'commitHash' is left to the reader's inference, and the description offers no additional context about the relationship between the two.

Input schemas describe structure but not intent. Descriptions should explain non-obvious parameter relationships and valid value ranges.

Purpose4/5

Does the description clearly state what the tool does and how it differs from similar tools?

The description clearly states the action ('save'), the resource ('a completed review'), the output format ('markdown report'), and the destination ('configured output folder'), making the tool's purpose immediately understandable. It is distinct from siblings like get_pr_review_template or generate_pr_diff, although it doesn't explicitly name them.

Agents choose between tools based on descriptions. A clear purpose with a specific verb and resource helps agents select the right tool.

Usage Guidelines3/5

Does the description explain when to use this tool, when not to, or what alternatives exist?

The description implies usage after a review is complete ('a completed review') and mentions a 'configured output folder,' giving some contextual clue. However, it provides no explicit when-to-use guidance, prerequisites, or alternatives, leaving the agent to infer the appropriate context from limited evidence.

Agents often have multiple tools that could apply. Explicit usage guidance like "use X instead of Y when Z" prevents misuse.

Tool Schema Changelog

Recent tool additions, removals, and schema changes observed during successful MCP inspections.

  1. 4 tool updatesv1.0.0
    • First observedgenerate_pr_diff
    • First observedget_pr_review_template
    • First observedread_pr_diff
    • First observedsave_pr_review

TDQS

A3.7/5.0

Scored across 4 tools

Disambiguation5/5

Each tool serves a distinct purpose: fetching the review template, generating a diff, reading a diff, and saving a review. There is no overlap or ambiguity in their functionalities.

Naming Consistency4/5

All tools follow a consistent verb_noun pattern (get, generate, read, save) with clear object references. The only minor deviation is that 'get_pr_review_template' and 'generate_pr_diff' could be seen as slightly more specific verbs, but they are still consistent with the overall style.

Tool Count5/5

With 4 tools, the server is well-scoped for a focused PR review workflow. Each tool covers an essential step: template retrieval, diff generation, diff reading, and saving results, without unnecessary extras.

Completeness4/5

The tool surface covers the core workflow of generating and reviewing a PR diff. However, a minor gap might be the lack of a tool to fetch PR metadata or commit details beyond the diff, but the current set is sufficient for the stated purpose.

Maintenance

ActivityMaintained
ResponsivenessNo issues

Related MCP Connectors

Related MCP Servers