Skip to content

fix: harden mac release service account reads - #32

Merged
steipete merged 1 commit into
mainfrom
steipete/vigorous-meitner-c861c5
Aug 8, 2026
Merged

fix: harden mac release service account reads#32
steipete merged 1 commit into
mainfrom
steipete/vigorous-meitner-c861c5

Conversation

@steipete

@steipete steipete commented Aug 8, 2026

Copy link
Copy Markdown
Owner

Summary

  • remove --account from explicit 1Password service-account reads and force desktop integration off
  • deliver the current OP_SERVICE_ACCOUNT_TOKEN to the shared op-work task window through a mode-0600 one-shot file, then delete it
  • preserve interactive desktop vault reads with account selection and pane TTY access

Tests

  • skills/release-mac-app/scripts/mac-release.test.sh
  • bash -n across the release helper scripts
  • shellcheck on the changed scripts
  • git diff --check
  • structured autoreview and secret scan

@steipete
steipete merged commit 1017707 into main Aug 8, 2026
2 checks passed
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.

1 participant