feat(aidd-dev): audit codebase compliance against project rules

オープン
#864 コメント 2 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度
5/5
見積もり時間
1週間以上
初心者へのやさしさ
45/100
issue の種類
機能追加
明瞭さ
おおむね明確
活発さ
活発
技術スタック
markdown
領域
devtools, tooling

調査の方向性

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.

索引モデルが issue の本文から書いたものです。

説明

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 violation with the Rule plus affected file:line and supporting evidence, interpretation-dependent, or not verifiable from repository evidence.
  • Produce a global clean result only when all applicable Rules that are verifiable from repository evidence are compliant.
  • Keep this within aidd-dev: it must not depend on aidd-context or call a skill from another plugin. aidd-dev evaluates code conformance; aidd-context owns 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-audit action discovers declared project Rules at runtime and determines applicability before evaluating code, reusing the approach used by 05-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, or not 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 clean result is possible only when every applicable, verifiable Rule is compliant.
  • 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-dev to aidd-context dependency or cross-plugin call, and does not change the existing diff-only 05-review contract.
Prior art in this repo
  • 04-audit/SKILL.md#L3-L33 is the read-only codebase audit, but its seven listed pillars do not explicitly include compliance with project Rules.
  • 05-review/SKILL.md#L3-L27 defines a read-only review of a diff; review-relevancy is its Rule-conformance axis.
  • 03-review-relevancy.md#L3-L24 discovers declared Rules at runtime, evaluates a diff, cites violated Rules, and writes None. when clean.
  • docs/ARCHITECTURE.md#L85-L109 assigns code transformation to aidd-dev and places a missing capability with the plugin whose concern owns it; this makes aidd-dev the 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.
主要言語
TypeScript
スター
481
フォーク
45
平均マージ
14時間 36分
マージ済み PR(30日)
110

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

ai-driven-dev/framework のほかの issue

ai-driven-dev/framework の issue をすべて見る

似ている issue

TypeScript の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。