Skip to content

chore(repo): remove the dead self-host Docker path, fix CI path filters and repo scripts - #2446

Merged
Makisuo merged 19 commits into
fix/audit-frontend-fixesfrom
chore/audit-selfhost-ci
Nov 10, 2026
Merged

Makisuo merged 19 commits into
fix/audit-frontend-fixesfrom
chore/audit-selfhost-ci

Merge branch 'fix/audit-frontend-fixes' into chore/audit-selfhost-ci

30b73be
Select commit
Loading
Failed to load commit list.
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-collector compose services and both Dockerfiles deleted
  • ts filter is now a positive apps/!(ingest|ios)/** extglob with the read-but-unlisted paths added
  • cli filter lists packages/** and lib/**; shellcheck runs the shell filter's globs
  • iOS OpenAPI allowlist drops getSpan/getLog in 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 up at 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's ai_session/** and package.json, knip.json, patches/** (ci.yml:63-95)
  • bun db:up still resolves: postgres, electric and postgres-data remain 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 calls getSpan/getLog

30b73be · Reply "won't fix" to dismiss a finding, or mention @maple-review-bot to ask about one.