Skip to content

[Maintainability] Centralize Global Ask session storage access #359

Description

@seonghobae

Problem

AskAgentPanel defines GLOBAL_ASK_SESSION_STORAGE_KEY but two new write/remove paths still repeat the literal "lineageweave.globalAskSessionId". The values currently match, so runtime behavior is correct, but a future key migration could silently split read, write, and invalidation behavior.

Required change

  • Route every read, write, and removal through the shared constant or a small typed storage adapter.
  • Keep current 404 session-recovery and 409 stale-session retry behavior.
  • Do not clear unrelated session-storage entries.
  • Do not change the server session contract or remove recovery behavior.

Acceptance

  • a source/behavior regression prevents a second hardcoded key from being introduced;
  • successful answers persist the returned session ID;
  • 404 recovery removes the same key and retries once without a session;
  • 409 recovery removes the same key and retries once;
  • malformed or unavailable storage fails safely without losing the current Ask answer;
  • frontend tests, lint, production build, and Storybook pass.

Origin

Tracked from the exact-head review of PR #342. It is a maintainability hardening item, not a reason to roll back the Global Ask session-recovery feature.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions