@testdriverai/mcp 7.10.0-canary.0 → 7.10.1-test

package/interfaces/vitest-plugin.mjs

@@ -10,6 +10,7 @@ import { setTestRunInfo } from "./shared-test-state.mjs";
 // Use createRequire to import CommonJS modules without esbuild processing
 const require = createRequire(import.meta.url);
 const channelConfig = require("../lib/resolve-channel.js");
+const environments = require("../lib/environments.json");
 
 // Import Sentry for error reporting
 const Sentry = require("@sentry/node");
@@ -39,8 +40,8 @@ function initializeSentry() {
         "https://452bd5a00dbd83a38ee8813e11c57694@o4510262629236736.ingest.us.sentry.io/4510480443637760",
       environment: channelConfig.sentryEnvironment,
       release: version,
-      sampleRate: 1.0,
-      tracesSampleRate: 1.0,
+      sampleRate: 0.01,
+      tracesSampleRate: 0.01,
       enableLogs: true,
       integrations: [Sentry.httpIntegration(), Sentry.nodeContextIntegration()],
       initialScope: {
@@ -820,10 +821,6 @@ class TestDriverReporter {
     // Initialize Sentry for error reporting
     initializeSentry();
 
-    // Store project root for making file paths relative
-    pluginState.projectRoot = ctx.config.root || process.cwd();
-    logger.debug("Project root:", pluginState.projectRoot);
-
     // NOW read the API key and API root (after setupFiles have run, including dotenv/config)
     pluginState.apiKey = this.options.apiKey || process.env.TD_API_KEY;
     pluginState.apiRoot =
@@ -875,7 +872,7 @@ class TestDriverReporter {
       // Create test run via direct API call
       const testRunData = {
         runId: pluginState.testRunId,
-        suiteName: getSuiteName(),
+        suiteName: getSuiteName(this.ctx),
         ...pluginState.gitInfo,
       };
 
@@ -1094,27 +1091,25 @@ class TestDriverReporter {
     const sessionId = meta.sessionId || null;
     const platform = meta.platform || null;
     const sandboxId = meta.sandboxId || null;
-    let testFile = meta.testFile || "unknown";
-    const testOrder = meta.testOrder !== undefined ? meta.testOrder : 0;
-
-    // If testFile not in meta, fallback to test object properties
-    if (testFile === "unknown") {
-      const absolutePath =
-        test.module?.task?.filepath ||
-        test.module?.file?.filepath ||
-        test.module?.file?.name ||
-        test.file?.filepath ||
-        test.file?.name ||
-        test.suite?.file?.filepath ||
-        test.suite?.file?.name ||
-        test.location?.file ||
-        "unknown";
-      testFile =
-        pluginState.projectRoot && absolutePath !== "unknown"
-          ? path.relative(pluginState.projectRoot, absolutePath)
-          : absolutePath;
-      logger.debug(`Resolved testFile from fallback: ${testFile}`);
+
+    // Test identity from Vitest's standard reporter API (TestCase in v4+).
+    const projectName = test.project?.name || null;
+    const projectRoot = test.project?.config?.root || process.cwd();
+    const moduleId = test.module?.moduleId || null;
+    let testFile =
+      moduleId && !moduleId.includes("://") ? path.relative(projectRoot, moduleId) : "unknown";
+    if (path.isAbsolute(testFile) || testFile.startsWith("..")) testFile = "unknown";
+
+    // Full nested describe path (top-level first), joined with " > ".
+    const suitePath = [];
+    {
+      let cursor = test.parent;
+      while (cursor && cursor.type === "suite") {
+        suitePath.unshift(cursor.name);
+        cursor = cursor.parent;
+      }
     }
+    const suiteName = suitePath.length > 0 ? suitePath.join(" > ") : null;
 
     // Update test run platform from first test that reports it
     if (platform && !pluginState.detectedPlatform) {
@@ -1150,7 +1145,6 @@ class TestDriverReporter {
         // We only want actual SDK crashes and exceptions reported to Sentry.
       }
 
-      const suiteName = test.suite?.name;
       const startTime = Date.now() - duration; // Calculate start time from duration
       // In Vitest v4, retryCount is on diagnostic(), not result()
       // result() only returns { state, errors }, while diagnostic() has retryCount, duration, etc.
@@ -1160,12 +1154,14 @@ class TestDriverReporter {
       const consoleUrl = getConsoleUrl(pluginState.apiRoot);
       const hasRetries = retryCount > 0 && dashcamUrls.length > 1;
 
-      // Record a single test case with all metadata
+      // Record a single test case with all metadata.
+      // Field names match the current API contract; `projectName` is a
+      // forward-compatible addition (ignored by the API today).
       const testCaseData = {
         runId: testRunId,
         testName: test.name,
         testFile: testFile,
-        testOrder: testOrder,
+        projectName: projectName,
         status,
         startTime: startTime,
         endTime: Date.now(),
@@ -1186,7 +1182,13 @@ class TestDriverReporter {
       // Include per-attempt replay URLs for retry visibility
       if (dashcamUrls.length > 0) {
         const attemptUrls = dashcamUrls
-          .map(a => ({ attempt: a.attempt, url: a.url || null, sessionId: a.sessionId || null }));
+          .map(a => ({
+            attempt: a.attempt,
+            url: a.url || null,
+            sessionId: a.sessionId || null,
+            errorMessage: a.errorMessage || null,
+            errorStack: a.errorStack || null,
+          }));
         testCaseData.replayUrls = attemptUrls;
       }
 
@@ -1195,7 +1197,7 @@ class TestDriverReporter {
       if (errorStack) testCaseData.errorStack = errorStack;
 
       logger.debug(
-        `Recording test case: ${test.name} (${status}) with testFile: ${testFile}, testOrder: ${testOrder}, duration: ${duration}ms, replay: ${dashcamUrl ? "yes" : "no"}`,
+        `Recording test case: ${test.name} (${status}) with testFile: ${testFile}, project: ${projectName}, suite: ${suiteName || "(none)"}, duration: ${duration}ms, replay: ${dashcamUrl ? "yes" : "no"}`,
       );
 
       const testCaseResponse = await recordTestCaseDirect(
@@ -1304,32 +1306,13 @@ class TestDriverReporter {
           interactions: testResult.interactions || { total: 0, cached: 0, byType: {} },
         };
 
-        // Sanitize testName for filesystem use
-        const safeName = (test.name || "unknown").replace(/[^a-zA-Z0-9_.-]/g, "_").substring(0, 200);
+        // Use Vitest's stable per-test id as the filename — it's unique within
+        // a run and avoids the name-collision problem entirely.
         const resultDir = path.join(process.cwd(), ".testdriver", "results", testFile);
         fs.mkdirSync(resultDir, { recursive: true });
-
-        // Include a stable unique suffix in the filename to avoid collisions
-        // when multiple tests in the same file share the same name.
-        const hashSourceParts = [];
-        if (test.id) {
-          hashSourceParts.push(String(test.id));
-        }
-        if (Array.isArray(test.suitePath)) {
-          hashSourceParts.push(test.suitePath.join(" > "));
-        }
-        if (test.file && (test.file.name || test.file.path)) {
-          hashSourceParts.push(test.file.name || test.file.path);
-        }
-        // Fallback to the test name if no other identifiers are available.
-        if (hashSourceParts.length === 0) {
-          hashSourceParts.push(test.name || "unknown");
-        }
-        const hashSource = hashSourceParts.join(" | ");
-        const uniqueHash = crypto.createHash("sha256").update(hashSource).digest("hex").slice(0, 8);
-
+        const safeId = (test.id || "unknown").replace(/[^\w.-]/g, "_");
         fs.writeFileSync(
-          path.join(resultDir, `${safeName}-${uniqueHash}.json`),
+          path.join(resultDir, `${safeId}.json`),
           JSON.stringify(resultData, null, 2),
         );
       }
@@ -1376,11 +1359,11 @@ function getConsoleUrl(apiRoot) {
     return `https://${flyMatch[1]}-web.fly.dev`;
   }
 
-  // Known channel API URLs -> console equivalents
-  // e.g. https://api-canary.testdriver.ai -> https://console-canary.testdriver.ai
-  for (const url of Object.values(channelConfig.channels)) {
-    if (url === apiRoot) {
-      return url.replace("api", "console").replace("1337", "3001");
+  // Known channel API URLs -> console equivalents (from environments.json)
+  // e.g. https://api-dev.testdriver.ai -> https://console-dev.testdriver.ai
+  for (const env of Object.values(environments)) {
+    if (env.apiRoot === apiRoot && env.consoleUrl) {
+      return env.consoleUrl;
     }
   }
 
@@ -1404,7 +1387,14 @@ function generateRunId() {
   return `${Date.now()}-${crypto.randomBytes(4).toString("hex")}`;
 }
 
-function getSuiteName() {
+function getSuiteName(ctx) {
+  // Derived from Vitest's configured projects (joined when multiple).
+  // Must always return a string: the test-run-create API requires suiteName.
+  const projects = ctx?.projects;
+  if (Array.isArray(projects) && projects.length > 0) {
+    const names = projects.map((p) => p?.name).filter(Boolean);
+    if (names.length > 0) return names.join(",");
+  }
   return process.env.npm_package_name || path.basename(process.cwd());
 }
 

package/lib/core/Dashcam.js

@@ -98,7 +98,7 @@ class Dashcam {
     if (!apiRoot) return "https://console.testdriver.ai";
 
     // Dash-separated environments: api-{env}.testdriver.ai -> console-{env}.testdriver.ai
-    const envMatch = apiRoot.match(/^https:\/\/api-(test|canary)\.testdriver\.ai/);
+    const envMatch = apiRoot.match(/^https:\/\/api-(dev|test|canary)\.testdriver\.ai/);
     if (envMatch) {
       return `https://console-${envMatch[1]}.testdriver.ai`;
     }
@@ -150,6 +150,39 @@ class Dashcam {
 
   }
 
+  /**
+   * Workaround for dashcam <1.5.0: webLogsDaemon.js writes its state to a
+   * `.dashcam` directory relative to process.cwd(). When launched by the
+   * runner, cwd is typically a root-owned global install path (for example
+   * /usr/lib/node_modules/@testdriverai/runner or
+   * /usr/local/lib/node_modules/@testdriverai/runner),
+   * so the daemon can't mkdir its state dir as user `user` and silently
+   * dies — recordings never start (or `dashcam logs --add` fails with EACCES).
+   * Pre-create world-writable .dashcam dirs at both the current cwd and the
+   * known global-install path so the daemon survives regardless of which path
+   * dashcam ends up using. Must run before ANY dashcam invocation, not just
+   * auth(), because addWebLog() is called before start()/auth() in provision.
+   * TODO: remove once dashcam >=1.5.0 (which uses $HOME) is bundled.
+   * @private
+   */
+  async _ensureDashcamStateDirs() {
+    if (this.client.os === "windows") return;
+    if (!this._dashcamDirsReady) {
+      this._dashcamDirsReady = this.client.exec(
+        this._getShell(),
+        `GLOBAL_NPM_ROOT="$(npm root -g 2>/dev/null || true)"; ` +
+          `PWD_DIR="$(pwd)"; REAL_PWD_DIR="$(pwd -P 2>/dev/null || pwd)"; ` +
+          `DIRS="$PWD_DIR/.dashcam $REAL_PWD_DIR/.dashcam /usr/lib/node_modules/@testdriverai/runner/.dashcam /usr/local/lib/node_modules/@testdriverai/runner/.dashcam"; ` +
+          `if [ -n "$GLOBAL_NPM_ROOT" ]; then DIRS="$DIRS $GLOBAL_NPM_ROOT/@testdriverai/runner/.dashcam"; fi; ` +
+          `for d in $DIRS; do sudo mkdir -p "$d" 2>/dev/null || true; sudo chmod 0777 "$d" 2>/dev/null || true; done; ` +
+          `true`,
+        10000,
+        process.env.TD_DEBUG == "true" ? false : true,
+      );
+    }
+    await this._dashcamDirsReady;
+  }
+
   /**
    * Authenticate dashcam with API key
    * @param {string} [apiKey] - Override API key
@@ -174,6 +207,8 @@ class Dashcam {
       this._log("debug", "Auth output:", authOutput);
     } else {
       // Linux/Mac authentication with TD_API_ROOT
+      await this._ensureDashcamStateDirs();
+
       const authOutput = await this.client.exec(
         shell,
         `TD_API_ROOT="${apiRoot}" dashcam auth ${key}`,
@@ -215,6 +250,7 @@ class Dashcam {
       );
       this._log("debug", "Add log tracking output:", addLogOutput);
     } else {
+      await this._ensureDashcamStateDirs();
       // Create log file
       await this.client.exec(
         shell,
@@ -254,6 +290,7 @@ class Dashcam {
       );
       this._log("debug", "Add application log tracking output:", addLogOutput);
     } else {
+      await this._ensureDashcamStateDirs();
       const addLogOutput = await this.client.exec(
         shell,
         `TD_API_ROOT="${apiRoot}" dashcam logs --add --type=application --application="${application}" --name="${name}"`,
@@ -288,6 +325,7 @@ class Dashcam {
         this._log("warn", "Add web log tracking failed:", err.message);
       }
     } else {
+      await this._ensureDashcamStateDirs();
       const addLogOutput = await this.client.exec(
         shell,
         `TD_API_ROOT="${apiRoot}" dashcam logs --add --type=web --pattern="${pattern}" --name="${name}"`,
@@ -368,6 +406,7 @@ class Dashcam {
       this._log("debug", "Dashcam recording started");
     } else {
       // Linux/Mac with TD_API_ROOT
+      await this._ensureDashcamStateDirs();
       this._log("debug", "Starting dashcam recording on Linux/Mac...");
       const dashcamPath = await this._getDashcamPath();
       const titleArg = this.title
@@ -430,10 +469,12 @@ class Dashcam {
       this._log("debug", "Dashcam stop command output:", output);
     } else {
       // Linux/Mac with TD_API_ROOT
+      // Use 2>&1 to merge stderr into stdout so the replay URL is captured
+      // regardless of which stream the dashcam CLI writes it to.
       const dashcamPath = await this._getDashcamPath();
       output = await this.client.exec(
         shell,
-        `TD_API_ROOT="${apiRoot}" "${dashcamPath}" stop`,
+        `TD_API_ROOT="${apiRoot}" "${dashcamPath}" stop 2>&1`,
         300000,
         process.env.TD_DEBUG == "true" ? false : true,
       );

package/lib/core/index.d.ts

@@ -38,6 +38,13 @@ export class Dashcam {
    */
   addApplicationLog(application: string, name: string): Promise<void>;
   
+  /**
+   * Add a web log to Dashcam
+   * @param pattern - URL pattern to match (e.g., "*example.com*")
+   * @param name - Name/description for the log entry
+   */
+  addWebLog(pattern: string, name: string): Promise<void>;
+  
   /**
    * Start recording
    * @returns Promise that resolves when recording starts
@@ -68,7 +75,7 @@ export interface LogConfig {
   /**
    * Type of log entry
    */
-  type: 'file' | 'application';
+  type: 'file' | 'application' | 'web';
   
   /**
    * Path to file (for file logs)
@@ -80,6 +87,11 @@ export interface LogConfig {
    */
   application?: string;
   
+  /**
+   * URL pattern (for web logs, e.g., "*example.com*")
+   */
+  pattern?: string;
+  
   /**
    * Name/description for the log entry
    */

package/lib/github-comment.mjs

@@ -8,7 +8,13 @@
  * - Links to test runs
  */
 
-import { Octokit } from '@octokit/rest';
+/**
+ * Lazily load Octokit to avoid hard failure when @octokit/rest is not installed
+ */
+async function getOctokit(token) {
+  const { Octokit } = await import('@octokit/rest');
+  return new Octokit({ auth: token });
+}
 
 /**
  * Format test duration in human-readable format
@@ -334,7 +340,7 @@ export async function postGitHubComment(options) {
     throw new Error('Either prNumber or commitSha must be provided');
   }
 
-  const octokit = new Octokit({ auth: token });
+  const octokit = await getOctokit(token);
 
   if (prNumber) {
     // Comment on PR
@@ -374,7 +380,7 @@ export async function updateGitHubComment(options) {
     throw new Error('Token, owner, repo, and commentId are required');
   }
 
-  const octokit = new Octokit({ auth: token });
+  const octokit = await getOctokit(token);
 
   const response = await octokit.rest.issues.updateComment({
     owner,
@@ -402,7 +408,7 @@ export async function findExistingComment(options) {
     return null;
   }
 
-  const octokit = new Octokit({ auth: token });
+  const octokit = await getOctokit(token);
 
   const comments = await octokit.rest.issues.listComments({
     owner,
@@ -435,7 +441,7 @@ export async function postOrUpdateTestResults(testRunData, testCases, githubOpti
 
     if (existingComment) {
       // Delete the old comment
-      const octokit = new Octokit({ auth: githubOptions.token });
+      const octokit = await getOctokit(githubOptions.token);
       await octokit.rest.issues.deleteComment({
         owner: githubOptions.owner,
         repo: githubOptions.repo,

package/lib/init-project.js

@@ -1,6 +1,7 @@
 const fs = require("fs");
 const path = require("path");
 const { execSync, spawn } = require("child_process");
+const { active: ACTIVE_CHANNEL } = require("./resolve-channel.js");
 
 /**
  * Run an npm install command with an animated progress bar
@@ -38,12 +39,18 @@ function runInstall(cmd, args, cwd, label) {
  * @param {string} [options.targetDir] - Target directory (defaults to current working directory)
  * @param {string} [options.apiKey] - TestDriver API key (will be saved to .env)
  * @param {boolean} [options.skipInstall=false] - Skip npm install step
+ * @param {boolean} [options.skipSampleTest=false] - Skip scaffolding the example test files (tests/example.test.js + tests/login.js). The tests/ dir, config, workflow, etc. are still created.
  * @param {boolean} [options.interactive=false] - Whether to prompt for missing values (CLI mode)
+ * @param {string[]} [options.clients] - AI client ids to wire up (e.g. ["claude-code","cursor"] or ["all"]). When set, native per-client config is written instead of the interactive add-mcp flow.
+ * @param {string} [options.channel] - Release channel (dev|test|canary|stable) the generated GitHub workflow should pin the action to. Defaults to the SDK's own active channel.
  * @param {function} [options.onProgress] - Callback for progress updates (receives message string)
  * @returns {Promise<{success: boolean, results: string[], errors: string[]}>}
  */
 async function initProject(options = {}) {
   const targetDir = options.targetDir || process.cwd();
+  // Which TestDriver channel the generated workflow should target. A -test SDK
+  // scaffolds a test-pinned workflow, -canary -> canary, clean semver -> stable.
+  const channel = options.channel || ACTIVE_CHANNEL;
   const results = [];
   const errors = [];
   
@@ -94,6 +101,11 @@ async function initProject(options = {}) {
       fs.mkdirSync(testDir, { recursive: true });
     }
 
+    // Sample test files (login snippet + example test). Skipped when the caller
+    // opts out (e.g. /eve, which scaffolds the project but writes its own tests).
+    if (options.skipSampleTest) {
+      progress("⊘ Skipped sample test files (tests/example.test.js, tests/login.js)");
+    } else {
     // Create login snippet file
     const loginSnippetFile = path.join(testDir, "login.js");
     if (!fs.existsSync(loginSnippetFile)) {
@@ -170,6 +182,7 @@ test('should login and add item to cart', async (context) => {
       fs.writeFileSync(testFile, vitestContent);
       progress("✓ Created test file: tests/example.test.js");
     }
+    }
 
     // 3. Create vitest.config.js
     const configFile = path.join(targetDir, "vitest.config.js");
@@ -234,6 +247,10 @@ export default defineConfig({
 
     const workflowFile = path.join(workflowDir, "testdriver.yml");
     if (!fs.existsSync(workflowFile)) {
+      // The published action is tagged test/canary/stable. `dev` runs locally and
+      // isn't published, so pin its `uses:` ref to the test action while still
+      // passing channel: dev so the action targets the dev API root.
+      const actionRef = channel === "dev" ? "test" : channel;
       const workflowContent = `name: TestDriver.ai Tests
 
 on:
@@ -241,10 +258,14 @@ on:
     branches: [ main, master ]
   pull_request:
     branches: [ main, master ]
+  workflow_dispatch:
 
 jobs:
   test:
     runs-on: ubuntu-latest
+    permissions:
+      id-token: write   # required to mint the OIDC token for auth
+      contents: read
 
     steps:
     - uses: actions/checkout@v4
@@ -258,9 +279,16 @@ jobs:
     - name: Install dependencies
       run: npm ci
 
+    # Authenticate to TestDriver via GitHub OIDC (no stored secret needed once the
+    # TestDriver GitHub App is authorized for your org). Falls back to a TD_API_KEY
+    # secret if you set one. Exports TD_API_KEY for the test step below.
+    - name: Authenticate to TestDriver
+      uses: testdriverai/action@${actionRef}
+      with:
+        channel: ${channel}
+        api-key: \${{ secrets.TD_API_KEY }}
+
     - name: Run TestDriver.ai tests
-      env:
-        TD_API_KEY: \${{ secrets.TD_API_KEY }}
       run: npx vitest run
 
     - name: Upload test results
@@ -277,12 +305,64 @@ jobs:
       progress("⊘ GitHub workflow already exists");
     }
 
+    // Resolve the bundled agent/skills source dirs once; reused by the
+    // client-aware installer (step 6) and the legacy copy steps (8 & 9).
+    const resolveSource = (...candidates) =>
+      candidates.find((c) => fs.existsSync(c)) || null;
+    const skillsSourceDir = resolveSource(
+      path.join(targetDir, "node_modules", "testdriverai", "ai", "skills"),
+      path.join(__dirname, "..", "ai", "skills"),
+    );
+    const agentsSourceDir = resolveSource(
+      path.join(targetDir, "node_modules", "testdriverai", "ai", "agents"),
+      path.join(__dirname, "..", "ai", "agents"),
+    );
+
     // 6. Setup MCP configuration
-    // When triggered from VS Code extension, create .vscode/mcp.json silently
-    // When triggered from CLI, use interactive add-mcp for user to select their MCP client
+    // Priority:
+    //   a) options.clients set        -> write native config per selected client
+    //   b) TD_INIT_SOURCE=vscode      -> create .vscode/mcp.json silently
+    //   c) otherwise (CLI)            -> interactive add-mcp client picker
     const isVscodeInit = process.env.TD_INIT_SOURCE === "vscode";
-
-    if (isVscodeInit) {
+    const requestedClients =
+      Array.isArray(options.clients) && options.clients.length
+        ? options.clients
+        : null;
+
+    if (requestedClients) {
+      const {
+        installClient,
+        resolveClientIds,
+      } = require("./install-clients.js");
+      const { valid, unknown } = resolveClientIds(requestedClients);
+      for (const id of unknown) {
+        progress(`⚠ Unknown client "${id}" - skipping`);
+      }
+      for (const id of valid) {
+        try {
+          const result = installClient(id, {
+            targetDir,
+            apiKey: options.apiKey,
+            agentSource: agentsSourceDir,
+            skillsSource: skillsSourceDir,
+          });
+          progress(`🔧 ${result.label}:`);
+          for (const step of result.steps) {
+            if (step.status === "manual") {
+              progress(`  ℹ ${step.message}`);
+            } else if (step.status === "error") {
+              progress(`  ⚠ ${step.message}`);
+            } else if (step.status === "skipped") {
+              progress(`  ⊘ ${step.message || step.file}`);
+            } else {
+              progress(`  ✓ ${step.message || step.file}`);
+            }
+          }
+        } catch (err) {
+          progress(`⚠ Failed to configure ${id}: ${err.message}`);
+        }
+      }
+    } else if (isVscodeInit) {
       // VS Code extension: create .vscode/mcp.json directly
       const vscodeDir = path.join(targetDir, ".vscode");
       if (!fs.existsSync(vscodeDir)) {
@@ -365,18 +445,6 @@ jobs:
 
     // 8. Copy TestDriver skills
     const skillsDestDir = path.join(targetDir, ".github", "skills");
-    const possibleSkillsSources = [
-      path.join(targetDir, "node_modules", "testdriverai", "ai", "skills"),
-      path.join(__dirname, "..", "ai", "skills"),
-    ];
-
-    let skillsSourceDir = null;
-    for (const source of possibleSkillsSources) {
-      if (fs.existsSync(source)) {
-        skillsSourceDir = source;
-        break;
-      }
-    }
 
     if (skillsSourceDir) {
       if (!fs.existsSync(skillsDestDir)) {
@@ -412,18 +480,6 @@ jobs:
 
     // 9. Copy TestDriver agents
     const agentsDestDir = path.join(targetDir, ".github", "agents");
-    const possibleAgentsSources = [
-      path.join(targetDir, "node_modules", "testdriverai", "ai", "agents"),
-      path.join(__dirname, "..", "ai", "agents"),
-    ];
-
-    let agentsSourceDir = null;
-    for (const source of possibleAgentsSources) {
-      if (fs.existsSync(source)) {
-        agentsSourceDir = source;
-        break;
-      }
-    }
 
     if (agentsSourceDir) {
       if (!fs.existsSync(agentsDestDir)) {
@@ -447,16 +503,6 @@ jobs:
       if (copiedCount > 0) {
         progress(`✓ Copied ${copiedCount} agent(s) to .github/agents/`);
       }
-
-      // Also set testdriver.md as copilot-instructions.md if it doesn't already exist
-      const copilotInstructionsPath = path.join(targetDir, ".github", "copilot-instructions.md");
-      const testdriverAgentSource = path.join(agentsSourceDir, "testdriver.md");
-      if (!fs.existsSync(copilotInstructionsPath) && fs.existsSync(testdriverAgentSource)) {
-        fs.copyFileSync(testdriverAgentSource, copilotInstructionsPath);
-        progress("✓ Created .github/copilot-instructions.md");
-      } else if (fs.existsSync(copilotInstructionsPath)) {
-        progress("⊘ copilot-instructions.md already exists, skipping");
-      }
     } else {
       progress("⚠ Agents directory not found (will be available after npm install)");
     }
@@ -485,21 +531,29 @@ jobs:
     }
 
     // 11. Install dependencies (unless skipped)
+    // Pin testdriverai to the SAME channel this CLI is running from (the npm
+    // dist-tags match the channel names: test | canary | stable). Otherwise a
+    // bare `testdriverai` resolves to `latest` (= stable) even when init was run
+    // via `npx testdriverai@test`, so the scaffolded project — and the vitest
+    // run in the generated GitHub action — would target stable instead. `dev`
+    // isn't a published tag, so fall back to `test` (matches the actionRef logic above).
+    const installTag = channel === "dev" ? "test" : channel;
+    const testdriveraiSpec = `testdriverai@${installTag}`;
     if (!options.skipInstall) {
       progress("\n📦 Installing dependencies...");
       try {
-        await runInstall("npm", ["install", "-D", "vitest", "testdriverai"], targetDir, "vitest testdriverai");
-        progress("✓ Installed vitest, testdriverai");
+        await runInstall("npm", ["install", "-D", "vitest", testdriveraiSpec], targetDir, `vitest ${testdriveraiSpec}`);
+        progress(`✓ Installed vitest, ${testdriveraiSpec}`);
         await runInstall("npm", ["install", "dotenv"], targetDir, "dotenv");
         progress("✓ Installed dotenv");
       } catch (error) {
         errors.push("Failed to install dependencies. Run manually:");
-        errors.push("  npm install -D vitest testdriverai");
+        errors.push(`  npm install -D vitest ${testdriveraiSpec}`);
         errors.push("  npm install dotenv");
       }
     } else {
       progress("\nℹ Skipped dependency installation. Run manually:");
-      progress("  npm install -D vitest testdriverai");
+      progress(`  npm install -D vitest ${testdriveraiSpec}`);
       progress("  npm install dotenv");
     }