@malloy-publisher/server 0.0.214 → 0.0.215

package/dist/server.mjs

@@ -248857,8 +248857,12 @@ class MaterializationService {
   }
   async executeInstructedBuild(compiled, instructions, seedEntries, signal) {
     const { graphs, sources, connectionDigests, connections } = compiled;
+    const bySourceID = new Map;
     const byBuildId = new Map;
     for (const instruction of instructions) {
+      if (instruction.sourceID) {
+        bySourceID.set(instruction.sourceID, instruction);
+      }
       byBuildId.set(instruction.buildId, instruction);
     }
     const manifest = new Manifest;
@@ -248878,7 +248882,7 @@ class MaterializationService {
         if (signal.aborted)
           throw new Error("Build cancelled");
         const buildId = computeBuildId(persistSource, connectionDigests);
-        const instruction = byBuildId.get(buildId);
+        const instruction = bySourceID.get(persistSource.sourceID) ?? byBuildId.get(buildId);
         if (!instruction)
           continue;
         const entry = await this.buildOneSource(persistSource, instruction, connection, connectionDigests, manifest);

package/package.json

@@ -1,7 +1,7 @@
 {
    "name": "@malloy-publisher/server",
    "description": "Malloy Publisher Server",
-   "version": "0.0.214",
+   "version": "0.0.215",
    "main": "dist/server.mjs",
    "bin": {
       "malloy-publisher": "dist/server.mjs"

package/src/service/materialization_service.ts

@@ -560,8 +560,20 @@ export class MaterializationService {
    ): Promise<Record<string, ManifestEntry>> {
       const { graphs, sources, connectionDigests, connections } = compiled;
 
+      // Index instructions by sourceID (the stable per-source handle) so the
+      // build no longer recomputes the buildId to find an instruction.
+      // Recomputing it here forced a caller's buildId to equal the publisher's
+      // content hash, so a caller that derives buildIds by any other scheme
+      // would have its sources silently skipped (the recomputed buildId would
+      // not match the instruction). buildId is treated as opaque, caller-assigned
+      // identity. A buildId index is kept as a fallback for instructions without
+      // a sourceID (e.g. standalone auto-run).
+      const bySourceID = new Map<string, BuildInstruction>();
       const byBuildId = new Map<string, BuildInstruction>();
       for (const instruction of instructions) {
+         if (instruction.sourceID) {
+            bySourceID.set(instruction.sourceID, instruction);
+         }
          byBuildId.set(instruction.buildId, instruction);
       }
 
@@ -587,8 +599,15 @@ export class MaterializationService {
          for (const persistSource of iterGraphSources(graph, sources)) {
             if (signal.aborted) throw new Error("Build cancelled");
 
+            // The manifest is keyed by the content buildId — what Malloy
+            // recomputes to resolve upstream persist references during SQL
+            // generation — independent of the instruction's identity buildId.
             const buildId = computeBuildId(persistSource, connectionDigests);
-            const instruction = byBuildId.get(buildId);
+            // Prefer sourceID matching (so the caller's buildId scheme stays
+            // opaque to the build); fall back to buildId for instructions
+            // without a sourceID (auto-run).
+            const instruction =
+               bySourceID.get(persistSource.sourceID) ?? byBuildId.get(buildId);
             if (!instruction) continue;
 
             const entry = await this.buildOneSource(