-
Notifications
You must be signed in to change notification settings - Fork 47
Description
Documentation Audit Report β 2026-03-05
Audited against codebase state as of the current main branch (post-merge of PR #45 on 2026-03-05). 8 documentation files reviewed.
Previous audit: #50 (2026-02-24). Most findings below are recurring from that issue. New items are marked π.
Resolved Since Last Audit
-
docs/REVERSE_ENGINEERING_ARCHITECTURE.md"Last Updated: December 2025" β date was updated to2026-02-23in a commit on 2026-02-23. (Now stale again after 2026-03-05 PR β see Critical section.)
Critical (documentation is wrong or references removed code)
-
π
CHANGELOG.md:[2.5.0]entry (line 8) is dated2026-02-23, but PR Pass reverse-engineered business logic into migration/conversion pipelineΒ #45 β which introduced all v2.5.0 features β was merged on2026-03-05. Per [Keep a Changelog]((keepachangelog.com/redacted), the release date should reflect when the version was released/merged. Fix: Change## [2.5.0] - 2026-02-23β## [2.5.0] - 2026-03-05. -
π
docs/REVERSE_ENGINEERING_ARCHITECTURE.md: After PR Pass reverse-engineered business logic into migration/conversion pipelineΒ #45 (merged 2026-03-05) the file was modified (+45/-7 lines covering business logic persistence,--reuse-re, and dependency mapping), but the header still reads> **Last Updated:** 2026-02-23and> **Version:** 0.2 (Smart Chunking). The version label is misleading given that v2.5.0 features are now documented here. Fix: UpdateLast Updatedto2026-03-05and version to e.g.0.3 (Business Logic Persistence & Dependency Mapping). -
π
.devcontainer/README.mdβ Configure AI section lists three wrong environment variable names:AZURE_OPENAI_ENDPOINTβ correct name is_MAIN_ENDPOINTAZURE_OPENAI_DEPLOYMENT_NAMEβ correct names are_CODE_MODEL/_CHAT_MODELAZURE_OPENAI_API_KEYβ correct name is_MAIN_API_KEY
These names do not match
Config/ai-config.local.envor the configuration loading documented inREADME.md. A developer following the devcontainer README will set the wrong variables and get a silent failure. -
README.md(lines ~164 and ~990): Links toazlogin-auth-guide.mdwhich does not exist anywhere in the repository. Fix: Either createdocs/azlogin-auth-guide.mdor remove/update the links. (Recurring from #50) -
README.md"Further Reading" section: Two links use paths without thedocs/prefix and will 404:[Smart Chunking Guide](Smart-chuncking-how%20it-works.md)β should bedocs/Smart-chuncking-how%20it-works.md[Architecture Documentation](REVERSE_ENGINEERING_ARCHITECTURE.md)β should bedocs/REVERSE_ENGINEERING_ARCHITECTURE.md
(Recurring from #50)
-
.devcontainer/README.md: References/workspace/WINDOWS_COMPATIBILITY.md(in the "Additional Resources" section and the Windows Compatibility section) but this file does not exist in the repository. (Recurring from #50) -
.devcontainer/README.md: References .NET 9.0 SDK throughout (What's Included, Build Errors troubleshooting, verification checklist:dotnet --version # Should be 9.0.x), but bothCobolToQuarkusMigration.csprojandMcpChatWeb/McpChatWeb.csprojtargetnet10.0. (Recurring from #50) -
docs/spec-approach-concept.md: DocumentsConversionPlannerAgent,IConversionPlannerAgent, and a Constitution loader β none of which exist in the codebase. The Spec-Driven Migration workflow was explicitly removed in v2.3.0. This file should be deleted or clearly marked as a rejected proposal. (Recurring from #50)
Missing (undocumented user-facing features)
- Speed Profile system (TURBO / FAST / BALANCED / THOROUGH, added in v2.4.0): Documented inline in
README.mdbut there is no dedicated deep-dive indocs/covering env var overrides (CODEX_STAGGER_DELAY_MS,CODEX_MAX_PARALLEL_CONVERSION,CODEX_RATE_LIMIT_SAFETY_FACTOR), theselect_speed_profile()function indoctor.sh, or the three-tier complexity scoring system that BALANCED mode relies on. (Recurring from #50)
Stale (documentation references that may be outdated)
-
docs/smart-chunking-architecture.md(**Last updated**: 2025-02-17): Predates v2.4.0 (2026-02-16) by nearly a year. TheMaxParallelConversionsetting, the new env var overrides, and the Adaptive Re-Chunking feature added in v2.4.0 are not reflected. (Recurring from #50)
Structural (guideline compliance issues)
-
docs/Smart-chuncking-how it-works.md: Filename contains a space and a typo ("chuncking" instead of "chunking"). Guidelines require a speaking filename without special characters. Suggestion: rename todocs/smart-chunking-deep-dive.md. (Recurring from #50) -
docs/Smart-chuncking-how it-works.md: Missing required**Last updated**: YYYY-MM-DDheader at the top of the file. (Recurring from #50) -
docs/legacy-modernization-flow.md: Missing required**Last updated**: YYYY-MM-DDheader. File also has no title or description β it starts directly with a Mermaid diagram. (Recurring from #50) -
docs/spec-approach-concept.md: Missing required**Last updated**: YYYY-MM-DDheader. (Recurring from #50) -
docs/REVERSE_ENGINEERING_ARCHITECTURE.md: Date format> **Last Updated:** 2026-02-23does not follow the required**Last updated**: YYYY-MM-DDconvention (wrong capitalisation, colon position, and surrounding>blockquote). (Recurring from #50)
Summary
- Total documentation files audited: 8 (
README.md,.devcontainer/README.md,CHANGELOG.md,docs/REVERSE_ENGINEERING_ARCHITECTURE.md,docs/Smart-chuncking-how it-works.md,docs/smart-chunking-architecture.md,docs/legacy-modernization-flow.md,docs/spec-approach-concept.md) - Issues found: 8 critical, 1 missing, 1 stale, 5 structural
- Previously reported issues resolved: 1 (
docs/REVERSE_ENGINEERING_ARCHITECTURE.mdDecember 2025 date β 2026-02-23) - New issues since last audit: 3 (CHANGELOG [2.5.0] date;
REVERSE_ENGINEERING_ARCHITECTURE.mdstale after PR Pass reverse-engineered business logic into migration/conversion pipelineΒ #45;.devcontainer/README.mdwrong env var names)
Generated by Documentation Full Audit
Generated by Documentation Full Audit