Skip to content

Fix editor thread storage host routing - #3605

Merged
SawyerHood merged 1 commit into
mainfrom
slopcop/issue-3604
Sep 14, 2026
Merged

SawyerHood merged 1 commit into
mainfrom
slopcop/issue-3604

Conversation

@bb-slop-cop

@bb-slop-cop bb-slop-cop Bot commented Sep 13, 2026

Copy link
Copy Markdown
Contributor

🚨 SLOP COP 🚨 · new-issue-autopilot

Human comments

What was wrong

The editor resolved thread-storage roots from server-local configuration and omitted the owning host ID. Remote-thread read, tree, and save requests consequently targeted the server. Reproduction report.

What changed

Resolve the existing thread storage location through the SDK and forward its root and host ID to file operations. Remove the local-root helper and add three routing regression tests. Total: 112 changed text lines across two files (103 additions, 9 deletions).

How you verified

  • Trusted main and a second clean checkout: three routing tests failed; 34 existing tests passed.
  • With the fix: corepack pnpm exec turbo run test typecheck --filter=bb-plugin-monaco-editor passed, including all 37 tests.
  • Full base Turbo build passed; git diff --check passed.
  • Tests exercise the real plugin RPC handlers with SDK test doubles. Live two-machine browser behavior and physical writes were not exercised.

Fixes #3604

AGENT GENERATED

@SawyerHood
SawyerHood merged commit 283e6d7 into main Sep 14, 2026
16 checks passed
@SawyerHood
SawyerHood deleted the slopcop/issue-3604 branch September 14, 2026 20:31
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.

File Editor (monaco-editor) cannot open thread storage for threads on a non-server machine: text files 404 and the tree is empty

1 participant