fix: address review comments — restore help tests and add .options assertions

- Restore deploy --help and dev --help flag-level assertions (integ smoke
  test only checks exit code and "Usage:", not specific flags)
- Restore --invoke negative regression guard in dev.test.ts
- Replace representative-sample enum tests with .options assertions for
  AgentCoreRegionSchema and GatewayTargetTypeSchema (catches accidental
  removal of enum values)

Author: tejaskash

src/cli/commands/deploy/__tests__/deploy.test.ts

@@ -5,6 +5,18 @@ import { tmpdir } from 'node:os';
 import { join } from 'node:path';
 import { afterAll, beforeAll, describe, expect, it } from 'vitest';
 
+describe('deploy --help', () => {
+  it('shows all deploy options', async () => {
+    const result = await runCLI(['deploy', '--help'], process.cwd());
+    expect(result.exitCode).toBe(0);
+    expect(result.stdout.includes('--yes')).toBeTruthy();
+    expect(result.stdout.includes('--verbose')).toBeTruthy();
+    expect(result.stdout.includes('--json')).toBeTruthy();
+    expect(result.stdout.includes('--dry-run')).toBeTruthy();
+    expect(result.stdout.includes('resource-level'), 'Should describe resource-level events').toBeTruthy();
+  });
+});
+
 describe('deploy without agents', () => {
   let noAgentTestDir: string;
   let noAgentProjectDir: string;

src/cli/commands/dev/__tests__/dev.test.ts

@@ -2,6 +2,27 @@ import { runCLI } from '../../../../test-utils/index.js';
 import { describe, expect, it } from 'vitest';
 
 describe('dev command', () => {
+  describe('--help', () => {
+    it('shows all options', async () => {
+      const result = await runCLI(['dev', '--help'], process.cwd());
+
+      expect(result.exitCode).toBe(0);
+      expect(result.stdout.includes('[prompt]'), 'Should show [prompt] positional argument').toBeTruthy();
+      expect(result.stdout.includes('--port'), 'Should show --port option').toBeTruthy();
+      expect(result.stdout.includes('--runtime'), 'Should show --runtime option').toBeTruthy();
+      expect(result.stdout.includes('--stream'), 'Should show --stream option').toBeTruthy();
+      expect(result.stdout.includes('--logs'), 'Should show --logs option').toBeTruthy();
+      expect(result.stdout.includes('8080'), 'Should show default port').toBeTruthy();
+    });
+
+    it('does not show --invoke flag', async () => {
+      const result = await runCLI(['dev', '--help'], process.cwd());
+
+      expect(result.exitCode).toBe(0);
+      expect(result.stdout.includes('--invoke'), 'Should not show removed --invoke option').toBeFalsy();
+    });
+  });
+
   describe('requires project context', () => {
     it('exits with error when run outside project', async () => {
       const result = await runCLI(['dev'], process.cwd());

src/schema/schemas/__tests__/aws-targets.test.ts

@@ -8,10 +8,25 @@ import {
 import { describe, expect, it } from 'vitest';
 
 describe('AgentCoreRegionSchema', () => {
-  it('accepts representative valid regions', () => {
-    expect(AgentCoreRegionSchema.safeParse('us-east-1').success).toBe(true);
-    expect(AgentCoreRegionSchema.safeParse('eu-west-1').success).toBe(true);
-    expect(AgentCoreRegionSchema.safeParse('us-gov-west-1').success).toBe(true);
+  it('enumerates all supported regions', () => {
+    expect(AgentCoreRegionSchema.options).toEqual([
+      'ap-northeast-1',
+      'ap-northeast-2',
+      'ap-south-1',
+      'ap-southeast-1',
+      'ap-southeast-2',
+      'ca-central-1',
+      'eu-central-1',
+      'eu-north-1',
+      'eu-west-1',
+      'eu-west-2',
+      'eu-west-3',
+      'sa-east-1',
+      'us-east-1',
+      'us-east-2',
+      'us-west-2',
+      'us-gov-west-1',
+    ]);
   });
 
   it('rejects unsupported regions and invalid values', () => {

src/schema/schemas/__tests__/mcp.test.ts

@@ -21,9 +21,15 @@ import {
 import { describe, expect, it } from 'vitest';
 
 describe('GatewayTargetTypeSchema', () => {
-  it('accepts valid target types', () => {
-    expect(GatewayTargetTypeSchema.safeParse('lambda').success).toBe(true);
-    expect(GatewayTargetTypeSchema.safeParse('lambdaFunctionArn').success).toBe(true);
+  it('enumerates all target types', () => {
+    expect(GatewayTargetTypeSchema.options).toEqual([
+      'lambda',
+      'mcpServer',
+      'openApiSchema',
+      'smithyModel',
+      'apiGateway',
+      'lambdaFunctionArn',
+    ]);
   });
 
   it('rejects invalid type', () => {