fix: address macOS rsync and Bun stdin issues (#12) #82

Merged

jonas.hanisch merged 2 commits from fix/issue-12-bugfixes into main

2026-01-22 18:29:22 +00:00

jack commented

2026-01-22 18:23:00 +00:00

Owner

Summary

Fix macOS rsync permission errors with --no-perms flag
Add xattr -cr cleanup before rsync on macOS (clears extended attributes)
Handle rsync exit codes 23/24 gracefully (partial transfer warnings)
Add isStdinValid() utility to prevent Bun crashes on invalid stdin fd

Bug Status from Issue #12

Bug	Status	Details
1. macOS rsync permissions	✅ Fixed	Added `--no-perms`, `xattr -cr`, exit code handling
2. Memory sessions in resume	✅ Already works	`cwd` is tracked in observations
3. Chroma sandbox fallback	⚠️ N/A	Project uses Qdrant by default
4. Bun stdin crash	✅ Fixed	Added `isStdinValid()` guard
5. HealthMonitor path	✅ Already fixed	Uses `MARKETPLACE_ROOT`
6-9. Additional fixes	✅ Already fixed	Merged from upstream

Test plan

Test rsync sync on macOS to verify no permission errors
Test stdin reading in environments where stdin is not connected
Verify build completes successfully

Partially addresses #12

Generated with Claude Code

## Summary - Fix macOS rsync permission errors with `--no-perms` flag - Add `xattr -cr` cleanup before rsync on macOS (clears extended attributes) - Handle rsync exit codes 23/24 gracefully (partial transfer warnings) - Add `isStdinValid()` utility to prevent Bun crashes on invalid stdin fd ## Bug Status from Issue #12 | Bug | Status | Details | |-----|--------|---------| | 1. macOS rsync permissions | ✅ Fixed | Added `--no-perms`, `xattr -cr`, exit code handling | | 2. Memory sessions in resume | ✅ Already works | `cwd` is tracked in observations | | 3. Chroma sandbox fallback | ⚠️ N/A | Project uses Qdrant by default | | 4. Bun stdin crash | ✅ Fixed | Added `isStdinValid()` guard | | 5. HealthMonitor path | ✅ Already fixed | Uses `MARKETPLACE_ROOT` | | 6-9. Additional fixes | ✅ Already fixed | Merged from upstream | ## Test plan - [ ] Test rsync sync on macOS to verify no permission errors - [ ] Test stdin reading in environments where stdin is not connected - [ ] Verify build completes successfully Partially addresses #12 --- Generated with [Claude Code](https://claude.com/claude-code)

jack added 1 commit

2026-01-22 18:23:00 +00:00

fix: address macOS rsync and Bun stdin issues (#12 )

CI / build (pull_request) Successful in 21s

Details

CI / validate-plugin (pull_request) Successful in 2s

Details

c3e4686f6b

- Add --no-perms flag to rsync commands to avoid permission errors on macOS
- Add xattr -cr cleanup before rsync on macOS (clears extended attributes)
- Handle rsync exit codes 23/24 gracefully (partial transfer warnings)
- Add isStdinValid() utility to prevent Bun crashes on invalid stdin fd
- Add timeout and error handling to stdin reader

Partially addresses #12

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

jack added the

labels

2026-01-22 18:23:02 +00:00

jack referenced this pull request

2026-01-22 18:23:13 +00:00

Fix 9 critical bugs (stdin crash, ESM/CJS, SSE drops) #12

jonas.hanisch added 1 commit

2026-01-22 18:27:00 +00:00

fix: add pip fallback for Chroma MCP in sandbox environments (#12 )

CI / build (pull_request) Successful in 20s

Details

CI / validate-plugin (pull_request) Successful in 2s

Details

c29fe3a9ee

- Try uvx first (preferred method)
- Fall back to pip-installed chroma-mcp via python -m
- Helpful error message if neither is available
- Fixes Chroma in Claude Code sandbox where uvx may panic

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>