chore: satisfies npm-pkg-lint & publint #94
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.
With the check of trending use of
npm-pkg-lint
andpublint
:Basically this means:
src/**/*.test.ts
files should not be bundled into the package1."main": "./dist/index.js"
doesn't exist, since this package is sub-directory oriented,index.ts
doesn't exist"types": "./dist/index.d.ts"
doesn't exist eithermain
is only for Node.js,"main"
is not neededkeywords
repository
engines.node
Motivation and Context
Follow the best practice of Node.js community projects and guidelines of how
package.json
and distributing packages can be organized.How Has This Been Tested?
Yes, locally tested with
verdaccio
(private npm registry) with my MCP projects.Breaking Changes
No.
Types of changes
Checklist
Additional context
Footnotes
node.js - Is it a good practices to add tests files in the tsconfig.json exclude field? - Stack Overflow ↩
https://rollupjs.org/es-module-syntax/ ↩
Modules: Packages | Node.js v23.4.0 Documentation ↩
javascript - What is the "module" package.json field for? - Stack Overflow ↩
https://github.com/nodejs/node-eps/blob/4217dca299d89c8c18ac44c878b5fe9581974ef3/002-es6-modules.md#51-determining-if-source-is-an-es-module ↩