Skip to content

fix: preserve control center context on refresh#155

Merged
saagpatel merged 1 commit into
mainfrom
codex/control-center-context-acks
Jul 4, 2026
Merged

fix: preserve control center context on refresh#155
saagpatel merged 1 commit into
mainfrom
codex/control-center-context-acks

Conversation

@saagpatel

Copy link
Copy Markdown
Owner

Summary

  • preserve enriched audit-row portfolio context when standalone control-center refreshes rebuild snapshot queues
  • keep catalog/scorecard/path context aligned between full audit reports and control-center artifacts
  • make acknowledgment capture skip already-acknowledged matching changes so repeated captures advance to the next open item

Verification

  • /opt/homebrew/bin/python3.13 -m pytest tests/test_cli_hardening.py tests/test_operator_acknowledgments.py tests/test_weekly_command_center.py -q
  • /opt/homebrew/bin/python3.13 -m src.cli security-gate
  • /opt/homebrew/bin/python3.13 -m src.cli saagpatel --control-center
  • jq confirmed AIWorkFlow has 0 rows in output/operator-control-center-saagpatel-2026-07-04.json operator_queue after acknowledgments

@saagpatel saagpatel merged commit 5d39e4c into main Jul 4, 2026
4 checks passed
@saagpatel saagpatel deleted the codex/control-center-context-acks branch July 4, 2026 14:04

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 4247a87599

ℹ️ 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".

Comment thread src/cli.py
if catalog_entry:
item["portfolio_catalog"] = dict(catalog_entry)
item["catalog_line"] = catalog_entry.get("catalog_line", "")
item["intent_alignment"] = catalog_entry.get("intent_alignment", "missing-contract")

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Recompute intent alignment for refreshed queues

When standalone --control-center rebuilds a snapshot whose queue differs from the stored report queue (for example a lane-filtered view or local acknowledgment/approval state), build_operator_snapshot has already attached intent alignment from the current queue item, but this line overwrites it with the audit row's older intent_alignment. A maintain/finish repo that is now urgent or blocked can therefore keep an aligned value, and _apply_operating_paths below will treat the path as healthier than the refreshed operator evidence supports.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant