feat(ripgrep): add Show Ripgrep Diagnostic command (#281)

* feat: add Zoo Code: Show Ripgrep Diagnostic command

Adds a user-triggerable diagnostic for the "Could not find ripgrep
binary" error class. The command runs the same hybrid resolution
that #248 tested — try require("@vscode/ripgrep"), then probe
known appRoot-relative paths — and writes a verbose report to a
dedicated output channel, also copying it to the clipboard.

Motivated by #248: debugging that bug required a custom test build
to inspect resolution state on the user's machine. With this
command shipped, future users hitting weird ripgrep-resolution
behavior can paste the diagnostic into a bug report without us
needing to build instrumented VSIXs.

Reintroduces the @vscode/ripgrep devDep and esbuild external entry
that #248 dropped — the diagnostic needs to call require() to
report what happens. The require attempt also serves as forward-
compat: when VS Code completes the @vscode/ripgrep → @vscode/
ripgrep-universal package-rename migration, it will start
succeeding broadly, and the diagnostic output will be the signal
that we can revisit the require approach in getBinPath itself.

- src/services/ripgrep/diagnostic.ts: new module (data fn + cmd wrapper)
- src/services/ripgrep/internal/loadRipgrep.ts: testable require wrapper
- zoo-code.showRipgrepDiagnostic command wired up in registerCommands.ts
  and contributed via package.json
- Tests for the data function

* fix: address review feedback on ripgrep diagnostic command

- diagnostic.ts: fix .asar->.asar.unpacked regex (the \b boundary was
  matching inside node_modules.asar.unpacked too, producing
  .unpacked.unpacked). Replaced with a path-separator lookahead.
- diagnostic.ts: create the OutputChannel once at registration and
  return a composite Disposable that disposes both the command and
  the channel; clear the channel before appending so repeated runs
  are readable.
- diagnostic.ts: route the command ID through getCommand() instead
  of hardcoding 'zoo-code.showRipgrepDiagnostic', and add
  'showRipgrepDiagnostic' to the CommandId union in @roo-code/types.
  Exclude it from getCommandsMap so the diagnostic's separate
  registration owns the OutputChannel lifecycle.
- package.json + package.nls.*.json: switch the command title to a
  %command.showRipgrepDiagnostic.title% NLS key across all 18 locale
  files.
- loadRipgrep.ts: preserve the require() error message in a
  loadError field instead of swallowing it; surface it in the
  diagnostic report.
- Tests updated for the loadError case, the already-unpacked path
  guard, and widened the mock value type.

* test(ripgrep diagnostic): add Windows-path coverage + appRoot validation

Two polish items from review:

- getRipgrepDiagnostic now early-returns an explanatory message
  when vscode.env.appRoot is empty, instead of silently producing
  a report with a meaningless path probe. Closes the input-
  validation gap surfaced in re-reading the diagnostic.
- New test exercises the .asar->.asar.unpacked substitution on
  Windows-style backslash paths. The regex already handles both
  separators via [\\/], this just pins it.

* refactor(registerCommands): tighten command map value type

Replace the residual `any` in the command-callback map value with a
`CommandCallback` alias of `(...args: any[]) => unknown`. Mirrors
VS Code's own `commands.registerCommand` signature while narrowing
the return to `unknown` so callers must inspect before use.

Rest-args stay `any[]` intentionally: the callbacks in this map are
heterogeneous (`importSettings` takes an optional `filePath?: string`,
the rest take none), and VS Code dispatches positional args
dynamically — a single tight per-arity type would be lossy without
splitting the map.

* feat(ripgrep): add Show Ripgrep Diagnostic command and translations

---------

Co-authored-by: 0xMink <260166390+0xMink@users.noreply.github.com>
Co-authored-by: edelauna <54631123+edelauna@users.noreply.github.com>
Co-authored-by: Elliott de Launay <edelauna@gmail.com>

Author: 0xMink

knip.json

@@ -21,6 +21,7 @@
 				"@types/node-cache",
 				"@types/vscode",
 				"@vscode/codicons",
+				"@vscode/ripgrep",
 				"esbuild-wasm",
 				"sambanova-ai-provider",
 				"tree-sitter-wasms",

packages/types/src/vscode.ts

@@ -46,6 +46,8 @@ export const commandIds = [
 	"acceptInput",
 	"focusPanel",
 	"toggleAutoApprove",
+
+	"showRipgrepDiagnostic",
 ] as const
 
 export type CommandId = (typeof commandIds)[number]

src/activate/__tests__/registerCommands.spec.ts

@@ -81,6 +81,10 @@ vi.mock("../../i18n", () => ({
 	t: (key: string) => key,
 }))
 
+vi.mock("../../services/ripgrep/diagnostic", () => ({
+	registerRipgrepDiagnosticCommand: vi.fn().mockReturnValue({ dispose: vi.fn() }),
+}))
+
 describe("getVisibleProviderOrLog", () => {
 	let mockOutputChannel: vscode.OutputChannel
 
@@ -172,6 +176,14 @@ describe("registerCommands handlers", () => {
 		setPanel(undefined, "tab")
 	})
 
+	it("registers the ripgrep diagnostic command and stores its disposable in context.subscriptions", async () => {
+		const { registerRipgrepDiagnosticCommand } = await import("../../services/ripgrep/diagnostic")
+		const mock = vi.mocked(registerRipgrepDiagnosticCommand)
+		const disposable = mock.mock.results[0]?.value
+		expect(mock).toHaveBeenCalled()
+		expect(mockContext.subscriptions).toContain(disposable)
+	})
+
 	it("settingsButtonClicked posts both settingsButtonClicked and didBecomeVisible actions", () => {
 		handlers["zoo-code.settingsButtonClicked"]()
 

src/activate/registerCommands.ts

@@ -13,6 +13,7 @@ import { handleNewTask } from "./handleTask"
 import { CodeIndexManager } from "../services/code-index/manager"
 import { importSettingsWithFeedback } from "../core/config/importExport"
 import { MdmService } from "../services/mdm/MdmService"
+import { registerRipgrepDiagnosticCommand } from "../services/ripgrep/diagnostic"
 import { t } from "../i18n"
 
 /**
@@ -68,9 +69,27 @@ export const registerCommands = (options: RegisterCommandOptions) => {
 		const command = getCommand(id as CommandId)
 		context.subscriptions.push(vscode.commands.registerCommand(command, callback))
 	}
+
+	context.subscriptions.push(registerRipgrepDiagnosticCommand())
 }
 
-const getCommandsMap = ({ context, outputChannel, provider }: RegisterCommandOptions): Record<CommandId, any> => ({
+// `showRipgrepDiagnostic` is registered separately by
+// `registerRipgrepDiagnosticCommand` (above), which owns the OutputChannel
+// lifecycle alongside the command registration, so it's intentionally
+// excluded from this map.
+//
+// Callback shape mirrors VS Code's own `commands.registerCommand` signature
+// (`(...args: any[]) => any`), with the return narrowed to `unknown` so
+// callers must inspect before using. `any[]` for args is unavoidable: the
+// callbacks here are heterogeneous (`importSettings` takes an optional
+// `filePath?: string`, others take none) and VS Code dispatches positional
+// args dynamically.
+type CommandCallback = (...args: any[]) => unknown
+const getCommandsMap = ({
+	context,
+	outputChannel,
+	provider,
+}: RegisterCommandOptions): Record<Exclude<CommandId, "showRipgrepDiagnostic">, CommandCallback> => ({
 	activationCompleted: () => {},
 	plusButtonClicked: async () => {
 		const visibleProvider = getVisibleProviderOrLog(outputChannel)

src/esbuild.mjs

@@ -126,7 +126,7 @@ async function main() {
 		// global-agent must be external because it dynamically patches Node.js http/https modules
 		// which breaks when bundled. It needs access to the actual Node.js module instances.
 		// undici must be bundled because our VSIX is packaged with `--no-dependencies`.
-		external: ["vscode", "esbuild", "global-agent"],
+		external: ["vscode", "esbuild", "global-agent", "@vscode/ripgrep"],
 	}
 
 	/**

src/package.json

@@ -160,6 +160,11 @@
 				"title": "%command.acceptInput.title%",
 				"category": "%configuration.title%"
 			},
+			{
+				"command": "zoo-code.showRipgrepDiagnostic",
+				"title": "%command.showRipgrepDiagnostic.title%",
+				"category": "%configuration.title%"
+			},
 			{
 				"command": "zoo-code.toggleAutoApprove",
 				"title": "%command.toggleAutoApprove.title%",