refactor: unify AbortError detection with isAbortError utility

[amabito]

Summary

• Abort error detection was scattered across multiple files with inconsistent patterns (name.includes, name.startsWith, === "cancel", code === "ABORT_ERR")
• Introduces a shared isAbortError(error) utility and replaces inline checks

Changes

• core/util/isAbortError.ts (new) — unified abort detection covering all known patterns
• core/util/isAbortError.test.ts (new) — 14 test cases (positive + negative)
• core/llm/index.ts — replaced 2 inline abort checks with isAbortError()
• core/llm/utils/retry.ts — replaced 1 inline abort check with isAbortError()

Semantics preserved

• Uses includes("AbortError") to match the original behavior at all replacement sites
• L506 replacement intentionally widens suppression to also cover "cancel" and code="ABORT_ERR" errors from console logging (these are all abort cases that should not pollute the console)
• No functional behavior change for end users

Test plan

• isAbortError.test.ts passes (14 cases)
• Existing tests unaffected
• core/llm/index.test.ts still passes

🤖 Generated with Claude Code

---

Continue Tasks: ✅ 1 no changes — View all

---

Summary by cubic

Unifies abort error detection with a shared isAbortError utility across LLM core to keep handling consistent and cut noisy abort logs. No behavior change for end users.

• Refactors
  • Added core/util/isAbortError.ts; detects "cancel", DOMException AbortError, Error.name containing "AbortError", code "ABORT_ERR", and plain objects with name.
  • Replaced inline checks in core/llm/index.ts and core/llm/utils/retry.ts with isAbortError(); console debug suppression now skips all abort variants.
  • Added 14 vitest cases in core/util/isAbortError.vitest.ts.

^{Written for commit ab9267b. Summary will update on new commits.}

[github-actions]

All contributors have signed the CLA ✍️ ✅
_{Posted by the CLA Assistant Lite bot.}

[amabito]

I have read the CLA Document and I hereby sign the CLA

[cubic-dev-ai]

No issues found across 4 files

[amabito]

I have read the CLA Document and I hereby sign the CLA

[amabito]

recheck

[amabito]

I have read the CLA Document and I hereby sign the CLA

[amabito]

Heads up: the only failure appears to be a flaky GUI e2e check (GUI.test.js e2e:ci:run-yaml) unrelated to this refactor. All other required checks are green.

A rerun of the failing job should unblock merge.