Skip to main content
← Back to list
01Issue
BugShippedSwamp CLIPublic
Assigneesstack72

Relationships

#1868 Windows test failures in resolveManagedConfigPaths after managedConfig merge

Opened by stack72 · 8/27/2026· Shipped 8/27/2026

Description

After merging the managedConfig PR (#2288, commit 661f4896), the Windows CI job fails with 5 test failures in resolveManagedConfigPaths tests in src/cli/repo_context_test.ts.

All 5 failures are the same root cause: tests use hardcoded Unix-style expected paths (e.g. /repo/extensions/models/upstream_extensions.json) but on Windows the actual resolved paths include the drive letter prefix (e.g. D:/repo/extensions/models/upstream_extensions.json).

Failing tests

All in src/cli/repo_context_test.ts:

  1. resolveManagedConfigPaths: null marker returns default lockfile path (line 2210)
  2. resolveManagedConfigPaths: marker without datastore returns default paths (line 2218)
  3. resolveManagedConfigPaths: managedConfig=false returns default paths (line 2234)
  4. resolveManagedConfigPaths: managedConfig=true without sentinel falls back to default paths (line 2324)
  5. resolveManagedConfigPaths: custom modelsDir is used when managedConfig=false (line 2344)

Error pattern

AssertionError: Values are not equal.

    [Diff] Actual / Expected

-   D:/repo/extensions/models/upstream_extensions.json
+   /repo/extensions/models/upstream_extensions.json

The tests are already using assertPathEquals from path_test_helpers.ts, but the expected values are hardcoded with Unix forward-slash paths that don't account for the Windows drive letter prefix.

Steps to reproduce

Run the test suite on Windows (or check the CI run):

Fix

The test expectations need to use @std/path join() or similar to construct platform-aware expected paths instead of hardcoded Unix-style string literals.

Environment

  • OS: Windows (GitHub Actions runner)
  • Deno: CI version
  • Commit: 661f4896 (feat(datastore): add managedConfig)
02Bog Flow
OPENTRIAGEDIN PROGRESSSHIPPED+ 1 MOREASSIGNED+ 2 MOREREVIEW+ 21 MOREPR_MERGED+ 2 MORESESSION_SUMMARIZED

Shipped

8/27/2026, 6:18:25 PM

Click a lifecycle step above to view its details.

03Sludge Pulse
stack72 assigned stack728/27/2026, 4:59:52 PM

Sign in to post a ripple.