review-module-loading

Étoiles10

Forks0

Mis à jour29 juillet 2026 à 13:40

Review changes to module loading, bootstrap, toy manifest, library resolution, or gamepad polling. Use when a PR touches src/js/bootstrap/, src/js/loader.ts, src/js/router.ts, src/js/toy-view.ts, src/data/toys.json, or gamepad polling code.

Installation

Installer avec Codex ou Claude Copiez ce prompt, collez-le dans Codex, Claude ou un autre assistant, puis laissez-le vérifier la page du skill et l'installer pour vous.

Source

zz-plant

zz-plant/stims

Ouvrir le dépôt GitHub Voir les dépôts du créateur

Téléchargement

Métiers associésSOC

Basé sur la classification professionnelle SOC

Analystes en assurance qualité des logiciels et testeursProfessions informatiques et mathématiques·SOC 15-1253

SKILL.md

readonly

name	review-module-loading
description	Review changes to module loading, bootstrap, toy manifest, library resolution, or gamepad polling. Use when a PR touches src/js/bootstrap/, src/js/loader.ts, src/js/router.ts, src/js/toy-view.ts, src/data/toys.json, or gamepad polling code.

Review Module Loading and Bootstrap

Use this skill when reviewing or authoring changes to src/js/bootstrap/*, src/js/loader.ts, src/js/library-view.js, src/js/toy-view.ts, src/js/milkdrop/catalog-store*.ts, src/data/toys.json, index.html, or gamepad polling code.

Why this exists

~11% of fix commits (125 sampled) are module-loading regressions: toy bundle loading failures, manifest resolution drift, library boot order, and gamepad/input polling lifecycle — the #5 category. This skill prevents those at review time.

Pre-merge checklist

1. All toys load

bun run check:toys passes — verifies toy manifest, generated artifacts, and entry points
If adding or removing a toy entry, the manifest is regenerated

2. Bootstrap order is explicit

No new implicit ordering dependency between loader, router, and toy-view
If adding a new bootstrap step, document its position in the sequence

3. Gamepad/input lifecycle

If touching gamepad or input polling, verify:
- Polling starts after user interaction (not on page load)
- Polling stops on unmount/navigation
- No orphaned requestAnimationFrame loops

4. Library/manifest consistency

If changing src/data/toys.json, the in-memory manifest matches the file
Library resolution does not depend on file-system paths that differ between dev and production

5. Index.html entry points

index.html changes must be verified across all route entry points (/, /?tool=, /?preset=)
No untested module preload or script-order changes in the HTML shell

What to reject in review

New import() calls in bootstrap that don't handle load failure
Toy manifest changes without regenerating toys.json
Gamepad polling that starts before user gesture
Implicit ordering assumptions between bootstrap phases

Related skills

review-workspace-ui-state — when the change involves the UI shell that wraps loaded toys
review-deploy-tooling — when the change involves build config that affects module bundling

Plus depuis ce dépôt

même dépôt

guard-agent-work

zz-plant/stims

Auto-guardrail for agent-authored changes. Use at the start of any coding session to prevent the most common agent mistakes in this repo.

2026-07-2910

iterate-visualizer-ui

zz-plant/stims

Iterate on visualizer UI, workspace chrome, and CSS with fast feedback loops. Use when changing frontend components, styling, layout, or animation without needing the full MilkDrop runtime.

2026-07-2910

modify-visualizer-runtime

zz-plant/stims

Modify the flagship visualizer runtime safely. Use when changing the shared runtime, loader, renderer, shell, controls, audio flows, capability checks, or canonical-route behavior.

2026-07-2910

review-renderer-fallback

zz-plant/stims

Review changes to renderer capability probing, fallback chains, timeout logic, or audio worklet initialization. Use when a PR touches src/js/core/renderer-*, src/js/core/audio-handler.ts, or src/js/milkdrop/runtime/backend-fallback.ts.

2026-07-2910

review-webgpu-parity

zz-plant/stims

Review changes to WebGPU/WebGL dual-backend parity. Use when a PR touches feedback managers, renderer adapters, shader lowering, or any code that affects both WebGPU and WebGL rendering paths.

2026-07-2910

review-workspace-ui-state

zz-plant/stims

Review changes to React workspace UI state, URL routing, toast/panel behavior, or engine adapter interactions. Use when a PR touches src/js/frontend/*, especially App.tsx, workspace hooks, or url-state.ts.

2026-07-2910

Review Module Loading and Bootstrap

Why this exists

Pre-merge checklist

1. All toys load

bun run check:toys passes — verifies toy manifest, generated artifacts, and entry points

If adding or removing a toy entry, the manifest is regenerated

2. Bootstrap order is explicit

No new implicit ordering dependency between loader, router, and toy-view

If adding a new bootstrap step, document its position in the sequence

3. Gamepad/input lifecycle

If touching gamepad or input polling, verify:

Polling starts after user interaction (not on page load)
Polling stops on unmount/navigation
No orphaned requestAnimationFrame loops

4. Library/manifest consistency

If changing src/data/toys.json, the in-memory manifest matches the file

Library resolution does not depend on file-system paths that differ between dev and production

5. Index.html entry points

index.html changes must be verified across all route entry points (/, /?tool=, /?preset=)

No untested module preload or script-order changes in the HTML shell

What to reject in review

New import() calls in bootstrap that don't handle load failure

Toy manifest changes without regenerating toys.json

Gamepad polling that starts before user gesture

Implicit ordering assumptions between bootstrap phases

Related skills

review-workspace-ui-state — when the change involves the UI shell that wraps loaded toys

review-deploy-tooling — when the change involves build config that affects module bundling