[Kwaliteit][P3] AI-statuscontrole stopt na de eerste instellingswijziging #2635

New issue

Closed

opened 2026-10-28 16:31:53 +00:00 by brenno · 1 comment

brenno commented

2026-10-28 16:31:53 +00:00

Owner

Symptoom

Het AI-statuslampje ververst niet meer nadat je één keer een AI-instelling hebt gewijzigd (of toestemming hebt gegeven); het blijft de laatste uitkomst tonen tot je erop klikt of de app herstart.

Oorzaak

AiStatusNotifier.build (lib/state/ai_status_provider.dart:70-98):

ref.onDispose(() => _timer?.cancel());   // regel 73
...
_timer ??= Timer.periodic(               // regel 92
  kAiStatusRefreshInterval,
  (_) => unawaited(_check(generation)),
);

Bij een herbouw (gewijzigde aiSettings of hasAccepted) annuleert onDispose de timer, maar zet _timer niet op null. De ??= maakt daarna geen nieuwe timer, dus de periodieke controle is weg. Daarbij legt de closure de generation van de éérste build vast; ook een timer die zou blijven lopen, zou door generation != _generation niets meer doen.

Oplossing

ref.onDispose(() {
  _timer?.cancel();
  _timer = null;
});
...
_timer = Timer.periodic(kAiStatusRefreshInterval, (_) => unawaited(_check(_generation)));

Toets

fake_async: instelling wijzigen, kAiStatusRefreshInterval laten verstrijken → testConnection is opnieuw aangeroepen.

Onderdeel van de prestatie- en stabiliteitsinspectie van 28-10-2026 (EPIC #2637).

## Symptoom Het AI-statuslampje ververst niet meer nadat je één keer een AI-instelling hebt gewijzigd (of toestemming hebt gegeven); het blijft de laatste uitkomst tonen tot je erop klikt of de app herstart. ## Oorzaak `AiStatusNotifier.build` (`lib/state/ai_status_provider.dart:70-98`): ```dart ref.onDispose(() => _timer?.cancel()); // regel 73 ... _timer ??= Timer.periodic( // regel 92 kAiStatusRefreshInterval, (_) => unawaited(_check(generation)), ); ``` Bij een herbouw (gewijzigde `aiSettings` of `hasAccepted`) annuleert `onDispose` de timer, maar zet `_timer` niet op `null`. De `??=` maakt daarna geen nieuwe timer, dus de periodieke controle is weg. Daarbij legt de closure de `generation` van de éérste build vast; ook een timer die zou blijven lopen, zou door `generation != _generation` niets meer doen. ## Oplossing ```dart ref.onDispose(() { _timer?.cancel(); _timer = null; }); ... _timer = Timer.periodic(kAiStatusRefreshInterval, (_) => unawaited(_check(_generation))); ``` ## Toets `fake_async`: instelling wijzigen, `kAiStatusRefreshInterval` laten verstrijken → `testConnection` is opnieuw aangeroepen. Onderdeel van de prestatie- en stabiliteitsinspectie van 28-10-2026 (EPIC #2637).

brenno added the

good first issue

bug

labels

2026-10-28 16:31:53 +00:00

brenno referenced this issue

2026-10-28 16:32:34 +00:00

[EPIC] Prestatie- en stabiliteitsinspectie 28-10-2026: traag bij groot, start niet, loopt vast #2637

brenno added the

in-progress

label

2026-10-29 00:00:14 +00:00