fix: log swallowed errors across scheduler, MCP, UI, and storage layers

[devin-ai-integration]

What Changed

Replaces ~30 bare catch {} / catch { return fallback } / .catch(() => fallback) patterns with structured console.warn calls that surface the error message using the existing [tag] prefix convention. Every catch still returns the same fallback value — no behavioral change — but now the error is logged so operators can diagnose failures.

Files (15):

Area	File	What was silent
Scheduler	service.ts	fireJobCompleteCallbacks swallowed callback errors
Scheduler	parse-with-sonnet.ts	Sonnet judge failure returned generic 422 with no server-side log
Scheduler	schedule.ts	Invalid cron in computeNextRunAt returned null with no log
Onboarding	profiler.ts	Three Slack API calls (.catch(() => null)) with no trace
MCP	tools-swe.ts	recallEpisodes/recallFacts failures in codebase_query and repo_info
MCP	tools-universal.ts	recallEpisodes/recallFacts/findProcedure in searchMemoryCore
MCP	resources.ts	healthCheck, recallEpisodes, recallFacts in resource handlers
UI	pages.ts	extractTitle empty catch
UI	identity.ts	readMetaSync, tmp-file cleanup, old avatar pruning, delete
UI	evolution.ts	listPoisonPile, statSync/readFileSync in readFilePreview
Secrets	crypto.ts	Encryption key file read fallback to generation
Persona	llm-caller.ts	JSON envelope parse failure
Skills	storage.ts	Previous-content read on upsert/delete
Subagents	storage.ts	Previous-content read on upsert/delete
Plugins	curated.ts	Overlay stat failure

Why

Bare catch {} blocks silently swallow errors, making it impossible to diagnose why graceful degradation kicked in. When memory, Slack, scheduling, or file I/O silently fails, the operator has no log trail. These warnings surface the root cause without changing any runtime behavior.

How I Tested

bun test          # 2640 pass, 0 fail
bun run lint      # clean
bun run typecheck # clean

All 2640 tests pass unchanged — every fix is additive logging, no behavior change.

Checklist

• Tests pass (bun test)
• Lint passes (bun run lint)
• Typecheck passes (bun run typecheck)
• No secrets or .env files included
• Files stay under 300 lines
• No Cardinal Rule violations (TypeScript does plumbing only, the Agent SDK does reasoning)
• No default exports or barrel files added

Link to Devin session: https://app.devin.ai/sessions/fcaafb7496fb40bb95440bc7c00d240b
Requested by: @mcheemaa

[devin-ai-integration]

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

• Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
• Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

• Disable automatic comment, CI, and merge conflict monitoring