Chart builder: fold render diagnostic behind a disclosure

src/app/components/ChartBuilderModal.tsx

@@ -737,7 +737,10 @@ function BuilderPreview() {
   const hostRef = useRef<HTMLDivElement>(null);
   const handleRef = useRef<RenderHandle | null>(null);
   const generationRef = useRef(0);
-  const [error, setError] = useState<string | null>(null);
+  // The error contract (docs/architecture/10): a plain headline that names the
+  // problem and points at the next step; any raw Vega diagnostic goes in `detail`,
+  // shown behind a disclosure rather than in the headline.
+  const [error, setError] = useState<{ message: string; detail?: string } | null>(null);
   // Set when the chart resolves larger than the canvas backend can draw — a
   // physical render-size limit, distinct from the readability cardinality warnings.
   const [tooLarge, setTooLarge] = useState<{ heightPx: number; limitPx: number } | null>(null);
@@ -809,16 +812,18 @@ function BuilderPreview() {
             setTooLarge({ heightPx: e.heightPx, limitPx: e.limitPx });
             setError(null);
           } else if (e instanceof DatasetNotFoundError) {
-            // TODO: this drops the next-step the error contract wants (arch 10); LivePreview
-            // gives "Create it from Datasets…". Near-unreachable here (the builder opens from
-            // an existing dataset), so it's terse — restore the next-step if it can be reached.
-            setError(`Dataset "${e.datasetName}" not found.`);
+            // Near-unreachable (the builder opens from an existing dataset), but if the
+            // backing dataset is deleted mid-session the contract still wants the next step.
+            setError({
+              message: `Dataset "${e.datasetName}" not found — recreate it from Datasets, then reopen the builder.`,
+            });
             setTooLarge(null);
           } else {
-            // TODO: arch 10 routes a raw diagnostic into a disclosure, not the headline. The
-            // editor surfaces the Vega message inline by design; the builder could fold it
-            // behind a details disclosure and keep the headline plain.
-            setError(`Couldn't render this chart: ${(e as Error).message}`);
+            // Plain headline; the raw Vega message folds into the disclosure below.
+            setError({
+              message: "Couldn't render this chart.",
+              detail: (e as Error).message,
+            });
             setTooLarge(null);
           }
         }
@@ -853,9 +858,15 @@ function BuilderPreview() {
         <div className={styles.previewHost} ref={hostRef} />
       </div>
       {valid && tooLarge === null && error !== null && (
-        <pre className={styles.previewError} role="alert">
-          {error}
-        </pre>
+        <div className={styles.previewError} role="alert">
+          <p className={styles.previewErrorHeadline}>{error.message}</p>
+          {error.detail !== undefined && (
+            <details className={styles.previewErrorDetails}>
+              <summary className={styles.previewErrorSummary}>Technical details</summary>
+              <pre className={styles.previewErrorDetail}>{error.detail}</pre>
+            </details>
+          )}
+        </div>
       )}
     </div>
   );