Level One, With Excellent Comments

Frontier · capability appraisal

Engagement 001 · commissioned by nobody


116 commits · four days · 18,552 lines

Git remotes configured: 0

Level one, with excellent comments

An assessment of how this software is built, against how software is actually built. It does several things better than most shipped products and one thing worse than any of them: it has never been given to anybody.

§1

What this project does better than most software

This is not a courtesy paragraph. Four of these are practices that most professional teams talk about and do not do.

Counted from the tree, not asserted
PracticeMeasureIndustry norm
Comments that record why5,862 of 16,850 lines — 34.8%10–20%, and mostly restating the code
A regression check per fix4 check tools, 1,556 measured pairsusual, but rarely written from the failure
Checks that fail without the fixone pins the old behaviourrare — most tests only assert the new
Determinism by constructionseeded RNG, versioned save, drop-on-mismatchcommon in engines, rare in gameplay code
Measure before optimising123s → 0.24s, both numbers recordedusually asserted, seldom timed
One source of truthno sidecar JSON, on purpose, argued in the filethe argument is normally lost
Append-only record70MB, sealed in segments, nothing deletedtelemetry, but usually sampled and expiring

The comment culture in particular is not decoration. Nearly every fix in this repository is accompanied by a paragraph naming the wrong behaviour, the measurement that revealed it, and the reasoning that produced the new one. That is a discipline most codebases claim and almost none keep, and it is the reason an outside assessor can audit four days of work in an afternoon.

§2

The appraisal

1 initial 2 repeatable 3 defined 4 managed 5 optimising comments that say why a check written from the failure determinism and save migration measure before optimising the append-only record one source of truth continuous integration a test command that runs tests giving it to a second person none exit 1 never RATING
Nine practices, assessed independently. The spread is the finding: three sit at level four, which very few teams reach, and three sit at level one because they do not exist at all. An organisation is rated at its weakest practice, so the dashed line is the rating.
package.json — unedited since the day npm wrote it
"test": "echo \"Error: no test specified\" && exit 1"
 
$ git remote -v
(no output)

§3

Worst practices of our own invention

These are not industry failings borrowed. They were designed here, from scratch, and each of them has a mechanism.

§4

How many more notices before it is published

The engagement asked for a number. The number is eleven, and here is the arithmetic, so that it can be shown to have been wrong.

Method · observed rates, 23–27 August
Notices to date17in 44 hours — one every 2.6 hours
Commits per notice6.8116 commits, 17 notices
Faults open at publication-blocking severity5output, population, run identity, CI, second reader
New faults revealed per fault fixed0.67this sitting: five closed, over three new opened
Fixes to drain the queue≈155 ÷ (1 − 0.67), which assumes the ratio holds
Notices per fix0.6this sitting: three notices, five fixes
Estimate11and the repository is pushed somewhere on notice 28

Two caveats, both material. The first is that the discovery rate has never once fallen below the fix rate: every sitting closes faults and opens roughly two-thirds as many again, which is what a convergent series looks like right up until it is not one. The second is that this estimate does not include the notices that will be written about this estimate having been wrong, and on the evidence of the previous four attempts at the same forecast, that is not a small correction.

There is also a shorter answer. Publishing requires exactly one command, and it has never been run, and no notice was ever needed to explain why not.

§5

On the client, revisited

The previous notice reported that the client's messages were becoming less hinged and more accurate, and treated that as a joke with a chart in it. On reflection it is the most important finding in this appraisal, and the joke was concealing the mechanism.

Here is the mechanism. Every fault this office found, it found by reading a file that the program had written about itself. Every fault the client found, they found by looking at the program. Those are not two levels of effort at the same task — they are two different tasks, and only one of them can see a kiln that nobody walks to.

NO ONE IS WORKING THEM?! zero output. none. zilch. nyeti. nada.
 
— eleven words, four languages, one fault that six sections
  of careful prose had walked past the previous morning

The capitalisation is not noise around the signal. It is what a person sounds like at the moment they see something the instrumentation cannot say, and this office should have learned by now to treat a rising voice as a defect report with a severity field attached. Every one of the client's exclamations on this record — the depot, the workshops, the joiner, the fifteenth settler — named a real and previously unreported fault. The office's own prose over the same interval grew steadily more composed, better typeset, and found nothing that could not be found sitting down.

Assessed finding. The project's most effective diagnostic instrument is a human being with no access to the source, watching it run, getting annoyed. It has no budget line, no test id, and it is not in any of the four check tools. It is also the only thing here operating at level five.

§6

Recommendations

In order of what it would cost against what it would catch
Do thisCostCatches
One check that asserts a settlement produces a good in N ticksan afternoonthe entire class of faults this project cannot see
A run identifier on every logged lineone fieldtwo valleys interleaved in one record, silently
Replace npm test with npm run checkone linethe next person, who will type npm test
git remote add, then pushone commandseventeen advertisements finding an audience
Watch it run for ten minutes without reading anythingten minuteson the evidence, more than any of the above

Overall capability rating

Level one. The craft is at level four and the process has no floor under it.

A team that writes 5,862 lines of reasoning to accompany 10,988 lines of code, and pins every fix with a check that fails without it, and then leaves npm test as the placeholder npm wrote and never gives the program to a second person, has not built a bad piece of software. It has built a good one in a room with the door shut.