refactor: v2 - ai assistant - split tools bridges

[maxy-shpfy]

Description

The monolithic toolBridge.ts file has been split into focused, single-responsibility modules under a new toolBridge/ directory. The three handler slices are:

• csomBridge.ts — spec-mutation handlers (add/delete/rename tasks, inputs, outputs, connections, subgraphs, validation)
• runBridge.ts — run lifecycle handlers (submitPipelineRun, getRunDetails, debugPipelineRun)
• debugBridge.ts — fine-grained read-only execution/container fetch handlers
• utils.ts — shared BridgeDeps interface, requireSpec, requireBackendUrl, errorMessage, and computeNextPosition
• index.ts — composes the three slices into the full ToolBridgeApi via spread

Truncation logic (log text, container state, execution details) that was duplicated between debugTools.ts and toolBridge.ts has been extracted into a single shared module at src/agent/util/truncate.ts. Both the worker-side per-tool path and the main-thread composite debugPipelineRun path now import from this one location, ensuring identical truncation caps regardless of which entry point the model uses.

The deepClone call on the serialized spec before pipeline submission has been removed since serializeComponentSpec already produces a plain object.

Related Issue and Pull requests

Type of Change

• Bug fix
• New feature
• Improvement
• Cleanup/Refactor
• Breaking change
• Documentation update

Checklist

• I have tested this does not break current pipelines / runs functionality
• I have tested the changes on staging

Screenshots (if applicable)

Test Instructions

1. Open a pipeline in the editor and confirm the AI chat agent can still read pipeline state, add/remove tasks and connections, and validate the pipeline.
2. Submit a pipeline run via the agent and verify the run appears in the runs list and the query cache is invalidated.
3. Trigger debugPipelineRun on a run with failed children and confirm truncated logs and container state are returned correctly.
4. Call getContainerLog, getExecutionDetails, and getContainerState directly through the debug assistant and verify raw payloads are returned without truncation at the bridge layer.
5. Confirm that truncation applied by debugTools.ts (worker side) and runBridge.ts (main thread) produces identical output for the same input by comparing results from both paths.

Additional Comments

The BridgeDeps type is re-exported from toolBridge/index.ts so call sites that previously imported from toolBridge.ts require only a path update with no interface changes.

[maxy-shpfy]

• chore: v2 - ai assistant - architecture md #2344
• chore: v2 - ai assistant - unit tests #2336
• feat: v2 - ai assistant - architect subagent #2335
• refactor: v2 - ai assistant - split tools bridges #2333 👈 (View in Graphite)
• feat: v2 - ai assistent - debug agent #2332
• feat: v2 - ai assistent - repair subagent #2331
• feat: v2 - ai assistant - tangle help subagent #2330
• feat: v2 - ai assistant llm proxy and dispatcher agent #2329
• feat: v2 - ai assistant web worker scaffolding #2328
• feat: v2 - ai assistant chat window #2327
• master

This stack of pull requests is managed by Graphite. Learn more about stacking.

[github-actions]

🎩 Preview

A preview build has been created at: 05-27-refactor_v2_-_ai_assistant_-_split_tools_bridges/51a1c12

[maxy-shpfy]

Merge activity

• Jun 2, 7:53 PM UTC: A user started a stack merge that includes this pull request via Graphite.
• Jun 2, 8:16 PM UTC: Graphite rebased this pull request as part of a merge.
• Jun 2, 8:19 PM UTC: @maxy-shpfy merged this pull request with Graphite.