project-audit

Project audit — what is true of this project right now

A gate asks whether an artifact is good. Acceptance asks whether anything on the list was lost. Neither asks what the project actually is today, and that is the only question an operator has when they open a repository they have not touched for a month.

This skill answers it from a cold start — no brief, no REQ table, no module map required — and leaves two artefacts: a page a person reads, and a sidecar the next audit reads.

Contents

• What this is not
• The six phases
• Three verdicts, and why the third one exists
• The class that version checks cannot see
• The two artefacts
• What the script does and what you do
• Exit criterion
• Rationalizations

What this is not

	Owns	Reach for it when
references/audit.md	the method — the L0→L7 ladder, seams, axis rotation	one deliverable is being walked inside a run
this skill	the procedure — cold start, probes, production, the report	a whole project is the subject
/skill-audit (make-skill)	a skill's construction against the standard	the thing audited is a skill or plugin
/ux-audit (super-ux)	code against documented scenarios	the question is user-facing behaviour
/seo-aeo-audit (seo-aeo-audit)	a public surface's search and answer-engine visibility	the question is whether a machine will find it

The method is not restated here. Phase 4 below hands off to audit.md and comes back; a second copy of the ladder would be a second rule, and the two would disagree within a release.

The six phases

Run them in order. Each writes into the same payload the report is rendered from.

1. Discover — what this project IS, before anything is measured

scripts/audit.py answers this mechanically: languages, package managers, monorepo shape, submodules, CI, deploy targets, declared telemetry, and the version the project states about itself. It reads committed state — git ls-files, never a directory walk — because a walk finds node_modules and build residue, and reports a project no clone would produce.

The output is a profile, and the profile chooses the probes. Skipping this and running a fixed checklist is how an audit produces the same nine findings on every repository it is ever pointed at.

2. Probe — the registry, chosen from the profile

Every probe declares what it needs (git, gh, npm, network, a connected MCP server). A need that is not met makes the probe blind, with the reason — never absent, and never clean. The catalogue by stack, and what to add for a project shape the script does not cover, is references/probes.md.

3. Prod — the evidence that only exists outside the tree

This is the phase most audits skip, and the one that finds what the repository cannot admit about itself:

• the published artefact against the source — see the next section;
• CI history: not "is it green" but what share of release runs failed, and whether a failure was noticed. gh run list --limit 60 --json conclusion,name is one command and it is the closest thing to a production log a package has;
• telemetry, present or absent. Absent is a finding only when something is deployed; for a library it is a design. Either way the report says which, because "no Sentry configured" and "no errors" must not render the same;
• adoption, where a registry serves it — downloads make "is this in prod" a measurement rather than an opinion;
• connected MCP servers — error trackers, analytics, hosting, databases. Ask what the project uses, use what is connected, and record the rest as blind. Read aggregates and pointers, never raw bodies (see The two artefacts).

Before ANY datastore read, in this order — a figure from the wrong database is worse than no figure, because it is quoted with the authority of a measurement:

1. resolve the datastore from the platform's own attachment, not from a script in the repository and not from a name in the documentation;
2. prove liveness with a freshest-write aggregate against a table the product writes constantly, and put that timestamp in the report beside every figure it underwrites;
3. where more than one datastore is attached, name them all with their freshness — an audit that reads one and does not mention the other cannot be reproduced by its reader;
4. the repository's connect helper is evidence about the repository, never about the platform. Where the two disagree that is a finding of its own, and it is the one this rule was written from.

4. Seams — hand off to the ladder

Now, and only now, walk audit.md's ladder over the capabilities the discovery found. Bottom-up, seam-ordered. Absences found here are findings like any other.

5. Report — two files, one command

python3 scripts/audit.py --root <path> writes the sidecar and prints a summary. Add --report for the HTML page, and --no-open beside it to write without opening. It is the same script the probes live in, so what the report claims and what ran are the same object.

6. Propose — rows, not edits

