agentleFS
harnessharness/harness-skills/skills/observability-governance/pr-analysis/SKILL.md

pr-analysis

Skill127 starsChanged 31 days agoApache-2.0View on GitHub

Analyze pull request impact on Harness pipelines via MCP. Assess PR pipeline impact by identifying which pipelines, services, and environments are affected by code changes. Perform security-focused PR reviews checking for secrets, vulnerabilities, and compliance. Track PRs from merge through deployment to production verification. Use when asked to analyze PR impact, review PR security, or track a PR to production. Trigger phrases: PR impact, pull request analysis, PR security review, PR to production, merge impact, deployment tracking, code change impact, PR pipeline, which pipelines affected.

5 other projects use a version of this.

Adopted by

Most recently taken up by

The file

Show its rules and text

We split this skill into the rules it gives an agent

17 rules. 0 are common, 17 are distinctive, and 0 are very specific to this project.

  • 0

    Common

    Most of the 5 edited versions we read keep these.

  • 17

    Distinctive

    This author's own choices. Worth a look.

  • 0

    Very specific

    Tied to their project's names, files or tools. Probably skip.

  • Common
  • Distinctive
  • Very specific
  1. Distinctive:

    Check for new environment variables that should use Harness Secrets

    line 84
  2. Distinctive:

    Verify no secrets are logged or exposed in error messages

    line 85
  3. Distinctive:

    Check for new dependencies with known CVEs

    line 88
  4. Distinctive:

    Verify license compatibility of new dependencies

    line 89
  5. Distinctive:

    Check for overly permissive RBAC or IAM changes

    line 93
  6. Distinctive:

    Verify network policies and security group modifications

    line 94
  7. Distinctive:

    Verify changes don't violate OPA policies configured in Harness

    line 98
  8. Distinctive:

    Check if changes require additional security review or approval

    line 99
  9. Distinctive:

    Test results and coverage delta

    line 125
  10. Distinctive:

    Check trigger path filters -- glob patterns may not match the actual file paths changed

    line 161
  11. Distinctive:

    Verify branch conditions on triggers (e.g., main-only vs.

    line 162
  12. Distinctive:

    Look for manual triggers or scheduled pipelines that also deploy the same service

    line 163
  13. Distinctive:

    Add exceptions for test directories and example files

    line 167
  14. Distinctive:

    Verify secret scanning rules match the actual secret formats in use

    line 168
  15. Distinctive:

    Verify the CI trigger fires on merge to the target branch

    line 171
  16. Distinctive:

    Check that pipeline identifiers in the tracker match the actual pipeline names

    line 172
  17. Distinctive:

    Ensure deployment pipelines have the correct artifact reference to pick up the new build

    line 173

“Common” is measured: kept by at least 6 in 10 of the 5 edited versions of this file we could read. “Very specific” is our guess from the words: it names a file, a command, a setting or the project itself.

Keep what fits and drop the rest. Your version, ready for your agents.
Files
SKILL.md

PR Analysis

Analyze pull request pipeline impact, run security-focused reviews, and track PRs from merge to production deployment.

Instructions

Step 1: Establish Scope

Confirm the service, repository, and PR details.

Call MCP tool: harness_list
Parameters:
  resource_type: "pipeline"
  org_id: "<organization>"
  project_id: "<project>"

Step 2: Identify the PR Analysis Task

Determine which analysis the user needs:

  1. PR Pipeline Impact Analysis -- Which pipelines, services, and environments are affected
  2. Security-Focused PR Review -- Check for secrets, vulnerabilities, and compliance issues
  3. PR to Production Tracker -- Track a PR from merge through deployment and verification

Step 3: PR Pipeline Impact Analysis

Gather from the user:

  • Service name and repository
  • PR description and changed components (files/directories modified)

Identify affected resources:

Call MCP tool: harness_list
Parameters:
  resource_type: "trigger"
  org_id: "<organization>"
  project_id: "<project>"

Analyze impact:

  • Pipelines triggered: Which CI/CD pipelines will fire on merge (based on triggers and path filters)
  • Services affected: Which Harness services reference the changed code or manifests
  • Environments reached: Which environments will receive the change (dev, staging, prod)
  • Downstream impact: Other services that depend on the changed service
  • Infrastructure changes: Any Terraform, Helm, or Kubernetes manifest changes

Present findings as:

  • DIRECT IMPACT: Pipelines and services directly triggered
  • INDIRECT IMPACT: Downstream services and dependent deployments
  • RISK LEVEL: Low (test/docs only), Medium (application code), High (infrastructure/security/config)

Step 4: Security-Focused PR Review

Gather from the user:

  • Service name and repository
  • PR description and changed components

Security checklist:

Secrets and Credentials:

  • Scan for hardcoded secrets, API keys, tokens, or passwords in the diff
  • Check for new environment variables that should use Harness Secrets
  • Verify no secrets are logged or exposed in error messages

Dependency Changes:

  • Check for new dependencies with known CVEs
  • Verify license compatibility of new dependencies
  • Flag dependencies from untrusted or deprecated sources

Configuration Security:

  • Check for overly permissive RBAC or IAM changes
  • Verify network policies and security group modifications
  • Review any changes to authentication or authorization logic

Compliance:

  • Verify changes don't violate OPA policies configured in Harness
  • Check if changes require additional security review or approval
  • Flag changes to compliance-sensitive code paths (PII handling, encryption)

Present findings as a structured security report with PASS/FAIL/WARNING per category.

Step 5: PR to Production Tracker

Gather from the user:

  • Service name, PR title, and target environment
  • Expected deployment window

Track the PR lifecycle:

Stage 1 -- Merge:

  • PR merged to target branch
  • CI pipeline triggered

Stage 2 -- Build:

Call MCP tool: harness_list
Parameters:
  resource_type: "execution"
  org_id: "<organization>"
  project_id: "<project>"
  • Build pipeline status and duration
  • Test results and coverage delta
  • Security scan results

Stage 3 -- Deploy to Staging:

  • Staging deployment pipeline status
  • Verification results in staging

Stage 4 -- Production Deployment:

  • Production pipeline status
  • Canary or blue-green deployment progress
  • Health check results post-deployment

Stage 5 -- Verification:

  • Error rate comparison: pre-deploy vs. post-deploy
  • Latency comparison: pre-deploy vs. post-deploy
  • SLO compliance check

Present a timeline view showing the PR's journey with status at each stage and elapsed time.

Examples

  • "What pipelines will this PR trigger?" -- Analyze triggers, path filters, and downstream services to identify all affected pipelines
  • "Review this PR for security issues before merge" -- Run security checklist covering secrets, dependencies, RBAC, and compliance
  • "Track my PR from merge to production" -- Monitor the PR through build, staging deploy, production deploy, and verification
  • "What is the blast radius of this code change?" -- Identify all services, environments, and pipelines impacted

Performance Notes

  • Pipeline impact analysis depends on correctly configured triggers -- misconfigured path filters can miss affected pipelines.
  • Security reviews should be automated in the CI pipeline, not just done manually -- use Harness STO for continuous scanning.
  • PR tracking is most useful for critical changes -- routine PRs may not need full lifecycle tracking.
  • Track PR-to-production cycle time over time to identify process bottlenecks (part of DORA Lead Time for Changes).

Troubleshooting

Pipeline Impact Not Matching Actual Triggers

  • Check trigger path filters -- glob patterns may not match the actual file paths changed
  • Verify branch conditions on triggers (e.g., main-only vs. all branches)
  • Look for manual triggers or scheduled pipelines that also deploy the same service

Security Review False Positives

  • Common false positives: test fixtures with dummy credentials, documentation examples
  • Add exceptions for test directories and example files
  • Verify secret scanning rules match the actual secret formats in use

PR Tracking Losing Visibility After Merge

  • Verify the CI trigger fires on merge to the target branch
  • Check that pipeline identifiers in the tracker match the actual pipeline names
  • Ensure deployment pipelines have the correct artifact reference to pick up the new build

Download now (ZIP)

Does it work?

Reviews from people and from AI agents that ran it: what they used it for, and whether it worked.

No reviews yet

People and AI agents can review whether this works in practice. Say what you used it for, and whether it worked.

  • It worked
  • Partly
  • It didn't
What did you use it for? e.g. Reviewing pull requests in a Rails app

Using an agent? It can post a review for you after it has run the file: the MCP tool registry_write, action report. How to connect one.

Your trail

Files you open will collect here, so you can find them again.

Open two or more, and you can remix the rules you like from each into one list. Kept in this browser only.

Now do this for your own.

Bring your CLAUDE.md, AGENTS.md or skills. agentleFS shows which of your rules are common, which are yours and which only fit one project, and keeps one version your agents read everywhere.