Skip to content

fix: @W-24754518 CI/CD - Migrate CLI plugin-data-setup-transfer repository to use NPM Trusted Publishing - #34

Merged
johnchenlive merged 1 commit into
mainfrom
trusted-publishing
Nov 3, 2026
Merged

johnchenlive merged 1 commit into
mainfrom
trusted-publishing

Conversation

@johnchenlive

Copy link
Copy Markdown
Collaborator

⚠️ Do not merge yet. Waiting on the Platform CLI team (#platform-cli-collaboration) to configure trusted publishing in NPM. A release must happen within 24 hours of that configuration.

What

Migrate npm publishing to trusted publishing, following plugin-agent#503 since this repo was on dev-scripts < v14.

  • onRelease.yml: add id-token: write / contents: read permissions and trustedPublishing: true
  • @salesforce/dev-scripts ^11.0.4 → ^15.0.0 (brings eslint 10 + TypeScript 6)
    • typescript → ^6.0.0, eslint-plugin-sf-plugin → ^3.0.4, add eslint, eslint-config-salesforce-typescript, @types/*
  • ESLint flat config: replace .eslintrc.cjs, test/.eslintrc.cjs, .eslintignore with eslint.config.mjs (keeps header rule off)
  • tsconfig.json: add types: ["mocha", "node"] (required under TS 6)
  • transfer.ts: fix new lint findings (unnecessary type assertion; attach cause to rethrown error)
  • No npm shrinkwrap usage to remove

Testing

  • yarn build ✅
  • yarn test ✅ (9 passing)

🤖 Generated with Claude Code

… to use NPM Trusted Publishing

- onRelease.yml: enable trustedPublishing with id-token: write permission
- bump @salesforce/dev-scripts to v15 (eslint 10, typescript 6)
- migrate to eslint flat config (eslint.config.mjs)
- add mocha/node types to tsconfig for typescript 6
- fix new lint findings in transfer command

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@johnchenlive johnchenlive changed the title @W-24754518 CI/CD - Migrate CLI plugin-data-setup-transfer repository to use NPM Trusted Publishing fix: @W-24754518 CI/CD - Migrate CLI plugin-data-setup-transfer repository to use NPM Trusted Publishing Nov 3, 2026
@johnchenlive
johnchenlive merged commit 34f7673 into main Nov 3, 2026
14 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