[PROPOSAL] Integrate Napi.rs to TSC to gradually migrate to Rust 🦀

.github/workflows/ci.yml

@@ -33,7 +33,7 @@ jobs:
           - '20'
           - '18'
           - '16'
-          - '14'
+        # - '14' does not support Nullish Coalescing in napi.rs
         bundle:
           - 'true'
         include:
@@ -56,6 +56,8 @@ jobs:
           node-version: ${{ matrix.node-version }}
           check-latest: true
       - run: npm ci
+      - uses: actions-rust-lang/setup-rust-toolchain@v1
+      - run: npm run build:napi
 
       - name: Tests
         id: test
@@ -70,6 +72,7 @@ jobs:
           git diff --staged --exit-code
 
   coverage:
+    if: ${{ github.repository_owner == 'microsoft' }}
     runs-on:
       - 'self-hosted'
       - '1ES.Pool=TypeScript-1ES-GitHub-Large'
@@ -86,6 +89,9 @@ jobs:
           node-version: 'lts/*'
       - run: npm ci
 
+      - uses: actions-rust-lang/setup-rust-toolchain@v1
+      - run: npm run build:napi
+
       - name: Run tests with coverage
         run: npm test -- --no-lint --coverage
 
@@ -157,6 +163,9 @@ jobs:
           node-version: 'lts/*'
       - run: npm ci
 
+      - uses: actions-rust-lang/setup-rust-toolchain@v1
+      - run: npm run build:napi
+
       - name: Installing browsers
         run: npx playwright install --with-deps
 
@@ -173,6 +182,9 @@ jobs:
           node-version: 'lts/*'
       - run: npm ci
 
+      - uses: actions-rust-lang/setup-rust-toolchain@v1
+      - run: npm run build:napi
+
       - name: Build src
         run: npx hereby build-src
 
@@ -193,6 +205,9 @@ jobs:
 
       - run: npm ci
 
+      - uses: actions-rust-lang/setup-rust-toolchain@v1
+      - run: npm run build:napi
+
       - run: npx hereby lkg
       - run: |
           node ./scripts/addPackageJsonGitHead.mjs package.json
@@ -237,6 +252,8 @@ jobs:
           npm --version
           # corepack enable npm
 
+      - uses: actions-rust-lang/setup-rust-toolchain@v1
+
       - run: |
           npm install -g $(jq -r '.packageManager' < package.json)
           npm --version
@@ -245,6 +262,10 @@ jobs:
       - run: npm ci
         working-directory: ./pr
 
+      # TODO: same on base once it has the command
+      - run: npm run build:napi
+        working-directory: ./pr
+
       - run: npm ci
         working-directory: ./base
 
@@ -284,6 +305,9 @@ jobs:
           node-version: 'lts/*'
       - run: npm ci
 
+      - uses: actions-rust-lang/setup-rust-toolchain@v1
+      - run: npm run build:napi
+
       - name: Build tsc
         run: npx hereby tsc
 
@@ -303,6 +327,9 @@ jobs:
           node-version: 'lts/*'
       - run: npm ci
 
+      - uses: actions-rust-lang/setup-rust-toolchain@v1
+      - run: npm run build:napi
+
       - name: Remove all baselines
         run: rm -rf tests/baselines/reference
 

.gitignore

@@ -64,3 +64,9 @@ package-lock.json
 .eslintcache
 *v8.log
 /lib/
+
+# napi.rs
+src/rs.d.ts
+src/rs.js
+target/*
+typescript.*