Skip to content

Report Unready Ticket blockers and retain graph titles - #1343

Merged
JacobStephens2 merged 6 commits into
mainfrom
issue-1113
Oct 25, 2026
Merged

JacobStephens2 merged 6 commits into
mainfrom
issue-1113

Conversation

@JacobStephens2

@JacobStephens2 JacobStephens2 commented Oct 25, 2026 •

Copy link
Copy Markdown
Owner

Summary

- #21 unready: labelled ready-for-human
+ #21 unready: labelled ready-for-human; blocked by #99 (outside the Spec; ready-for-agent)

The Tickets checklist and Command log retain the Unready reason and name open blockers. Command records also retain Ticket and blocker titles for show --json; missing historical titles are documented as best effort. Outcomes, exit codes and scheduling stay unchanged.

Closes #1113

Evidence

  • Before: the Ticket-board regression omitted blockers. The reviewer’s live GitHub reproduction and the retained regression rejected the initial expanded query at 1,020,100 possible nodes, exceeding GitHub’s node limit.
    After: the board, graph-adapter and Spec-run CLI regressions pass. The same live reproduction passes after limiting the initial query to 420,100 possible nodes. The CLI regression exercises a ready-for-agent label beyond the first 40 labels, the PR checklist, Command log, retained titles, unchanged failure/draft outcome and independent Ticket dispatch.
  • Final affected checks: /usr/bin/time -p scripts/affected-tests main passed on 2c3d52a0: full Rust suite through the cargo test fallback and all 50 site tests. Wall time: 2,271.83s (37m 52s), including compilation. All applicable CI checks also pass on this exact head.
  • Formatting and cargo clippy --all-targets -- -D warnings pass. cargo test --locked --test pickup_model -- --nocapture passes its four checks, including both safeguard mutations; the known pre-Claim counterexample remains recorded and its literal assertion remains ignored.

Seams tested: TicketBoard’s lines/checklist and next-Ticket interface; GitHub’s Ticket graph reader; Spec-run CLI output and retained show --json observations.

Review coverage: Standards: 0 findings. Spec: 1 finding, SPEC-001, fixed and independently rechecked. Both axes read all 13 changed files, including generated site/docs/index.html. Their review did not reread the final optional-title-map lookup correction in tests/fakes/gh.rs; its existing Activity log integration target passes (12 passed, 1 ignored). No other unread changed files or generated-output exclusions.

Unaddressed findings

  • Standards: None.
  • Spec: None.
  • Security: None.

Security review outcome

No unaddressed introduced findings.

Merge Danger

Door: two-way

No record migration; optional titles remain compatible with existing records.

Blast Radius: Commands

The shared GitHub graph reader is used during Ticket and Spec startup and Pickup. Reporting changes do not widen dispatch to outside blockers.

Built with codex codex-cli 0.164.0 · gpt-6.1-sol · xhigh

@JacobStephens2
JacobStephens2 merged commit 21e8b5b into main Oct 25, 2026
18 checks passed
@JacobStephens2
JacobStephens2 deleted the issue-1113 branch October 25, 2026 18:58
@JacobStephens2 JacobStephens2 mentioned this pull request Oct 25, 2026
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.

A Spec run that stops on a Ticket doesn't name the Ticket's blockers outside the Spec

1 participant