Split group and seq

Author: dy

feature/group.js

@@ -1,15 +1,6 @@
-import { nary, group, operator, compile } from '../parse.js';
+import { group } from '../parse.js';
 
-const STATEMENT = 5, SEQ = 10, ACCESS = 170;
+const ACCESS = 170;
 
 // (a,b,c), (a) — uses ACCESS to avoid conflict with ?.
 group('()', ACCESS);
-
-// Sequences
-nary(',', SEQ);
-nary(';', STATEMENT, true);  // right-assoc to allow same-prec statements
-
-// Compile sequences - returns last evaluated value
-const seq = (...args) => (args = args.map(compile), ctx => { let r; for (const a of args) r = a(ctx); return r; });
-operator(',', seq);
-operator(';', seq);

index.html

@@ -326,6 +326,7 @@ <h1><a href="https://github.com/dy/subscript" class="logo-link" target="_blank"
   literal: 'feature/literal.js',
   access: 'feature/access.js',
   group: 'feature/group.js',
+  seq: 'feature/seq.js',
 
   // Operators (from subscript.js and justin.js)
   assign: 'feature/op/assignment.js',
@@ -380,7 +381,7 @@ <h1><a href="https://github.com/dy/subscript" class="logo-link" target="_blank"
   // Control flow foundations (needed by group via loop)
   'control', 'block', 'destruct', 'loop',
   // Expression features
-  'group', 'access',
+  'seq', 'group', 'access',
   // Justin additions
   'comment', 'identity', 'nullish', 'pow', 'unary',
   'literal', 'ternary', 'arrow', 'spread', 'optional',
@@ -392,7 +393,7 @@ <h1><a href="https://github.com/dy/subscript" class="logo-link" target="_blank"
 
 // UI feature groups (maps to internal IDs)
 const FEATURES = [
-  { cat: 'Core', items: ['number', 'string', 'group', 'access'] },
+  { cat: 'Core', items: ['number', 'string', 'group', 'seq', 'access'] },
   { cat: 'Operators', items: ['assign', 'arithmetic', 'logical', 'bit', 'cmp', 'equality', 'increment', 'pow'] },
   { cat: 'Advanced Ops', items: ['ternary', 'arrow', 'optional', 'spread', 'unary', 'identity', 'nullish'] },
   { cat: 'Literals', items: ['literal', 'collection', 'template', 'regex', 'unit', 'comment'] },
@@ -402,7 +403,7 @@ <h1><a href="https://github.com/dy/subscript" class="logo-link" target="_blank"
 
 const DESC = {
   // Core
-  number: '123, 1.5', string: '"x", \'y\'', group: '()', access: 'a.b, a[i]',
+  number: '123, 1.5', string: '"x", \'y\'', group: '()', seq: ', ;', access: 'a.b, a[i]',
   // Operators
   assign: '=, +=', arithmetic: '+-*/%', logical: '&& || !', bit: '& | ^ ~',
   cmp: '< > <= >=', equality: '== !=', increment: '++ --', pow: '**',
@@ -424,7 +425,8 @@ <h1><a href="https://github.com/dy/subscript" class="logo-link" target="_blank"
 const TIPS = {
   number: 'Decimal numbers<br><code>42</code> <code>3.14</code> <code>1e3</code> <code>0xFF</code> <code>0b101</code>',
   string: 'String literals<br><code>"hello"</code> <code>\'world\'</code> with escapes',
-  group: 'Grouping & sequences<br><code>(a + b)</code> <code>a, b</code> <code>a; b</code>',
+  group: 'Grouping<br><code>(a + b)</code>',
+  seq: 'Sequences<br><code>a, b</code> <code>a; b</code>',
   access: 'Property access<br><code>a.b</code> <code>a[i]</code> <code>fn()</code>',
   assign: 'Assignment operators<br><code>=</code> <code>+=</code> <code>-=</code> <code>*=</code> <code>/=</code>',
   arithmetic: 'Math operators<br><code>+ - * / %</code> unary: <code>+x</code> <code>-x</code>',
@@ -658,15 +660,15 @@ <h1><a href="https://github.com/dy/subscript" class="logo-link" target="_blank"
 
 const PRESETS = {
   // Minimal: just numbers and basic arithmetic
-  minimal: ['number', 'string', 'group', 'arithmetic'],
+  minimal: ['number', 'string', 'seq', 'group', 'arithmetic'],
   // Expressions: subscript.js features
-  expr: ['number', 'string', 'assign', 'logical', 'bit', 'cmp', 'equality', 'arithmetic', 'increment', 'group', 'access'],
+  expr: ['number', 'string', 'assign', 'logical', 'bit', 'cmp', 'equality', 'arithmetic', 'increment', 'seq', 'group', 'access'],
   // Justin: JSON + expressions (no statements)
-  justin: ['number', 'string', 'assign', 'logical', 'bit', 'cmp', 'equality', 'arithmetic', 'increment', 'group', 'access',
+  justin: ['number', 'string', 'assign', 'logical', 'bit', 'cmp', 'equality', 'arithmetic', 'increment', 'seq', 'group', 'access',
            'comment', 'identity', 'nullish', 'pow', 'unary', 'literal', 'ternary', 'arrow', 'spread', 'optional',
            'collection', 'template'],
   // Jessie: practical JS subset with statements
-  jessie: ['number', 'string', 'assign', 'logical', 'bit', 'cmp', 'equality', 'arithmetic', 'increment', 'group', 'access',
+  jessie: ['number', 'string', 'assign', 'logical', 'bit', 'cmp', 'equality', 'arithmetic', 'increment', 'seq', 'group', 'access',
            'comment', 'identity', 'nullish', 'pow', 'unary', 'literal', 'ternary', 'arrow', 'spread', 'optional',
            'collection', 'template', 'regex', 'block', 'var', 'function', 'destruct', 'if', 'loop', 'try', 'switch', 'asi'],
   // Full: all features

package.json

@@ -29,8 +29,9 @@
   ],
   "scripts": {
     "build": "esbuild subscript.js --bundle --minify --format=esm --outfile=subscript.min.js && esbuild justin.js --bundle --minify --format=esm --outfile=justin.min.js && esbuild jessie.js --bundle --minify --format=esm --outfile=jessie.min.js",
-    "prepublishOnly": "npm run build",
+    "prepublishOnly": "npm run build && npm run test:e2e",
     "test": "node --import ./test/https-loader.js test/test.js",
+    "test:e2e": "npx playwright test test/index.spec.js",
     "check-types": "tsc --noEmit subscript.d.ts"
   },
   "repository": {

subscript.js

@@ -24,7 +24,8 @@ import './feature/op/equality.js';    // == !=
 import './feature/op/arithmetic.js';  // + - * / %
 import './feature/op/increment.js';   // ++ --
 
-import './feature/group.js';          // Grouping: (a), sequences: a, b; a; b
+import './feature/seq.js';            // Sequences: a, b; a; b
+import './feature/group.js';          // Grouping: (a)
 import './feature/access.js';         // Property access: a.b, a[b], f(), [a,b]
 
 import { parse, compile } from './parse.js';