Skip to content

Extension: key analysis cache by workspace path for multi-root support #56

@digitarald

Description

@digitarald

The extension's cachedAnalysis in vscode-extension/src/commands/analyze.ts is a single global variable. With pickWorkspacePath() enabling multi-root selection, if a user analyzes one folder then runs commands against a different folder, the cached analysis may be reused incorrectly (areas/apps from the wrong root).

Current: let cachedAnalysis: RepoAnalysis | undefined
Proposed: Map<string, RepoAnalysis> keyed by fsPath, with invalidation when the picked path differs from what's cached.

This affects getCachedAnalysis() consumers in:

  • vscode-extension/src/commands/readiness.ts
  • vscode-extension/src/commands/instructions.ts

Ref: PR #55 review comments analyze.ts, readiness.ts

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions