[Eduardo Villalpando Mello (edvilme)]

Skips method-based capability inference (legacy) when a manager advertises its capabilities.

[Copilot]

🟢 Approval recommended

The focused guards match the intended behavior, preserve legacy fallback, and have regression coverage for both manager types.

No open issues 🎉

What changed in this PR

This PR keeps explicit manager capability declarations separate from legacy method-based inference in the Python Environments extension.

Changes:

• Skip legacy inference for package and environment managers that advertise capabilities.
• Add regression tests for both manager types, verifying that available methods do not trigger legacy support detection.

File	Description
src/​test/​legacyCapabilities.unit.test.ts	Adds regression coverage for managers with advertised capabilities.
src/​legacyCapabilities.ts	Adds guards to bypass legacy inference when capabilities are declared.

_{Review effort: Balanced}

[Bill Schnurr (bschnurr)]

🔒 Automated review in progress — Bill Schnurr (@bschnurr) is auto-reviewing this PR.

[Bill Schnurr (bschnurr)]

Result: 🔴 could-not-verify

Verification details

Verification: The relevant tests could not be fully run in the isolated environment; this review is not fully verified.

Details: Verification could not complete: timeout must be None or at least 2073600 seconds (24 days); shorter completion deadlines are unsupported.

Test runs: none recorded.

[Bill Schnurr (bschnurr)]

Approved via Review Center.

[Heejae Chang (heejaechang)]

Approved via Review Center.