Skip to content

Land six ready PRs in one batch - #2160

Merged
trissim merged 74 commits into
mainfrom
batch/cl-49
Oct 20, 2026
Merged

trissim merged 74 commits into
mainfrom
batch/cl-49

Conversation

@trissim

@trissim trissim commented Oct 20, 2026

Copy link
Copy Markdown
Contributor

One batch: #2124, #2127, #2025, #2095, #2115, #2118 merged on current main, plus the fix commits the merge needed (14772e3, d711e5a, b562ad9).

#1937 and #1940 are not in this batch; each is being ported onto current main in its own PR.

Gates: compile, imports, src and tools debt ratchets against the merge base (exit 0).

Closes #2124, closes #2127, closes #2025, closes #2095, closes #2115, closes #2118.

🤖 Generated with Claude Code

trissim and others added 30 commits October 19, 2026 18:52
#2011 made every transcript event carry its time but left 45,458 saved Claude tool events undated, so every rotate crashed decoding them. The carry takes each event's time from its Claude Code transcript (all 45,458 found, 6 s) and refuses if any stays undated; the unjudged query loses its undated branch.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…in one update

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
trissim and others added 27 commits October 20, 2026 00:06
# Conflicts:
#	src/agent_comms/todos.py
# Conflicts:
#	src/agent_comms/commander/send_guard.py
#	src/agent_comms/external_transcript.py
#	src/agent_comms/recorded_time.py
#	tests/test_claude_turn.py
#	tests/test_codex_items.py
The #2025 resolution stamped every completed event with the item start
time, so a tool end recorded at 2.0 read as 1.0. Completions already carry
their own decoded times; stamp had no other caller.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
#2025 tested send_guard.simple_commands, a second shell-word parse that P294 (#2149)
deleted from main. ShellInvocation.parse owns the parse and refuses malformed words,
which the bash guard denies; the test now asserts that refusal.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
# Conflicts:
#	src/agent_comms/todos.py
#	tests/test_runtime_publication.py
#	tools/publish_runtime.py
# Conflicts:
#	src/agent_comms/commander/order.py
#	src/agent_comms/commander_parity/lifecycle.py
#	src/agent_comms/commander_parity/rows.py
#	src/agent_comms/todos.py
# Conflicts:
#	src/agent_comms/todos.py
…them

#2118's invoke_command declared cwd and env with None defaults, which restates
BoundedRun.run's own options and their absent defaults (debt ratchet: NoneAbsence at
lifecycle.py:205-206). BoundedRun.run owns them; invoke_command now forwards them.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings October 20, 2026 07:04

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing.

@trissim
trissim merged commit fd7d2dc into main Oct 20, 2026
0 of 3 checks passed
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