Skip to content

fix(gbrain): probe CLI without command builtin#1485

Open
jbetala7 wants to merge 1 commit into
garrytan:mainfrom
jbetala7:oss/fix-1386-gbrain-windows-probe
Open

fix(gbrain): probe CLI without command builtin#1485
jbetala7 wants to merge 1 commit into
garrytan:mainfrom
jbetala7:oss/fix-1386-gbrain-windows-probe

Conversation

@jbetala7
Copy link
Copy Markdown
Contributor

Fixes #1386

Summary

  • remove the shell-based command -v gbrain probe from gstack-memory-ingest so Windows/Bun does not try to spawn the POSIX shell builtin as an executable
  • use a direct gbrain --help probe in /sync-gbrain while keeping dry-run code-sync previews independent of local gbrain availability
  • add focused regressions that prevent these paths from reintroducing command -v gbrain

Testing

  • bun test test/gstack-gbrain-sync.test.ts
  • bun test test/gstack-memory-ingest.test.ts
  • git diff --check

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.

Windows: gstack-memory-ingest.ts can't ingest at scale (follow-up to #1271)

1 participant