[jeffmaki]

Sentry wasn't receiving any backend events. The code was fine, but the App Services never set SENTRY_DSN, and without a DSN the SDK silently sends nothing. Prod now has SENTRY_DSN and ENV, set by hand. This PR makes a missing setting visible and fills in the release.

Startup log line. Sentry setup moves into init_sentry() in api/main.py, with the same settings as before; logging is now configured just before it. At startup it logs either:

• WARNING … SENTRY_DSN is not set, so errors are not reported to Sentry, or
• INFO … Reporting errors to Sentry: environment='production', release='<sha>'.

Release from the commit. Every event reported release unknown, because nothing set CODE_VERSION.

• The Dockerfile now takes a CODE_VERSION build argument and sets it as an environment variable, defaulting to unknown. It's declared after the dependency install, so a new commit doesn't invalidate the cached layers.
• push-docker-image.yml (the deploy workflow, also run by trigger-deploy-on-merge.yml) and docker-image.yml pass CODE_VERSION=${{ github.sha }}. That's the same SHA the deploy workflow uses as an image tag, so a Sentry release maps to the image you'd roll back to.
• Don't set CODE_VERSION as an App Service setting: it would override the image's value.

/health reports the running commit. It now returns {"status": "ok", "version": "<CODE_VERSION>"}, so you can confirm which commit is deployed with curl. status is unchanged, and probes only check the status code. The repo is public, so the SHA reveals nothing new.

Still to do outside this PR: add SENTRY_DSN and ENV (stage, dev) on ws-backend-stage and ws-backend-dev. Prod already has them.

Testing:

• New unit tests cover the startup warning and info line, with the Sentry SDK stubbed out.
• Two /health tests cover version with and without CODE_VERSION.
• A local Docker build confirmed the build arg reaches the container (and defaults to unknown without it).
• Run against a local fake Sentry server, the app reported an unhandled route exception, a logger.error call and a capture_message call once a DSN was set.
• ./scripts/ci.sh passes.

🤖 Generated with Claude Code

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

Run configuration

• Source mode: Auto
• Repository: Unavailable
• Branch: Unavailable
• File: No configuration file from this repository applied
• Commit: Unavailable
• Layers: Organization UI
• Review profile: CHILL
• Plan: Advanced
• Run ID: a5f5f6bc-ba9d-4f40-9956-220c23164964

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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