fix(chat): name stackless error subclasses and compact stored unfinished tool params

[waleedlatif1]

Summary

• Function errors: recognize Node's stackless [Class [name]: message] headline (an error subclass that keeps its parent's name), so it reports the thrown error instead of falling back to Execution failed. Names match the stacked form (QuotaExceeded [Error])
• Chat transcript: stored assistant turns now compact params of unfinished tool calls too — they render as interrupted and are never re-run or approved. Only the live assistant message keeps unfinished params whole
• One shared isStoredAssistantMessage predicate for display and compaction so the two can't drift

Type of Change

• Bug fix

Testing

• code-failure.test.ts and lib/mothership/chat/** tests pass (559); new cases fail on the pre-fix code
• Real Node 22 / Python stderr harness: 5 of 61 cases improve (all previously Execution failed), the rest unchanged; non-error bracket forms ([Object: …], [Function: …], [class A [T]]) still don't match
• biome on changed files; type-check and audits in CI

Checklist

• Code follows project style guidelines
• Self-reviewed my changes
• Tests added/updated and passing (new tests pass the test-audit authoring gate)
• No new warnings introduced
• I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

[cubic-dev-ai]

No issues found across 6 files

Confidence score: 5/5

• Automated review surfaced no issues in the provided summaries.
• No files require special attention.

<sub>Heads up: you’ve reached your flex budget. Increase your flex budget or wait for usage to reset.

Turn on auto-fix | Re-trigger cubic</sub>

[greptile-apps]

Confidence Score: 5/5

[Medium impact] This PR appears safe to merge; no actionable issues were found.

Summary

This PR recognizes Node's stackless error subclass headlines and shortens long tool params in stored assistant turns.

• Stackless Node errors keep the subclass name they were thrown with.
• Stored unfinished tool calls show shorter params, while live calls keep full params.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart TD
  A[Assistant message] --> B{Stored message?}
  B -->|Yes| C[Compact long tool params]
  C --> D[Show unfinished calls as interrupted]
  B -->|No| E{Unfinished call?}
  E -->|Yes| F[Keep full params]
  E -->|No| C

Loading

_{Reviews (1) · Last reviewed commit: "fix(chat): name stackless error subclass..." · Reviewed by Greptile}

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated
docs	Skipped		Oct 12, 2026 6:45pm UTC