Skip to content

probe: verbatim commands with an exact-match allowlist - #138

Merged
yihanzhu merged 1 commit into
mainfrom
probe-tune
Aug 27, 2026
Merged

probe: verbatim commands with an exact-match allowlist#138
yihanzhu merged 1 commit into
mainfrom
probe-tune

Conversation

@yihanzhu

Copy link
Copy Markdown
Owner

Attempt 3 proved the token and execution work; its two denials were allowlist/spelling mismatches on the writes. The probe now gives Claude the exact commands verbatim and allowlists those strings one-for-one — determinism on both sides.

Also learned (attempt 4): the action skips execution for workflow_dispatch on non-default refs, so probe changes only test after merging — hence this PR.

Stack B rule this bakes in: stage prompts state exact commands that mirror their allowlists.

🤖 Generated with Claude Code

Attempt 3 proved the token works and Claude executes; the two denials
were allowlist/spelling mismatches on the writes. The prompt now gives
exact commands and the allowlist mirrors them string-for-string.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@yihanzhu
yihanzhu merged commit 437b15b into main Aug 27, 2026
2 of 3 checks passed
@yihanzhu
yihanzhu deleted the probe-tune branch August 27, 2026 15:01
@yihanzhu yihanzhu added the merge-ready Exact reviewed head+base passed; YOUR merge; void if either moves label Aug 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merge-ready Exact reviewed head+base passed; YOUR merge; void if either moves

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant