squirreling 0.12.23 → 0.12.24

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "squirreling",
-  "version": "0.12.23",
+  "version": "0.12.24",
   "description": "Squirreling Async SQL Engine",
   "author": "Hyperparam",
   "homepage": "https://hyperparam.app",
@@ -39,11 +39,11 @@
     "test": "vitest run"
   },
   "devDependencies": {
-    "@types/node": "25.9.1",
-    "@vitest/coverage-v8": "4.1.7",
+    "@types/node": "25.9.3",
+    "@vitest/coverage-v8": "4.1.8",
     "eslint": "9.39.4",
-    "eslint-plugin-jsdoc": "63.0.0",
+    "eslint-plugin-jsdoc": "63.0.2",
     "typescript": "6.0.3",
-    "vitest": "4.1.7"
+    "vitest": "4.1.8"
   }
 }

package/src/ast.d.ts

@@ -212,6 +212,7 @@ export interface JoinClause extends AstBase {
   on?: ExprNode
   using?: string[]
   fromFunction?: FromFunction
+  subquery?: FromSubquery
 }
 
 // All AST node derive from this base, which includes position info for error reporting and other purposes

package/src/expression/binary.js

@@ -55,7 +55,7 @@ export function applyBinaryOp(op, a, b) {
       .replace(/[.*+?^${}()|[\]\\]/g, '\\$&')
       .replace(/%/g, '.*')
       .replace(/_/g, '.')
-    const regex = new RegExp(`^${regexPattern}$`, 'i')
+    const regex = new RegExp(`^${regexPattern}$`, 'is')
     return regex.test(str)
   }
 

package/src/parse/extractTables.js

