agentleFS
dougcunhadougcunha/TokenHound/.agents/skills/sdd-execute-corrections/SKILL.md

sdd-execute-corrections

Skill3 starsChanged 35 days agoMITView on GitHub

SDD execution when tasks from a code review need correction; implements each correction task in this session with read-only explorers and moves on between tasks until the context threshold, and delegates the re-review to a fresh-context subagent; does not create or reclassify findings.

The file

We split this skill into the rules it gives an agent

4 rules. About 0 are common, 1 is distinctive, and 3 are very specific to this project.

  • ≈0

    Common · estimate

    Many files like this have a rule of this kind.

  • ≈1

    Distinctive · estimate

    This author's own choices. Worth a look.

  • 3

    Very specific

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

  • Common
  • Distinctive
  • Very specific
  1. Very specific:

    If the caller limits execution to one task, return after step 6 instead of running the session pause: task-completed with the next eligible …

    line 12
  2. Very specific:

    Resume: Read .agents/skills/sdd-orchestrate-tasks/references/session-continuity.md in full once per session: its context measurement …

    line 14
  3. Very specific:

    Update only the task and its single ## Handoff with result, files, commands, validated version, and pending items.

    line 21
  4. Distinctive:

    If the environment prevents acceptance, keep the affected task pending with command, error, and impact.

    line 30

≈ “Common” is an estimate: at least 1 in 5 of the 350,495 skill files we read have a rule of the same kind, not the same words. “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

Execute SDD corrections

The session that runs this skill is the only writer: it implements every correction task itself and owns the moves inside the report folder. Subagents are read-only explorers; they never edit files, run builds or tests that write shared resources, or talk to the user. The exception is the delegated reviewer of step 7, with its own contract. Execution is one task at a time; between tasks the session moves on by itself until the session pause says stop. The session that corrects the code does not issue the re-review: it delegates it.

If the caller limits execution to one task, return after step 6 instead of running the session pause: task-completed with the next eligible IDs, or blocked with evidence. When all tasks are complete, run step 7 before returning. A task return does not end the review cycle.

  1. Resume. Read .agents/skills/sdd-orchestrate-tasks/references/session-continuity.md in full once per session: its context measurement applies to every task. If tasks/prd-[slug]/context-snapshot.md exists, load it through the Load branch of .agents/skills/sdd-snapshot/SKILL.md, validating covers_through against the report folder. Output: snapshot applied, partially trusted with the suspect entries named, or absent.
  2. Fix one report folder by argument, snapshot, or context and require its codereview.md. Inventory root and done/ tasks by ID, findings, acceptance, dependencies, and files. If ambiguous, request a choice; if the plan is missing, direct to sdd-plan-corrections. Output: exact report, with no duplicates or missing or circular dependencies. If all tasks are complete, continue to step 7.
  3. Select one eligible task whose dependencies are complete, preferring the snapshot's next_step when it is still eligible. Confirm correction authorization. Output: one task with its findings, limits, and matching snapshot entries loaded.
  4. Read the stable report and task once per version; then retrieve relevant PRD and TechSpec sections, code, AGENTS.md, and skills. Send a read-only explorer only for sweeps across many files, such as every caller of a symbol a finding names, and verify the lines it cites. Trace the cause and implement within the limits. Apply the TechSpec validation; in desktop C#/.NET, omit E2E even in legacy commands. Record unit, integration, and required manual checks and limitations; when available, use dotnet-efficient-validation. Run the quality profile's blocking commands over the files the correction touched: fixing one finding without introducing another is part of the task's acceptance. Serialize builds and test runs that share bin/, obj/, or TestResults/. Update only the task and its single ## Handoff with result, files, commands, validated version, and pending items. Preserve the report, other tasks, global state, and other features' artifacts, even when the code comes to contradict them. Output: implementation and evidence, or a reproducible block.
  5. Re-read the diff and handoff against the finding, acceptance, and rules as if another author wrote them. Reuse proven tests from the same state; run only missing or invalidated checks. Fix what fails; after two attempts without new evidence, record a block and advance independent work. An architecture or scope divergence requires HIL when not covered by existing authorization. Output: acceptance of the task proven, or a specific pending item; unexecuted essential manual work prevents approval.
  6. Move an approved task to done/, preserving its name and checking absolute paths inside the report folder. Preserve the immutable report. Recalculate the DAG from the remaining files. Then, with no explorer or process running, run the session pause from session-continuity.md with stage corrections, authored_code: yes, and the next eligible task as next step; on the Move on destination, return to step 3. Output: approved task moved, pending tasks at the root; the next task started, or the snapshot written before the question and the user's choice applied. On interruption, check report and handoff before inferring completion from the folder.
  7. Validate the integrated set without repeating already valid commands. Check every actionable finding and its evidence. The re-review runs in a context that did not make these corrections: in the orchestrated flow, return the execution report so the caller delegates sdd-review-code; in standalone use, apply the independence rule of the session pause: delegate the re-review under the protocol in .agents/skills/sdd-review-code/references/delegated-review.md, with this review as the previous one, read the status in the new report, and run the end-of-standalone-use session pause with the next step it points to. Without an eligible reviewer, the pause names sdd-review-code and recommends ending this session. Output: tasks complete with integrated evidence and the re-review delegated and received, left to an independent session, or explicitly returned to the caller; persistent or new findings remain open until a decision.

If the environment prevents acceptance, keep the affected task pending with command, error, and impact. Pre-existing or foreign changes that collide with the task's files stop the task until they are reconciled.

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.