fix: resolve blox workspace provisioning and ACP issues#38
Merged
wesbillman merged 2 commits intomainfrom Feb 11, 2026
Merged
Conversation
Three fixes for remote workspace support: 1. Resolve local repo path to HTTPS remote URL before passing to `blox ws start` (blox cannot fetch from local filesystem paths, and SSH URLs are also unsupported). 2. Handle integer status codes in blox JSON output — the CLI returns status as a numeric enum (e.g. 3 = running) rather than a string. Added a custom deserializer that accepts both formats. 3. Split goose --with-builtin args so each gets its own flag, avoiding the comma-in-value conflict with blox's comma-separated --command format. Co-Authored-By: Claude Opus 4.6 <[email protected]>
When running through `blox acp`, the local working_dir (e.g. /Users/.../builderbot) was being sent in ACP NewSessionRequest and LoadSessionRequest. The remote agent would report this non-existent path as its working directory. Now AcpDriver tracks `is_remote` and sends "." instead, so the remote agent resolves its own workspace path. Co-Authored-By: Claude Opus 4.6 <[email protected]>
cd7111e to
30f7936
Compare
matt2e
approved these changes
Feb 11, 2026
loganj
added a commit
that referenced
this pull request
Feb 26, 2026
Seed recent activity from filesystem on startup
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
blox ws promptapproach with proper ACP protocol proxied throughblox acp <workspace>, enabling full session/timeline support for remote branches/Users/.../builderbot) in ACP session requests — sends"."so the remote agent uses its own workspace directoryblox ws info/blox ws list(was expecting strings, causing parse failures)[email protected]:...to HTTPS before passing toblox ws start(blox needs a fetchable URL)--with-builtin developer,extensionmanagerinto separate flags per CLI conventionsend_workspace_promptcommand andws_promptfunctionTest plan
🤖 Generated with Claude Code