@malloydata/malloy 0.0.420 → 0.0.422

package/dist/api/foundation/core.js

@@ -1300,11 +1300,16 @@ class PersistSource {
     getSQL(options) {
         const sd = this.persistableDef;
         const queryModel = this.model.queryModel;
+        // Compile with finalize=false so this SQL is the bare source SELECT.
+        // The build-time key (makeBuildId over this SQL) must equal the serve-time
+        // manifest lookup key, which query_query.ts recomputes from the same
+        // unfinalized SELECT; finalizing would diverge the two on dialects with a
+        // final stage (Postgres) and mis-materialize the table.
         if (sd.type === 'sql_select') {
-            return (0, model_1.getCompiledSQL)(sd, options !== null && options !== void 0 ? options : {}, (query, opts) => queryModel.compileQuery(query, opts).sql);
+            return (0, model_1.getCompiledSQL)(sd, options !== null && options !== void 0 ? options : {}, (query, opts) => queryModel.compileQuery(query, opts, false).sql);
         }
         else {
-            const compiled = queryModel.compileQuery(sd.query, options);
+            const compiled = queryModel.compileQuery(sd.query, options, false);
             return compiled.sql;
         }
     }

package/dist/lang/test/test-translator.d.ts

@@ -20,6 +20,7 @@ export declare function humanify(value: unknown): string;
 export declare const TEST_DIALECT = "duckdb";
 export declare const aTableDef: TableSourceDef;
 export declare const bqTableDef: SourceDef;
+export declare const pgTableDef: SourceDef;
 /**
  * A TestTranlator never actually talks to connection, instead uses
  * some mocked schema definitions.

package/dist/lang/test/test-translator.js

@@ -5,7 +5,7 @@
  * SPDX-License-Identifier: MIT
  */
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.BetaExpression = exports.TestTranslator = exports.TestChildTranslator = exports.mockSchema = exports.bqTableDef = exports.aTableDef = exports.TEST_DIALECT = void 0;
+exports.BetaExpression = exports.TestTranslator = exports.TestChildTranslator = exports.mockSchema = exports.pgTableDef = exports.bqTableDef = exports.aTableDef = exports.TEST_DIALECT = void 0;
 exports.pretty = pretty;
 exports.humanify = humanify;
 exports.getExplore = getExplore;
@@ -171,6 +171,17 @@ exports.bqTableDef = {
     connection: '_bq_',
     fields: baseFields,
 };
+// Postgres table definition — Postgres is the only dialect with
+// `hasFinalStage = true`, so it is the one that exercises the persist-source
+// finalize=false path (see PersistSource.getSQL).
+exports.pgTableDef = {
+    type: 'table',
+    name: 'aTable',
+    dialect: 'postgres',
+    tablePath: 'aTable',
+    connection: '_pg_',
+    fields: baseFields,
+};
 /**
  * A TestTranlator never actually talks to connection, instead uses
  * some mocked schema definitions.
@@ -178,6 +189,7 @@ exports.bqTableDef = {
 exports.mockSchema = [
     exports.aTableDef,
     exports.bqTableDef,
+    exports.pgTableDef,
     {
         type: 'table',
         name: 'carriers',
@@ -350,6 +362,7 @@ class TestTranslator extends parse_malloy_1.MalloyTranslator {
             contents: {
                 _db_: { type: 'connection', name: '_db_' },
                 _bq_: { type: 'connection', name: '_bq_' },
+                _pg_: { type: 'connection', name: '_pg_' },
                 a: { ...exports.aTableDef, primaryKey: 'astr', name: 'a' },
                 b: { ...exports.aTableDef, primaryKey: 'astr', name: 'b' },
                 bq_a: { ...exports.bqTableDef, primaryKey: 'astr', name: 'bq_a' },
@@ -405,6 +418,7 @@ class TestTranslator extends parse_malloy_1.MalloyTranslator {
         }
         this.connectionDialectZone.define('_db_', exports.TEST_DIALECT);
         this.connectionDialectZone.define('_bq_', 'standardsql');
+        this.connectionDialectZone.define('_pg_', 'postgres');
         for (const flag of (_e = options.compilerFlags) !== null && _e !== void 0 ? _e : []) {
             this.compilerFlagSrc.push(`##! ${flag}\n`);
         }

package/dist/version.d.ts

@@ -1 +1 @@
-export declare const MALLOY_VERSION = "0.0.420";
+export declare const MALLOY_VERSION = "0.0.422";

package/dist/version.js

@@ -2,5 +2,5 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.MALLOY_VERSION = void 0;
 // generated with 'generate-version-file' script; do not edit manually
-exports.MALLOY_VERSION = '0.0.420';
+exports.MALLOY_VERSION = '0.0.422';
 //# sourceMappingURL=version.js.map

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@malloydata/malloy",
-  "version": "0.0.420",
+  "version": "0.0.422",
   "license": "MIT",
   "exports": {
     ".": "./dist/index.js",
@@ -51,9 +51,9 @@
     "generate-version-file": "VERSION=$(npm pkg get version --workspaces=false | tr -d \\\")\necho \"// generated with 'generate-version-file' script; do not edit manually\\nexport const MALLOY_VERSION = '$VERSION';\" > src/version.ts"
   },
   "dependencies": {
-    "@malloydata/malloy-filter": "0.0.420",
-    "@malloydata/malloy-interfaces": "0.0.420",
-    "@malloydata/malloy-tag": "0.0.420",
+    "@malloydata/malloy-filter": "0.0.422",
+    "@malloydata/malloy-interfaces": "0.0.422",
+    "@malloydata/malloy-tag": "0.0.422",
     "@noble/hashes": "^1.8.0",
     "antlr4ts": "^0.5.0-alpha.4",
     "assert": "^2.0.0",