Skip to content

fix: Default atag poller to Opus low - #39

Merged
yolo-sam merged 3 commits into
mainfrom
codex/atag-opus-default-clean
May 28, 2026
Merged

yolo-sam merged 3 commits into
mainfrom
codex/atag-opus-default-clean

Conversation

@yolo-sam

Copy link
Copy Markdown
Collaborator

Summary

  • Set the atag-poll.sh default Claude invocation to Opus low effort while preserving caller overrides after --.
  • Add an interim ADR documenting the run-3b/3c/3d/3e evidence, costs, timings, and why this is temporary.
  • Add handoff notes and link follow-up issue atag: run full model and effort eval for poller defaults #37 for the broader model/effort evaluation.
  • Sync canonical atag changes into Claude/Codex plugin copies.

Decision

Interim default: claude -p --model opus --permission-mode acceptEdits --effort low.

This is explicitly temporary pending issue #37. The scanner still avoids no-op token burn; when the poller wakes Claude, the run-3 data says correctness favors Opus low.

Verification

  • bash -n on canonical/plugin atag-poll.sh copies
  • diff -qr -x dev canonical vs plugin copies
  • stale-default grep for Sonnet defaults
  • bun test skills/atag/reference/markdown-agent-tags.spec.test.ts skills/atag/reference/atag-poll.test.ts claude-plugins/atag/skills/atag/reference/markdown-agent-tags.spec.test.ts claude-plugins/atag/skills/atag/reference/atag-poll.test.ts codex-plugins/atag/skills/atag/reference/markdown-agent-tags.spec.test.ts codex-plugins/atag/skills/atag/reference/atag-poll.test.ts — 246 pass, 0 fail
  • Stubbed default invocation printed --model opus --permission-mode acceptEdits --effort low

Follow-up eval: #37.

Supersedes closed PR #38, which used a stacked branch and showed conflicts.

@yolo-sam

Copy link
Copy Markdown
Collaborator Author

Follow-up review note:

  • Independent Haiku review found one launch blocker in the first PR cut: run_with_timeout() had lost exec, so timeout/interrupt handling could kill only the wrapper subshell and potentially leave Claude orphaned.
  • Fixed in 0d752df by restoring exec "$@" in canonical + plugin poller scripts.
  • Added a timeout regression test in all synced copies.

Verification after the fix:

  • bash -n for canonical, plugin, and active installed poller scripts
  • diff -qr -x dev canonical vs Claude plugin, Codex plugin, and active installed copies
  • bun test focused atag canonical/plugin suites: 249 pass, 0 fail
  • git diff --check

@yolo-sam
yolo-sam merged commit 89a4c11 into main May 28, 2026
1 check passed
@yolo-sam
yolo-sam deleted the codex/atag-opus-default-clean branch May 28, 2026 20:06
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