Skip to content

Draft release until all assets upload, then publish#242

Merged
gewenyu99 merged 1 commit into
mainfrom
posthog-code/fix-release-draft-window
Jul 16, 2026
Merged

Draft release until all assets upload, then publish#242
gewenyu99 merged 1 commit into
mainfrom
posthog-code/fix-release-draft-window

Conversation

@gewenyu99

Copy link
Copy Markdown
Collaborator

Fixes a race that bricks the wizard on every release.

Problem

The release was created published (draft: false) before any assets were uploaded. Because the wizard resolves releases/latest, the new release went live the instant it was created — but skill-menu.json is uploaded last, a few minutes later. For that window releases/latest points at a release with no skill-menu.json, so the wizard 404s.

Fix

Create the release as a draft, upload every asset, then flip it to published as the final step. releases/latest ignores drafts, so it keeps pointing at the previous complete release until the new one is fully populated — closing the window entirely.

Why

Raised in a Slack thread: context-mill builds take a few minutes, and the publish-then-upload ordering left a ~3-minute window that broke the wizard on every release.


Created with PostHog from a Slack thread

The release was created published (draft: false) before any assets were
uploaded. Since the wizard resolves releases/latest, the release went live
pointing at the new version while skill-menu.json — uploaded last — was still
minutes away, 404ing the wizard for that window.

Create the release as a draft, upload every asset, then flip it to published
as the final step. releases/latest ignores drafts, so it keeps pointing at the
previous complete release until the new one is fully populated.

Generated-By: PostHog Code
Task-Id: 58d8e9e2-5c34-49cb-a53e-f0d51d2079c4
@github-actions

Copy link
Copy Markdown

🧙 Wizard CI

Run the Wizard CI and test your changes against wizard-workbench example apps by replying with a GitHub comment using one of the following commands:

Test all apps:

  • /wizard-ci all

Test all apps in a directory:

  • /wizard-ci basic-integration
  • /wizard-ci mcp-analytics
  • /wizard-ci revenue

Test an individual app:

  • /wizard-ci basic-integration/android
  • /wizard-ci basic-integration/angular
  • /wizard-ci basic-integration/astro
Show more apps
  • /wizard-ci basic-integration/django
  • /wizard-ci basic-integration/fastapi
  • /wizard-ci basic-integration/flask
  • /wizard-ci basic-integration/javascript-node
  • /wizard-ci basic-integration/javascript-web
  • /wizard-ci basic-integration/laravel
  • /wizard-ci basic-integration/next-js
  • /wizard-ci basic-integration/nuxt
  • /wizard-ci basic-integration/python
  • /wizard-ci basic-integration/rails
  • /wizard-ci basic-integration/react-native
  • /wizard-ci basic-integration/react-router
  • /wizard-ci basic-integration/sveltekit
  • /wizard-ci basic-integration/swift
  • /wizard-ci basic-integration/tanstack-router
  • /wizard-ci basic-integration/tanstack-start
  • /wizard-ci basic-integration/vue
  • /wizard-ci mcp-analytics/custom-dispatcher
  • /wizard-ci mcp-analytics/typescript-sdk
  • /wizard-ci revenue/stripe

Results will be posted here when complete.

@gewenyu99
gewenyu99 marked this pull request as ready for review July 15, 2026 23:45
@gewenyu99
gewenyu99 requested a review from a team as a code owner July 15, 2026 23:45

@sarahxsanders sarahxsanders left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good catch

@gewenyu99
gewenyu99 merged commit fb90251 into main Jul 16, 2026
15 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.

2 participants