pandorga 1.4.5 → 1.4.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c08f73c874e192ee0b1bede450515781124405c6875afec2ac4f6500e3a1db46
-  data.tar.gz: 55c89932abd93336991692769b088ca9dc798d04c2f7615cd175f67f156f06d4
+  metadata.gz: b72923b7c78519c2e27d696f963cf147445dd0d8dd089f90a69cde8c2247acbd
+  data.tar.gz: 703d26b69d010bcd5e535eee0dfd64da725887439cd68edb7cd8da827acb69d8
 SHA512:
-  metadata.gz: '00088be394eb9440225561ca538fd7aef974688c9563d293627b5f3e7ee1f5058c80a6ffb70734d7eaec249a1b50ccf81df68a8dce557519730d666b5da1e3a3'
-  data.tar.gz: a3a0687866fe28126982d3564595fa6e2193034a2bd0f2e288ccf2e19d180765d5a4e2ca3d8ae92604a8d1c4eb92cf81136076ca292077ad9e6e14d92fbceb3b
+  metadata.gz: a974cbf723b43a543a0af0a489c74c08bac2bddd4b77275baf5e74c106fde22aa69ea4e0085f0a1a71ba9f25514db61d0e6fc96dd9f03f749645e38e93dfd704
+  data.tar.gz: 164b159223d96b978e251f271fc8aea4b6535ad3397205efa88860dd1211b31ace3661b3425d6540b72874f1cd8d4348f8480ae305eccce115d45756dd0bfb5b

data/CHANGELOG.md

@@ -1,5 +1,26 @@
 # Changelog
 
+## [1.4.7] — 2026-10-07
+
+### Fixed
+
+- Studio keeps the editor focused while the content-pipeline chip is spinning.
+  Status polls patch the toolbar chip instead of rebuilding the shell.
+- Home draft cards use 16:9 thumbnails and are 20% shorter on desktop and mobile.
+
+## [1.4.6] — 2026-10-07
+
+### Fixed
+
+- Pipeline config resolution no longer swallows `_config.yml` read failures as
+  a silent “unconfigured”; returns a clear message when
+  `pandorga.content.workflow` is missing
+- Studio chip surfaces pipeline-status API errors on boot, not only after Save
+
+### Added
+
+- Gate `test-studio-pipeline-config.mjs` for `parseContentWorkflowFromConfig`
+
 ## [1.4.5] — 2026-10-07
 
 ### Changed

data/assets/css/ledger/home.css

@@ -845,10 +845,9 @@ html[data-theme="light"] .home-hero-plate {
     border-color var(--motion-duration-fast) var(--motion-ease-default);
 }
 
-/* ~40% taller poster than the shared 16/9 plate so the dropped-sheet body
-   keeps room for the excerpt (four-up columns were clipping copy to a sliver). */
+/* Shared 16:9 plate. The previous 16/12.6 crop made these thumbs tall. */
 .home-draft-card .ledger-poster {
-  aspect-ratio: 16 / 12.6;
+  aspect-ratio: 16 / 9;
 }
 
 /* A handful of fixed degrees per grid position — not a JS random() — so the
@@ -875,8 +874,9 @@ html[data-theme="light"] .home-hero-plate {
 }
 
 .home-draft-card .ledger-card-body {
-  padding: var(--space-md);
-  gap: var(--space-xs);
+  /* 20% shorter stack than the previous dropped-sheet body, both breakpoints. */
+  padding: calc(var(--space-md) * 0.8);
+  gap: calc(var(--space-xs) * 0.8);
 }
 
 /* Up from 1rem. The four dropped sheets are the only place on the home page
@@ -903,19 +903,29 @@ html[data-theme="light"] .home-hero-plate {
      `flex: 1` gives the paragraph a definite block size from the flex
      allocation, so line-clamp/overflow clip against that height instead of
      growing past the ellipsis (the old flex:1 + clamp:2 bug that left text
-     after "…" and sliced mid-glyph at the card edge).
-     min-height ~40% above the old six-line band so short posters still leave
-     readable excerpt room. */
+     after "…" and sliced mid-glyph at the card edge). */
   flex: 1 1 auto;
