Merge pull request #176 from zardoy/develop

Author: zardoy

.github/workflows/ci.yml

@@ -33,9 +33,6 @@ jobs:
  # if command with timeout already failed on unix, Windows job will be cancelled
  - run: pnpm test-plugin
  if: ${{ runner.os == 'Windows' }}
- - uses: GabrielBB/xvfb-action@v1.6
- with:
- run: pnpm integration-test
  - run: cd out && npx @vscode/vsce package --out ../extension.vsix
  - name: Archive production artifacts
  uses: actions/upload-artifact@v3

.vscode/settings.json

@@ -1,10 +1,13 @@
 {
  "tsEssentialPlugins.autoImport.changeToNamespaceImport": {
- "typescript": {
+ "typescript/lib/tsserverlibrary": {
  "namespace": "ts",
  "addImport": false
  }
  },
+ "typescript.preferences.autoImportFileExcludePatterns": [
+ "./typescript/src/ts.d.ts"
+ ],
  "tsEssentialPlugins.suggestions.ignoreAutoImports": [
  "typescript-full",
  "@volar/language-service/*"
@@ -13,6 +16,13 @@
  "develop"
  ],
  "cSpell.words": [
+ "tsserverlibrary",
  "unpatch"
+ ],
+ "vitest.enable": true,
+ "vitest.commandLine": "pnpm test-plugin",
+ "vitest.showFailMessages": true,
+ "vitest.include": [
+ "typescript/test/**/*.{test,spec}.{js,mjs,cjs,ts,mts,cts,jsx,tsx}"
  ]
 }

CONTRIBUTING.MD

@@ -31,6 +31,8 @@ To start the VS Code plugin extension locally for developing:
 
 #### Unit Tests
 
+> Note: currently project doesn't use integration testing so you can ignore `integration` folder
+
 They are in `typescript/test` and using vitest, so they faster than integration. Feel free to add new tests here. But note that most of tests are completion tests, but I do hope to add more types tests in the future.
 
 To launch them run `pnpm test-plugin`.

README.MD

@@ -3,17 +3,17 @@
 Feature-complete TypeScript plugin that improves every single builtin feature such as completions, definitions, references and so on, and also adds even new TypeScript killer features, so you can work with large codebases faster!
 We make completions more informative. Definitions, references (and sometimes even completions) less noisy. And finally our main goal is to provide most customizable TypeScript experience for IDE features.
 
-TOC:
+## TOC
 
 - [Top Features](#top-features)
 - [Minor Useful Features](#minor-useful-features)
 - [Method Snippets](#method-snippets)
-- [Volar (Vue) support](#vue-support)
+- [Volar (Vue) Support](#vue-support)
 - [Auto Imports](#auto-imports)
 - [Type Driven Completions](#type-driven-completions)
-- [Rename Features](#rename-features)
 - [Special Commands List](#special-commands-list)
 - [Contributed Code Actions](#contributed-code-actions)
+- [Rename Features](#rename-features)
 - [Even Even More](#even-even-more)
 
 > *Note*: You can disable all optional features with `> Disable All Optional Features` command right after install.
@@ -133,6 +133,8 @@ This also makes plugin work in Volar's takeover mode!
 
 There is web-only feature: fix clicking on relative `import` paths.
 
+Alternative you can try to use [vscode-typescript-web extension](https://github.com/volarjs/vscode-typescript-web/) for better web intellisense.
+
 ### `in` Keyword Suggestions
 
 [Demo](https://github.com/zardoy/typescript-vscode-plugins/pull/19)
@@ -319,7 +321,7 @@ function Foo() {
 }
 ```
 
-`tsEssentialPlugins.methodSnippetsInsertText`:
+`tsEssentialPlugins.methodSnippets.previewSignature`:
 
 Optionally resolve insertText of all completion at suggest trigger:
 
@@ -448,9 +450,7 @@ You can add this to `keybindings.json` to disable previewing before renaming:
 }
 ```
 
-Another options that is accepted is
-
-> Note: VS Code has builtin setting to disable introducing aliases (e.g. for imports & object properties)
+> Note: VS Code has builtin setting to disable introducing aliases (e.g. for imports & object properties): `typescript.preferences.useAliasesForRenames`
 
 ## Special Commands List
 

buildTsPlugin.mjs

@@ -1,6 +1,7 @@
 //@ts-check
 import buildTsPlugin from '@zardoy/vscode-utils/build/buildTypescriptPlugin.js'
 import { build, analyzeMetafile } from 'esbuild'
+import fs from 'fs'
 
 const enableWatch = process.argv.includes('--watch')
 await build({
@@ -29,6 +30,22 @@ const result = await buildTsPlugin('typescript', undefined, undefined, {
  js: 'let ts, tsFull;',
  // js: 'const log = (...args) => console.log(...args.map(a => JSON.stringify(a)))',
  },
+ plugins: [
+ {
+ name: 'watch-notifier',
+ setup(build) {
+ const writeStatus = (/** @type {number} */ signal) => {
+ fs.writeFileSync('./out/build_plugin_result', signal.toString())
+ }
+ build.onStart(() => {
+ writeStatus(0)
+ })
+ build.onEnd(({ errors }) => {
+ writeStatus(errors.length ? 2 : 1)
+ })
+ },
+ },
+ ],
 })
 
 // @ts-ignore

package.json

@@ -66,6 +66,10 @@
  {
  "command": "getArgumentReferencesFromCurrentParameter",
  "title": "Get Argument References from Current Parameter"
+ },
+ {
+ "command": "printPerformanceMemoryInfo",
+ "title": "Print Performance & Memory Info"
  }
  ],
  "keybindings": [

src/autoPluginReload.ts

@@ -0,0 +1,27 @@
+import * as vscode from 'vscode'
+import fs from 'fs'
+import { extensionCtx } from 'vscode-framework'
+
+export default () => {
+ const status = vscode.window.createStatusBarItem('plugin-auto-reload', vscode.StatusBarAlignment.Left, 1000)
+ status.show()
+ const watcher = vscode.workspace.createFileSystemWatcher(new vscode.RelativePattern(extensionCtx.extensionUri, 'build_plugin_result'))
+ const updateStatus = uri => {
+ const newStatus = fs.readFileSync(uri.fsPath, 'utf8')
+ if (newStatus === '1') {
+ void vscode.commands.executeCommand('typescript.restartTsServer')
+ status.text = 'Latest'
+ }
+
+ if (newStatus === '0') {
+ status.text = 'Rebuilding'
+ }
+
+ if (newStatus === '2') {
+ status.text = 'Build errored'
+ }
+ }
+
+ watcher.onDidChange(updateStatus)
+ watcher.onDidCreate(updateStatus)
+}

src/configurationType.ts

@@ -87,7 +87,8 @@ export type Configuration = {
  * Format of this setting is very close to `jsxCompletionsMap` setting:
  * `path#symbol` (exact) or `path/*#symbol` (`#symbol` part can be omitted)
  *
- * Note: Please use `javascript`/`typescript.preferences.autoImportFileExcludePatterns` when possible, to achieve better performance!
+ * **Note**: Please use builtin `javascript`/`typescript.preferences.autoImportFileExcludePatterns` when possible, to achieve better performance!
+ * The builtin function expects relative paths to .d.ts / .ts files that contains declarations.
  *
  * e.g. instead of declaring `@mui/icons-material` here, declare `node_modules/@mui/icons-material` in aforementioned setting.
  *
@@ -99,7 +100,7 @@ export type Configuration = {
  * - `path/*#join` - ignore path, path/posix and path/win32, but only join symbol
  * - `path/*#join,resolve` - ignore path, path/posix and path/win32, but only join and resolve symbol
  *
- * - jquery/* - ignore absolutely all auto imports from jquery, even if it was declared virtually (declare module)
+ * - jquery/* - ignore absolutely all auto imports from jquery, even if it was declared using ambient declaration (`declare module`)
  * @default []
  */
  'suggestions.ignoreAutoImports': string[]
@@ -384,7 +385,7 @@ export type Configuration = {
  * Recommended to try!
  * @default disable
  */
- methodSnippetsInsertText: 'disable' | 'only-local' | 'all'
+ 'methodSnippets.previewSignature': 'disable' | 'only-local' | 'all'
  /**
  * ```ts
  * const example = ({ a }, b?, c = 5, ...d) => { }
@@ -571,7 +572,7 @@ export type Configuration = {
  'experiments.excludeNonJsxCompletions': boolean
  /**
  * Wether to change function completions to function kind
- * @deprecated Use `methodSnippetsInsertText` instead
+ * @deprecated Use `methodSnippets.previewSignature` instead
  * @default false
  */
  'experiments.changeKindToFunction': boolean

src/extension.ts

@@ -96,6 +96,10 @@ export const activateTsPlugin = (tsApi: { configurePlugin; onCompletionAccepted
 
  figIntegration()
  vueVolarSupport()
+
+ if (process.env.PLATFORM === 'node' && process.env.NODE_ENV === 'development') {
+ require('./autoPluginReload').default()
+ }
 }
 
 export const activate = async () => {

src/migrateSettings.ts

@@ -33,6 +33,13 @@ export default () => {
  mustBePrimitive: false,
  },
  },
+ {
+ rename: {
+ old: 'methodSnippetsInsertText',
+ new: 'methodSnippets.previewSignature',
+ mustBePrimitive: false,
+ },
+ },
  {
  async detect(configuration) {
  return !!(await migrateSettingValues(configuration, true))