Repository navigation
fix(labs): pin claude-remote to claude-haiku-5-5 - #1786
mctl-claude-remote[bot] wants to merge 1 commit into
Conversation
Operator requested pinning Haiku 5.5 for the interactive claude-remote session instead of the current sonnet pin. This skips the GO/NO-GO validation proposed in mctl-claude-remote#79 (Sonnet-vs-Haiku-5.5 audit-stream comparison for ack_event correctness) — accepted explicitly despite the known mctl-claude-remote#66 regression, where a prior Haiku pin never called ack_event and silently broke the events Channel. Uses the explicit model id rather than a bare alias so a future Haiku release can't re-open #66 unreviewed. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01DQGNcdu462hMZJSk6httXA
Antigravity (agy) review — informationalModel: gemini-3.8-flash-medium · commit No significant issues found. |
|
Hold — do not merge yet. mctl-claude-remote#80 (opened after this PR) reveals that the currently-deployed Correct order per #80:
Merging this PR before #80's image is live would set an env var value the running binary doesn't recognize — a worse failure mode than the #66 regression this PR already discloses. Leaving this open until #80 and the image bump land. 🤖 Generated with Claude Code |
|
Claude finished @mctl-claude-remote[bot]'s task in 3m 16s —— View job Review complete ✅
Verdict: APPROVED — review The actual diff introduced by this PR's single commit (
No template/logic, RBAC, resource, or secret-handling changes — this falls squarely in the "env var" trivial-change category, and the risk + rollback plan are already thoroughly documented by the PR author itself. Approved per the CLAUDE.md "trivial changes merge immediately" convention. Note: I diffed against the commit's actual parent rather than the |
Summary
claude-remotesession (labs) toclaude-haiku-5-5instead ofsonnet, per explicit operator request.ack_eventcorrectness) — the operator accepted this explicitly, aware of the known mctl-claude-remote#66 regression.Background / risk
mctl-claude-remote#66: a prior Haiku pin (0.12.1, rolled 2026-09-19) never called
mcp__mctl-events__ack_eventfor inbound events on this exact deployment — it fabricated a fake acknowledgement viaBash//tmpinstead. Nothing was actually acknowledged,max_inflightsaturated, and the events Channel adapter stopped reading.pr-stewardis unaffected (it passes--modelexplicitly, default sonnet) — this only affects the live interactive session, which is also what processes inboundmctl-eventsnotifications.mctl-claude-remote#79 ("feat-models-validate-and-move") already proposes a mechanical GO/NO-GO gate for exactly this kind of model change (baseline comparison + audit-stream proof that
ack_eventis really being called, not faked). That validation has not been run for this change — the operator chose to pin directly instead, explicitly accepting the risk.Used the explicit model id (
claude-haiku-5-5) rather than a barehaikualias, per #79's rationale: an alias floats to future Haiku releases and could silently re-open #66 without review.Rollback
One-line revert: set
CLAUDE_REMOTE_MODELback to"sonnet"ifack_eventstops being called after rollout (symptom:max_inflightsaturates, events stop being read/acknowledged).Test plan
helm template/kubeconformvalidation (CI)ackedvsdeliveredin the mctl-events audit stream on the next live session for signs of the chore(mctl-agents): bump 1.4.0 → 1.7.0 (3 features in one cycle) #66 regression repeating"sonnet"🤖 Generated with Claude Code