Fix repl

Author: dy

debug-bundle-node.mjs

@@ -0,0 +1,76 @@
+import fs from 'fs'
+
+// Get source files
+const constSrc = fs.readFileSync('./src/const.js', 'utf8')
+const parseSrc = fs.readFileSync('./src/parse.js', 'utf8')
+const compileSrc = fs.readFileSync('./src/compile.js', 'utf8')
+
+// Replicate the bundle generation from repl.html
+const stripImports = code => code.replace(/^import\s+.*$/gm, '')
+const stripExports = code => code
+  .replace(/^export\s+default\s+/gm, '')
+  .replace(/^export\s+(const|let|function)\s+/gm, '$1 ')
+  .replace(/^export\s+\{[^}]+\}.*$/gm, '')
+
+let bundle = `// Bundled subscript
+// https://github.com/phtml/subscript
+
+(function() {
+'use strict';
+
+`
+
+// 1. Constants
+bundle += `// === src/const.js ===\n`
+bundle += stripExports(constSrc) + '\n\n'
+
+// 2. Parse
+bundle += `// === src/parse.js ===\n`
+let parseModule = stripImports(parseSrc)
+  .replace(/^export\s+let\s+/gm, 'let ')
+  .replace(/^export\s+default\s+parse\s*;?\s*$/gm, '')
+bundle += parseModule + '\n\n'
+
+// 3. Compile
+bundle += `// === src/compile.js ===\n`
+let compileModule = stripImports(compileSrc)
+  .replace(/^export\s+const\s+/gm, 'const ')
+  .replace(/^export\s+default\s+compile\s*;?\s*$/gm, '')
+bundle += compileModule + '\n\n'
+
+// Close IIFE
+bundle += `window.parse = parse;
+window.compile = compile;
+})();
+`
+
+console.log('Bundle size:', bundle.length, 'bytes')
+
+// Test if it's valid JavaScript
+try {
+  new Function(bundle)
+  console.log('✓ Bundle is syntactically valid!')
+} catch (e) {
+  console.log('❌ Bundle has syntax error:', e.message)
+  // Find the line with error
+  const lines = bundle.split('\n')
+  const match = e.message.match(/position (\d+)/)
+  if (match) {
+    let pos = parseInt(match[1])
+    let lineNum = 0
+    for (let i = 0; i < lines.length; i++) {
+      if (pos <= lines[i].length) {
+        console.log(`Error around line ${i}: ${lines[i].slice(0, 100)}`)
+        break
+      }
+      pos -= lines[i].length + 1
+    }
+  }
+}
+
+// Show compile.js section
+console.log('\n=== COMPILE.JS SECTION (first 30 lines) ===')
+const compileStart = bundle.indexOf('// === src/compile.js ===')
+const compileEnd = bundle.indexOf('window.parse')
+const compileSection = bundle.slice(compileStart, compileEnd).split('\n').slice(0, 35)
+compileSection.forEach((line, i) => console.log(`${i}: ${line.slice(0, 120)}`))

debug-bundle.mjs

