fix(ci): skip Coveralls upload when token unavailable

The coverage workflow was failing on Dependabot PRs because COVERALLS_REPO_TOKEN is not available to PRs from forks for security reasons. Add a conditional check to skip the upload step when the token is missing, while still running coverage generation. This allows PRs to pass CI while retaining coverage reports on master.
fix(web): resolve leaderboard production crashes by removing react-animated-numbers
2025-12-05 23:15:40 -06:00 · 2025-11-23 01:38:41 -06:00 · 2025-11-23 01:21:02 -06:00 · 2025-11-23 01:21:02 -06:00
6 changed files with 28 additions and 187 deletions
--- a/.github/dependabot.yml
+++ b/.github/dependabot.yml
@@ -4,135 +4,40 @@
 # Strategy:
 # - Weekly checks for faster vulnerability detection
 # - Separate patch/minor/major updates to prevent blocking
-# - Group by crate (game vs server) for easier review
 # - Auto-merge patches via GitHub branch protection rules
 # - Limit concurrent PRs to avoid spam

 version: 2
 updates:
-  # Game: Patch updates (auto-mergeable)
+  # Cargo workspace (all Rust crates)
  - package-ecosystem: "cargo"
-    directory: "/pacman"
+    directory: "/"
    schedule:
      interval: "weekly"
      day: "monday"
    open-pull-requests-limit: 5
-    groups:
-      game-patches:
-        applies-to: "version-updates"
-        update-types:
-          - "patch"
    ignore:
-      # Bevy ECS 0.17+ requires API migration - ignore until manual update
+      # Bevy ECS 0.17+ requires API migration
      - dependency-name: "bevy_ecs"
        versions: ["0.17.x", "0.18.x", "0.19.x"]
-    labels:
-      - "dependencies"
-      - "dependencies:patch"
-      - "game"
-
-  # Game: Minor updates (grouped, manual review)
-  - package-ecosystem: "cargo"
-    directory: "/pacman"
-    schedule:
-      interval: "weekly"
-      day: "monday"
-    open-pull-requests-limit: 5
-    groups:
-      game-minor:
-        applies-to: "version-updates"
-        update-types:
-          - "minor"
-    ignore:
-      - dependency-name: "bevy_ecs"
-        versions: ["0.17.x", "0.18.x", "0.19.x"]
-    labels:
-      - "dependencies"
-      - "dependencies:minor"
-      - "game"
-
-  # Game: Major updates (separate PRs, manual review)
-  - package-ecosystem: "cargo"
-    directory: "/pacman"
-    schedule:
-      interval: "weekly"
-      day: "monday"
-    open-pull-requests-limit: 5
-    groups:
-      game-major:
-        applies-to: "version-updates"
-        update-types:
-          - "major"
-    ignore:
-      - dependency-name: "bevy_ecs"
-        versions: ["0.17.x", "0.18.x", "0.19.x"]
-    labels:
-      - "dependencies"
-      - "dependencies:major"
-      - "game"
-
-  # Server: Patch updates (auto-mergeable)
-  - package-ecosystem: "cargo"
-    directory: "/pacman-server"
-    schedule:
-      interval: "weekly"
-      day: "monday"
-    open-pull-requests-limit: 5
-    groups:
-      server-patches:
-        applies-to: "version-updates"
-        update-types:
-          - "patch"
-    ignore:
-      # jsonwebtoken 10+ requires crypto backend feature flag - ignore until manual migration
+      # jsonwebtoken 10+ requires crypto backend feature flag
      - dependency-name: "jsonwebtoken"
        versions: ["10.x", "11.x"]
-    labels:
-      - "dependencies"
-      - "dependencies:patch"
-      - "server"
-
-  # Server: Minor updates (grouped, manual review)
-  - package-ecosystem: "cargo"
-    directory: "/pacman-server"
-    schedule:
-      interval: "weekly"
-      day: "monday"
-    open-pull-requests-limit: 5
    groups:
-      server-minor:
+      rust-patches:
        applies-to: "version-updates"
-        update-types:
-          - "minor"
-    ignore:
-      - dependency-name: "jsonwebtoken"
-        versions: ["10.x", "11.x"]
+        update-types: ["patch"]
+      rust-minor:
+        applies-to: "version-updates"
+        update-types: ["minor"]
+      rust-major:
+        applies-to: "version-updates"
+        update-types: ["major"]
    labels:
      - "dependencies"
