Keep dependency refresh surgical and verify sample logger

Author: niemyjski

.github/workflows/build.yml

@@ -1,31 +1,29 @@
 name: Build
 on: [push, pull_request]
 
-concurrency:
-  group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}
-  cancel-in-progress: true
-
 permissions:
   contents: read
+  id-token: write
+  packages: write
 
 jobs:
-  version:
-    name: Determine build version
-    runs-on: ubuntu-latest
-    outputs:
-      version: ${{ steps.version.outputs.version }}
+  build:
+    runs-on: ${{ matrix.os }}
+    strategy:
+      matrix:
+        os:
+          - ubuntu-latest
+          - macos-latest
+          - windows-latest
+        node_version:
+          - 24
+    name: Node ${{ matrix.node_version }} on ${{ matrix.os }}
     steps:
       - name: Checkout
-        uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7.0.1
+        uses: actions/checkout@v7
         with:
           fetch-depth: 0
-          persist-credentials: false
-      - name: Setup .NET SDK for MinVer
-        uses: actions/setup-dotnet@a98b56852c35b8e3190ac28c8c2271da59106c68 # v6.0.0
-        with:
-          dotnet-version: "10.0.x"
-      - name: Build Version
-        id: version
+      - name: Build Reason
         shell: bash
         run: |
           branch=${GITHUB_REF##*/}.
@@ -38,124 +36,68 @@ jobs:
           elif [[ "${GITHUB_REF}" = refs/pull* ]]; then
             branch=""
           fi
-
+          echo "GIT_BRANCH_SUFFIX=$branch" >> $GITHUB_ENV
+          echo "ref: $GITHUB_REF event: $GITHUB_EVENT_NAME branch_suffix: $branch"
+      - name: Setup Node.js environment
+        uses: actions/setup-node@v7
+        with:
+          node-version: ${{ matrix.node_version }}
+          registry-url: "https://registry.npmjs.org"
+      - name: Cache node_modules
+        uses: actions/cache@v6
+        with:
+          path: node_modules
+          key: ${{ matrix.node_version }}-${{ runner.os }}-node-modules-${{ hashFiles('package-lock.json') }}
+      - name: Setup .NET SDK for MinVer
+        uses: actions/setup-dotnet@v6
+        with:
+          dotnet-version: "10.0.x"
+      - name: Build Version
+        id: version
+        shell: bash
+        run: |
           dotnet tool install --global minver-cli --version 7.0.0
-          version=$(minver --tag-prefix v --default-pre-release-identifiers "preview.${branch}0" --minimum-major-minor 3.0)
+          version=$(minver --tag-prefix v --default-pre-release-identifiers "preview.${GIT_BRANCH_SUFFIX}0" --minimum-major-minor 3.0)
 
-          if [ -n "$branch" ]; then
-            branch_name="${branch%.}"
+          # If on a non-main branch, insert branch name before the height (last numeric segment)
+          if [ -n "$GIT_BRANCH_SUFFIX" ]; then
+            branch_name="${GIT_BRANCH_SUFFIX%.}"
             if [[ "$version" != *"$branch_name"* ]]; then
-              version=$(echo "$version" | sed -E "s/\.([0-9]+)$/.${branch}\1/")
+              version=$(echo "$version" | sed -E "s/\.([0-9]+)$/.${GIT_BRANCH_SUFFIX}\1/")
             fi
           fi
 
-          echo "version=$version" >> "$GITHUB_OUTPUT"
+          echo "version=$version" >> $GITHUB_OUTPUT
           echo "Version: $version"
-          echo "### Version: $version" >> "$GITHUB_STEP_SUMMARY"
+          echo "### Version: $version" >> $GITHUB_STEP_SUMMARY
 
-  build:
-    needs: version
-    runs-on: ${{ matrix.os }}
-    strategy:
-      matrix:
-        os:
-          - ubuntu-latest
-          - macos-latest
-          - windows-latest
-        node_version:
-          - 24
-    name: Node ${{ matrix.node_version }} on ${{ matrix.os }}
-    steps:
-      - name: Checkout
-        uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7.0.1
-        with:
-          persist-credentials: false
-      - name: Setup Node.js environment
-        uses: actions/setup-node@820762786026740c76f36085b0efc47a31fe5020 # v7.0.0
-        with:
-          node-version: ${{ matrix.node_version }}
-          registry-url: "https://registry.npmjs.org"
-          cache: npm
-          cache-dependency-path: package-lock.json
-      - name: Apply Build Version
-        env:
-          BUILD_VERSION: ${{ needs.version.outputs.version }}
-        run: node scripts/set-build-version.mjs
-      - name: Install Dependencies
-        run: npm ci
+          npm install --global replace-in-files-cli
+          replace-in-files --string="3.0.0-dev" --replacement=$version packages/core/src/configuration/Configuration.ts
+          replace-in-files --string="3.0.0-dev" --replacement=$version **/package*.json
+          npm ci
       - name: Build
         run: npm run build
       - name: Lint
         run: npm run lint
       - name: Run Tests
         run: npm test
-
-  publish-release:
-    name: Publish npm release
-    if: startsWith(github.ref, 'refs/tags/v')
-    needs: [version, build]
-    runs-on: ubuntu-latest
-    permissions:
-      contents: read
-      id-token: write # Required for npm trusted publishing.
-    steps:
-      - name: Checkout
-        uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7.0.1
-        with:
-          persist-credentials: false
-      - name: Setup Node.js environment
-        uses: actions/setup-node@820762786026740c76f36085b0efc47a31fe5020 # v7.0.0
-        with:
-          node-version: 24
-          registry-url: "https://registry.npmjs.org"
-          cache: npm
-          cache-dependency-path: package-lock.json
-      - name: Apply Build Version
-        env:
-          BUILD_VERSION: ${{ needs.version.outputs.version }}
-        run: node scripts/set-build-version.mjs
-      - name: Install Dependencies
-        run: npm ci
-      - name: Build
-        run: npm run build
       - name: Publish Release Packages
+        if: startsWith(github.ref, 'refs/tags/v') && matrix.os == 'ubuntu-latest'
         run: npm publish --workspaces --access public
-
-  publish-github:
-    name: Publish GitHub CI packages
-    if: github.event_name == 'push' && startsWith(github.ref, 'refs/heads/') && contains(needs.version.outputs.version, '-')
-    needs: [version, build]
-    runs-on: ubuntu-latest
-    permissions:
-      contents: read
-      packages: write # Required to publish with GITHUB_TOKEN.
-    steps:
-      - name: Checkout
-        uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7.0.1
-        with:
-          persist-credentials: false
-      - name: Setup Node.js environment
-        uses: actions/setup-node@820762786026740c76f36085b0efc47a31fe5020 # v7.0.0
+      - name: Setup GitHub CI Node.js environment
+        if: github.event_name != 'pull_request' && startsWith(github.ref, 'refs/heads/') && matrix.os == 'ubuntu-latest' && contains(steps.version.outputs.version, '-')
+        uses: actions/setup-node@v7
         with:
-          node-version: 24
-          registry-url: "https://registry.npmjs.org"
-          cache: npm
-          cache-dependency-path: package-lock.json
-      - name: Apply Build Version
-        env:
-          BUILD_VERSION: ${{ needs.version.outputs.version }}
-        run: node scripts/set-build-version.mjs
-      - name: Install Dependencies
-        run: npm ci
-      - name: Build
-        run: npm run build
-      - name: Setup GitHub Packages registry
-        uses: actions/setup-node@820762786026740c76f36085b0efc47a31fe5020 # v7.0.0
-        with:
-          node-version: 24
+          node-version: ${{ matrix.node_version }}
           registry-url: "https://npm.pkg.github.com"
           scope: "@exceptionless"
       - name: Push GitHub CI Packages
-        run: node scripts/publish-ci-packages.mjs # zizmor: ignore[use-trusted-publishing] GitHub Packages uses GITHUB_TOKEN.
+        if: github.event_name != 'pull_request' && startsWith(github.ref, 'refs/heads/') && matrix.os == 'ubuntu-latest' && contains(steps.version.outputs.version, '-')
+        shell: bash
+        run: |
+          TAG_BRANCH="${GIT_BRANCH_SUFFIX%.}"
+          TAG_BRANCH="${TAG_BRANCH:-main}"
+          TAG_BRANCH="${TAG_BRANCH//\//-}"
+          npm publish --workspaces --access public --tag "ci-${TAG_BRANCH}" || true
         env:
-          NODE_AUTH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+          NODE_AUTH_TOKEN: ${{secrets.GITHUB_TOKEN}}

.npmrc

@@ -1,2 +1,3 @@
 legacy-peer-deps=true
 min-release-age=7
+min-release-age-exclude[]=source-map-js

eslint.config.mjs

@@ -44,17 +44,6 @@ export default defineConfig(
       ]
     }
   },
