[codex] Reconcile portfolio working set contracts#135
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 5108f364ce
ℹ️ 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".
| agent-guestbook-inbox: | ||
| owner: d | ||
| lifecycle_state: experimental | ||
| review_cadence: monthly | ||
| intended_disposition: experiment | ||
| maturity_program: experiment | ||
| category: infrastructure |
There was a problem hiding this comment.
Set experiment contracts to a foundation target
These new experiment entries inherit target_maturity: operating from the catalog defaults because no target is set. resolve_target_maturity() prefers the catalog/default target before the experiment program's own foundation target, so every repo in this added experiment block will be graded against the stricter Operating bar instead of the intended lightweight experiment bar. Add an explicit target_maturity: foundation (or another deliberate target) to the entries in this block.
Useful? React with 👍 / 👎.
| sae-synth-spike: | ||
| owner: d | ||
| lifecycle_state: archived | ||
| review_cadence: quarterly | ||
| intended_disposition: archive |
There was a problem hiding this comment.
Set archive contracts to the archive program
These archive entries inherit the default maturity_program: maintain because the block only sets intended_disposition: archive. Since resolve_program_key() chooses maturity_program before intended_disposition, these newly archived repos will be scored under Maintain and build_operating_path_entry() records a program/disposition conflict instead of a clean archive posture. Add maturity_program: archive (and an explicit archive target such as foundation) to the entries in this block.
Useful? React with 👍 / 👎.
What
Adds explicit portfolio catalog contracts for the current working set and refreshes the local Notion project map entry for cost-tracker.
Why
The canonical portfolio truth snapshot was treating many settled repos as
decision-neededbecause they lacked explicit catalog contracts. That made the operator queue noisy and weakened the usefulness of default attention.Review Of What Was Built
peer-agent-toolsas active infrastructure with a maintain path.Cleanup Review
Kept this to catalog/map source changes. Generated compatibility outputs were refreshed locally during verification but are not part of this PR.
Verification Summary
uv run operator-os-seam-linter --truth output/portfolio-truth-latest.json --jsonpassed.uv run ruff check .passed.uv run pytest -q tests/test_portfolio_catalog.py tests/test_portfolio_pathing.py tests/test_portfolio_truth.py tests/test_weekly_command_center.pypassed: 80 passed.uv run pytest -qpassed: 2,657 passed, 2 skipped.git diff --checkpassed.Shipped Summary
With the companion context PRs for
agent-session-replayandpeer-agent-toolsapplied locally, the two approved remaining decision items clear from the portfolio truth queue.Next Phase
Merge the two companion context PRs first or alongside this PR, then rerun
audit report saagpatel --portfolio-truth --portfolio-truth-allow-empty-notionand the seam linter to refresh canonical local truth.Remaining Roadmap