Skip to content

[codex] Report blocked close-item closeouts#1217

Draft
rickardvh wants to merge 1 commit into
masterfrom
codex/1216-close-item-blocked-status
Draft

[codex] Report blocked close-item closeouts#1217
rickardvh wants to merge 1 commit into
masterfrom
codex/1216-close-item-blocked-status

Conversation

@rickardvh
Copy link
Copy Markdown
Owner

Summary

  • Add close-item JSON status fields for blocked, closed, dry-run, and no-change outcomes.
  • Surface closed, mutation_applied, next_action, and blocked_reason when archive-flow close-item is blocked.
  • Cover blocked archive closeout and CLI JSON output in Planning close-item tests.

Root cause

close-item routed execplan ids through archive_execplan, but blocked archive attempts only returned warnings/actions. Automation could miss that the requested close mutation did not happen.

Validation

  • uv run pytest packages/planning/tests/test_close_item.py -q
  • make test-planning
  • make lint-planning
  • make typecheck-planning
  • uv run python scripts/check/check_planning_surfaces.py

Closes #1216

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.

close-item should clearly report blocked archive closeout

1 participant