fix: Apply 5-point punch list for electric field physics, classroom WS, panel init, eslint ignores, and unit tests

Author: ShyamAlancode

eslint.config.js

@@ -22,6 +22,7 @@ export default [
       "models/**",
       "scratch/**",
       "test_api.mjs",
+      "dist/**",
     ],
   },
   js.configs.recommended,

package-lock.json

@@ -29,6 +29,7 @@
     "@eslint/js": "^9.22.0",
     "eslint": "^9.22.0",
     "globals": "^16.0.0",
+    "three": "^0.164.1",
     "vite": "^8.2.1"
   }
 }

package.json

@@ -25,7 +25,7 @@ export function initClassroomWebSocketServer(server) {
 
   wss.on("connection", (ws) => {
     let currentRoomId = null;
-    let userId = `user_${Math.random().toString(36).substring(2, 9)}`;
+    const userId = `user_${Math.random().toString(36).substring(2, 9)}`;
     let userName = "Learner";
 
     ws.on("message", (rawMessage) => {

server/services/classroomServer.js

@@ -39,6 +39,9 @@ export class ElectricFieldManager {
     this.fluxLines = null;
     this.particles = null;
     this.particleStates = [];
+    this.coulombLines = null;
+    this.velocities = new Map();
+    this.simulationRunning = false;
   }
 
   collectPhysicsObjects() {

src/render/electricFieldManager.js

@@ -6,7 +6,7 @@ export function initClassroomUI(appContext) {
   classroomClient = new ClassroomClient(appContext);
 
   const topbarTools = document.querySelector(".topbar-tools");
-  if (!topbarTools) return;
+  if (!topbarTools) return classroomClient;
 
   const classroomCluster = document.createElement("div");
   classroomCluster.className = "tool-cluster classroom-cluster";

src/ui/classroomPanel.js

@@ -0,0 +1,34 @@
+import { test } from "node:test";
+import assert from "node:assert/strict";
+import * as THREE from "three";
+import { ElectricFieldManager } from "../src/render/electricFieldManager.js";
+
+test("simulateCoulombMotion does not throw once simulation is running", () => {
+  const fakeSceneApi = {
+    snapshot: () => ({ objects: [] }),
+    updateObjectPosition: () => {},
+  };
+  const fakeWorld = { scene: { add: () => {} } };
+  const manager = new ElectricFieldManager(fakeWorld, fakeSceneApi);
+
+  manager.setSimulationPlay(true);
+
+  const charges = [
+    {
+      id: "charge_a",
+      position: new THREE.Vector3(-1, 1, 0),
+      charge: 1,
+      strength: 1,
+      radius: 0.35,
+    },
+    {
+      id: "charge_b",
+      position: new THREE.Vector3(1, 1, 0),
+      charge: -1,
+      strength: 1,
+      radius: 0.35,
+    },
+  ];
+
+  assert.doesNotThrow(() => manager.simulateCoulombMotion(0.016, charges));
+});