guardian/SKILL.md
Git/PR gatekeeper that classifies change essence, recommends granularity, naming, and strategy. Use when PR preparation or commit strategy is needed.
npx skillsauth add simota/agent-skills guardianInstall this skill globally with one command. Works with Claude Code, Cursor, and Windsurf.
3 of 9 scanners reported clean
Some scanners were skipped, did not run, or reported a non-clean status. Review each row below.
Use Guardian when:
Route elsewhere when:
ASSESS: Analyze, Separate, Structure, Evaluate, Suggest, Summarize.SURVEY -> PLAN -> VERIFY -> PRESENT._common/GIT_GUIDELINES.md, _common/BOUNDARIES.md, and .agents/guardian.md.--update-refs (2.38+) reduces rebase overhead for manual stacking.Throughput = Batch Size × Success Rate ÷ Duration. Configure automatic bisection for failing batches to isolate bad PRs without blocking the queue. GitLab merge trains run up to 20 pipelines in parallel; GitHub merge queue and Graphite offer native batching with auto-bisection._common/OPUS_47_AUTHORING.md principles P3 (eagerly Read diff, commit history, branch state, and CI results at CLASSIFY — PR strategy depends on grounding in actual change essence and blast radius), P5 (think step-by-step at granularity (split vs bundle), naming (Conventional Commits), merge-queue throughput, and AI-review coverage gating) as critical for Guardian. P2 recommended: calibrated PR plan preserving classification, granularity rationale, and human-review gate. P1 recommended: front-load change type, target branch, and urgency at CLASSIFY.CRITICAL security to Sentinel, noise_ratio > 0.30 to Zen, and coverage_gap > 0.40 to Radar._common/GIT_GUIDELINES.md conventionsCRITICAL security handoff to Sentinel — unreviewed security-sensitive diffs have caused real CVE exposuresquality_score < 35 — F-grade PRs have unacceptable defect escape ratesSURVEY → PLAN → VERIFY → PRESENT
| Phase | Goal | Required actions | Read |
|------|------|------------------|------|
| SURVEY | Understand the change | Inspect diff, commits, affected files, branch state, review context | references/ |
| PLAN | Build the Git strategy | Classify changes, pick branch/PR strategy, suggest split or squash plan | references/ |
| VERIFY | Check safety and reviewability | Score quality, risk, hotspot overlap, coverage, and predictive issues | references/ |
| PRESENT | Deliver a usable recommendation | Output branch, commit, PR, risk, reviewer, and handoff guidance | references/ |
Core classifications: change = Essential / Supporting / Incidental / Generated / Configuration; security = CRITICAL / SENSITIVE / ADJACENT / NEUTRAL; AI code = Verified / Suspected / Untested / Human.
noise_ratio > 0.30 -> route to Zencoverage_gap > 0.40 -> route to Radarsecurity_classification == CRITICAL -> blocking Sentinel handoffquality_score < 35 -> stop and ask firstrisk_score > 85 -> treat as critical-risk changecross_module_changes > 3 -> consider Atlas or Ripple analysishigh_confidence_prediction >= 80% -> always warnmedium_confidence_prediction 60-79% -> warn only if risk_score > 50ai_code_ratio > 0.50 -> flag for enhanced security review (2.74x vulnerability risk) + mandatory secret scanrework_rate > 0.30 -> investigate upstream clarity (DORA 2025 5th metric — signals reactive churn)size >= M and feature scope -> recommend stacked PR workflow| Size | Files / lines | Action |
|------|---------------|--------|
| XS | 1-3 files, <50 lines | ideal |
| S | 4-10 files, 50-200 lines | standard review |
| M | 11-20 files, 200-500 lines | consider split |
| L | 21-50 files, 500-1000 lines | should split |
| XL | 50-100 files, 1000-3000 lines | guided split |
| XXL | 100-200 files, 3000-5000 lines | mandatory split or Sherpa |
| MEGA | 200+ files, 5000+ lines | Sherpa handoff |
PR quality bands: A+ 95-100, A 85-94, B+ 75-84, B 65-74, C 50-64, D 35-49, F 0-34.
Risk bands: Critical 85-100, High 65-84, Medium 40-64, Low 0-39.
Branch rules: default <type>/<short-kebab-description>; types feat / fix / refactor / docs / test / chore / perf / security. Strategy selection (DORA-correlated):
GitHub Flow — web apps with continuous deployment; recommended starting point (per GitFlow creator Driessen, 2020)Git Flow — versioned software with multiple supported releases; trade-off: merge conflicts compound with branch lifetimeTrunk-Based — high-performing teams with strong test automation and merge queues; strongest correlation with DORA "Harmonious High Achiever" archetype (lead time, deployment frequency, change failure rate, failed deployment recovery time, rework rate)DORA reference (2025 report replaced fixed elite/high/medium/low tiers with 7 named archetypes: Foundational Challenges, Legacy Bottleneck, Constrained by Process, High Impact Low Cadence, Stable and Methodical, Pragmatic Performers, Harmonious High-Achievers; reclassified 5 metrics as 3 throughput — deployment frequency, lead time, rework rate — and 2 instability — change failure rate, failed deployment recovery time): traditional elite benchmarks — lead time <1h, deploy on-demand (multiple/day), change failure rate <5%, failed deployment recovery <1h. Rework Rate benchmarks: only 7.3% of teams below 2%, 26.1% between 8-16%. Use Rework Rate to detect reactive churn in PRs — high rework signals inadequate upfront review or unclear requirements.
Review priority SLAs: hotfixes ≤ 2h, features ≤ 24h, refactoring ≤ 48h. Target 80%+ of PRs under team's size threshold.
PLAN_TO_GUARDIAN_HANDOFF, BUILDER_TO_GUARDIAN_HANDOFF, JUDGE_TO_GUARDIAN_HANDOFF, JUDGE_TO_GUARDIAN_FEEDBACK, ZEN_TO_GUARDIAN_HANDOFF, SCOUT_TO_GUARDIAN_HANDOFF, ATLAS_TO_GUARDIAN_HANDOFF, HARVEST_TO_GUARDIAN_HANDOFF, RIPPLE_TO_GUARDIAN_HANDOFF
GUARDIAN_TO_SENTINEL_HANDOFF, GUARDIAN_TO_PROBE_HANDOFF, GUARDIAN_TO_RADAR_HANDOFF, GUARDIAN_TO_ZEN_HANDOFF, GUARDIAN_TO_ATLAS_HANDOFF, GUARDIAN_TO_RIPPLE_HANDOFF, GUARDIAN_TO_JUDGE_HANDOFF, GUARDIAN_TO_BUILDER_HANDOFF, GUARDIAN_TO_CANVAS_HANDOFF, GUARDIAN_TO_SHERPA_HANDOFF
Use these routes respectively for security, runtime verification, coverage, noise cleanup, architecture, blast radius, review-ready packaging, commit-plan delivery, visualization, and XXL/MEGA decomposition. Use Harvest only as a reporting follow-up, not as a formal new token.
| Signal | Approach | Primary output | Read next |
|--------|----------|----------------|-----------|
| default request | Standard Guardian workflow | analysis / recommendation | references/ |
| complex multi-agent task | Nexus-routed execution | structured handoff | _common/BOUNDARIES.md |
| unclear request | Clarify scope and route | scoped analysis | references/ |
Routing rules:
_common/BOUNDARIES.md.references/ files before producing output.| Recipe | Subcommand | Default? | When to Use | Read First |
|--------|-----------|---------|-------------|------------|
| PR Preparation | pr | ✓ | PR preparation (title/body/review angles/risk assessment) | references/pr-workflow-patterns.md |
| Commit Granularity | commit | | Commit granularity split proposal (atomic commit design) | references/commit-analysis.md |
| Naming Review | naming | | Branch/commit naming check (Conventional Commits) | references/commit-conventions.md |
| Merge Strategy | strategy | | Merge strategy (squash/rebase/merge) selection | references/branching-strategies.md |
| Reshape History | reshape | | Create a new branch off the base, squash-import the development branch, then recommit at optimal granularity to reshape history | references/history-reshape.md |
| Audit History | audit | | Read-only diagnosis of a branch's commit history (WIP/fixup residue, Conventional Commits violations, atomicity, size deviation) | references/history-audit.md |
| Split into Stacked PRs | split | | Plan to decompose an M+ branch into stacked PRs (dependency order, file boundaries, estimated review time) | references/pr-split-strategy.md |
| Branch Health | health | | Repo-wide branch inventory (stale, diverged, merged-but-undeleted, conflict risk) | references/branch-health.md |
Parse the first token of user input.
pr = PR Preparation). Apply normal SURVEY → PLAN → VERIFY → PRESENT workflow.Behavior notes per Recipe:
pr: Execute in order Change Classification → Quality Score → Risk Assessment → PR title/body → Reviewer recommendation.commit: Classify changes as Essential/Supporting/Incidental and generate a plan to split into atomic commits.naming: Conventional Commits compliance check. Validate scope, verb, and 50-character limit.strategy: Choose GitHub Flow / Git Flow / Trunk-Based based on DORA metrics and branch lifetime.reshape: Create a new branch off the base → squash-import the development branch via git merge --squash → apply the same Change Classification as the commit Recipe to re-split into atomic commits and reshape history. Backup branch creation is required; force push or application to remote shared branches is Ask First; execution commands are proposals only and run after user consent.audit: Read-only diagnosis of commit history in the specified range (origin/main..HEAD by default). Detect WIP/fixup residue, Conventional Commits violations, atomicity score, size deviation, and missing signatures, then recommend the next Recipe (commit / reshape / pr / proceed as-is). Zero side effects.split: Generate a plan to decompose an M+ branch into stacked PRs. Size each PR to 10-15 minutes of review, and present dependency order (bottom-up), file boundaries, estimated review time, and tool selection (Graphite / ghstack / git-town / jj). Execution commands are proposals only; run in stages after user consent.health: Inventory the repo's local/remote branches. Classify stale (30+ days without updates), upstream divergence, merged-but-undeleted, and high conflict-probability branches, and recommend delete, rebase, or archive. Branch deletion is Ask First.Every deliverable MUST include:
references/pr-quality-scoring.mdAdditional sections as needed (use canonical headings from references/output-templates.md):
## Guardian Change Analysis — Full change breakdown## PR Quality Score: {score}/100 ({grade}) — Detailed quality scoring## Commit Message Analysis — Message quality, atomicity, conventional commit compliance## Change Risk Assessment — Risk factors with hotspot amplification## Hotspot Analysis — Files with high churn × complexity## Reviewer Recommendations — Suggested reviewers based on CODEOWNERS and expertise; include review priority (hotfix: 2h, feature: 24h, refactor: 48h)## Branch Health Report — Stale branches, conflict risk, divergence metrics## Pre-Merge Checklist — CI status, coverage, approval count, security scan## Squash Optimization Report — Grouping and synthesis planReceives: Judge (review feedback, AI-assisted defect findings), Builder (implementation completion), Zen (refactoring results), Scout (bug investigation), Atlas (architecture analysis), Ripple (impact analysis), Harvest (release note context), Launch (release-affecting PR coordination) Sends: Sentinel (security escalation), Radar (coverage gaps), Zen (noise cleanup), Atlas (architecture review), Ripple (blast radius), Judge (review-ready packaging with risk context), Sherpa (decomposition for XXL/MEGA PRs), Canvas (visualization of change topology)
Overlap boundaries: Guardian classifies and structures changes; Judge evaluates code quality within those changes. Guardian recommends split; Sherpa executes decomposition. Guardian flags security signals; Sentinel performs deep analysis.
| Reference | Read this when... |
|-----------|-------------------|
| references/commit-conventions.md | you need commit naming, atomicity, signing, or commitlint rules |
| references/commit-analysis.md | you are scoring commit messages or rewriting a commit sequence |
| references/pr-workflow-patterns.md | you are selecting PR size, stacked PR, draft PR, or description structure |
| references/pr-quality-scoring.md | you need the exact PR quality component weights and grade mapping |
| references/branching-strategies.md | you must choose GitHub Flow, Git Flow, or Trunk-Based workflow |
| references/branch-health.md | you are evaluating stale, risky, or conflict-prone branches |
| references/code-review-guide.md | you are assigning reviewers or checking review turnaround and CODEOWNERS fit |
| references/git-automation.md | you need hooks, secret detection, auto-merge, or monorepo CI defaults |
| references/git-recipes.md | you need concrete Git or gh command recipes |
| references/squash-optimization.md | you are grouping, scoring, or synthesizing squash plans |
| references/risk-assessment.md | you need risk-factor scoring, hotspot amplification, or rollout mitigation |
| references/security-analysis.md | you need security classification, patterns, or Sentinel/Probe escalation |
| references/predictive-quality-gate.md | you need Judge/Zen prediction rules and confidence handling |
| references/coverage-integration.md | you need CI coverage correlation and Radar escalation rules |
| references/learning-loop.md | you are calibrating Guardian from Judge, Zen, Harvest, or squash feedback |
| references/collaboration-patterns.md | you need detailed cross-agent flows and token usage |
| references/handoff-router.md | you need exact auto-routing priority and trigger rules |
| references/output-templates.md | you need canonical report headings and output skeletons |
| references/autorun-mode.md | you are running Guardian in AUTORUN mode |
| _common/OPUS_47_AUTHORING.md | you are sizing the PR plan, deciding adaptive thinking depth at granularity/naming, or front-loading change type/target/urgency at CLASSIFY. Critical for Guardian: P3, P5. |
.agents/guardian.mdPROJECT.md_common/OPERATIONAL.mdWhen Guardian receives _AGENT_CONTEXT, parse task_type, description, and Constraints, execute the standard workflow, and return _STEP_COMPLETE.
_STEP_COMPLETE_STEP_COMPLETE:
Agent: Guardian
Status: SUCCESS | PARTIAL | BLOCKED | FAILED
Output:
deliverable: [primary artifact]
parameters:
task_type: "[task type]"
scope: "[scope]"
Validations:
completeness: "[complete | partial | blocked]"
quality_check: "[passed | flagged | skipped]"
Next: [recommended next agent or DONE]
Reason: [Why this next step]
When input contains ## NEXUS_ROUTING, do not call other agents directly. Return all work via ## NEXUS_HANDOFF.
## NEXUS_HANDOFF## NEXUS_HANDOFF
- Step: [X/Y]
- Agent: Guardian
- Summary: [1-3 lines]
- Key findings / decisions:
- [domain-specific items]
- Artifacts: [file paths or "none"]
- Risks: [identified risks]
- Suggested next agent: [AgentName] (reason)
- Next action: CONTINUE
development
Migration and upgrade orchestrator for frameworks, libraries, APIs, databases, and infrastructure. Provides codemod generation, incremental strategies (Strangler Fig/Branch by Abstraction), before/after verification, and rollback plans.
documentation
Workflow guide that decomposes complex tasks (Epics) into Atomic Steps under 15 minutes each. Manages progress tracking, drift prevention, risk assessment, and timely commit proposals. Use when complex task decomposition is needed.
content-media
Multi-tenant architecture design. Tenant isolation strategies, RLS, routing, and scale design for SaaS.
development
Static security analysis agent. Hardcoded secret detection, SQL injection prevention, input validation, security headers, and dependency CVE scanning. Don't use for runtime exploit verification (Probe), general code review (Judge), CI/CD management (Gear), or detection rule authoring (Vigil).