Skip to content

Fix release publish package metadata#143

Merged
petyosi merged 1 commit into
mainfrom
petyosi/add-vitest-catalog-entry
Jun 9, 2026
Merged

Fix release publish package metadata#143
petyosi merged 1 commit into
mainfrom
petyosi/add-vitest-catalog-entry

Conversation

@petyosi

@petyosi petyosi commented Jun 9, 2026

Copy link
Copy Markdown
Member

Summary

  • Add the missing vitest catalog entry used by packages/logfire-api.
  • Move package LICENSE cleanup from postpack to postpublish for all publishable packages.

Root Cause

After #142 fixed the frozen-lockfile failure, the main release job reached changesets/action publish mode and failed inside pnpm run release.

The first publish blocker was:

[ERR_PNPM_CATALOG_ENTRY_NOT_FOUND_FOR_SPEC] No catalog entry 'vitest' was found for catalog 'default'.

packages/logfire-api/package.json uses "vitest": "catalog:", but pnpm-workspace.yaml only had a vitest override, not a catalog entry.

While verifying the fix with pnpm publish --dry-run, pnpm 11 exposed a second publish lifecycle issue: postpack removed the copied LICENSE before pnpm finished publishing, causing an ENOENT on LICENSE. Moving cleanup to postpublish keeps LICENSE present through packing/publish and still cleans it afterward.

Verification

  • pnpm publish -r --access public --dry-run --no-git-checks
  • vp install --frozen-lockfile
  • vp check
  • vp run --filter "./packages/*" build

@petyosi
petyosi merged commit ce583e1 into main Jun 9, 2026
2 checks passed
@petyosi
petyosi deleted the petyosi/add-vitest-catalog-entry branch June 9, 2026 18:16
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