chore: prebuild dist folder | add changelog

Author: JPeer264

.cursor/rules/sentry-cli-project.mdc

@@ -20,7 +20,7 @@ This is **sentry-cli**, a command-line utility for working with Sentry. It's pri
 ## Project Structure
 
 - `src/` - Core Rust source code with command modules and utilities
-- `js/` - JavaScript wrapper and npm package code
+- `lib/` - JavaScript wrapper and npm package code
 - `scripts/` - Build and utility scripts
 - `tests/integration/` - Integration tests using `.trycmd` format
 - `npm-binary-distributions/` - Platform-specific binary packages

.github/CODEOWNERS

@@ -4,7 +4,7 @@
 /.cursor/rules/javascript-development.mdc @getsentry/team-javascript-sdks @getsentry/owners-sentry-cli
 /.npmignore @getsentry/team-javascript-sdks @getsentry/owners-sentry-cli
 /bin/ @getsentry/team-javascript-sdks @getsentry/owners-sentry-cli  # these are the JS "binaries"
-/js @getsentry/team-javascript-sdks @getsentry/owners-sentry-cli
+/lib @getsentry/team-javascript-sdks @getsentry/owners-sentry-cli
 /package.json @getsentry/team-javascript-sdks @getsentry/owners-sentry-cli
 /scripts/*.js @getsentry/team-javascript-sdks @getsentry/owners-sentry-cli
 /tsconfig.json @getsentry/team-javascript-sdks @getsentry/owners-sentry-cli

.github/workflows/test_node.yml

@@ -26,7 +26,7 @@ jobs:
 
       - name: Install dependencies via npm ci
         if: ${{ !inputs.triggered-by-release }}
-        run: npm ci
+        run: npm ci --ignore-scripts
 
       # For pushes to the release branch, we need to install the dependencies via `npm install`
       # because the `package-lock.json` is not updated with the new versions of the optional
@@ -37,7 +37,30 @@ jobs:
 
       - run: npm run check:types
 
+  prepare_build:
+    name: Prepare Build
+    runs-on: ubuntu-24.04
+    steps:
+      - uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # 5.0.0
+
+      - name: Use Node.js
+        uses: actions/setup-node@2028fbc5c25fe9cf00d9f06a71cc4710d4507903 # 6.0.0
+        with:
+          node-version-file: package.json
+
+      - name: Install dependencies
+        run: npm ci --ignore-scripts
+
+      - name: Build
+        run: npm run build
+
+      - uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # 4.6.2
+        with:
+          name: artifact-build
+          path: js
+
   test_node:
+    needs: prepare_build
     strategy:
       fail-fast: false
       matrix:
@@ -56,7 +79,7 @@ jobs:
 
       - name: Install dependencies via npm ci
         if: ${{ !inputs.triggered-by-release }}
-        run: npm ci
+        run: npm ci --ignore-scripts
 
       # For pushes to the release branch, we need to install the dependencies via `npm install`
       # because the `package-lock.json` is not updated with the new versions of the optional
@@ -65,6 +88,15 @@ jobs:
         if: ${{ inputs.triggered-by-release }}
         run: npm install --omit=optional --ignore-scripts
 
+      - name: Download build artifact
+        uses: actions/download-artifact@634f93cb2916e3fdff6788551b99b062d0335ce0 # 5.0.0
+        with:
+          name: artifact-build
+          path: js
+
+      - name: Install CLI
+        run: npm run install-cli
+
       # older node versions need an older nft
       - run: SENTRYCLI_SKIP_DOWNLOAD=1 npm install @vercel/nft@0.22.1
         if: matrix.node-version == '10.x' || matrix.node-version == '12.x'

.gitignore

@@ -4,7 +4,7 @@ target
 node_modules
 coverage
 dist
-js-dist
+js
 dump
 checksums.txt
 yarn-error.log

.npmignore

@@ -6,7 +6,7 @@
 !README.md
 !LICENSE
 !bin/sentry-cli
-!js-dist
+!js
 !scripts/install.js
 !checksums.txt
 

CHANGELOG.md

@@ -46,6 +46,10 @@
 
 - Slightly sped up the `sentry-cli sourcemaps upload` command by eliminating an HTTP request to the Sentry server, which was not required in most cases ([#2913](https://github.com/getsentry/sentry-cli/pull/2913)).
 
+### Internal changes
+
+- Migrated JavaScript wrapper to TypeScript for better type safety ([#2910](https://github.com/getsentry/sentry-cli/pull/2910))
+
 ## 2.57.0
 
 ### New Features

bin/sentry-cli

@@ -3,7 +3,7 @@
 'use strict';
 
 const childProcess = require('child_process');
-const SentryCli = require('../js-dist');
+const SentryCli = require('../js');
 
 const child = childProcess
   .spawn(SentryCli.getPath(), process.argv.slice(2), {