refactor(cli): reorganize commands — add test, alias orchestrate to o, extract shared options

- Add `test` / `t` command mirroring unity-test-runner (EditMode, PlayMode, All,
  code coverage, test filters)
- Add `o` short alias for `orchestrate` command
- Extract shared option builders (project, docker, android, orchestrator) to
  eliminate duplication across build/test/orchestrate
- Make `build` local-only — remove --provider-strategy (use `orchestrate` for
  remote builds)
- Absorb `cache` command into `status` (--cache-dir flag) — remove standalone
  cache command that was half-implemented
- Fix `list-worfklow` typo in internal CLI → `list-workflow`
- Add test-related fields to CliArguments input mapper
- Update all unit and integration tests (64 passing)

Command structure is now:
  game-ci build          Local build (Docker/macOS)
  game-ci test / t       Run Unity tests
  game-ci orchestrate / o  Remote build (AWS/K8s/etc)
  game-ci activate       License validation
  game-ci status         Project info + cache status
  game-ci version        Version info
  game-ci update         Self-update

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: frostebite

dist/index.js

@@ -4,9 +4,9 @@ import yargs from 'yargs';
 // eslint-disable-next-line import/no-unresolved
 import { hideBin } from 'yargs/helpers';
 import buildCommand from './cli/commands/build';
-import activateCommand from './cli/commands/activate';
+import testCommand from './cli/commands/test';
 import orchestrateCommand from './cli/commands/orchestrate';
-import cacheCommand from './cli/commands/cache';
+import activateCommand from './cli/commands/activate';
 import statusCommand from './cli/commands/status';
 import versionCommand from './cli/commands/version';
 import updateCommand from './cli/commands/update';
@@ -16,9 +16,9 @@ const cli = yargs(hideBin(process.argv))
   .scriptName('game-ci')
   .usage('$0 <command> [options]')
   .command(buildCommand)
-  .command(activateCommand)
+  .command(testCommand)
   .command(orchestrateCommand)
-  .command(cacheCommand)
+  .command(activateCommand)
   .command(statusCommand)
   .command(versionCommand)
   .command(updateCommand)

dist/index.js.map

