Skip to content

fix(homebrew): Disable telemetry during completion generation - #9

Merged
philprime merged 1 commit into
mainfrom
philprime/fix/homebrew-completion-telemetry
Sep 16, 2026
Merged

philprime merged 1 commit into
mainfrom
philprime/fix/homebrew-completion-telemetry

Conversation

@philprime

Copy link
Copy Markdown
Member

Disable telemetry inside the formula's install method before generating shell completions. Under Homebrew's macOS sandbox, Sentry cannot create its cache directory and writes initialization diagnostics to stdout, contaminating all three completion files and preventing Zsh from recognizing the leading #compdef directive.

Publish Apple Docs v0.0.5 as Homebrew revision 2 so existing installations regenerate their completions. This only disables telemetry during installation and leaves runtime telemetry and release binaries unchanged.

Validation

  • The regression fixture fails when telemetry output precedes completion content and passes with the install-time opt-out, independently of the parent environment.
  • make test-homebrew passes, including rendered-formula style checks and version-scoped revision assertions.
  • The actual v0.0.5 binary reproduces stdout contamination under a macOS sandbox denying filesystem writes and network access. With the updated formula, Bash, Zsh, and Fish output has clean headers and no Sentry diagnostics.
  • Interactive Zsh Tab checks pass for technologies, types view, and the technology option after a type name.
  • Formatting and pre-commit checks pass. Full publisher tests remain blocked locally by missing rpmbuild.

After merge, rerun publication for techprimate/apple-docs-cli@v0.0.5 to release 0.0.5_2 through the automated tap PR.

@philprime
philprime enabled auto-merge (squash) September 16, 2026 09:28
@philprime
philprime merged commit a14d098 into main Sep 16, 2026
1 of 2 checks passed
@philprime
philprime deleted the philprime/fix/homebrew-completion-telemetry branch September 16, 2026 09:29
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