Skip to content

fix(ci): put the helper signing keychain ON the search list - #458

Merged
theogravity merged 1 commit into
mainfrom
fix/desktop-helper-keychain-list
Oct 26, 2026
Merged

theogravity merged 1 commit into
mainfrom
fix/desktop-helper-keychain-list

Conversation

@theogravity

Copy link
Copy Markdown
Contributor

Follow-up to #457, which stopped unlisting the helper keychain but never listed it. Probe run 40325026239 on macos-14 settled the mechanism: security create-keychain does not append to the user's search list on this image, and codesign resolves a named identity through the search list only - --keychain alone never helps. Same keychain: unlisted = "item could not be found", listed = signs clean. The fix makes the keychain the sole search-list entry exactly as the CLI darwin shards already do (the path that signed cli-server-v3.0.2 today); the step's trap restores the baseline. Both duplicated desktop job bodies edited identically (release-cut-order drift test passes); docs paragraph rewritten to the probe-proven account.

#457 stopped unlisting the helper keychain but never listed it: on this
runner image 'security create-keychain' does not append to the user's
search list, so the identity stayed invisible to codesign and
cli/desktop darwin shards still died with 'item could not be found'.
Probe run 40325026239 on macos-14 proved the split: the same keychain
fails while unlisted and signs cleanly once listed, and --keychain
alone never helps. List it as the sole entry exactly as the CLI darwin
shards already do (the path that signed cli-server-v3.0.2 today); the
trap restores the baseline at step end.
@theogravity
theogravity merged commit ed59dac into main Oct 26, 2026
21 of 23 checks passed
@theogravity
theogravity deleted the fix/desktop-helper-keychain-list branch October 26, 2026 13:08
@github-actions github-actions Bot locked and limited conversation to collaborators Oct 26, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant