Skip to content

feat(desktop,core): remote workspace tool context and snapshot/stream fixes#602

Merged
bobleer merged 1 commit intoGCWing:mainfrom
bobleer:feat/remote-workspace-tool-context-snapshot-stream
May 8, 2026
Merged

feat(desktop,core): remote workspace tool context and snapshot/stream fixes#602
bobleer merged 1 commit intoGCWing:mainfrom
bobleer:feat/remote-workspace-tool-context-snapshot-stream

Conversation

@bobleer
Copy link
Copy Markdown
Collaborator

@bobleer bobleer commented May 8, 2026

Summary

  • Desktop tool_api builds ToolUseContext with remote WorkspaceBinding and attaches workspace_services from the remote workspace manager when validating and executing tools.
  • ExecutionEngine forwards workspace_services into the tool-availability probe so tool listings match runtime filesystem access (e.g. project skills on remote hosts).
  • Stream processing treats whitespace-only text chunks as non-effective output so first-visible timing is not skewed.
  • Snapshot APIs return empty files and zeroed stats for remote workspace paths instead of initializing a local snapshot manager.
  • Adds unit tests for remote skill tool description indexing and whitespace-only stream chunks.

Test plan

  • cargo check --workspace
  • cargo test -p bitfun-core skill_tool::tests -- --nocapture
  • cargo test -p bitfun-core whitespace_only_text -- --nocapture

… fixes

- Wire desktop tool API ToolUseContext with remote WorkspaceBinding and workspace services
- Propagate workspace_services into execution engine tool availability context
- Ignore whitespace-only stream chunks for effective output / first visible timing
- Short-circuit snapshot session files and stats for remote workspace paths
- Add unit tests for remote skill discovery and whitespace stream handling
@bobleer bobleer merged commit 935a9a4 into GCWing:main May 8, 2026
4 checks passed
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