[logging] inspector: convert chat ingest, tunnels, widgets to typed events

[Vu-John]

Summary

• Converts persistChatSessionToConvex to emit chat.session.persist.failed typed events (with version_conflict, http_error, timeout, exception failure kinds) when a Hono Context is available; falls back to logger.warn for callers without a request context
• Converts the widget-content POST handler to emit widget.resource.served on success and widget.resource.failed (with classifyWidgetError error codes) on early-return 404s and catch errors
• Converts tunnels.ts /create and /create/:serverId to emit tunnel.created (for both new and pre-existing tunnels) and tunnel.creation_failed on catch; adds tunnel.record_failed inside recordTunnel's catch block via an optional c?: Context parameter
• Converts OAuth proxy routes (/debug/proxy, /proxy, /metadata) to emit mcp.oauth.proxy.failed with targetUrlHost, oauthPhase, errorCode, and optional statusCode

Test plan

• 4 new tests for chat-ingestion event emission: VERSION_CONFLICT JSON, http_error, timeout/AbortError, exception
• All 759 server tests pass

🤖 Generated with Claude Code

[chelojimenez]

✅ Snyk checks have passed. No issues have been found so far.

Status	Scan Engine	Critical	High	Medium	Low	Total (0)
✅	Open Source Security	0	0	0	0	0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

[Vu-John]

• [logging] inspector: add deprecation guards, LOGGING.md, and ESLint enforcement #1934
• [logging] inspector: convert second-wave hosted routes to typed events #1933
• [logging] inspector: convert chat ingest, tunnels, widgets to typed events #1932 👈 (View in Graphite)
• [logging] inspector: consume backend InternalLogContext in authorize wrapper #1931
• [logging] inspector: add typed event foundation #1930
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[github-actions]

Internal preview

Preview URL will appear in Railway after the deploy finishes.
Deployed commit: ff4d7be
PR head commit: 3aa9855
Backend target: staging fallback.
Access is employee-only in non-production environments.

[chatgpt-codex-connector]

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 979e9dfc-72f0-47e6-80f7-3772b682e089

📥 Commits

Reviewing files that changed from the base of the PR and between 094c84f and 3aa9855.

📒 Files selected for processing (5)

• mcpjam-inspector/server/routes/apps/mcp-apps/index.ts
• mcpjam-inspector/server/routes/mcp/oauth.ts
• mcpjam-inspector/server/routes/mcp/tunnels.ts
• mcpjam-inspector/server/utils/__tests__/chat-ingestion.test.ts
• mcpjam-inspector/server/utils/chat-ingestion.ts

---

Walkthrough

This pull request instruments multiple server routes and utilities with structured request-scoped event logging. Changes add getRequestLogger integration across widget resource serving, OAuth proxy operations, and tunnel lifecycle handlers to emit classified error events and success metrics. The persistChatSessionToConvex utility is updated to accept an optional Hono Context parameter to enable conditional event emission on failure scenarios. New test cases validate event emission behavior across distinct failure paths. No response signatures or HTTP behaviors were altered.

---

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