Skip to content

chore: standardize release process#82

Merged
tylerjroach merged 4 commits into
masterfrom
release/standardize-release-process
May 20, 2026
Merged

chore: standardize release process#82
tylerjroach merged 4 commits into
masterfrom
release/standardize-release-process

Conversation

@tylerjroach
Copy link
Copy Markdown
Contributor

Summary

Standardizes this repo's release process to match the rest of the Mixpanel SDK fleet, per the SDK Release Process Standardization Effort.

After merge, releases follow a uniform two-step ceremony:

  1. Run the Prepare Release workflow (prepare-release.yml) — opens a release PR with the version bump, changelog section, and README header line updated.
  2. Push the release tag — release-packagist.yml validates, gates on the release GitHub environment for human approval, and publishes (where applicable for this ecosystem).

What's added

  • .github/modules.json — single source of truth for module config (paths, tag prefixes, package names)
  • .github/workflows/prepare-release.yml — manual dispatch, opens release PR
  • .github/workflows/release-packagist.yml — tag-triggered publish workflow
  • .github/workflows/pr-title-check.yml — conventional-commit enforcement (regex built from modules.json)
  • .github/scripts/generate-changelog.sh — shared changelog generator (verbatim from mixpanel-android)
  • README version-header line on each module's README, seeded with the current latest tag
  • CHANGELOG.md per module where missing
  • Single module analytics (mixpanel/mixpanel-php on Packagist)
  • Tag style preserved as bare semver (2.12.0, not v2.12.0) — Packagist resolves both styles, but switching would break Composer constraint resolution for downstream consumers
  • composer.json has no version field — Packagist reads from the git tag. The publish workflow has no upload step; Packagist auto-syncs via the existing GitHub webhook
  • New CHANGELOG.md seeded from the existing changelog history embedded in the README

How releases work after this lands

Full ceremony, one-time setup, and PR title conventions: PHP Release Runbook

One-time setup before the first release

The runbook lists the full setup. The work that requires repo-admin access (cannot be done in this PR):

  • Create a GitHub Environment named release with required reviewer(s)
  • Branch protection on the default branch (require PR review + the new Validate PR title check)
  • Tag rulesets (restrict creation/update/deletion of release tags)
  • Workflow permissions: enable Read and write permissions and Allow GitHub Actions to create and approve pull requests (needed by prepare-release.yml's gh pr create)

Notes

  • This is a WIP draft to be reviewed alongside the equivalent PRs in the other SDK repos. The Flutter and React Native ports are tracked at mixpanel-flutter#238, mixpanel-flutter-session-replay#30, mixpanel-react-native#408, mixpanel-react-native-session-replay#64.
  • This PR does not change any source code or build behavior — only adds release infrastructure.
  • The standardization commit is followed by a small fix: commit correcting a jq tag-resolution snippet (the same bug exists in the WIP Flutter/RN PRs and should be back-ported there).

Adds the shared release infrastructure (prepare-release, release-<ecosystem>,
pr-title-check workflows + modules.json + generate-changelog.sh) so this
repo's release flow matches the other Mixpanel SDK repositories.

See: https://www.notion.so/348e0ba925628029af63c779caa835f9
Aligns the changelog-section extraction with the deployed mixpanel-android
release workflow, which uses a sed range. The Python regex implementation
was an accident of port-time authorship; sed is the proven approach in the
gold-standard Maven Central pipeline.

Uses `\@...@` as the sed address delimiter so tags containing `/` (e.g.
`openfeature/v0.1.0`) don't conflict with the default `/`. Behavior is
otherwise preserved: file-based release_notes.md output, fallback to
"Release $TAG" placeholder when the section is missing or empty, and the
two-step structure for log visibility in the workflow run.
Aligns with the Android fleet's convention of allowing `feat(all): ...`,
`fix(all): ...`, `chore(all): ...` for cross-cutting changes that should
appear in every module's changelog. The shared generate-changelog.sh
already matches `all` (it was copied verbatim from mixpanel-android), so
this regex change is the only piece needed to make the end-to-end flow
accept `all`-scoped PR titles.

For single-module repos, `feat(all): foo` is functionally equivalent to
`feat(<only-module>): foo` — kept for fleet-wide consistency.
@tylerjroach tylerjroach force-pushed the release/standardize-release-process branch from 796198e to f9cf084 Compare May 14, 2026 17:15
@tylerjroach tylerjroach marked this pull request as ready for review May 19, 2026 13:29
@tylerjroach tylerjroach requested review from a team and jakewski May 19, 2026 13:29
Mirrors the same fix applied to mixpanel-flutter and mixpanel-react-native.

Previously the generator required every commit to be scoped to either
`<module>` or `all` — meaning a perfectly valid bare-titled PR like
`feat: add foo` would silently drop out of the changelog despite passing
the pr-title-check workflow (which accepts bare titles).

Split the rule so it matches pr-title-check semantics:

- feat / fix: bare, scoped to the current module, or scoped to `all`
- chore: explicit scope required — bare `chore:` is the convention for
  changes intentionally hidden from the changelog (release prep, CI
  tweaks, lockfile bumps, internal docs)

Other-module scopes (e.g. `feat(other):`) remain excluded from this
module's changelog, as before.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@tylerjroach tylerjroach merged commit f5de17e into master May 20, 2026
1 check 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