Skip to content

test(agent-isolation): keep the shim test's commit from signing - #1831

Merged
potiuk merged 1 commit into
mainfrom
fix/shim-test-ignore-signing
Oct 25, 2026
Merged

potiuk merged 1 commit into
mainfrom
fix/shim-test-ignore-signing

Conversation

@potiuk

@potiuk potiuk commented Oct 25, 2026

Copy link
Copy Markdown
Member

test_a_linked_worktree_targets_the_main_checkout makes one commit in a throwaway repository.
That commit picked up the developer's global commit.gpgsign, so with a hardware signing key it waited for a touch and then failed: the test failed locally while passing in CI.
The commit now runs with -c commit.gpgsign=false; the repository has nothing to sign.

Found while preparing #1829. The whole test_hook_manager_shim.py suite passes with a global commit.gpgsign=true and a YubiKey-backed signing key.


Generative AI disclosure: drafted with Claude Code (Claude Opus 5) and reviewed by the committer.

🤖 Generated with Claude Code

test_a_linked_worktree_targets_the_main_checkout makes one commit in a
throwaway repository, which picked up the developer's global
commit.gpgsign. With a hardware signing key the commit waited for a
touch and then failed, so the test failed locally while passing in CI.
The commit now runs with commit.gpgsign=false.

Generated-by: Claude Opus 5
@potiuk
potiuk enabled auto-merge (squash) October 25, 2026 20:27
@github-actions github-actions Bot added family:tools tools/* substrate:sandbox Tool substrate: agent isolation, egress control, settings audit labels Oct 25, 2026
@potiuk
potiuk merged commit 9634e3e into main Oct 25, 2026
10 checks passed
@potiuk
potiuk deleted the fix/shim-test-ignore-signing branch October 25, 2026 20:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

family:tools tools/* substrate:sandbox Tool substrate: agent isolation, egress control, settings audit

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant