Skip to content

chore: publish hygiene — LICENSE, package metadata, drop src from tarball#9

Merged
AlejandroFabianCampos merged 2 commits into
mainfrom
chore/publish-hygiene
Apr 30, 2026
Merged

chore: publish hygiene — LICENSE, package metadata, drop src from tarball#9
AlejandroFabianCampos merged 2 commits into
mainfrom
chore/publish-hygiene

Conversation

@AlejandroFabianCampos
Copy link
Copy Markdown
Collaborator

@AlejandroFabianCampos AlejandroFabianCampos commented Apr 30, 2026

Summary

Three pre-publish hygiene fixes flagged in the v0.2.0 review:

  1. Add LICENSEpackage.json declares MIT but the file was missing. Standard MIT, copyright Quiknode Inc., 2026.
  2. Add package.json metadatarepository, homepage, bugs, engines.node (≥22). Without these, npm and GitHub presentation are degraded and the documented Node-22 requirement isn't enforced.
  3. files field aligned with the MPP first-party SDK spec["dist", "src", "!src/**/*.test.ts", "!src/__tests__"]. The spec recommends shipping src so consumers get "Go to Source Definition" in editors (matches mppx, tempo, ecosystem convention). Tests are excluded via gitignore-style negation in the files array — no .npmignore needed.

Test plan

  • npm run verify (lint + typecheck + 82 tests pass, 9 skipped)
  • npm pack --dry-run — 91 files, 46.9 kB
    • dist/ + src/ source included
    • 0 test files (*.test.ts, src/__tests__/) leak into tarball
    • LICENSE and README.md auto-included
    • Declaration maps in dist/ now resolve correctly (sources are present)

Comment thread package.json Outdated
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 1dc06bf. Configure here.

Comment thread package.json
@AlejandroFabianCampos AlejandroFabianCampos merged commit a578c03 into main Apr 30, 2026
2 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