Integrate Godaudits 2.0 canonical state#64
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 37289f5951
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| const canonical = entry.id === 'godplans' ? PLAN_PATH : resolveAuditSource(projectRoot); | ||
| const files = Array.isArray(entry.files) && entry.files.length > 0 | ||
| ? entry.files | ||
| : [resolveArtifact(projectRoot, canonical)]; |
There was a problem hiding this comment.
Rehash the active godaudits source for staleness
When a project imported a legacy .godaudits/AUDIT.mdx (so state.json records that path in source-systems[].files) and later receives canonical .godaudits/AUDIT.json, this code reports the active path as JSON but still hashes the old recorded MDX file because entry.files wins. As a result state.detectDrift and /god-status do not flag the new canonical audit as stale, so /god-migrate is not prompted and the harden seed plus managed GA todos can remain based on the old audit.
Useful? React with 👍 / 👎.
Summary
.godaudits/AUDIT.jsonwith generated MDX and legacy Markdown fallbacksVerification
npm run release:checknode scripts/test-sibling-artifacts.js(49 passed)git diff --check