jspsych-tangram 0.0.4 → 0.0.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "jspsych-tangram",
-  "version": "0.0.4",
+  "version": "0.0.5",
   "description": "Tangram tasks for jsPsych: prep and construct.",
   "type": "module",
   "main": "dist/index.cjs",

package/src/core/components/board/GameBoard.tsx

@@ -357,8 +357,11 @@ export default function GameBoard(props: GameBoardProps) {
         setGameCompleted(true);
 
         // Finalize trial data tracking (which calls onTrialEnd)
+        // Defer to avoid unmounting React while still in commit phase
         if (tracker) {
-          tracker.finalizeTrial('auto_complete');
+          setTimeout(() => {
+            tracker.finalizeTrial('auto_complete');
+          }, 0);
         }
       }
     };

package/src/core/engine/state/BaseGameController.ts

@@ -196,7 +196,6 @@ export class BaseGameController {
     const allDone = Object.values(this.state.sectors).every((s: SectorState) => !!s.completedAt);
     if (allDone && !this.state.endedAt) {
       this.state.endedAt = NOW();
-      console.log("[BaseGameController] all sectors complete");
     }
   }
 

package/src/plugins/tangram-construct/ConstructionApp.tsx

@@ -58,42 +58,24 @@ export function startConstructionTrial(
   ]);
 
   // Convert JSON plugin parameters to internal Sector[] format
