@oscarpalmer/timer 0.31.0 → 0.33.0

package/dist/timer.js

@@ -1,8 +1,8 @@
 import { noop } from '@oscarpalmer/atoms/function';
+import { stop, work } from './work.js';
 
 class Timer {
-  constructor(type, worker, state, options, start) {
-    this.worker = worker;
+  constructor(type, state, options, start) {
     this.options = options;
     this.$timer = type;
     this.state = {
@@ -58,8 +58,17 @@ class Timer {
     this.options.onAfter = noop;
     this.options.onError = noop;
     this.state.callback = noop;
-    this.state.trace = void 0;
-    this.#work("stop");
+    if (!globalThis._oscarpalmer_timer_debug) {
+      this.state.trace = void 0;
+    }
+    stop(
+      {
+        instance: this,
+        type: this.$timer
+      },
+      this.state,
+      this.options
+    );
   }
   /**
    * Pause the timer _(if it's running)_
@@ -86,7 +95,7 @@ class Timer {
     return this.#work("stop");
   }
   #work(type) {
-    return this.worker(
+    return work(
       type,
       {
         instance: this,

package/dist/wait.cjs

@@ -4,14 +4,13 @@ Object.defineProperty(exports, Symbol.toStringTag, { value: 'Module' });
 
 const constants = require('./constants.cjs');
 const get = require('./get.cjs');
+require('./global.cjs');
 const models = require('./models.cjs');
 const timer = require('./timer.cjs');
-const work = require('./work.cjs');
 
 function wait(callback, time) {
   return new timer.Timer(
     "wait",
-    work.work,
     {
       callback: get.getCallback(callback),
       trace: new models.TimerTrace().stack

package/dist/wait.js

@@ -1,13 +1,12 @@
 import { milliseconds } from './constants.js';
 import { getCallback, getValidNumber } from './get.js';
+import './global.js';
 import { TimerTrace } from './models.js';
 import { Timer } from './timer.js';
-import { work } from './work.js';
 
 function wait(callback, time) {
   return new Timer(
     "wait",
-    work,
     {
       callback: getCallback(callback),
       trace: new TimerTrace().stack

package/dist/when.cjs

@@ -5,13 +5,19 @@ Object.defineProperty(exports, Symbol.toStringTag, { value: 'Module' });
 const _function = require('@oscarpalmer/atoms/function');
 const constants = require('./constants.cjs');
 const get = require('./get.cjs');
+require('./global.cjs');
 const models = require('./models.cjs');
 const timer = require('./timer.cjs');
-const work = require('./work.cjs');
 
 class When {
   $timer = "when";
-  state;
+  state = {
+    promise: void 0,
+    rejecter: void 0,
+    resolver: void 0,
+    started: false,
+    timer: void 0
+  };
   /**
    * Is the timer active?
    */
@@ -36,8 +42,47 @@ class When {
   get trace() {
     return globalThis._oscarpalmer_timer_debug ?? false ? this.state.timer?.trace : void 0;
   }
-  constructor(state) {
-    this.state = state;
+  constructor(condition, options) {
+    const { state } = this;
+    state.promise = new Promise((resolve, reject) => {
+      state.resolver = resolve;
+      state.rejecter = reject;
+    });
+    let result = false;
+    this.state.timer = new timer.Timer(
+      "when",
+      {
+        callback() {
+          try {
+            if (condition()) {
+              result = true;
+              state.timer.stop();
+            }
+          } catch {
+            state.timer.stop();
+          }
+        },
+        trace: new models.TimerTrace().stack
+      },
+      {
+        onAfter: () => {
+          if (result) {
+            state.resolver?.();
+          } else {
+            state.rejecter?.();
+          }
+          this.destroy();
+        },
+        onError: () => {
+          state.rejecter?.();
+          this.destroy();
+        },
+        count: get.getValidNumber(options?.count),
+        interval: get.getValidNumber(options?.interval, constants.milliseconds),
+        timeout: get.getValidNumber(options?.timeout)
+      },
+      false
+    );
   }
   /**
    * Continues the timer _(if it was paused)_
@@ -50,11 +95,12 @@ class When {
    * Destroys the timer _(and stops it,if it was running)_
    */
   destroy() {
-    this.state.timer?.destroy();
-    this.state.promise = void 0;
-    this.state.resolver = _function.noop;
-    this.state.rejecter = _function.noop;
-    this.state.timer = void 0;
+    const { state } = this;
+    state.timer?.destroy();
+    state.promise = void 0;
+    state.resolver = _function.noop;
+    state.rejecter = _function.noop;
+    state.timer = void 0;
   }
   /**
    * Pauses the timer _(if it was running)_
@@ -75,63 +121,20 @@ class When {
    */
   // biome-ignore lint/suspicious/noThenProperty: returning a promise-like object, so it's ok ;)
   then(resolve, reject) {
-    if (this.state.timer == null || this.state?.started) {
-      throw new Error(
-        this.state.timer == null ? constants.destroyedMessage : constants.startedMessage
-      );
+    const { state } = this;
+    if (state.timer == null) {
+      throw new Error(constants.destroyedMessage);
+    }
+    if (state.started) {
+      throw new Error(constants.startedMessage);
     }
-    this.state.started = true;
-    this.state.timer.start();
-    return this.state.promise.then(resolve ?? _function.noop, reject ?? _function.noop);
+    state.started = true;
+    state.timer.start();
+    return state.promise.then(resolve ?? _function.noop, reject ?? _function.noop);
   }
 }
 function when(condition, options) {
-  let called = false;
-  let result = false;
-  const state = {
-    started: false,
-    timer: new timer.Timer(
-      "when",
-      work.work,
-      {
-        callback() {
-          if (condition()) {
-            result = true;
-            state.timer.stop();
-          }
-        },
-        trace: new models.TimerTrace().stack
-      },
-      {
-        onAfter() {
-          if (!(state.timer?.paused ?? false) && !called) {
-            called = true;
-            if (result) {
-              state.resolver?.();
-            } else {
-              state.rejecter?.();
-            }
-            instance.destroy();
-          }
-        },
-        onError() {
-          state.rejecter?.();
-          instance.destroy();
-        },
-        count: get.getValidNumber(options?.count),
-        interval: get.getValidNumber(options?.interval, constants.milliseconds),
-        timeout: get.getValidNumber(options?.timeout)
-      },
-      false
-    )
-  };
-  const promise = new Promise((resolve, reject) => {
-    state.resolver = resolve;
-    state.rejecter = reject;
-  });
-  state.promise = promise;
-  const instance = new When(state);
-  return instance;
+  return new When(condition, options);
 }
 
 exports.when = when;

package/dist/when.js

@@ -1,13 +1,19 @@
 import { noop } from '@oscarpalmer/atoms/function';
 import { milliseconds, destroyedMessage, startedMessage } from './constants.js';
 import { getValidNumber } from './get.js';
+import './global.js';
 import { TimerTrace } from './models.js';
 import { Timer } from './timer.js';
-import { work } from './work.js';
 
 class When {
   $timer = "when";
-  state;
+  state = {
+    promise: void 0,
+    rejecter: void 0,
+    resolver: void 0,
+    started: false,
+    timer: void 0
+  };
   /**
    * Is the timer active?
    */
@@ -32,8 +38,47 @@ class When {
   get trace() {
     return globalThis._oscarpalmer_timer_debug ?? false ? this.state.timer?.trace : void 0;
   }
-  constructor(state) {
-    this.state = state;
+  constructor(condition, options) {
+    const { state } = this;
+    state.promise = new Promise((resolve, reject) => {
+      state.resolver = resolve;
+      state.rejecter = reject;
+    });
+    let result = false;
+    this.state.timer = new Timer(
+      "when",
+      {
+        callback() {
+          try {
+            if (condition()) {
+              result = true;
+              state.timer.stop();
+            }
+          } catch {
+            state.timer.stop();
+          }
+        },
+        trace: new TimerTrace().stack
+      },
+      {
+        onAfter: () => {
+          if (result) {
+            state.resolver?.();
+          } else {
+            state.rejecter?.();
+          }
+          this.destroy();
+        },
+        onError: () => {
+          state.rejecter?.();
+          this.destroy();
+        },
+        count: getValidNumber(options?.count),
+        interval: getValidNumber(options?.interval, milliseconds),
+        timeout: getValidNumber(options?.timeout)
+      },
+      false
+    );
   }
   /**
    * Continues the timer _(if it was paused)_
@@ -46,11 +91,12 @@ class When {
    * Destroys the timer _(and stops it,if it was running)_
    */
   destroy() {
-    this.state.timer?.destroy();
-    this.state.promise = void 0;
-    this.state.resolver = noop;
-    this.state.rejecter = noop;
-    this.state.timer = void 0;
+    const { state } = this;
+    state.timer?.destroy();
+    state.promise = void 0;
+    state.resolver = noop;
+    state.rejecter = noop;
+    state.timer = void 0;
   }
   /**
    * Pauses the timer _(if it was running)_
@@ -71,63 +117,20 @@ class When {
    */
   // biome-ignore lint/suspicious/noThenProperty: returning a promise-like object, so it's ok ;)
   then(resolve, reject) {
-    if (this.state.timer == null || this.state?.started) {
-      throw new Error(
-        this.state.timer == null ? destroyedMessage : startedMessage
-      );
+    const { state } = this;
+    if (state.timer == null) {
+      throw new Error(destroyedMessage);
+    }
+    if (state.started) {
+      throw new Error(startedMessage);
     }
-    this.state.started = true;
-    this.state.timer.start();
-    return this.state.promise.then(resolve ?? noop, reject ?? noop);
+    state.started = true;
+    state.timer.start();
+    return state.promise.then(resolve ?? noop, reject ?? noop);
   }
 }
 function when(condition, options) {
-  let called = false;
-  let result = false;
-  const state = {
-    started: false,
-    timer: new Timer(
-      "when",
-      work,
-      {
-        callback() {
-          if (condition()) {
-            result = true;
-            state.timer.stop();
-          }
-        },
-        trace: new TimerTrace().stack
-      },
-      {
-        onAfter() {
-          if (!(state.timer?.paused ?? false) && !called) {
-            called = true;
-            if (result) {
-              state.resolver?.();
-            } else {
-              state.rejecter?.();
-            }
-            instance.destroy();
-          }
-        },
-        onError() {
-          state.rejecter?.();
-          instance.destroy();
-        },
-        count: getValidNumber(options?.count),
-        interval: getValidNumber(options?.interval, milliseconds),
-        timeout: getValidNumber(options?.timeout)
-      },
-      false
-    )
-  };
-  const promise = new Promise((resolve, reject) => {
-    state.resolver = resolve;
-    state.rejecter = reject;
-  });
-  state.promise = promise;
-  const instance = new When(state);
-  return instance;
+  return new When(condition, options);
 }
 
 export { when };

package/dist/work.cjs

@@ -4,6 +4,16 @@ Object.defineProperty(exports, Symbol.toStringTag, { value: 'Module' });
 
 const constants = require('./constants.cjs');
 
+function endOrRestart(type, timer, state, options) {
+  cancelAnimationFrame(state.frame);
+  if (type === "stop") {
+    options.onAfter?.(false);
+  }
+  state.active = false;
+  state.frame = void 0;
+  state.paused = type === "pause";
+  return type === "restart" ? work("start", timer, state, options) : timer.instance;
+}
 function finish(timer, state, options, success) {
   constants.activeTimers.delete(timer.instance);
   state.active = false;
@@ -15,46 +25,29 @@ function finish(timer, state, options, success) {
     options.onAfter?.(success);
   }
 }
-function work(type, timer, state, options) {
-  if (state.destroyed && type !== "stop" || (state.active ? constants.beginTypes.has(type) : constants.endTypes.has(type))) {
-    return timer.instance;
-  }
-  const pausable = constants.pauseTypes.has(type);
-  state.elapsed = pausable ? state.elapsed : 0;
-  state.index = pausable ? state.index : 0;
-  state.total = pausable ? state.total : 0;
-  if (constants.endOrRestartTypes.has(type)) {
-    constants.activeTimers.delete(timer.instance);
-    cancelAnimationFrame(state.frame);
-    if (type === "stop") {
-      options.onAfter?.(false);
-    }
-    state.active = false;
-    state.frame = void 0;
-    state.paused = type === "pause";
-    return type === "restart" ? work("start", timer, state, options) : timer.instance;
-  }
-  state.active = true;
-  state.paused = false;
-  let start;
-  function step(now) {
+function ignore(type, state) {
+  return state.destroyed && type !== "stop" || (state.active ? constants.beginTypes.has(type) : constants.endTypes.has(type));
+}
+function run(timer, state, options) {
+  let last;
+  return function step(now) {
     if (!state.active) {
       return;
     }
-    start ??= now;
-    const difference = now - start;
+    last ??= now;
+    const difference = now - last;
     state.elapsed += difference;
     state.total += difference;
+    last = now;
     if (options.timeout > 0 && state.total >= options.timeout) {
       options.onError?.();
       finish(timer, state, options, false);
       return;
     }
-    if (options.interval === constants.milliseconds || state.elapsed >= options.interval - 5) {
+    if (options.interval === constants.milliseconds || state.elapsed >= options.interval - constants.intervalBuffer) {
       if (options.count > -1) {
         state.callback(state.index);
       }
-      start = now;
       state.elapsed = 0;
       state.index += 1;
       if (options.count === -1 || options.count > 0 && state.index >= options.count) {
@@ -63,10 +56,32 @@ function work(type, timer, state, options) {
       }
     }
     state.frame = requestAnimationFrame(step);
+  };
+}
+function setState(type, state) {
+  const pausable = constants.pauseTypes.has(type);
+  state.elapsed = pausable ? state.elapsed : 0;
+  state.index = pausable ? state.index : 0;
+  state.total = pausable ? state.total : 0;
+}
+function stop(timer, state, options) {
+  endOrRestart("stop", timer, state, options);
+}
+function work(type, timer, state, options) {
+  if (ignore(type, state)) {
+    return timer.instance;
   }
+  setState(type, state);
+  if (constants.endOrRestartTypes.has(type)) {
+    return endOrRestart(type, timer, state, options);
+  }
+  state.active = true;
+  state.paused = false;
   constants.activeTimers.add(timer.instance);
-  state.frame = requestAnimationFrame(step);
+  const runner = run(timer, state, options);
+  state.frame = requestAnimationFrame(runner);
   return timer.instance;
 }
 
+exports.stop = stop;
 exports.work = work;

package/dist/work.js

@@ -1,5 +1,15 @@
-import { beginTypes, endTypes, pauseTypes, endOrRestartTypes, activeTimers, milliseconds } from './constants.js';
+import { endOrRestartTypes, activeTimers, beginTypes, endTypes, pauseTypes, milliseconds, intervalBuffer } from './constants.js';
 
+function endOrRestart(type, timer, state, options) {
+  cancelAnimationFrame(state.frame);
+  if (type === "stop") {
+    options.onAfter?.(false);
+  }
+  state.active = false;
+  state.frame = void 0;
+  state.paused = type === "pause";
+  return type === "restart" ? work("start", timer, state, options) : timer.instance;
+}
 function finish(timer, state, options, success) {
   activeTimers.delete(timer.instance);
   state.active = false;
@@ -11,46 +21,29 @@ function finish(timer, state, options, success) {
     options.onAfter?.(success);
   }
 }
-function work(type, timer, state, options) {
-  if (state.destroyed && type !== "stop" || (state.active ? beginTypes.has(type) : endTypes.has(type))) {
-    return timer.instance;
-  }
-  const pausable = pauseTypes.has(type);
-  state.elapsed = pausable ? state.elapsed : 0;
-  state.index = pausable ? state.index : 0;
-  state.total = pausable ? state.total : 0;
-  if (endOrRestartTypes.has(type)) {
-    activeTimers.delete(timer.instance);
-    cancelAnimationFrame(state.frame);
-    if (type === "stop") {
-      options.onAfter?.(false);
-    }
-    state.active = false;
-    state.frame = void 0;
-    state.paused = type === "pause";
-    return type === "restart" ? work("start", timer, state, options) : timer.instance;
-  }
-  state.active = true;
-  state.paused = false;
-  let start;
-  function step(now) {
+function ignore(type, state) {
+  return state.destroyed && type !== "stop" || (state.active ? beginTypes.has(type) : endTypes.has(type));
+}
+function run(timer, state, options) {
+  let last;
+  return function step(now) {
     if (!state.active) {
       return;
     }
-    start ??= now;
-    const difference = now - start;
+    last ??= now;
+    const difference = now - last;
     state.elapsed += difference;
     state.total += difference;
+    last = now;
     if (options.timeout > 0 && state.total >= options.timeout) {
       options.onError?.();
       finish(timer, state, options, false);
       return;
     }
-    if (options.interval === milliseconds || state.elapsed >= options.interval - 5) {
+    if (options.interval === milliseconds || state.elapsed >= options.interval - intervalBuffer) {
       if (options.count > -1) {
         state.callback(state.index);
       }
-      start = now;
       state.elapsed = 0;
       state.index += 1;
       if (options.count === -1 || options.count > 0 && state.index >= options.count) {
@@ -59,10 +52,31 @@ function work(type, timer, state, options) {
       }
     }
     state.frame = requestAnimationFrame(step);
+  };
+}
+function setState(type, state) {
+  const pausable = pauseTypes.has(type);
+  state.elapsed = pausable ? state.elapsed : 0;
+  state.index = pausable ? state.index : 0;
+  state.total = pausable ? state.total : 0;
+}
+function stop(timer, state, options) {
+  endOrRestart("stop", timer, state, options);
+}
+function work(type, timer, state, options) {
+  if (ignore(type, state)) {
+    return timer.instance;
   }
+  setState(type, state);
+  if (endOrRestartTypes.has(type)) {
+    return endOrRestart(type, timer, state, options);
+  }
+  state.active = true;
+  state.paused = false;
   activeTimers.add(timer.instance);
-  state.frame = requestAnimationFrame(step);
+  const runner = run(timer, state, options);
+  state.frame = requestAnimationFrame(runner);
   return timer.instance;
 }
 
-export { work };
+export { stop, work };

package/package.json

@@ -11,15 +11,15 @@
 		"@biomejs/biome": "^1.9",
 		"@rollup/plugin-node-resolve": "^16",
 		"@rollup/plugin-typescript": "^12.1",
-		"@types/node": "^22.15",
-		"@vitest/coverage-istanbul": "^3.1",
+		"@types/node": "^24",
+		"@vitest/coverage-istanbul": "^3.2",
 		"dts-bundle-generator": "^9.5",
 		"glob": "^11",
 		"jsdom": "^26.1",
 		"tslib": "^2.8",
 		"typescript": "^5.8",
 		"vite": "^6.3",
-		"vitest": "^3.1"
+		"vitest": "^3.2"
 	},
 	"exports": {
 		".": {
@@ -94,5 +94,5 @@
 	},
 	"type": "module",
 	"types": "types/index.d.cts",
-	"version": "0.31.0"
+	"version": "0.33.0"
 }