@@ -0,0 +1,61 @@
+import { chromium } from 'playwright'
+
+const browser = await chromium.launch({ headless: true })
+const page = await browser.newPage()
+
+// Capture console messages
+const consoleLogs = []
+page.on('console', msg => consoleLogs.push(`[${msg.type()}] ${msg.text()}`))
+
+// Navigate to REPL
+await page.goto('http://localhost:8765/repl.html', { waitUntil: 'domcontentloaded' })
+await page.waitForTimeout(3000) // Wait for init
+
+// Enable bundle toggle and click get bundle
+try {
+  const toggle = await page.$('#bundleToggle')
+  if (toggle) {
+    await toggle.check()
+    await page.waitForTimeout(500)
+  }
+  
+  const btn = await page.$('#getBundleBtn')
+  if (btn) {
+    await btn.click()
+    await page.waitForTimeout(2000)
+  }
+} catch (e) {
+  console.log('Error interacting with page:', e.message)
+}
+
+// Print all console logs
+console.log('\n=== CONSOLE OUTPUT ===')
+consoleLogs.forEach(log => console.log(log))
+
+// Get the modal code content (what bundle was generated)
+try {
+  const modalText = await page.textContent('#modalCode')
+  if (modalText && modalText.length > 0) {
+    // Just show first 3000 chars to see structure
+    console.log('\n=== BUNDLE (first 3000 chars) ===')
+    console.log(modalText.slice(0, 3000))
+    
+    // Find where "parse" appears
+    console.log('\n=== PARSE DECLARATIONS ===')
+    const lines = modalText.split('\n')
+    let parseCount = 0
+    lines.forEach((line, i) => {
+      if (/\bparse\s*[=\(]|^parse|function parse|const parse|let parse|var parse/.test(line)) {
+        parseCount++
+        console.log(`Line ${i}: ${line.trim().slice(0, 100)}`)
+        if (parseCount > 10) console.log('... (more matches)')
+      }
+    })
+  }
+} catch (e) {
+  console.log('Error getting modal text:', e.message)
+}
+
+await browser.close()
+console.log('\n✓ Debug complete')
+

feature/asi.js

@@ -0,0 +1,69 @@
+// ASI (Automatic Semicolon Insertion) preprocessor
+// Wraps input to insert semicolons where JS would infer them
+// Not a full JS spec implementation - covers common practical cases
+
+// Tokens that REQUIRE continuation (don't insert ; before these)
+// Note: [ and { can START statements (array literal, block), so not included
+const CONT_STARTS = /^[\.\(\+\-\*\/\%\&\|\^\~\?\:\,\<\>]|^&&|^\|\||^\.\.\.$/
+
+// Lines that can't be statements by themselves (incomplete)
+const INCOMPLETE = /[\+\-\*\/\%\&\|\^\~\?\:\,\=\<\>\(\{]$/
+
+export function asi(src, { keepNewlines = false } = {}) {
+  const lines = src.split('\n')
+  const result = []
+  
+  for (let i = 0; i < lines.length; i++) {
+    const line = lines[i].trim()
+    
+    // Skip empty lines  
+    if (!line) {
+      if (keepNewlines) result.push(lines[i])
+      continue
+    }
+    
+    // Skip comment lines
+    if (line.startsWith('//') || line.startsWith('/*')) {
+      if (keepNewlines) result.push(lines[i])
+      continue
+    }
+    
+    result.push(lines[i])
+    
+    // Already ends with semicolon, opening brace, comma, or closing brace - no insert
+    if (line.endsWith(';') || line.endsWith('{') || line.endsWith(',') || line.endsWith('}')) continue
+    
+    // Line is incomplete (ends with operator/opening paren) - no insert
+    if (INCOMPLETE.test(line)) continue
+    
+    // Find next non-empty, non-comment line
+    let nextLine = ''
+    for (let j = i + 1; j < lines.length; j++) {
+      const nl = lines[j].trim()
+      if (nl && !nl.startsWith('//') && !nl.startsWith('/*')) {
+        nextLine = nl
+        break
+      }
+    }
+    
+    // Next line starts with continuation token - no insert
+    const nextToken = nextLine.match(/^[^\s\w]*|^\w+/)?.[0] || ''
+    if (nextLine && CONT_STARTS.test(nextToken)) continue
+    
+    // If this is NOT the last code line, add semicolon
+    // (Don't add trailing semicolon - it makes result undefined)
+    if (nextLine) {
+      result[result.length - 1] = result[result.length - 1] + ';'
+    }
+  }
+  
+  // Join with space instead of newline for parser compatibility
+  return result.join(keepNewlines ? '\n' : ' ')
+}
+
+// Wrap parse function with ASI preprocessing
+export function withASI(parse) {
+  return (src, ...args) => parse(asi(src), ...args)
+}
+
+export default asi

feature/loop.js

@@ -13,7 +13,7 @@ import { PREC_STATEMENT, OPAREN, CPAREN, CBRACE, PREC_SEQ, PREC_TOKEN } from '..
 import { parseBody, loop, BREAK, CONTINUE, Return_ as Return } from './block.js'
 export { BREAK, CONTINUE } from './block.js'
 
-const { token, expr, skip, space, err } = P
+const { token, expr, skip, space, err, next, parse } = P
 const SEMI = 59
 
 // while (cond) body
@@ -40,11 +40,35 @@ operator('while', (cond, body) => {
 })
 
 // for (init; cond; step) body
+// Note: init supports both expressions AND let/const declarations
 token('for', PREC_STATEMENT, a => {
   if (a) return
   space() === OPAREN || err('Expected (')
   skip()
-  const init = space() === SEMI ? null : expr(PREC_SEQ)
+  // Parse init: can be expression (PREC_SEQ) or let/const declaration (PREC_STATEMENT)
+  // Using expr(PREC_SEQ) excludes statement-level tokens like let/const
+  // So we detect let/const keywords and parse the declaration inline
+  let init
+  const cc = space()
+  if (cc === SEMI) init = null
+  else if (cc === 108 && P.cur.substr(P.idx, 3) === 'let' && !parse.id(P.cur.charCodeAt(P.idx + 3))) {
+    skip(); skip(); skip() // skip 'let'
+    space()
+    const name = next(parse.id)
+    if (!name) err('Expected identifier')
+    space()
+    if (P.cur.charCodeAt(P.idx) === 61 && P.cur.charCodeAt(P.idx + 1) !== 61) {
+      skip(); init = ['let', name, expr(PREC_SEQ)]
+    } else init = ['let', name]
+  } else if (cc === 99 && P.cur.substr(P.idx, 5) === 'const' && !parse.id(P.cur.charCodeAt(P.idx + 5))) {
+    skip(); skip(); skip(); skip(); skip() // skip 'const'
+    space()
+    const name = next(parse.id)
+    if (!name) err('Expected identifier')
+    space()
+    P.cur.charCodeAt(P.idx) === 61 && P.cur.charCodeAt(P.idx + 1) !== 61 || err('Expected =')
+    skip(); init = ['const', name, expr(PREC_SEQ)]
+  } else init = expr(PREC_SEQ)
   space() === SEMI ? skip() : err('Expected ;')
   const cond = space() === SEMI ? null : expr(PREC_SEQ)
   space() === SEMI ? skip() : err('Expected ;')

feature/unit.js

@@ -53,6 +53,7 @@ const wrapHandler = (charCode) => {
 }
 
 // Wrap all number entry points (0-9 and .)
-const PERIOD = 46, _0 = 48, _9 = 57
+// PERIOD, _0, _9 are from src/const.js
+import { PERIOD, _0, _9 } from '../src/const.js'
 wrapHandler(PERIOD)
 for (let i = _0; i <= _9; i++) wrapHandler(i)