@@ -36,9 +36,9 @@ describe('CLI integration', () => {
     expect(result.code).toStrictEqual(0);
     expect(result.stdout).toContain('game-ci');
     expect(result.stdout).toContain('build');
-    expect(result.stdout).toContain('activate');
+    expect(result.stdout).toContain('test');
     expect(result.stdout).toContain('orchestrate');
-    expect(result.stdout).toContain('cache');
+    expect(result.stdout).toContain('activate');
     expect(result.stdout).toContain('status');
     expect(result.stdout).toContain('version');
     expect(result.stdout).toContain('update');
@@ -62,7 +62,24 @@ describe('CLI integration', () => {
     expect(result.stdout).toContain('--builds-path');
     expect(result.stdout).toContain('--build-method');
     expect(result.stdout).toContain('--custom-parameters');
-    expect(result.stdout).toContain('--provider-strategy');
+    expect(result.stdout).toContain('--versioning');
+  });
+
+  it('exits 0 and shows test flags for test --help', async () => {
+    const result = await runCli(['test', '--help']);
+
+    expect(result.code).toStrictEqual(0);
+    expect(result.stdout).toContain('--target-platform');
+    expect(result.stdout).toContain('--test-mode');
+    expect(result.stdout).toContain('--test-results-path');
+    expect(result.stdout).toContain('--enable-code-coverage');
+  });
+
+  it('exits 0 for test alias t --help', async () => {
+    const result = await runCli(['t', '--help']);
+
+    expect(result.code).toStrictEqual(0);
+    expect(result.stdout).toContain('--test-mode');
   });
 
   it('exits non-zero for an unknown command', async () => {
@@ -85,18 +102,26 @@ describe('CLI integration', () => {
     expect(result.stdout).toContain('--provider-strategy');
   });
 
+  it('exits 0 for orchestrate alias o --help', async () => {
+    const result = await runCli(['o', '--help']);
+
+    expect(result.code).toStrictEqual(0);
+    expect(result.stdout).toContain('--provider-strategy');
+  });
+
   it('exits 0 for activate --help', async () => {
     const result = await runCli(['activate', '--help']);
 
     expect(result.code).toStrictEqual(0);
     expect(result.stdout).toContain('activate');
   });
 
-  it('exits 0 for cache --help', async () => {
-    const result = await runCli(['cache', '--help']);
+  it('exits 0 for status --help', async () => {
+    const result = await runCli(['status', '--help']);
 
     expect(result.code).toStrictEqual(0);
-    expect(result.stdout).toContain('cache');
+    expect(result.stdout).toContain('status');
+    expect(result.stdout).toContain('--cache-dir');
   });
 
   it('exits 0 for update --help', async () => {

src/cli.ts

@@ -1,7 +1,7 @@
 import buildCommand from '../commands/build';
+import testCommand from '../commands/test';
 import activateCommand from '../commands/activate';
 import orchestrateCommand from '../commands/orchestrate';
-import cacheCommand from '../commands/cache';
 import statusCommand from '../commands/status';
 import versionCommand from '../commands/version';
 import updateCommand from '../commands/update';
@@ -54,7 +54,7 @@ describe('CLI commands', () => {
 
       (buildCommand.builder as Function)(yargs);
 
-      // Core build flags
+      // Core build flags (from shared + build-specific)
       expect(options['target-platform']).toBeDefined();
       expect(options['target-platform'].demandOption).toStrictEqual(true);
       expect(options['unity-version']).toBeDefined();
@@ -88,10 +88,10 @@ describe('CLI commands', () => {
       expect(options['run-as-host-user']).toBeDefined();
       expect(options['chown-files-to']).toBeDefined();
 
-      // Provider flags
-      expect(options['provider-strategy']).toBeDefined();
-      expect(options['skip-activation']).toBeDefined();
-      expect(options['unity-licensing-server']).toBeDefined();
+      // Build should NOT have orchestrator-specific flags
+      expect(options['provider-strategy']).toBeUndefined();
+      expect(options['aws-stack-name']).toBeUndefined();
+      expect(options['kube-config']).toBeUndefined();
     });
 
     it('sets correct default values', () => {
@@ -107,7 +107,6 @@ describe('CLI commands', () => {
       expect(options['enable-gpu'].default).toStrictEqual(false);
       expect(options['android-export-type'].default).toStrictEqual('androidPackage');
       expect(options['android-symbol-type'].default).toStrictEqual('none');
-      expect(options['provider-strategy'].default).toStrictEqual('local');
     });
 
     it('provides camelCase aliases for kebab-case options', () => {
@@ -124,6 +123,62 @@ describe('CLI commands', () => {
     });
   });
 
+  describe('test command', () => {
+    it('exports command with alias', () => {
+      expect(testCommand.command).toStrictEqual(['test', 't']);
+    });
+
+    it('has a description', () => {
+      expect(testCommand.describe).toBeTruthy();
+    });
+
+    it('has a builder function', () => {
+      expect(typeof testCommand.builder).toStrictEqual('function');
+    });
+
+    it('has a handler function', () => {
+      expect(typeof testCommand.handler).toStrictEqual('function');
+    });
+
+    it('defines test-specific flags', () => {
+      const { yargs, options } = createFakeYargs();
+
+      (testCommand.builder as Function)(yargs);
+
+      expect(options['test-mode']).toBeDefined();
+      expect(options['test-mode'].default).toStrictEqual('All');
+      expect(options['test-mode'].choices).toStrictEqual(['EditMode', 'PlayMode', 'All']);
+      expect(options['test-results-path']).toBeDefined();
+      expect(options['test-category']).toBeDefined();
+      expect(options['test-filter']).toBeDefined();
+      expect(options['enable-code-coverage']).toBeDefined();
+      expect(options['coverage-options']).toBeDefined();
+    });
+
+    it('includes shared project options', () => {
+      const { yargs, options } = createFakeYargs();
+
+      (testCommand.builder as Function)(yargs);
+
+      expect(options['target-platform']).toBeDefined();
+      expect(options['target-platform'].demandOption).toStrictEqual(true);
+      expect(options['unity-version']).toBeDefined();
+      expect(options['project-path']).toBeDefined();
+      expect(options['custom-image']).toBeDefined();
+    });
+
+    it('includes docker options but not orchestrator options', () => {
+      const { yargs, options } = createFakeYargs();
+
+      (testCommand.builder as Function)(yargs);
+
+      expect(options['docker-cpu-limit']).toBeDefined();
+      expect(options['docker-memory-limit']).toBeDefined();
+      expect(options['provider-strategy']).toBeUndefined();
+      expect(options['aws-stack-name']).toBeUndefined();
+    });
+  });
+
   describe('activate command', () => {
     it('exports the correct command name', () => {
       expect(activateCommand.command).toStrictEqual('activate');
@@ -143,8 +198,8 @@ describe('CLI commands', () => {
   });
 
   describe('orchestrate command', () => {
-    it('exports the correct command name', () => {
-      expect(orchestrateCommand.command).toStrictEqual('orchestrate');
+    it('exports command with alias', () => {
+      expect(orchestrateCommand.command).toStrictEqual(['orchestrate', 'o']);
     });
 
     it('has a description', () => {
@@ -175,23 +230,16 @@ describe('CLI commands', () => {
       expect(options['watch-to-end']).toBeDefined();
       expect(options['clone-depth']).toBeDefined();
     });
-  });
-
-  describe('cache command', () => {
-    it('exports the correct command name', () => {
-      expect(cacheCommand.command).toStrictEqual('cache <action>');
-    });
 
-    it('has a description', () => {
-      expect(cacheCommand.describe).toBeTruthy();
-    });
+    it('does not include build-only options', () => {
+      const { yargs, options } = createFakeYargs();
 
-    it('has a builder function', () => {
-      expect(typeof cacheCommand.builder).toStrictEqual('function');
-    });
+      (orchestrateCommand.builder as Function)(yargs);
 
-    it('has a handler function', () => {
-      expect(typeof cacheCommand.handler).toStrictEqual('function');
+      expect(options['build-profile']).toBeUndefined();
+      expect(options['manual-exit']).toBeUndefined();
+      expect(options['enable-gpu']).toBeUndefined();
+      expect(options['android-version-code']).toBeUndefined();
     });
   });
 
@@ -207,6 +255,15 @@ describe('CLI commands', () => {
     it('has a handler function', () => {
       expect(typeof statusCommand.handler).toStrictEqual('function');
     });
+
+    it('includes cache-dir option', () => {
+      const { yargs, options } = createFakeYargs();
+
+      (statusCommand.builder as Function)(yargs);
+
+      expect(options['cache-dir']).toBeDefined();
+      expect(options['project-path']).toBeDefined();
+    });
   });
 
   describe('version command', () => {