-      - "dependencies:minor"
-      - "server"
+      - "rust"

-  # Server: Major updates (separate PRs, manual review)
-  - package-ecosystem: "cargo"
-    directory: "/pacman-server"
-    schedule:
-      interval: "weekly"
-      day: "monday"
-    open-pull-requests-limit: 5
-    groups:
-      server-major:
-        applies-to: "version-updates"
-        update-types:
-          - "major"
-    ignore:
-      - dependency-name: "jsonwebtoken"
-        versions: ["10.x", "11.x"]
-    labels:
-      - "dependencies"
-      - "dependencies:major"
-      - "server"
-
-  # Frontend: Patch updates (auto-mergeable)
+  # Frontend (web/)
  - package-ecosystem: "npm"
    directory: "/web"
    schedule:
@@ -142,65 +47,21 @@ updates:
    groups:
      frontend-patches:
        applies-to: "version-updates"
-        update-types:
-          - "patch"
-    labels:
-      - "dependencies"
-      - "dependencies:patch"
-      - "frontend"
-
-  # Frontend: Minor updates (grouped, manual review)
-  - package-ecosystem: "npm"
-    directory: "/web"
-    schedule:
-      interval: "weekly"
-      day: "monday"
-    open-pull-requests-limit: 5
-    groups:
+        update-types: ["patch"]
      frontend-minor:
        applies-to: "version-updates"
-        update-types:
-          - "minor"
-    labels:
-      - "dependencies"
-      - "dependencies:minor"
-      - "frontend"
-
-  # Frontend: Major updates (separate PRs for critical deps)
-  - package-ecosystem: "npm"
-    directory: "/web"
-    schedule:
-      interval: "weekly"
-      day: "monday"
-    open-pull-requests-limit: 5
-    groups:
+        update-types: ["minor"]
      frontend-major-framework:
        applies-to: "version-updates"
-        update-types:
-          - "major"
+        update-types: ["major"]
        patterns:
          - "react"
          - "react-dom"
          - "vike"
          - "vite"
-    labels:
-      - "dependencies"
-      - "dependencies:major"
-      - "frontend"
-      - "framework"
-
-  # Frontend: Other major updates (grouped)
-  - package-ecosystem: "npm"
-    directory: "/web"
-    schedule:
-      interval: "weekly"
-      day: "monday"
-    open-pull-requests-limit: 5
-    groups:
      frontend-major-other:
        applies-to: "version-updates"
-        update-types:
-          - "major"
+        update-types: ["major"]
        exclude-patterns:
          - "react"
          - "react-dom"
@@ -208,10 +69,9 @@ updates:
          - "vite"
    labels:
      - "dependencies"
-      - "dependencies:major"
      - "frontend"

-  # GitHub Actions: All updates grouped (low risk)
+  # GitHub Actions
  - package-ecosystem: "github-actions"
    directory: "/"
    schedule:
@@ -220,8 +80,7 @@ updates:
    open-pull-requests-limit: 5
    groups:
      github-actions:
-        patterns:
-          - "*"
+        patterns: ["*"]
    labels:
      - "dependencies"
      - "github-actions"
--- a/.github/workflows/coverage.yaml
+++ b/.github/workflows/coverage.yaml
@@ -57,6 +57,7 @@ jobs:
        working-directory: pacman

      - name: Coveralls upload
+        if: ${{ secrets.COVERALLS_REPO_TOKEN != '' }}
        uses: coverallsapp/github-action@v2
        with:
          github-token: ${{ secrets.COVERALLS_REPO_TOKEN }}
--- a/web/bun.lock
+++ b/web/bun.lock
@@ -9,7 +9,6 @@
        "@tabler/icons-react": "^3.35.0",
        "@vitejs/plugin-react": "^5.0.2",
        "react": "^19.1.1",
-        "react-animated-numbers": "^1.1.1",
        "react-dom": "^19.1.1",
        "vike": "^0.4.240",
        "vike-react": "^0.6.5",
@@ -465,8 +464,6 @@

    "for-each": ["for-each@0.3.5", "", { "dependencies": { "is-callable": "^1.2.7" } }, "sha512-dKx12eRCVIzqCxFGplyFKJMPvLEWgmNtUrpTiJIR5u97zEhRG8ySrtboPHZXx7daLxQVrl643cTzbab2tkQjxg=="],

