feat: better error logging

[premiumjibles]

Summary by CodeRabbit

• Chores
  • Enhanced error handling and logging throughout chat functionality to capture and report more comprehensive diagnostic information when errors occur. The system now logs all errors with detailed contextual information including error messages, error names, and stack traces to improve troubleshooting and system monitoring.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[vercel]

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

Project	Deployment	Preview	Comments	Updated (UTC)
shapeshift-agentic	Ready	Preview	Comment	Dec 7, 2025 10:57pm

[coderabbitai]

📝 Walkthrough

Walkthrough

These changes add client-side and server-side error logging to the chat system. The ChatProvider adds an onError callback for console logging, while the chat route adds stream error handling and structured error responses with detailed error information.

Changes

Cohort / File(s)	Summary
Error Handling Enhancement apps/agentic-chat/src/providers/ChatProvider.tsx, apps/agentic-server/src/routes/chat.ts	ChatProvider adds onError callback to log errors to console. Chat route adds onError handler for stream errors and enhances catch block to log detailed error information (message, name, stack) and return structured 500 response with error details.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

• Both changes involve straightforward error callback additions without complex logic
• Only 2 files affected with isolated modifications
• Error handling patterns are straightforward logging and response formatting

Poem

🐰 When errors hop across the chat,
We catch them now—imagine that!
With logs so clear, both high and low,
The bugs have nowhere left to go! 🪲✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title directly and accurately reflects the main change: both modified files add error logging functionality to chat error handling.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch better-logging

---

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.}

[coderabbitai]

Actionable comments posted: 1

🧹 Nitpick comments (2)

apps/agentic-chat/src/providers/ChatProvider.tsx (1)

82-89: LGTM! Consider adding error tracking integration.

The client-side error logging implementation is solid and provides good debugging visibility. The structured format with message, name, cause, and stack will help diagnose transport and chat errors.

For production monitoring, consider integrating an error tracking service (e.g., Sentry, LogRocket) alongside console logging:

 onError: error => {
   console.error('[Chat Error]', {
     message: error.message,
     name: error.name,
     cause: error.cause,
     stack: error.stack,
   })
+  // Example: Sentry.captureException(error)
 },

apps/agentic-server/src/routes/chat.ts (1)

262-268: LGTM! Consider structured logging service.

The stream error handler correctly captures and logs error details during the streaming phase. The type guards and fallbacks handle various error types appropriately.

For production environments, consider using a structured logging service (e.g., Winston, Pino) instead of console.error:

onError: ({ error }) => {
  logger.error('[Stream Error]', {
    message: error instanceof Error ? error.message : String(error),
    name: error instanceof Error ? error.name : 'Unknown',
    cause: error instanceof Error ? error.cause : undefined,
  })
},

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

• Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 71a38a1 and 32679fc.

📒 Files selected for processing (2)

• apps/agentic-chat/src/providers/ChatProvider.tsx (1 hunks)
• apps/agentic-server/src/routes/chat.ts (1 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: main