Skip to content

fix(ci): upgrade PostHog CLI for release races - #3952

Merged
trunk-io[bot] merged 1 commit into
mainfrom
fix/sourcemap-release-race
Jul 29, 2026
Merged

fix(ci): upgrade PostHog CLI for release races#3952
trunk-io[bot] merged 1 commit into
mainfrom
fix/sourcemap-release-race

Conversation

@jonathanlab

@jonathanlab jonathanlab commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Summary

  • upgrade the root @posthog/cli dependency from 0.7.5 to 0.8.4
  • use the upstream concurrent release creation fix shipped in CLI 0.8.2
  • remove the need for workflow-level build retries

Why

Parallel release jobs can both resolve the same sourcemap release as absent, then race to create it. The losing process receives release_hash_in_use, which CLI 0.7.5 treats as fatal. CLI 0.8.2 added explicit handling for this exact race: it recognizes the structured conflict and resolves the release created by the winning process.

The Rollup plugin resolves posthog-cli from the repository-level node_modules/.bin, so the root dependency controls the binary used in release builds.

Upstream fix: PostHog/posthog#70061

Validation

  • pnpm install
  • node_modules/.bin/posthog-cli --versionposthog-cli 0.8.4
  • inspected the upstream fix and confirmed it handles structured release_hash_in_use conflicts

@trunk-io

trunk-io Bot commented Jul 29, 2026

Copy link
Copy Markdown

😎 Merged successfully - details.

Comment thread scripts/build-electron-vite-with-retry.mjs Outdated
@greptile-apps

greptile-apps Bot commented Jul 29, 2026

Copy link
Copy Markdown
Contributor
Prompt To Fix All With AI
### Issue 1
scripts/build-electron-vite-with-retry.mjs:35-40
**Retry flow lacks automated coverage**

All platform release builds now depend on branches for successful retries, exhausted retries, and immediate non-retryable failures, but these paths have only manual simulation coverage. Add parameterized automated tests so changes to error matching, attempt limits, or exit-code propagation do not silently break releases.

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

Reviews (1): Last reviewed commit: "fix(ci): retry sourcemap release creatio..." | Re-trigger Greptile

@jonathanlab
jonathanlab force-pushed the fix/sourcemap-release-race branch from fde94e8 to a9f1d20 Compare July 29, 2026 14:41
@jonathanlab jonathanlab changed the title fix(ci): retry sourcemap release creation races fix(ci): upgrade PostHog CLI for release races Jul 29, 2026
@jonathanlab jonathanlab added the Create Release This will trigger a new release label Jul 29, 2026
@trunk-io
trunk-io Bot merged commit 6e5802d into main Jul 29, 2026
39 checks passed
@trunk-io
trunk-io Bot deleted the fix/sourcemap-release-race branch July 29, 2026 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Create Release This will trigger a new release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants