@cplieger/actions 2.0.0 → 2.0.2

package/README.md

@@ -1,9 +1,10 @@
 # actions
 
-[![CI](https://github.com/cplieger/actions/actions/workflows/ci.yaml/badge.svg)](https://github.com/cplieger/actions/actions/workflows/ci.yaml)
 [![npm](https://img.shields.io/npm/v/@cplieger/actions)](https://www.npmjs.com/package/@cplieger/actions)
 [![JSR](https://jsr.io/badges/@cplieger/actions)](https://jsr.io/@cplieger/actions)
-[![License: GPL-3.0](https://img.shields.io/badge/License-GPL--3.0-blue.svg)](./LICENSE)
+[![Test coverage](https://img.shields.io/endpoint?url=https://raw.githubusercontent.com/cplieger/actions/badges/coverage.json)](https://github.com/cplieger/actions/actions/workflows/coverage.yml)
+[![OpenSSF Best Practices](https://www.bestpractices.dev/projects/13197/badge)](https://www.bestpractices.dev/projects/13197)
+[![OpenSSF Scorecard](https://api.scorecard.dev/projects/github.com/cplieger/actions/badge)](https://scorecard.dev/viewer/?uri=github.com/cplieger/actions)
 
 > Declarative UI-actions framework with lifecycle management, retry, debounce, and polling.
 

package/jsr.json

@@ -1,6 +1,6 @@
 {
   "name": "@cplieger/actions",
-  "version": "2.0.0",
+  "version": "2.0.2",
   "exports": {
     ".": "./src/index.ts",
     "./testing": "./src/testing.ts"

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cplieger/actions",
-  "version": "2.0.0",
+  "version": "2.0.2",
   "type": "module",
   "sideEffects": false,
   "exports": {
@@ -13,18 +13,18 @@
     "test": "vitest --run"
   },
   "dependencies": {
-    "@cplieger/reactive": "1.1.0"
+    "@cplieger/reactive": "1.2.0"
   },
   "devDependencies": {
     "@eslint/js": "10.0.1",
-    "@types/node": "24.13.1",
-    "@vitest/coverage-v8": "4.1.8",
-    "eslint": "10.4.1",
+    "@types/node": "24.13.2",
+    "@vitest/coverage-v8": "4.1.9",
+    "eslint": "10.5.0",
     "fast-check": "4.8.0",
-    "happy-dom": "20.10.2",
-    "prettier": "3.8.3",
-    "typescript-eslint": "8.60.1",
-    "vitest": "4.1.8"
+    "happy-dom": "20.10.3",
+    "prettier": "3.8.4",
+    "typescript-eslint": "8.61.1",
+    "vitest": "4.1.9"
   },
   "description": "Declarative async UI-action framework for TypeScript",
   "license": "GPL-3.0-or-later",

package/src/registry.ts

@@ -33,6 +33,13 @@ let _pendingTotal = 0;
 let _liveCount = 0;
 let _head = 0;
 
+// Test-only: when true, a detected _pendingTotal underflow throws instead of
+// the production warn + clamp, so an add/removePending pairing bug surfaces in
+// tests rather than being silently masked. Production default is false.
+// Toggled via _setThrowOnInvariantViolationForTest (same _*ForTest convention
+// as _resetForTest); reset to false by _resetForTest.
+let _throwOnInvariantViolation = false;
+
 // Reactive mirrors of the pending state. pendingByName remains the source of
 // truth; these signals expose the derived counts so isPending/pendingCount can
 // be read reactively (e.g. by bindLoadingState's effect).
@@ -153,6 +160,9 @@ export function record(instance: ActionInstance): void {
     compact();
   }
   if (_pendingTotal < 0) {
+    if (_throwOnInvariantViolation) {
+      throw new Error("[actions] _pendingTotal went negative — invariant violation");
+    }
     console.warn("[actions] _pendingTotal went negative — invariant violation; clamping to 0");
     _pendingTotal = 0;
     pendingTotalSig.value = 0;
@@ -243,6 +253,26 @@ export function _resetForTest(): void {
   listeners.clear();
   namedListeners.clear();
   _pendingTotal = 0;
+  _throwOnInvariantViolation = false;
   pendingSigs.clearAll();
   pendingTotalSig.value = 0;
 }
+
+/**
+ * @internal Test-only: toggle whether a detected `_pendingTotal` underflow
+ * throws (true) instead of the production warn + clamp (false). Lets tests
+ * assert that an add/removePending pairing bug surfaces loudly. Reset to false
+ * by `_resetForTest`.
+ */
+export function _setThrowOnInvariantViolationForTest(enabled: boolean): void {
+  _throwOnInvariantViolation = enabled;
+}
+
+/**
+ * @internal Test-only: simulate an unpaired `removePending` (an add/remove
+ * mismatch) by decrementing the pending total without a matching add, driving
+ * the invariant negative. The next `record()` then hits the clamp branch.
+ */
+export function _forcePendingUnderflowForTest(): void {
+  _pendingTotal--;
+}