Skip to content

Commit b4c915d

Browse files
Adapt inline manager to tracking boolean
Use the merged createWithProgress boolean parameter while preserving that inline-script cache entries are not tracked as workspace uv environments. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 39dcc6a3-0fbd-4f36-9d0f-68677de49c27
1 parent c7d13a0 commit b4c915d

2 files changed

Lines changed: 6 additions & 2 deletions

File tree

src/managers/builtin/inlineScriptEnvManager.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -363,7 +363,7 @@ export class InlineScriptEnvManager implements EnvironmentManager, Disposable {
363363
cacheRoot,
364364
envDir.fsPath,
365365
{ install: [...packages], uninstall: [] },
366-
{ trackUvEnvironment: false },
366+
false, // trackUvEnvironment
367367
);
368368
} catch (error) {
369369
this.log.error(`Failed to build inline-script environment: ${this.errorMessage(error)}`);

src/test/managers/builtin/inlineScriptEnvManager.unit.test.ts

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -309,7 +309,11 @@ suite('InlineScriptEnvManager', () => {
309309
lastUsedAt: NOW.toISOString(),
310310
},
311311
]);
312-
assert.deepStrictEqual(createWithProgressStub.firstCall.args[8], { trackUvEnvironment: false });
312+
assert.strictEqual(
313+
createWithProgressStub.firstCall.args[8],
314+
false,
315+
'inline-script cache entries must not be tracked as workspace uv environments',
316+
);
313317
assert.ok(releaseLockStub.calledOnce);
314318
});
315319

0 commit comments

Comments
 (0)