Remove window active middleware for now

[lramos15]

No description provided.

[Copilot]

Pull request overview

This PR updates the CAPI-backed FetchedValue helper to stop blocking requests when the VS Code window is inactive by removing the windowActiveMiddleware from the advanced-fetch middleware stack.

Changes:

• Removed windowActiveMiddleware import and usage from createCapiClientFetchedValue’s middleware pipeline.
• Left the remaining middleware stack intact (etag, authBlocked, serverErrorBackoff).

Show a summary per file

File	Description
src/platform/networking/common/capiClientFetchedValue.ts	Removes the window-activity gating middleware from the CAPI FetchedValue fetch pipeline.

Copilot's findings

Comments suppressed due to low confidence (2)

src/platform/networking/common/capiClientFetchedValue.ts:111

• Removing windowActiveMiddleware(envService) means envService is now unused in this function, but it remains part of the public signature. This is likely to trigger unused-parameter linting and also leaves the API/doc intent unclear (callers still pass an envService even though window activity is no longer considered). Consider either removing the envService parameter and updating call sites, or renaming it to _envService/adding an explicit comment explaining why it’s intentionally unused until the middleware is reintroduced.

		middleware: [
			etagMiddleware(),
			authBlockedMiddleware(),
			serverErrorBackoffMiddleware(),
		],

src/platform/networking/common/capiClientFetchedValue.ts:111

• Dropping windowActiveMiddleware changes behavior for any FetchedValue created here with keepCacheHot: true: the 60s refresh timer will now continue to hit the network even when IEnvService.isActive is false (previously it would throw a FetchBlockedError and reuse the last-good cached value). If the goal is only to work around incorrect window-activity reporting, consider an alternative guard (e.g., feature-flag this middleware off, or only disable it for specific request types) to avoid increased background traffic when the window is genuinely inactive.

		middleware: [
			etagMiddleware(),
			authBlockedMiddleware(),
			serverErrorBackoffMiddleware(),
		],

• Files reviewed: 1/1 changed files
• Comments generated: 0