Merge branch 'main' into fix/chromatic-timeout-settings

Author: DereC4

.github/workflows/best-practices.yml

@@ -1,5 +1,9 @@
 name: Best Practices
-on: [push, pull_request]
+on:
+  push:
+    branches:
+      - main
+  pull_request:
 jobs:
   check:
     runs-on: ubuntu-latest

.github/workflows/check-types.yml

@@ -1,5 +1,9 @@
 name: Type Check
-on: [push, pull_request]
+on:
+  push:
+    branches:
+      - main
+  pull_request:
 jobs:
   type-check:
     runs-on: ubuntu-latest

.github/workflows/chromatic.yml

@@ -1,5 +1,9 @@
 name: 'Chromatic'
-on: [push, pull_request]
+on:
+  push:
+    branches:
+      - main
+  pull_request:
 jobs:
   chromatic:
     runs-on: ubuntu-latest

.github/workflows/nix-flake-check.yml

@@ -1,5 +1,9 @@
 name: Nix Flake Check
-on: [push, pull_request]
+on:
+  push:
+    branches:
+      - main
+  pull_request:
 jobs:
   nix-flake:
     name: Nix Flake Check

.github/workflows/tests.yml

@@ -1,5 +1,9 @@
 name: Tests
-on: [push, pull_request]
+on:
+  push:
+    branches:
+      - main
+  pull_request:
 jobs:
   test:
     runs-on: ubuntu-latest

.storybook/chrome-mock.ts

@@ -1,5 +1,6 @@
 /// <reference types="chrome" />
 import { version } from '../package.json';
+import deepEqual from 'fast-deep-equal/es6';
 
 type ListenerFunction = (
     changes: { [key: string]: chrome.storage.StorageChange },
@@ -18,13 +19,13 @@ function createMockStorageArea(areaName: chrome.storage.AreaName) {
             if (Array.isArray(keys)) {
                 return keys.reduce(
                     (acc, key) => {
-                        acc[key] = data[key];
+                        acc[key] = structuredClone(data[key]);
                         return acc;
                     },
                     {} as Record<string, any>
                 );
             }
-            if (typeof keys === 'string') return { [keys]: data[keys] };
+            if (typeof keys === 'string') return { [keys]: structuredClone(data[keys]) };
             return keys;
         },
         async getBytesInUse() {
@@ -40,11 +41,20 @@ function createMockStorageArea(areaName: chrome.storage.AreaName) {
             }
         },
         async set(items: { [key: string]: any }) {
-            for (const key in items) {
+            let hasUpdates = false;
+            const updates: Record<string, chrome.storage.StorageChange> = {};
+            for (const [key, newValue] of Object.entries(items)) {
                 const oldValue = data[key];
-                data[key] = JSON.parse(JSON.stringify(items[key]));
+                if (!deepEqual(newValue, oldValue)) {
+                    hasUpdates = true;
+                    data[key] = structuredClone(newValue);
+                    updates[key] = { oldValue, newValue };
+                }
+            }
+
+            if (hasUpdates) {
                 for (const listener of allListeners.values()) {
-                    listener({ [key]: { oldValue, newValue: data[key] } }, areaName);
+                    listener(structuredClone(updates), areaName);
                 }
             }
         },

.storybook/fetch-mock.ts

@@ -0,0 +1,25 @@
+// Must load before any module that calls fetch() (e.g. GitHubStatsService) evaluates.
+// GitHubStatsService hits the real GitHub API on mount; in Chromatic's sandboxed
+// browser that request hangs/fails with no network access, timing out story capture.
+const realFetch = globalThis.fetch;
+
+const MOCKED_HOSTS = ['github.cachedapi.com', 'api.github.com'];
+
+function resolveUrl(input: RequestInfo | URL): string {
+    if (typeof input === 'string') return input;
+    if (input instanceof URL) return input.href;
+    return input.url;
+}
+
+globalThis.fetch = async (input: RequestInfo | URL, init?: RequestInit): Promise<Response> => {
+    const url = resolveUrl(input);
+
+    if (MOCKED_HOSTS.some(host => url.includes(host))) {
+        return new Response(JSON.stringify([]), {
+            status: 200,
+            headers: { 'Content-Type': 'application/json' },
+        });
+    }
+
+    return realFetch(input, init);
+};

.storybook/main.ts

@@ -2,7 +2,7 @@ import type { StorybookConfig } from '@storybook/react-vite';
 
 const config: StorybookConfig = {
     stories: ['../src/**/*.mdx', '../src/**/*.stories.@(js|jsx|mjs|ts|tsx)'],
-    addons: ['@storybook/addon-designs', '@chromatic-com/storybook'],
+    addons: ['@storybook/addon-docs', '@storybook/addon-designs', '@chromatic-com/storybook'],
     framework: {
         name: '@storybook/react-vite',
         options: {

.storybook/preview.tsx

@@ -1,5 +1,6 @@
-// Must be first import — sets up globalThis.chrome before any other module evaluates
+// Must be first imports — set up globalThis.chrome/fetch before any other module evaluates
 import './chrome-mock';
+import './fetch-mock';
 
 import { UserScheduleStore } from '@shared/storage/UserScheduleStore';
 import type { Preview } from '@storybook/react-vite';

README.md

@@ -218,3 +218,9 @@ Special thanks to the developers and contributors behind these amazing tools and
    <img alt="Star History Chart" src="https://api.star-history.com/svg?repos=Longhorn-Developers/UT-Registration-Plus&type=Date" />
  </picture>
 </a>
+
+## Contributors
+
+<a href="https://github.com/Longhorn-Developers/UT-Registration-Plus/graphs/contributors">
+<img src="https://contrib.rocks/image?repo=Longhorn-Developers/UT-Registration-Plus&columns=24&max=480" />
+</a>