synstate 6.2.1 → 7.0.1

package/src/core/predefined/operators/skip.mts

@@ -1,4 +1,4 @@
-import { PositiveSafeInt } from 'ts-data-forge';
+import { isPositiveSafeInt } from 'ts-data-forge';
 import type { PositiveSafeIntWithSmallInt } from 'ts-type-forge';
 import { skipWhile } from '../../operators/index.mjs';
 import type { DropInitialValueOperator } from '../../types/index.mjs';
@@ -53,6 +53,6 @@ import type { DropInitialValueOperator } from '../../types/index.mjs';
 export const skip = <A,>(
   n: PositiveSafeIntWithSmallInt,
 ): DropInitialValueOperator<A, A> =>
-  !PositiveSafeInt.is(n) ? idFn : skipWhile((_, index) => index + 1 <= n);
+  !isPositiveSafeInt(n) ? idFn : skipWhile((_, index) => index + 1 <= n);
 
 const idFn = <T,>(value: T): T => value;

package/src/core/utils/index.mts

@@ -1,5 +1,7 @@
 export * from './current-scope.mjs';
 export * from './id-maker.mjs';
+export * from './is-development-build.mjs';
 export * from './max-depth.mjs';
 export * from './observable-brand.mjs';
 export * from './observable-utils.mjs';
+export * from './rethrow-out-of-promise.mjs';

package/src/core/utils/is-development-build.mts

@@ -0,0 +1,32 @@
+/**
+ * Whether this is a development build: `NODE_ENV` is not `production`.
+ *
+ * `process.env.NODE_ENV` is written out in full because that is the expression
+ * bundlers (Vite, webpack, esbuild) replace, in the application's build and in
+ * a pre-bundled dependency alike: with the string `"development"` in a
+ * development build and `"production"` in a production one. Where nothing
+ * replaces it and there is no `process` (an unbundled module in a browser),
+ * reading it throws and the answer is `false`.
+ *
+ * synstate-router reads it the same way
+ * (`libs/synstate-router/src/internal/is-development.mts`).
+ */
+export const isDevelopmentBuild = (): boolean => {
+  // @sumi-expect-error exceptions/no-try
+  try {
+    return process.env.NODE_ENV !== 'production';
+  } catch {
+    return false;
+  }
+};
+
+/**
+ * The one part of Node.js's `process` read here, declared in this module so
+ * that the dot form type-checks without the Node.js types (which this package's
+ * Sumi check does not load) and without the bracket form their index signature
+ * asks for, which bundlers do not replace. `declare` emits nothing:
+ * the emitted code reads the global `process`.
+ */
+declare const process: Readonly<{
+  env: Readonly<{ NODE_ENV?: string | undefined }>;
+}>;

package/src/core/utils/is-development-build.test.mts

@@ -0,0 +1,44 @@
+// cspell:ignore unstub
+import { isDevelopmentBuild } from './is-development-build.mjs';
+
+describe(isDevelopmentBuild, () => {
+  test('is true when NODE_ENV is development', () => {
+    restoreAfterTest();
+
+    vi.stubEnv('NODE_ENV', 'development');
+
+    assert.isTrue(isDevelopmentBuild());
+  });
+
+  test('is true when NODE_ENV is not set', () => {
+    restoreAfterTest();
+
+    vi.stubEnv('NODE_ENV', undefined);
+
+    assert.isTrue(isDevelopmentBuild());
+  });
+
+  test('is false when NODE_ENV is production', () => {
+    restoreAfterTest();
+
+    vi.stubEnv('NODE_ENV', 'production');
+
+    assert.isFalse(isDevelopmentBuild());
+  });
+
+  test('is false where there is no process and nothing replaced the expression', () => {
+    restoreAfterTest();
+
+    vi.stubGlobal('process', undefined);
+
+    assert.isFalse(isDevelopmentBuild());
+  });
+});
+
+const restoreAfterTest = (): void => {
+  onTestFinished(() => {
+    vi.unstubAllEnvs();
+
+    vi.unstubAllGlobals();
+  });
+};

package/src/core/utils/rethrow-out-of-promise.mts

@@ -0,0 +1,16 @@
+/**
+ * Throws `error` again from a task of its own, for a `.catch` after each step
+ * of a promise chain that runs subscribers (`fromPromise`): after the step that
+ * emits and after the one that completes, so that neither exception replaces
+ * the other. A subscriber's exception then surfaces as an uncaught error, as
+ * one thrown during an update a timer started does (`debounce`, `timer`),
+ * rather than as a rejection or not at all.
+ */
+export const rethrowOutOfPromise = (error: unknown): void => {
+  setTimeout(() => {
+    // The subscriber's own exception, passed on unchanged.
+    // @sumi-expect-error exceptions/no-throw
+    // eslint-disable-next-line @typescript-eslint/only-throw-error -- the subscriber's own exception, passed on unchanged
+    throw error;
+  }, 0);
+};