LibreKAT/Ocideck

Watch

Fork 0

[Bug] Three file paths in GIT_STORAGE.md name files that do not exist #637

New issue

Closed

opened 2026-07-22 16:23:35 +00:00 by brenno · 1 comment

brenno commented

2026-07-22 16:23:35 +00:00

Owner

Found in the pre-publication documentation review.

Evidence: a script over 242 backtick-quoted paths in the public documentation found 15 non-existent paths, of which 12 are legitimate (future paths in the two unbuilt designs, COLLABORATION.md and PROCESS_IMPROVEMENT.md). The other three are in docs/design/GIT_STORAGE.md:1102-1106 — a document that states phases 0-6 have landed and that it is "written to be picked up cold: exact file paths…":

native_git_mirror.dart — exists as native_git_mirror_io.dart / _api / _factory / _stub
draft_mirror.dart — exists as draft_store*.dart (4 files)
git_cli_stub.dart — exists as git_cli_web.dart

The implementation is there; the names changed during construction and the map was not updated.

Why this matters: small, but it hits exactly the promise that document makes in its opening lines, and it is the one document where a wrong path leads a reader to conclude the feature does not exist. Five minutes of work.

Proposal: correct the three lines to the real filenames with a *(Corrected …)* note per the house style. Consider afterwards a gate in tool/ that checks backtick-quoted paths to lib/ in docs/ for existence, with the 12 design paths on a baseline, so this class of drift cannot recur.

Found in the pre-publication documentation review. **Evidence:** a script over 242 backtick-quoted paths in the public documentation found 15 non-existent paths, of which 12 are legitimate (future paths in the two unbuilt designs, `COLLABORATION.md` and `PROCESS_IMPROVEMENT.md`). The other three are in `docs/design/GIT_STORAGE.md:1102-1106` — a document that states phases 0-6 have landed and that it is "written to be picked up cold: exact file paths…": - `native_git_mirror.dart` — exists as `native_git_mirror_io.dart` / `_api` / `_factory` / `_stub` - `draft_mirror.dart` — exists as `draft_store*.dart` (4 files) - `git_cli_stub.dart` — exists as `git_cli_web.dart` The implementation is there; the names changed during construction and the map was not updated. **Why this matters:** small, but it hits exactly the promise that document makes in its opening lines, and it is the one document where a wrong path leads a reader to conclude the feature does not exist. Five minutes of work. **Proposal:** correct the three lines to the real filenames with a `*(Corrected …)*` note per the house style. Consider afterwards a gate in `tool/` that checks backtick-quoted paths to `lib/` in `docs/` for existence, with the 12 design paths on a baseline, so this class of drift cannot recur.

brenno added the

docs

bug

good first issue

labels

2026-07-22 16:23:35 +00:00

brenno referenced this issue from a commit

2026-07-22 17:07:10 +00:00

docs: maak het bijdragerspad waar wat het belooft (#588 #625 #642 #596 #595 #594)

brenno referenced this issue from a commit

2026-07-22 17:10:07 +00:00

docs: haal negen achterhaalde beweringen uit de publieke teksten

brenno referenced this issue from a commit

2026-07-22 17:12:52 +00:00

docs: maak het bijdragerspad waar wat het belooft (#588 #625 #642 #596 #595 #594)

brenno referenced this issue

2026-07-22 17:13:34 +00:00

docs: haal negen achterhaalde beweringen uit de publieke teksten #655

brenno referenced this issue

2026-07-22 17:14:10 +00:00

docs: de publicatieklaar-tros — 26 docs-issues in zeven commits #656

brenno commented

2026-07-22 17:19:18 +00:00

Author

Owner

Opgelost in `cc00070` (PR #655).

De drie paden wijzen nu naar native_git_mirror_io.dart, draft_store.dart en git_cli_web.dart, met vermelding van de _api/_factory/_stub-broers ernaast, plus een gedateerde correctienotitie.

De voorgestelde poort in tool/ die backtick-paden naar lib/ toetst zit hier niet in — dat is code en verdient een eigen afweging (de twaalf legitieme toekomstige paden in de ongebouwde ontwerpen zouden een basislijst nodig hebben).

Opgelost in \`cc00070\` (PR #655). De drie paden wijzen nu naar `native_git_mirror_io.dart`, `draft_store.dart` en `git_cli_web.dart`, met vermelding van de `_api`/`_factory`/`_stub`-broers ernaast, plus een gedateerde correctienotitie. De voorgestelde poort in `tool/` die backtick-paden naar `lib/` toetst zit hier niet in — dat is code en verdient een eigen afweging (de twaalf legitieme toekomstige paden in de ongebouwde ontwerpen zouden een basislijst nodig hebben).

brenno closed this issue

2026-07-22 17:19:18 +00:00

brenno referenced this issue

2026-07-22 17:24:19 +00:00

docs: de publicatieklaar-tros — 26 docs-issues in zeven commits #656

brenno referenced this issue from a commit

2026-07-22 17:29:21 +00:00

docs: maak het bijdragerspad waar wat het belooft (#588 #625 #642 #596 #595 #594)

brenno referenced this issue from a commit

2026-07-22 17:58:58 +00:00

refactor(state): geef DeckNotifier lucht, en ruim drie kleine bugs op

brenno referenced this issue from a pull request that will close it,