@@ -59,6 +59,8 @@ function walkStatement(stmt, cteScope, refs) {
   for (const j of stmt.joins) {
     if (j.fromFunction) {
       for (const a of j.fromFunction.args) walkExpr(a, cteScope, refs)
+    } else if (j.subquery) {
+      walkStatement(j.subquery.query, cteScope, refs)
     } else if (!cteScope.has(j.table.toLowerCase())) {
       refs.add(j.table)
     }

package/src/parse/joins.js

@@ -2,11 +2,11 @@ import { expectNoAggregate } from '../validation/aggregates.js'
 import { isTableFunction, validateFunctionArgs } from '../validation/functions.js'
 import { ParseError } from '../validation/parseErrors.js'
 import { parseExpression } from './expression.js'
-import { isTableFunctionStart, parseFromFunction, parseTableAlias, tableFunctionColumnCount, tableFunctionDefaultColumns } from './parse.js'
+import { isTableFunctionStart, parseFromFunction, parseStatement, parseTableAlias, tableFunctionColumnCount, tableFunctionDefaultColumns } from './parse.js'
 import { consume, current, expect, match } from './state.js'
 
 /**
- * @import { ExprNode, FromFunction, JoinClause, JoinType, ParserState } from '../types.js'
+ * @import { ExprNode, FromFunction, FromSubquery, JoinClause, JoinType, ParserState } from '../types.js'
  */
 
 /**
@@ -218,9 +218,42 @@ export function parseJoins(state) {
       })
     }
 
-    // Parse table name and optional alias
-    const tableTok = expect(state, 'identifier')
-    const tableAlias = parseTableAlias(state)
+    // Subquery on the right side: JOIN (SELECT ...) AS alias ON ...
+    const rightTok = current(state)
+    /** @type {FromSubquery | undefined} */
+    let subquery
+    let tableName
+    /** @type {string | undefined} */
+    let tableAlias
+    let endPos
+    if (rightTok.type === 'paren' && rightTok.value === '(') {
+      consume(state)
+      const query = parseStatement(state)
+      expect(state, 'paren', ')')
+      tableAlias = parseTableAlias(state)
+      if (!tableAlias) {
+        throw new ParseError({
+          message: 'Subquery in JOIN must have an alias',
+          positionStart: rightTok.positionStart,
+          positionEnd: state.lastPos,
+        })
+      }
+      endPos = state.lastPos
+      subquery = {
+        type: 'subquery',
+        query,
+        alias: tableAlias,
+        positionStart: rightTok.positionStart,
+        positionEnd: endPos,
+      }
+      tableName = tableAlias
+    } else {
+      // Parse table name and optional alias
+      const tableTok = expect(state, 'identifier')
+      tableName = tableTok.value
+      tableAlias = parseTableAlias(state)
+      endPos = tableTok.positionEnd
+    }
 
     // Parse ON condition or USING column list (not for POSITIONAL joins)
     /** @type {ExprNode | undefined} */
@@ -246,12 +279,13 @@ export function parseJoins(state) {
 
     joins.push({
       joinType,
-      table: tableTok.value,
+      table: tableName,
       alias: tableAlias,
       on: condition,
       using,
+      subquery,
       positionStart: tok.positionStart,
-      positionEnd: tableTok.positionEnd,
+      positionEnd: endPos,
     })
   }
 

package/src/plan/columns.js

@@ -420,6 +420,10 @@ export function inferSelectSourceColumns({ select, cteColumns, tables }) {
         for (const col of tableFunctionColumnNames(join.fromFunction)) {
           result.push(`${joinAlias}.${col}`)
         }
+      } else if (join.subquery) {
+        for (const col of inferStatementColumns({ stmt: join.subquery.query, cteColumns, tables })) {
+          result.push(`${joinAlias}.${col}`)
+        }
       } else {
         for (const col of lookupTableColumns(join.table, cteColumns, tables)) {
           result.push(`${joinAlias}.${col}`)
@@ -446,6 +450,10 @@ export function inferSelectSourceColumns({ select, cteColumns, tables }) {
       for (const col of tableFunctionColumnNames(join.fromFunction)) {
         result.push(`${joinAlias}.${col}`)
       }
+    } else if (join.subquery) {
+      for (const col of inferStatementColumns({ stmt: join.subquery.query, cteColumns, tables })) {
+        result.push(`${joinAlias}.${col}`)
+      }
     } else {
       for (const col of lookupTableColumns(join.table, cteColumns, tables)) {
         result.push(`${joinAlias}.${col}`)

package/src/plan/plan.js

@@ -447,18 +447,47 @@ function planJoin({ left, joins, leftTable, ctePlans, cteColumns, perTableColumn
       continue
     }
 
-    const ctePlan = ctePlans?.get(join.table.toLowerCase())
-    /** @type {ScanOptions} */
-    const rightHints = {}
-    if (!ctePlan) {
-      rightHints.columns = perTableColumns.get(rightTable)
-      validateScan({ ...join, hints: rightHints, tables })
+    /** @type {QueryPlan} */
+    let rightScan
+    if (join.subquery) {
+      // Subquery on the right side of the join (derived table). Mirror the
+      // FROM-clause subquery handling: plan the inner statement, push down the
+      // columns the outer query needs, and wrap in the inner scope so
+      // correlated subqueries inside resolve against the right aliases.
+      let subColumns = perTableColumns.get(rightTable)
+      // Empty array means no columns referenced, but the derived table still
+      // needs its own columns. Treat empty as unrestricted.
+      if (subColumns?.length === 0) subColumns = undefined
+      const subPlan = planStatement({
+        stmt: join.subquery.query,
+        ctePlans,
+        cteColumns,
+        tables,
+        outerScope,
+        parentColumns: subColumns?.map(name => ({ type: 'identifier', name, positionStart: 0, positionEnd: 0 })),
+      })
+      const availableColumns = inferStatementColumns({ stmt: join.subquery.query, cteColumns, tables })
+      if (subColumns && availableColumns.length) {
+        const missingColumn = subColumns.find(col => !availableColumns.includes(col))
+        if (missingColumn) {
+          throw new ColumnNotFoundError({ missingColumn, availableColumns, ...join.subquery })
+        }
+      }
+      const innerScope = statementScope(join.subquery.query)
+      rightScan = innerScope ? { type: 'Subquery', scope: innerScope, child: subPlan } : subPlan
     } else {
-      // For CTE joins, use CTE column metadata for hints
-      rightHints.columns = perTableColumns.get(rightTable) ?? cteColumns?.get(join.table.toLowerCase())
+      const ctePlan = ctePlans?.get(join.table.toLowerCase())
+      /** @type {ScanOptions} */
+      const rightHints = {}
+      if (!ctePlan) {
+        rightHints.columns = perTableColumns.get(rightTable)
+        validateScan({ ...join, hints: rightHints, tables })
+      } else {
+        // For CTE joins, use CTE column metadata for hints
+        rightHints.columns = perTableColumns.get(rightTable) ?? cteColumns?.get(join.table.toLowerCase())
+      }
+      rightScan = ctePlan ?? { type: 'Scan', table: join.table, hints: rightHints }
     }
-    /** @type {QueryPlan} */
-    const rightScan = ctePlan ?? { type: 'Scan', table: join.table, hints: rightHints }
 
     if (join.joinType === 'POSITIONAL') {
       plan = { type: 'PositionalJoin', leftAlias: currentLeftTable, rightAlias: rightTable, left: plan, right: rightScan }