Repository navigation
🔧 chore: migrate to pnpm + Node 22 and adopt workflow-scripts release flow - #15
Merged
Merged
Conversation
… flow Replace the localazy/release@v2 composite actions with the @localazy/workflow-scripts release flow, and switch the package manager from npm to pnpm on Node 22 — matching the current org convention (directus-extension-localazy #101/#102). - release.yml: two commit-message-gated jobs (create-release-pr / publish-release) driven by @localazy/workflow-scripts@latest; publish via `pnpm publish` - qa.yml: pnpm + Node 22 via .nvmrc; single `pnpm run check` + build - package.json: packageManager pnpm@11.4.0, engines node>=22, @types/node ^22, internal scripts npm->pnpm - declare typescript ~5.4.2 as a direct devDependency and run `tsc` instead of `npx tsc`: pnpm's strict node_modules does not hoist transitive binaries, so tsc must be a direct dependency - add pnpm-workspace.yaml (allowBuilds: esbuild) and pnpm-lock.yaml, remove package-lock.json - .prettierignore: ignore dist and pnpm-lock.yaml - vitest: passWithNoTests so the empty suite exits 0 - reformat service.ts / scope.ts under the bumped prettier (no behaviour change; enum values untouched) Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
elisiondan
force-pushed
the
chore-migrate-pnpm-node22-release-flow
branch
from
July 14, 2026 08:49
e5273ec to
c14ac7c
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Modernises the build/release toolchain for this package, following the current org convention set by
directus-extension-localazy(#101 "adopt release flow via@localazy/workflow-scripts" and #102 "migrate from npm workspaces to pnpm").localazy/release@v2composite actions with two commit-message-gated jobs driven by@localazy/workflow-scripts@latest:create-release-pron normal pushes tomainpublish-releaseon the merged🚀 release:commit → build,pnpm publish,create-git-tag,create-github-releasepackageManagerpin);package-lock.json→pnpm-lock.yaml; newpnpm-workspace.yamlwith anallowBuildsallowlist (esbuild)..nvmrc,engines.node >= 22,@types/node ^22).qa.ymlnow runs on pnpm + Node 22 and uses a singlepnpm run check+ build.vitestpassWithNoTests(empty suite exits 0);.prettierignoreignoresdist+pnpm-lock.yaml; reformattedservice.ts/scope.tsunder the bumped prettier (no behaviour change — enum values untouched).Verified locally (Node 22.21.1 / pnpm 11.4.0)
pnpm install --frozen-lockfile— clean (noERR_PNPM_IGNORED_BUILDSafter theallowBuildsallowlist)pnpm run check(typecheck + test + lint + prettier) — passpnpm run build— passpnpm publish --dry-run— resolves@localazy/generic-connector-client@0.4.0Relationship to #14
This PR should merge first. #14 ("reassign
IN_CONTEXT_EDITORto plugin id 8") is the follow-up and contains only the enum value change. Because this PR reformatssrc/enums/service.ts(unquoting members), #14 will need a trivial rebase ontomainafterwards so its7 → 8re-applies in the new unquoted style.Test plan
qa.ymlpasses on this PRmainopens a proper release PR viacreate-release-prNPM_AUTH_TOKEN_PUBLIC,AUTH_APP_ID,AUTH_APP_KEYreach this repo (already used by the oldrelease.yml)🤖 Generated with Claude Code