squirreling 0.7.4 → 0.7.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "squirreling",
-  "version": "0.7.4",
+  "version": "0.7.5",
   "description": "Squirreling SQL Engine",
   "author": "Hyperparam",
   "homepage": "https://hyperparam.app",
@@ -40,7 +40,7 @@
     "@types/node": "25.0.3",
     "@vitest/coverage-v8": "4.0.16",
     "eslint": "9.39.2",
-    "eslint-plugin-jsdoc": "61.5.0",
+    "eslint-plugin-jsdoc": "62.0.0",
     "typescript": "5.9.3",
     "vitest": "4.0.16"
   }

package/src/execute/expression.js

@@ -1,5 +1,5 @@
 import { unknownFunctionError } from '../parseErrors.js'
-import { invalidContextError } from '../executionErrors.js'
+import { columnNotFoundError, invalidContextError } from '../executionErrors.js'
 import { aggregateError, argValueError, castError } from '../validationErrors.js'
 import { isAggregateFunc, isMathFunc, isRegexpFunc, isStringFunc } from '../validation.js'
 import { applyIntervalToDate } from './date.js'
@@ -32,17 +32,23 @@ export async function evaluateExpr({ node, row, tables, functions, rowIndex, row
 
   if (node.type === 'identifier') {
     // Try exact match first (handles both qualified and unqualified names)
-    if (row.cells[node.name]) {
+    if (node.name in row.cells) {
       return row.cells[node.name]()
     }
     // For qualified names like 'users.id', also try just the column part
     if (node.name.includes('.')) {
       const colName = node.name.split('.').pop()
-      if (colName && row.cells[colName]) {
+      if (colName && colName in row.cells) {
         return row.cells[colName]()
       }
     }
-    return null
+    throw columnNotFoundError({
+      columnName: node.name,
+      availableColumns: Object.keys(row.cells),
+      positionStart: node.positionStart,
+      positionEnd: node.positionEnd,
+      rowNumber: rowIndex,
+    })
   }
 
   // Scalar subquery - returns a single value

package/src/executionErrors.js

@@ -24,8 +24,6 @@ export class ExecutionError extends Error {
 }
 
 /**
- * Error for missing table.
- *
  * @param {Object} options
  * @param {string} options.tableName - The missing table name
  * @returns {Error}
@@ -50,8 +48,6 @@ export function invalidContextError({ item, validContext, positionStart, positio
 }
 
 /**
- * Error for unsupported operation combinations.
- *
  * @param {Object} options
  * @param {string} options.operation - The unsupported operation
  * @param {string} [options.hint] - How to fix it
@@ -61,3 +57,24 @@ export function unsupportedOperationError({ operation, hint }) {
   const suffix = hint ? `. ${hint}` : ''
   return new Error(`${operation}${suffix}`)
 }
+
+/**
+ * @param {Object} options
+ * @param {string} options.columnName - The missing column name
+ * @param {string[]} options.availableColumns - List of available column names
+ * @param {number} options.positionStart - Start position in query
+ * @param {number} options.positionEnd - End position in query
+ * @param {number} [options.rowNumber] - 1-based row number where error occurred
+ * @returns {ExecutionError}
+ */
+export function columnNotFoundError({ columnName, availableColumns, positionStart, positionEnd, rowNumber }) {
+  const available = availableColumns.length > 0
+    ? `. Available columns: ${availableColumns.join(', ')}`
+    : ''
+  return new ExecutionError({
+    message: `Column "${columnName}" not found${available}`,
+    positionStart,
+    positionEnd,
+    rowNumber,
+  })
+}

package/src/parse/parse.js

@@ -97,29 +97,32 @@ export function parseSql({ query, functions }) {
 function parseSelectList(state) {
   /** @type {SelectColumn[]} */
   const cols = []
-  const tok = current(state)
 
-  // Check for qualified asterisk (table.*)
-  if (tok.type === 'identifier') {
-    const next = peekToken(state, 1)
-    const nextNext = peekToken(state, 2)
-    if (next.type === 'dot' && nextNext.type === 'operator' && nextNext.value === '*') {
-      const tableTok = consume(state) // consume table name
-      consume(state) // consume dot
-      consume(state) // consume asterisk
-      cols.push({ kind: 'star', table: tableTok.value })
-      return cols
+  while (true) {
+    const tok = current(state)
+
+    // Check for qualified asterisk (table.*)
+    if (tok.type === 'identifier') {
+      const next = peekToken(state, 1)
+      const nextNext = peekToken(state, 2)
+      if (next.type === 'dot' && nextNext.type === 'operator' && nextNext.value === '*') {
+        const tableTok = consume(state) // consume table name
+        consume(state) // consume dot
+        consume(state) // consume asterisk
+        cols.push({ kind: 'star', table: tableTok.value })
+        if (!match(state, 'comma')) break
+        continue
+      }
     }
-  }
 
-  // Check for unqualified asterisk (*)
-  if (tok.type === 'operator' && tok.value === '*') {
-    consume(state)
-    cols.push({ kind: 'star' })
-    return cols
-  }
+    // Check for unqualified asterisk (*)
+    if (tok.type === 'operator' && tok.value === '*') {
+      consume(state)
+      cols.push({ kind: 'star' })
+      if (!match(state, 'comma')) break
+      continue
+    }
 
-  while (true) {
     cols.push(parseSelectItem(state))
     if (!match(state, 'comma')) break
   }

package/src/types.d.ts

@@ -38,7 +38,7 @@ export interface ScanOptions {
  * All hints are optional and "best effort" - sources may ignore them.
  */
 export interface QueryHints {
-  columns?: string[] // columns needed
+  columns?: string[] // columns needed (undefined means all columns)
   where?: ExprNode // where clause
   // important: only apply limit/offset if where is fully applied by the data source
   // otherwise, the data source must return at least enough rows to ensure the engine