Skip to content

Add prepare script to ensure build artifacts are included in published package #179

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
syavorsky merged 3 commits into from
Jan 16, 2026
Merged

Add prepare script to ensure build artifacts are included in published package #179

syavorsky merged 3 commits into from
Jan 16, 2026

Conversation

Copy link
Contributor

Copilot AI commented Jan 16, 2026 •
edited
Loading

The published package is missing built artifacts (lib/, es6/, browser/), causing module resolution failures:

Error: Cannot find module '.../comment-parser/lib/index.cjs'

These directories are gitignored and were not being built before npm publish unless npm version was run first.

Changes

  • Add prepare script to run build before npm pack and npm publish
  • Add prepublishOnly script as additional safeguard before publish
  • Add *.tgz to .gitignore to exclude build artifacts

The prepare hook ensures built files exist in the tarball regardless of publish workflow.

Original prompt

This section details on the original issue you should resolve

<issue_title>broken release</issue_title>
<issue_description>```
Error: Cannot find module '/Users/gajus/Developer/gajus/slonik/node_modules/.pnpm/eslint-plugin-regexp@2.10.0_eslint@9.39.2_jiti@2.6.1_/node_modules/comment-parser/lib/index.cjs'
at createEsmNotFoundErr (node:internal/modules/cjs/loader:1457:15)
at finalizeEsmResolution (node:internal/modules/cjs/loader:1446:9)
at resolveExports (node:internal/modules/cjs/loader:678:14)


## Comments on the Issue (you are @copilot in this section)

<comments>
<comment_new><author>@syavorsky</author><body>
@gajus can you hint me how I can reproduce this issue?</body></comment_new>
</comments>

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 2 commits January 16, 2026 22:58
@syavorsky
Initial analysis of broken release issue
45670a8 
Co-authored-by: syavorsky <455779+syavorsky@users.noreply.github.com>
@syavorsky
Add prepare and prepublishOnly scripts to ensure build runs before pu…
a5a65e0 
…blishing

Co-authored-by: syavorsky <455779+syavorsky@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix module not found error in release Add prepare script to ensure build artifacts are included in published package Jan 16, 2026
Copilot AI requested a review from syavorsky January 16, 2026 23:06
@syavorsky syavorsky marked this pull request as ready for review January 16, 2026 23:09
@syavorsky syavorsky merged commit b1b19d6 into master Jan 16, 2026
6 checks passed
@syavorsky syavorsky deleted the copilot/fix-broken-release-issue branch January 16, 2026 23:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@syavorsky syavorsky Awaiting requested review from syavorsky

Assignees

@syavorsky syavorsky

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

broken release

2 participants

@syavorsky