Skip to content

docs(tick): 2349Z Otto-CLI background worker — PR #3956 thread sweep → merged#3980

Merged
AceHack merged 3 commits into
mainfrom
shard/tick-2349z-otto-cli-2026-05-16
May 17, 2026
Merged

docs(tick): 2349Z Otto-CLI background worker — PR #3956 thread sweep → merged#3980
AceHack merged 3 commits into
mainfrom
shard/tick-2349z-otto-cli-2026-05-16

Conversation

@AceHack
Copy link
Copy Markdown
Member

@AceHack AceHack commented May 16, 2026

Summary

Thread disposition

# Reviewer Verdict Action
1, 3 codex P1 + copilot Already fixed in bdc56f232 (null-state skip in runOneCycle) Reply + resolve
2 codex P2 Real per-window-vs-process limitation; full fix needs AX traversal (contradicts cheap-probe design) Substrate-honest scope-defer reply + resolve
4 copilot False positive. `indexOf(" ")returns first
5 copilot Already fixed in bdc56f232 (FRONTMOST_HISTORY_MAX=256 + LRU) Reply + resolve
6 copilot Already fixed in bdc56f232 (docstring rewrite) Reply + resolve

Substrate-honest framing

No code change was pushed — all four substantive concerns were already in code at the PR head. Threads were stale acknowledgment work, not action work. The blocker was required_conversation_resolution holding open over threads pointing at already-shipped fixes (per the catalog in blocked-green-ci-investigate-threads.md: "stale ≠ false; just means the action window closed").

Budget state

  • Session start: 136/5000 GraphQL (pure-git tier)
  • Post-work: 46/5000 (12 mutations + 1 status check)
  • Reset: 23:58:24Z (~9 min ETA at commit time)
  • --all-open batch poll deferred to post-reset cron tick

Test plan

🤖 Generated with Claude Code

…→ merged

PR #3956 (B-0402 shadow freshness fix #3) cleared BLOCKED → CLEAN →
MERGED via 6-thread substrate-honest sweep. Four threads (1, 3, 5, 6)
were stale acknowledgment work — fixes already shipped in commit
bdc56f2. Thread #4 verified as false positive against source. Thread
#2 acknowledged as real per-window-vs-process refinement gap, deferred
to B-0402 follow-up scope.

Verified each finding against source at PR head before mutating
threads per blocked-green-ci-investigate-threads.md suspect-by-default
discipline.

GraphQL budget on session start: 136/5000 → pure-git tier. Post-work:
46/5000. Reset 23:58Z. --all-open batch poll deferred to post-reset
cron tick.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings May 16, 2026 23:53
@AceHack AceHack enabled auto-merge (squash) May 16, 2026 23:53
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a new hygiene-history tick shard documenting the 2026-05-16T23:49Z Otto-CLI session that resolved stale PR #3956 review threads (with no code changes) so auto-merge could complete, including budget state and substrate links.

Changes:

  • Adds a new tick file capturing the PR #3956 thread sweep and merge outcome.
  • Records GraphQL budget usage / deferrals and the rationale for skipping --all-open.
  • Links the work to relevant .claude/rules/* substrate documents.
Comments suppressed due to low confidence (1)

docs/hygiene-history/ticks/2026/05/16/2349Z.md:81

  • These .claude/rules/* links also use ../../../../../ from this directory, which resolves under docs/ rather than repo-root. Update them to ../../../../../../.claude/rules/... (consistent with other tick files in this folder).
- [`.claude/rules/holding-without-named-dependency-is-standing-by-failure.md`](../../../../../.claude/rules/holding-without-named-dependency-is-standing-by-failure.md)
  — rate-limit reset is a named bounded dependency; counter does
  not tick during pure-git deferral
- [`.claude/rules/zeta-expected-branch.md`](../../../../../.claude/rules/zeta-expected-branch.md)
  — `ZETA_EXPECTED_BRANCH` set + `git branch --show-current`
  guard applied before commit

Comment thread docs/hygiene-history/ticks/2026/05/16/2349Z.md Outdated
Comment thread docs/hygiene-history/ticks/2026/05/16/2349Z.md
AceHack and others added 2 commits May 16, 2026 20:26
Bare-pipe code spans (`"|"` and `|`) confused the markdownlint table
parser into counting 7 cells where 5 were expected (MD056), and the
broken code-span boundaries triggered MD038 false-positives. Reworded
row 4 to describe pipe handling in prose without code-span pipes.

Verified locally: markdownlint-cli2 on the file exits 0.

Co-Authored-By: Claude <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings May 17, 2026 04:10
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@AceHack AceHack merged commit d6bb8a4 into main May 17, 2026
28 of 30 checks passed
@AceHack AceHack deleted the shard/tick-2349z-otto-cli-2026-05-16 branch May 17, 2026 04:14
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.

2 participants