@teachinglab/omd 0.1.0 → 0.1.2

package/omd/core/omdEquationStack.js

@@ -75,11 +75,9 @@ export class omdEquationStack extends jsvgGroup {
         if (this.toolbar) {
             if (overlayBottom) {
                 // For overlay positioning, add toolbar directly to this (not layoutGroup)
-                console.log('[constructor] Adding toolbar as overlay to stack');
                 this.addChild(this.toolbar.elements.toolbarGroup);
             } else {
                 // For in-flow positioning, add to layout group
-                console.log('[constructor] Adding toolbar to layout group');
                 this.layoutGroup.addChild(this.toolbar.elements.toolbarGroup);
             }
         }
@@ -238,16 +236,6 @@ export class omdEquationStack extends jsvgGroup {
             const name = st?.constructor?.name;
             if (st instanceof omdOperationDisplayNode || name === 'omdOperationDisplayNode') { startIndex = i; break; }
         }
-
-        try {
-            console.log('[undoLastOperation] before', {
-                totalSteps: seq.steps.length,
-                eqIndex,
-                startIndex,
-                stepTypes: seq.steps.map(s => s?.constructor?.name)
-            });
-        } catch (_) {}
-
         // Remove DOM children and steps from startIndex to end
         for (let i = seq.steps.length - 1; i >= startIndex; i--) {
             const step = seq.steps[i];
@@ -271,7 +259,6 @@ export class omdEquationStack extends jsvgGroup {
         // If this is a step visualizer, rebuild its dots/lines
         if (typeof seq.rebuildVisualizer === 'function') {
             try {
-                console.log('[undoLastOperation] rebuilding visualizer');
                 seq.rebuildVisualizer();
             } catch (_) {}
         } else if (typeof seq._initializeVisualElements === 'function') {
@@ -286,12 +273,7 @@ export class omdEquationStack extends jsvgGroup {
         try {
             const isEquation = (s) => (s instanceof omdEquationNode) || (s?.constructor?.name === 'omdEquationNode');
             const equationsCount = Array.isArray(seq.steps) ? seq.steps.filter(isEquation).length : 0;
-            console.log('[undoLastOperation] after removal', {
-                remainingSteps: seq.steps.length,
-                equationsCount,
-                dotCount: Array.isArray(seq.stepDots) ? seq.stepDots.length : 'n/a',
-                lineCount: Array.isArray(seq.stepLines) ? seq.stepLines.length : 'n/a'
-            });
+
             // Remove dots whose equationRef is no longer present in steps
             if (Array.isArray(seq.stepDots) && seq.visualContainer) {
                 const eqSet = new Set(seq.steps.filter(isEquation));
@@ -336,12 +318,6 @@ export class omdEquationStack extends jsvgGroup {
 
         // Refresh stack layout
         this.updateLayout();
-        try {
-            console.log('[undoLastOperation] done', {
-                finalDotCount: Array.isArray(seq.stepDots) ? seq.stepDots.length : 'n/a',
-                finalLineCount: Array.isArray(seq.stepLines) ? seq.stepLines.length : 'n/a'
-            });
-        } catch (_) {}
         return true;
     }
 } 

package/omd/core/omdUtilities.js

@@ -49,7 +49,6 @@ export function astToOmdType(type, ast) {
             }
             return omdFunctionNode;
         default:
-            console.log(`Unknown AST node type: ${type}, using default omdNode`);
             return omdNode;
     }
 }

package/omd/display/omdDisplay.js

@@ -143,7 +143,6 @@ export class omdDisplay {
     
     fitToContent() {
         if (!this.node) {
-            console.log('No node to fit');
             return;
         }
         
@@ -160,7 +159,6 @@ export class omdDisplay {
             if (sequence && sequence.width && sequence.height) {
                 sequenceWidth = sequence.width;
                 sequenceHeight = sequence.height;
-                console.log('Sequence dimensions:', sequenceWidth, 'x', sequenceHeight);
                 
                 // Check current step dimensions too
                 if (sequence.getCurrentStep) {
@@ -168,14 +166,12 @@ export class omdDisplay {
                     if (currentStep && currentStep.width && currentStep.height) {
                         stepWidth = currentStep.width;
                         stepHeight = currentStep.height;
-                        console.log('Current step dimensions:', stepWidth, 'x', stepHeight);
                     }
                 }
                 
                 // Use the larger of sequence or step dimensions
                 actualWidth = Math.max(sequenceWidth, stepWidth);
                 actualHeight = Math.max(sequenceHeight, stepHeight);
-                console.log('Using maximum dimensions:', actualWidth, 'x', actualHeight);
             }
         }
         
@@ -183,26 +179,22 @@ export class omdDisplay {
         if ((actualWidth === 0 || actualHeight === 0) && this.node.width && this.node.height) {
             actualWidth = this.node.width;
             actualHeight = this.node.height;
-            console.log('Using node dimensions:', actualWidth, 'x', actualHeight);
         }
         
         // Fallback dimensions
         if (actualWidth === 0 || actualHeight === 0) {
             actualWidth = 200;
             actualHeight = 60;
-            console.log('Using fallback dimensions:', actualWidth, 'x', actualHeight);
         }
         
         const padding = 10; // More comfortable padding to match user expectation
         const newWidth = actualWidth + (padding * 2);
         const newHeight = actualHeight + (padding * 2);
         
-        console.log('Setting container to:', newWidth, 'x', newHeight);
         
         // Position the content at the minimal padding offset FIRST
         if (this.node && this.node.setPosition) {
             this.node.setPosition(padding, padding);
-            console.log('Positioning content at:', padding, padding);
         }
         
         // Update SVG dimensions with viewBox starting from 0,0 since we repositioned content

package/omd/display/omdToolbar.js

@@ -25,7 +25,7 @@ export class omdToolbar {
             inputFontSize: 28, menuFontSize: 24, inputWidth: 120,
             popupDirection: 'below',
             showUndoButton: false,
-            undoIconUrl: './assets/undo.svg',
+            undoIconUrl: 'data:image/svg+xml;base64,PHN2ZyB3aWR0aD0iMzciIGhlaWdodD0iNDQiIHZpZXdCb3g9IjAgMCAzNyA0NCIgZmlsbD0ibm9uZSIgeG1sbnM9Imh0dHA6Ly93d3cudzMub3JnLzIwMDAvc3ZnIj4NCjxwYXRoIGQ9Ik0xOC4zNjc2IDQzLjgzMDFDMTMuNTkxMyA0My44MjM3IDkuMDEyNTYgNDEuOTIzNCA1LjYzNTI1IDM4LjU0NTlDMi4yNTc5MiAzNS4xNjg1IDAuMzU3NjYgMzAuNTg5OSAwLjM1MTA3NCAyNS44MTM2QzAuMzUxMDc0IDI1LjMxOTQgMC41NDc0NDEgMjQuODQ1MiAwLjg5Njk2MSAyNC40OTU4QzEuMjQ2NDggMjQuMTQ2MiAxLjcyMDU1IDIzLjk0OTggMi4yMTQ4NSAyMy45NDk4QzIuNzA5MTUgMjMuOTQ5OCAzLjE4MzIyIDI0LjE0NjIgMy41MzI3NCAyNC40OTU4QzMuODgyMjYgMjQuODQ1MiA0LjA3ODYzIDI1LjMxOTQgNC4wNzg2MyAyNS44MTM2QzQuMDc4NjMgMjguNjM5NiA0LjkxNjY2IDMxLjQwMjIgNi40ODY3NSAzMy43NTIxQzguMDU2ODUgMzYuMTAxOSAxMC4yODg1IDM3LjkzMzQgMTIuODk5NCAzOS4wMTQ5QzE1LjUxMDMgNDAuMDk2NCAxOC4zODM1IDQwLjM3OTQgMjEuMTU1MyAzOS44MjhDMjMuOTI3MSAzOS4yNzY4IDI2LjQ3MyAzNy45MTU3IDI4LjQ3MTUgMzUuOTE3NUMzMC40Njk3IDMzLjkxOTEgMzEuODMwOCAzMS4zNzMxIDMyLjM4MTkgMjguNjAxM0MzMi45MzM0IDI1LjgyOTUgMzIuNjUwMyAyMi45NTYzIDMxLjU2ODggMjAuMzQ1NkMzMC40ODczIDE3LjczNDUgMjguNjU1OSAxNS41MDI5IDI2LjMwNiAxMy45MzI4QzIzLjk1NjIgMTIuMzYyNyAyMS4xOTM2IDExLjUyNDcgMTguMzY3NiAxMS41MjQ3SDEyLjE1NUMxMS42NjA3IDExLjUyNDcgMTEuMTg2NiAxMS4zMjgzIDEwLjgzNzEgMTAuOTc4OEMxMC40ODc2IDEwLjYyOTMgMTAuMjkxMiAxMC4xNTUyIDEwLjI5MTIgOS42NjA5QzEwLjI5MTIgOS4xNjY2IDEwLjQ4NzYgOC42OTI1MyAxMC44MzcxIDguMzQzMDFDMTEuMTg2NiA3Ljk5MzQ5IDExLjY2MDcgNy43OTcxMiAxMi4xNTUgNy43OTcxMkgxOC4zNjc2QzIzLjE0NTggNy43OTcxMiAyNy43Mjg1IDkuNjk1MjkgMzEuMTA3MSAxMy4wNzRDMzQuNDg2IDE2LjQ1MjggMzYuMzg0MSAyMS4wMzU0IDM2LjM4NDEgMjUuODEzNkMzNi4zODQxIDMwLjU5MTkgMzQuNDg2IDM1LjE3NDUgMzEuMTA3MSAzOC41NTMyQzI3LjcyODUgNDEuOTMyMSAyMy4xNDU4IDQzLjgzMDEgMTguMzY3NiA0My44MzAxWiIgZmlsbD0id2hpdGUiLz4NCjxwYXRoIGQ9Ik0xOC4zNjc1IDE4Ljk3OThDMTguMTIyNyAxOC45ODEgMTcuODc5OSAxOC45MzMzIDE3LjY1MzggMTguODM5NEMxNy40Mjc3IDE4Ljc0NTQgMTcuMjIyNyAxOC42MDczIDE3LjA1MDQgMTguNDMzMUw5LjU5NTM2IDEwLjk3OEM5LjI0NjM0IDEwLjYyODYgOS4wNTAyOSAxMC4xNTQ5IDkuMDUwMjkgOS42NjA5NkM5LjA1MDI5IDkuMTY3MDYgOS4yNDYzNCA4LjY5MzM2IDkuNTk1MzYgOC4zNDM4OUwxNy4wNTA0IDAuODg4Nzg5QzE3LjIyMTIgMC43MDU2NjcgMTcuNDI2OSAwLjU1ODgwMSAxNy42NTU1IDAuNDU2OTM5QzE3Ljg4NDIgMC4zNTUwNzggMTguMTMwOSAwLjMwMDMwOCAxOC4zODEyIDAuMjk1ODg0QzE4LjYzMTQgMC4yOTE0NjEgMTguODc5OSAwLjMzNzUwOCAxOS4xMTIgMC40MzEyNDRDMTkuMzQ0MSAwLjUyNDk3OSAxOS41NTQ5IDAuNjY0NDg5IDE5LjczMTggMC44NDE0NzNDMTkuOTA5IDEuMDE4NDYgMjAuMDQ4NCAxLjIyOTI5IDIwLjE0MjEgMS40NjEzNEMyMC4yMzYgMS42OTM0MiAyMC4yODIgMS45NDIgMjAuMjc3NSAyLjE5MjI0QzIwLjI3MyAyLjQ0MjUxIDIwLjIxODQgMi42ODkzIDIwLjExNjUgMi45MTc5MkMyMC4wMTQ2IDMuMTQ2NTQgMTkuODY3NyAzLjM1MjMgMTkuNjg0NiAzLjUyMjkzTDEzLjU0NjUgOS42NjA5NkwxOS42ODQ2IDE1Ljc5OUMyMC4wMzM3IDE2LjE0ODUgMjAuMjI5OCAxNi42MjIyIDIwLjIyOTggMTcuMTE2QzIwLjIyOTggMTcuNjEgMjAuMDMzNyAxOC4wODM3IDE5LjY4NDYgMTguNDMzMUMxOS41MTI2IDE4LjYwNzMgMTkuMzA3MyAxOC43NDU0IDE5LjA4MTIgMTguODM5NEMxOC44NTUxIDE4LjkzMzMgMTguNjEyNSAxOC45ODEgMTguMzY3NSAxOC45Nzk4WiIgZmlsbD0id2hpdGUiLz4NCjwvc3ZnPg0K',
             onUndo: null,
             ...options
         };

package/omd/nodes/omdEquationSequenceNode.js

@@ -587,40 +587,22 @@ export class omdEquationSequenceNode extends omdNode {
      * @returns {string} result.message - Human-readable description of the result
      */
     simplify() {
-        console.log(`🔍 simplify called`);
-        
         const currentStep = this.steps[this.steps.length - 1];
         if (!currentStep) {
-            console.log(`❌ No current step found`);
             return { success: false, message: 'No expression found to simplify' };
         }
 
-        console.log(`📋 Current step:`, {
-            type: currentStep.type,
-            constructorName: currentStep.type,
-            toString: currentStep.toString()
-        });
-
         try {
             const stepToSimplify = currentStep.clone();
-            console.log(`🔄 Cloned step for simplification:`, {
-                type: stepToSimplify.type,
-                constructorName: stepToSimplify.type,
-                toString: stepToSimplify.toString()
-            });
-            
             const simplificationResult = simplifyStep(stepToSimplify);
-            console.log(`📊 simplifyStep result:`, simplificationResult);
 
             if (simplificationResult.foldedCount > 0) {
-                console.log(`✅ Simplification successful, handling result...`);
                 return this._handleSuccessfulSimplification(currentStep, simplificationResult);
             } else {
-                console.log(`❌ No simplifications applied`);
                 return { success: false, foldedCount: 0, message: 'No simplifications available' };
             }
         } catch (error) {
-            console.error(`❌ Error during simplification:`, error);
+            console.error(`Error during simplification:`, error);
             return { success: false, message: `Simplification error: ${error.message}` };
         }
     }
@@ -665,40 +647,22 @@ export class omdEquationSequenceNode extends omdNode {
      * @returns {string} result.message - Human-readable description of the final result
      */
     simplifyAll(maxIterations = 50) {
-        console.log(`🚀 simplifyAll called with maxIterations: ${maxIterations}`);
-        console.log(`📋 Current steps:`, {
-            stepsLength: this.steps.length,
-            steps: this.steps.map((step, i) => ({
-                index: i,
-                type: step.type,
-                constructorName: step.type,
-                toString: step.toString()
-            }))
-        });
-
         let iteration = 0;
         let stepsBefore;
         let totalSteps = 0;
 
         do {
             stepsBefore = this.steps.length;
-            console.log(`🔄 Iteration ${iteration + 1}: starting with ${stepsBefore} steps`);
-            
             const result = this.simplify();
-            console.log(`📊 Iteration ${iteration + 1} result:`, result);
             
             if (result.success) {
                 totalSteps++;
-                console.log(`✅ Iteration ${iteration + 1} successful, totalSteps: ${totalSteps}`);
-            } else {
-                console.log(`❌ Iteration ${iteration + 1} failed: ${result.message}`);
             }
             
             iteration++;
         } while (this.steps.length > stepsBefore && iteration < maxIterations);
 
         if (iteration >= maxIterations) {
-            console.log(`⚠️ Stopped after ${maxIterations} iterations to avoid infinite loop`);
             return {
                 success: false,
                 totalSteps,
@@ -706,11 +670,6 @@ export class omdEquationSequenceNode extends omdNode {
                 message: `Stopped after ${maxIterations} iterations to avoid an infinite loop.`
             };
         } else {
-            console.log(`✅ simplifyAll completed successfully:`, {
-                totalSteps,
-                iterations: iteration,
-                finalStepsLength: this.steps.length
-            });
             return {
                 success: true,
                 totalSteps,
@@ -738,9 +697,9 @@ export class omdEquationSequenceNode extends omdNode {
             if (typeof result === 'object' && result.left !== undefined && result.right !== undefined) {
                 const { left, right } = result;
                 const isEqual = Math.abs(left - right) < 1e-9;
-                console.log(`Evaluation: ${Number(left.toFixed(4))} = ${Number(right.toFixed(4))}. Result: ${isEqual}`);
+
             } else {
-                console.log(`Evaluation result: ${result}`);
+
             }
         } catch (error) {
             console.error("Evaluation failed:", error.message);

package/omd/simplification/omdSimplification.js

@@ -15,11 +15,6 @@ export function findSimplificationOpportunities(rootNode) {
         if (!node || visitedNodes.has(node)) return;
         visitedNodes.add(node);
 
-
-        if (node.type === 'omdEquationNode') {
-
-        }
-
         // Traverse children first (depth-first)
         if (node.childList && node.childList.length > 0) {
 
@@ -43,33 +38,23 @@ export function findSimplificationOpportunities(rootNode) {
     }
 
     traverse(rootNode);
-    console.log(`📊 Total opportunities found: ${opportunities.length}`);
     return opportunities;
 }
 
 export function simplifyStep(rootNode) {
-    console.log(`🚀 simplifyStep called for root node:`, {
-        nodeType: rootNode.type,
-        constructorName: rootNode.type,
-        nodeId: rootNode.id
-    });
-
     let foldedCount = 0;
     let newRoot = rootNode;
     let historyEntry = null;
 
     // Find all simplification opportunities
     const opportunities = findSimplificationOpportunities(rootNode);
-    console.log(`📊 Found ${opportunities.length} simplification opportunities`);
 
     if (opportunities.length > 0) {
         // Apply the first opportunity
         const opportunity = opportunities[0];
-        console.log(`🔧 Applying simplification rule:`, opportunity.rule.name);
         
         try {
             const result = opportunity.rule.apply(opportunity.node, opportunity.ruleData, rootNode);
-            console.log(`✅ Rule "${opportunity.rule.name}" applied successfully:`, result);
             
             if (result && result.success && result.newRoot) {
                 newRoot = result.newRoot;
@@ -79,27 +64,12 @@ export function simplifyStep(rootNode) {
                     affectedNodes: [rootNode.id],
                     message: `Applied ${opportunity.rule.name}`
                 };
-                console.log(`✅ Simplification successful! New root:`, {
-                    nodeType: newRoot.type,
-                    nodeId: newRoot.id
-                });
-            } else {
-                console.log(`❌ Rule "${opportunity.rule.name}" returned no result`);
             }
         } catch (error) {
-            console.error(`❌ Error applying rule "${opportunity.rule.name}":`, error);
+            console.error(`Error applying rule '${opportunity.rule.name}':`, error);
         }
-    } else {
-        console.log(`❌ No simplification opportunities found`);
     }
 
-    console.log(`📊 simplifyStep result:`, {
-        foldedCount,
-        hasNewRoot: newRoot !== rootNode,
-        newRootType: newRoot.type,
-        newRootId: newRoot.id
-    });
-
     return { newRoot, foldedCount, historyEntry };
 }
 

package/omd/step-visualizer/omdStepVisualizer.js

@@ -45,29 +45,12 @@ export class omdStepVisualizer extends omdEquationSequenceNode {
      * Force rebuild visual container (dots/lines) from scratch
      */
     rebuildVisualizer() {
-        try {
-            if (this.visualContainer) {
-                try {
-                    console.log('[rebuildVisualizer] removing old visualContainer', {
-                        childCount: Array.isArray(this.visualContainer.childList) ? this.visualContainer.childList.length : 'n/a'
-                    });
-                } catch (_) {}
-                this.removeChild(this.visualContainer);
-                try {
-                    const stillAttached = !!(this.visualContainer?.svgObject?.parentNode);
-                    console.log('[rebuildVisualizer] removed old visualContainer, stillAttached?', stillAttached);
-                } catch (_) {}
-            }
-        } catch (_) {}
+        if (this.visualContainer) {
+            this.removeChild(this.visualContainer);
+        }
         this.visualContainer = new jsvgLayoutGroup();
         this.addChild(this.visualContainer);
         this._initializeVisualElements();
-        try {
-            console.log('[rebuildVisualizer] new visualContainer', {
-                dots: Array.isArray(this.stepDots) ? this.stepDots.length : 'n/a',
-                lines: Array.isArray(this.stepLines) ? this.stepLines.length : 'n/a'
-            });
-        } catch (_) {}
         this.computeDimensions();
         this.updateLayout();
     }

package/omd/utils/omdPopup.js

@@ -752,8 +752,7 @@ export class omdPopup {
             const expr2Trimmed = expr2.trim();
             const node1 = window.math.simplify(expr1Trimmed);
             const node2 = window.math.simplify(expr2Trimmed);
-            // For debugging, log the stringified forms
-            console.log('Simplified expressions:', { simplified1: node1.toString(), simplified2: node2.toString() });
+
             // If ASTs match, return true
             if (node1.equals(node2)) return true;
 
@@ -1005,7 +1004,7 @@ export class omdPopup {
             const result = await transcriptionService.transcribeWithFallback(imageBlob, {
                 prompt: 'Transcribe this handwritten mathematical expression. Return ONLY the pure mathematical expression with no formatting, no LaTeX, no dollar signs, no explanations. Use ^ for powers (e.g., 3^2), use / for fractions (e.g., (2x+1)/(x-3)), use * for multiplication, use + and - for addition/subtraction. Return only the expression.'
             });
-            console.log('Transcription result:', result);
+
             if (result.text) {
                 this._setSubmitButtonLoading(false);
                 this.flashValidation(true);

package/omd/utils/omdStepVisualizerInteractiveSteps.js

@@ -322,9 +322,6 @@ export class omdStepVisualizerInteractiveSteps {
             stepBox.div.style.backgroundColor = omdColor.mediumGray; // Slightly darker version of explainColor
             stepBox.div.style.transform = 'translateX(2px)';
             
-            // Console logging for hover enter
-
-            
             // Call hover callback if provided
             this.onStepHover?.(stepBox.stepIndex, stepBox.stepMessage, true);
         });
@@ -333,18 +330,12 @@ export class omdStepVisualizerInteractiveSteps {
             stepBox.div.style.backgroundColor = 'transparent';
             stepBox.div.style.transform = 'translateX(0)';
             
-            // Console logging for hover leave
-
-            
             // Call hover callback if provided
             this.onStepHover?.(stepBox.stepIndex, stepBox.stepMessage, false);
         });
         
         // Click interactions
         stepBox.div.addEventListener('click', () => {
-            // Console logging for clicks
-
-            
             this.onStepClick?.(stepBox.stepIndex, stepBox.stepMessage);
         });
     }

package/omd/utils/omdTranscriptionService.js

@@ -12,11 +12,6 @@ export class omdTranscriptionService {
             defaultProvider: options.defaultProvider || 'gemini',
             ...options
         };
-        
-        console.log('omdTranscriptionService constructor called with options:', {
-            endpoint: this.options.endpoint,
-            defaultProvider: this.options.defaultProvider
-        });
     }
     
     /**
@@ -52,7 +47,7 @@ export class omdTranscriptionService {
      */
     async transcribe(imageBlob, options = {}) {
         try {
-            console.log('Starting transcription with endpoint:', this.options.endpoint);
+
             
             // Convert blob to base64
             const base64Image = await this._blobToBase64(imageBlob);
@@ -74,7 +69,7 @@ export class omdTranscriptionService {
             }
             
             const result = await response.json();
-            console.log('Transcription result:', result);
+
             
             return {
                 text: result.text,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@teachinglab/omd",
-  "version": "0.1.0",
+  "version": "0.1.2",
   "description": "omd",
   "main": "./index.js",
   "module": "./index.js",
@@ -42,5 +42,7 @@
     "start": "node --env-file=.env server.js",
     "dev": "node --env-file=.env server.js"
   },
-  "publishConfig": { "access": "public" }
+  "publishConfig": {
+    "access": "public"
+  }
 }