Skip to content

fix(package.json): move peer deps, add keywords, include CHANGELOG#233

Open
deborahamoni0-prog wants to merge 1 commit into
Sorokit:mainfrom
deborahamoni0-prog:fix/package-json-peer-deps
Open

fix(package.json): move peer deps, add keywords, include CHANGELOG#233
deborahamoni0-prog wants to merge 1 commit into
Sorokit:mainfrom
deborahamoni0-prog:fix/package-json-peer-deps

Conversation

@deborahamoni0-prog

Copy link
Copy Markdown

Description

Fixes #203: Addresses three package.json issues affecting publishing, peer dependencies, and npm discoverability.

Changes

  • Peer Dependencies: Move sorokit-core and @creit.tech/stellar-wallets-kit from dependencies to peerDependencies. This prevents duplicate copies in consuming apps and reduces bundle bloat. Version ranges: ^0.1.0 and ^0.0.0-beta.0 respectively.
  • Files Field: Add "files": ["dist", "README.md", "LICENSE", "CHANGELOG.md"] to exclude development artifacts (src/, _issues/, etc.) from npm publish.
  • Keywords: Add missing terms for npm search discoverability: dapp, defi, xlm, freighter, wallet-kit, soroban-contracts.

Verification

npm pack --dry-run:

  • Before: 3 files (LICENSE, README.md, package.json) - 13.3 kB unpacked
  • After: 4 files (CHANGELOG.md, LICENSE, README.md, package.json) - 16.1 kB unpacked

npm run build: Passed

  • ES module: 3.49 kB (gzip: 1.27 kB)
  • CommonJS: 2.59 kB (gzip: 1.11 kB)

Impact

  • Consuming apps no longer bundle duplicate peer dependencies
  • Smaller published package size (excludes development files)
  • Better npm search rankings for DeFi and Stellar dApp developers

… package

- Move sorokit-core and @creit.tech/stellar-wallets-kit from dependencies to peerDependencies
- Add 'files' field to include CHANGELOG.md in npm publish
- Add missing keywords: dapp, defi, xlm, freighter, wallet-kit, soroban-contracts

This prevents duplicate peer dependencies in consuming apps and reduces npm package bloat by excluding development artifacts. The keywords improve npm discoverability for DeFi and Stellar dApp developers.

npm pack --dry-run verification:
- Before: 3 files (LICENSE, README.md, package.json) - 13.3 kB unpacked
- After: 4 files (CHANGELOG.md, LICENSE, README.md, package.json) - 16.1 kB unpacked
- Build: ✓ ES module (3.49 kB gzip) + CommonJS (2.59 kB gzip)
@drips-wave

drips-wave Bot commented Jun 30, 2026

Copy link
Copy Markdown

@deborahamoni0-prog Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

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.

fix(package.json): peerDeps listed as deps, missing files field includes scratch dirs in publish, keywords missing DeFi terms

1 participant