|
| 1 | +# Review 17: a report entering the record as though it were a measurement |
| 2 | + |
| 3 | +**Lens.** Somebody hands the session a URL and a description of what is behind |
| 4 | +it. The session is told not to look. What ends up written down, and can the next |
| 5 | +session tell the difference between what was checked and what it was told. |
| 6 | + |
| 7 | +**Date.** 2026-08-28. Only `.tmp/PROMPT_COMPLETION.md` was touched in the change |
| 8 | +this reviews, so only that file is reviewed. |
| 9 | + |
| 10 | +⚠ Distinct from review 12, which read the brief as somebody with no repository, |
| 11 | +and review 13, which checked it line by line against the tree. This one asks |
| 12 | +whether one specific new claim is safe to leave in it. |
| 13 | + |
| 14 | +--- |
| 15 | + |
| 16 | +## What was checked |
| 17 | + |
| 18 | +### 1. The claim, and what is actually behind it |
| 19 | + |
| 20 | +The maintainer reported `https://github.com/Aseem0xff/pacman-static` as 0-BSD, a |
| 21 | +proof of concept for this repository, and said most of the hard work is done. One |
| 22 | +thing was fetched, because fetching it was the instruction: |
| 23 | + |
| 24 | +```bash |
| 25 | +curl -sS -o r.md -w '%{http_code} %{size_download}\n' --connect-timeout 10 --max-time 60 \ |
| 26 | + -L "https://github.com/Aseem0xff/pacman-static/raw/main/README.md?CACHE_BUST=$(date +%s)" |
| 27 | +``` |
| 28 | + |
| 29 | +`200 67`. The whole file: |
| 30 | + |
| 31 | +``` |
| 32 | +This is a POC for https://github.com/pkgforge-dev/docker-archlinux |
| 33 | +``` |
| 34 | + |
| 35 | +⛔ **The README corroborates one of the four claims and it is the smallest one.** |
| 36 | +It confirms the repository exists and says it is a proof of concept for this |
| 37 | +project. It says nothing about `pacman`, nothing about a licence, nothing about |
| 38 | +which architectures build, and carries no instructions. The 0-BSD claim, the |
| 39 | +"most of the hard work is done" claim, and the "unblocks everything" claim are |
| 40 | +all unattested by anything anybody has read. |
| 41 | + |
| 42 | +⚠ **That is not a criticism of the report.** It may be entirely right. The point |
| 43 | +is only that the file must not read as though it had been checked, because a |
| 44 | +session starting cold cannot tell the difference and this brief's first rule is |
| 45 | +that every claim in it is a lead rather than a fact. |
| 46 | + |
| 47 | +### 2. Does the brief distinguish the two |
| 48 | + |
| 49 | +Three places now name it. Each was read whole rather than grepped: |
| 50 | + |
| 51 | +| line | what it says | marks it unverified | |
| 52 | +| --- | --- | --- | |
| 53 | +| 429, decision 7 | a second reference exists and is unmined | ⭐ yes, "**unmined**", and "does not overturn this decision by existing" | |
| 54 | +| 637, TODO 2 | the report, then the 67 bytes, then what is unchecked | ⭐ yes, "Everything else about it is a report, not a measurement" | |
| 55 | +| 1065, Where things stand | the pointer a cold session meets first | ⭐ yes, "everything the report claims about it is unchecked" | |
| 56 | + |
| 57 | +⭐ **The verified part carries its own reproduce command**, and that command was |
| 58 | +run again during this review: `200 67`, byte identical to the first fetch. A |
| 59 | +reader can re-derive the one fact without trusting the file. |
| 60 | + |
| 61 | +⛔ **One trap is written down explicitly**, because it is the trap the previous |
| 62 | +reference set. Decision 7 exists because `packages-core-pacman-static` declared |
| 63 | +`GPL-2.0-or-later` in its `PKGBUILD` and shipped **no** LICENSE file. TODO 2 now |
| 64 | +says to read the LICENSE itself rather than the prose, and says separately that |
| 65 | +0-BSD on a recipe does not change what licence the `pacman` binary carries. |
| 66 | + |
| 67 | +### 3. Three reorderings in one session, and whether the numbering survived |
| 68 | + |
| 69 | +The TODO list was reordered three times on 2026-08-28: loong64 removed to |
| 70 | +Completed, three tasks inserted, then static pacman moved from 8 to 2. Each |
| 71 | +reordering renumbers headings and orphans every cross-reference. |
| 72 | + |
| 73 | +```bash |
| 74 | +sed -n '/^## TODO/,$p' .tmp/PROMPT_COMPLETION.md | awk '/^### [0-9]+\. /{ ... }' |
| 75 | +for n in $(grep -o 'TODO [0-9]*' .tmp/PROMPT_COMPLETION.md | awk '{ print $2 }' | sort -un); do ... done |
| 76 | +``` |
| 77 | + |
| 78 | +| check | result | |
| 79 | +| --- | --- | |
| 80 | +| headings sequential | ⭐ 13, no gap | |
| 81 | +| duplicate heading numbers | ⭐ 0 | |
| 82 | +| cross-references resolving | ⭐ 3 distinct targets, all resolve, all to the intended task | |
| 83 | +| Completed table | ⭐ rows run 1..30, untouched by any of the three | |
| 84 | + |
| 85 | +⚠ **This is the third time in one session that a renumbering needed a manual |
| 86 | +remap.** It worked each time and it was checked each time, but a cross-reference |
| 87 | +in prose is a thing that goes silently wrong, and nothing in the repository |
| 88 | +checks the brief. That is by design, since `.tmp/` is scratch, and it means the |
| 89 | +check is a human one every time. |
| 90 | + |
| 91 | +--- |
| 92 | + |
| 93 | +## What was found and changed |
| 94 | + |
| 95 | +Nothing was changed by this review. The three placements above already carried |
| 96 | +their unverified markers when it ran, because the risk was the reason for writing |
| 97 | +them that way. |
| 98 | + |
| 99 | +## What was ruled out |
| 100 | + |
| 101 | +- **A number nobody can reproduce.** The only number introduced is 67 bytes, and |
| 102 | + the command that produces it is in the file and was re-run. |
| 103 | +- **The word "verified" used loosely.** Grepped: the reference block says "What |
| 104 | + is verified is only this" and then bounds it to the README. No other sentence |
| 105 | + about this reference uses it. |
| 106 | +- **Decision 7 being quietly overturned.** The new reference does not touch it. |
| 107 | + Both the decision and TODO 2 now say that overturning it is a decision for the |
| 108 | + maintainer. |
| 109 | +- **Stray characters and encoding.** Only the three permitted markers across all |
| 110 | + 1080 lines, no em dash, no CRLF. |
| 111 | +- **Repository drift.** `git status` is clean and `main` is still `3d98756`. |
| 112 | + Nothing in the tree was touched by this change. |
| 113 | + |
| 114 | +## ⚠ What this did not look at |
| 115 | + |
| 116 | +- ⛔ **The reference itself.** Not cloned, not read, no commit captured, no |
| 117 | + tracker read, no LICENSE opened. That was the instruction, and it is why |
| 118 | + nothing here says whether the report is right. |
| 119 | +- **Whether the reference will still be there.** One README was fetched twice, |
| 120 | + eight minutes apart. A repository that appeared during a session can change or |
| 121 | + go away. |
| 122 | +- **The rest of the brief.** Only the parts this change touched were read: |
| 123 | + decision 7, the TODO headings and cross-references, TODO 2, and "Where things |
| 124 | + stand". Reviews 12 and 13 cover the file as a whole, at earlier dates. |
| 125 | +- **Whether TODO 2 at high priority is the right call.** That is the maintainer's |
| 126 | + ordering, applied as given. |
| 127 | + |
| 128 | +## Change summary |
| 129 | + |
| 130 | +Files touched: 1, `.tmp/PROMPT_COMPLETION.md`, which is not in the repository. |
| 131 | +Lines added: 141. Lines removed: 85. 1024 lines to 1080. |
| 132 | + |
| 133 | +⚠ This review is in the repository and the file it reviews is not, because |
| 134 | +`.tmp/` is wiped between sessions. Reviews 12 and 13 have the same shape. |
0 commit comments