LibreKAT/Ocideck

Watch

Fork 0

fix(module): laat de drie achtergebleven oppervlakken meebewegen (#648) #662

Merged

brenno merged 1 commit from fix/module-lekt into main

2026-07-22 17:50:56 +00:00

brenno commented

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

Owner

Instellingen die je kunt openen, invullen en dan niet gebruiken, omdat de enige afnemer van het resultaat achter de modulepoort zit.

De regel is niet "verbergen"

Het is: verbergen zolang er niets ligt, en tónen zodra er wél iets ligt. Wie sjablonen heeft gemaakt of de lokale CVE-database al heeft gedownload en dan de module uitzet, moet zijn werk kunnen zien en weghalen — anders laat een schakelaar stilzwijgend een gigabyte op schijf achter zonder route terug.

Die tweede helft heeft zijn eigen test, want dat is de helft die het makkelijkst per ongeluk sneuvelt. Beide richtingen geplant en rood gezien.

Een gat dat ik zelf maakte

De zoekindex moest mee. Zonder die stap vond zoeken nog steeds "CVE opzoeken" en sprong dan naar een tabblad waar dat blok niet meer staat — verwarrender dan de knop die we net weghaalden. Dat kwam pas boven doordat een bestaande test omviel; het stond niet in de issue.

Het derde oppervlak: bewust ongemoeid

De docs-tegel van docs/design/PENTEST_MIAUW.md blijft staan, en ik volg daarin de redenering uit de issue. De documentatielezer is een transparantiefunctie: het punt is dat je kunt lezen wát deze applicatie is zonder eerst iets aan te zetten. Die tegel verbergen zou dezelfde logica doortrekken naar de hele Ontwerp-groep, en dan onthult de app minder naarmate je minder aan hebt staan. Een ontwerpdocument verbergen is een andere handeling dan een functie verbergen.

Vier bestaande tests

Ze gingen uit van de oude zichtbaarheid. Ze zetten nu de module aan in plaats van dat ik hun bakens verzwakte — de regressie die ze bewaken (de koppeling zoektreffer→tabblad, die ooit brak toen er een tabblad tussen werd geschoven) blijft precies zoals hij was.

Eén uitzondering: de sprongtest gebruikte de CVE-schakelaar als baken, en toetste daarmee sinds vandaag de módule in plaats van de sprong. Die kreeg een baken uit een sectie die er altijd staat.

make check groen.

Closes #648

Instellingen die je kunt openen, invullen en dan niet gebruiken, omdat de enige afnemer van het resultaat achter de modulepoort zit. ## De regel is niet "verbergen" Het is: **verbergen zolang er niets ligt, en tónen zodra er wél iets ligt.** Wie sjablonen heeft gemaakt of de lokale CVE-database al heeft gedownload en dan de module uitzet, moet zijn werk kunnen zien en weghalen — anders laat een schakelaar stilzwijgend een gigabyte op schijf achter zonder route terug. Die tweede helft heeft zijn eigen test, want dat is de helft die het makkelijkst per ongeluk sneuvelt. Beide richtingen geplant en rood gezien. ## Een gat dat ik zelf maakte De **zoekindex** moest mee. Zonder die stap vond zoeken nog steeds "CVE opzoeken" en sprong dan naar een tabblad waar dat blok niet meer staat — verwarrender dan de knop die we net weghaalden. Dat kwam pas boven doordat een bestaande test omviel; het stond niet in de issue. ## Het derde oppervlak: bewust ongemoeid De docs-tegel van `docs/design/PENTEST_MIAUW.md` blijft staan, en ik volg daarin de redenering uit de issue. De documentatielezer is een transparantiefunctie: het punt is dat je kunt lezen wát deze applicatie is zonder eerst iets aan te zetten. Die tegel verbergen zou dezelfde logica doortrekken naar de hele *Ontwerp*-groep, en dan onthult de app minder naarmate je minder aan hebt staan. Een ontwerpdocument verbergen is een andere handeling dan een functie verbergen. ## Vier bestaande tests Ze gingen uit van de oude zichtbaarheid. Ze zetten nu de module aan in plaats van dat ik hun bakens verzwakte — de regressie die ze bewaken (de koppeling zoektreffer→tabblad, die ooit brak toen er een tabblad tussen werd geschoven) blijft precies zoals hij was. Eén uitzondering: de sprongtest gebruikte de CVE-schakelaar als baken, en toetste daarmee sinds vandaag de módule in plaats van de sprong. Die kreeg een baken uit een sectie die er altijd staat. `make check` groen. Closes #648

brenno added 1 commit

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

fix(module): laat de drie achtergebleven oppervlakken meebewegen

CI / Gate (Linux) · Format · Analyze · Coverage (push) Failing after 23s

Details

CI / Web hardening (push) Failing after 21s

Details

CI / Supply-chain (Trivy · advisory) (push) Failing after 26s

Details

CI / Docs links (push) Failing after 27s

Details

CI / Web hardening (pull_request) Failing after 26s

Details

CI / Gate (Linux) · Format · Analyze · Coverage (pull_request) Failing after 27s

Details

CI / Docs links (pull_request) Failing after 26s

Details

CI / Supply-chain (Trivy · advisory) (pull_request) Failing after 24s

Details

CI / Test (macos-latest) (push) Has been cancelled

Details

CI / Test (windows-latest) (push) Has been cancelled

Details

CI / Test (macos-latest) (pull_request) Has been cancelled

Details

CI / Test (windows-latest) (pull_request) Has been cancelled

Details

22cd3b280a

Instellingen die je kunt openen, invullen en dan niet gebruiken, omdat de enige
afnemer van het resultaat achter de modulepoort zit. Een checklist-sjabloon dat
nergens te laden is; een aanbod om honderden megabytes CVE-data binnen te halen
voor een zoekfunctie die niet opengaat.

De regel is niet "verbergen" maar: verbergen zolang er niets ligt, en tónen
zodra er wél iets ligt. Wie sjablonen heeft gemaakt of de database al heeft
gedownload en dan de module uitzet, moet zijn werk kunnen zien en weghalen —
anders laat een schakelaar stilzwijgend een gigabyte op schijf achter zonder
route terug. Die tweede helft heeft zijn eigen test, want dat is de helft die
het makkelijkst sneuvelt.

De zoekindex moest mee, en dat was geen bijvangst maar een gat dat ik zelf
maakte: zonder die stap vond zoeken nog steeds "CVE opzoeken" en sprong dan
naar een tabblad waar dat blok niet meer staat. Verwarrender dan de knop die we
net weghaalden.

Het derde oppervlak uit de issue — de docs-tegel van het ontwerpdocument — is
bewust ongemoeid. De documentatielezer is een transparantiefunctie: het punt is
dat je kunt lezen wat deze applicatie is zonder eerst iets aan te zetten. Die
tegel verbergen zou dezelfde redenering doortrekken naar de hele Ontwerp-groep,
en dan onthult de app minder naarmate je minder aan hebt staan.

Vier bestaande tests gingen uit van de oude zichtbaarheid. Ze zetten nu de
module aan in plaats van dat ik hun bakens verzwakte — de regressie die ze
bewaken (de koppeling zoektreffer→tabblad) blijft precies zoals hij was. Eén
uitzondering: de sprongtest gebruikte de CVE-schakelaar als baken en toetste
daarmee sinds vandaag de module in plaats van de sprong; die kreeg een baken
uit een sectie die er altijd staat.

Closes #648

brenno merged commit f5a8ed0c12 into main

2026-07-22 17:50:56 +00:00

brenno referenced this pull request from a commit

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

Merge pull request 'fix(module): laat de drie achtergebleven oppervlakken meebewegen (#648)' (#662) from fix/module-lekt into main