Skip to content

Replace .npmignore with files field to slim published package - #344

Merged
RomiC merged 1 commit into
mainfrom
chore/slim-package
Aug 8, 2026
Merged

Replace .npmignore with files field to slim published package#344
RomiC merged 1 commit into
mainfrom
chore/slim-package

Conversation

@RomiC

@RomiC RomiC commented Aug 8, 2026

Copy link
Copy Markdown
Owner

Switched from deny-list .npmignore to allow-list "files" in package.json. Published package drops from 17 files to 8 — only index.js, lib/, dist/, package.json, README.md, and LICENSE are included.

@RomiC RomiC self-assigned this Aug 8, 2026
@RomiC RomiC added enhancement dx Changes relative to the Developer eXperience labels Aug 8, 2026
@greptile-apps

greptile-apps Bot commented Aug 8, 2026

Copy link
Copy Markdown

Greptile Summary

The PR replaces the package publication deny-list with an explicit npm allow-list to reduce the published artifact.

  • Deletes .npmignore.
  • Publishes source entry points and runtime code from index.js, lib/, and dist/.
  • Retains all declared CommonJS and ESM package entry points.

Confidence Score: 5/5

The PR appears safe to merge because the reduced package contents retain all declared entry points and runtime dependencies.

The allow-list contains index.js, the complete runtime lib directory, and the generated dist directory, while npm retains required package metadata and documentation automatically.

Important Files Changed

Filename Overview
package.json Adds a publication allow-list that includes every declared entry point and required runtime file.
.npmignore Removes the obsolete deny-list now superseded by package.json's explicit files field.

Reviews (1): Last reviewed commit: "Chore: Replace .npmignore with files fie..." | Re-trigger Greptile

@coveralls

Copy link
Copy Markdown

Coverage Status

coverage: 100.0%. remained the same — chore/slim-package into main

@RomiC
RomiC merged commit ef94999 into main Aug 8, 2026
8 checks passed
@RomiC
RomiC deleted the chore/slim-package branch August 8, 2026 15:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dx Changes relative to the Developer eXperience enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants