IDEA-230: voorkom paginaverversing bij mapprullenbakklik #99

Merged

janpeter merged 2 commits from codex/idea-230-no-refresh into main 2026-09-29 20:06:14 +02:00

Conversation 2 Commits 2 Files changed 4 +26 -11

janpeter commented 2026-09-29 19:56:55 +02:00

Owner

Copy link

Wijziging

De mapprullenbakknop gebruikte na bevestigde status router.refresh(), terwijl de rode of neutrale knopstatus al lokaal beschikbaar is. Die geforceerde paginaverversing is verwijderd; focusherstel blijft werken. Bij herstel invalideert de serveractie ook niet meer de huidige /library-route. De prullenbakroute blijft invalideerbaar. De read-only poll voor een inventarisatie die na herladen uit de serverprojectie komt, blijft bestaan.

Verificatie

• Gemounte kliktest was eerst rood: één router.refresh() na markeren. Na de wijziging: nul na markeren en herstellen, met behoud van status, selectie en focus; vijf opeenvolgende gerichte runs groen.
• Serveractietest was eerst rood omdat herstel /library en /library/system/trash invalideerde. Nu alleen de prullenbakroute.
• Volledige lokale suite met aparte allowlisted PostgreSQL 17-testdatabase en Node 22.23.2: 1.175 tests, 1.144 geslaagd, 0 mislukt, 31 geconfigureerde skips.
• npm run typecheck en npm run build: geslaagd. Lint van de gewijzigde bestanden: 0 fouten, 1 reeds bestaande waarschuwing in folder-trash-core.ts.

Deze PR bevat alleen de vervolgfix op de reeds gemergede PR #98. Geen merge of uitrol uitgevoerd.

## Wijziging De mapprullenbakknop gebruikte na bevestigde status `router.refresh()`, terwijl de rode of neutrale knopstatus al lokaal beschikbaar is. Die geforceerde paginaverversing is verwijderd; focusherstel blijft werken. Bij herstel invalideert de serveractie ook niet meer de huidige `/library`-route. De prullenbakroute blijft invalideerbaar. De read-only poll voor een inventarisatie die na herladen uit de serverprojectie komt, blijft bestaan. ## Verificatie - Gemounte kliktest was eerst rood: één `router.refresh()` na markeren. Na de wijziging: nul na markeren en herstellen, met behoud van status, selectie en focus; vijf opeenvolgende gerichte runs groen. - Serveractietest was eerst rood omdat herstel `/library` en `/library/system/trash` invalideerde. Nu alleen de prullenbakroute. - Volledige lokale suite met aparte allowlisted PostgreSQL 17-testdatabase en Node 22.23.2: 1.175 tests, 1.144 geslaagd, 0 mislukt, 31 geconfigureerde skips. - `npm run typecheck` en `npm run build`: geslaagd. Lint van de gewijzigde bestanden: 0 fouten, 1 reeds bestaande waarschuwing in `folder-trash-core.ts`. Deze PR bevat alleen de vervolgfix op de reeds gemergede PR #98. Geen merge of uitrol uitgevoerd.

janpeter added 1 commit 2026-09-29 19:56:55 +02:00

Keep folder trash toggle in place after status changes a8ca3c86c3

s4m-codex-reviewer reviewed 2026-09-29 20:01:57 +02:00

s4m-codex-reviewer left a comment

Copy link

COMMENT

• LOW — src/components/useFolderTrashToggle.ts:249: onChanged wordt na deze wijziging nergens meer aangeroepen, maar blijft wel onderdeel van de hook-signatuur en dependency-array. Verwijder de callback-API (en pas de consumer aan) om geen onjuiste cache-invalideringsbelofte in stand te houden.
• INFO — src/components/LibraryFolderView.folder-trash.mounted.test.ts:47: de test borgt terecht dat zowel markeren als terugzetten geen router-refresh uitvoeren.

geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden.

## COMMENT - **LOW** — `src/components/useFolderTrashToggle.ts:249`: `onChanged` wordt na deze wijziging nergens meer aangeroepen, maar blijft wel onderdeel van de hook-signatuur en dependency-array. Verwijder de callback-API (en pas de consumer aan) om geen onjuiste cache-invalideringsbelofte in stand te houden. - **INFO** — `src/components/LibraryFolderView.folder-trash.mounted.test.ts:47`: de test borgt terecht dat zowel markeren als terugzetten geen router-refresh uitvoeren. geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden.

janpeter added 1 commit 2026-09-29 20:02:29 +02:00

Avoid library route invalidation on folder restore 24df818f82

janpeter merged commit e23ca503ad into main 2026-09-29 20:06:14 +02:00

janpeter referenced this pull request from a commit 2026-09-29 20:06:14 +02:00

Merge pull request 'IDEA-230: voorkom paginaverversing bij mapprullenbakklik' (#99) from codex/idea-230-no-refresh into main

s4m-codex-reviewer approved these changes 2026-09-29 20:07:42 +02:00

s4m-codex-reviewer left a comment

Copy link

APPROVED

Bevindingen

• Geen bevindingen met blokkerende of error-severity.

Gerichte controle: de coretest voor route-invalidatie slaagt (10/10); de gemounte UI-test kon in deze runner niet starten doordat /tmp met noexec is gemount, niet door een test- of codefout. De diff is syntactisch schoon en de wijziging past bij de productspecificatie: de huidige bibliotheekweergave mag na de mapactie niet verversen, terwijl de prullenbakroute wel wordt geïnvalideerd.

Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden.

## APPROVED ### Bevindingen - Geen bevindingen met blokkerende of error-severity. Gerichte controle: de coretest voor route-invalidatie slaagt (10/10); de gemounte UI-test kon in deze runner niet starten doordat `/tmp` met `noexec` is gemount, niet door een test- of codefout. De diff is syntactisch schoon en de wijziging past bij de productspecificatie: de huidige bibliotheekweergave mag na de mapactie niet verversen, terwijl de prullenbakroute wel wordt geïnvalideerd. Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden.

Sign in to join this conversation.

Reviewers

No reviewers

s4m-codex-reviewer

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

2 participants

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

janpeter/Media-Organizer!99

No description provided.