docs: add PRD-53 codegen pipeline audit

[DevonMorris]

Comprehensive audit of the clifford-codegen pipeline identifying:

• 31 #[allow(...)] suppressions violating CLAUDE.md principles
• 12+ dead conditional branches with identical if/else
• 4 algebras missing [norm] sections
• Code duplication in traits.rs (sandwich/antisandwich, project/antiproject)
• Legacy stub module (constraint_simplify.rs)
• Unused functions and error variants
• Inconsistent TOML field naming across 15 algebra specs

Includes proposal for formal TOML schema specification with:

• Complete field reference documentation
• JSON Schema for IDE support
• validate/migrate CLI commands
• Effort estimates (~97 hours total)

https://claude.ai/code/session_01TQJ63pYbHmpLhLphtB1xAx

[greptile-apps]

Greptile Overview

Greptile Summary

This PR adds PRD-53, a comprehensive audit document of the clifford-codegen pipeline that systematically identifies and categorizes technical debt across multiple dimensions.

Strengths:

• Thorough and well-structured audit with verifiable claims (spot-checked: traits.rs is indeed 5,558 lines, 4 algebras confirmed missing [norm] sections, 15 TOML files exist)
• Clear categorization by severity with actionable remediation steps
• Practical implementation plan broken into phases with realistic effort estimates
• Proposes documentation-first approach for TOML schema rather than over-engineering with JSON Schema
• Directly addresses CLAUDE.md violation (31 #[allow(...)] suppressions)

Issues identified:

• Minor discrepancy: PR description mentions "~97 hours" but document shows "89 hours" total effort
• Document still mentions JSON Schema as an option (line 276) while PR description indicates it was removed from the proposal

This is planning documentation only—no implementation changes included.

Confidence Score: 5/5

• Safe to merge - adds planning documentation only, no code changes
• Documentation-only PR with thorough, well-researched audit. The two minor issues identified (effort estimate discrepancy and JSON Schema reference) are trivial and don't affect the quality or correctness of the technical analysis.
• No files require special attention

Important Files Changed

Filename	Overview
docs/prd/prd-53-codegen-audit.md	Comprehensive audit document identifying technical debt in codegen pipeline with actionable remediation plan

[greptile-apps]

_{1 file reviewed, 2 comments}

_{Edit Code Review Agent Settings | Greptile}

[greptile-apps]

PR description mentions "~97 hours total" but this line shows "89 hours". Should be consistent.

Prompt To Fix With AI

This is a comment left during a code review.
Path: docs/prd/prd-53-codegen-audit.md
Line: 369:369

Comment:
PR description mentions "~97 hours total" but this line shows "89 hours". Should be consistent.

How can I resolve this? If you propose a fix, please make it concise.

[greptile-apps]

Note mentions "While JSON Schema can be used" - the PR description states "remove JSON Schema" but this doc still references it as an option. Consider clarifying the decision to explicitly exclude JSON Schema approach.

Prompt To Fix With AI

This is a comment left during a code review.
Path: docs/prd/prd-53-codegen-audit.md
Line: 276:276

Comment:
Note mentions "While JSON Schema can be used" - the PR description states "remove JSON Schema" but this doc still references it as an option. Consider clarifying the decision to explicitly exclude JSON Schema approach.

How can I resolve this? If you propose a fix, please make it concise.