Update CLI to TypeScript

Author: harmony7

.github/workflows/main.yml

@@ -164,6 +164,11 @@ jobs:
     - uses: actions/checkout@v4
       with:
         submodules: true
+    - name: Set up Node LTS
+      uses: actions/setup-node@v3
+      with:
+        node-version: 'lts/*'
+    - run: npm ci
     - name: Install Rust 1.81.0
       run: |
         rustup toolchain install 1.81.0
@@ -174,6 +179,8 @@ jobs:
       with:
         path: "/home/runner/.cargo/bin/wasm-tools"
         key: crate-cache-wasm-tools-${{ env.wasm-tools_version }}
+    - name: Build CLI
+      run: npm run build:cli
     - name: Build with full debug info
       run: npm run build:debug:info
     - uses: actions/upload-artifact@v4
@@ -193,6 +200,11 @@ jobs:
     - uses: actions/checkout@v4
       with:
         submodules: true
+    - name: Set up Node LTS
+      uses: actions/setup-node@v3
+      with:
+        node-version: 'lts/*'
+    - run: npm ci
     - name: Install Rust 1.81.0
       run: |
         rustup toolchain install 1.81.0
@@ -203,6 +215,8 @@ jobs:
       with:
         path: "/home/runner/.cargo/bin/wasm-tools"
         key: crate-cache-wasm-tools-${{ env.wasm-tools_version }}
+    - name: Build CLI
+      run: npm run build:cli
     - name: Build
       if: matrix.profile == 'release'
       run: npm run build:release
@@ -229,9 +243,11 @@ jobs:
     - uses: actions/checkout@v4
       with:
         submodules: true
-    - uses: actions/setup-node@v3
+    - name: Set up Node LTS
+      uses: actions/setup-node@v3
       with:
         node-version: 'lts/*'
+    - run: npm ci
 
     - name: Download Engine
       uses: actions/download-artifact@v4
@@ -250,8 +266,9 @@ jobs:
       with:
         path: "/home/runner/.cargo/bin/wasm-tools"
         key: crate-cache-wasm-tools-${{ env.wasm-tools_version }}
-    
-    - run: npm install
+
+    - name: Build CLI
+      run: npm run build:cli
 
     - name: Build WPT runtime
       run: tests/wpt-harness/build-wpt-runtime.sh --debug-build
@@ -277,9 +294,11 @@ jobs:
     - uses: actions/checkout@v4
       with:
         submodules: true
-    - uses: actions/setup-node@v3
+    - name: Set up Node LTS
+      uses: actions/setup-node@v3
       with:
         node-version: 'lts/*'
+    - run: npm ci
 
     - name: Download Engine
       uses: actions/download-artifact@v4
@@ -305,7 +324,8 @@ jobs:
         path: "/home/runner/.cargo/bin/wasm-tools"
         key: crate-cache-wasm-tools-${{ env.wasm-tools_version }}
 
-    - run: npm install
+    - name: Build CLI
+      run: npm run build:cli
 
     - name: Build WPT runtime
       run: tests/wpt-harness/build-wpt-runtime.sh ${{matrix.profile == 'weval' && '--enable-experimental-aot' || matrix.profile == 'debug' && '--debug-build' || ''}}

.github/workflows/release-please.yml

@@ -89,6 +89,11 @@ jobs:
     - uses: actions/checkout@v3
       with:
         submodules: true
+    - name: Set up Node LTS
+      uses: actions/setup-node@v3
+      with:
+        node-version: 'lts/*'
+    - run: npm ci
     - name: Install Rust 1.81.0
       run: |
         rustup toolchain install 1.81.0
@@ -99,6 +104,8 @@ jobs:
       with:
         path: "/home/runner/.cargo/bin/wasm-tools"
         key: crate-cache-wasm-tools-${{ env.wasm-tools_version }}
+    - name: Build CLI
+      run: npm run build:cli
     - name: Build
       if: ${{ matrix.profile == 'release' }}
       run: npm run build:release

.gitignore

@@ -1,3 +1,6 @@
+# TypeScript build output
+dist/
+
 # Integration Tests build output
 /integration-tests/**/fixtures/**/*.tar.gz
 /integration-tests/**/fixtures/**/fastly.toml

DEVELOPMENT.md

@@ -33,9 +33,9 @@ To build from source, you need to have the following tools installed to successf
   ```sh
   sudo apt install binaryen
   ```
-- rust target wasm32-wasi
+- rust target wasm32-wasip1
   ```sh
-  rustup target add wasm32-wasi
+  rustup target add wasm32-wasip1
   ```
 - [cbindgen](https://github.com/eqrion/cbindgen#quick-start)
   ```sh
@@ -87,9 +87,9 @@ npm run build
   # then, restart shell or run:
   source $HOME/.cargo/env
   ```
-- rust target wasm32-wasi
+- rust target wasm32-wasip1
   ```sh
-  rustup target add wasm32-wasi
+  rustup target add wasm32-wasip1
   ```
 - [cbindgen](https://github.com/eqrion/cbindgen#quick-start)
   ```sh
@@ -180,10 +180,10 @@ In addition the following flags can be added after the command (passed via `npm
 A typical development test command is therefore something like:
 
 ```
-npm run build:debug && npm run test:integration -- --debug-build --debug-log --local --bail /crypto
+npm run build:cli && npm run build:debug && npm run test:integration -- --debug-build --debug-log --local --bail /crypto
 ```
 
-Which would run a debug build, enable debugging logging, and then that build against all the crypto tests locally on Viceroy, throwing an error as soon as one is found.
+Which would build the CLI TypeScript to JavaScript, run a debug build, enable debugging logging, and then that build against all the crypto tests locally on Viceroy, throwing an error as soon as one is found.
 
 Some tests can only be run on Compute and not Viceroy and will be automatically skipped. A green tick is always shown for a test that ran successfully - if it is missing that means it did not run.
 

integration-tests/cli/env.test.js

@@ -1,5 +1,5 @@
 import test from 'brittle';
-import { EnvParser } from '../../src/env.js';
+import { EnvParser } from '../../dist/env.js';
 
 test('EnvParser should parse single key-value pair', function (t) {
   const parser = new EnvParser();

integration-tests/js-compute/fixtures/app/fastly.toml.in

@@ -9,7 +9,7 @@ name = "js-test-app"
 service_id = ""
 
 [scripts]
-  build = "node ../../../../js-compute-runtime-cli.js --env FASTLY_DEBUG_LOGGING,ACL_NAME,CONFIG_STORE_NAME,DICTIONARY_NAME,KV_STORE_NAME,SECRET_STORE_NAME,LOCAL_TEST,TEST=\"foo\" --enable-experimental-high-resolution-time-methods src/index.js"
+  build = "node ../../../../dist/cli/js-compute-runtime-cli.js --env FASTLY_DEBUG_LOGGING,ACL_NAME,CONFIG_STORE_NAME,DICTIONARY_NAME,KV_STORE_NAME,SECRET_STORE_NAME,LOCAL_TEST,TEST=\"foo\" --enable-experimental-high-resolution-time-methods src/index.js"
 
 [local_server]
 

integration-tests/js-compute/fixtures/module-mode/fastly.toml.in

@@ -9,7 +9,7 @@ name = "js-test-app"
 service_id = ""
 
 [scripts]
-  build = "node ../../../../js-compute-runtime-cli.js --env FASTLY_DEBUG_LOGGING,ACL_NAME,CONFIG_STORE_NAME,DICTIONARY_NAME,KV_STORE_NAME,SECRET_STORE_NAME,LOCAL_TEST,TEST=\"foo\" --enable-experimental-high-resolution-time-methods --module-mode src/index.js"
+  build = "node ../../../../dist/cli/js-compute-runtime-cli.js --env FASTLY_DEBUG_LOGGING,ACL_NAME,CONFIG_STORE_NAME,DICTIONARY_NAME,KV_STORE_NAME,SECRET_STORE_NAME,LOCAL_TEST,TEST=\"foo\" --enable-experimental-high-resolution-time-methods --module-mode src/index.js"
 
 [local_server]