npm pack rollbar for examples on postbuild (#1277)

matux · web-flow · commit ea292b9be349 · 2025-08-04T11:03:46.000-03:00
diff --git a/.gitignore b/.gitignore
@@ -8,7 +8,8 @@ coverage
 .DS_Store
 *.swp
 npm-debug.log
-.pack
+examples/rollbar*.tgz
+
 .local
 CLAUDE.*.*
 .aider*
diff --git a/examples/angular/package.json b/examples/angular/package.json
@@ -22,7 +22,7 @@
     "@angular/router": "^19.0.0",
     "@angular/ssr": "^19.0.6",
     "express": "^4.18.2",
-    "rollbar": "file:../..",
+    "rollbar": "file:../rollbar.tgz",
     "rxjs": "~7.8.0",
     "tslib": "^2.3.0",
     "zone.js": "~0.15.0"
diff --git a/examples/browserify/package.json b/examples/browserify/package.json
@@ -10,7 +10,7 @@
   "author": "",
   "license": "ISC",
   "dependencies": {
-    "rollbar": "file:../.."
+    "rollbar": "file:../rollbar.tgz"
   },
   "devDependencies": {
     "browserify": "^17.0.0"
diff --git a/examples/no-conflict/.gitignore b/examples/no-conflict/.gitignore
@@ -0,0 +1 @@
+dist
diff --git a/examples/no-conflict/package.json b/examples/no-conflict/package.json
@@ -12,7 +12,7 @@
   "license": "ISC",
   "dependencies": {
     "express": "^4.14.1",
-    "rollbar": "file:../.."
+    "rollbar": "file:../rollbar.tgz"
   },
   "devDependencies": {
     "babel-core": "^6.24.1",
diff --git a/examples/react-19-typescript/package.json b/examples/react-19-typescript/package.json
@@ -51,7 +51,7 @@
     "react-refresh": "^0.11.0",
     "resolve": "^1.20.0",
     "resolve-url-loader": "^4.0.0",
-    "rollbar": "file:../../.pack/rollbar.tgz",
+    "rollbar": "file:../rollbar.tgz",
     "sass-loader": "^12.3.0",
     "semver": "^7.3.5",
     "source-map-loader": "^3.0.0",
diff --git a/examples/react/package.json b/examples/react/package.json
@@ -20,6 +20,6 @@
     "babel-loader": "^7.1.5",
     "react": "^16.8.6",
     "react-dom": "^16.8.6",
-    "rollbar": "file:../../.pack/rollbar.tgz"
+    "rollbar": "file:../rollbar.tgz"
   }
 }
diff --git a/examples/universal-node/.gitignore b/examples/universal-node/.gitignore
@@ -0,0 +1 @@
+dist
diff --git a/examples/universal-node/package.json b/examples/universal-node/package.json
@@ -21,7 +21,7 @@
     "babel-core": "^6.22.1",
     "react": "^0.14.3",
     "react-dom": "^0.14.3",
-    "rollbar": "file:../..",
+    "rollbar": "file:../rollbar.tgz",
     "webpack-node-externals": "^1.5.4"
   },
   "devDependencies": {
diff --git a/examples/vuejs3/.gitignore b/examples/vuejs3/.gitignore
@@ -0,0 +1 @@
+dist
diff --git a/examples/vuejs3/package.json b/examples/vuejs3/package.json
@@ -8,7 +8,7 @@
     "preview": "vite preview"
   },
   "dependencies": {
-    "rollbar": "file:../..",
+    "rollbar": "file:../rollbar.tgz",
     "vue": "^3.3.4"
   },
   "devDependencies": {
diff --git a/examples/webpack/package.json b/examples/webpack/package.json
@@ -6,7 +6,7 @@
   },
   "private": true,
   "dependencies": {
-    "rollbar": "file:../.."
+    "rollbar": "file:../rollbar.tgz"
   },
   "devDependencies": {
     "webpack": "^4.29.6",
diff --git a/package.json b/package.json
@@ -101,7 +101,7 @@
   },
   "scripts": {
     "build": "webpack --mode production",
-    "postbuild": "npm run validate:es5 && npm run update:snippets:examples",
+    "postbuild": "npm run validate:es5 && npm run update:snippets:examples && npm run pack",
     "build:dev": "webpack --mode development",
     "postbuild:dev": "npm run postbuild",
     "test": "./node_modules/.bin/grunt test",
@@ -112,6 +112,7 @@
     "test-replay-integration": "./node_modules/.bin/grunt test-replay-integration",
     "test-ci": "./node_modules/.bin/grunt test && npm run test-server",
     "lint": "./node_modules/.bin/eslint .",
+    "pack": "node scripts/pack.js",
     "validate:es5": "es-check es5 './dist/**/*.js' --verbose",
     "update:snippets:examples": "node scripts/update-snippets.js examples"
   }
diff --git a/scripts/pack.js b/scripts/pack.js
@@ -0,0 +1,34 @@
+#!/usr/bin/env node
+
+/**
+ * Pack the root package, then move the resulting .tgz file into the project’s
+ * `examples` directory as `rollbar.tgz`.
+ *
+ * This assumes all examples point to `rollbar.tgz` in the `examples` directory
+ * for their rollbar package.
+ */
+
+import { rename } from 'node:fs/promises';
+import { fileURLToPath } from 'node:url';
+import path from 'node:path';
+
+import { findUp, npm } from './util.js';
+
+async function pack() {
+  const cwd = path.dirname(fileURLToPath(import.meta.url));
+  const root = await findUp({ fileName: 'package.json', dir: cwd });
+
+  const stdout = await npm(['pack'], { cwd: root });
+  const tgz = stdout.trim().split('\n').pop();
+  const src = path.join(root, tgz);
+  const dest = path.join(root, 'examples', 'rollbar.tgz');
+
+  await rename(src, dest);
+
+  console.log(`Packaged Rollbar into ${path.relative(root, dest)}`);
+}
+
+pack().catch((err) => {
+  console.error('Error packing Rollbar JS:', err);
+  process.exit(1);
+});
diff --git a/scripts/util.js b/scripts/util.js
@@ -1,4 +1,6 @@
 import { access } from 'node:fs/promises';
+import { once } from 'node:events';
+import { spawn } from 'node:child_process';
 import path from 'node:path';
 
 /**
@@ -27,3 +29,48 @@ export async function findUp({ fileName, dir = process.cwd() }) {
 
   throw new Error(`No ${fileName} found in or above ${origin}`);
 }
+
+/**
+ * Runs an npm command in a given directory, capturing output.
+ *
+ * @param {string[]} args - Arguments to pass to npm (e.g., `['install']`, `['run', 'build']`).
+ * @param {Object} options - Options for the npm command.
+ * @param {string} [options.cwd=process.cwd()] - Optional directory to run the command in. Defaults to `process.cwd()`.
+ * @param {string} [options.id] - Optional identifier for error reporting.
+ * @returns {Promise<string>} Resolves with stdout if successful.
+ * @throws {Error} If the npm process fails; error includes stderr or stdout.
+ * @example
+ * // Install dependencies in the current directory:
+ * await npm(['install']);
+ *
+ * // Run a custom npm script in a specific directory:
+ * await npm(['run', 'build'], { cwd: '/path/to/project' });
+ */
+export async function npm(args, { cwd = process.cwd(), id = '' }) {
+  const child = spawn('npm', args, {
+    cwd,
+    shell: true,
+    stdio: ['inherit', 'pipe', 'pipe'],
+  });
+
+  const std = { out: '', err: '' };
+  child.stdout.setEncoding('utf8');
+  child.stderr.setEncoding('utf8');
+  child.stdout.on('data', (chunk) => (std.out += chunk));
+  child.stderr.on('data', (chunk) => (std.err += chunk));
+
+  const [code, signal] = await once(child, 'close');
+
+  if (code !== 0 || signal) {
+    const msg = std.err.trim() || std.out.trim();
+    throw new Error(
+      `npm ` +
+        `${args.join(' ')} ` +
+        `${signal ? `terminated by ${signal}` : `exited with ${code}`}` +
+        `${id ? ` (${id})` : ''}` +
+        `${msg ? `:\n${msg}` : ''}`,
+    );
+  }
+
+  return std.out;
+}

Original file line number	Diff line number	Diff line change
`@@ -20,6 +20,6 @@`
`20`	`20`	`"babel-loader": "^7.1.5",`
`21`	`21`	`"react": "^16.8.6",`
`22`	`22`	`"react-dom": "^16.8.6",`
`23`		`- "rollbar": "file:../../.pack/rollbar.tgz"`
	`23`	`+ "rollbar": "file:../rollbar.tgz"`
`24`	`24`	`}`
`25`	`25`	`}`