refactor(cue): cue-polish merge and cleanup

[reachraza]

Summary

• Fix Cue pipeline ENOENT spawn error — Cue executor was using the bare command name (e.g., claude) instead of the resolved binary path, causing spawn claude ENOENT failures. Now resolves the full path via agentDetector.getAgent() before spawning, matching the normal process flow.
• Remove unused catch (e) binding in web-server-factory.ts (eslint no-unused-vars fix)
• Fix duplicate fontSizeReset shortcut entry in shortcuts.ts
• Sync package.json version

This branch was rebased on top of rc after the cue-engine decomposition and review fixes were merged via PR #578. No new cue logic changes — all decomposition modules (cue-activity-log, cue-heartbeat, cue-subscription-setup, cue-fan-in-tracker, cue-run-manager) and bug fixes (concurrency slot leak, stale nextTriggers, fan-in guard, reconciler dispatch routing) are already in rc.

Test plan

• npm run lint and npm run lint:eslint clean (0 errors/warnings)
• Full test suite passing (23,491+ tests)
• Cue tests: 477 passing, 0 regressions

[greptile-apps]

Greptile Summary

This is a small cleanup/polish PR that applies three targeted fixes with no functional logic changes: it removes an unused catch binding in web-server-factory.ts, de-duplicates the fontSizeReset shortcut entry in shortcuts.ts, and adds --ignore-unknown to the prettier lint-staged command in package.json.

• web-server-factory.ts: catch (e) → catch {} — valid TypeScript optional-catch-binding syntax (TS 4.0+), silences the ESLint no-unused-vars warning without altering error-handling behavior.
• shortcuts.ts: The fontSizeReset shortcut (['Meta', 'Shift', '0']) was present twice in DEFAULT_SHORTCUTS; the duplicate occurrence (before openCue) is removed, leaving a single canonical entry after jumpToTerminal.
• package.json: prettier --write → prettier --write --ignore-unknown in lint-staged prevents prettier from throwing on file types it cannot process (binary assets, uncommon extensions), which is a commonly recommended configuration for projects with mixed file types.

Confidence Score: 5/5

• This PR is safe to merge — all three changes are minimal, well-scoped, and carry no risk.
• All changes are purely cosmetic/lint fixes: an optional catch-binding removal, a duplicate shortcut entry de-duplication, and a safer prettier flag. No logic paths are altered, no new dependencies are introduced, and the stated test results (23 k+ tests, 477 cue tests, 0 regressions) align with the trivial scope of the diff.
• No files require special attention.

Important Files Changed

Filename	Overview
src/main/web-server/web-server-factory.ts	Removes unused e binding from a catch block (catch (e) → catch {}), a clean ESLint no-unused-vars fix; the error path is correctly handled via logger.warn.
src/renderer/constants/shortcuts.ts	Removes the duplicate fontSizeReset entry (previously appeared twice in DEFAULT_SHORTCUTS); the remaining single entry is placed after jumpToTerminal with correct keys ['Meta', 'Shift', '0'].
package.json	Adds --ignore-unknown to the prettier --write command in lint-staged, preventing prettier from erroring on file types it cannot format (e.g. images, binaries).

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[lint-staged triggered on commit] --> B{"File type known\nby prettier?"}
    B -- "Yes (--ignore-unknown)" --> C[prettier --write]
    B -- "No (--ignore-unknown)" --> D[Skip silently ✓]
    C --> E[eslint --fix\n*.js/.ts/.tsx etc.]
    D --> E
    E --> F[Commit proceeds]

    G[web-server-factory.ts\npersistentWebLink = true] --> H{Valid UUID\nin store?}
    H -- Yes --> I[Use stored token]
    H -- No --> J[Generate randomUUID]
    J --> K[try: persist token]
    K -- success --> L[Server starts\nwith persistent token]
    K -- "catch {}\nnon-fatal" --> M[logger.warn\nServer starts\nwith ephemeral token]

    N[DEFAULT_SHORTCUTS\nshortcuts.ts] --> O[...\ndirectorNotes\nopenCue\nfilterUnreadAgents\njumpToTerminal\nfontSizeReset ✓ single entry]

Loading

_{Last reviewed commit: "fix: remove unused c..."}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: a27a8173-6e3d-4390-8464-7490ad1fbf77

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch cue-polish

📝 Coding Plan

• Generate coding plan for human review comments

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}