-  console.log('[ConstructionApp] Starting tangram conversion...');
-  console.log('[ConstructionApp] Received tangrams:', params.tangrams);
-  console.log('[ConstructionApp] Number of tangrams:', params.tangrams.length);
-
-  const sectors: Sector[] = params.tangrams.map((tangramSpec, index) => {
-    console.log(`\n[ConstructionApp] Processing tangram ${index}:`, tangramSpec);
-    console.log(`[ConstructionApp] tangramID: ${tangramSpec.tangramID}`);
-    console.log(`[ConstructionApp] setLabel: ${tangramSpec.setLabel}`);
-    console.log(`[ConstructionApp] solutionTans count: ${tangramSpec.solutionTans?.length}`);
-    console.log(`[ConstructionApp] solutionTans:`, tangramSpec.solutionTans);
-
+    const sectors: Sector[] = params.tangrams.map((tangramSpec, index) => {
+  
     // Filter to canonical pieces only and convert vertices to polygon format
     const filteredTans = tangramSpec.solutionTans.filter((tan: any) => {
       // Support both "name" and "kind" fields (different JSON formats)
       const tanName = tan.name ?? tan.kind;
       const isCanonical = CANON.has(tanName);
-      console.log(`[ConstructionApp]   Tan "${tanName}": canonical=${isCanonical}, vertices count=${tan.vertices?.length}`);
       return isCanonical;
     });
 
-    console.log(`[ConstructionApp] Filtered to ${filteredTans.length} canonical pieces`);
-
     const mask = filteredTans.map((tan: any, tanIndex: number) => {
-      const tanName = tan.name ?? tan.kind;
       const polygon = tan.vertices.map(([x, y]: number[]) => ({ x: x ?? 0, y: -(y ?? 0) }));
-      console.log(`[ConstructionApp]   Polygon ${tanIndex} (${tanName}): ${tan.vertices.length} vertices -> ${polygon.length} points`);
-      console.log(`[ConstructionApp]     First vertex: [${tan.vertices[0]?.[0]}, ${tan.vertices[0]?.[1]}] -> {x: ${polygon[0]?.x}, y: ${polygon[0]?.y}}`);
       return polygon;
     });
 
     // Assign sector ID from alphabetical sequence
     const sectorId = `sector${index}`;
     
-    console.log(`[ConstructionApp] Assigned sector ID: ${sectorId}`);
-    console.log(`[ConstructionApp] Final mask has ${mask.length} polygons`);
-
     const sector = {
       id: sectorId,
       tangramId: tangramSpec.tangramID,
@@ -103,27 +85,17 @@ export function startConstructionTrial(
       },
     };
 
-    console.log(`[ConstructionApp] Created sector:`, sector);
     return sector;
   });
 
-  console.log('\n[ConstructionApp] Final sectors array:', sectors);
-  console.log(`[ConstructionApp] Total sectors created: ${sectors.length}`);
-
   // Convert quickstash_macros to Blueprint[] format
   // Handle both anchor-based composites and pre-converted blueprints
-  console.log('\n[ConstructionApp] Processing quickstash macros...');
-  console.log('[ConstructionApp] quickstash_macros:', params.quickstash_macros);
-  console.log('[ConstructionApp] quickstash_macros count:', params.quickstash_macros?.length ?? 0);
-
   let quickstash: Blueprint[] = [];
 
   if (params.quickstash_macros && params.quickstash_macros.length > 0) {
     // Check if the first item has anchorOffset (anchor-based) or offset (pixel-based)
     const firstMacro = params.quickstash_macros[0];
-    console.log('[ConstructionApp] First macro:', firstMacro);
     if (firstMacro && 'parts' in firstMacro && firstMacro.parts && firstMacro.parts[0] && 'anchorOffset' in firstMacro.parts[0]) {
-      console.log('[ConstructionApp] Detected anchor-based composites, converting to pixels...');
 
       // Create primitive map for conversion
       const primsByKind = new Map<TanKind, PrimitiveBlueprint>();
@@ -133,14 +105,11 @@ export function startConstructionTrial(
       quickstash = (params.quickstash_macros as AnchorComposite[]).map(anchorComposite =>
         convertAnchorCompositeToPixels(anchorComposite, primsByKind, CONFIG.layout.grid.stepPx) // Use current CONFIG grid step
       );
-      console.log('[ConstructionApp] Converted to pixel-based blueprints:', quickstash);
     } else {
-      console.log('[ConstructionApp] Already pixel-based blueprints');
       // Already pixel-based blueprints
       quickstash = params.quickstash_macros as Blueprint[];
     }
   } else {
-    console.log('[ConstructionApp] No quickstash macros provided');
   }
 
   // Create React root and render GameBoard
@@ -148,31 +117,18 @@ export function startConstructionTrial(
     sectors,
     quickstash,
     primitives: PRIMITIVE_BLUEPRINTS,
-    layout: (params.layout || "semicircle") as LayoutMode,
-    target: (params.target || "silhouette") as PlacementTarget,
-    input: (params.input || "drag") as InputMode,
-    timeLimitMs: params.time_limit_ms || 0,
+    layout: params.layout as LayoutMode,
+    target: params.target as PlacementTarget,
+    input: params.input as InputMode,
+    timeLimitMs: params.time_limit_ms,
     maxQuickstashSlots: CONFIG.layout.defaults.maxQuickstashSlots,
-    mode: 'construction' as const, // Explicit construction mode
+    mode: 'construction' as const,
     ...(params.onInteraction && { onInteraction: params.onInteraction }),
     ...(params.onTrialEnd && { onTrialEnd: params.onTrialEnd })
   };
 
-  console.log('\n[ConstructionApp] Final GameBoard props:');
-  console.log('[ConstructionApp]   sectors count:', gameBoardProps.sectors.length);
-  console.log('[ConstructionApp]   quickstash count:', gameBoardProps.quickstash.length);
-  console.log('[ConstructionApp]   primitives count:', gameBoardProps.primitives.length);
-  console.log('[ConstructionApp]   layout:', gameBoardProps.layout);
-  console.log('[ConstructionApp]   target:', gameBoardProps.target);
-  console.log('[ConstructionApp]   input:', gameBoardProps.input);
-  console.log('[ConstructionApp]   timeLimitMs:', gameBoardProps.timeLimitMs);
-  console.log('[ConstructionApp]   mode:', gameBoardProps.mode);
-  console.log('[ConstructionApp] Full props:', gameBoardProps);
-
   const root = createRoot(display_element);
   root.render(React.createElement(GameBoard, gameBoardProps));
 
-  console.log('[ConstructionApp] GameBoard rendered successfully');
-
   return { root, display_element, jsPsych: _jsPsych };
 }

package/src/plugins/tangram-construct/index.ts

@@ -140,7 +140,7 @@ class TangramConstructPlugin implements JsPsychPlugin<Info> {
       target: trial.target,
       input: trial.input,
       layout: trial.layout,
-      time_limit_ms: trial.time_limit_ms || 0,
+      time_limit_ms: trial.time_limit_ms,
       onInteraction: trial.onInteraction,
       onTrialEnd: wrappedOnTrialEnd
     };

package/src/plugins/tangram-prep/index.ts

@@ -101,7 +101,7 @@ class TangramPrepPlugin implements JsPsychPlugin<Info> {
     };
 
     const params: StartPrepTrialParams = {
-      numQuickstashSlots: trial.num_quickstash_slots || 4,
+      numQuickstashSlots: trial.num_quickstash_slots,
       maxPiecesPerMacro: trial.max_pieces_per_macro,
       minPiecesPerMacro: trial.min_pieces_per_macro,
       inputMode: trial.input as "click" | "drag",