Repository navigation
chore(repo): remove the dead self-host Docker path, fix CI path filters and repo scripts - #2446
Merged
Merged
Maple Review Bot / Maple / review
succeeded
Nov 10, 2026 in 1m 35s
Confidence 9/10 · No issues found
🟢 Confidence 9/10 · safe to merge. CI-config and infra deletions only, no runtime path changed; the enlarged shellcheck globs could not be run in the sandbox.
no findings · tests partial −1 · risk medium
Removes the non-functional self-host Docker Compose path, turns the CI ts filter into a positive extglob covering the paths its jobs read, widens the cli and shell filters, and trims the iOS OpenAPI allowlist. No runtime code changes; safe to merge.
- Self-host
api/web/ingest/alerting/scraper/otel-collectorcompose services and both Dockerfiles deleted tsfilter is now a positiveapps/!(ingest|ios)/**extglob with the read-but-unlisted paths addedclifilter listspackages/**andlib/**; shellcheck runs theshellfilter's globs- iOS OpenAPI allowlist drops
getSpan/getLogin the spec and the domain test
Before merge
- Infra · Check every environment has what this config now expects ·
docker-compose.yml - Manual · Confirm no self-hoster still runs
docker compose upat the repo root: the API, web, ingest and scraper services are gone.
Production impact
Production traffic of the changed files (last 7 days)
| File | Calls/day | Busiest operations |
|---|---|---|
scripts/install.sh |
2.5k | maple 2.5k/day, 2.0% err |
scripts/uninstall.sh |
2.5k | maple 2.5k/day, 2.0% err |
What was checked
- New filters cover every input the TS jobs read:
apps/*/alchemy.run.ts, ingest'sai_session/**andpackage.json,knip.json,patches/**(ci.yml:63-95) bun db:upstill resolves:postgres,electricandpostgres-dataremain in docker-compose.yml:5-44- No references left to the deleted Dockerfiles,
docker:up/docker:build,nginx.conf, or the two removed scripts; no Swift code callsgetSpan/getLog
30b73be · Reply "won't fix" to dismiss a finding, or mention @maple-review-bot to ask about one.
Loading