Fix moduleResolution -> customConditions test to actually apply both conditions

[auvred]

webpack, browser string is splitted by ,

TypeScript/tsc/internal/tsoptions/commandlineparser.go

Line 363 in 0681ef7

values := strings.Split(value, ",")

and later isn't trimmed

TypeScript/tsc/internal/tsoptions/commandlineparser.go

Line 367 in 0681ef7

val, err := validateJsonOptionValue(opt.Elements(), v, nil, nil)

However, enum options, like lib are trimmed

TypeScript/tsc/internal/tsoptions/commandlineparser.go

Line 381 in 0681ef7

val, err := convertJsonOptionOfEnumType(opt.Elements(), strings.TrimFunc(v, stringutil.IsWhiteSpaceLike), nil, nil)

---

Other string-list options are passed without spaces (https://github.com/search?q=repo%3Amicrosoft%2FTypeScript+path%3A%2F%5Etsc%5C%2Ftestdata%5C%2Ftests%5C%2Fcases%5C%2F%2F+%2F%5C%2F%5C%2F%5Cs*%40%28types%7CtypeRoots%29.*%2C.*%2F&type=code) so I thought this is the intended way, and this is enough to fix this on test-case level

---

Aside:

> tsc --noEmit --showConfig --customConditions 'webpack, browser' --lib 'es2015, dom'            
{
    "compilerOptions": {
        "customConditions": [
            "webpack",
            " browser"
        ],
        "lib": [
            "es6",
            "dom"
        ],
        "noEmit": true
    },
    "files": [
        "./index.ts"
    ]
}

See how --lib trims space and correctly produces ["es6", "dom"] while --customConditions has ["webpack", " browser"]. Should I open a separate issue for this?

[typescript-automation]

This PR doesn't have any linked issues. Please open an issue that references this PR. From there we can discuss and prioritise.

[Copilot]

Copilot review overview

🟢 Approval recommended

The updated baselines consistently verify both conditions and the expected resolution behavior.

Review effort: Balanced
Findings: None

What changed in this PR

Corrects the custom conditions conformance test so both conditions are applied.

Changes:

• Removes whitespace from the comma-separated directive.
• Updates resolution traces and inferred type baselines.

File	Description
customConditions.ts	Corrects the test directive.
customConditions(resolvepackagejsonexports=true).types	Expects the browser export type.
customConditions(resolvepackagejsonexports=true).trace.json	Records browser-condition resolution.
customConditions(resolvepackagejsonexports=false).trace.json	Records both parsed conditions.

---

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.