feat(hooks): add question.asked and session.error notifications

[orientpine]

Summary

Add OS notifications for two additional event types that were previously not covered:

• question.asked: Notifies when the agent asks a question (e.g., during Prometheus interview sessions)
• session.error: Notifies when the session encounters an error (e.g., "The final block in an assistant message cannot be thinking")

Problem

Currently, session-notification.ts only triggers notifications on session.idle events. This means:

1. When Prometheus asks clarifying questions, users don't get notified and may miss the prompt
2. When sessions encounter errors and stop, users don't get notified about the failure

Solution

Added handlers for both event types following the existing patterns:

• Skip subagent sessions
• Only notify for main session
• Immediate notification (no confirmation delay needed)
• Error messages are truncated to 100 characters for readability

Test Coverage

Added 7 new test cases covering:

• question.asked notification for main session
• question.asked skipped for subagent
• question.asked skipped for non-main session
• session.error notification for main session
• session.error skipped for subagent
• session.error skipped for non-main session
• session.error with missing error message (fallback to default)

Verification

• All 18 tests pass (bun test src/hooks/session-notification.test.ts)
• TypeScript type check passes (tsc --noEmit)

---

Summary by cubic

Add OS notifications for question.asked and session.error so users see prompts and failures immediately. This prevents missed questions during interviews and silent session errors.

• New Features
  • Notify on question.asked for the main session; ignore subagent and non-main sessions.
  • Notify on session.error with the error message truncated to 100 chars.
  • Immediate notification; optional sound uses existing config.

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

[github-actions]

All contributors have signed the CLA. Thank you! ✅
_{Posted by the CLA Assistant Lite bot.}

[orientpine]

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

[cubic-dev-ai]

No issues found across 2 files

Confidence score: 5/5

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