@energy8platform/shell 0.7.0 → 0.7.2

package/dist/pixi.d.ts

@@ -140,6 +140,9 @@ interface CurrencyConfig {
      *  trimmed down to this many places so small wins keep their significant digits (e.g. 0.0673)
      *  while round amounts stay compact (e.g. 0.30). Everything else is shown at exactly this many. */
     minDecimals?: number;
+    /** Defaults to the platform convention: `{ thousands: ',', decimal: '.' }` → `€1,234.50`.
+     *  Override only for a jurisdiction that requires another convention — a comma decimal reads
+     *  as a thousands group to most players and inflates the perceived payout. */
     separator?: {
         thousands?: string;
         decimal?: string;
@@ -630,7 +633,7 @@ declare class ShellController extends EventEmitter<ShellEvents> implements Shell
     tokens: ShellTokens;
     layout: ShellLayoutMode;
     soundOn: boolean;
-    readonly engineVersion = "0.7.0";
+    readonly engineVersion = "0.7.2";
     readonly actions: ShellActions;
     private renderer;
     private i18n;
@@ -775,7 +778,7 @@ interface I18n {
 declare function createI18n(opts: I18nOptions): I18n;
 
 /** The @energy8platform/shell package version, stamped into the game-info footer. */
-declare const PACKAGE_VERSION = "0.7.0";
+declare const PACKAGE_VERSION = "0.7.2";
 
 /** A shell: the renderer-agnostic controller plus the surface facade (safeArea/barHeight/setVisible)
  *  an embedding host reads. `createShell`, `createGameShell` and `createPixiShell` all return this. */

package/dist/pixi.esm.js

@@ -402,15 +402,19 @@ function resolveTheme(theme = {}) {
  *  trailing zeros are trimmed down to (but never past) `minDecimals`, so small wins keep their
  *  significant digits. Balance / bet / prices stay fixed at `minDecimals`.
  *
+ *  Separators default to the platform convention — `.` decimal, `,` thousands (`€1,234.50`).
+ *  A comma decimal is NOT safe as a default: players read "2,50" as 250 and believe they were
+ *  paid far more than they were. Games that need another convention pass `currency.separator`.
+ *
  *  Example with `maxDecimals: 4, minDecimals: 2`:
- *    fixed    → 0.0673 → 0,07     0.3 → 0,30
- *    variable → 0.0673 → 0,0673   0.067 → 0,067   0.3 → 0,30   0 → 0,00
+ *    fixed    → 0.0673 → 0.07     0.3 → 0.30
+ *    variable → 0.0673 → 0.0673   0.067 → 0.067   0.3 → 0.30   0 → 0.00
  */
 function formatCurrency(value, currency, variableDecimals = false) {
     const maxDecimals = currency.maxDecimals ?? 2;
     const minDecimals = Math.max(0, Math.min(maxDecimals, currency.minDecimals ?? maxDecimals));
-    const thousands = currency.separator?.thousands ?? '.';
-    const decimal = currency.separator?.decimal ?? ',';
+    const thousands = currency.separator?.thousands ?? ',';
+    const decimal = currency.separator?.decimal ?? '.';
     const safe = Number.isFinite(value) ? value : 0;
     // fixed callers round at minDecimals; variable callers round at maxDecimals then trim back down.
     const places = variableDecimals ? maxDecimals : minDecimals;
@@ -1688,7 +1692,7 @@ class KeyboardController {
 
 // AUTO-GENERATED by scripts/gen-version.mjs — do not edit. Mirrors package.json "version".
 /** The @energy8platform/shell package version, stamped into the game-info footer. */
-const PACKAGE_VERSION = '0.7.0';
+const PACKAGE_VERSION = '0.7.2';
 
 /** Apply defaults to the raw config (the mount target lives on the renderer, not here). */
 function resolveConfig(config) {
@@ -2320,7 +2324,22 @@ function prefersReducedMotion() {
 }
 const easeOutCubic = (p) => 1 - Math.pow(1 - p, 3);
 
-/** Tween 0→1 on the Pixi ticker. Returns a canceler. Skips to the end when motion is reduced. */
+/**
+ * Tween 0→1 on the Pixi ticker. Returns a canceler. Skips to the end when motion is reduced.
+ *
+ * CONTRACT — when motion is reduced (or `duration <= 0`) this tween does NOT animate: it applies
+ * the final value and calls `onComplete` SYNCHRONOUSLY, before returning. Callers depend on that
+ * (`PixiRenderer.destroy` resolves its teardown promise from `onComplete`, and must not wait on a
+ * ticker that may already be stopped), so it must stay synchronous.
+ *
+ * The consequence for callers: `onComplete` is NOT an async boundary. An `onComplete` that starts
+ * another tween is then direct recursion with no unwind — tween → onComplete → tween → … until
+ * `RangeError: Maximum call stack size exceeded`. That shipped once: the CTA pulse loop
+ * (`widgets.ts` startPulse) restarted itself from `onComplete`, so every player running the OS
+ * "reduce motion" setting crashed the instant the buy-bonus panel painted its hovered CTA. Any
+ * looping animation must therefore check `prefersReducedMotion()` and not loop — which is also
+ * what reduced motion is asking for.
+ */
 function tween(ticker, opts) {
     const ease = opts.ease ?? easeOutCubic;
     if (prefersReducedMotion() || opts.duration <= 0) {
@@ -3292,6 +3311,15 @@ class BuyBonusBadge extends Container {
     startPulse() {
         if (this.pulseCancel)
             return;
+        // A pulse is exactly the kind of looping decoration "reduce motion" asks us to drop — and
+        // under it `tween` has nothing to animate anyway: it snaps to the end and calls `onComplete`
+        // SYNCHRONOUSLY (see its contract), which this loop would answer by starting another one, and
+        // another, with no unwind — `RangeError: Maximum call stack size exceeded`. That was a live
+        // Stake crash: with the OS "reduce motion" setting on, opening the buy-bonus panel painted its
+        // hovered CTA and killed the game, so the player could never buy. Now the button simply sits
+        // still at scale 1.
+        if (prefersReducedMotion())
+            return;
         // both the label (anchor 0.5) and the icon (pivot centred) scale around their centre
         const loop = () => {
             this.pulseCancel = tween(this.ticker, {