Gatekeeper round 2 on PR #161 (head ffe46c7): BLOCK. Details are on the PR.
The unit tests now pin readOpened, Rel and WalkRegular. Remaining: nothing checks that ReadRegular goes through…
Gatekeeper verdict, round 2: BLOCK (head ffe46c7)
Close. One gap is left, with the same shape as #156 round 2: the check has a unit test, but nothing checks that its caller uses it.
###…
pr-gatekeeper verdict, round 2: BLOCK (PR #160 @ 01de163). Not merged.
Round-1 findings 1, 3 and 4 are closed, and I verified each by execution. CI run 8916 is green. The fix for finding 2…
pr-gatekeeper verdict, round 2: BLOCK (PR #160 @ 01de163). Not merged.
Round-1 findings 1, 3 and 4 are closed, and I verified each by execution. CI run 8916 is green. The fix for finding 2…
Gatekeeper review of PR #161 (slice 1, search, head 9c9ba31): BLOCK. Details are on the PR.
Symlinks at every depth, symlinked wing and hall filters, and symlinked wiki/ and knowledge/ roots…
Gatekeeper verdict: BLOCK (head 9c9ba31)
The control held under every probe. One load-bearing check has no test.
What survived (run in a scratch clone, sentinel counted, contents never…
One consequence of the auto-remove decision that the record above didn't spell out (from the PR #160 review): the brain dir is a Syncthing sendreceive folder, so a symlink the tripwire removes…
pr-gatekeeper verdict: BLOCK (PR #160 @ 504feb3). Not merged.
The removal itself is correct, and I verified that by execution (table below). Two things block. CI is red, and the alert path…
pr-gatekeeper verdict: BLOCK (PR #160 @ 504feb3). Not merged.
The removal itself is correct, and I verified that by execution (table below). Two things block. CI is red, and the alert path…