Skip to content

fix stale helper exports for build plugins#14

Closed
jamilahmadzai wants to merge 1 commit into
storm-software:mainfrom
jamilahmadzai:codex/fix-powerlines-helper-exports
Closed

fix stale helper exports for build plugins#14
jamilahmadzai wants to merge 1 commit into
storm-software:mainfrom
jamilahmadzai:codex/fix-powerlines-helper-exports

Conversation

@jamilahmadzai

Copy link
Copy Markdown
Contributor

Summary

This removes stale helper subpath exports from the build plugin package manifests:

  • @powerlines/plugin-tsup
  • @powerlines/plugin-vite
  • @powerlines/plugin-tsdown
  • @powerlines/plugin-esbuild
  • @powerlines/plugin-rolldown

Those package.json exports point at dist/helpers/... files that are not present in the published packages or in these package source trees, so consumers can resolve a helper subpath and then fail at runtime with ERR_MODULE_NOT_FOUND.

The root exports, ./types exports, and ./package.json exports are unchanged.

Reproduction

For example, with the current published @powerlines/plugin-tsup@0.12.596 package:

mkdir -p /tmp/powerlines-plugin-tsup-pack-repro/node_modules/@powerlines
cd /tmp/powerlines-plugin-tsup-pack-repro
npm pack --silent @powerlines/plugin-tsup@0.12.596
mkdir -p node_modules/@powerlines/plugin-tsup
tar -xzf powerlines-plugin-tsup-0.12.596.tgz -C node_modules/@powerlines/plugin-tsup --strip-components=1
node --input-type=module -e "console.log(await import.meta.resolve('@powerlines/plugin-tsup/helpers'))"
node --input-type=module -e "await import('@powerlines/plugin-tsup/helpers')"

import.meta.resolve() points to dist/helpers/index.mjs, but that file is not included in the package, so importing it fails.

Validation

  • Checked the changed package manifests no longer expose any ./helpers* export keys.
  • git diff --check
  • pnpm install --frozen-lockfile --ignore-scripts

I also attempted to run the plugin-tsup Vitest entry directly, but the test process stops during setup before executing tests because the installed @storm-software/tsup package imports @nx/js/src/utils/buildable-libs-utils without the .js extension under this Node/ESM setup. This change is limited to package export metadata.

@jamilahmadzai jamilahmadzai requested a review from a team June 20, 2026 13:07

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello, I'm 🤖 Stormie-Bot! The Storm team sincerely appreciates your effort/interest in contributing to this project. A Storm developer will review this change and get back to you ASAP. Please feel free to reach out to the Storm team (contact@stormsoftware.com) if you have any questions/comments.

@jamilahmadzai jamilahmadzai changed the title [codex] fix stale helper exports for build plugins fix stale helper exports for build plugins Jun 20, 2026
@jamilahmadzai jamilahmadzai deleted the codex/fix-powerlines-helper-exports branch June 20, 2026 13:09
@storm-software storm-software locked and limited conversation to collaborators Jun 22, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant