refactor(ai): deel de AI-suggestiecontrol en foutafhandeling

brenno commented

2026-08-01 08:26:11 +00:00

Owner

Context
AiSuggestField en ImprovementAiSuggestField bevatten ongeveer 120 regels vrijwel identieke UI en control-flow: vervangen bevestigen, busy-state, lege respons, AiGate/AiRequest-fouten, logging, knop en AI-conceptbadge. Alleen beschikbaarheid, client/service-call en padding verschillen.

Risico
Een wijziging aan provenance, foutmelding of interactie moet nu tweemaal identiek worden uitgevoerd en getest.

Relevante code

lib/widgets/editors/ai_suggest_field.dart:43
lib/widgets/editors/improvement_ai_suggest_field.dart:40

Acceptatie

Extraheer één kleine gedeelde presentatielaag/control met callbacks voor beschikbaarheid en de daadwerkelijke suggest-call.
Houd FindingAiService en ImprovementAiService als aparte domeinservices; introduceer geen generieke servicehiërarchie.
Bestaande widgettests voor beide varianten blijven groen en de gedeelde fout-/confirm-/badgepaden krijgen één bron.

Context AiSuggestField en ImprovementAiSuggestField bevatten ongeveer 120 regels vrijwel identieke UI en control-flow: vervangen bevestigen, busy-state, lege respons, AiGate/AiRequest-fouten, logging, knop en AI-conceptbadge. Alleen beschikbaarheid, client/service-call en padding verschillen. Risico Een wijziging aan provenance, foutmelding of interactie moet nu tweemaal identiek worden uitgevoerd en getest. Relevante code - lib/widgets/editors/ai_suggest_field.dart:43 - lib/widgets/editors/improvement_ai_suggest_field.dart:40 Acceptatie - Extraheer één kleine gedeelde presentatielaag/control met callbacks voor beschikbaarheid en de daadwerkelijke suggest-call. - Houd FindingAiService en ImprovementAiService als aparte domeinservices; introduceer geen generieke servicehiërarchie. - Bestaande widgettests voor beide varianten blijven groen en de gedeelde fout-/confirm-/badgepaden krijgen één bron.

brenno added the

enhancement

accepted

in-progress

labels

2026-08-01 08:26:11 +00:00

brenno commented

2026-08-01 08:36:41 +00:00

Author

Owner

Opgepakt. Tak: codex/refactor-ai-suggest-control-1054. Verwachte reikwijdte: lib/widgets/editors/ai_suggest_field.dart, lib/widgets/editors/improvement_ai_suggest_field.dart, één gedeelde AI-control en betrokken widgettests.

brenno referenced this issue from a pull request that will close it,

2026-08-01 09:42:11 +00:00

refactor(ai): deel de suggestiebediening #1065

brenno referenced this issue from a commit

2026-08-01 09:42:13 +00:00

refactor(ai): deel suggestiebediening (#1054)

brenno referenced this issue from a commit

2026-08-01 09:54:51 +00:00

refactor(ai): deel suggestiebediening (#1054)

brenno closed this issue

2026-08-01 09:58:39 +00:00

brenno referenced this issue from a commit

2026-08-01 09:58:40 +00:00

Merge pull request #1065 from codex/refactor-ai-suggest-control-1054

brenno commented

2026-08-01 09:59:34 +00:00

Author

Owner

Gelanden op main via mergecommit 1c99fdfc17. De gedeelde presentatielaag is ingevoerd met behoud van de aparte FindingAiService- en ImprovementAiService-adapters en de oorspronkelijke marges. Gevalideerd met make check-full: 7.814 tests, 87,5% regeldekking en alle beveiligings-, keten-, referentie- en webpoorten groen. Niet inbegrepen: wijzigingen aan de AI-services, backend/configuratie of Matrix-functionaliteit.

Gelanden op main via mergecommit 1c99fdfc178356e9f2f2adf6f0350056e7e1d8c3. De gedeelde presentatielaag is ingevoerd met behoud van de aparte FindingAiService- en ImprovementAiService-adapters en de oorspronkelijke marges. Gevalideerd met make check-full: 7.814 tests, 87,5% regeldekking en alle beveiligings-, keten-, referentie- en webpoorten groen. Niet inbegrepen: wijzigingen aan de AI-services, backend/configuratie of Matrix-functionaliteit.

Rows
Columns

refactor(ai): deel de AI-suggestiecontrol en foutafhandeling #1054