saf-fix-pr · git:20260826.655710e · 2026-08-26 · sha256 6f42f3c6d61e73a6

saf-fix-pr git:20260826.655710eA

Immutable. This exact content is served forever at /api/v1/blob/6f42f3c6d61e73a6.

---
name: saf-fix-pr
description: Apply the smallest task-scoped fixes for verified SDD pull-request findings. Use only when the user explicitly asks to repair actionable PR findings; not for a general refactor or automatic push.
compatibility: Requires Git and a compatible Agent Skills host.
---

# Fix SDD pull-request findings

## When to use

Use for repairs to verified findings from one task-scoped local change review. In `manual` and
`supervised`, the user explicitly requests the repair. In `autonomous`, the original bounded
delegation is standing authorization for verified local findings only. Read [the TLC baseline](../sdd-agentic-flow-shared/references/tlc-baseline.md), [engineering principles](../sdd-agentic-flow-shared/references/engineering-principles.md), and [safety rules](../sdd-agentic-flow-shared/references/workflow-safety.md).

## When not to use

Do not use for unverified comments, broad cleanup, feature redesign, sibling tasks, or automatic commits and pushes.

## Inputs

- One task reference and a review report, PR findings, or user-supplied evidence.
- Optional `.sdd-agentic-flow/config.yml` overrides, SDD artifacts, current diff, and validation commands.

## Workflow

1. Read `.sdd-agentic-flow/config.yml` when present; otherwise use canonical effective defaults, then resolve one task and its permitted scope.
2. Build a findings ledger, applying `../sdd-agentic-flow-shared/references/evidence-standard.md`. Fix only findings with reproducible evidence; classify preferences, missing evidence, and spec drift without changing them. In autonomous mode, reconcile intent-preserving spec drift through `saf-create-spec` rather than editing the contract here. Do not close findings by reclassifying missing evidence as preference or by silently changing delegated intent.
3. Apply `../sdd-agentic-flow-shared/references/engineering-principles.md`. Apply the smallest patch per actionable finding and add or update focused regression evidence. No opportunistic cleanup.
4. Run configured targeted checks, update the ledger, and hand off to `saf-review-pr` for focused re-review.

## Safety

Preserve unrelated changes. Stop for SDD reconciliation, sibling scope, unsafe environments, or unresolved identity. Do not commit, push, amend, post comments, update PR metadata, mutate trackers, or make network/default mutations unless explicitly authorized.

## Output

Return the findings ledger, changes and checks, unresolved items, re-review scope, and next step. Include `Status`, `Next recommended skill`, and `Reason`. When actionable findings remain unresolved across a session or agent boundary, write or update `handoff.md` per `../sdd-agentic-flow-shared/references/handoff-standard.md`, referencing the findings ledger rather than duplicating it.

### Autonomy

Supports `manual`, `supervised`, and `autonomous` autonomy levels. In autonomous mode, advancing
back to `saf-review-pr` requires `fix-evidence` and every actionable finding resolved or explicitly
deferred with a reason. An unresolved authority boundary or scope violation blocks the transition;
verified local findings do not require a new human confirmation. See
`../sdd-agentic-flow-shared/references/autonomy-guardrails.md`.