Skip to content

Fix package types entry#60

Open
luuhongyii wants to merge 1 commit into
rpf3:mainfrom
luuhongyii:codex/fix-types-entry
Open

Fix package types entry#60
luuhongyii wants to merge 1 commit into
rpf3:mainfrom
luuhongyii:codex/fix-types-entry

Conversation

@luuhongyii

Copy link
Copy Markdown

Summary

  • point the top-level types field to the generated declaration file at dist/index.d.ts
  • leave the existing exports and typesVersions mappings aligned with the same declaration entry

Problem

@rpf3/spek@0.95.0 publishes dist/index.d.ts, but the package manifest currently advertises:

"types": "index.d.ts"

The published package tarball does not include a root index.d.ts, so tools that read the package-level types field can resolve to a missing declaration file even though the correct declaration file is already present under dist/.

Verification

  • npm view @rpf3/spek@0.95.0 name version types exports --json
  • npm pack @rpf3/spek@0.95.0 --dry-run --ignore-scripts --json confirms root index.d.ts is absent and dist/index.d.ts is present
  • corepack pnpm@8.15.9 install --frozen-lockfile --ignore-scripts
  • corepack pnpm@8.15.9 exec svelte-kit sync
  • corepack pnpm@8.15.9 run package
  • corepack pnpm@8.15.9 run check passes with 0 errors and 4 existing accessibility warnings
  • local npm pack --dry-run --ignore-scripts --json after package generation confirms the configured types file exists in the packed file list
  • git diff --check

Note

corepack pnpm@8.15.9 test -- --reporter=list currently exits with No tests found; I left that unrelated test-suite setup issue out of this focused manifest fix.

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