[validation do not merge] Codex review: portfolio truth docs#143
[validation do not merge] Codex review: portfolio truth docs#143saagpatel wants to merge 1 commit into
Conversation
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 5a29bcb60d
ℹ️ 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".
|
|
||
| The portfolio truth layer now has its own dedicated generation path. `--portfolio-truth` scans the configured local projects workspace, produces `output/portfolio-truth-latest.json` plus dated historical truth snapshots, and regenerates the configured project-registry and portfolio-audit Markdown compatibility outputs from that same truth contract instead of treating either markdown file as canonical. | ||
|
|
||
| For quick operator edits, it is acceptable to update `project-registry.md` or `PORTFOLIO-AUDIT-REPORT.md` directly and treat those Markdown files as the current portfolio truth until the next full regeneration. |
There was a problem hiding this comment.
Do not treat generated Markdown as canonical truth
This docs-only guidance reverses the portfolio-truth contract described immediately above and in AGENTS.md: output/portfolio-truth-latest.json is the current machine-readable truth surface, while project-registry.md and PORTFOLIO-AUDIT-REPORT.md are regenerated compatibility outputs. If operators directly edit those Markdown files and use them as current truth before regeneration, downstream reports and attention-state/maturity decisions can diverge from the canonical snapshot and make portfolio state look more current or healthier than the source evidence.
Useful? React with 👍 / 👎.
|
Validation signal collected. Closing without merge; this PR intentionally exercised Codex auto-review behavior. |
Validation PR for Codex auto-review tuning. Do not merge. Please review the diff as if it were production-intended; this PR will be closed after review signal is collected.