microsoft / microsoft/aspire

Remove AI assistant references from VS Code extension

Open Beginner friendly
#18,751 1 comment 0 reactions 0 assignees View on GitHub
area-vscode
Dominant language
C#
Stars
6.3k
Forks
991
Avg merge
2d 15h
Merged PRs (30d)
196

Description

After removing the AI assistant feature from the dashboard (#18726), the VS Code extension still has leftover references that should be cleaned up:

1. **`extension/src/debugger/languages/dotnet.ts`** — Injects `ASPIRE_DASHBOARD_AI_DISABLED=true` env var for every AppHost launch. The dashboard no longer recognizes this variable.

2. **`extension/src/debugger/languages/maui.ts`** — Lists `ASPIRE_DASHBOARD_AI_DISABLED` in `isKnownMauiApplicationEnvironmentVariable`.

3. **`extension/src/dcp/DashboardTelemetryPassthrough.ts`** — Allowlists 5 `Aspire.Dashboard.AIAssistant.*` property keys in `ALLOWED_DASHBOARD_PROPERTY_KEYS` and `ALLOWED_STRING_ARRAY_PROPERTY_KEYS`.

4. **`extension/src/test/dashboardTelemetryPassthrough.test.ts`** — Tests exercise the removed AI property keys.

These are harmless (they emit a now-ignored env var and allowlist telemetry keys that will never arrive), but should be removed to keep the extension in sync with the dashboard.

Contributor guide

Open the contributing guide

Research direction

Start with the four paths named in the issue: the .NET and MAUI debugger files, extension/src/dcp/DashboardTelemetryPassthrough.ts, and extension/src/test/dashboardTelemetryPassthrough.test.ts. Review the AI-related environment-variable and telemetry-key references, then run the dashboard telemetry passthrough tests. Done means the obsolete references and their tests are removed without breaking the remaining extension tests.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
tooling
Issue type
Refactor
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Quiet
Clarity
Clearly specified
Newbie friendliness
78/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.