fix(codemode): restore npm trusted publishing metadata#24
Merged
Conversation
Rationale: npm trusted publishing rejected the 0.4.1 release because the package still declared the old repository coordinate. npm requires the exact GitHub repository for the configured publisher. Risk: Metadata-only change; publishing remains blocked until npm's trusted publisher is configured for publish.yml if it is not already. Tested: mise exec -- packages/codemode/node_modules/.bin/vitest run packages/codemode/test/package-publication.test.ts; mise exec -- pnpm --filter @robinbraemer/codemode run typecheck.
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
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.
Intent
Ship CodeMode 0.4.1 to production for CNAP by fixing the release-only npm authentication failure without adding long-lived publishing credentials. The npm package metadata must declare the exact GitHub repository coordinate akua-dev/codemode required by npm trusted publishing, and a regression test must prevent the legacy cnap-tech coordinate from returning. Preserve the existing OIDC release workflow, public package identity, and all previously merged OpenAPI-search safety fixes.
What Changed
akua-dev/codemode.Risk Assessment
✅ Low: The change is narrowly scoped to correcting CodeMode’s repository metadata and adding a regression assertion, while preserving package identity, OIDC workflow permissions, and prior safety changes.
Testing
Installed the pinned pnpm toolchain dependencies, built the local LLRT and CodeMode artifacts needed for isolated-worktree testing, reran the focused publication/OpenAPI/sandbox suites, verified package metadata and OIDC workflow assertions, and exercised npm pack plus public npm publish in dry-run mode. The initial missing-LLRT setup issue was resolved; transient build and dependency artifacts were removed, leaving the worktree source-clean.
Evidence: Targeted Vitest evidence
Evidence: npm metadata and OIDC workflow evidence
Evidence: npm publish dry-run evidence
Pipeline
Updates from git push no-mistakes
✅ **intent** - passed
✅ No issues found.
✅ **Rebase** - passed
✅ No issues found.
✅ **Review** - passed
✅ No issues found.
✅ **Test** - passed
✅ No issues found.
npm exec --yes --package=pnpm@10 -- pnpm install --filter '@robinbraemer/codemode...' --frozen-lockfile --ignore-scriptsnpm exec --yes --package=pnpm@10 -- pnpm --filter @robinbraemer/llrt buildnpm exec --yes --package=pnpm@10 -- pnpm --filter @robinbraemer/codemode exec vitest run test/package-publication.test.ts test/spec.test.ts test/data-only.test.ts test/codemode.test.tsnode --input-type=module -e '...metadata/workflow assertions...'npm exec --yes --package=pnpm@10 -- pnpm --filter @robinbraemer/codemode buildnpm pack --dry-run --json --ignore-scriptsnpm publish --dry-run --access public --ignore-scriptsgit diff --quiet 97995819183de3edcf0b82812e84ffa5351ec038 78b8a3fe52dd0c0002640aacd1e84174f54b6d20 -- .github/workflows/publish.yml packages/codemode/src packages/codemode/test/codemode.test.ts packages/codemode/test/spec.test.ts packages/codemode/test/data-only.test.tsgit status --short --untracked-files=all🔧 **Document** - 1 issue found → auto-fixed ✅
packages/llrt/package.json:51- LLRT and its generated native package manifests still advertise cnap-tech/codemode; updating them coherently requires a separate package-metadata regeneration change.🔧 Fix: Refresh repository metadata and pass lint checks
✅ Re-checked - no issues remain.
✅ **Lint** - passed
✅ No issues found.
✅ **Push** - passed
✅ No issues found.