refactor: move CLI to orchestrator, fix validate-orchestrator workflow

- Delete src/cli.ts, src/cli/ (commands, tests, input-mapper) — moved
  to game-ci/orchestrator repo (PR #813 reference)
- Delete .github/workflows/release-cli.yml — moved to orchestrator
- Remove bin, pkg, yargs, @types/yargs, pkg from package.json
- Fix validate-orchestrator.yml:
  - Build TypeScript before running require() smoke tests
  - Remove || echo fallback that swallowed errors
  - Add smoke test that installs orchestrator via npm pack and
    verifies loadOrchestrator() returns defined exports

Legacy src/model/cli/ (Cli class, CliFunctionsRepository) preserved —
used by Input.getInput() and build-parameters.ts on main.

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

Author: frostebite

.github/workflows/release-cli.yml

@@ -51,21 +51,20 @@ jobs:
       - name: Install unity-builder dependencies
         run: yarn install --frozen-lockfile
 
-      - name: Verify unity-builder compiles without orchestrator
+      - name: Build unity-builder
         run: |
-          echo "Verifying unity-builder compiles without @game-ci/orchestrator installed..."
-          npx tsc --noEmit
+          echo "Building unity-builder TypeScript..."
+          npx tsc
           echo "✓ unity-builder compiles successfully"
 
       - name: Run unity-builder tests
         run: |
           echo "Running unity-builder tests..."
           npx jest --no-cache --passWithNoTests 2>&1 | tail -10
 
-      - name: Verify plugin loader gracefully handles missing orchestrator
+      - name: Verify plugin loader returns undefined without orchestrator
         run: |
-          echo "Checking that orchestrator-plugin.ts handles missing package..."
-          # The plugin loader should return undefined when @game-ci/orchestrator is not installed
+          echo "Checking plugin loader handles missing @game-ci/orchestrator..."
           node -e "
             const { loadOrchestrator, loadEnterpriseServices } = require('./lib/model/orchestrator-plugin');
             (async () => {
@@ -83,7 +82,7 @@ jobs:
               }
               console.log('✓ loadEnterpriseServices() returns undefined when package not installed');
             })();
-          " 2>&1 || echo "::warning::Plugin loader test requires compiled JS (run yarn build first)"
+          "
 
       - name: Verify orchestrator type declarations exist
         run: |
@@ -94,16 +93,46 @@ jobs:
             exit 1
           fi
 
-      - name: Run orchestrator standalone tests
+      - name: Build and install orchestrator standalone
         working-directory: orchestrator-standalone
         run: |
           yarn install --frozen-lockfile
+          echo "Building orchestrator standalone..."
+          npx tsc
+          echo "✓ orchestrator standalone compiles successfully"
+          echo "Packing orchestrator as tarball..."
+          npm pack
+
+      - name: Run orchestrator standalone tests
+        working-directory: orchestrator-standalone
+        run: |
           echo "Running orchestrator standalone tests..."
           npx jest --no-cache 2>&1 | tail -10
 
-      - name: Verify orchestrator standalone compiles
-        working-directory: orchestrator-standalone
+      - name: Verify plugin loader returns exports with orchestrator installed
         run: |
-          echo "Verifying orchestrator standalone compiles..."
-          npx tsc --noEmit
-          echo "✓ orchestrator standalone compiles successfully"
+          echo "Installing orchestrator into unity-builder workspace..."
+          npm install ./orchestrator-standalone/game-ci-orchestrator-*.tgz --no-save
+          echo "Checking plugin loader returns defined exports..."
+          node -e "
+            const { loadOrchestrator, loadEnterpriseServices } = require('./lib/model/orchestrator-plugin');
+            (async () => {
+              const orch = await loadOrchestrator();
+              if (orch === undefined) {
+                console.error('ERROR: loadOrchestrator should return defined exports when package is installed');
+                process.exit(1);
+              }
+              if (typeof orch.run !== 'function') {
+                console.error('ERROR: loadOrchestrator().run should be a function');
+                process.exit(1);
+              }
+              console.log('✓ loadOrchestrator() returns defined exports with orchestrator installed');
+
+              const services = await loadEnterpriseServices();
+              if (services === undefined) {
+                console.error('ERROR: loadEnterpriseServices should return defined exports when package is installed');
+                process.exit(1);
+              }
+              console.log('✓ loadEnterpriseServices() returns defined exports with orchestrator installed');
+            })();
+          "

.github/workflows/validate-orchestrator.yml

@@ -3,24 +3,6 @@
   "version": "3.0.0",
   "description": "Build Unity projects for different platforms.",
   "main": "dist/index.js",
-  "bin": {
-    "game-ci": "./lib/cli.js"
-  },
-  "pkg": {
-    "scripts": "lib/**/*.js",
-    "assets": [
-      "lib/**/*.json",
-      "package.json"
-    ],
-    "targets": [
-      "node20-linux-x64",
-      "node20-linux-arm64",
-      "node20-macos-x64",
-      "node20-macos-arm64",
-      "node20-win-x64"
-    ],
-    "outputPath": "dist-binaries"
-  },
   "repository": "git@github.com:game-ci/unity-builder.git",
   "author": "Webber <webber@takken.io>",
   "license": "MIT",
@@ -29,8 +11,6 @@
     "build": "yarn && tsc && ncc build lib --source-map --license licenses.txt",
     "lint": "prettier --check \"src/**/*.{js,ts}\" && eslint src/**/*.ts",
     "format": "prettier --write \"src/**/*.{js,ts}\"",
-    "cli": "yarn ts-node src/index.ts -m cli",
-    "game-ci": "ts-node src/cli.ts",
     "test": "jest",
     "test:ci": "jest --config=jest.ci.config.js --runInBand"
   },
@@ -50,14 +30,12 @@
     "semver": "^7.5.2",
     "ts-md5": "^1.3.1",
     "unity-changeset": "^3.1.0",
-    "yaml": "^2.2.2",
-    "yargs": "^17.7.2"
+    "yaml": "^2.2.2"
   },
   "devDependencies": {
     "@types/jest": "^27.4.1",
     "@types/node": "^17.0.23",
     "@types/semver": "^7.3.9",
-    "@types/yargs": "^17.0.35",
     "@typescript-eslint/parser": "4.8.1",
     "@vercel/ncc": "^0.36.1",
     "cross-env": "^7.0.3",
@@ -73,7 +51,6 @@
     "js-yaml": "^4.1.0",
     "lefthook": "^1.6.1",
     "node-fetch": "2",
-    "pkg": "^5.8.1",
     "prettier": "^2.5.1",
     "ts-jest": "^27.1.3",
     "ts-node": "10.8.1",