-  min-height: calc(var(--type-subtitle) * var(--leading-body) * 8.4);
+  /* With the 16:9 plate, this band keeps the whole card about 20% shorter
+     than the old 16/12.6 poster and 8.4-line excerpt, on narrow and wide grids. */
+  min-height: calc(var(--type-subtitle) * var(--leading-body) * 5.33);
   height: 0;
   display: -webkit-box;
-  -webkit-line-clamp: 8;
-  line-clamp: 8;
+  -webkit-line-clamp: 5;
+  line-clamp: 5;
   -webkit-box-orient: vertical;
   overflow: hidden;
 }
 
+/* Four-up posters were the tall part of the old card. Give the excerpt a
+   little of that height back so the wide card is 20% shorter too, not more. */
+@media (min-width: 1024px) {
+  .home-draft-card .ledger-card-copy {
+    min-height: calc(var(--type-subtitle) * var(--leading-body) * 6.05);
+    -webkit-line-clamp: 6;
+    line-clamp: 6;
+  }
+}
+
 .home-draft-card .ledger-card-foot .ledger-cta {
   font-size: var(--type-cta);
 }

data/functions/api/studio/_lib/github.js

@@ -307,21 +307,40 @@ export async function resolveContentWorkflowConfig(env) {
 
   try {
     const file = await getFile(env, "_config.yml");
-    if (file?.content) {
-      const parsed = parseContentWorkflowFromConfig(file.content);
-      if (parsed.workflow) {
-        return {
-          workflow: parsed.workflow,
-          ref: parsed.workflowRef || envRef || "main",
-          source: "config",
-        };
-      }
+    if (!file?.content) {
+      return {
+        workflow: null,
+        ref: envRef || "main",
+        source: null,
+        error: "config_missing",
+        message: "_config.yml was not found on main.",
+      };
+    }
+    const parsed = parseContentWorkflowFromConfig(file.content);
+    if (parsed.workflow) {
+      return {
+        workflow: parsed.workflow,
+        ref: parsed.workflowRef || envRef || "main",
+        source: "config",
+      };
     }
-  } catch {
-    /* fall through to unconfigured */
+    return {
+      workflow: null,
+      ref: envRef || "main",
+      source: null,
+      error: "workflow_unset",
+      message:
+        "Set pandorga.content.workflow in _config.yml (e.g. content-pipeline.yml).",
+    };
+  } catch (err) {
+    return {
+      workflow: null,
+      ref: envRef || "main",
+      source: null,
+      error: err.code || "config_read_failed",
+      message: String(err.message || err),
+    };
   }
-
-  return { workflow: null, ref: envRef || "main", source: null };
 }
 
 /**
@@ -345,8 +364,11 @@ export async function getContentPipelineStatus(env, sha) {
       sha: commitSha,
       state: "unconfigured",
       run: null,
-      error: null,
-      source: null,
+      error: resolved.error || "workflow_unset",
+      source: resolved.source,
+      message:
+        resolved.message ||
+        "Set pandorga.content.workflow in _config.yml (e.g. content-pipeline.yml).",
     };
   }
   if (commitSha && !/^[0-9a-f]{7,40}$/.test(commitSha)) {

data/lib/pandorga/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Pandorga
-  VERSION = "1.4.5"
+  VERSION = "1.4.7"
 end

data/scripts/test/render-parity/blocks/project-specimen.json

@@ -32,7 +32,6 @@
     "article.home-project-specimen",
     "div.home-specimen-strip",
     "div.ledger-meta",
-    "span.ledger-status",
     "div.home-specimen-figure",
     "div.ledger-poster",
     "img",
@@ -45,13 +44,16 @@
     "span.home-specimen-open"
   ],
   "forbidden": [
-    "projects-plate"
+    "projects-plate",
+    "span.ledger-status"
+  ],
+  "excludes": [
+    ">Ongoing<"
   ],
   "contains": [
     ">#01<",
     ">Simulation<",
     ">2024<",
-    ">Ongoing<",
     ">ARCO<",
     ">Repository<",
     ">Article<",

data/scripts/test/render-parity/blocks/system-card.json

@@ -27,22 +27,27 @@
     "div.ledger-poster",
     "img",
     "div.ledger-meta",
-    "span.ledger-status",
     "h3.ledger-card-title",
     "a.ledger-card-stretch",
     "p.ledger-card-copy",
     "div.ledger-chip-row",
     "div.ledger-card-links"
   ],
+  "forbidden": [
+    "span.ledger-status"
+  ],
+  "excludes": [
+    ">Ongoing<"
+  ],
   "contains": [
     ">Repository<",
     ">Article<",
     ">Visit<",
     ">ARCO<",
-    ">Simulation"
+    ">Simulation<"
   ],
   "counts": {
-    "ledger-meta-sep": 1,
+    "ledger-meta-sep": 0,
     "ledger-cta": 3
   }
 }

data/scripts/test/test-studio-api-contract.rb

@@ -45,6 +45,76 @@ check!(REPO_ROOT.join("studio-app/src/main.js").read.include?("armPipelineWatch"
        "SPA must watch the content pipeline after save")
 check!(REPO_ROOT.join("studio-app/src/main.js").read.include?("refreshPipelineStatus"),
        "SPA must hydrate pipeline status on boot")
+
+# Pipeline polls used to call renderShell(), which wipes #app and remounts
+# TipTap. That drops editor focus for the whole run.
+def function_source(source, name)
+  re = /(?:async\s+)?function\s+#{Regexp.escape(name)}\s*\(/
+  m = source.match(re)
+  raise "missing function #{name}" unless m
+  i = m.end(0)
+  paren = 1
+  while i < source.length && paren.positive?
+    c = source[i]
+    if c == "'" || c == '"' || c == "`"
+      q = c
+      i += 1
+      while i < source.length && source[i] != q
+        i += source[i] == "\\" ? 2 : 1
+      end
+      i += 1
+    elsif c == "/" && source[i + 1] == "/"
+      nl = source.index("\n", i)
+      i = nl ? nl + 1 : source.length
+    else
+      paren += 1 if c == "("
+      paren -= 1 if c == ")"
+      i += 1
+    end
+  end
+  i += 1 while i < source.length && source[i] != "{"
+  raise "missing body #{name}" if i >= source.length
+  depth = 0
+  start = i
+  while i < source.length
+    c = source[i]
+    if c == "{"
+      depth += 1
+      i += 1
+    elsif c == "}"
+      depth -= 1
+      return source[start..i] if depth == 0
+      i += 1
+    elsif c == "'" || c == '"' || c == "`"
+      q = c
+      i += 1
+      while i < source.length && source[i] != q
+        i += source[i] == "\\" ? 2 : 1
+      end
+      i += 1
+    elsif c == "/" && source[i + 1] == "/"
+      nl = source.index("\n", i)
+      i = nl ? nl + 1 : source.length
+    else
+      i += 1
+    end
+  end
+  raise "unclosed function #{name}"
+end
+
+studio_main = REPO_ROOT.join("studio-app/src/main.js").read
+%w[pollPipelineStatus refreshPipelineStatus armPipelineWatch].each do |fn|
+  body = function_source(studio_main, fn)
+  check!(!body.include?("renderShell"),
+         "#{fn} must not rebuild the shell (that drops editor focus while the chip spins)")
+  check!(body.include?("syncPipelineChip"),
+         "#{fn} must patch the pipeline chip in place")
+end
+reexport = function_source(studio_main, "reexportLocalContent")
+check!(!reexport.include?("renderShell"),
+       "local export must not rebuild the shell while the button spins")
+check!(reexport.include?("syncLocalExportButton"),
+       "local export must patch the export button in place")
 check!(!REPO_ROOT.join("studio-app/src/main.js").read.match?(/action:\s*"delete"/),
        "toolbar must not include a Delete action")
 

data/scripts/test/test-studio-pipeline-config.mjs

@@ -0,0 +1,35 @@
+#!/usr/bin/env node
+/**
+ * parseContentWorkflowFromConfig — reads pandorga.content.workflow from _config.yml.
+ */
+import assert from "node:assert/strict";
+import { parseContentWorkflowFromConfig } from "../../functions/api/studio/_lib/github.js";
+
+const sample = `# site
+title: Example
+
+pandorga:
+  # comment above content
+  content:
+    backend: r2
+    base_url: https://example.invalid
+    # Actions workflow Studio watches after Save
+    workflow: content-pipeline.yml
+    workflow_ref: main
+  identity:
+    name: Example
+`;
+
+const parsed = parseContentWorkflowFromConfig(sample);
+assert.equal(parsed.workflow, "content-pipeline.yml");
+assert.equal(parsed.workflowRef, "main");
+
+const missing = parseContentWorkflowFromConfig("pandorga:\n  identity:\n    name: x\n");
+assert.equal(missing.workflow, null);
+
+const quoted = parseContentWorkflowFromConfig(
+  "pandorga:\n  content:\n    workflow: \"my-workflow.yml\"\n"
+);
+assert.equal(quoted.workflow, "my-workflow.yml");
+
+console.log("All Studio pipeline config parse checks passed.");

data/scripts/test/test-studio-pipeline-focus.mjs

@@ -0,0 +1,308 @@
+#!/usr/bin/env node
+/**
+ * Pipeline status updates must patch the toolbar chip. Rebuilding the shell
+ * drops editor focus for the whole run.
+ */
+import assert from "node:assert/strict";
+import fs from "node:fs";
+import vm from "node:vm";
+import { fileURLToPath } from "node:url";
+import path from "node:path";
+
+const root = path.resolve(path.dirname(fileURLToPath(import.meta.url)), "../..");
+const source = fs.readFileSync(path.join(root, "studio-app/src/main.js"), "utf8");
+
+function skipString(i) {
+  const q = source[i];
+  i += 1;
+  while (i < source.length && source[i] !== q) {
+    if (source[i] === "\\") i += 2;
+    else i += 1;
+  }
+  return i + 1;
+}
+
+function functionSource(name) {
+  const re = new RegExp(`(?:async\\s+)?function\\s+${name}\\s*\\(`);
+  const m = re.exec(source);
+  if (!m) throw new Error(`missing function ${name}`);
+  let i = m.index + m[0].length;
+  let paren = 1;
+  while (i < source.length && paren > 0) {
+    const c = source[i];
+    if (c === "'" || c === '"' || c === "`") i = skipString(i);
+    else if (c === "/" && source[i + 1] === "/") {
+      const nl = source.indexOf("\n", i);
+      i = nl === -1 ? source.length : nl + 1;
+    } else {
+      if (c === "(") paren += 1;
+      else if (c === ")") paren -= 1;
+      i += 1;
+    }
+  }
+  while (i < source.length && source[i] !== "{") i += 1;
+  if (i >= source.length) throw new Error(`missing body ${name}`);
+  let depth = 0;
+  while (i < source.length) {
+    const c = source[i];
+    if (c === "{") {
+      depth += 1;
+      i += 1;
+    } else if (c === "}") {
+      depth -= 1;
+      if (depth === 0) return source.slice(m.index, i + 1);
+      i += 1;
+    } else if (c === "'" || c === '"' || c === "`") {
+      i = skipString(i);
+    } else if (c === "/" && source[i + 1] === "/") {
+      const nl = source.indexOf("\n", i);
+      i = nl === -1 ? source.length : nl + 1;
+    } else {
+      i += 1;
+    }
+  }
+  throw new Error(`unclosed function ${name}`);
+}
+
+class MiniNode {
+  constructor(tag) {
+    this.nodeType = 1;
+    this.tagName = String(tag || "").toUpperCase();
+    this.childNodes = [];
+    this.parentNode = null;
+    this.attributes = new Map();
+    this.className = "";
+    this.disabled = false;
+    this.title = "";
+    this.value = "";
+    this.selectionStart = 0;
+    this.selectionEnd = 0;
+    this._text = "";
+    this.listeners = new Map();
+    const node = this;
+    this.classList = {
+      toggle(name, force) {
+        const parts = node.className.split(/\s+/).filter(Boolean);
+        const has = parts.includes(name);
+        const on = force === undefined ? !has : !!force;
+        const next = parts.filter((part) => part !== name);
+        if (on) next.push(name);
+        node.className = next.join(" ");
+        return on;
+      },
+    };
+  }
+
+  get textContent() {
+    if (this.childNodes.length === 0) return this._text;
+    return this.childNodes.map((child) => child.textContent).join("");
+  }
+
+  set textContent(value) {
+    this._text = value == null ? "" : String(value);
+    this.childNodes = [];
+  }
+
+  setAttribute(name, value) {
+    this.attributes.set(name, String(value));
+  }
+
+  getAttribute(name) {
+    return this.attributes.has(name) ? this.attributes.get(name) : null;
+  }
+
+  addEventListener(type, fn) {
+    this.listeners.set(type, fn);
+  }
+
+  append(...kids) {
+    for (const kid of kids) {
+      if (kid == null || kid === false) continue;
+      kid.parentNode = this;
+      this.childNodes.push(kid);
+    }
+  }
+
+  replaceWith(next) {
+    const parent = this.parentNode;
+    if (!parent) return;
+    const idx = parent.childNodes.indexOf(this);
+    if (idx < 0) return;
+    next.parentNode = parent;
+    parent.childNodes[idx] = next;
+    this.parentNode = null;
+    if (document.activeElement === this) document.activeElement = document.body;
+  }
+
+  querySelector(sel) {
+    return querySelector(this, sel);
+  }
+
+  focus() {
+    document.activeElement = this;
+  }
+
+  setSelectionRange(start, end) {
+    this.selectionStart = start;
+    this.selectionEnd = end;
+  }
+}
+
+function matches(node, sel) {
+  const attr = sel.match(/^\[([^=]+)='([^']*)'\]$/);
+  if (attr) return node.getAttribute(attr[1]) === attr[2];
+  if (sel.startsWith(".")) {
+    const name = sel.slice(1);
+    return node.className.split(/\s+/).includes(name);
+  }
+  return false;
+}
+
+function querySelector(root, sel) {
+  for (const child of root.childNodes || []) {
+    if (child.nodeType === 1 && matches(child, sel)) return child;
+    const found = querySelector(child, sel);
+    if (found) return found;
+  }
+  return null;
+}
+
+const document = {
+  activeElement: null,
+  body: null,
+  createElement(tag) {
+    return new MiniNode(tag);
+  },
+  querySelector(sel) {
+    return querySelector(document.body, sel);
+  },
+};
+document.body = new MiniNode("body");
+document.body.parentNode = document;
+
+const editor = new MiniNode("textarea");
+editor.value = "keep typing";
+editor.className = "body-editor";
+const chipHost = new MiniNode("div");
+document.body.append(chipHost, editor);
+
+const state = {
+  pipeline: {
+    state: "idle",
+    sha: "",
+    runUrl: "",
+    message: "",
+    pollGen: 0,
+  },
+  reexporting: false,
+  error: "",
+  errorDetail: "",
+  status: "",
+  file: "content/collections/articles/hello.md",
+  saving: false,
+  dirty: false,
+};
+
+let statusCalls = 0;
+const studioApi = {
+  async pipelineStatus() {
+    statusCalls += 1;
+    if (statusCalls < 3) {
+      return {
+        state: "running",
+        sha: "abc",
+        run: { html_url: "https://example.com/actions/runs/9" },
+      };
+    }
+    return {
+      state: "ok",
+      sha: "abc",
+      run: { html_url: "https://example.com/actions/runs/9" },
+    };
+  },
+};
+
+const context = {
+  state,
+  document,
+  studioApi,
+  sleep: async () => {},
+  console,
+  JSON,
+  String,
+  window: { open() {}, location: { hostname: "studio.example.com" } },
+};
+vm.createContext(context);
+vm.runInContext(
+  [
+    functionSource("el"),
+    functionSource("pipelineChipMeta"),
+    functionSource("pipelineChipViewKey"),
+    functionSource("renderPipelineChip"),
+    functionSource("syncPipelineChip"),
+    functionSource("applyPipelinePayload"),
+    functionSource("localExportPresentation"),
+    functionSource("syncLocalExportButton"),
+    functionSource("pollPipelineStatus"),
+  ].join("\n"),
+  context
+);
+
+chipHost.append(vm.runInContext("renderPipelineChip()", context));
+editor.focus();
+editor.setSelectionRange(5, 5);
+const editorAtFocus = document.activeElement;
+
+await vm.runInContext(
+  `(async () => {
+    state.pipeline.pollGen = 1;
+    state.pipeline.state = "awaiting_run";
+    syncPipelineChip();
+    await pollPipelineStatus(1, "abc");
+  })()`,
+  context
+);
+
+assert.equal(document.activeElement, editorAtFocus, "editor focus survives pipeline polls");
+assert.equal(editor.parentNode, document.body, "editor stays mounted");
+assert.equal(editor.value, "keep typing");
+assert.equal(editor.selectionStart, 5);
+assert.equal(state.pipeline.state, "ok");
+
+const chip = document.querySelector("[data-studio-action='pipeline-status']");
+assert.ok(chip, "pipeline chip remains");
+assert.match(chip.className, /pipeline-ok/);
+assert.equal(chip.querySelector(".btn-label").textContent, "Live");
+assert.equal(chip.querySelector(".studio-icon-spin"), null);
+assert.equal(chip.disabled, false);
+
+editor.focus();
+state.reexporting = true;
+state.pipeline.state = "syncing";
+vm.runInContext("syncLocalExportButton()", context);
+const exportBtn = document.querySelector("[data-studio-action='reexport']");
+assert.equal(exportBtn, null, "production shell has no local export button");
+
+const exportHost = new MiniNode("button");
+exportHost.setAttribute("data-studio-action", "reexport");
+exportHost.className = "btn btn-tool btn-reexport";
+const icon = new MiniNode("span");
+icon.className = "material-symbols-outlined";
+icon.textContent = "sync";
+const label = new MiniNode("span");
+label.className = "btn-label";
+label.textContent = "Export";
+exportHost.append(icon, label);
+document.body.append(exportHost);
+
+editor.focus();
+vm.runInContext("syncLocalExportButton()", context);
+assert.equal(document.activeElement, editor, "local export spin does not steal editor focus");
+assert.equal(exportHost.disabled, true);
+assert.match(exportHost.className, /pipeline-running/);
+assert.equal(icon.textContent, "progress_activity");
+assert.match(icon.className, /studio-icon-spin/);
+assert.equal(label.textContent, "Exporting…");
+assert.equal(editor.value, "keep typing");
+
+console.log("All Studio pipeline focus checks passed.");

data/scripts/validate.sh

@@ -68,6 +68,16 @@ for test in "${TESTS[@]}"; do
   fi
 done
 
+echo "-- scripts/test/test-studio-pipeline-config.mjs"
+if ! node scripts/test/test-studio-pipeline-config.mjs; then
+  failed=1
+fi
+
+echo "-- scripts/test/test-studio-pipeline-focus.mjs"
+if ! node scripts/test/test-studio-pipeline-focus.mjs; then
+  failed=1
+fi
+
 if [[ "$failed" -ne 0 ]]; then
   echo "VALIDATE FAILED"
   exit 1