sflow-review
Perform an independent Singularity Flow review, record actionable findings, and register the review decision.
SKILL.mdplugin/skills/sflow-review/SKILL.md
GitHubPortable review bundle and independent review
<!-- sflow-copilot-pause -->Reuse a verified pause-aware entry supplied in this invocation before any mutation or selection change; otherwise first run singularity-flow session current --for-agent --json once; pause precedes Git. If paused: native Copilot, only /sf-pause off, never resume implicitly. Reuse binding/personalization.replyName; fresh operation checks/consent remain required.
Output contract: Present artifact references, hashes, identity warnings, and the exact confirmation before recording any decision. Summarize artifacts with verified Markdown-preview links; never paste document bodies, code or diffs into Copilot. For code, list touched paths and observed added/deleted/created line counts; unavailable stays unavailable. Use existing evidence, not extra calls solely for display. Summaries, links and opened previews are not human review or approval. For every next action, honor returned commandGuidance in order: Copilot, Shell, VS Code (model-free) using modelFreeCommand; if absent, say "not available for this action". Missing Copilot: "no verified equivalent". Never invent routes. Run a check only when a returned field or command asks for it; never decide from a step's name, kind or route that it applies. A refusal saying a check does not apply to this step is not a blocker: skip it and continue. Act on a result's act first: its state, do routes and any ask. A bounded result names agentOutput.read ranges and omittedFields text files; read all in one response with your file tool. If your host saved a result to a file you cannot open, run that command again with --for-agent --json; never read the host's copy with shell tools.
Boundary: reuse entry ready/workId, cwd=repositoryPath; CLI/workItemRoot paths only; never $HOME. No duplicate lookup. Use exact CLI-returned paths. repositoryPath is command cwd, not a prefix for absolute file paths. Never rebase paths or strip .git/worktrees. Issued private/external files are exact exceptions to repository confinement, not wider read/write permission; writes still need authorization. Resolve relative paths only against their declared base. On failure, preserve the path and error; never search or recreate it.
Run singularity-flow review <phase> --format json for the explicit or active phase. Inspect artifacts, checks, decisions, diff/evidence; not retained source review or approval. HTML: singularity-flow review <phase> --format html --out <file>.
- Run
singularity-flow status --for-agent --jsonandsingularity-flow phase show <phase> --for-agent --json. Author only the current in-progress phase with verified effective authoring skill/sf-review. Otherwise perform inspection steps 3–6 and handoff 10; never compose, publish or invent areviewphase. - For that authoring phase only, read Story context and
singularity-flow wm compose --phase <phase> --evidence; use its complete prompt. Never derive--taskfrom Story text. - Read approved inputs, implementation summary, verification evidence, diff and cited source.
- Review correctness, acceptance coverage, maintainability, architecture alignment, security, failures, observability, rollout, rollback, and tests.
- Rank findings by severity with file/line references.
- Do not silently fix findings unless explicitly asked.
- Prepare/author that configured review phase, then
singularity-flow phase prepublish <phase> --for-agent --json --publish-if-ready(includes draft-check). If unready, act on itscommands.nextandrepairLoop.protocol; stay here. Correct every agent authoring finding now only whencorrection.sameTurn; route other producers to their owner. Never delete markers blindly, invent facts/padding, nest models or overwrite producers. Substantive findings need a separate human fix decision: publish them in the review, then offer the approver's post-submission/sf-reject --to <OWNING-STEP>, orsingularity-flow upstream-issue raise --phase <OWNING-STEP> --about <ID> --problem "<TEXT>"for/sf-decide. Follow returnedrepairLoop.protocol; stop unchanged. - Use returned
commands.publishwhenreadyunlesspublishIfReady.published; absent: relaycommands.next, stop. Race-timeARTIFACT_AUTHORING_INCOMPLETE: recheck once, retry once if ready, never loop. Never submit or approve. - If published, from
publishIfReady.reviewsummarize findings/status and give verified Markdown-preview links with generation/hashes and currentreviewBinding. Read exact source/artifact bytes for the independent review, but never reproduce bodies/code/diffs in chat. Summaries/links are not human review; document reuse never reuses approval consent. - After authoring, refresh
singularity-flow review <phase> --format json. Relaycontinuation.actions: phase and all three surfaces per the Output contract. Correction, review, human disposition and recovery precede Submit. If absent,singularity-flow nextsteps <WORK-ID> --for-agent --json. Never submit or approve automatically.