feat(aidd-dev): audit codebase compliance against project rules
Nobody has claimed this yet.
Assessment
- Difficulty
- 5/5
- Estimated time
- Over a week
- Newbie friendliness
- 45/100
Research direction
Start with plugins/aidd-dev/skills/04-audit/SKILL.md and its existing actions, then read 05-review/SKILL.md and actions/03-review-relevancy.md to trace Rule discovery and report conventions. Define the optional audit flow within the existing report and findings table, covering repository-wide code and the four required outcomes. Done means the acceptance criteria hold without adding a skill or pillar or changing 05-review.
Written by the indexing model from the issue text.
Description
Problem
aidd-dev:05-review discovers declared project Rules at runtime and checks conformance, but only for a diff. aidd-dev:04-audit assesses the existing codebase read-only, but its current quality pillars do not explicitly cover compliance with project Rules.
There is therefore no explicit repository-wide workflow that answers: does the existing codebase comply with all applicable project Rules? This leaves legacy code, Rules introduced after code was written, code created outside AIDD, and code never included in a reviewed diff without that assessment.
Scope
- Implement this as an optional rule-conformance action within
aidd-dev:04-audit, reusing the existing audit report and findings table. - Discover applicable project Rules at runtime and determine their scope before applying them, using the existing Rule-discovery approach from
05-review. - Evaluate existing repository code, not only a diff, against those Rules.
- Produce one outcome for each applicable Rule:
compliant,confirmed violationwith the Rule plus affectedfile:lineand supporting evidence,interpretation-dependent, ornot verifiable from repository evidence. - Produce a global
cleanresult only when all applicable Rules that are verifiable from repository evidence arecompliant. - Keep this within
aidd-dev: it must not depend onaidd-contextor call a skill from another plugin.aidd-devevaluates code conformance;aidd-contextowns Rule content and context health (placement, scope, duplication, conflicts, and loading). Both inspect project artifacts directly, and neither plugin calls the other. - Do not introduce a new skill or audit pillar, and do not change the existing diff-only contract of
05-review.
Acceptance criteria
- The optional
04-auditaction discovers declared project Rules at runtime and determines applicability before evaluating code, reusing the approach used by05-review. - It evaluates existing repository code, including code outside the current diff.
- It reuses the existing audit report and findings table.
- It produces exactly one outcome for each applicable Rule, using one of:
compliant,confirmed violation,interpretation-dependent, ornot verifiable from repository evidence. - Each confirmed violation identifies the Rule, affected
file:line, and repository evidence. - Interpretation-dependent findings are distinct from confirmed violations.
- A Rule that cannot reasonably be verified from repository evidence is reported as
not verifiable from repository evidence, not asserted as a violation. - A global
cleanresult is possible only when every applicable, verifiable Rule iscompliant. - The default workflow is read-only and does not modify Rules or application code.
- The implementation adds neither a new skill nor a new audit pillar, creates no
aidd-devtoaidd-contextdependency or cross-plugin call, and does not change the existing diff-only05-reviewcontract.
Prior art in this repo
04-audit/SKILL.md#L3-L33is the read-only codebase audit, but its seven listed pillars do not explicitly include compliance with project Rules.05-review/SKILL.md#L3-L27defines a read-only review of a diff;review-relevancyis its Rule-conformance axis.03-review-relevancy.md#L3-L24discovers declared Rules at runtime, evaluates a diff, cites violated Rules, and writesNone.when clean.docs/ARCHITECTURE.md#L85-L109assigns code transformation toaidd-devand places a missing capability with the plugin whose concern owns it; this makesaidd-devthe appropriate scope without requiring a new skill or audit pillar.- #791 is candidate Rule to existing Rules during Rule generation; #792 is Rules to Rules; #793 is context artifacts to ownership and placement; #796 is Rule to development decision to artifact. None evaluates existing Rules against the existing codebase.
I checked the current main contracts and searched open and closed issues and pull requests for codebase Rule compliance, Rule conformance, and equivalent wording. No duplicate was found.
Out of scope
- Automatically fixing violations or modifying Rules or application code.
- Auditing Rule quality, conflicts, duplication, or drift; that is #792.
- Auditing context-artifact ownership or placement; that is #793.
- Tracing a Rule's influence on development decisions; that is #796.
- Defining universal Rule precedence across AI hosts.
- Treating every Rule as statically verifiable.
- Dominant language
- TypeScript
- Stars
- 481
- Forks
- 45
- Avg merge
- 17h 40m
- Merged PRs (30d)
- 106
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from ai-driven-dev/framework
-
refactor(aidd-orchestrator): the check zone says when to stop, and reviews its axes in one round Open
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
ai-driven-dev/framework#887 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
ai-driven-dev/framework#873 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 85/100
ai-driven-dev/framework#872 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
ai-driven-dev/framework#625 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
ai-driven-dev/framework#467 · 1 comment ·
All issues in ai-driven-dev/framework
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
Eynzof/Hermes-CN-Desktop#610 ·
-
bug clawsweeper:linked-pr-open clawsweeper:needs-live-repro clawsweeper:no-new-fix-pr impact:message-loss issue-rating: 🐚 platinum hermit P2 regression
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
-
enhancement
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
-
calcite-components needs triage refactor
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
Esri/calcite-design-system#15203 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 90/100
danielmiessler/LifeOS#2218 ·