feat: respect CURSORLESS_VSCODE_COMMAND in generateTestSubsetFile script

bluedrink9 · bluedrink9 · commit da9dee7472fc · 2025-11-29T18:02:21.000+13:00
Otherwise this breaks on systems without code installed
diff --git a/packages/test-harness/src/scripts/generateTestSubsetFile.ts b/packages/test-harness/src/scripts/generateTestSubsetFile.ts
@@ -20,6 +20,8 @@ languages/go/
  */
 function run() {
   const testSubsetGrepPath = testSubsetFilePath();
+  // Use CURSORLESS_VSCODE_COMMAND environment variable if set, otherwise default to 'code'
+  const vscodeCommand = process.env.CURSORLESS_VSCODE_COMMAND || "code";
 
   const exists = fs.existsSync(testSubsetGrepPath);
 
@@ -28,7 +30,7 @@ function run() {
   }
 
   if (!exists || process.argv.includes("--always-open")) {
-    child.execSync(`code ${testSubsetGrepPath}`);
+    child.execSync(`${vscodeCommand} ${testSubsetGrepPath}`);
   }
 
   if (!exists && process.argv.includes("--fail-if-not-exists")) {

Original file line number	Diff line number	Diff line change
`@@ -20,6 +20,8 @@ languages/go/`
`20`	`20`	`*/`
`21`	`21`	`function run() {`
`22`	`22`	`const testSubsetGrepPath = testSubsetFilePath();`
	`23`	`+ // Use CURSORLESS_VSCODE_COMMAND environment variable if set, otherwise default to 'code'`
	`24`	`+ const vscodeCommand = process.env.CURSORLESS_VSCODE_COMMAND \|\| "code";`
`23`	`25`
`24`	`26`	`const exists = fs.existsSync(testSubsetGrepPath);`
`25`	`27`
`@@ -28,7 +30,7 @@ function run() {`
`28`	`30`	`}`
`29`	`31`
`30`	`32`	`if (!exists \|\| process.argv.includes("--always-open")) {`
`31`		- child.execSync(`code ${testSubsetGrepPath}`);
	`33`	+ child.execSync(`${vscodeCommand} ${testSubsetGrepPath}`);
`32`	`34`	`}`
`33`	`35`
`34`	`36`	`if (!exists && process.argv.includes("--fail-if-not-exists")) {`