airkjw/claude-mem
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

MAESTRO: Mark PR #771 complete in PR-Triage-11 - stdin hang fix merged

Browse Source 
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
Alex Newman
2026-02-06 03:06:38 -05:00
parent 830d7a2b23
commit 2f935a56b9
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Auto Run Docs/PR-Triage/PR-Triage-11.md
+2 -1
View File
@@ -26,7 +26,8 @@ Standalone bug fixes that don't group neatly into other phases.
- [x] Review PR #524 (`fix: add minimum bun version check to smart-install.js` by @quicktime). File: `plugin/scripts/smart-install.js`. Steps: (1) `gh pr checkout 524` (2) Review version check logic — what minimum version? Is it still relevant? (3) If clean: `gh pr merge 524 --rebase --delete-branch`
- **Merged.** Adds `MIN_BUN_VERSION = '1.1.14'` check with auto-upgrade via `bun upgrade`. Clean semver `compareVersions()` function, `isBunVersionSufficient()` check inserted between bun install and uv install steps. Minimum version is justified: bun 1.1.14+ required for `.changes` property on SQLite `.run()` and multi-statement SQL support (1.0.26+). Falls back gracefully with manual upgrade instructions if auto-upgrade fails. Fixes #519.
- [ ] Review PR #771 (`fix: handle stdin unavailability and timeout to prevent hook hangs` by @rajivsinclair). File: `src/cli/stdin-reader.ts`. Steps: (1) `gh pr checkout 771` (2) Review — stdin may not be available in all environments (CI, some Windows configs) (3) Should add timeout and graceful fallback (4) Run `npm run build` (5) If clean: `gh pr merge 771 --rebase --delete-branch`
- [x] Review PR #771 (`fix: handle stdin unavailability and timeout to prevent hook hangs` by @rajivsinclair). File: `src/cli/stdin-reader.ts`. Steps: (1) `gh pr checkout 771` (2) Review — stdin may not be available in all environments (CI, some Windows configs) (3) Should add timeout and graceful fallback (4) Run `npm run build` (5) If clean: `gh pr merge 771 --rebase --delete-branch`
- **Merged.** Replaces naive `stdin.on('end')` listener (which hangs when Claude Code doesn't close stdin) with JSON self-delimiting detection — tries `JSON.parse()` after each chunk and resolves immediately on valid JSON. Adds `isStdinAvailable()` guard for Bun EINVAL crash (#646), 30s safety timeout for malformed input, and graceful error handling (returns `undefined` instead of crashing). Fixes #727, addresses #646. TypeScript compilation clean.
## Cursor Integration