Make the README safe to share, and show the product instead of describing it - #309
Merged
Merged
Conversation
…bing it
Two problems with the README as it stood: it published operational detail that
should not be public, and it described the product without ever showing it.
REMOVED — things a public repo should not advertise:
- The "Status - honest" section named which production environment variables
were unset and spelled out the consequence ("a paid order writes no row,
sends no receipt, raises no alert"). That is a live misconfiguration map for
anyone reading. Runtime configuration health is now read from the service
itself via /api/health, which cannot go stale and is not a standing
invitation.
- `AUTH_SERVER_ROUTES=off` was documented as a rollback that disables the
server-side credential routes and "takes effect without a rebuild". Naming a
kill switch for auth hardening, with its no-restart property, is not
documentation a reader needs.
- Exact lockout and timing-defence thresholds (attempt count, lockout window,
latency floor). The control mapping stays - NIST 3.1.8 / CMMC AC.2.008 - but
the tunable numbers are gone. The code is MIT and readable; the README does
not need to hand over the dial settings.
- Business strategy that is not repository documentation: the pricing rationale
about procurement thresholds, customer counts, and the DOJ False Claims Act
enforcement angle. The forward-looking roadmap link is dropped from the
header for the same reason.
ADDED - safe, verifiable, and visual:
- A mermaid architecture diagram showing the interception path, with the
trust boundary drawn explicitly so the local-only claim is visible rather
than asserted. Renders natively on GitHub, no build step and no new asset.
- The 16 detection engines as a table grouped by what they protect, plus a
worked blocked-request example - the same curl and JSON already published in
the public API docs.
- A "Compliance context" section carrying only the standing obligations the
product maps to (DFARS 7012, the 110 NIST 800-171 Rev 2 controls, the SPRS
self-assessment, HIPAA Part 164), with the certification claims pointed at
/trust rather than restated here.
Also corrected: the API route count in the repository map read 59 against 66
actual, and the screenshots are now date-stamped 2026-08-15 because the landing
page has changed twice since they were captured. docs/assets/README.md no longer
claims production has not deployed since #288 - it has.
Gates: 2,993 passed / 212 files, exit 0. lib/detection/__tests__/doc-counts.test.ts
scans the README against the shipped registry and passes, so the engine and
pattern figures in the new table are computed truth rather than prose.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_018UjEyfCdsoUjYXo6aAaQ4D
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
thecelestialmismatch
marked this pull request as ready for review
August 19, 2026 07:48
thecelestialmismatch
added a commit
that referenced
this pull request
Aug 20, 2026
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Description
Two problems with the README as it stood: it published operational detail that should not be public, and it described the product without ever showing it.
Removed — things a public repo should not advertise
/api/health, which cannot go stale and is not a standing invitation.AUTH_SERVER_ROUTES=offAdded — safe, verifiable, and visual
/trustrather than restated.Also corrected
docs/assets/README.mdno longer claims production has not deployed since ⛔ BLOCKED ON A DASHBOARD SETTING — delete the repo-root vercel.json so middleware executes #288. It has.Type of change
Jordan's Test Plan (CMMC Buyer — required before merge)
npm run buildpasses incompliance-firewall-agent/— unchanged by this PR; no application file touchedproxy/patterns/modifiedVerification
Two markdown files changed, +109 / −44. No application code.
lib/detection/__tests__/doc-counts.test.tsscans the README against the shipped registry and passes, so the engine and pattern figures in the new table are computed truth, not prosemermaidblockResidual-check for everything this PR set out to remove — all zero:
Checklist
anytypes added in compliance-critical codeNote
The limitations that are genuinely useful to a reader are kept, not buried: Docker Hub publication is still pending, there is no SOC 2 report and no FedRAMP authorization, and the hosted trial is explicitly not a CUI or PHI environment. Honesty about what the product does not do is different from publishing its runtime configuration.
Generated by Claude Code