@malloydata/malloy 0.0.434 → 0.0.436

package/dist/index.js

@@ -37,9 +37,9 @@ var __importStar = (this && this.__importStar) || (function () {
     };
 })();
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.Model = exports.MalloyTranslator = exports.malloyToQuery = exports.constantExprToSQL = exports.isDateUnit = exports.isTimestampUnit = exports.composeSQLExpr = exports.indent = exports.expressionIsUngroupedAggregate = exports.expressionIsScalar = exports.expressionIsCalculation = exports.expressionIsAnalytic = exports.expressionIsAggregate = exports.mkFieldDef = exports.mkArrayDef = exports.isBasicArray = exports.isRepeatedRecord = exports.isSamplingRows = exports.isSamplingPercent = exports.isSamplingEnable = exports.isJoinedSource = exports.isJoined = exports.isCompoundArrayData = exports.isBasicAtomic = exports.isAtomic = exports.isSourceDef = exports.validateDottedTablePath = exports.decodeDottedTablePath = exports.Dialect = exports.spread = exports.literal = exports.variadicParam = exports.param = exports.makeParam = exports.sql = exports.maxScalar = exports.minAggregate = exports.anyExprType = exports.minScalar = exports.overload = exports.qtz = exports.arg = exports.registerDialect = exports.DatabricksDialect = exports.MySQLDialect = exports.SnowflakeDialect = exports.PostgresDialect = exports.TrinoDialect = exports.StandardSQLDialect = exports.DuckDBDialect = void 0;
-exports.Reference = exports.EMPTY_BUILD_MANIFEST = exports.PersistSource = exports.annotationToTaglines = exports.annotationToTag = exports.tagFromAnnotations = exports.annotationsForRoute = exports.payloadOf = exports.routeOf = exports.RoutedNote = exports.Annotations = exports.sqlKey = exports.API = exports.sourceDefToSourceInfo = exports.modelDefToModelInfo = exports.toAsyncGenerator = exports.createConnectionsFromConfig = exports.getRegisteredConnectionTypes = exports.getConnectionTypeDisplayName = exports.getConnectionProperties = exports.registerConnectionType = exports.discoverConfig = exports.defaultConfigOverlays = exports.contextOverlay = exports.envOverlay = exports.MalloyConfig = exports.Manifest = exports.CacheManager = exports.InMemoryModelCache = exports.Explore = exports.DataWriter = exports.Parse = exports.JSONWriter = exports.CSVWriter = exports.QueryMaterializer = exports.Result = exports.PreparedResult = exports.TimestampTimeframe = exports.DateTimeframe = exports.SourceRelationship = exports.JoinRelationship = exports.MalloyError = exports.FixedConnectionMap = exports.InMemoryURLReader = exports.EmptyURLReader = exports.SingleConnectionRuntime = exports.ConnectionRuntime = exports.AtomicFieldType = exports.Runtime = exports.Malloy = void 0;
-exports.makeDigest = void 0;
+exports.malloyToQuery = exports.constantExprToSQL = exports.isDateUnit = exports.isTimestampUnit = exports.composeSQLExpr = exports.indent = exports.expressionIsUngroupedAggregate = exports.expressionIsScalar = exports.expressionIsCalculation = exports.expressionIsAnalytic = exports.expressionIsAggregate = exports.mkFieldDef = exports.mkArrayDef = exports.isBasicArray = exports.isRepeatedRecord = exports.isSamplingRows = exports.isSamplingPercent = exports.isSamplingEnable = exports.isJoinedSource = exports.isJoined = exports.isCompoundArrayData = exports.isBasicAtomic = exports.isAtomic = exports.isSourceDef = exports.validateDottedTablePath = exports.decodeDottedTablePath = exports.Dialect = exports.spread = exports.literal = exports.variadicParam = exports.param = exports.makeParam = exports.sql = exports.maxScalar = exports.minAggregate = exports.anyExprType = exports.minScalar = exports.overload = exports.queryZone = exports.arg = exports.getDialect = exports.registerDialect = exports.SQLServerDialect = exports.DatabricksDialect = exports.MySQLDialect = exports.SnowflakeDialect = exports.PostgresDialect = exports.TrinoDialect = exports.StandardSQLDialect = exports.DuckDBDialect = void 0;
+exports.PersistSource = exports.annotationToTaglines = exports.annotationToTag = exports.tagFromAnnotations = exports.annotationsForRoute = exports.payloadOf = exports.routeOf = exports.RoutedNote = exports.Annotations = exports.sqlKey = exports.API = exports.sourceDefToSourceInfo = exports.modelDefToModelInfo = exports.toAsyncGenerator = exports.createConnectionsFromConfig = exports.getRegisteredConnectionTypes = exports.getConnectionTypeDisplayName = exports.getConnectionProperties = exports.registerConnectionType = exports.discoverConfig = exports.defaultConfigOverlays = exports.contextOverlay = exports.envOverlay = exports.MalloyConfig = exports.Manifest = exports.CacheManager = exports.InMemoryModelCache = exports.Explore = exports.DataWriter = exports.Parse = exports.JSONWriter = exports.CSVWriter = exports.QueryMaterializer = exports.Result = exports.PreparedResult = exports.TimestampTimeframe = exports.DateTimeframe = exports.SourceRelationship = exports.JoinRelationship = exports.MalloyError = exports.FixedConnectionMap = exports.InMemoryURLReader = exports.EmptyURLReader = exports.SingleConnectionRuntime = exports.ConnectionRuntime = exports.AtomicFieldType = exports.Runtime = exports.Malloy = exports.Model = exports.MalloyTranslator = void 0;
+exports.makeDigest = exports.Reference = exports.EMPTY_BUILD_MANIFEST = void 0;
 var dialect_1 = require("./dialect");
 Object.defineProperty(exports, "DuckDBDialect", { enumerable: true, get: function () { return dialect_1.DuckDBDialect; } });
 Object.defineProperty(exports, "StandardSQLDialect", { enumerable: true, get: function () { return dialect_1.StandardSQLDialect; } });
