Skip to content

fix: trigger publish workflows on lerna tag format - #3

Merged
MD4 merged 2 commits into
masterfrom
fix/publish-workflow-tag-triggers
Jun 11, 2026
Merged

fix: trigger publish workflows on lerna tag format#3
MD4 merged 2 commits into
masterfrom
fix/publish-workflow-tag-triggers

Conversation

@MD4

@MD4 MD4 commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Problem

The five JS SDK publish workflows (angular, browser, react, vue, react-native) never run on release. They trigger on tags like `sdk-/v*`, but the documented release flow (`npm run version` → Lerna in independent mode) creates tags as `@screeb/@`. No Lerna tag ever matched the trigger patterns, so the workflows never fired.

Existing tags confirm the format: `@screeb/sdk-react@0.7.0`, `@screeb/react-native@3.3.0`, `@screeb/sdk-browser@0.6.0`, etc.

Flutter and MAUI are unaffected — `CONTRIBUTING.md` documents tagging those manually as `sdk-flutter/vX.Y.Z` / `sdk-maui/vX.Y.Z`, which already matches their triggers.

Changes

  • Align the 5 JS workflow triggers to Lerna's actual tag format:
    Workflow New trigger
    publish-angular `@screeb/sdk-angular@*`
    publish-browser `@screeb/sdk-browser@*`
    publish-react `@screeb/sdk-react@*`
    publish-vue `@screeb/sdk-vue@*`
    publish-reactnative `@screeb/react-native@*` (distinct package name)
  • Remove the root `publish` script. With CI publishing on tag push, the documented flow is `npm run version` (tags only); the local `lerna publish` would double-publish.

The glob patterns are scoped per-package — e.g. `@screeb/sdk-angular@*` does not match `@screeb/sdk-angular-example@...` (the char after the package name must be `@`).

Follow-up (not in this PR)

npm trusted publishing: the workflows use `--provenance` + OIDC (`id-token: write`). Each package must have a trusted publisher configured on npmjs.com pointing at its workflow file, or the publish step will fail at auth on the first run.

Verification

  • After merge, run `npm run version` to bump a package; the pushed `@screeb/@` tag should now trigger the matching workflow.
  • Optional dry run: push a throwaway tag (e.g. `@screeb/sdk-browser@0.6.1-test`) and confirm the workflow starts in the Actions tab, then delete the tag.

MD4 added 2 commits June 11, 2026 11:31
The JS publish workflows triggered on 'sdk-<name>/v*' tags, but lerna
(independent mode) creates tags as '@screeb/<name>@<version>', so the
workflows never ran on release. Align the 5 JS workflow triggers to
lerna's actual tag format.

Also remove the root 'publish' script: with CI publishing on tag push,
'npm run version' (tags only) is the documented flow, and the local
'lerna publish' would double-publish.
@MD4
MD4 merged commit a9732a7 into master Jun 11, 2026
2 of 3 checks passed
@Alexays
Alexays deleted the fix/publish-workflow-tag-triggers branch June 18, 2026 12:59
Alexays pushed a commit that referenced this pull request Jul 9, 2026
fix: trigger publish workflows on lerna tag format
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