@malloy-publisher/server 0.9.10 → 0.9.11

package/dist/app/index.html

@@ -16,7 +16,7 @@ SPDX-License-Identifier: MIT
          href="https://fonts.googleapis.com/css2?family=Inter:wght@400;500;600;700&family=JetBrains+Mono:wght@400;500&display=swap"
       />
       <title>Malloy Publisher</title>
-     <script type="module" crossorigin src="/assets/index-BvSbJtsT.js"></script>
+     <script type="module" crossorigin src="/assets/index-gQEuz0Cg.js"></script>
      <link rel="stylesheet" crossorigin href="/assets/index-DUqldpo0.css">
    </head>
    <body>

package/dist/package_load_worker.mjs

@@ -21077,7 +21077,7 @@ ${body}`;
   if (artifactAt < 0) {
     return refuse({
       line: 1,
-      message: "Line 1: this notebook has no ## artifact note, so its header cannot be told from its cells. Fix: add `## artifact { layout=linear }` after the ##! line."
+      message: "Line 1: this notebook has no ## artifact note, so its header cannot be told from its cells. Fix: add `## artifact { layout=linear tiles=[…] }` after the ##! line."
     });
   }
   const runCount = items.filter((item) => item.kind === "statement" && item.run).length;
@@ -21277,12 +21277,14 @@ function readReportTag(modelNotes) {
   const kind = tagText(artifact, "kind");
   if (kind !== undefined)
     return { status: "kind-removed", kind };
-  const layout = parseLayoutValue(tagText(artifact, "layout"));
-  if (layout)
-    return { status: "report", layout, decidedBy: "layout" };
-  if (artifact.has("tiles") || tags?.tag("dashboard") !== undefined)
-    return { status: "report", layout: "grid", decidedBy: "tiles" };
-  return { status: "report", layout: "linear", decidedBy: "default" };
+  const hasTiles = artifact.has("tiles");
+  if (hasTiles && artifact.array("tiles") === undefined)
+    return { status: "unparsed" };
+  const declared = tagText(artifact, "layout");
+  const layout = parseLayoutValue(declared) ?? (hasTiles || tags?.tag("dashboard") !== undefined ? "grid" : "linear");
+  if (layout === "linear" && !hasTiles)
+    return declared !== undefined && parseLayoutValue(declared) === undefined ? { status: "cells-removed", unknownLayout: declared } : { status: "cells-removed" };
+  return { status: "report", layout };
 }
 function readReportTagOfText(text) {
   return readReportTag(modelNotesOfText(text));
@@ -23726,6 +23728,11 @@ function readGivenControlSpec(annotationTexts2) {
   }
   return spec;
 }
+function givenDeclaresNoDefault(given) {
+  const loose = given;
+  const parsed = "default" in loose ? loose.default : loose._internal?.default;
+  return parsed === undefined && loose._internal?.defaultText === undefined;
+}
 function malloyGivenToApi(given) {
   const type = given.type;
   const renderedType = type.type === "filter expression" ? `filter<${type.filterType}>` : type.type === "array" && type.elementTypeDef?.type ? `${type.elementTypeDef.type}[]` : type.type;
@@ -23736,7 +23743,8 @@ function malloyGivenToApi(given) {
     annotations: allNotes.filter((note) => !isReservedRoute(note.route) && note.route !== MARKDOWN_ROUTE).map((note) => note.text),
     ...readGivenControlSpec(allNotes.map((note) => note.text)),
     ...allNotes.some((note) => note.route === "secure") && { secure: true },
-    default: given._internal?.defaultText
+    default: given._internal?.defaultText,
+    ...givenDeclaresNoDefault(given) && { required: true }
   };
 }
 function collectGivenRefs(value, into) {
@@ -26381,7 +26389,13 @@ async function prepareForMainThread(job, malloyConfig, load, model) {
   const pastDeadline = () => performance.now() >= load.softDeadline;
   const { modelPath, materializer, importBaseURL } = model;
   const renderTagResults = [];
-  for (const target of renderTagTargets(model.queries, model.sources)) {
+  const targets = renderTagTargets(model.queries, model.sources);
+  if (targets.length > 0 && !load.failed && !pastDeadline()) {
+    try {
+      model.model.queryModel;
+    } catch {}
+  }
+  for (const target of targets) {
     if (load.failed || pastDeadline())
       break;
     try {