Skip to content

fix(js): allow clean consumer installs#49

Merged
konard merged 2 commits into
mainfrom
issue-48-clean-consumer-install
Jul 27, 2026
Merged

fix(js): allow clean consumer installs#49
konard merged 2 commits into
mainfrom
issue-48-clean-consumer-install

Conversation

@konard

@konard konard commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

Summary

  • remove the obsolete published patch-package lifecycle and patch payload
  • require command-stream@^0.17.2, which contains the renderer fixes formerly carried by the patch
  • add a package-contract regression test and patch changeset

Reproduction

Installing agent-commander@0.10.0 as a dependency in a clean consumer project runs its published postinstall. The lifecycle executes from node_modules/agent-commander, cannot find the hoisted command-stream package there, and aborts the install with:

Error: Patch file found for package command-stream which is not present at node_modules/command-stream

This was reproduced by formal-ai Desktop Release run 30228722022 while consuming the package in link-assistant/formal-ai#851.

Verification

  • npm ci
  • npm test — 240 passed
  • bun test — 240 passed
  • deno test --allow-read --allow-run --allow-env --allow-net test/ — 234 passed, 6 runtime-specific ignores
  • npm run check
  • packed js/ and installed the tarball in an empty npm project; install succeeded with command-stream@0.17.2 and no patch payload

Closes #48

Downstream: link-assistant/formal-ai#851

@konard
konard merged commit d3296f5 into main Jul 27, 2026
50 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Published 0.10.0 fails clean npm installs in consumers

1 participant