Publish the compiled output, which is the whole package #5

Merged
jschoubben merged 1 commits from feat/publish-what-it-builds into main 2026-09-14 09:08:50 +00:00
Owner

This package compiles itself on install and then shipped everything except the result.

With no explicit file list, npm falls back to .gitignore — and .gitignore excludes the build output. So the install hook ran, compiled the package, and npm then packed the sources and left the compiled directory behind. Every consumer received a package whose every entry point pointed at a directory that had just been built and then excluded.

The workaround for this lived in mesh-tools, which installed this package and then compiled it by hand, with a comment saying the hook "does not run". The hook runs. The output was being thrown away afterwards.

This package compiles itself on install and then shipped everything except the result. With no explicit file list, npm falls back to `.gitignore` — and `.gitignore` excludes the build output. So the install hook ran, compiled the package, and npm then packed the sources and left the compiled directory behind. Every consumer received a package whose every entry point pointed at a directory that had just been built and then excluded. The workaround for this lived in mesh-tools, which installed this package and then compiled it by hand, with a comment saying the hook "does not run". The hook runs. The output was being thrown away afterwards.
jschoubben added 1 commit 2026-09-14 09:08:43 +00:00
It compiles itself on install and then shipped everything except the result.
With no explicit file list npm falls back to .gitignore, which ignores dist —
so every consumer received a package whose every entry point pointed at a
directory that had just been built and then excluded.

The workaround for this lived in mesh-tools, which compiled the dependency by
hand after installing it.
jschoubben merged commit 77ebcb3904 into main 2026-09-14 09:08:50 +00:00
Sign in to join this conversation.
No Reviewers
No labels
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: novox/mesh-sdk#5