-    "framer-motion": ["framer-motion@11.18.2", "", { "dependencies": { "motion-dom": "^11.18.1", "motion-utils": "^11.18.1", "tslib": "^2.4.0" }, "peerDependencies": { "@emotion/is-prop-valid": "*", "react": "^18.0.0 || ^19.0.0", "react-dom": "^18.0.0 || ^19.0.0" }, "optionalPeers": ["@emotion/is-prop-valid", "react", "react-dom"] }, "sha512-5F5Och7wrvtLVElIpclDT0CBzMVg3dL22B64aZwHtsIY8RB4mXICLrkajK4G9R+ieSAGcgrLeae2SeUTg2pr6w=="],
-
    "fsevents": ["fsevents@2.3.3", "", { "os": "darwin" }, "sha512-5xoDfX+fL7faATnagmWPpbFtwh/R77WmMMqqHGS65C3vvB0YHrgF+B1YmZ3441tMj5n63k0212XNoJwzlhffQw=="],

    "function-bind": ["function-bind@1.1.2", "", {}, "sha512-7XHNxH7qX9xG5mIwxkhumTox/MIRNcOgDrxWsMt2pAr23WHp6MrRlN7FBSFpCpr+oVO0F744iUgR82nJMfG2SA=="],
@@ -641,12 +638,6 @@

    "minimatch": ["minimatch@3.1.2", "", { "dependencies": { "brace-expansion": "^1.1.7" } }, "sha512-J7p63hRiAjw1NDEww1W7i37+ByIrOWO5XQQAzZ3VOcL0PNybwpfmV/N05zFAzwQ9USyEcX6t3UO+K5aqBQOIHw=="],

-    "motion": ["motion@11.18.2", "", { "dependencies": { "framer-motion": "^11.18.2", "tslib": "^2.4.0" }, "peerDependencies": { "@emotion/is-prop-valid": "*", "react": "^18.0.0 || ^19.0.0", "react-dom": "^18.0.0 || ^19.0.0" }, "optionalPeers": ["@emotion/is-prop-valid", "react", "react-dom"] }, "sha512-JLjvFDuFr42NFtcVoMAyC2sEjnpA8xpy6qWPyzQvCloznAyQ8FIXioxWfHiLtgYhoVpfUqSWpn1h9++skj9+Wg=="],
-
-    "motion-dom": ["motion-dom@11.18.1", "", { "dependencies": { "motion-utils": "^11.18.1" } }, "sha512-g76KvA001z+atjfxczdRtw/RXOM3OMSdd1f4DL77qCTF/+avrRJiawSG4yDibEQ215sr9kpinSlX2pCTJ9zbhw=="],
-
-    "motion-utils": ["motion-utils@11.18.1", "", {}, "sha512-49Kt+HKjtbJKLtgO/LKj9Ld+6vw9BjH5d9sc40R/kVyH8GLAXgT42M2NnuPcJNuA3s9ZfZBUcwIgpmZWGEE+hA=="],
-
    "mrmime": ["mrmime@2.0.1", "", {}, "sha512-Y3wQdFg2Va6etvQ5I82yUhGdsKrcYox6p7FfL1LbK2J4V01F9TGlepTIhnK24t7koZibmg82KGglhA1XK5IsLQ=="],

    "ms": ["ms@2.1.3", "", {}, "sha512-6FlzubTLZG3J2a/NVCAleEhjzq5oxgHyaCU9yYXvcLsvoVaHJq/s5xXI6/XXP6tz7R9xAOtHnSO/tXtF3WRTlA=="],
@@ -711,8 +702,6 @@

    "react": ["react@19.2.0", "", {}, "sha512-tmbWg6W31tQLeB5cdIBOicJDJRR2KzXsV7uSK9iNfLWQ5bIZfxuPEHp7M8wiHyHnn0DD1i7w3Zmin0FtkrwoCQ=="],

-    "react-animated-numbers": ["react-animated-numbers@1.1.1", "", { "dependencies": { "motion": "^11.16.0" }, "peerDependencies": { "react": ">=16.8.0", "react-dom": ">=16.8.0" } }, "sha512-Jr2vbDWjo5wW+X8wBYRBACpuKdPkLa4A2ZYfxlAR0oc/gXWVeWyAmAWzZj73276IQxycxrTL7aGn4quWV7+l2Q=="],
-
    "react-dom": ["react-dom@19.2.0", "", { "dependencies": { "scheduler": "^0.27.0" }, "peerDependencies": { "react": "^19.2.0" } }, "sha512-UlbRu4cAiGaIewkPyiRGJk0imDN2T3JjieT6spoL2UeSf5od4n5LB/mQ4ejmxhCFT1tYe8IvaFulzynWovsEFQ=="],

    "react-is": ["react-is@16.13.1", "", {}, "sha512-24e6ynE2H+OKt4kqsOvNd8kBpV65zoxbA4BVsEOB3ARVWQki/DHzaUoC5KuON/BiccDaCCTZBuOcfZs70kR8bQ=="],
@@ -803,8 +792,6 @@

    "ts-api-utils": ["ts-api-utils@2.1.0", "", { "peerDependencies": { "typescript": ">=4.8.4" } }, "sha512-CUgTZL1irw8u29bzrOD/nH85jqyc74D6SshFgujOIA7osm2Rz7dYH77agkx7H4FBNxDq7Cjf+IjaX/8zwFW+ZQ=="],

-    "tslib": ["tslib@2.8.1", "", {}, "sha512-oJFu94HQb+KVduSUQL7wnpmqnfmLsOA/nAh6b6EH0wCEoK0/mPeXU6c3wKDV83MkOuHPRHtSXKKU99IBazS/2w=="],
-
    "type-check": ["type-check@0.4.0", "", { "dependencies": { "prelude-ls": "^1.2.1" } }, "sha512-XleUoc9uwGXqjWwXaUTZAmzMcFZ5858QA2vvx1Ur5xIcixXIP+8LnFDgRplU30us6teqdlskFfu+ae4K79Ooew=="],

    "typed-array-buffer": ["typed-array-buffer@1.0.3", "", { "dependencies": { "call-bound": "^1.0.3", "es-errors": "^1.3.0", "is-typed-array": "^1.1.14" } }, "sha512-nAYYwfY3qnzX30IkA6AQZjVbtK6duGontcQm1WSG1MD94YLqK0515GNApXkoxKOWMusVssAHWLh9SeaoefYFGw=="],
--- a/web/package.json
+++ b/web/package.json
@@ -14,7 +14,6 @@
    "@tabler/icons-react": "^3.35.0",
    "@vitejs/plugin-react": "^5.0.2",
    "react": "^19.1.1",
-    "react-animated-numbers": "^1.1.1",
    "react-dom": "^19.1.1",
    "vike": "^0.4.240",
    "vike-react": "^0.6.5"
--- a/web/pages/leaderboard/+Page.tsx
+++ b/web/pages/leaderboard/+Page.tsx
@@ -1,8 +1,5 @@
 import { useState } from "react";
 import { IconTrophy, IconCalendar } from "@tabler/icons-react";
-import { clientOnly } from "vike-react/clientOnly";
-
-const AnimatedNumbers = clientOnly(() => import("react-animated-numbers"));

 interface LeaderboardEntry {
  id: number;
@@ -238,15 +235,7 @@ function LeaderboardTable({ data }: { data: LeaderboardEntry[] }) {
            </td>
            <td className="py-2">
              <span className="text-yellow-300 font-[600] text-lg">
-                <AnimatedNumbers
-                  fallback={<span className="text-transparent">{entry.score.toLocaleString()}</span>}
-                  useThousandsSeparator
-                  transitions={(digitIndex) => ({
-                    type: "easeIn",
-                    duration: 0.75 + digitIndex * 0.25 + entryIndex * 0.2,
-                  })}
-                  animateToNumber={entry.score}
-                />
+                {entry.score.toLocaleString()}
              </span>
            </td>
            <td className="py-2">
--- a/web/pages/leaderboard/+config.ts
+++ b/web/pages/leaderboard/+config.ts
@@ -0,0 +1,6 @@
+import type { Config } from "vike/types";
+
+export default {
+  prerender: true,  // Generate static HTML for deployment
+  ssr: false,       // Force client-side only rendering
+} satisfies Config;