sdd-plan-audit
Skill3 starsChanged 35 days agoMITView on GitHub
SDD audit when an architectural-analysis report must become workstreams with TechSpec and tasks; does not audit or implement.
In this repository26 other files it gives its agents.
- AGENTS.mdAGENTS.md
- ai-ready-fixSkill
- ai-ready-scoreSkill
- architectural-analysisSkill
- commitSkill
- deep-reviewSkill
- domain-modelingSkill
- dotnet-efficient-validationSkill
- herdr-orchestrationSkill
- no-workaroundsSkill
- repository-cli-efficiencySkill
- sdd-create-prdSkill
- sdd-create-techspecSkill
- sdd-execute-correctionsSkill
- sdd-execute-taskSkill
- sdd-orchestrate-flowSkill
- sdd-orchestrate-prdsSkill
- sdd-orchestrate-tasksSkill
- sdd-plan-correctionsSkill
- sdd-plan-refactoringSkill
- sdd-plan-tasksSkill
- sdd-review-codeSkill
- sdd-snapshotSkill
- sdd-triageSkill
- update-nuget-packagesSkill
- writing-agents-mdSkill
The file
We split this skill into the rules it gives an agent
5 rules. About 0 are common, 2 are distinctive, and 3 are very specific to this project.
≈0
Common · estimate
Many files like this have a rule of this kind.
≈2
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
- Distinctive: line 17
Output: every finding has an ID, class, and current evidence.
- Very specific: line 27
When writing the map, read assets/destinations.template.md in full.
- Distinctive: line 28
Output: every actionable finding belongs to exactly one workstream.
- Very specific: line 32
After each workstream except the last, run the session pause from session-continuity.md with stage planning and the next workstream as next …
- Very specific: line 38
Output: no finding without a destination in the map.
≈ “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.
Files
Plan SDD audit
An architectural-analysis report becomes workstreams: each workstream is an SDD feature in tasks/prd-[slug]/, executable by sdd-orchestrate-tasks. Contracts and tasks follow the owning skills, which this session runs itself; this skill slices, sequences, and maintains the destination map, which links each finding to the workstream and task that resolves it. Subagents are read-only explorers. The report is immutable and no code is changed.
- Fix the report. Resolve
.audits/architectural-analysis-[timestamp].mdby argument; without one, use the most recent by timestamp and report the choice. An argument that does not resolve while other reports exist: list them and request a choice. With no report at all, stop without writing anything and tell the user to runarchitectural-analysis, which only they invoke. Read the resolved report once. Look for the map.audits/architectural-analysis-[timestamp].destinations.md: if it exists, reuse IDs, workstreams, and states, and change it only with--update. When the map lists workstreams, check their folders for anactivecontext-snapshot.mdand, if one exists, load it through the Load branch of.agents/skills/sdd-snapshot/SKILL.mdand resume from its next step. Locatesdd-plan-refactoring,sdd-create-prd,sdd-create-techspec, andsdd-plan-tasksin.agents/skills/. A missing skill blocks only the route that depends on it. Output: exact report, previous map and snapshot reconciled, and each route's skills located; or a stop with no files written and the missing report reported. - Inventory findings. Walk the five dimensions — dead code, duplication, anti-patterns, type safety, smells — by content, not by headings: real reports rename sections, write findings in prose, and record corrected false positives. Every table row, duplication group, or described hotspot gets
AA-NNin order of appearance, with dimension,file:line, severity or confidence, and recommendation. An aggregate count beyond the listed items becomes a single finding for the rule, with the count as baseline; without named files, it enters a workstream only at the user's request and is otherwise deferred. Summary, statistics, impact, "None found", and corrected false positives are context and get no ID. Check each finding against the smallest excerpt of current code — the report ages. With many findings, split the check by dimension or directory across parallel read-only explorers that return the proposed class and cited evidence; this session assigns the final class. Classify: actionable (confirmed), resolved (no longer exists), discarded (contrary evidence cited), or pending (depends on a decision or environment). ForPossibly dead, LOW confidence, and use through reflection, dependency injection, XAML bindings, or configuration, first run the verification the report names; only what it does not decide stays pending. Output: every finding has an ID, class, and current evidence. Slice workstreams. Group actionable findings by cause, with one reviewable outcome per workstream: confirmed dead code; one duplication or contract group; one god object or layer boundary; defects with a concrete failure path (async misuse, synchronous I/O on the UI thread, captive dependency, swallowed exception); nullability posture per project; local smells per target file. Merge findings the same extraction resolves —
Prioritized ActionsandRisk Areaspoint to that coupling. Order removal before consolidation and consolidation before decomposition; a target file shared between workstreams becomes an explicit dependency. Name them[NN]-arch-[YYYYMMDD]-[workstream]with the report date, numbered after the highest folder undertasks/. Decide each workstream's route — this is where it is decided whether a TechSpec is needed:Workstream Route Preserves observable behavior: removal, consolidation, decomposition, coupling, layers, types, smells sdd-plan-refactoringwritesprd.mdandtechspec.mdFixes a defect and changes observable behavior: a failure becomes logged or propagated, synchronous blocking becomes async sdd-create-prd, thensdd-create-techspectasks/prd-[slug]/of a feature not yet completed already has a PRD and TechSpec valid for the scopeReused: no new contract When writing the map, read assets/destinations.template.md in full; workstreams enter as
proposed. Output: every actionable finding belongs to exactly one workstream; each workstream has a route and dependencies without cycles; map written.- Approve the slicing. Present each workstream with slug,
AA-NN, route, dependencies, and risk, and list resolved, discarded, and pending findings separately. Ask which workstreams to plan — through the available question tool (AskUserQuestion) when the options fit, in text when they do not — and accept regrouping. A workstream not chosen becomesdeferredin the map with its findings. Output: each workstreamapprovedordeferredby the user's answer, recorded in the map;tasks/untouched until then. - Generate contracts. For each approved, non-reused workstream, in dependency order, follow the route's skill by exact path in this session with slug, the workstream's findings (ID,
file:line, current evidence, recommendation), and sibling workstreams as boundaries. KeepAA-NNas the obligation ID in the PRD scope and in the TechSpec'sDEC-NNandQA-NNthat eliminate it: that is how tasks inherit traceability. On the refactoring route, the finding's counts are the quality profile baseline and the target is to eliminate them. Read-only explorers may trace several workstreams in parallel; this session writes one workstream's contracts at a time. After each workstream except the last, run the session pause fromsession-continuity.mdwith stageplanningand the next workstream as next step; a written snapshot goes to that workstream's folder and the previous one is markedsuperseded. Present the written contracts at a single HIL — preserved behaviors, decisions, and pending items for all workstreams — before decomposing; correct whatever is refused through the owning skill. Output: each approved workstream has acceptedprd.mdandtechspec.md, or a block with a concrete cause that affects only it. - Plan tasks. For each workstream with accepted contracts, follow
.agents/skills/sdd-plan-tasks/SKILL.mdfor--prd [slug]in this session. Rules, templates, validation profile, and coverage gate are its own, applied without local adaptation. Run the session pause between workstreams as in step 5. Output: each workstream hastasks.mdandtask_*.mdwith the coverage gate filled in, or a block returned by the skill. - Close the map. Record on each
AA-NNthe workstream and tasks that cover it, read from the traceability matrix intasks.md, or its final destination. Check that every actionable finding of a planned workstream appears in at least one task and that the map's links resolve. Present workstreams, execution order, risks, and pending items; implementation authorization stays with the user. Point the snapshot's next step atsdd-orchestrate-tasksfor the first workstream, or mark itclosed. Output: no finding without a destination in the map; execution indicated assdd-orchestrate-tasks --prd [slug]in map order.
New report on code with a previous map: reconcile findings by file and rule, preserve workstreams in progress, and number new workstreams after the highest existing one. A workstream whose source changes during planning invalidates only its own derivatives. A completed workstream whose folder the commit skill already removed counts as completed; the broken link in the map is not an error.
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
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.

