Skip to content

PR-Review #781: Speicherfehler-Pfad des AI-Chat-Stores (Quota/blockierter Storage) ist ungetestet #791

Closed

Description

@Sev7eNup

PR: #781 · Autor: @Sev7eNup · Status: merged

  • mittel – src/nodepilot-ui/src/stores/chatPersistenceStatus.ts:9-24 und src/nodepilot-ui/src/components/ai/ChatPersistenceNotice.tsx: Der Kern der Änderung ist, dass der Chat weiterläuft, wenn sessionStorage voll oder blockiert ist. In dem Fall setzt der Store unavailable, und der Hinweis erscheint. Dafür gibt es keinen Test: Kein Test lässt sessionStorage.setItem/getItem werfen (z. B. QuotaExceededError), und ChatPersistenceNotice wird nirgends gerendert. Der einzige neue Store-Test prüft das Größenbudget. Fällt das try/catch weg, bleibt alles grün, und ein voller Storage wirft wieder in den Sende-Pfad.
    Vorschlag: Einen Store-Test mit gemocktem, werfendem setItem ergänzen, der prüft, dass updateMessages nicht wirft und unavailable === true ist. Dazu einen kleinen Render-Test für den Hinweis.

Automatischer Review-Fund des nächtlichen PR-Review-Laufs – braucht Triage.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions