coc-vscode-loader 1.2.5 → 1.2.7

package/README.md

@@ -37,9 +37,10 @@ npm install coc-vscode-loader
 | `x` | Toggle mark package for batch operations |
 | `f` | Cycle filter: all → installed → available |
 | `s` | Cycle sort: default → name → status → type |
+| `j` / `k` | Scroll through packages (virtual scroll) |
 | `gg` | Jump to first package |
 | `G` | Jump to last package |
-| `<CR>` | Toggle details (commit / type / source) or install log |
+| `<CR>` | Open detail popup (info / install log with syntax highlights) |
 | `/` | Search filter |
 | `q` | Close (auto `:CocRestart` if changes detected) |
 | `<Esc>` | Help→Search→Clear marks→Cancel|Close |
@@ -60,12 +61,13 @@ npm install coc-vscode-loader
 
 - **Real conversion pipeline** — git clone → converter → npm install → esbuild → register to coc
 - **Auto-fetch registry** — remote registry fetched in background when TUI opens, no manual refresh needed
+- **Virtual scrolling** — `j`/`k` smooth scroll through packages, handles 100k+ registry entries
 - **Incremental cache** — source/ keeps git repo, updates via git pull only
 - **Commit tracking** — records commit SHA after install, visible in detail view
 - **Update check** — `C` key compares against remote HEAD, shows `↑` when outdated
 - **Auto restart** — `:CocRestart` triggered automatically on close when changes detected
 - **Manual registry update** — `:CocCommand loader.updateRegistry` also available for re-fetch
-- **Install logs** — real command output per step, expandable
+- **Detail popup** — `<CR>` opens centered float window with package info or live install log (syntax highlighted, auto-scroll to latest)
 - **Mark & batch** — `x` toggle mark, visual indicator, `D` clean orphaned packages
 - **Filter & sort** — `f` cycle view filter, `s` cycle sort order (name/status/type)
 - **Concurrency limit** — max 3 parallel operations for `U` (Update All)

package/converter/README.md

@@ -18,11 +18,25 @@ cd ~/.config/coc/extensions && npm install /path/to/coc-ext
 
 ## Verified conversions
 
-| Plugin | Type | Auto-detected | Build | Working | Notes |
-|--------|------|---------------|-------|---------|-------|
-| Volar (Vue) | TS bridge | `@vue/language-server` + `typescript` | ✅ | ✅ | Requires modified coc-tsserver |
-| Prisma | Pure LSP | `@prisma/language-server` | ✅ | ✅ | Auto-detects bin entry |
-| HTML CSS Support | Direct API | — | ✅ | ✅ | Handles API differences |
+| Plugin | Type | Notes |
+|--------|------|-------|
+| Volar (Vue) | TS bridge | Requires modified coc-tsserver |
+| Prisma | Pure LSP | Auto-detects bin entry |
+| HTML CSS Support | Direct API | Handles API differences |
+| Deno | Pure LSP | Binary server download |
+| TOML (Taplo) | Pure LSP | Binary server download |
+| Ansible | Pure LSP | npm package server + pip install |
+| YAML | Pure LSP | npm package server |
+| Tailwind CSS | Pure LSP | npm package server, bin entry |
+| Biome | Pure LSP | Binary server download |
+| Stylelint | Pure LSP | npm package server |
+| Prettier | Direct API | Source transforms |
+| Svelte | Pure LSP | npm package server |
+| Astro | Pure LSP | npm package server |
+| Lua | Pure LSP | npm package server |
+| gitignore | Direct API | Source transforms |
+
+See the [registry](https://github.com/coc-plugin/coc-vscode-registry) for the full list and latest status.
 
 ### Plugin types
 
@@ -77,21 +91,21 @@ const PRESETS = {
 `convert.ts` only calls `getActivePresets()` + `generateBridgeCode()`, it never touches bridge logic directly.
 Adding a new bridge type = add a new preset in `presets.ts`, no changes to main flow.
 
-See [coc-vscode-registry/docs/converter-design-v2.md](https://github.com/coc-plugin/coc-vscode-registry/blob/main/docs/converter-design-v2.md).
+See [../docs/converter-design-v2.md](../docs/converter-design-v2.md).
 
 ## File structure
 
 | File | Lines | Description |
 |------|-------|-------------|
-| `src/cli.ts` | 28 | CLI entry |
-| `src/convert.ts` | 484 | Main flow + template generation + API replacement |
-| `src/scanner.ts` | 136 | API scanner + plugin classification |
-| `src/transforms/import-mapping.ts` | 47 | Import replacement |
+| `src/cli.ts` | 59 | CLI entry |
+| `src/convert.ts` | 461 | Main flow + template generation + API replacement |
+| `src/scanner.ts` | 52 | API scanner + plugin classification |
+| `src/transforms/import-mapping.ts` | 193 | Import replacement |
 | `src/transforms/language-client.ts` | 48 | LanguageClient adaptation |
-| `src/transforms/class-to-factory.ts` | 54 | new Xxx() → Xxx.create() |
-| `src/transforms/provider-register.ts` | 55 | Provider registration signature fixes |
-| `src/transforms/enum-offset.ts` | 49 | Enum value offset annotations |
-| **Total** | **~870** | |
+| `src/transforms/class-to-factory.ts` | 53 | new Xxx() → Xxx.create() |
+| `src/transforms/provider-register.ts` | 61 | Provider registration signature fixes |
+| `src/transforms/enum-offset.ts` | 32 | Enum value offset annotations |
+| **Total** | **~959** | |
 
 ## Handled API differences
 

package/converter/package-lock.json

@@ -1,12 +1,12 @@
 {
   "name": "converter",
-  "version": "1.2.5",
+  "version": "1.2.7",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "converter",
-      "version": "1.2.5",
+      "version": "1.2.7",
       "dependencies": {
         "commander": "^15.0.0",
         "ts-morph": "^28.0.0",

package/converter/package.json

@@ -1,6 +1,6 @@
 {
   "name": "converter",
-  "version": "1.2.5",
+  "version": "1.2.7",
   "private": true,
   "description": "vscode → coc.nvim converter prototype",
   "type": "module",

package/converter/src/convert.ts

@@ -326,6 +326,7 @@ export async function convert(opts: ConvertOptions): Promise<void> {
         command: c.command,
         title: c.title,
       })) || undefined,
+      snippets: origPkg.contributes?.snippets || undefined,
       ...(tsPlugins.length > 0 ? {
         typescriptServerPlugins: tsPlugins.map(p => ({
           ...p,
@@ -338,6 +339,7 @@ export async function convert(opts: ConvertOptions): Promise<void> {
   // Clean null fields
   if (!pkg.contributes?.configuration) delete (pkg.contributes as any).configuration
   if (!pkg.contributes?.commands) delete (pkg.contributes as any).commands
+  if (!pkg.contributes?.snippets) delete (pkg.contributes as any).snippets
   if (Object.keys(pkg.contributes).length === 0) delete (pkg as any).contributes
 
   fs.writeFileSync(path.join(output, 'package.json'), JSON.stringify(pkg, null, 2))

package/converter/src/scanner.ts

@@ -23,7 +23,7 @@ export function scan(dir: string): ScanResult {
     const apis: string[] = []
     const relative = path.relative(dir, filePath)
 
-    if (content.includes("from 'vscode'") || content.includes('from "vscode"') || content.includes('require("vscode")')) {
+    if (content.includes("from 'vscode'") || content.includes('from "vscode"') || content.includes('require("vscode")') || content.includes("require('vscode')")) {
       apis.push('vscode')
     }
 

package/converter/src/steps/index.ts

@@ -3,6 +3,7 @@ import { languageClientGenerator } from './language-client.js'
 import { sourceGenerator } from './source.js'
 import { bridgeGenerator } from './bridge.js'
 import { markUnsupportedGenerator } from './mark-unsupported.js'
+import { snippetsGenerator } from './snippets.js'
 
 const REGISTRY: Record<string, StepGenerator> = {}
 
@@ -27,3 +28,4 @@ registerGenerator(languageClientGenerator)
 registerGenerator(sourceGenerator)
 registerGenerator(bridgeGenerator)
 registerGenerator(markUnsupportedGenerator)
+registerGenerator(snippetsGenerator)

package/converter/src/steps/language-client.ts

@@ -108,7 +108,7 @@ ${ls.verbose ? `    console.log('[${escapeStr(id)}] creating LanguageClient')\n`
         {
           documentSelector: ${docSelectorCode},
           outputChannelName: '${escapeStr(description)}',
-          ${ls.initializationOptions ? `initializationOptions: ${ls.initializationOptions},` : ''}
+          ${ls.initializationOptions ? `initializationOptions: ${ls.initializationOptions.replace(/`/g, '\\`').replace(/\$\{/g, '\\${')},` : ''}
         },
       )
       context.subscriptions.push({ dispose: () => c.stop() })
@@ -117,23 +117,27 @@ ${ls.verbose ? `    console.log('[${escapeStr(id)}] creating LanguageClient')\n`
     }
 
 ${ls.verbose ? `    console.log('[${escapeStr(id)}] registering LanguageClient')\n` : ''}\
-    let client: LanguageClient
+    let clients: LanguageClient[]
     if (${multiRoot ? 'workspace.workspaceFolders && workspace.workspaceFolders.length > 1' : 'false'}) {
 ${ls.verbose ? `      console.log('[${escapeStr(id)}] multiRoot mode')\n` : ''}\
-      for (const folder of workspace.workspaceFolders) {
-        client = createClient()
-        client.start()
-      }
+      clients = workspace.workspaceFolders.map(folder => {
+        const c = createClient()
+        c.start()
+        return c
+      })
     } else {
-      client = createClient()
+      const c = createClient()
 ${ls.verbose ? `      console.log('[${escapeStr(id)}] starting client')\n` : ''}\
-      client.start()
+      c.start()
+      clients = [c]
     }
 
     context.subscriptions.push(
       commands.registerCommand('${escapeStr(pluginName)}.restart', async () => {
-        await client.stop()
-        await client.start()
+        for (const c of clients) {
+          await c.stop()
+          await c.start()
+        }
       }),
     )
   } catch (e: any) {

package/converter/src/steps/snippets.ts

@@ -0,0 +1,122 @@
+import * as fs from 'fs'
+import * as path from 'path'
+import { execFileSync } from 'child_process'
+import { StepGenerator, StepContext, SnippetsStep, StepResult } from '../types.js'
+
+export const snippetsGenerator: StepGenerator = {
+  type: 'snippets',
+
+  generate(ctx: StepContext, step: any): StepResult {
+    const ss = step as SnippetsStep
+    const { input, output, origPkg, verbose } = ctx
+
+    const contributedSnippets: Array<{ language: string; path: string }> =
+      origPkg.contributes?.snippets || []
+
+    if (contributedSnippets.length === 0 && !ss.languages) {
+      throw new Error('snippets step: source package.json has no contributes.snippets, and no languages specified in step config')
+    }
+
+    // Collect unique (sourcePath → [languages]) mappings
+    const fileToLanguages = new Map<string, string[]>()
+    if (ss.languages) {
+      for (const lang of ss.languages) {
+        const entry = contributedSnippets.find(s => s.language === lang)
+        if (entry) {
+          const langs = fileToLanguages.get(entry.path) || []
+          langs.push(lang)
+          fileToLanguages.set(entry.path, langs)
+        } else {
+          const defaultPath = `./snippets/${lang}.json`
+          const fp = path.join(input, defaultPath)
+          if (fs.existsSync(fp)) {
+            const langs = fileToLanguages.get(defaultPath) || []
+            langs.push(lang)
+            fileToLanguages.set(defaultPath, langs)
+          } else if (verbose) {
+            console.warn(`  snippets: no snippet file found for language "${lang}", skipping`)
+          }
+        }
+      }
+    } else {
+      for (const s of contributedSnippets) {
+        const langs = fileToLanguages.get(s.path) || []
+        langs.push(s.language)
+        fileToLanguages.set(s.path, langs)
+      }
+    }
+
+    if (fileToLanguages.size === 0) {
+      throw new Error('snippets step: no snippet files found to copy')
+    }
+
+    // Create output directories and copy files to original relative paths
+    const srcDir = path.join(output, 'src')
+    fs.mkdirSync(srcDir, { recursive: true })
+
+    const generatedFiles: Array<{ path: string; content: string }> = []
+    let copiedCount = 0
+    const allLanguages: string[] = []
+
+    for (const [sourceRelPath, languages] of fileToLanguages) {
+      const sourceFile = path.join(input, sourceRelPath)
+      if (!fs.existsSync(sourceFile)) {
+        if (verbose) console.warn(`  snippets: source file not found: ${sourceFile}, skipping`)
+        continue
+      }
+      const dest = path.join(output, sourceRelPath)
+      fs.mkdirSync(path.dirname(dest), { recursive: true })
+      fs.copyFileSync(sourceFile, dest)
+      copiedCount++
+      allLanguages.push(...languages)
+      if (verbose) console.log(`  snippets: copied ${sourceRelPath} (${languages.join(', ')})`)
+    }
+
+    if (copiedCount === 0 && ss.build) {
+      // Run build script to generate snippet files (e.g. node merge.js)
+      if (verbose) console.log(`  snippets: running build: ${ss.build}`)
+      execFileSync('npm', ['install', '--legacy-peer-deps'], { cwd: input, stdio: verbose ? 'inherit' : 'pipe' })
+      const [cmd, ...args] = ss.build.split(' ')
+      execFileSync(cmd, args, { cwd: input, stdio: verbose ? 'inherit' : 'pipe' })
+      // Retry copying
+      for (const [sourceRelPath, languages] of fileToLanguages) {
+        const sourceFile = path.join(input, sourceRelPath)
+        if (fs.existsSync(sourceFile)) {
+          const dest = path.join(output, sourceRelPath)
+          fs.mkdirSync(path.dirname(dest), { recursive: true })
+          fs.copyFileSync(sourceFile, dest)
+          copiedCount++
+          allLanguages.push(...languages)
+          if (verbose) console.log(`  snippets: copied ${sourceRelPath} (${languages.join(', ')})`)
+        }
+      }
+    }
+
+    if (copiedCount === 0) {
+      throw new Error('snippets step: no snippet files were copied')
+    }
+
+    // Generate empty src/index.ts
+    const indexContent = `\
+import { ExtensionContext } from 'coc.nvim'
+
+export function activate(context: ExtensionContext): void {
+  // coc-snippets discovers snippets via package.json's contributes.snippets
+}
+`
+    generatedFiles.push({ path: 'src/index.ts', content: indexContent })
+
+    const activationEvents = [...new Set(allLanguages)].map(l => `onLanguage:${l}`)
+
+    if (verbose) {
+      console.log(`  snippets: ${copiedCount} files, ${new Set(allLanguages).size} languages`)
+    }
+
+    return {
+      generatedFiles,
+      entryPoint: 'src/index.ts',
+      keepDeps: {},
+      activationEvents,
+    }
+  },
+}

package/converter/src/transforms/class-to-factory.ts

@@ -20,16 +20,14 @@ export const transformClassToFactory: Transform = (ctx) => {
 
   // AST approach: try to replace via ts-morph
   const nodes = file.getDescendantsOfKind(SyntaxKind.NewExpression)
-  const astReplacements: Array<{ node: any, text: string }> = []
+  // Sort by position descending so inner nodes are processed first
+  nodes.sort((a, b) => b.getPos() - a.getPos())
   for (const expr of nodes) {
     const text = expr.getText()
     const m = text.match(/^new\s+(\w+)\(/)
     if (!m || !FACTORY_TYPES.has(m[1])) continue
     const args = text.slice(m[0].length, -1)
-    astReplacements.push({ node: expr, text: `${m[1]}.create(${args})` })
-  }
-  for (const { node, text } of astReplacements) {
-    try { node.replaceWithText(text) } catch {}
+    try { expr.replaceWithText(`${m[1]}.create(${args})`) } catch {}
   }
 
   // Text fallback: catch remaining new Xxx() that AST might have missed

package/converter/src/transforms/provider-register.ts

@@ -44,14 +44,36 @@ export const transformProviderRegister: Transform = (ctx) => {
       `registerCompletionItemProvider('${pluginName}', '${shortcut}', `
     )
     // Wrap the last argument in an array if it's a string (trigger chars)
-    content = content.replace(
-      /(registerCompletionItemProvider\([^)]+),\s*'([^']+)'\)/g,
-      '$1, ["$2"])'
-    )
-    content = content.replace(
-      /(registerCompletionItemProvider\([^)]+),\s*"([^"]+)"\)/g,
-      '$1, ["$2"])'
-    )
+    // Use paren-balancing to handle nested parentheses in arguments
+    // Build result by iterating over all matches, replacing each full call
+    const providerRe = /registerCompletionItemProvider\(/g
+    let result = ''
+    let lastIdx = 0
+    let m: RegExpExecArray | null
+    while ((m = providerRe.exec(content)) !== null) {
+      const start = m.index
+      let depth = 1
+      let i = start + m[0].length
+      while (i < content.length && depth > 0) {
+        if (content[i] === '(') depth++
+        else if (content[i] === ')') depth--
+        i++
+      }
+      const end = i
+      const fullCall = content.slice(start, end)
+      const lastStrMatch = fullCall.match(/,?\s*'([^']+)'\s*\)$/)
+      const lastDblMatch = fullCall.match(/,?\s*"([^"]+)"\s*\)$/)
+      let replacement = fullCall
+      if (lastStrMatch) {
+        replacement = fullCall.slice(0, fullCall.length - lastStrMatch[0].length) + ', ["' + lastStrMatch[1] + '"])'
+      } else if (lastDblMatch) {
+        replacement = fullCall.slice(0, fullCall.length - lastDblMatch[0].length) + ', ["' + lastDblMatch[1] + '"])'
+      }
+      result += content.slice(lastIdx, start) + replacement
+      lastIdx = end
+    }
+    result += content.slice(lastIdx)
+    content = result
     changed = true
   }
 

package/converter/src/types.ts

@@ -73,7 +73,15 @@ export interface MarkUnsupportedStep {
   verbose?: boolean
 }
 
-export type ConvertStep = LanguageClientStep | SourceStep | BridgeStep | MarkUnsupportedStep
+export interface SnippetsStep {
+  type: 'snippets'
+  /** Optional: override languages to generate (default: read from source package.json's contributes.snippets) */
+  languages?: string[]
+  /** Optional: build command to run in source dir before collecting snippet files (e.g. "node merge.js") */
+  build?: string
+}
+
+export type ConvertStep = LanguageClientStep | SourceStep | BridgeStep | MarkUnsupportedStep | SnippetsStep
 
 // ---- Step execution ----