Skip to content

chore(release): 0.39.1#664

Merged
esengine merged 1 commit into
mainfrom
fix/postinstall-patch-package
May 11, 2026
Merged

chore(release): 0.39.1#664
esengine merged 1 commit into
mainfrom
fix/postinstall-patch-package

Conversation

@esengine

Copy link
Copy Markdown
Owner

Summary

Hotfix for 0.39.0 install regression that broke fresh npm install / npx reasonix@latest for every user. 0.39.0 has been deprecated on npm and latest dist-tag rolled back to 0.38.0 — this PR ships the replacement.

Root cause (0.39.0)

package.json added "postinstall": "patch-package" to apply an ink alt-screen render patch for #639. But:

  • patch-package was in devDependencies only, so it didn't ship to end-users — 'patch-package' is not recognized as an internal or external command.
  • patches/ was missing from the files array, so the patch wasn't even in the published tarball.
  • Most fundamentally, patch-package is designed for consumer projects to patch their own deps — it looks at ./node_modules/<dep>, but npm hoists ink to the consumer's top-level node_modules/ink, where patch-package running from inside node_modules/reasonix/ can't find it. Even if (1) and (2) had been fixed, the patch would never have applied.

Fix

Remove the entire patch-package machinery: postinstall script, patches/ directory, and the patch-package dependency. The ink alt-screen fix (#639) returns via a forked ink + npm: alias in a later release — tracked at #663. Until then alt-screen ghosting on CJK terminals matches 0.38.0 behavior (i.e., still present but not regressed).

Piggyback changes

Three PRs that were in flight land in 0.39.1:

Test plan

  • npm run verify — 2616 tests pass on the rebased branch
  • npm pack — verified tarball has no patches/, no postinstall, no patch-package in deps/devDeps
  • Install gauntlet — npm install ./reasonix-0.39.1.tgz into a fresh empty dir succeeds (added 101 packages in 9s, no postinstall warnings)
  • reasonix --version0.39.1
  • reasonix --help renders all commands
  • reasonix doctor --json → 9/9 ok

Post-merge

  • Tag v0.39.1, push tag
  • npm publish
  • npm dist-tag add reasonix@0.39.1 latest

Hotfix for 0.39.0 install failure: the `postinstall: patch-package`
hook + missing `patch-package` runtime dep + missing `patches/` in the
files array meant fresh `npm install` / `npx reasonix@latest` runs
crashed before the binary was usable. 0.39.0 has been deprecated on
npm and `latest` rolled back to 0.38.0.

Removes the patch-package machinery entirely — it can't work for a
published library since npm hoists `ink` out of reasonix's own
`node_modules`. The ink alt-screen render fix (#639) will return via
a forked ink + `npm:` alias (#663) in a later release.

Piggybacks three PRs that landed since 0.39.0:
- #632 fix(web): actionable web_search / web_fetch errors
- #661 feat(composer): Ctrl+X opens \$EDITOR
- #662 feat(composer): Ctrl+P / Ctrl+N picker navigation
@esengine esengine merged commit daf5399 into main May 11, 2026
3 checks passed
@esengine esengine deleted the fix/postinstall-patch-package branch May 11, 2026 08:55
ChasLui pushed a commit to ChasLui/DeepSeek-Reasonix that referenced this pull request May 23, 2026
Hotfix for 0.39.0 install failure: the `postinstall: patch-package`
hook + missing `patch-package` runtime dep + missing `patches/` in the
files array meant fresh `npm install` / `npx reasonix@latest` runs
crashed before the binary was usable. 0.39.0 has been deprecated on
npm and `latest` rolled back to 0.38.0.

Removes the patch-package machinery entirely — it can't work for a
published library since npm hoists `ink` out of reasonix's own
`node_modules`. The ink alt-screen render fix (esengine#639) will return via
a forked ink + `npm:` alias (esengine#663) in a later release.

Piggybacks three PRs that landed since 0.39.0:
- esengine#632 fix(web): actionable web_search / web_fetch errors
- esengine#661 feat(composer): Ctrl+X opens \$EDITOR
- esengine#662 feat(composer): Ctrl+P / Ctrl+N picker navigation
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.

1 participant