Skip to content

resolve fails on Windows #1182

Open
Open

Description

@MikeMcC399

Situation

Running citgm resolve fails on Node.js 22-26 (windows)

All 825 tests pass, however there are failures in posttest
(see attached logs resolve-1.22.13-windows-citgm.log)

Item Status
npm package https://www.npmjs.com/package/resolve
npm status active
last published 1.22.13 Oct 7, 2026
weekly downloads 233,891,835
repo location https://github.com/browserify/resolve
repo last update to default branch Oct 7, 2026
repo status active
repo CI status runs in Node.js all versions, ubuntu only
lookup - skip -
lookup maintainer 1 ljharb

Repo CI https://github.com/browserify/resolve/blob/main/.github/workflows/node-twenties.yml runs in Node.js >=20 on ubuntu.

Steps to reproduce

Run https://github.com/nodejs/citgm/actions/workflows/test-module.yml

Module to test: resolve
CITGM repository to checkout: nodejs/citgm
CITGM GitHub ref to checkout: main

Alternatively on Windows 11 26H2, Node.js 24.21.0 LTS

npm install citgm@latest -g
citgm resolve

Result

Fails on Node.js 22-26 (windows)

Node.js npm macos ubuntu windows
22.23.3 10.9.9 PASS PASS FAIL
24.21.0 11.19.0 PASS PASS FAIL
26.10.0 11.19.1 PASS PASS FAIL

References

citgm/lib/lookup.json

Lines 356 to 360 in 4d3d256

"resolve": {
"prefix": "v",
"maintainers": "ljharb",
"scripts": ["tests-only", "posttest"]
},

cc: @ljharb

Activity

  1. ljharb commented on Oct 7, 2026

    @ljharb
    SponsorMember

    Looks like the issue is in npm run test:multirepo (which is part of posttest). In the short term, could perhaps only tests-only be invoked on windows?

    In the longer term, I'll need to find a Windows instance to test this on so I can fix it.

  2. MikeMcC399 commented on Oct 8, 2026

    @MikeMcC399
    ContributorAuthor

    Looks like the issue is in npm run test:multirepo (which is part of posttest). In the short term, could perhaps only tests-only be invoked on windows?

    I don't believe there is a way to remove posttest only on Windows. It would have to be removed for all OS. Will you submit a PR to make this change?

  3. ljharb commented on Oct 8, 2026

    @ljharb
    SponsorMember

    sure

  4. ljharb commented on Oct 8, 2026

    @ljharb
    SponsorMember

    I added such a way: #1191

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions