Skip to content

Conversation

@johnfav03
Copy link
Contributor

@johnfav03 johnfav03 commented Oct 2, 2025

Implements and tests User Preferences for Auto Imports; the specific preferences included are shown below.

importModuleSpecifierPreference: "shortest" | "project-relative" | "relative" | "non-relative";
importModuleSpecifierEnding: "auto" | "minimal" | "index" | "js";
includePackageJsonAutoImports: "auto" | "on" | "off";
allowRenameOfImportPath: boolean;
autoImportFileExcludePatterns: string[];
autoImportSpecifierExcludeRegexes: string[];
preferTypeOnlyAutoImports: boolean;

@johnfav03 johnfav03 changed the title added userpref gating for package.json auto imports added userpref gating for user preferences for auto imports Nov 13, 2025
@johnfav03 johnfav03 changed the title added userpref gating for user preferences for auto imports added gating for user preferences for auto imports Nov 13, 2025
@johnfav03 johnfav03 requested a review from jakebailey November 21, 2025 18:49
@jakebailey
Copy link
Member

Is the test generator just missing these options such that all tests are new handwritten ones?

@johnfav03
Copy link
Contributor Author

Is the test generator just missing these options such that all tests are new handwritten ones?

Isabel mentioned that I'd need to handwrite tests using BaselineAutoImportsCompletions as codefixProvider hasn't been ported yet

@andrewbranch
Copy link
Member

Codefixes were ported in #2053

@johnfav03 johnfav03 requested a review from jakebailey December 9, 2025 21:17
Copy link
Member

@jakebailey jakebailey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems good, just a few small things.

return true
}
}
// !! SymlinkCache
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The symlink cache should now be available.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't bother changing anything in this file more than you need to

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

alright, ignore me, I guess this is going away soon 😄

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds good, should be ready for final review then

@johnfav03 johnfav03 added this pull request to the merge queue Dec 10, 2025
Merged via the queue into microsoft:main with commit 31ce997 Dec 10, 2025
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants