Spaces:
Running
Running
solidprivacy-nl commited on
Commit ·
46e3ed3
1
Parent(s): c9079fc
Add TXT reinsert UI app verification closeout handover
Browse files
handover/workpackages/20260608_0015_v13_7_txt_reinsert_upload_download_ui_app_closeout.md
ADDED
|
@@ -0,0 +1,96 @@
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 1 |
+
# Handover — WP13B — v13.7 — TXT reinsert upload/download UI app verification closeout
|
| 2 |
+
|
| 3 |
+
Repository: `solidprivacy-nl/scrub`
|
| 4 |
+
Status: completed and app-verified after Actions/sync verification
|
| 5 |
+
|
| 6 |
+
## Summary
|
| 7 |
+
|
| 8 |
+
WP13B was performed as closeout-only. The TXT reinsert upload/download UI from WP13 is now recorded as completed and app-verified after coordinator evidence showed GitHub Actions and Hugging Face sync green, and the Hugging Face app verification confirmed TXT reinsert behavior.
|
| 9 |
+
|
| 10 |
+
## Files added
|
| 11 |
+
|
| 12 |
+
- `handover/workpackages/20260608_0015_v13_7_txt_reinsert_upload_download_ui_app_closeout.md`
|
| 13 |
+
|
| 14 |
+
## Files changed
|
| 15 |
+
|
| 16 |
+
- `WORKPACKAGES.md`
|
| 17 |
+
- `CHANGELOG.md`
|
| 18 |
+
|
| 19 |
+
## Code files changed
|
| 20 |
+
|
| 21 |
+
None.
|
| 22 |
+
|
| 23 |
+
The following files were intentionally not changed:
|
| 24 |
+
|
| 25 |
+
- `fix_streamlit_nested_expanders.py`
|
| 26 |
+
- `presidio_streamlit.py`
|
| 27 |
+
- `scrub_key_document_reinsert.py`
|
| 28 |
+
- `scrub_key_reinsert.py`
|
| 29 |
+
- `scrub_key.py`
|
| 30 |
+
- `scrub_key_import.py`
|
| 31 |
+
- `tests/*`
|
| 32 |
+
|
| 33 |
+
## Tests
|
| 34 |
+
|
| 35 |
+
No new tests were added because this was a closeout-only workpackage.
|
| 36 |
+
|
| 37 |
+
Existing technical verification was recorded from coordinator evidence:
|
| 38 |
+
|
| 39 |
+
```text
|
| 40 |
+
Tests #164 green — commit e442d28
|
| 41 |
+
Sync to Hugging Face Space #178 green — commit e442d28
|
| 42 |
+
Tests #165 green — commit ce7721d
|
| 43 |
+
Sync to Hugging Face Space #179 green — commit ce7721d
|
| 44 |
+
Tests #166 green — commit 6bf359f
|
| 45 |
+
Sync to Hugging Face Space #180 green — commit 6bf359f
|
| 46 |
+
Tests #167 green — commit 443d6af
|
| 47 |
+
Sync to Hugging Face Space #181 green — commit 443d6af
|
| 48 |
+
```
|
| 49 |
+
|
| 50 |
+
## Validation
|
| 51 |
+
|
| 52 |
+
- GitHub Actions: green based on coordinator evidence.
|
| 53 |
+
- Hugging Face sync: green based on coordinator evidence.
|
| 54 |
+
- App verification: confirmed by coordinator/user.
|
| 55 |
+
|
| 56 |
+
Confirmed app behavior:
|
| 57 |
+
|
| 58 |
+
- `Originele waarden terugzetten` mode is active.
|
| 59 |
+
- `Scrub Key laden` is visible.
|
| 60 |
+
- Scrub Key JSON is loaded.
|
| 61 |
+
- Pasted-text reinsert remains available.
|
| 62 |
+
- `TXT-bestand terugzetten` is visible.
|
| 63 |
+
- TXT file upload works.
|
| 64 |
+
- `Zet TXT-bestand lokaal terug` works with a valid Scrub Key.
|
| 65 |
+
- Result message appears: `37 waarde(n) lokaal teruggezet in het TXT-bestand.`
|
| 66 |
+
- `Herstelde TXT-tekst` appears.
|
| 67 |
+
- `Download hersteld TXT-bestand (.txt)` is visible.
|
| 68 |
+
- `Controleverslag TXT terugzetten` appears.
|
| 69 |
+
- Audit shows document type `txt`.
|
| 70 |
+
- Audit shows mapping counts, restored value count, not-found placeholders, unknown placeholders and validation issues.
|
| 71 |
+
- Audit shows `local_only=True`, `ai_processing=False` and `cloud_processing=False`.
|
| 72 |
+
|
| 73 |
+
## Boundaries preserved
|
| 74 |
+
|
| 75 |
+
- No DOCX upload reinsert UI was added.
|
| 76 |
+
- No PDF reinsert was added.
|
| 77 |
+
- No AI calls were added.
|
| 78 |
+
- No cloud processing was added.
|
| 79 |
+
- No automatic document rehydration was added.
|
| 80 |
+
- Existing pasted-text reinsert remains available.
|
| 81 |
+
- Existing Scrub Key import/export remains available.
|
| 82 |
+
- Existing scrubbed TXT/CSV/DOCX/PDF export/download behavior remains unchanged.
|
| 83 |
+
- No Scrub Key storage, secrets, tokens or real personal data were added.
|
| 84 |
+
|
| 85 |
+
## Remaining risks
|
| 86 |
+
|
| 87 |
+
- TXT reinsert is app-verified, but DOCX reinsert UI is not yet implemented.
|
| 88 |
+
- PDF reinsert remains intentionally out of scope until a separate reliability review.
|
| 89 |
+
- WP10 helper foundation still has coordinator-verification nuance recorded because Actions/sync were not visible through the connector during WP10B, although later UI lines relying on the helpers have been verified through their own Actions/sync.
|
| 90 |
+
|
| 91 |
+
## Next recommended step
|
| 92 |
+
|
| 93 |
+
- Start `WP14 — v13.8 — DOCX reinsert upload/download UI` sequentially.
|
| 94 |
+
- Do not run WP14 in parallel with any other UI workpackage.
|
| 95 |
+
- Keep PDF full reinsert out of scope until a separate reliability review.
|
| 96 |
+
- Keep AI/cloud behavior out unless explicitly approved.
|