This skill commits nothing. Findings leave as board rows in the project's own vocabulary, priced with the board header's declared formula — the shipped default is Sev × Blast + age_bonus (references/backlog.md, the pipeline's board doctrine) — and the operator accepts them. Effort never ranks inside an audit: what a fix costs is the fixer's decision, not the finder's (references/prioritisation.md). An audit that edits while it reads cannot be re-run to check itself.

A finding keeps its axes apart, or it is a guess wearing a verdict

Mechanism status, exploit/reproduction, exposure, observed incidence and impact uncertainty are FIVE axes, not one. A row written from the mechanism alone is indistinguishable from a real finding until each axis says what it knows — and one audit had eight consecutive rows rewritten by measurement, two of which would have destroyed inventory if remedied as written, because both read an absence of sales as an absence of demand. But the correction cuts both ways:

• A proven defect may have incidence UNKNOWN. An auth bypass or a race reproduced locally is a code defect BEFORE any incident — the reproduction is the proof, and no production log is needed to license the row. A confirmed local crash with no production logs stays a code defect, external incidence UNKNOWN.
• UNKNOWN ≠ 0. A zero sample is not zero risk; absent telemetry lowers what can be said about EXPOSURE, never the technical truth — and the row says which.
• Unknown attacker control lowers exploitability CONFIDENCE, never the observed behaviour: what was watched happening stays written as watched.
• A documented exception does not turn a failed invariant into PASS — decision status and technical validity are the separate axes of the section below.

The incidence axis is still asked: how often does the mechanism fire? A query, a log count, a telemetry read. Where the answer is never, keep the row priced as latent (the remedy weighed against zero); where the measurement is impossible, the INCIDENCE axis records blind — and the finding stands on its mechanism and reproduction axes. Each row records its observation scope and time, keeping code mechanism, deployment observation and assumptions apart — templates/finding-evidence.json in task-pipeline is the minimal schema, and the collector writes mechanism/incidence/observed_scope/observed_at into the sidecar. Ask an operator only when the unknown would change the action — an interview is not a prerequisite for a row.

Two rules follow from the same place:

• Check a remedy against the population it would touch before proposing it. Two rows prescribed releasing an asset class; the measurement showed the asset class was the product's own inventory, deliberately held.
• An absence is not evidence until the path that would produce the presence has been walked. Zero sales means no demand or no working path, and those two want opposite remedies.

Already decided is not a finding, and it is not nothing either

An audit that reads a module and not the places the module is used reports, as defects, the things the project has already decided — in the project's own words, because the decision is usually written a few lines from the code the finding cites. Five of eight rows in one run were that.

• Read the call site, not only the definition. A finding about a module is not written until the places that use it have been read.
• Every row states which of three it is: (a) undecided, (b) decided and documented right here, (c) decided elsewhere and not propagated. (a) and (c) are work — and so is a (b) that is still WRONG, because decision status and technical validity are DIFFERENT AXES. A documented decision can be mistaken, stale, or break an external contract; "it's by design" is not a proof of correctness. So a (b) splits:
  • accepted trade-off — a conscious limitation with a real, named cost the project chose to pay (single-browser support under a matching contract): mark it accepted limitation with its cost, not a defect.
  • documented violation — a decision that STILL breaks a security invariant, an external contract, or its own stated goal (an ADR that permits logging a refresh token): this REMAINS a finding, carrying the decision_id, the reason to revisit, and the counter-evidence. Recording a (b) the audit got wrong (a false positive) is worth more than deleting the row; but excluding every documented decision from findings is a systemic source of false NEGATIVES, which is the more expensive miss.
• Where the verdict is (c), the remedy is a mechanical check, not an edit. A written rule nobody verifies reaches exactly as far as the place it was written; the durable fix in all five cases was a guard that asks the project's own instruction of every copy, not a patch to the copy that happened to be found.

Three verdicts, and why the third one exists

clean · finding · blind. The vocabulary is closed; a fourth value is refused at construction.

blind is the whole design. Without it, a probe that could not look and a probe that found nothing produce the same empty section, and a reader takes the second meaning every time. This is audit.md's silence is not a reading raised from a command to a probe: a zero exit with no output has not answered.

The page renders the blind list as a section of its own, never an appendix. An audit's blind spots are part of its result.

The class that version checks cannot see

One version string, more than one tree. A package's channels do not all serve the same thing: a registry serves the tag, while a plugin marketplace and a skills CLI serve the branch tip. When the branch has moved past the tag without a version bump, every channel answers the same number and ships different code.

Measured in this family on 2026-08-22: npm served one file at 4344 lines while the marketplace served it at 4575, and all three channels reported 1.15.0. The pin checker was green throughout — correctly, because it compared the two strings.

So compare trees, never labels, and compare the pair that can actually disagree. The first draft of this probe compared the registry tarball against the tag and reported clean: those agree by construction, because the registry publishes from the tag. A tautology returning green is the false success shape — a mechanism trusted by its own reply.

Two more traps, both of which shipped in the first draft and are now fixtures:

• A path in one channel and not the other is packaging, not divergence. A tarball ships what its files allowlist permits; counting .github/ as a disagreement produced 22 findings where one file had moved.
• Compare only when both sides claim the same version. A branch already bumped past its tag makes no common claim, and is blind, not clean.

The artefacts — the sidecar always, the page on request

docs/audit/<date>-audit.json is written on every run. The HTML page is written only with --report. Three output modes, and only one needs a browser: stdout (--stdout — the summary in the terminal, nothing written but the sidecar), json (the sidecar alone, the default — a machine reads it, no page and no browser), html (--report — the page too). A json-only or stdout run never opens a browser and never requires one; the browser is a concern of --report alone.

The split is not symmetry. The sidecar is what makes this a ratchet rather than a snapshot, and the next run reads it — skipping it would silently turn every future run into a first run. The page is a report, and a report is an artefact that outlives the conversation: one nobody asked for is a document nobody ordered and nobody maintains, sitting untracked under docs/ one git add -A from the product's history. Answer in the conversation; write the page when somebody wants a page.

The page carries aggregates and pointers, never raw bodies. Counts, top classes, trends, and a link to the issue in its own system — never a stack trace, a log line or a row of data. The report is a file people forward, and a report that cannot be shared is one nobody writes twice.

A secret is reported by place and class, never by value. file:line, which credential it is, and the remedy. The value appears in neither artefact nor on stdout: an audit must not become the second place a credential leaks. Redaction is total rather than a prefix — half a credential plus its context is often enough to finish.

"Read-only" is about the TARGET, not the disk. The audit reads the project's source, data and production evidence and mutates NONE of it, and it commits nothing. It DOES write its own artefacts into its allowed output directory (docs/audit/ by default, --out-dir to relocate) — the sidecar always, the page on --report. Writing the sidecar there is not a violation of read-only; writing into the target's source, or git add-ing anything, is.

The sidecar is what makes this a ratchet rather than a snapshot. Each finding carries an id derived from its probe and its place, so it survives a rewording; the next run prints what closed, what is new, and what has now survived three audits. That last number is itself a finding: a defect nobody picks up is a decision nobody wrote down.

What the script does and what you do

The script	You
discovery, the registry, mechanical probes, the sidecar, the page on --report, the diff	the seam walk, MCP evidence, judgement about what a finding means
refuses a fourth verdict, redacts secrets, excludes its own output	deciding severity and effort, writing the remedy that fits this project

The split is not tidiness. A judgement encoded in a script becomes a gate nobody agreed to; a mechanic left in prose becomes a step nobody runs. Both have shipped in this family and both are on its boards.

Run the script first, read its blind list, then spend your reading where it could not look.

Exit criterion

An audit is finished when — the criterion is CONDITIONAL on the deliverables requested, not a fixed page-and-browser:

1. every probe has a verdict, and every blind one names why;
2. the sidecar is written (always). On --report, and only then, the HTML page also exists, its internal links are safe and resolve, and its inspect/render status is recorded (opened, or --no-open noted) — WITHOUT --report no page is created and the run is still complete;
3. every number in the report was produced by a command this run executed — a restated count is an assertion (evidence-docs);
4. at least one figure was re-derived by a differently-shaped command and both were printed. Re-running the same command is a spell-check of the first run;
5. the proposed rows are printed for the operator, with nothing written to the target.

Rationalizations

Excuse	Reality
"The tests pass, so the project is healthy"	Tests compare what somebody thought to write down. This phase 3 exists because the sharpest defect in this family was invisible to a green suite in every repository it touched.
"There's no Sentry, so there are no errors to report"	Those are the same empty section and opposite facts. That is what blind is for.
"The versions match, so the channels agree"	Measured: three channels, one version string, 231 lines of difference. Compare trees.
"I'll note the raw log lines so we have context"	Then the report cannot be shared, and it will not be written again. Aggregate and point.
"I found a credential — I'll paste it so we can check it's live"	The report becomes the second leak. Place and class; rotation is the remedy.
"Findings should just be fixed while I'm in there"	An audit that edits while it reads cannot re-run to check itself, and the next run cannot tell a fix from a rewording.
"It's the same nine checks every time, so I can skip discovery"	Then the instrument returns the same answer for every input, which is a fact about the instrument and not about the project.
"No findings — clean bill"	Only against the probes that ran. Read the blind list before saying that sentence out loud.