@@ -48,9 +48,11 @@ Object.defineProperty(exports, "PostgresDialect", { enumerable: true, get: funct
 Object.defineProperty(exports, "SnowflakeDialect", { enumerable: true, get: function () { return dialect_1.SnowflakeDialect; } });
 Object.defineProperty(exports, "MySQLDialect", { enumerable: true, get: function () { return dialect_1.MySQLDialect; } });
 Object.defineProperty(exports, "DatabricksDialect", { enumerable: true, get: function () { return dialect_1.DatabricksDialect; } });
+Object.defineProperty(exports, "SQLServerDialect", { enumerable: true, get: function () { return dialect_1.SQLServerDialect; } });
 Object.defineProperty(exports, "registerDialect", { enumerable: true, get: function () { return dialect_1.registerDialect; } });
+Object.defineProperty(exports, "getDialect", { enumerable: true, get: function () { return dialect_1.getDialect; } });
 Object.defineProperty(exports, "arg", { enumerable: true, get: function () { return dialect_1.arg; } });
-Object.defineProperty(exports, "qtz", { enumerable: true, get: function () { return dialect_1.qtz; } });
+Object.defineProperty(exports, "queryZone", { enumerable: true, get: function () { return dialect_1.queryZone; } });
 Object.defineProperty(exports, "overload", { enumerable: true, get: function () { return dialect_1.overload; } });
 Object.defineProperty(exports, "minScalar", { enumerable: true, get: function () { return dialect_1.minScalar; } });
 Object.defineProperty(exports, "anyExprType", { enumerable: true, get: function () { return dialect_1.anyExprType; } });

package/dist/lang/ast/expressions/boolean-value.d.ts

@@ -0,0 +1,24 @@
+import type { ExprValue } from '../types/expr-value';
+import type { FieldSpace } from '../types/field-space';
+import type { MalloyElement } from '../types/malloy-element';
+/**
+ * On a dialect with booleanType 'none' a boolean is a condition and never a
+ * value. When `type` is a boolean about to be used as a value, logs an error
+ * at `logTo` and returns an error-typed value for the caller to return in
+ * place of what it was building, so the enclosing expressions see an error
+ * and stay quiet. Otherwise returns undefined. `where` says how the boolean
+ * was used, `hint` what to write instead.
+ */
+export declare function booleanValueError(fs: FieldSpace, type: {
+    type: string;
+} | undefined, where: string, logTo: MalloyElement, hint: string): ExprValue | undefined;
+/** The hint where a boolean's unknown state is examined: `is null`, `??` */
+export declare const CONDITION_HINT = "test the condition itself; unknown already tests as false";
+/** The hint for a boolean compared with anything */
+export declare const COMPARE_HINT = "test the condition itself, or its not";
+/** The hint where a number would do: a function argument, a cast */
+export declare const NUMBER_HINT = "write the value, pick 1 when \u2026 else 0";
+/** The hint for a boolean as a result column, from the source text it came from */
+export declare function resultColumnHint(src: string): string;
+/** The hint for a `pick` or `case` whose result is a boolean */
+export declare const CHOICE_HINT = "write values, pick 'yes' when \u2026 else 'no', or the condition itself with and, or and not";

package/dist/lang/ast/expressions/boolean-value.js

@@ -0,0 +1,41 @@
+"use strict";
+/*
+ * Copyright Contributors to the Malloy project
+ * SPDX-License-Identifier: MIT
+ */
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.CHOICE_HINT = exports.NUMBER_HINT = exports.COMPARE_HINT = exports.CONDITION_HINT = void 0;
+exports.booleanValueError = booleanValueError;
+exports.resultColumnHint = resultColumnHint;
+/**
+ * On a dialect with booleanType 'none' a boolean is a condition and never a
+ * value. When `type` is a boolean about to be used as a value, logs an error
+ * at `logTo` and returns an error-typed value for the caller to return in
+ * place of what it was building, so the enclosing expressions see an error
+ * and stay quiet. Otherwise returns undefined. `where` says how the boolean
+ * was used, `hint` what to write instead.
+ */
+function booleanValueError(fs, type, where, logTo, hint) {
+    const dialect = fs.dialectObj();
+    if ((type === null || type === void 0 ? void 0 : type.type) !== 'boolean' || (dialect === null || dialect === void 0 ? void 0 : dialect.booleanType) !== 'none') {
+        return undefined;
+    }
+    return logTo.loggedErrorExpr('boolean-value-unsupported', {
+        dialect: dialect.name,
+        where,
+        hint,
+    });
+}
+/** The hint where a boolean's unknown state is examined: `is null`, `??` */
+exports.CONDITION_HINT = 'test the condition itself; unknown already tests as false';
+/** The hint for a boolean compared with anything */
+exports.COMPARE_HINT = 'test the condition itself, or its not';
+/** The hint where a number would do: a function argument, a cast */
+exports.NUMBER_HINT = 'write the value, pick 1 when … else 0';
+/** The hint for a boolean as a result column, from the source text it came from */
+function resultColumnHint(src) {
+    return `write the value, pick 'yes' when ${src} else 'no'`;
+}
+/** The hint for a `pick` or `case` whose result is a boolean */
+exports.CHOICE_HINT = "write values, pick 'yes' when … else 'no', or the condition itself with and, or and not";
+//# sourceMappingURL=boolean-value.js.map

package/dist/lang/ast/expressions/case.js

@@ -42,6 +42,7 @@ const expr_value_1 = require("../types/expr-value");
 const expression_def_1 = require("../types/expression-def");
 const malloy_element_1 = require("../types/malloy-element");
 const TDU = __importStar(require("../typedesc-utils"));
+const boolean_value_1 = require("./boolean-value");
 function typeCoalesce(ev1, ev2) {
     return ev1 === undefined || ev1.type === 'null' || ev1.type === 'error'
         ? ev2
@@ -68,6 +69,9 @@ class Case extends expression_def_1.ExpressionDef {
         let value = undefined;
         if (this.value) {
             const v = this.value.getExpression(fs);
+            const noSubject = (0, boolean_value_1.booleanValueError)(fs, v, 'as the subject of a case', this.value, 'write the conditions, case when … then …');
+            if (noSubject)
+                return noSubject;
             dependents.push(v);
             resultExpr.kids.caseValue = v.value;
             value = v;
@@ -118,6 +122,9 @@ class Case extends expression_def_1.ExpressionDef {
             dependents.push(elseValue);
             resultExpr.kids.caseElse = elseValue.value;
         }
+        const noCase = (0, boolean_value_1.booleanValueError)(fs, returnType, 'as the result of a case', this, boolean_value_1.CHOICE_HINT);
+        if (noCase)
+            return noCase;
         return (0, expr_value_1.computedExprValue)({
             value: resultExpr,
             dataType: returnType ? TDU.atomicDef(returnType) : { type: 'null' },

package/dist/lang/ast/expressions/expr-alternation-tree.js

@@ -8,6 +8,7 @@ exports.ExprAlternationTree = void 0;
 const expr_value_1 = require("../types/expr-value");
 const expression_def_1 = require("../types/expression-def");
 const binary_operators_1 = require("../types/binary_operators");
+const boolean_value_1 = require("./boolean-value");
 /**
  * Return a flattened version of an alternation tree, if the tree is
  * composed entirely values and "or"s.
@@ -58,6 +59,16 @@ class ExprAlternationTree extends expression_def_1.ExpressionDef {
             if (inList.length > 0 && (applyOp === '=' || applyOp === '!=')) {
                 const isIn = expr.getExpression(fs);
                 const values = inList.map(v => v.getExpression(fs));
+                // A boolean on either side of the comparison, the tested value or
+                // any of the choices, is a boolean compared with something.
+                for (const [el, val] of [
+                    [expr, isIn],
+                    ...inList.map((el, i) => [el, values[i]]),
+                ]) {
+                    const noIn = (0, boolean_value_1.booleanValueError)(fs, val, `compared with ${applyOp}`, el, boolean_value_1.COMPARE_HINT);
+                    if (noIn)
+                        return noIn;
+                }
                 return (0, expr_value_1.computedExprValue)({
                     dataType: { type: 'boolean' },
                     value: {

package/dist/lang/ast/expressions/expr-cast.js

@@ -8,6 +8,7 @@ exports.ExprCast = void 0;
 const time_utils_1 = require("../time-utils");
 const expr_value_1 = require("../types/expr-value");
 const expression_def_1 = require("../types/expression-def");
+const boolean_value_1 = require("./boolean-value");
 class ExprCast extends expression_def_1.ExpressionDef {
     constructor(expr, castType, safe = false) {
         super({ expr: expr });
@@ -38,6 +39,10 @@ class ExprCast extends expression_def_1.ExpressionDef {
                 }
             }
         }
+        const noCast = (0, boolean_value_1.booleanValueError)(fs, expr, 'in a cast', this, boolean_value_1.NUMBER_HINT) ||
+            (0, boolean_value_1.booleanValueError)(fs, dataType, 'as the type of a cast', this, 'write a comparison');
+        if (noCast)
+            return noCast;
         return (0, expr_value_1.computedExprValue)({
             dataType,
             value: (0, time_utils_1.castTo)(this.castType, expr.value, expr.type, this.safe),

package/dist/lang/ast/expressions/expr-coalesce.js

@@ -42,6 +42,7 @@ const model_1 = require("../../../model");
 const composite_source_utils_1 = require("../../composite-source-utils");
 const TDU = __importStar(require("../typedesc-utils"));
 const expression_def_1 = require("../types/expression-def");
+const boolean_value_1 = require("./boolean-value");
 class ExprCoalesce extends expression_def_1.ExpressionDef {
     constructor(expr, altExpr) {
         super({ expr, altExpr });
@@ -53,6 +54,9 @@ class ExprCoalesce extends expression_def_1.ExpressionDef {
     computeExpression(fs) {
         const maybeNull = this.expr.getExpression(fs);
         const whenNull = this.altExpr.getExpression(fs);
+        const noCoalesce = (0, boolean_value_1.booleanValueError)(fs, maybeNull, 'in ??', this, boolean_value_1.CONDITION_HINT);
+        if (noCoalesce)
+            return noCoalesce;
         if (maybeNull.type === 'null') {
             return whenNull;
         }

package/dist/lang/ast/expressions/expr-func.js

@@ -48,6 +48,7 @@ const field_space_1 = require("../types/field-space");
 const utils_2 = require("../../../model/utils");
 const TDU = __importStar(require("../typedesc-utils"));
 const composite_source_utils_1 = require("../../composite-source-utils");
+const boolean_value_1 = require("./boolean-value");
 // Built-in functions that take a string literal of user-supplied SQL
 // and emit it directly. Gated by experimental.sql_functions in plain
 // Malloy; rejected unconditionally in restricted queries because they
@@ -111,9 +112,14 @@ class ExprFunc extends expression_def_1.ExpressionDef {
         return { found: func, error: undefined };
     }
     getPropsExpression(fs, props) {
-        var _a, _b, _c, _d, _e;
+        var _a, _b, _c, _d, _e, _f;
         const argExprsWithoutImplicit = this.args.map(arg => arg.getExpression(fs));
         if (this.isRaw) {
+            for (const [i, argExpr] of argExprsWithoutImplicit.entries()) {
+                const noArg = (0, boolean_value_1.booleanValueError)(fs, argExpr, `as an argument of ${this.name}`, this.args[i], boolean_value_1.NUMBER_HINT);
+                if (noArg)
+                    return noArg;
+            }
             const funcCall = [`${this.name}(`];
             argExprsWithoutImplicit.forEach((expr, i) => {
                 if (i !== 0) {
@@ -182,6 +188,11 @@ class ExprFunc extends expression_def_1.ExpressionDef {
                 .join(', ')})`);
         }
         const { overload, expressionTypeErrors, evalSpaceErrors, nullabilityErrors, returnType, } = result;
+        for (const [i, argExpr] of argExprs.entries()) {
+            const noArg = (0, boolean_value_1.booleanValueError)(fs, argExpr, `as an argument of ${this.name}`, (_e = this.args[i - (implicitExpr ? 1 : 0)]) !== null && _e !== void 0 ? _e : this, boolean_value_1.NUMBER_HINT);
+            if (noArg)
+                return noArg;
+        }
         // Report errors for expression type mismatch
         for (const error of expressionTypeErrors) {
             const adjustedIndex = error.argIndex - (implicitExpr ? 1 : 0);
@@ -214,7 +225,7 @@ class ExprFunc extends expression_def_1.ExpressionDef {
         }
         const type = overload.returnType;
         const expressionType = (0, malloy_types_1.maxOfExpressionTypes)([
-            (_e = type.expressionType) !== null && _e !== void 0 ? _e : 'scalar',
+            (_f = type.expressionType) !== null && _f !== void 0 ? _f : 'scalar',
             ...argExprs.map(e => e.expressionType),
         ]);
         if (!(0, malloy_types_1.expressionIsAggregate)(overload.returnType.expressionType) &&

package/dist/lang/ast/expressions/expr-minus.js

@@ -38,7 +38,6 @@ var __importStar = (this && this.__importStar) || (function () {
 })();
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.ExprMinus = void 0;
-const ast_utils_1 = require("../ast-utils");
 const TDU = __importStar(require("../typedesc-utils"));
 const expression_def_1 = require("../types/expression-def");
 class ExprMinus extends expression_def_1.ExpressionDef {
@@ -50,6 +49,10 @@ class ExprMinus extends expression_def_1.ExpressionDef {
     }
     computeExpression(fs) {
         const expr = this.expr.getExpression(fs);
+        // An error is already logged; it stays an error so nothing else logs it
+        if (expr.type === 'error') {
+            return expr;
+        }
         if (TDU.typeIn(expr, this.legalChildTypes)) {
             return {
                 ...expr,
@@ -57,7 +60,7 @@ class ExprMinus extends expression_def_1.ExpressionDef {
                 value: { node: 'unary-', e: expr.value },
             };
         }
-        return (0, ast_utils_1.errorFor)('negate requires number');
+        return this.expr.loggedErrorExpr('expression-type-error', `'${this.elementType}' Can't use type ${TDU.inspect(expr)}`);
     }
 }
 exports.ExprMinus = ExprMinus;

package/dist/lang/ast/expressions/expr-null.js

@@ -7,11 +7,15 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.ExprIsNull = exports.PartialIsNull = exports.ExprNULL = void 0;
 const expr_value_1 = require("../types/expr-value");
 const expression_def_1 = require("../types/expression-def");
+const boolean_value_1 = require("./boolean-value");
 function doIsNull(fs, op, expr) {
     // Build a new value rather than retyping the operand's in place: what
     // getExpression hands back is shared, and the operand is usually asked for
     // its value again after this.
     const nullCmp = expr.getExpression(fs);
+    const noTest = (0, boolean_value_1.booleanValueError)(fs, nullCmp, 'in a null test', expr, boolean_value_1.CONDITION_HINT);
+    if (noTest)
+        return noTest;
     return (0, expr_value_1.computedExprValue)({
         dataType: { type: 'boolean' },
         value: {

package/dist/lang/ast/expressions/pick-when.js

@@ -42,6 +42,7 @@ const TDU = __importStar(require("../typedesc-utils"));
 const expr_value_1 = require("../types/expr-value");
 const expression_def_1 = require("../types/expression-def");
 const malloy_element_1 = require("../types/malloy-element");
+const boolean_value_1 = require("./boolean-value");
 function typeCoalesce(ev1, ev2) {
     if (ev1 === undefined || ev1.type === 'null' || ev1.type === 'error') {
         return ev2;
@@ -131,6 +132,9 @@ class Pick extends expression_def_1.ExpressionDef {
         if (definedReturnType.type === 'filter expression') {
             return this.loggedErrorExpr('filter-expression-error', 'Pick statments cannot have filter expression values');
         }
+        const noPick = (0, boolean_value_1.booleanValueError)(fs, definedReturnType, 'as the result of a pick', this, boolean_value_1.CHOICE_HINT);
+        if (noPick)
+            return noPick;
         caseValue.kids.caseElse = defaultVal.value;
         return (0, expr_value_1.computedExprValue)({
             dataType: definedReturnType,
@@ -193,6 +197,9 @@ class Pick extends expression_def_1.ExpressionDef {
         if (definedReturnType.type === 'filter expression') {
             return this.loggedErrorExpr('filter-expression-error', 'Pick statments cannot have filter expression values');
         }
+        const noPick = (0, boolean_value_1.booleanValueError)(fs, definedReturnType, 'as the result of a pick', this, boolean_value_1.CHOICE_HINT);
+        if (noPick)
+            return noPick;
         pick.kids.caseElse = defVal.value;
         return (0, expr_value_1.computedExprValue)({
             dataType: definedReturnType,

package/dist/lang/ast/field-space/index-field-space.js

@@ -8,6 +8,7 @@ exports.IndexFieldSpace = void 0;
 const composite_source_utils_1 = require("../../composite-source-utils");
 const malloy_types_1 = require("../../../model/malloy_types");
 const error_factory_1 = require("../error-factory");
+const boolean_value_1 = require("../expressions/boolean-value");
 const field_references_1 = require("../query-items/field-references");
 const space_field_1 = require("../types/space-field");
 const query_spaces_1 = require("./query-spaces");
@@ -98,10 +99,21 @@ class IndexFieldSpace extends query_spaces_1.QueryOperationSpace {
             }
             else {
                 const eTypeDesc = entry.typeDesc();
+                if (
                 // Don't index arrays and records
-                if (malloy_types_1.TD.isBasicAtomic(eTypeDesc) &&
+                malloy_types_1.TD.isBasicAtomic(eTypeDesc) &&
                     (0, malloy_types_1.expressionIsScalar)(eTypeDesc.expressionType) &&
                     (dialect === undefined || !dialect.ignoreInProject(name))) {
+                    if (eTypeDesc.type === 'boolean' && (dialect === null || dialect === void 0 ? void 0 : dialect.booleanType) === 'none') {
+                        const path = [...joinPath, name].join('.');
+                        wild.logWarning('wildcard-omits-boolean', {
+                            dialect: dialect.name,
+                            wildcard: wild.refString,
+                            name: path,
+                            hint: (0, boolean_value_1.resultColumnHint)(path),
+                        });
+                        continue;
+                    }
                     expandEntries.push({ name: indexName, entry });
                     this.expandedWild.set(indexName, {
                         path: joinPath.concat(name),

package/dist/lang/ast/field-space/query-spaces.js

@@ -48,6 +48,7 @@ const static_space_1 = require("./static-space");
 const query_input_space_1 = require("./query-input-space");
 const composite_source_utils_1 = require("../../composite-source-utils");
 const error_factory_1 = require("../error-factory");
+const boolean_value_1 = require("../expressions/boolean-value");
 const reference_field_1 = require("./reference-field");
 const refine_from_space_field_1 = require("./refine-from-space-field");
 /**
@@ -158,6 +159,16 @@ class QueryOperationSpace extends refined_space_1.RefinedSpace {
                 if (model.TD.isAtomic(eType) &&
                     model.expressionIsScalar(eType.expressionType) &&
                     (dialect === undefined || !dialect.ignoreInProject(name))) {
+                    if (eType.type === 'boolean' && (dialect === null || dialect === void 0 ? void 0 : dialect.booleanType) === 'none') {
+                        const path = joinPath.concat(name).join('.');
+                        wild.logWarning('wildcard-omits-boolean', {
+                            dialect: dialect.name,
+                            wildcard: wild.refString,
+                            name: path,
+                            hint: (0, boolean_value_1.resultColumnHint)(path),
+                        });
+                        continue;
+                    }
                     expandEntries.push({ name, entry });
                     this.expandedWild.set(name, {
                         path: joinPath.concat(name),

package/dist/lang/ast/query-builders/reduce-builder.js

@@ -55,7 +55,7 @@ class QuerySegmentBuilder {
     }
     refineFrom(from, to) {
         if (from && from.type !== 'index' && from.type !== 'raw') {
-            if (!this.limit && from.orderBy && !from.defaultOrderBy) {
+            if (!this.order && from.orderBy && !from.defaultOrderBy) {
                 to.orderBy = from.orderBy;
             }
             if (!this.limit && from.limit) {

package/dist/lang/ast/query-items/field-declaration.d.ts

@@ -19,24 +19,29 @@ export declare abstract class AtomicFieldDeclaration extends MalloyElement imple
     fieldDef(fs: FieldSpace, exprName: string): FieldDef;
     abstract typecheckExprValue(expr: ExprValue): void;
     executesInOutputSpace(): boolean;
+    outputsValue(): boolean;
     queryFieldDef(exprFS: FieldSpace, exprName: string): AtomicFieldDef;
     makeEntry(fs: DynamicSpace): void;
 }
 export declare class CalculateFieldDeclaration extends AtomicFieldDeclaration {
     elementType: string;
+    outputsValue(): boolean;
     typecheckExprValue(expr: ExprValue): void;
     executesInOutputSpace(): boolean;
 }
 export declare class AggregateFieldDeclaration extends AtomicFieldDeclaration {
     elementType: string;
+    outputsValue(): boolean;
     typecheckExprValue(expr: ExprValue): void;
 }
 export declare class GroupByFieldDeclaration extends AtomicFieldDeclaration {
     elementType: string;
+    outputsValue(): boolean;
     typecheckExprValue(expr: ExprValue): void;
 }
 export declare class ProjectFieldDeclaration extends AtomicFieldDeclaration {
     elementType: string;
+    outputsValue(): boolean;
     typecheckExprValue(expr: ExprValue): void;
 }
 export declare class DeclareFieldDeclaration extends AtomicFieldDeclaration {

package/dist/lang/ast/query-items/field-declaration.js

@@ -45,6 +45,7 @@ const malloy_element_1 = require("../types/malloy-element");
 const typecheck_utils_1 = require("./typecheck_utils");
 const space_field_1 = require("../types/space-field");
 const def_space_1 = require("../field-space/def-space");
+const boolean_value_1 = require("../expressions/boolean-value");
 class AtomicFieldDeclaration extends malloy_element_1.MalloyElement {
     constructor(expr, defineName, exprSrc) {
         super({ expr: expr });
@@ -71,6 +72,10 @@ class AtomicFieldDeclaration extends malloy_element_1.MalloyElement {
     executesInOutputSpace() {
         return false;
     }
+    // The declaration is a result column of a query
+    outputsValue() {
+        return false;
+    }
     queryFieldDef(exprFS, exprName) {
         var _a;
         let exprValue;
@@ -105,27 +110,16 @@ class AtomicFieldDeclaration extends malloy_element_1.MalloyElement {
         }
         if ((0, malloy_types_1.isAtomicFieldType)(exprValue.type) && exprValue.type !== 'error') {
             this.typecheckExprValue(exprValue);
+            if (this.outputsValue() &&
+                (0, boolean_value_1.booleanValueError)(exprFS, exprValue, 'as a result column', this.expr, (0, boolean_value_1.resultColumnHint)((_a = this.exprSrc) !== null && _a !== void 0 ? _a : '…'))) {
+                return { name: exprName, type: 'error' };
+            }
             const ret = (0, malloy_types_1.mkFieldDef)(TDU.atomicDef(exprValue), exprName);
             if (malloy_types_1.TD.isTemporal(ret) && (0, granular_result_1.isGranularResult)(exprValue)) {
                 ret.timeframe = exprValue.timeframe;
             }
             ret.location = this.location;
-            if (exprValue.type === 'boolean' &&
-                ((_a = exprFS.dialectObj()) === null || _a === void 0 ? void 0 : _a.booleanType) === 'none') {
-                // when generating a boolean field on a database without boolean support
-                // map it to integers
-                ret.e = {
-                    node: 'case',
-                    kids: {
-                        caseWhen: [exprValue.value],
-                        caseThen: [{ node: 'numberLiteral', literal: '1' }],
-                        caseElse: { node: 'numberLiteral', literal: '0' },
-                    },
-                };
-            }
-            else {
-                ret.e = exprValue.value;
-            }
+            ret.e = exprValue.value;
             ret.drillExpression = this.expr.drillExpression();
             ret.refSummary = exprValue.refSummary;
             ret.ungroupings = exprValue.ungroupings;
@@ -163,6 +157,9 @@ class CalculateFieldDeclaration extends AtomicFieldDeclaration {
         super(...arguments);
         this.elementType = 'calculateFieldDeclaration';
     }
+    outputsValue() {
+        return true;
+    }
     typecheckExprValue(expr) {
         (0, typecheck_utils_1.typecheckCalculate)(expr, this);
     }
@@ -176,6 +173,9 @@ class AggregateFieldDeclaration extends AtomicFieldDeclaration {
         super(...arguments);
         this.elementType = 'aggregateFieldDeclaration';
     }
+    outputsValue() {
+        return true;
+    }
     typecheckExprValue(expr) {
         (0, typecheck_utils_1.typecheckAggregate)(expr, this);
     }
@@ -186,6 +186,9 @@ class GroupByFieldDeclaration extends AtomicFieldDeclaration {
         super(...arguments);
         this.elementType = 'groupByFieldDeclaration';
     }
+    outputsValue() {
+        return true;
+    }
     typecheckExprValue(expr) {
         (0, typecheck_utils_1.typecheckGroupBy)(expr, this);
     }
@@ -196,6 +199,9 @@ class ProjectFieldDeclaration extends AtomicFieldDeclaration {
         super(...arguments);
         this.elementType = 'projectFieldDeclaration';
     }
+    outputsValue() {
+        return true;
+    }
     typecheckExprValue(expr) {
         (0, typecheck_utils_1.typecheckProject)(expr, this);
     }

package/dist/lang/ast/query-items/field-references.d.ts

@@ -25,6 +25,7 @@ export declare abstract class FieldReference extends ListOf<FieldName> implement
     get sourceString(): string | undefined;
     get nameString(): string;
     abstract typecheck(type: TypeDesc): void;
+    outputsValue(): boolean;
     getField(fs: FieldSpace): LookupResult;
 }
 export declare class AcceptExceptFieldReference extends FieldReference {
@@ -61,27 +62,32 @@ export declare class DrillFieldReference extends FieldReference {
 }
 export declare class CalculateFieldReference extends FieldReference {
     elementType: string;
+    outputsValue(): boolean;
     protected autoRenameDeclarationCtor(): typeof CalculateFieldDeclaration;
     typecheck(type: TypeDesc): void;
 }
 export declare class IndexFieldReference extends FieldReference {
     elementType: string;
+    outputsValue(): boolean;
     typecheck(type: TypeDesc): void;
     static indexOutputName(path: string[]): string;
     get outputName(): string;
 }
 export declare class AggregateFieldReference extends FieldReference {
     elementType: string;
+    outputsValue(): boolean;
     protected autoRenameDeclarationCtor(): typeof AggregateFieldDeclaration;
     typecheck(type: TypeDesc): void;
 }
 export declare class GroupByFieldReference extends FieldReference {
     elementType: string;
+    outputsValue(): boolean;
     protected autoRenameDeclarationCtor(): typeof GroupByFieldDeclaration;
     typecheck(type: TypeDesc): void;
 }
 export declare class ProjectFieldReference extends FieldReference {
     elementType: string;
+    outputsValue(): boolean;
     protected autoRenameDeclarationCtor(): typeof ProjectFieldDeclaration;
     typecheck(type: TypeDesc): void;
 }

package/dist/lang/ast/query-items/field-references.js

@@ -10,6 +10,7 @@ const definition_list_1 = require("../types/definition-list");
 const field_space_1 = require("../types/field-space");
 const malloy_element_1 = require("../types/malloy-element");
 const expr_id_reference_1 = require("../expressions/expr-id-reference");
+const boolean_value_1 = require("../expressions/boolean-value");
 const field_declaration_1 = require("./field-declaration");
 const typecheck_utils_1 = require("./typecheck_utils");
 class FieldReference extends malloy_element_1.ListOf {
@@ -108,11 +109,18 @@ class FieldReference extends malloy_element_1.ListOf {
     get nameString() {
         return this.list[this.list.length - 1].refString;
     }
+    // The reference is a result column of a query
+    outputsValue() {
+        return false;
+    }
     getField(fs) {
         const result = fs.lookup(this.list);
         if (result.found) {
             const actualType = result.found.typeDesc();
             this.typecheck(actualType);
+            if (this.outputsValue()) {
+                (0, boolean_value_1.booleanValueError)(fs, actualType, 'as a result column', this, (0, boolean_value_1.resultColumnHint)(this.refString));
+            }
         }
         return result;
     }
@@ -210,6 +218,9 @@ class CalculateFieldReference extends FieldReference {
         super(...arguments);
         this.elementType = 'calculateFieldReference';
     }
+    outputsValue() {
+        return true;
+    }
     autoRenameDeclarationCtor() {
         return field_declaration_1.CalculateFieldDeclaration;
     }
@@ -223,6 +234,9 @@ class IndexFieldReference extends FieldReference {
         super(...arguments);
         this.elementType = 'indexFieldReference';
     }
+    outputsValue() {
+        return true;
+    }
     typecheck(type) {
         (0, typecheck_utils_1.typecheckIndex)(type, this);
     }
@@ -240,6 +254,9 @@ class AggregateFieldReference extends FieldReference {
         super(...arguments);
         this.elementType = 'aggregateFieldReference';
     }
+    outputsValue() {
+        return true;
+    }
     autoRenameDeclarationCtor() {
         return field_declaration_1.AggregateFieldDeclaration;
     }
@@ -253,6 +270,9 @@ class GroupByFieldReference extends FieldReference {
         super(...arguments);
         this.elementType = 'groupByFieldReference';
     }
+    outputsValue() {
+        return true;
+    }
     autoRenameDeclarationCtor() {
         return field_declaration_1.GroupByFieldDeclaration;
     }
@@ -266,6 +286,9 @@ class ProjectFieldReference extends FieldReference {
         super(...arguments);
         this.elementType = 'projectFieldReference';
     }
+    outputsValue() {
+        return true;
+    }
     autoRenameDeclarationCtor() {
         return field_declaration_1.ProjectFieldDeclaration;
     }

package/dist/lang/ast/query-properties/filters.js

@@ -50,7 +50,9 @@ class FilterElement extends malloy_element_1.MalloyElement {
     filterCondition(fs) {
         const exprVal = this.expr.getExpression(fs);
         if (exprVal.type !== 'boolean') {
-            this.expr.logError('non-boolean-filter', 'Filter expression must have boolean value');
+            if (exprVal.type !== 'error') {
+                this.expr.logError('non-boolean-filter', 'Filter expression must have boolean value');
+            }
             return {
                 node: 'filterCondition',
                 code: this.exprSrc,

package/dist/lang/ast/source-elements/named-source.js

@@ -52,7 +52,6 @@ class NamedSource extends source_1.Source {
         }
     }
     modelStruct() {
-        var _a;
         const modelEnt = this.modelEntry(this.ref);
         const entry = modelEnt === null || modelEnt === void 0 ? void 0 : modelEnt.entry;
         if (!entry) {
@@ -89,7 +88,6 @@ class NamedSource extends source_1.Source {
             return;
         }
         else {
-            (_a = this.document()) === null || _a === void 0 ? void 0 : _a.checkExperimentalDialect(this, entry.dialect);
             if ((0, malloy_types_1.isSourceDef)(entry)) {
                 // This struct is created as an unmodified reference to `entry`. Mark it
                 // with entry's own identity; the modification path (DynamicSpace) clears