-  {
-    files: ["scripts/**/*.mjs"],
-    ...tseslint.configs.disableTypeChecked,
-    languageOptions: {
-      ...tseslint.configs.disableTypeChecked.languageOptions,
-      globals: {
-        console: "readonly",
-        process: "readonly"
-      }
-    }
-  },
   eslintConfigPrettier,
   {
     plugins: {

example/browser/index.html

@@ -15,7 +15,7 @@ <h1>Error Submission</h1>
     <button onclick="throwReferenceError('function-does-exist')">Throw uncaught reference error</button>
     <button id="throw-jquery-ajax-error">Throw jQuery ajax error</button>
     <button id="throw-promise-unhandled-rejection">Throw promise unhandled rejection</button>
-    <button id="throw-browser-extension-error">Throw browser extension error</button>
+    <button id="throw-browser-extension-error">Throw brower extension error</button>
 
     <h1>Log Submission</h1>
     <button class="submit-log">Submit log event</button>

example/browser/package.json

@@ -17,7 +17,7 @@
     "access": "restricted"
   },
   "devDependencies": {
-    "vite": "^8.2.2"
+    "vite": "^8.3.1"
   },
   "dependencies": {
     "@exceptionless/browser": "3.0.0-dev",

example/browser/test/text-area-logger.test.js

@@ -0,0 +1,64 @@
+import assert from "node:assert/strict";
+import { afterEach, test } from "vitest";
+import { JSDOM } from "jsdom";
+
+import { TextAreaLogger } from "../text-area-logger.js";
+
+let dom;
+afterEach(() => {
+  dom?.window.close();
+  delete globalThis.document;
+});
+
+function createDocument(readyState) {
+  dom = new JSDOM('<textarea id="log"></textarea>');
+  globalThis.document = dom.window.document;
+  Object.defineProperty(document, "readyState", { value: readyState });
+  return document.getElementById("log");
+}
+
+test("logs literal text and preserves the textarea's edited value", () => {
+  const element = createDocument("complete");
+  element.value = "edited text";
+  const logger = new TextAreaLogger("log");
+  logger.info("<b>literal</b> & text");
+  assert.match(element.value, /^edited text\n.* \[info\] <b>literal<\/b> & text$/);
+  assert.equal(element.innerHTML, "");
+});
+
+test("flushes buffered messages when the document becomes ready", () => {
+  const element = createDocument("loading");
+  const logger = new TextAreaLogger("log");
+  logger.info("<script>literal</script>");
+  logger.warn("second message");
+  assert.equal(element.value, "");
+  document.dispatchEvent(new dom.window.Event("DOMContentLoaded"));
+  assert.match(element.value, /\[info\] <script>literal<\/script>\n.* \[warn\] second message$/);
+  assert.deepEqual(logger.messageBuffer, []);
+  logger.error("third message");
+  assert.match(element.value, /second message\n.* \[error\] third message$/);
+});
+
+test("logs immediately when DOMContentLoaded has already fired", () => {
+  const element = createDocument("interactive");
+  new TextAreaLogger("log").info("ready");
+  assert.match(element.value, /\[info\] ready$/);
+});
+
+test("forwards every level to the supplied logger", () => {
+  createDocument("complete");
+  const calls = [];
+  const target = Object.fromEntries(["trace", "info", "warn", "error"].map((level) => [level, (message) => calls.push([level, message])]));
+  const logger = new TextAreaLogger("log", target);
+  for (const level of ["trace", "info", "warn", "error"]) logger[level](level);
+  assert.deepEqual(calls, [
+    ["trace", "trace"],
+    ["info", "info"],
+    ["warn", "warn"],
+    ["error", "error"]
+  ]);
+});
+
+test("requires an element id", () => {
+  assert.throws(() => new TextAreaLogger(""), /elementId is required/);
+});

example/browser/text-area-logger.js

@@ -6,7 +6,7 @@ export class TextAreaLogger {
 
     this.logger = logger;
     this.messageBuffer = [];
-    if (document.readyState === "complete") {
+    if (document.readyState !== "loading") {
       this.element = document.getElementById(elementId);
     } else {
       document.addEventListener("DOMContentLoaded", () => {

example/expo/package.json

@@ -14,11 +14,11 @@
   },
   "dependencies": {
     "@exceptionless/react-native": "3.0.0-dev",
-    "@expo/metro-runtime": "~57.0.15",
+    "@expo/metro-runtime": "~57.0.16",
     "@react-native-async-storage/async-storage": "2.2.0",
-    "expo": "~57.0.22",
+    "expo": "~57.0.25",
     "expo-application": "~57.0.3",
-    "expo-constants": "~57.0.18",
+    "expo-constants": "~57.0.19",
     "expo-dev-client": "~57.0.19",
     "expo-device": "~57.0.2",
     "expo-splash-screen": "~57.0.9",
@@ -28,7 +28,7 @@
     "react-native": "0.86.3",
     "react-native-safe-area-context": "~5.7.0",
     "react-native-screens": "~4.26.2",
-    "react-native-web": "^0.21.2"
+    "react-native-web": "^0.21.3"
   },
   "devDependencies": {
     "@react-native/js-polyfills": "^0.86.3",