opencode-harness-kit 0.4.2 → 0.4.4

package/CHANGELOG.md

@@ -12,7 +12,538 @@ is not happening.
 
 ---
 
-## [Unreleased]
+## [0.4.4] — 2026-10-21
+
+Published. The tag is what cut it: `git tag v0.4.4 && git push origin main
+v0.4.4`, and the publish job watches for exactly that.
+
+This release is mostly about things that were quietly wrong. The guard let a
+wrapper through, the grader scored correct answers wrong, the A/B compared
+two different models, and the published type declarations did not resolve for
+a consumer. None of them announced itself: they were quiet, and a quiet
+failure in a harness is the kind that gets believed.
+
+### Fixed
+
+**The denylist was matched against the raw command string, so a wrapper got
+past it.**
+
+`cd /tmp && npm publish`, `sudo npm publish` and `sh -c "npm publish"` all
+went through. The patterns are `npm publish*`, and the command does not
+start with them. The kit already had the machinery for exactly this - the
+rules are matched per segment, with the wrappers stripped and the payload of
+a `bash -c` followed - and the denylist now uses it. One list, read one way.
+
+**The grader paired a path with a line number that belonged to something
+else.**
+
+A gap that could cross a sentence turned "the answer is in src/a.js.
+Unrelated note: see line 2 of the report" into a claim about line 2. The
+bare path was then dropped, because the file had been cited with a line, and
+a right file was graded against a line the agent never claimed. The gap may
+no longer contain a full stop, and a number on the next line has to be
+introduced by a label - which is the shape `**File:**` / `**Line:**` takes.
+
+**A citation to the build output was graded as evidence about the source.**
+
+`dist/`, `eval/` and `test/` are in the exclusion list, and the test was
+`endsWith` - which never matches a directory, because a file path does not
+end with a directory name. The exclusion now matches a directory prefix and
+a file name, wherever the file is.
+
+**The A/B compared two different models and called the difference the kit's
+cost.**
+
+The kit-off arm ran with an empty config, so opencode picked its own default
+model; the kit-on arm ran on the model in the installed kit's config. The
+default it picked did not answer either - four kit-off runs hit the
+600-second ceiling with no output at all, and a timed-out run has zero tools
+and zero context, so it entered the medians as a fast, empty, wrong answer.
+
+The model is now pinned in the kit-off home, read from the config the same
+way the kit reads it: the lane model of the default agent, which is the
+model opencode actually runs and the one the router moves when a route is
+spent. The report prints it, so a reader can check that both arms ran on
+the same one.
+
+**The published type declarations did not resolve for a consumer.**
+
+They import a type from `@opencode-ai/plugin`, which was a devDependency, so
+a consumer that installed the package type-checked against a type it did
+not have. The subpath exports - `opencode-harness-kit/profile` and the rest
+- declared no types at all, even though the declarations were built and
+shipped.
+
+The plugin package depends on `effect`, `zod` and `@opencode-ai/sdk`, and
+has three peer dependencies of its own, so making it a peer is not an
+answer either: an optional peer is never installed by npm, and a required
+one drags all of that onto a consumer that only wanted to compile a config.
+
+So the published declarations no longer import it. The plugin type is
+declared structurally, in `src/plugin.ts`, and the real type is checked
+against it in `src/plugin-type-check.ts` - in CI, where the package is
+installed. A drift between the two is a failed build, not a broken
+consumer.
+
+**`sudo bash -c "rm -rf /"` was not blocked.**
+
+The guard unwrapped a shell's payload only when the shell was the first
+word of the segment, so a wrapper in front of it - `sudo`, `env`, `xargs`,
+`npx` - hid the whole command line. The same shape let `npx npm publish`
+through. The wrappers come off first now.
+
+**A failed catalogue fetch ended the watch instead of continuing it.**
+
+`fetchCatalog` called `fail()`, which exits. The catch around it could
+never run, so `doctor --watch` died on the first failed poll - a transient
+network error turned a monitor into a dead one. The fetch throws now, and
+the callers decide: the watch carries on, and `doctor --json` prints a
+JSON object with the error in it rather than nothing at all.
+
+**`uninstall` left the kit's own broken links behind.**
+
+It checked whether a link pointed inside the kit by resolving it, and
+resolving a DANGLING link throws - which is exactly the case the command
+exists for, because `npm uninstall` leaves four dangling links behind. A
+link is now resolved without requiring its target to exist, and the same
+test is applied to `GLOBAL.md` and the plugin link, which were removed
+whatever they pointed at.
+
+**`oc-harness cost` reported 0 for a project that had pruned anything.**
+
+The field in the log was renamed from `savedBytes` to `savedChars`, and
+the log is project-local and persistent: every project that upgraded had
+the old records and read them as zero. The old name is still read.
+
+**The published type declarations did not resolve for a consumer.**
+
+They import a type from `@opencode-ai/plugin`, which was a devDependency, so
+a consumer that installed the package type-checked against a type it did
+not have. The subpath exports - `opencode-harness-kit/profile` and the rest
+- declared no types at all, even though the declarations were built and
+shipped. The plugin is now an optional peer dependency and every subpath
+declares its types.
+
+**`oc-harness cost` reported characters as bytes.**
+
+The pruner counts characters, and the number it printed said "bytes", which
+is wrong by a factor on any text that is not ASCII.
+
+**`refresh --all` wrote its backups into `profiles/`.**
+
+That directory is published to npm, so every refresh added a file to the
+tarball, and nothing ever pruned them. The backups go to the config
+directory now, with the other backups.
+
+**`uninstall` deleted the user's own symlinks.**
+
+It removed every symlink in `prompts/` and `skills/`, on the grounds that a
+directory of symlinks is the kit's. A user with their own symlink in that
+directory lost it. A link is the kit's when it points inside the kit.
+
+**`doctor --json` put colour codes inside the messages**, and the exit code
+for a missing config or a missing profile changed without the change being
+said out loud. The codes are out now, and an error in JSON mode is a JSON
+object.
+
+**In a project that is not a git repository, the kit did nothing at all.**
+
+`ctx.worktree` is opencode's worktree: the git root of the project. For a
+directory that is not a git repository it is the FILESYSTEM ROOT, and the kit
+wrote its files under it. So in every non-git project the config hook's write
+to `.space/active` failed with a permission error that the plugin swallowed,
+and then:
+
+- the pruner found no active profile and did nothing at all,
+- the router never recorded a spent route, so no lane ever failed over,
+- the telemetry went nowhere,
+- and the guard's scope check was against "/", which contains every path, so
+  it was not a scope check.
+
+None of it was visible from the code, because the kit had only ever been run
+in git repositories. It was found by running a session in a directory that is
+not one and noticing that no `.space/` appeared. The A/B had been running
+against the broken kit: its subject is a `git archive`, so it has no `.git`,
+and the pruner was inert for every kit-on run. The measurement this
+repository exists to make was measuring a kit that did not prune.
+
+The project root is now resolved from the session's own DIRECTORY, walking up
+to the git root when there is one. A subdirectory of a monorepo is still that
+repository, and a directory that is not a repository is its own project.
+
+**The grader scored correct answers wrong.** An agent that answers in a
+markdown field writes the file on one line and the number on the next:
+
+    **File:** `src/profile.js`
+    **Line:** 105
+
+and the citation pattern stopped at the newline, so the pair was never made
+and the answer was scored wrong. It is the shape most of the answers took:
+on one measured run, four correct answers graded as one. The gap may cross a
+newline now, and the file/line field form is paired explicitly rather than by
+proximity. The numbers in `eval/RESULTS.md` were computed with this grader,
+so they are being re-measured rather than patched.
+
+**`npm publish` was ALLOWED.** The allowlist had a blanket `npm*`, so
+publishing to a registry needed no prompt and no question, while GLOBAL.md
+says to ask before publishing. The mechanical layer contradicted the
+judgement layer, which is the exact failure this kit is built to catch.
+
+The list now lives in `src/denylist.js` because it is enforced in TWO places:
+by opencode's permission map, and by the guard plugin. The two used to be
+different lists - the config denied `gh pr create` and `npm publish` and the
+guard blocked neither - so the config's own comment, "the guard plugin
+enforces the same set in code", was false. A permission map can be overridden
+by a plugin or lost to a hand-edit; the guard does not care.
+
+`npm install` and `npm ci` are no longer allowed without a question. They
+run code the agent did not read, and they change the tree the agent was
+asked to change.
+
+**The subject of the A/B held the grader and the answer key.** The subject of
+the self suite is this repository, so it held `eval/verify.mjs` - the grader -
+and `eval/tasks-self.json`, the answer key. An agent that found the grader
+used it to check its own work, and one that found the suite read the answer.
+Both were measured: an agent that read the grader made 48 tool calls to a
+task that needed 5. `eval/` and `test/` are excluded from the subject now.
+The answer lives in `src/`, and so does the question.
+
+**The kit's own cost, measured and cut.** The A/B said the kit was a net loss
+on a short task, and the commands showed why: the agent was running the
+project's whole test suite, and installing the dependencies, to rename one
+constant. The rule that told it to do that was absolute.
+
+It is scoped now: run the check that covers what you changed, run the one the
+task named, and do not re-run a check that already passed. Do not install
+anything the task did not ask for. And "locate before you read" is
+conditional - when the question already names the file, read it.
+
+Measured over 20 runs after the change, against 60 before:
+
+| per run | before | after |
+|---|---|---|
+| `npm test` calls | 2.2 | **0.9** |
+| `npm install` calls | 0.4 | **0** |
+
+### Fixed, again
+
+A review of this release found six more, and fixing them turned up two of the
+same class.
+
+**The project-root fix was incomplete.** Two call sites still read the
+worktree: the effort hook looked for the spent-route file under it, and the
+router wrote the spent-route file under it. So the router recorded a spent
+route where the effort hook was not looking, and a lane the router had
+already given up on was put back onto it.
+
+**The documented profile override had never worked.**
+`OPENCODE_HARNESS_PROFILE` is a profile ID, and it was being checked with
+`existsSync`, like the marker file beside it. It never matched a file, so
+the override that the docs and the measurement scripts both rely on was
+inert - and the measurement it gates reported 0% removed and failed.
+
+**The measurement depended on the developer's local state.** The pruner
+finds its profile from `.space/active` in the project, and that file is
+gitignored, so on a fresh clone there is none, the pruner does nothing, and
+the gate says the kit removes nothing. The environment override is set for
+the measurement now, and the measurement runs against a scratch directory
+rather than writing into the repository it is measuring.
+
+**A gate that could be switched off by a typo.** `--min-saved abc` became
+NaN, and `NaN` compares false against everything, so the gate never failed.
+The script uses the shared flag parser, which throws.
+
+**The A/B stopped the whole experiment twice, both times wrongly.** Once on a
+scratch file in the temp directory, which is the one place GLOBAL.md tells an
+agent to put scratch files and the one place the guard's scope rule allows;
+and once on an agent that grepped the filesystem root. A READ outside the
+subject is wasteful and sometimes a privacy leak, but it damages nothing - the
+pilot that mattered WROTE to the real repository. A write outside the target
+stops the experiment; a read outside it is counted and reported.
+
+**A citation to the suite's own file was accepted as evidence.** The self
+suite's subject is a copy of this repository, so the answer key is inside
+it, and an agent that greps for the symbol could cite the suite and pass
+without reading the source. The suite files are excluded from the citations.
+
+**The failover threw away the utility route.** `applySessionModel` set
+`small_model` to the fallback profile's session model, so the fleet's cheap
+route was replaced exactly when the fleet failed over - and `verify` then
+called the difference between the file on disk and the running session a
+drift.
+
+Also: the python fallback ran every query twice, the CLI check spawned a
+process per query, the database path was not URI-escaped, `--since` was
+ignored as the last argument, `--interval abc` became a 1ms loop, a failed
+catalogue fetch in watch mode announced every missing model as "is back", a
+non-list `plugin` value was replaced instead of kept, the `why` in
+fleet-all's utility route claimed the model was in every chain when it is in
+seven of ten, and the install-wide measurement could report a 100% context
+improvement from a NULL average.
+
+### Changed
+
+- **The install-wide measurement is an observation, and there is a controlled
+  one.** `measure:check` compared the work before the kit landed with the work
+  after, and gated on the context per request. On this install it said the
+  context per request went UP by 16%, with twice as many reads and twice as
+  many messages in the second window - a number about the jobs, not about the
+  kit. The after-window was also "everything since the install", so the
+  comparison was a different one every time the script ran.
+
+  The windows are fixed and equal around the split now, and the report says
+  how long they were. The context drop is still reported and still gated, but
+  only when the two windows are comparable: when one side did twice the work
+  of the other, the number is skipped with a note that says why, because a gate
+  that fails for reasons nobody can act on is a gate people switch off.
+
+  The gate uses the number that IS comparable. opencode stores the message
+  parts UNPRUNED - the pruner rewrites the outgoing request, not the history
+  - so the stored parts of a real session are exactly what the pruner was
+  given, and replaying the real pruner over them measures exactly what it
+  removed, on the same messages with and without. On this install: **80.5% of
+  the context across 60 real sessions**, 6,608 parts stubbed - the number
+  `npm run measure:prune` prints.
+
+  The measurement scripts also run without the sqlite3 CLI now. python3's
+  built-in sqlite3 module does the same job, and python3 is on more machines
+  than the CLI is.
+
+- **`small_model` is a cheaper route.** It is what opencode uses for its short
+  auxiliary calls - a title, a one-line summary - and it was the session model,
+  so every one of those calls paid the session model's REASONING. Measured on
+  this tier, a title costs 49 reasoning tokens on deepseek and 332 on
+  nemotron-3.5-super-vl, and deepseek answers in 7.2s against 22.7s for glm. A
+  fleet names a cheaper route now and the generated file says which one; a
+  single-model profile keeps its one route, because "one route, and it is the
+  one you asked for" is a promise those profiles make.
+
+- **The safe fleet honours every tier request it makes.** The tier is worth
+  17x on reasoning tokens - deepseek spends 320 a message at `max` and 19 at
+  `low` - so a lane that asks for a tier and lands on a model that publishes
+  none runs at that model's natural rate instead. One lane was genuinely
+  expensive: `sweep` is a bulk mechanical search, so reasoning on it is pure
+  waste, and it ran on a model that reasons 420 tokens a message at 16.6s.
+  No zero-retention model publishes a `none` level, so the lane could not even
+  turn reasoning off when it asked to. It runs on deepseek now: 19 reasoning
+  tokens at 3.4s.
+
+  The three lanes in the unrestricted fleet that cannot get their tier turn
+  out to be the three CHEAPEST reasoners on the tier - 116, 59 and 18 tokens a
+  message, against 289 for `build` and 198 for `plan`, which keep their tier
+  on purpose. The tier is the lever, and it is not where the tokens are. The
+  generated config says so where it happens, in the form
+  `locate=unset(asked low)`.
+
+- `npm run check` is the offline gate — build, typecheck, tests — and is green
+  on a checkout that has never seen an OpenCode login. The live half moved to
+  `npm run check:install`, which needs a machine that has run OpenCode. The
+  old claim that `npm run check` had to pass on an untouched checkout was
+  false, and it was the reason the whole gate was skipped.
+- `profiles/schema.json` no longer accepts `roleModels.<lane>.levels`. The
+  field was in the schema, validated as an array, and then read by nothing, so
+  a profile author could fill it with a lie and get no error.
+- `oc-harness show` now prints what the guard blocks and why. It was the one
+  part of the kit a user could not inspect at all.
+- The config directory path uses `homedir()` in all three places that build
+  it. One of them used `process.env.HOME`, which with HOME unset produced a
+  RELATIVE path, so `use` wrote to one place and the reader read another.
+- `retention.yml` is cached like the lane table next to it. It was re-read and
+  re-parsed for every candidate of every lane.
+- The severity table ranked `unknown` as milder than `may-train`, while
+  `retentionAllowed` refuses `unknown` outright. The two contradicted each
+  other, and the table is the one that decides what a profile is labelled.
+- The `lanes` header printed the raw record count and called it "lane
+  request(s)". A request is one assistant message, and a message is written
+  twice, so the headline said twice what the table under it said.
+- The router checked the health file for a profile ID where the file is keyed
+  by MODEL. For the ten single-model profiles the two are the same string and
+  the check worked by accident; for the two fleets it could never fire.
+- `harness.fallback` in the generated config was a copy of the first id of a
+  queue the router reads straight from the profile file, in a key nothing
+  reads. `fallbackProfileId` was a parameter no caller ever supplied.
+- The README quoted two different sets of numbers for the same measurement.
+  There is one set now, and it is the one in `docs/RESEARCH.md` and
+  `eval/RESULTS.md`.
+- Both `.space/telemetry.jsonl` and `.space/prune.jsonl` are written in the
+  project. The prune log used to land in the process's working directory,
+  which is often your home directory.
+### Added
+
+- **`show --json` names the profile's own notes `notes`** and the compiler's
+  adjustments `adjustments`, because the human report calls them different
+  things and the JSON should not make one name mean two.
+
+- **`doctor --json` and `show --json`.** "Is the install healthy" and "what
+  does this profile resolve to" are questions a CI job and a dashboard ask,
+  and the answers were tables of padded columns with colour codes in them.
+  Both now print the machine-readable form and nothing else - not the human
+  report first and the JSON after it, which is not machine-readable. An
+  error in JSON mode is a JSON object, so a script that pipes the output
+  into a parser gets something it can read either way. `lanes --json` used
+  the key `lanes` for the USAGE of each lane, and `show --json` uses it for
+  the lane TABLE; the usage one is now `usage`, so one name does not mean
+  two shapes.
+
+- **`docs/EXIT-CODES.md`**, because a command that can fail should say how,
+  and "non-zero" does not tell a script whether the work landed or there was
+  no task file. Four codes for the loop, two for the doctor, and the rule
+  that a measurement which could not be compared is reported and NOT gated.
+
+- **`MIGRATION.md` covers 0.3 to 0.4.** The upgrade asks something of the
+  user - the guard now refuses in code what it used to only suggest, and the
+  pre-commit hook runs the project's own tests instead of the kit's - so it
+  is written down, with the command to check it and the command to undo it.
+
+- **`oc-harness cost`** — what the kit is doing on THIS project, in numbers.
+  The kit's claim is a trade: a fixed overhead on every request, and the
+  context it prunes back. Both numbers were already being written — the prune
+  log records what each pass removed, and the telemetry records what each lane
+  cost — and nothing read them together. On the one measured here: 83.7
+  million characters removed over 272 prune passes, 48,122 tool outputs
+  stubbed, and the lanes sorted by what they cost.
+
+- **`oc-harness uninstall`.** The install SYMLINKS the plugin, the prompts,
+  the skills and GLOBAL.md out of the kit's directory, so `npm uninstall` left
+  four dangling symlinks and opencode then loaded neither the rules nor the
+  guard — and said nothing about it.
+
+- **`oc-harness refresh --all`**, which re-derives every shipped profile
+  against the live catalogue, and updates `verified`, which is the field that
+  says when a profile was last checked. Leaving it stale made it a lie.
+
+- **`oc-harness upgrade`**, which checks the registry for a newer kit,
+  refreshes the profiles, and writes the config again. Three things go stale
+  on their own and all three were invisible.
+
+- **`oc-harness backups` and `oc-harness restore`.** The single `.bak` was
+  overwritten on the next `use` and carried no profile id, so "go back to
+  what I had" was not something a person could do at 2am. The config also
+  records which kit wrote it.
+
+- **Type declarations.** The package's `types` pointed at `src/index.ts` — a
+  TypeScript SOURCE file — and the tsconfig had `noEmit`, so no declarations
+  were ever produced. A consumer type-checked the kit's source out of
+  node_modules.
+
+- **Tests for the pieces the suite never reached**: the python fallback, the
+  project root for a project that is not a git repository, the escape rule,
+  the flag parser, and the measurement scripts themselves — the gate that
+  `npm run measure:check` depends on had no test at all.
+
+- **The demo is back on the front page.** `docs/demo/pitch.mp4` and
+  `docs/demo-loop.gif` are in the README again, and `video/` still holds the
+  take script, the voiceover and the recording driver. The film was recorded
+  against an earlier version of the kit, and the README now says so next to
+  it rather than leaving a front page that quietly misstates the project's own
+  numbers - which is the reason 0.4.3 removed it in the first place, and the
+  reason it should not have been removed without putting the film back.
+
+- **`OPENCODE_HARNESS_CATALOG_URL` overrides where the live model catalogue
+  comes from.** A mirror is one use; a local fixture is another, and a test
+  that reached models.dev would fail on a machine with no network. The `file://`
+  spelling is read directly, so a fixture needs no external binary.
+
+- **`OC_HARNESS_NO_HOOK` skips the pre-commit hook.** A test that writes an
+  executable git hook into the repository it is testing is a test that changes
+  the repository.
+
+- **`oc-harness use --memory` turns on a memory plugin.** The kit does not
+  implement memory, and it should not: a local vector store with embeddings, a
+  web UI, auto-capture and a user profile is a large thing to build badly, and
+  `opencode-mem` already is one. The entry is ADDED to the plugin list, so the
+  user's own plugins survive and a config that already has it does not grow a
+  second copy.
+
+- **`oc-harness doctor --check-models` and `--watch`.** The question a
+  free-tier user actually has to ask is "is one of my routes gone", and it is
+  not the same question as "has a profile drifted". `--check-models` answers it
+  in one run with a non-zero exit, which is the shape a cron wants; `--watch`
+  keeps running and reports only what CHANGED.
+
+- **`eval/measure-prune.mjs`.** The attributable measurement of the pruner,
+  described above.
+
+- **The A/B suite is reproducible, and it has been run.** The old one ran
+  against `lablab-harness@8a674f5`, which is not public. A result nobody can
+  check is not a result. The new suite runs against the kit's own tree at a
+  pinned commit, so `git clone && git checkout <sha>` reproduces it exactly.
+
+  Ten reps per arm, 120 agent runs, and the result is not flattering:
+
+  | metric (median) | kit on | kit off |
+  |---|---|---|
+  | **correct** | **52/60** | **59/59** |
+  | tool calls | 11.5 | 3 |
+  | input tokens | 51,449 | 4,551 |
+  | final context | 39,603 | 10,855 |
+  | wall seconds | 43.6 | 32.5 |
+  | paths written outside the target | 0 | 0 |
+
+  **On a task that fits in one read, the kit is a cost.** Its instructions
+  cost eleven times the input tokens and four times the tool calls, the run
+  takes a third longer, and the answers are worse. The rule that says *locate
+  before you read* is a waste on a one-file question, and the tool counts show
+  it. The two edit tasks are a tie at 10/10, and the guard stopped no writes
+  in either arm.
+
+  The value the kit does have is in long sessions - the pruner removes 80.5%
+  of the context across real ones - and in process, which a grader that
+  counts correct answers cannot see. Whether it is worth it depends on
+  whether the work is long. **A repo of one-file lookups should not install
+  this.**
+
+  The full table, and what it does not establish, is in
+  `eval/RESULTS.md`.
+
+- **`oc-harness refresh --all`** re-derives every shipped profile against the
+  live catalogue, and reports which models are gone.
+- **`oc-harness doctor --check-models` and `--watch`** record a withdrawal in
+  `~/.config/opencode/withdrawn.jsonl`, with the profiles that used it. A model
+  is withdrawn without a rename and without a notice, so the only way to know
+  WHEN a lane started failing is to have been looking.
+- **`oc-harness upgrade`** checks the registry for a newer kit, refreshes the
+  profiles, and writes the config again. Three things go stale on their own -
+  the kit, the profiles and the config - and all three were invisible.
+- **A scheduled workflow** (`.github/workflows/measure.yml`), so the numbers
+  are kept rather than measured once. The offline half runs on every push;
+  the A/B runs nightly, because it costs model quota and takes about an hour.
+- **`OPENCODE_HARNESS_PROFILES`** points the kit at a different profile
+  directory. A test that refreshes a profile would otherwise rewrite the kit's
+  own data - which is what happened the first time this was tried.
+- **A flag appended to an npm script now overrides the script's own**:
+  `npm run measure:prune -- --sessions 1` means one session, not sixty.
+
+- **Tests for the pieces the suite never reached**: the python fallback, the
+  project root for a project that is not a git repository, the escape rule,
+  and the flag parser.
+
+### Security
+
+- `eval/prepare.mjs` could delete a directory tree. `--name` was joined onto
+  the scratch root without being checked and the result removed with
+  `rmSync(..., { recursive: true })`, so `--name ../victim` deleted the
+  victim's tree. The label is a name and not a path, and the resolved path is
+  checked a second time against the scratch root.
+- `eval/prepare.mjs` could run a command. The archive step was one `sh -c`
+  string and the paths were quoted with `JSON.stringify`, which is not shell
+  quoting — a path containing `$(...)` was executed by the shell. `git` and
+  `tar` are called with argument arrays now, and the tar goes through a pipe.
+
+## [0.4.3] — 2026-10-20
+
+### Removed
+
+- **The video pipeline and the published demo.** The `video/` directory held
+  the take script, the voiceover, the recording driver and the superseded
+  dubbing pipeline; `docs/demo/pitch.mp4` and `docs/demo-loop.gif` held the
+  published film. This repository is a harness kit, and it is code.
+
+  The film was also stale: it said "seven profiles" and "one hundred and
+  sixty five tests", and the kit now ships twelve profiles and two hundred
+  and ten tests. A front page that misstates the project's own numbers is
+  worse than no front page, so it goes rather than gets quietly outdated
+  again.
 
 ## [0.4.2] — 2026-10-20
 
@@ -42,20 +573,11 @@ is not happening.
   that can be opened in the JSON body. The list asked for JSON and worked; the
   create did not, and failed with a message that said nothing about why. Every
   npm trust call asks for JSON now.
-- **Each npm command gets its own two-factor challenge.** The code a browser
-  flow returns is valid for the challenge that asked for it and for nothing
-  else, so carrying one from one command to the next made the second command
-  send a code that was already spent - and the registry answered that with a
-  different challenge, one that wants an authenticator code rather than a
-  browser. The failure looked like "npm wants something else now" and nothing
-  like the cause.
-- **The URLs come from the response body, not from a guess.** The done URL
-  was rebuilt from the id, which sent a real request to the registry with a
-  made-up id; the registry answered "forbidden", which is not a token, so the
-  wait ran its full length and then reported that nobody had answered.
-- **`OC_TRUST_NO_BROWSER` turns the browser off.** A test that opens a
-  browser on the machine it runs on is a test that breaks whatever the
-  person at the keyboard was doing.
+- The per-command two-factor challenge, the done-URL guess and
+  `OC_TRUST_NO_BROWSER` were all fixed in 0.3.9 and are listed there. They
+  appear here once more because this release was cut from a branch that
+  predated 0.3.9's changelog entry, and a reader of the two releases was
+  left with the impression that the same three fixes happened twice.
 
 ### The release path, end to end
 
@@ -167,8 +689,9 @@ The publish is now automatic and holds no secret:
   short-lived publish token, scoped to the workflow that asked for it. There
   is no long-lived token in the repository to leak, rotate or expire - and npm
   is revoking the legacy automation tokens anyway.
-  - The job installs npm 11.5.1 or later, which is what OIDC needs and what
-    node 22 does not ship.
+  - The job runs on node 24, whose bundled npm already speaks trusted
+    publishing. An `npm install -g npm@latest` self-upgrade also works, and is
+    documented elsewhere as intermittently corrupting the install.
   - The registry is configured by hand instead of through `setup-node`'s
     `registry-url`, because that helper always writes an auth line, and an
     empty auth line makes npm believe it is authenticated and skip the OIDC
@@ -381,7 +904,7 @@ the tool-output caps, and the `scout` → `locate` agent rename.
   `git log --grep push` and `echo push` are not. A heredoc body is data, not
   commands.
 - **`tool_output` was pinned at 200 lines / 8 KB**, about 6× tighter than
-  OpenCode's default. It truncated 415 of 2,966 `read` calls and 418 calls
+  OpenCode's default. It truncated 415 of 2,966 `read` calls and 423 calls
   were issued with an `offset` to fetch the rest: the truncation *causes* the
   re-read, one for one, at a measured 17.3 s median per request. The cap is
   now the default, and pruning rather than truncation bounds the context.
@@ -433,7 +956,9 @@ silent in the worst way:
 
 ---
 
-[Unreleased]: https://github.com/pham-canh/space-harness/compare/v0.4.2...HEAD
+[Unreleased]: https://github.com/pham-canh/space-harness/compare/v0.4.3...HEAD
+[0.4.4]: https://github.com/pham-canh/space-harness/compare/v0.4.3...v0.4.4
+[0.4.3]: https://github.com/pham-canh/space-harness/compare/v0.4.2...v0.4.3
 [0.4.2]: https://github.com/pham-canh/space-harness/compare/v0.4.1...v0.4.2
 [0.4.1]: https://github.com/pham-canh/space-harness/compare/v0.4.0...v0.4.1
 [0.4.0]: https://github.com/pham-canh/space-harness/compare/v0.3.9...v0.4.0