Skip to content

staleCache has no lifecycle management in CommentThreadSync #33

Description

@pedrofuentes

Sentinel finding #1 (SNTNL-20260408-081014-c054ab7)

\comment-thread-sync.ts:19,284-287,318\ — \staleCache\ is populated on every successful fetch but never cleared — not by \invalidateCache(), not by \dispose(), and not on PR transitions. It grows monotonically for the session lifetime.

Consider clearing \staleCache\ for a given key when the PR context changes, or clearing all entries in \dispose().

Priority: 🟡 IMPORTANT

Metadata

Metadata

Assignees

No one assigned

    Labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions