PRs reviewed
6
in scope
Inline comments
11
across reviews
Security findings
2
flagged by AI
Gates failing
2
quality gate
Bamboo red
0
builds
Repos connected
5
GitHub + GitLab + Bitbucket
Source
feat(i18n): RTL scaffold + locale switcher persistence
Lena Schmidt · opened 2026-05-13 14:18 · 14 files · +318 −47
AI review·Adds RTL CSS direction inheritance + persists locale across configurator steps. 4 inline comments — 1 security, 2 perf, 1 test gap.Risk6/10
Quality gate· would not allow mergewarn
- linteslint clean
- tests184 / 184 green · 6m 12s
- securityXSS finding @ LocaleSwitcher.tsx:84
- coveragepatch coverage 71% (target 80%)
Bamboo#4421
acme-ccc · main · queued
security·
src/configurator/LocaleSwitcher.tsx:84Locale value from URL is concatenated into innerHTML — possible XSS. Sanitize or use textContent.
perf·
src/configurator/StepRouter.tsx:142Whole step list re-renders on every locale change. Memoize with useMemo or split components.
perf·
src/configurator/RTLProvider.tsx:31Direction lookup runs on every render. Cache via useRef.
test-gap·
src/configurator/__tests__/RTL edge case (mixed bidi text + form alignment) not covered.
fix(i18n): persist locale across step navigation
Lena Schmidt · opened 2026-05-12 11:02 · 6 files · +94 −32
AI review·Small + targeted. URL+cookie persistence + 1 reset bug fixed. 2 inline comments — both style.Risk2/10
Quality gate· would allow mergepass
- linteslint clean
- tests184 / 184 green · 5m 48s
- securityno findings
- coveragepatch coverage 91%
Bamboo#4419
acme-ccc · main · green
style·
src/configurator/useLocale.ts:17Naming: hookLocale → localeFromUrl is clearer.
maintainability·
src/configurator/useLocale.ts:5Cookie name "loc" is too generic — prefix with app key (e.g. "ccc.locale") to avoid collision.
feat(inventory): add preStop hook for graceful shutdown
Markus Bauer · opened 2026-05-13 09:14 · 3 files · +41 −0
AI review·Root cause fix for last night SEV2 — adds preStop hook + in-flight request drain. 2 inline comments — 1 security, 1 test-gap.Risk4/10
Quality gate· would not allow mergewarn
- lintgolangci-lint clean
- tests62 / 62 green · 2m 04s
- securityhardcoded drain timeout flagged
- coverageno integration test for SIGTERM path
Bamboo#812
fleet-api · main · green
security·
cmd/inventory/main.go:18Drain timeout is hardcoded to 30s. Externalize via env to avoid future drift.
test-gap·
cmd/inventory/lifecycle_test.goNo integration test that simulates K8s SIGTERM + verifies in-flight requests complete.
fix(qa): stabilise quote-builder e2e teardown
Jonas Wagner · opened 2026-05-12 16:48 · 4 files · +56 −88
AI review·Replaces shared module-level mocks with per-test fixtures. 1 comment — style.Risk2/10
Quality gate· would allow mergepass
- linteslint clean
- tests212 / 212 green · 8m 33s
- securityno findings
- coveragepatch coverage 87%
Bamboo#1249
dealer-portal · staging · green
style·
tests/e2e/quote-builder.spec.ts:24Use beforeEach for fixture creation rather than inline within each test.
feat(grafana): add SLO rule pack for fleet-api
John Becker · opened 2026-05-13 11:04 · 9 files · +184 −22
AI review·Adds 4 new Grafana SLO recording rules + alert routes for fleet-api. 2 inline comments — 1 perf, 1 style.Risk3/10
Quality gate· would allow mergepass
- lintyamllint clean
- testspromtool unit-tests green
- securityno findings
- coveragen/a (config)
Bamboo#091
infra-monitoring · main · green
perf·
monitoring/rules/fleet-api.yaml:425s evaluation interval is aggressive — consider 15s to ease load on Prometheus.
style·
monitoring/rules/fleet-api.yaml:18Alert name uses snake_case; convention in this repo is kebab-case.
feat(ci): add buildx remote cache
John Becker · opened 2026-05-11 18:30 · 2 files · +38 −12
AI review·CI optimization. Saves ~6 min off cold-cache builds. 0 inline issues — clean.Risk1/10
Quality gate· would allow mergepass
- lintshellcheck clean
- testspipeline self-test green
- securityno findings
- coveragen/a (infra)
Bamboo#207
ci-tooling · main · green