Skip to content

Add protected read-only installation inventory for internal admin - #397

Merged
neonwatty merged 2 commits into
mainfrom
codex/legacy-admin-usage-read
Sep 19, 2026
Merged

neonwatty merged 2 commits into
mainfrom
codex/legacy-admin-usage-read

Conversation

@neonwatty

Copy link
Copy Markdown
Collaborator

Summary

  • Add a server-to-server, read-only paginated inventory of legacy installations and nullable best-effort successful-feedback counts.
  • Require a dedicated bearer credential before any KV or Durable Object read; return no-store responses and no browser CORS access.
  • Check identity and deletion guards, keep cursors out of URL logs, and fail closed on malformed records or unavailable storage.
  • Preserve the historical public-plane fingerprint as V1 and add an explicitly reviewed V2 baseline for this intentional public Worker change.

Verification

  • Independent read-only review found no confirmed blocking code defect.
  • npm test: 2,525 passing tests across 151 files.
  • npm run typecheck, npm run lint, targeted Prettier check, npm run verify:legacy-compat, and git diff checks passed.
  • Adversarial tests cover auth-before-storage, missing versus zero counts, opaque pagination, deletion between list and read, empty continuation pages, Durable Object failures, and cursor logging.

Rollout gates

  • Informational best-effort counts only: no measured freshness age or guaranteed immediate disappearance after uninstall; not a billing meter or authoritative active-installation registry.
  • No deployment or secret provisioning is included. Provision a high-entropy dedicated secret on both servers only after review, with a rotation plan.
  • Qualify a nonproduction smoke path first. Preview currently has installation usage collection disabled, so the inventory route returns unavailable there.
  • The bugdrop.dev staff sign-in, role checks, access audit, and dashboard are separate follow-up work before administrator use.

@neonwatty
neonwatty marked this pull request as ready for review September 19, 2026 21:39
@neonwatty
neonwatty added this pull request to the merge queue Sep 19, 2026
Merged via the queue into main with commit 8c0d2b3 Sep 19, 2026
11 checks passed
@neonwatty
neonwatty deleted the codex/legacy-admin-usage-read branch September 19, 2026 22:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant