Fix template review findings: cruft, locale guard, docs, lint, config type-checking

biome.json

@@ -183,6 +183,16 @@
           }
         }
       }
+    },
+    {
+      "includes": ["tests/**"],
+      "linter": {
+        "rules": {
+          "nursery": {
+            "useExplicitType": "off"
+          }
+        }
+      }
     }
   ],
   "javascript": {

playwright.config.ts

@@ -11,7 +11,7 @@ export default defineConfig({
     timeout: 10_000,
   },
   fullyParallel: false,
-  forbidOnly: !!process.env.CI,
+  forbidOnly: !!process.env["CI"],
   retries: 0,
   workers: 1,
   reporter: [["list"], ["html", { open: "never" }]],
@@ -24,7 +24,7 @@ export default defineConfig({
   webServer: {
     command: "npm run start",
     url: "http://localhost:5173",
-    reuseExistingServer: !process.env.CI,
+    reuseExistingServer: !process.env["CI"],
     timeout: 120_000,
   },
   projects: [

src/common/TimeModel.ts

@@ -31,7 +31,6 @@
  *   import { TimeControlNode } from "scenerystack/scenery-phet";
  *
  *   const timeControl = new TimeControlNode( model.timer.isPlayingProperty, {
- *     timeSpeedProperty: model.timer.timeSpeedProperty, // optional
  *     playPauseStepButtonOptions: {
  *       stepForwardButtonOptions: {
  *         listener: () => model.step( 1 / 60 ),

src/i18n/StringManager.ts

@@ -26,11 +26,17 @@ import stringsEs from "./strings_es.json";
 import stringsFr from "./strings_fr.json";
 
 // ── Compile-time key-parity check ─────────────────────────────────────────────
-// TypeScript errors here if any locale file is missing a key from English.
+// English is the canonical shape; every other locale must match it exactly.
+// TypeScript errors here if any locale file is missing (or adds) a key relative to
+// English. Add one `satisfies` line per new locale so the check stays exhaustive.
+// biome-ignore lint/complexity/noVoid: intentional compile-time type assertion
+void (stringsFr satisfies typeof stringsEn);
 // biome-ignore lint/complexity/noVoid: intentional compile-time type assertion
 void (stringsEn satisfies typeof stringsFr);
 // biome-ignore lint/complexity/noVoid: intentional compile-time type assertion
-void (stringsFr satisfies typeof stringsEn);
+void (stringsEs satisfies typeof stringsEn);
+// biome-ignore lint/complexity/noVoid: intentional compile-time type assertion
+void (stringsEn satisfies typeof stringsEs);
 
 // ── Build the reactive string property tree ───────────────────────────────────
 const stringProperties = LocalizedString.getNestedStringProperties({

src/main.ts

@@ -74,6 +74,3 @@ onReadyToLaunch(() => {
 
   sim.start();
 });
-// hook test
-// test
-// test

tsconfig.scripts.json

@@ -4,5 +4,5 @@
     "noEmit": true,
     "types": ["node"]
   },
-  "include": ["scripts"]
+  "include": ["scripts", "*.config.ts"]
 }