You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Install the Debian CA certificate bundle in the REES runtime image so sentry-cli can verify TLS when uploading source maps from Railway deploys.
No issue because this is a small follow-up to production deploy verification: the previous image built and validated source maps, but runtime upload failed with unable to get local issuer certificate.
Scope
The PR title follows type(scope): short summary Conventional Commit format, for example fix(api): restore profile access checks.
This PR is focused and does not mix unrelated backend, UI, MCP, docs, dependency, and deploy changes.
This follows CONTRIBUTING.md and does not reintroduce GitHub Pages, VitePress, site/, or CNAME.
I linked an issue, or this is small enough that the summary explains why an issue is not needed.
Validation
git diff --check
npm run actionlint
npm run typecheck
npm run test:coverage locally; codecov/patch requires ≥97% coverage of the lines AND branches you changed (aim for 98%+ on your diff so CI variance does not fail near the threshold). Global coverage is a non-blocking trend with a loose 90% backstop, not the gate.
npm run test:workers
npm run build:mcp
npm run test:mcp-pack
npm run ui:openapi:check
npm run ui:lint
npm run ui:typecheck
npm run ui:build
npm audit --audit-level=moderate
New or changed behavior has unit/integration tests for new branches, fallback paths, and sanitizer boundaries
Container smoke: /health returned ok, /etc/ssl/certs/ca-certificates.crt exists, and no dist/*.map files remained after startup.
If any required check was skipped, explain why:
None.
Safety
No secrets, wallet details, hotkeys, coldkeys, user PATs, private keys, raw trust scores, private rankings, or private maintainer evidence are exposed.
Public GitHub text stays sanitized, low-noise, and does not imply compensation guarantees or optimization tactics.
Auth, cookie, CORS, GitHub App, Cloudflare, or session changes include negative-path tests.
API/OpenAPI/MCP behavior is updated and tested where needed.
UI changes use live API data or real empty/error/loading states, not production mock/demo fallbacks.
Visible UI changes include a UI Evidence section below with JPG/JPEG or PNG screenshots arranged as organized, captioned, clickable thumbnails. SVG screenshots are not used as review evidence. Review-only screenshots or recordings are not committed to the repository.
Public docs/changelogs are updated where needed; changelogs are only edited for release-prep PRs.
UI Evidence
Not applicable; runtime image only.
Notes
This keeps source maps private: they are uploaded by the release step and deleted from the running container before the service starts.
1 file · 1 AI reviewer · no blockers · readiness 93/100 · CI pending · blocked
⏸️ Held for maintainer review
Review summary
The change installs Debian's CA certificate bundle in the REES runtime image before dropping privileges, which is the right layer for runtime TLS verification during source-map uploads. The package install is scoped to the runtime stage and removes apt metadata in the same layer, so it addresses the certificate availability gap without broadening the image more than necessary. I do not see a reachable break in this diff.
Signal
Result
Evidence
Code review
✅ No blockers
1 reviewer
Linked issue
✅ No-issue rationale
PR body explains why no issue is linked.
Related work
✅ No active overlap found
No same-issue or scoped active PR overlap found.
Review load
✅ 20/20
Readiness component derived from cached public PR metadata and labels; size label size:XS.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
gittensor:bugGittensor-scored bug fix — scores a 0.05x multiplier.
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
sentry-clican verify TLS when uploading source maps from Railway deploys.unable to get local issuer certificate.Scope
type(scope): short summaryConventional Commit format, for examplefix(api): restore profile access checks.CONTRIBUTING.mdand does not reintroduce GitHub Pages, VitePress,site/, orCNAME.Validation
git diff --checknpm run actionlintnpm run typechecknpm run test:coveragelocally;codecov/patchrequires ≥97% coverage of the lines AND branches you changed (aim for 98%+ on your diff so CI variance does not fail near the threshold). Global coverage is a non-blocking trend with a loose 90% backstop, not the gate.npm run test:workersnpm run build:mcpnpm run test:mcp-packnpm run ui:openapi:checknpm run ui:lintnpm run ui:typechecknpm run ui:buildnpm audit --audit-level=moderateAdditional validation:
npm run test:cinpm run rees:testdocker build -f review-enrichment/Dockerfile review-enrichment -t gittensory-rees:ca-smoke/healthreturned ok,/etc/ssl/certs/ca-certificates.crtexists, and nodist/*.mapfiles remained after startup.If any required check was skipped, explain why:
Safety
UI Evidencesection below with JPG/JPEG or PNG screenshots arranged as organized, captioned, clickable thumbnails. SVG screenshots are not used as review evidence. Review-only screenshots or recordings are not committed to the repository.UI Evidence
Not applicable; runtime image only.
Notes