Skip to content

fix(research): add console.warn to silent catch blocks in research/maritime handlers#817

Open
NewCoder3294 wants to merge 1 commit intokoala73:mainfrom
NewCoder3294:fix/research-handler-observability
Open

fix(research): add console.warn to silent catch blocks in research/maritime handlers#817
NewCoder3294 wants to merge 1 commit intokoala73:mainfrom
NewCoder3294:fix/research-handler-observability

Conversation

@NewCoder3294
Copy link
Contributor

Summary

  • Adds console.warn logging to 8 silent catch blocks across 4 handler files in the research and maritime modules
  • Follows the same pattern established in PR fix(intel): add console.warn to silent catch blocks for observability #807 (which covered intelligence handlers)
  • Each warning includes the handler name in brackets for grep-ability, a brief description of what failed, and the caught error object

Files modified:

  • server/worldmonitor/research/v1/list-arxiv-papers.ts — 1 catch block
  • server/worldmonitor/research/v1/list-trending-repos.ts — 3 catch blocks (primary API, fallback API, handler)
  • server/worldmonitor/research/v1/list-hackernews-items.ts — 2 catch blocks (per-item fetch, handler)
  • server/worldmonitor/maritime/v1/get-vessel-snapshot.ts — 2 catch blocks (relay fetch, handler)

No logic changes — only added logging to previously silent catch blocks.

Test plan

  • Verify TypeScript compilation passes (npm run typecheck)
  • Confirm no runtime behavior changes — all catch blocks still return the same fallback values
  • Optionally trigger a failure scenario (e.g., network timeout) and verify the warning appears in server logs

🤖 Generated with Claude Code

…ritime handlers

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@vercel
Copy link

vercel bot commented Mar 2, 2026

@NewCoder3294 is attempting to deploy a commit to the Elie Team on Vercel.

A member of the Team first needs to authorize it.

@koala73 koala73 added Ready to Merge PR is mergeable, passes checks, and adds value Low Value Trivial, unnecessary, or not aligned with project needs labels Mar 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Low Value Trivial, unnecessary, or not aligned with project needs Ready to Merge PR is mergeable, passes checks, and adds value

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants