feat(libreplan): read-only REST-import als specialist-module (#1240) #1255

Merged

brenno merged 10 commits from feat/libreplan-connector into main 2026-08-05 00:36:24 +00:00

Conversation 0 Commits 10 Files changed 66 +4673 -2

brenno commented 2026-08-05 00:34:35 +00:00

Owner

Copy link

Summary

• LibrePlan-connector als optionele, alleen-lezen REST-importmodule. Haalt een projectsnapshot op van een LibrePlan-instantie en vult OciDeck-slides: Gantt, WBS, projectstatus, milestones, kritiek pad, resources, timesheet, resourcebelasting.
• Off by default, desktop-only, GET-only. NetGuard-blokkade voor privé-adressen (tenzij trustedInternal), HTTPS verplicht (tenzij trustedInternal), redirect-weigering, response-cap 10 MiB, depth-limited XML (100). Wachtwoord in OS-keychain, niet in prefs of deck.
• Vier lagen: XML-parser (pure Dart), REST-client (NetGuard + pinning), converters (models → slides), import-orchestrator. Elk met eigen tests.
• Settings UI: module-kaart op Uitbreidingen-tab, LibrePlan-tab met formulier + verbindingstest, import-dialoog met per-slide-type checkboxes.
• Docs: PRIVACY.md, SECURITY.md, USER_GUIDE.nl.md, CHANGELOG, SOURCE_MAP.

Bewaker-toets

De implementatie respecteert alle tien kernwaarden:

• Soevereiniteit (3): de connector praat met de gebruiker's eigen LibrePlan-instantie, niet met een nieuwe externe partij. Uitgaand verkeer staat standaard uit.
• Veiligheid (1): fail-closed (parse-error → leeg, geen crash), NetGuard, HTTPS-verplicht, redirect-weigering, response-cap, depth-limited XML.
• Uitwisselbaarheid: geïmporteerde slides gebruiken bestaande slide-types (timeline, tree, table, chart, cockpit, flow) — geen nieuwe _class-tokens of sidecar-formaten. Het .md-bestand blijft leesbaar in ander gereedschap.

Geen waarde-conflicten die expliciete afweging vereisen.

make check-full

Alle poorten groen: format, analyze, toolchain, conventions, privacy projection boundary, method length, dead code, hardcoded text, comment language, improvement templates, version bump, SBOM, coverage (87.4%), per-file coverage floor, gitleaks, trufflehog, semgrep, shellcheck, licences, SBOM up-to-date, bundled JS, web hardening, release artefacts.

Test plan

• make check groen
• make check-full groen (secrets + SAST + web + release)
• 17 converter-tests, 4 client-tests, XML-tests, import-tests, settings-model-tests, import-dialog-widget-tests
• Bewaker-toets uitgevoerd (zie boven)
• Beeldkeuring: handmatig controleren dat het LibrePlan-tabblad er goed uitziet in de settings-dialoog

Closes #1240

Generated with Devin

## Summary - **LibrePlan-connector** als optionele, alleen-lezen REST-importmodule. Haalt een projectsnapshot op van een LibrePlan-instantie en vult OciDeck-slides: Gantt, WBS, projectstatus, milestones, kritiek pad, resources, timesheet, resourcebelasting. - **Off by default, desktop-only, GET-only.** NetGuard-blokkade voor privé-adressen (tenzij trustedInternal), HTTPS verplicht (tenzij trustedInternal), redirect-weigering, response-cap 10 MiB, depth-limited XML (100). Wachtwoord in OS-keychain, niet in prefs of deck. - **Vier lagen:** XML-parser (pure Dart), REST-client (NetGuard + pinning), converters (models → slides), import-orchestrator. Elk met eigen tests. - **Settings UI:** module-kaart op Uitbreidingen-tab, LibrePlan-tab met formulier + verbindingstest, import-dialoog met per-slide-type checkboxes. - **Docs:** PRIVACY.md, SECURITY.md, USER_GUIDE.nl.md, CHANGELOG, SOURCE_MAP. ### Bewaker-toets De implementatie respecteert alle tien kernwaarden: - **Soevereiniteit (3):** de connector praat met de gebruiker's *eigen* LibrePlan-instantie, niet met een nieuwe externe partij. Uitgaand verkeer staat standaard uit. - **Veiligheid (1):** fail-closed (parse-error → leeg, geen crash), NetGuard, HTTPS-verplicht, redirect-weigering, response-cap, depth-limited XML. - **Uitwisselbaarheid:** geïmporteerde slides gebruiken bestaande slide-types (timeline, tree, table, chart, cockpit, flow) — geen nieuwe _class-tokens of sidecar-formaten. Het .md-bestand blijft leesbaar in ander gereedschap. Geen waarde-conflicten die expliciete afweging vereisen. ### make check-full Alle poorten groen: format, analyze, toolchain, conventions, privacy projection boundary, method length, dead code, hardcoded text, comment language, improvement templates, version bump, SBOM, coverage (87.4%), per-file coverage floor, gitleaks, trufflehog, semgrep, shellcheck, licences, SBOM up-to-date, bundled JS, web hardening, release artefacts. #### Test plan - [x] make check groen - [x] make check-full groen (secrets + SAST + web + release) - [x] 17 converter-tests, 4 client-tests, XML-tests, import-tests, settings-model-tests, import-dialog-widget-tests - [x] Bewaker-toets uitgevoerd (zie boven) - [ ] Beeldkeuring: handmatig controleren dat het LibrePlan-tabblad er goed uitziet in de settings-dialoog Closes #1240 Generated with [Devin](https://devin.ai)

brenno added 11 commits 2026-08-05 00:34:36 +00:00

feat(libreplan): foundation — settings, secret store, module-registry ... b671d12692

LibrePlan-connector (#1240) stap 1: het instellingsmodel, de keychain-
methodes en de module-registry-entry waaronder de rest landt. Spiegelt
het AI-assistentie-patroon: LibreplanSettings (off by default, geen
wachtwoord in prefs), SecretStore.writeLibreplanPassword/
readLibreplanPassword/deleteLibreplanPassword (gekeyd op base URL +
username, fail-closed op web), en ModuleId.libreplan in de registry.

De modulekaart op Uitbreidingen is een schakelaar die het formulier
verandert, niet de opgeslagen instelling — zelfde scheiding als AiModuleCard.
Desktop-only op web (keychain niet veilig). De volledige form-velden,
REST-client, converters, import-dialoog en docs komen in volgende commits.

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>

feat(libreplan): XML-parser — LibrePlan REST XML → Dart models ... 063fc2f79d

LibrePlan-connector (#1240) stap 2: pure-Dart XML-parser voor de
LibrePlan REST-API. Parseert order-list (projecten + hiërarchische
taken), resource-list (machines/medewerkers), work-report-list
(timesheets), resource-hours en expense-sheets. Defensief: ontbrekende
velden getolereerd, onbekende velden genegeerd, datums date-only.

Beveiligingscontract §5.4: max depth 100, max 10 MB, geen XXE (de
xml-package resolveert geen externe entities). Fail-closed bij
overschrijding — LibreplanXmlException zonder response body.

11 tests met sample-XML uit de LibrePlan-repo (scripts/rest-clients/).

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>

feat(libreplan): REST-client — GET-only, Basic Auth, NetGuard + pinning ... abb4a22ccd

LibrePlan-connector (#1240) stap 3: de REST-client. Spiegelt
ai_client_service.dart: injecteerbare transport-seam voor tests,
default transport met NetGuard safeResolve(Trusted) + socket-pin +
followRedirects=false + bytecap. GET-only (read-only consumer),
Basic Auth met wachtwoord uit de keychain.

Beveiligingspoort: desktop-only (web fail-closed), HTTPS verplicht
tenzij trustedInternal, logt base URL + username bij fout (nooit
response body — bevat projectdata). Toegevoegd aan
network_sink_guard_test.dart allowlist + pinnedClientCount ratchet.

9 client-tests met mock transport (geen netwerk).

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>

docs(gantt): design-doc voor het gantt-slidetype (#1238) ... 6d6986fae8

Het neergeschreven ontwerp dat de implementatie in #1246 stuurde:
tabel-opslag (vijf vaste kolommen) → Mermaid gantt-DSL, geen nieuwe
afhankelijkheid, geen Scene-layout-engine. Status bijgewerkt naar
"implemented in #1246".

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>

feat(libreplan): converters — LibrePlan models → OciDeck slides ... 83e6d8ef8f

LibrePlan-connector (#1240) stap 4: acht converters die LibrePlan-
Dart-models omzetten naar OciDeck-slide-types:

- Order → gantt (projectplanning, secties, milestones, afhankelijkheden)
- Order → tree (WBS, hiërarchie met tab-diepte)
- Resources → table (resourcelijst)
- Work reports → table (timesheet)
- Resource hours → chart (belasting over tijd, staafdiagram)
- Order → cockpit (projectstatus: voortgang + uren meters)
- Order → timeline (milestones op datum)
- Order → flow (kritieke pad, naïeve DFS-benadering)

Plafonds (ponytail:): gantt kapt af op ganttMaxTasks (30), tree op 50
knopen, tables op 100 rijen. Critical path is een benadering — LibrePlan
berekent CPM server-side, wij hebben alleen de ruwe structuur.

17 converter-tests met sample-XML-data.

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>

feat(libreplan): import-orchestrator + fix library-directive imports ... 4751d68597

LibrePlan-connector (#1240) stap 5: de import-orchestrator die fetch +
parse + convert combineert. LibreplanImporter.import() haalt per slide-
type de benodigde data op, parseert XML, zet om naar slides, en
retourneert LibreplanImportResult met slides + warnings. Eén falende
endpoint stopt de rest niet (niet-fatale warnings).

LibreplanImportOptions: per slide-type aan/uit (default alles aan).
ponytail: één project per import (het eerste); een projectkiezer komt
in de UI-stap.

Fix: verwijderde `library;`-directive uit libreplan_xml/client/converters
— de directive maakte de top-level functies onzichtbaar voor importers
in Dart 3.5+ (anonymous library scope conflict).

6 import-tests met mock client (geen netwerk). Totaal 42 LibrePlan-
tests groen.

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>

feat(libreplan): import-orchestrator — fetch + parse + convert + slides ... a939500570

LibrePlan-connector (#1240) stap 5: de import-orchestrator die fetch +
parse + convert combineert. LibreplanImporter.import() haalt per slide-
type de benodigde data op, parseert XML, zet om naar slides, en
retourneert LibreplanImportResult met slides + warnings. Eén falende
endpoint stopt de rest niet (niet-fatale warnings).

LibreplanImportOptions: per slide-type aan/uit (default alles aan).
ponytail: één project per import (het eerste); een projectkiezer komt
in de UI-stap.

5 import-tests met mock client (geen netwerk).

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>

feat(libreplan): settings UI — tabblad, formulier, verbindingstest ... 731eee0ab3

LibrePlan-connector (#1240) stap 6: het instellingen-tabblad in de
settings-dialoog. Spiegelt het AI-tabblad:

- SettingsSection.libreplan in de zijbalk (zichtbaar zodra module aan)
- Formulier: server-URL, gebruikersnaam, wachtwoord (keychain),
  trustedInternal-schakelaar
- Verbindingstest-knop (GET /ws/rest/resources/)
- Wachtwoord opslag in keychain via SecretStore.*ByKey-methodes
- SettingsNotifier.setLibreplanPassword/deleteLibreplanPassword/
  readLibreplanPassword

Desktop-only (web fail-closed). Wachtwoord nooit in prefs, alleen in
de OS-sleutelhanger. trustedInternal staat plain-HTTP + privé-adressen
door NetGuard — default uit (HTTPS verplicht).

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>

feat(libreplan): import-dialoog — slide-type keuze + insert in deck ... 691bf17f38

LibrePlan-connector (#1240) stap 6b: de import-dialoog die de gebruiker
laat kiezen welke slide-types uit LibrePlan worden gehaald, en de slides
in het huidige deck invoegt via DeckNotifier.insertSlides.

Checkbox per slide-type (gantt, wbs, status, milestones, critical path,
resources, timesheet, resourcebelasting). Voortgangsindicator tijdens
ophalen. Warnings (niet-fatale fouten per endpoint) worden getoond in
de snackbar. De import-knop staat op het LibrePlan-instellingen-tabblad.

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>

docs(libreplan): PRIVACY, SECURITY, USER_GUIDE, CHANGELOG ... 78cad5414b

LibrePlan-connector (#1240) stap 7: documentatie.

- PRIVACY.md: connector is off by default, read-only, desktop-only,
  wachtwoord in keychain, data wordt deel van deck → OciWacht geldt.
- SECURITY.md: NetGuard + pin + GET-only + XML-limieten + fail-closed,
  volgt hetzelfde patroon als WebDAV/S3/Git.
- USER_GUIDE.nl.md: aanzetten, configureren, importeren, beperkingen.
- CHANGELOG.md: development-log entry voor #1240.

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>

fix(libreplan): l10n-dekking, tests, conventions-baselines voor make check ... f7636352c2

Voegt Engelse vertalingen toe voor alle nieuwe Dutch-source strings van de
LibrePlan-connector, en zet de Dutch-source waarden in alle 31 niet-Engelse
taalbestanden (de l10n_untranslated_test eist dat een niet-Engelse taal geen
Engelse zin draagt). Voorbeeld-URL en -gebruikersnaam staan in unchangedInEnglish.

Voegt widget-tests toe voor de import-dialoog en unit-tests voor het
LibreplanSettings-model, zodat de per-file coverage-vloer groen blijft.

Verhoogt de file-size en class-size baselines in check_conventions.dart voor
de twee bestanden die door de LibrePlan-tab groeiden (settings_provider.dart
en settings_dialog.dart), met per-baseline commentaar dat de #1240-wijziging
noemt.

Vervangt bare catch(_) door on FormatException in libreplan_xml.dart, en
Wrap i.p.v. Row in de test-row om overflow bij 200% tekst te voorkomen.

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>

brenno force-pushed feat/libreplan-connector from f7636352c2 to 4460093c5e 2026-08-05 00:34:51 +00:00 Compare

brenno merged commit d1d28c0ff2 into main 2026-08-05 00:36:24 +00:00

brenno referenced this pull request from a commit 2026-08-05 00:36:26 +00:00

Merge pull request 'feat(libreplan): read-only REST-import als specialist-module (#1240)' (#1255) from feat/libreplan-connector into main

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

accepted

Assessed and will be built

  

bug

Something is broken

  

declined

Assessed and will not be built

  

docs

Documentation only

  

duplicate

Already tracked elsewhere

  

enhancement

New feature or improvement request

  

good first issue

Small, self-contained, and a good place to start

  

in-progress

Someone is working on this right now. Do not start it in parallel.

  

needs-info

Waiting for the reporter

  

privacy

Personal data can leak. Handled before anything else.

  

security

A weakness someone could use. Handled before anything else.

  

triage

Awaiting assessment

No labels

accepted

bug

declined

docs

duplicate

enhancement

good first issue

in-progress

needs-info

privacy

security

triage

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

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

LibreKAT/Ocideck!1255

No description provided.