fix: restore preview's integration tests for create/evaluator

Main's integration tests expect telemetry instrumentation in the create
CLI path which preview doesn't have (preview's create/command.tsx has
the harness path instead of main's telemetry wrapping). Use preview's
test versions which match preview's behavior.

Author: notgitika

integ-tests/add-remove-evaluator.test.ts

@@ -1,13 +1,10 @@
 import { createTestProject, parseJsonOutput, readProjectConfig, runCLI } from '../src/test-utils/index.js';
 import type { TestProject } from '../src/test-utils/index.js';
-import { createTelemetryHelper } from '../src/test-utils/telemetry-helper.js';
 import { afterAll, beforeAll, describe, expect, it } from 'vitest';
 
-const telemetry = createTelemetryHelper();
-
 /** Run a CLI command and assert it succeeds, returning parsed JSON output. */
 async function runSuccess(args: string[], cwd: string) {
-  const result = await runCLI(args, cwd, { env: telemetry.env });
+  const result = await runCLI(args, cwd);
   expect(result.exitCode, `stdout: ${result.stdout}, stderr: ${result.stderr}`).toBe(0);
   const json: unknown = parseJsonOutput(result.stdout);
   expect(json).toHaveProperty('success', true);
@@ -16,7 +13,7 @@ async function runSuccess(args: string[], cwd: string) {
 
 /** Run a CLI command and assert it fails, returning parsed JSON output. */
 async function runFailure(args: string[], cwd: string) {
-  const result = await runCLI(args, cwd, { env: telemetry.env });
+  const result = await runCLI(args, cwd);
   expect(result.exitCode).toBe(1);
   const json: unknown = parseJsonOutput(result.stdout);
   expect(json).toHaveProperty('success', false);
@@ -38,7 +35,6 @@ describe('integration: add and remove evaluators and online eval configs', () =>
 
   afterAll(async () => {
     await project.cleanup();
-    telemetry.destroy();
   });
 
   describe('evaluator and online eval lifecycle', () => {
@@ -127,29 +123,25 @@ describe('integration: add and remove evaluators and online eval configs', () =>
 
       const config = await readProjectConfig(project.projectPath);
       expect(config.onlineEvalConfigs.find(c => c.name === configName)).toBeUndefined();
-      telemetry.assertMetricEmitted({ command: 'remove.online-eval', exit_reason: 'success' });
     });
 
     it('removes the evaluator after online eval is gone', async () => {
       await runSuccess(['remove', 'evaluator', '--name', evalName, '--json'], project.projectPath);
 
       const config = await readProjectConfig(project.projectPath);
       expect(config.evaluators.find(e => e.name === evalName)).toBeUndefined();
-      telemetry.assertMetricEmitted({ command: 'remove.evaluator', exit_reason: 'success' });
     });
   });
 
   describe('error cases', () => {
     it('fails to remove non-existent evaluator', async () => {
       const json = await runFailure(['remove', 'evaluator', '--name', 'NonExistent', '--json'], project.projectPath);
       expect(json.error).toContain('not found');
-      telemetry.assertMetricEmitted({ command: 'remove.evaluator', exit_reason: 'failure' });
     });
 
     it('fails to remove non-existent online eval config', async () => {
       const json = await runFailure(['remove', 'online-eval', '--name', 'NonExistent', '--json'], project.projectPath);
       expect(json.error).toContain('not found');
-      telemetry.assertMetricEmitted({ command: 'remove.online-eval', exit_reason: 'failure' });
     });
 
     it('rejects evaluator with missing --level', async () => {

integ-tests/create-edge-cases.test.ts

@@ -1,6 +1,5 @@
 /* eslint-disable security/detect-non-literal-fs-filename */
 import { exists, prereqs, runCLI } from '../src/test-utils/index.js';
-import { createTelemetryHelper } from '../src/test-utils/telemetry-helper.js';
 import { randomUUID } from 'node:crypto';
 import { mkdir, rm } from 'node:fs/promises';
 import { tmpdir } from 'node:os';
@@ -10,21 +9,18 @@ import { afterAll, beforeAll, describe, expect, it } from 'vitest';
 describe.skipIf(!prereqs.npm || !prereqs.git)('integration: create edge cases', () => {
   let testDir: string;
 
-  const telemetry = createTelemetryHelper();
-
   beforeAll(async () => {
     testDir = join(tmpdir(), `agentcore-integ-edge-${randomUUID()}`);
     await mkdir(testDir, { recursive: true });
   });
 
   afterAll(async () => {
-    telemetry.destroy();
     await rm(testDir, { recursive: true, force: true });
   });
 
   describe('reserved names', () => {
     it('rejects reserved name "Test"', async () => {
-      const result = await runCLI(['create', '--name', 'Test', '--json'], testDir, { env: telemetry.env });
+      const result = await runCLI(['create', '--name', 'Test', '--json'], testDir);
 
       expect(result.exitCode).toBe(1);
       const json = JSON.parse(result.stdout);
@@ -34,11 +30,6 @@ describe.skipIf(!prereqs.npm || !prereqs.git)('integration: create edge cases',
         json.error.toLowerCase().includes('reserved') || json.error.toLowerCase().includes('conflict'),
         `Error should mention reserved/conflict: ${json.error}`
       ).toBeTruthy();
-
-      telemetry.assertMetricEmitted({
-        command: 'create',
-        exit_reason: 'failure',
-      });
     });
 
     it('rejects reserved name "bedrock"', async () => {
@@ -130,21 +121,12 @@ describe.skipIf(!prereqs.npm || !prereqs.git)('integration: create edge cases',
   describe('flag interactions', () => {
     it('--defaults creates project with default settings', async () => {
       const name = `Def${Date.now().toString().slice(-6)}`;
-      const result = await runCLI(['create', '--name', name, '--defaults', '--json'], testDir, { env: telemetry.env });
+      const result = await runCLI(['create', '--name', name, '--defaults', '--json'], testDir);
 
       expect(result.exitCode, `stderr: ${result.stderr}`).toBe(0);
       const json = JSON.parse(result.stdout);
       expect(json.success).toBe(true);
       expect(json.projectPath).toBeTruthy();
-
-      telemetry.assertMetricEmitted({
-        command: 'create',
-        exit_reason: 'success',
-        language: 'python',
-        framework: 'strands',
-        model_provider: 'bedrock',
-        has_agent: 'true',
-      });
     });
 
     it('--dry-run shows what would be created without writing files', async () => {

integ-tests/create-frameworks.test.ts

@@ -1,5 +1,4 @@
 import { exists, prereqs, readProjectConfig, runCLI } from '../src/test-utils/index.js';
-import { createTelemetryHelper } from '../src/test-utils/telemetry-helper.js';
 import { randomUUID } from 'node:crypto';
 import { mkdir, readFile, rm } from 'node:fs/promises';
 import { tmpdir } from 'node:os';
@@ -9,15 +8,12 @@ import { afterAll, beforeAll, describe, expect, it } from 'vitest';
 describe.skipIf(!prereqs.npm || !prereqs.git)('integration: create with different frameworks', () => {
   let testDir: string;
 
-  const telemetry = createTelemetryHelper();
-
   beforeAll(async () => {
     testDir = join(tmpdir(), `agentcore-integ-frameworks-${randomUUID()}`);
     await mkdir(testDir, { recursive: true });
   });
 
   afterAll(async () => {
-    telemetry.destroy();
     await rm(testDir, { recursive: true, force: true });
   });
 
@@ -38,8 +34,7 @@ describe.skipIf(!prereqs.npm || !prereqs.git)('integration: create with differen
         'none',
         '--json',
       ],
-      testDir,
-      { env: telemetry.env }
+      testDir
     );
 
     expect(result.exitCode, `stderr: ${result.stderr}`).toBe(0);
@@ -65,15 +60,6 @@ describe.skipIf(!prereqs.npm || !prereqs.git)('integration: create with differen
     expect(agents).toBeDefined();
     expect(agents.length).toBe(1);
     expect(agents[0]!.name).toBe(agentName);
-
-    telemetry.assertMetricEmitted({
-      command: 'create',
-      exit_reason: 'success',
-      language: 'python',
-      framework: 'langchain_langgraph',
-      model_provider: 'bedrock',
-      has_agent: 'true',
-    });
   });
 
   it('creates GoogleADK project with Gemini provider', async () => {