pandorga 2.0.1 → 2.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8ae9f3cb8f8c39b2e9af7ee8b582a35cdc389f61302704888d3c85adb88de394
-  data.tar.gz: 0d313d288d44e2d249751ceda633d6f58d981c42561a116c3faad0aac8e2cd9b
+  metadata.gz: 221dacb578ada5bb29237157b9e2fcffc4789053fe0566eae9f626967334b291
+  data.tar.gz: eedd7e320f83cf654525e469eddf66f013f7229324304d13ed983ad38a0b58eb
 SHA512:
-  metadata.gz: 91f82a29a1d5b7890fc2955ba3522d35f5c992a57dfda3b2f6e815e7ca613278aa1910be945170351d82ce5675045994070c6a6c8e9a8e20c3bf8d7b2448736d
-  data.tar.gz: 7478438f607e96460805455bf6873b0a97ccf35804a215f0d5422a168eb91228a91ef6907d9240b8d63f173e1cebc031c45070d0c8dfa4ae09b3ab8bf05ac088
+  metadata.gz: 41b852a904d013dbc020d1d14fe33b56d9355a5cf201255fc14df14ae095c2457da292ba2f357cce007b45a886fd047b2c9312a4bc1d3eb9f2f69b995270845c
+  data.tar.gz: d48976bdd3f548a0480766c9e6449566ce3527c5e78bc603711681d32c0ab144d6a8c5c7c60c02fd65553a1160c02eb3f10a8791e284ec0e3c98cdd4019f7f19

data/CHANGELOG.md

@@ -2,6 +2,22 @@
 
 ## [Unreleased]
 
+## [2.0.3] — 2026-10-25
+
+### Fixed
+
+- Entry titles on the Articles, Sources and Bibliography pages use the Rascunhos size at every width. The Articles rows and Sources cards were a step smaller and shrank again below 768px, and the Bibliography cards were a step smaller (PLT-AC-55).
+
+## [2.0.2] — 2026-10-25
+
+### Added
+
+- The home hero summary has its own file, `content/pages/home/summary.md`, editable in Studio as "Hero summary" under Presentation, so the site presentation and the CV page summary can differ. Without the file the hero keeps showing `content/pages/cv/summary.md`; saved empty, the hero shows no summary. The Studio CV summary entry for the public page is now labelled "CV page" (PLT-AC-54).
+
+### Changed
+
+- The object-store publish syncs `collections/`, `data/`, `pages/` and `media/` as parallel jobs, prints each job's log in a fixed order, and uploads `manifest.json` last, only after every job succeeded. Upload, delete and verification semantics are unchanged (PLT-AC-53).
+
 ## [2.0.1] — 2026-10-24
 
 ### Fixed

data/DESIGN.md

@@ -762,7 +762,7 @@ those tokens — no one-off rem sizes for roles that already have a step.
 | Page H1 | Cinzel Decorative | 400 | `--type-page` / `-lg` |
 | Section | Cinzel Decorative | 400 | `--type-section` / `-lg` |
 | Detail title | Newsreader | 600 | `--type-title` / `-lg` |
-| Card / row title | Newsreader | 600 | `--type-card` / `-lg` |
+| Card / row title | Newsreader | 600 | `--type-card` / `-lg`; listing entry titles always `-lg` (PLT-AC-55) |
 | Prose | Newsreader | 300 | `--type-prose` |
 | UI body | Newsreader | 300 | `--type-body` |
 | Detail standfirst | Newsreader | 300 | `--type-subtitle` (= body − 1pt) |

data/assets/css/base.css

@@ -527,11 +527,11 @@ html[data-theme="dark"] {
   --color-link-hover: color-mix(in srgb, var(--color-tertiary) 72%, var(--color-on-surface));
 }
 
-/* Card / row / collection entry titles — content-title voice (Regular 400). */
+/* Card / row / collection entry titles — content-title voice (Regular 400).
+   Listing rows and source cards also carry .card-title, so they get the voice
+   here and their size from the listing entry rule below. */
 .card-title,
-.ledger-card-title,
-.ledger-row-title,
-.source-card-title {
+.ledger-card-title {
   font-family: var(--font-content-title);
   font-size: var(--type-card);
   font-weight: var(--weight-title);
@@ -554,6 +554,17 @@ html[data-theme="dark"] {
   }
 }
 
+/* Listing entry titles (PLT-AC-55). The Articles and Rascunhos rows and the
+   Sources and Bibliography cards set their entry title at the Rascunhos step
+   at every width. This rule follows the small-screen step above on purpose,
+   so it holds below 768px too. */
+.ledger-row-title,
+.source-card-title,
+.bib-card-title {
+  font-size: var(--type-card-lg);
+  line-height: 1.25;
+}
+
 /* Label-face meta on CV identity lines, etc. */
 .card-meta {
   font-family: var(--font-label);

data/assets/css/ledger/bibliography.css

@@ -138,8 +138,6 @@
 
 .bib-card-title {
   margin: 0;
-  font-size: var(--type-card);
-  line-height: 1.25;
   color: var(--color-on-surface);
   overflow-wrap: anywhere;
 }

data/assets/css/ledger/home.css

@@ -222,7 +222,8 @@
 
 /* No empty min-height. A guessed line count shifts the call to action
    when the summary is shorter or taller than the reserve. The paragraph
-   is in the HTML when content/pages/cv/summary.md exists. */
+   is in the HTML when content/pages/home/summary.md or
+   content/pages/cv/summary.md exists. */
 
 .home-hero-summary p {
   margin: 0 0 var(--space-md);

data/assets/css/ledger/rascunhos.css

@@ -204,13 +204,6 @@
   text-transform: uppercase;
 }
 
-/* Entry titles — a step above the shared ledger-row scale so the notebook
-   line reads as a title, not a caption. */
-.rascunhos-row .ledger-row-title {
-  font-size: var(--type-card-lg);
-  line-height: 1.25;
-}
-
 /* ══ Ledger entry ══
    Ruled paper, not a stack of cards: the shared row's card fill and padding
    come off, and what separates two entries is the hairline under each one.

data/lib/pandorga/commands/new.rb

@@ -24,7 +24,7 @@ module Pandorga
 
         gem "pandorga", "~> 2.0"
         # Local checkout: gem "pandorga", path: "../pandorga"
-        # GitHub tag: gem "pandorga", github: "alexandrelheinen/pandorga", tag: "v2.0.1"
+        # GitHub tag: gem "pandorga", github: "alexandrelheinen/pandorga", tag: "v2.0.3"
         gem "csv"
         gem "base64"
       RUBY

data/lib/pandorga/hero_summary.rb

@@ -1,12 +1,14 @@
 # frozen_string_literal: true
 
-# Home hero summary from content/pages/cv/summary.md.
+# Home hero summary from content/pages/home/summary.md, falling back to
+# content/pages/cv/summary.md when the hero file is absent (PLT-AC-54).
 # That paragraph is the LCP element. Leaving it empty until JSON arrives
 # makes most of LCP render delay, and a guessed empty box shifts the
 # call to action when the real text replaces it.
 # Refs: PLT-AC-23
 module Pandorga
   module HeroSummary
+    HOME_RELATIVE = "content/pages/home/summary.md"
     RELATIVE = "content/pages/cv/summary.md"
     FALLBACK = '<div id="home-cv-summary" class="home-hero-summary font-body" ' \
                'data-content-fragment="cv/summary" data-content-quiet></div>'
@@ -14,11 +16,24 @@ module Pandorga
     module_function
 
     def markdown(source_root)
+      home = read_body(source_root, HOME_RELATIVE)
+      return home unless home.nil?
+
+      read_body(source_root, RELATIVE).to_s
+    end
+
+    # A hero file saved with a blank body hides the hero summary.
+    def hidden?(source_root)
+      read_body(source_root, HOME_RELATIVE) == ""
+    end
+
+    # The stripped body of a content file, or nil when it is absent.
+    def read_body(source_root, relative)
       root = File.expand_path(source_root.to_s)
-      path = File.expand_path(RELATIVE, root)
+      path = File.expand_path(relative, root)
       prefix = root.end_with?(File::SEPARATOR) ? root : root + File::SEPARATOR
-      return "" unless path.start_with?(prefix)
-      return "" unless File.file?(path)
+      return nil unless path.start_with?(prefix)
+      return nil unless File.file?(path)
 
       strip_front_matter(File.read(path, encoding: "UTF-8")).strip
     end

data/lib/pandorga/jekyll/plugins/hero_summary.rb

@@ -5,10 +5,13 @@ require_relative "../../hero_summary"
 
 module Pandorga
   # Emits the hero summary element. HTML is raw so the paragraph is in the
-  # first response when the markdown file exists.
+  # first response when the markdown file exists. A blank hero file
+  # renders nothing (PLT-AC-54).
   class HeroSummaryTag < Liquid::Tag
     def render(context)
       site = context.registers[:site]
+      return "" if HeroSummary.hidden?(site.source)
+
       HeroSummary.element(HeroSummary.html_for(site))
     end
   end

data/lib/pandorga/version.rb

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

data/scripts/content/publish-content-to-object-store.sh

@@ -58,36 +58,71 @@ run_ruby() {
 
 sync_json_prefix() {
   local prefix="$1"
-  local src="$EXPORT_ROOT/$prefix"
-  if [ -d "$src" ]; then
-    echo "==> Object store replace sync $prefix/"
-    run_ruby "$ROOT/scripts/content/sync-content-json-to-r2.rb" "$EXPORT_ROOT" "$prefix"
-  fi
+  echo "==> Object store replace sync $prefix/"
+  run_ruby "$ROOT/scripts/content/sync-content-json-to-r2.rb" "$EXPORT_ROOT" "$prefix"
 }
 
-echo "==> Publishing content JSON to bucket: ${S3_BUCKET}"
-echo "==> Endpoint: ${S3_ENDPOINT}"
-echo "==> Site root: ${SITE_ROOT}"
-sync_json_prefix collections
-sync_json_prefix data
-sync_json_prefix pages
-
-echo "==> Upload manifest.json"
-aws s3 cp "$EXPORT_ROOT/manifest.json" "s3://${S3_BUCKET}/manifest.json" \
-  --endpoint-url "$S3_ENDPOINT" \
-  --cache-control "public,max-age=60" \
-  --content-type "application/json"
-
 MEDIA_ROOT="$SITE_ROOT/content/media"
-if [ -d "$MEDIA_ROOT" ]; then
+
+sync_media() {
   echo "==> Sync media/ from ${MEDIA_ROOT}"
   aws s3 sync "$MEDIA_ROOT" "s3://${S3_BUCKET}/media" \
     --endpoint-url "$S3_ENDPOINT" \
     --delete \
     --size-only \
     --cache-control "public,max-age=31536000,immutable"
+}
+
+echo "==> Publishing content JSON to bucket: ${S3_BUCKET}"
+echo "==> Endpoint: ${S3_ENDPOINT}"
+echo "==> Site root: ${SITE_ROOT}"
+
+# The prefixes are disjoint, so their syncs run as parallel jobs (PLT-AC-53).
+# Each job logs to its own file; the logs print in a fixed order afterwards.
+JOB_LOGS="$(mktemp -d)"
+trap 'rm -rf "$JOB_LOGS"' EXIT
+JOB_NAMES=()
+JOB_PIDS=()
+
+start_job() {
+  local name="$1"
+  shift
+  ( "$@" ) > "$JOB_LOGS/$name.log" 2>&1 &
+  JOB_NAMES+=("$name")
+  JOB_PIDS+=("$!")
+}
+
+for prefix in collections data pages; do
+  if [ -d "$EXPORT_ROOT/$prefix" ]; then
+    start_job "$prefix" sync_json_prefix "$prefix"
+  fi
+done
+
+if [ -d "$MEDIA_ROOT" ]; then
+  start_job media sync_media
 else
   echo "==> No content/media at ${MEDIA_ROOT} (skipping media sync)"
 fi
 
+FAILED=()
+for i in "${!JOB_PIDS[@]}"; do
+  if ! wait "${JOB_PIDS[$i]}"; then
+    FAILED+=("${JOB_NAMES[$i]}")
+  fi
+  cat "$JOB_LOGS/${JOB_NAMES[$i]}.log"
+done
+
+if [ "${#FAILED[@]}" -gt 0 ]; then
+  echo "error: object-store sync failed for: ${FAILED[*]}; manifest.json not uploaded." >&2
+  exit 1
+fi
+
+# Readers start from the manifest, so it goes up only after every object it
+# can lead to is in place.
+echo "==> Upload manifest.json"
+aws s3 cp "$EXPORT_ROOT/manifest.json" "s3://${S3_BUCKET}/manifest.json" \
+  --endpoint-url "$S3_ENDPOINT" \
+  --cache-control "public,max-age=60" \
+  --content-type "application/json"
+
 echo "==> Object-store publish complete"

data/scripts/content/sync-content-json-to-r2.rb

@@ -39,7 +39,7 @@ endpoint = ENV["R2_ENDPOINT"].to_s
 bucket = ENV["R2_BUCKET"].to_s
 fail!("R2_ENDPOINT is required") if endpoint.empty?
 fail!("R2_BUCKET is required") if bucket.empty?
-fail!("aws CLI is required") if Open3.capture3("bash", "-lc", "command -v aws").last.exitstatus != 0
+fail!("aws CLI is required") if Open3.capture3("bash", "-c", "command -v aws").last.exitstatus != 0
 
 SRC = EXPORT_ROOT.join(PREFIX)
 fail!("Missing local prefix directory: #{SRC}") unless SRC.directory?

data/scripts/test/test-hero-summary.rb

@@ -33,6 +33,47 @@ Dir.mktmpdir("hero-summary") do |dir|
   fail!("front matter leaked: #{mattered.inspect}") unless mattered == "After the matter."
 end
 
+# PLT-AC-54: the hero has its own file; absent falls back, blank hides.
+Dir.mktmpdir("hero-summary-field") do |dir|
+  cv = File.join(dir, "content/pages/cv")
+  home = File.join(dir, "content/pages/home")
+  FileUtils.mkdir_p([cv, home])
+  File.write(File.join(cv, "summary.md"), "Professional summary.\n")
+
+  fail!("absent hero file must fall back to the CV summary") unless Pandorga::HeroSummary.markdown(dir) == "Professional summary."
+  fail!("absent hero file must not hide the hero summary") if Pandorga::HeroSummary.hidden?(dir)
+
+  File.write(File.join(home, "summary.md"), "What the site holds.\n")
+  own = Pandorga::HeroSummary.markdown(dir)
+  fail!("hero file must win over the CV summary, got #{own.inspect}") unless own == "What the site holds."
+  fail!("a hero file with text must not hide") if Pandorga::HeroSummary.hidden?(dir)
+
+  File.write(File.join(home, "summary.md"), "---\ntitle: Hero\n---\n  \n")
+  fail!("a blank hero file must hide the hero summary") unless Pandorga::HeroSummary.hidden?(dir)
+  fail!("a blank hero file must not fall back to the CV summary") unless Pandorga::HeroSummary.markdown(dir) == ""
+
+  FileUtils.rm_rf(cv)
+  FileUtils.rm_f(File.join(home, "summary.md"))
+  fail!("with neither file the hero must stay a fragment, not hide") if Pandorga::HeroSummary.hidden?(dir)
+end
+
+tag = ROOT.join("lib/pandorga/jekyll/plugins/hero_summary.rb").read
+fail!("hero tag ignores a blank hero file") unless tag.include?("HeroSummary.hidden?")
+
+cv_layout = ROOT.join("_layouts/cv.html").read
+fail!("CV page Summary must keep the cv/summary fragment") unless cv_layout.include?('data-content-fragment="cv/summary"')
+fail!("CV page must not read the hero file") if cv_layout.include?("home/summary")
+
+schema = ROOT.join("studio/schema.yml").read
+hero_entry = schema[/- name: home_summary\n(?:        .*\n)+/]
+fail!("Studio has no Hero summary entry") unless hero_entry
+fail!("Studio Hero summary label") unless hero_entry.include?("label: Hero summary")
+fail!("Studio Hero summary path") unless hero_entry.include?("path: content/pages/home/summary.md")
+fail!("Studio Hero summary is not a markdown body") unless hero_entry.include?("component: markdown_body")
+presentation = schema[/- name: presentation\n.*?(?=\n  - name: )/m]
+fail!("Hero summary is not in the Presentation group") unless presentation&.include?("- name: home_summary")
+fail!("Studio public CV entry is not labelled CV page") unless schema.match?(/- name: cv_summary_public\n\s+label: CV page\n/)
+
 fallback = Pandorga::HeroSummary.element("")
 fail!("missing summary must stay a quiet fragment") unless fallback.include?('data-content-fragment="cv/summary" data-content-quiet')
 fail!("missing summary must not claim to be static") if fallback.include?("data-content-static")

data/scripts/test/test-listing-entry-titles.rb

@@ -0,0 +1,107 @@
+#!/usr/bin/env ruby
+# frozen_string_literal: true
+
+# PLT-AC-55: listing entry titles on the Articles, Rascunhos, Sources and
+# Bibliography pages share one size, the Rascunhos step, at every width.
+
+require "pathname"
+
+ROOT = Pathname.new(__dir__).join("..", "..").expand_path
+CSS_ROOT = ROOT.join("assets/css")
+TITLE_CLASSES = %w[.ledger-row-title .source-card-title .bib-card-title].freeze
+
+def fail!(message)
+  warn "FAIL: #{message}"
+  exit 1
+end
+
+def strip_comments(source)
+  source.gsub(%r{/\*.*?\*/}m, "")
+end
+
+# Yields [selectors, declarations, offset] for every rule, descending into
+# at-rule blocks such as media queries.
+def each_rule(source, &block)
+  depth_start = 0
+  i = 0
+  while (open = source.index("{", i))
+    prelude = source[i...open].strip
+    depth = 1
+    j = open + 1
+    while depth.positive? && j < source.length
+      depth += 1 if source[j] == "{"
+      depth -= 1 if source[j] == "}"
+      j += 1
+    end
+    body = source[(open + 1)...(j - 1)]
+    if prelude.start_with?("@")
+      each_rule(body, &block) if body.include?("{")
+    else
+      selectors = prelude.split(",").map { |s| s.strip.gsub(/\s+/, " ") }
+      yield selectors, body, open + depth_start
+    end
+    i = j
+  end
+end
+
+def last_compound(selector)
+  selector.split(/\s*[ >+~]\s*/).last.to_s
+end
+
+def declares?(body, property)
+  body.match?(/(?:^|;|\s)#{Regexp.escape(property)}\s*:/)
+end
+
+puts "Testing listing entry titles (PLT-AC-55)..."
+
+base = strip_comments(CSS_ROOT.join("base.css").read)
+shared = []
+each_rule(base) do |selectors, body, offset|
+  shared << [body, offset] if selectors.sort == TITLE_CLASSES.sort
+end
+fail!("base.css needs exactly one rule for #{TITLE_CLASSES.join(', ')}") unless shared.size == 1
+body, shared_offset = shared.first
+fail!("shared rule must set font-size: var(--type-card-lg)") unless body.match?(/font-size:\s*var\(--type-card-lg\)\s*;/)
+fail!("shared rule must set line-height: 1.25") unless body.match?(/line-height:\s*1\.25\s*;/)
+
+small_step = base.index("var(--type-card-sm)", base.index("--type-card-sm:") + 1)
+fail!("small-screen .card-title step not found in base.css") unless small_step
+fail!("shared rule must follow the small-screen .card-title step") unless shared_offset > small_step
+puts "  shared rule: OK"
+
+offenders = []
+Dir.glob(CSS_ROOT.join("**", "*.css")).sort.each do |path|
+  source = strip_comments(File.read(path))
+  each_rule(source) do |selectors, rule_body, _offset|
+    next if selectors.sort == TITLE_CLASSES.sort && path == CSS_ROOT.join("base.css").to_s
+
+    targeted = selectors.select do |sel|
+      compound = last_compound(sel)
+      TITLE_CLASSES.any? { |cls| compound.include?(cls) && !compound.match?(/#{Regexp.escape(cls)}[-\w]/) }
+    end
+    next if targeted.empty?
+
+    %w[font-size line-height].each do |prop|
+      next unless declares?(rule_body, prop)
+
+      offenders << "#{Pathname.new(path).relative_path_from(ROOT)}: #{targeted.join(', ')} sets #{prop}"
+    end
+  end
+end
+fail!("entry title overrides:\n  #{offenders.join("\n  ")}") unless offenders.empty?
+puts "  no overrides: OK"
+
+markup = {
+  "_includes/content-runtime/43-blocks.html" => "ledger-row-title",
+  "_layouts/resources.html" => "source-card-title",
+  "_layouts/bibliography.html" => "bib-card-title"
+}
+markup.each do |file, cls|
+  fail!("#{file} no longer emits #{cls}") unless ROOT.join(file).read.include?(cls)
+end
+%w[_layouts/articles.html _layouts/blog.html].each do |file|
+  fail!("#{file} no longer renders rows with renderLedgerRow") unless ROOT.join(file).read.include?("renderLedgerRow(")
+end
+puts "  markup: OK"
+
+puts "All listing entry title checks passed."

data/scripts/test/test-parallel-content-publish.rb

@@ -0,0 +1,171 @@
+#!/usr/bin/env ruby
+# frozen_string_literal: true
+
+# PLT-AC-53: the object-store publish runs its prefix syncs in parallel and
+# uploads manifest.json last, only when every job succeeded. Uses a stateful
+# fake `aws` on PATH; no credentials or network.
+
+$stdout.sync = true
+$stderr.sync = true
+
+require "fileutils"
+require "json"
+require "open3"
+require "pathname"
+require "tmpdir"
+
+ROOT = Pathname.new(__dir__).join("../..").expand_path
+SCRIPT = ROOT.join("scripts/content/publish-content-to-object-store.sh")
+SYNC = ROOT.join("scripts/content/sync-content-json-to-r2.rb")
+
+def fail!(message)
+  warn "FAIL test-parallel-content-publish: #{message}"
+  exit 1
+end
+
+def check!(condition, message)
+  fail!(message) unless condition
+end
+
+# The fake keeps bucket objects under $FAKE_AWS_BUCKET_DIR and appends one
+# JSON line per call ({argv, start, end}) to $FAKE_AWS_LOG. Listing sleeps so
+# overlapping prefix syncs are observable. FAKE_AWS_FAIL_PREFIX makes the
+# listing of that prefix fail.
+FAKE_AWS = <<~'RUBY'
+  #!/usr/bin/env ruby
+  require "digest"
+  require "fileutils"
+  require "json"
+  bucket_dir = ENV.fetch("FAKE_AWS_BUCKET_DIR")
+  start = Process.clock_gettime(Process::CLOCK_REALTIME)
+  args = ARGV.dup
+  status = 0
+  key_of = ->(url) { url.sub(%r{\As3://[^/]+/?}, "") }
+  opt = ->(name) { (i = args.index(name)) ? args[i + 1] : nil }
+  case args[0, 2]
+  when %w[s3api list-objects-v2]
+    prefix = opt.call("--prefix").to_s
+    sleep 0.4
+    if (bad = ENV["FAKE_AWS_FAIL_PREFIX"].to_s) != "" && prefix == "#{bad}/"
+      warn "fake aws: listing #{prefix} failed"
+      status = 1
+    else
+      contents = Dir.glob(File.join(bucket_dir, "**", "*")).select { |f| File.file?(f) }.map do |f|
+        key = f.delete_prefix("#{bucket_dir}/")
+        next unless key.start_with?(prefix)
+        { "Key" => key, "ETag" => "\"#{Digest::MD5.file(f).hexdigest}\"" }
+      end.compact
+      puts JSON.generate({ "Contents" => contents, "IsTruncated" => false })
+    end
+  when %w[s3 cp]
+    dest = File.join(bucket_dir, key_of.call(args[3]))
+    FileUtils.mkdir_p(File.dirname(dest))
+    FileUtils.cp(args[2], dest)
+  when %w[s3 rm]
+    FileUtils.rm_f(File.join(bucket_dir, key_of.call(args[2])))
+  when %w[s3 sync]
+    src = args[2]
+    prefix = key_of.call(args[3])
+    local = Dir.glob(File.join(src, "**", "*")).select { |f| File.file?(f) }.map { |f| f.delete_prefix("#{src}/") }
+    local.each do |rel|
+      dest = File.join(bucket_dir, prefix, rel)
+      FileUtils.mkdir_p(File.dirname(dest))
+      FileUtils.cp(File.join(src, rel), dest)
+    end
+    if args.include?("--delete")
+      Dir.glob(File.join(bucket_dir, prefix, "**", "*")).select { |f| File.file?(f) }.each do |f|
+        FileUtils.rm_f(f) unless local.include?(f.delete_prefix("#{File.join(bucket_dir, prefix)}/"))
+      end
+    end
+  else
+    warn "fake aws: unsupported #{args.inspect}"
+    status = 2
+  end
+  finish = Process.clock_gettime(Process::CLOCK_REALTIME)
+  File.open(ENV.fetch("FAKE_AWS_LOG"), "a") { |io| io.puts(JSON.generate({ "argv" => args, "start" => start, "end" => finish })) }
+  exit status
+RUBY
+
+check!(SCRIPT.file?, "missing #{SCRIPT}")
+check!(!File.read(SYNC).include?("bash\", \"-lc\""),
+       "sync-content-json-to-r2.rb must check for aws on its own PATH, not a login shell's")
+
+def write(path, body)
+  FileUtils.mkdir_p(File.dirname(path))
+  File.write(path, body)
+end
+
+def run_publish(tmp, fail_prefix: nil)
+  site = File.join(tmp, "site")
+  export = File.join(site, "_content_json")
+  bucket = File.join(tmp, "bucket")
+  bin = File.join(tmp, "bin")
+  log = File.join(tmp, "aws.log")
+  FileUtils.rm_rf([bucket, log])
+  write(File.join(export, "collections/articles/a.json"), %({"a":1}\n))
+  write(File.join(export, "data/b.json"), %({"b":1}\n))
+  write(File.join(export, "pages/c.json"), %({"c":1}\n))
+  write(File.join(export, "manifest.json"), %({"version":1}\n))
+  write(File.join(site, "content/media/images/x.png"), "png")
+  write(File.join(bucket, "collections/articles/stale.json"), "{}")
+  write(File.join(bucket, "media/old.png"), "old")
+  write(File.join(bin, "aws"), FAKE_AWS)
+  FileUtils.chmod(0o755, File.join(bin, "aws"))
+
+  env = ENV.to_h.reject { |k, _| k.start_with?("S3_", "R2_", "AWS_") }.merge(
+    "PATH" => "#{bin}:#{ENV.fetch('PATH')}",
+    "PANDORGA_SITE_ROOT" => site,
+    "S3_ENDPOINT" => "https://example.invalid",
+    "S3_BUCKET" => "test-bucket",
+    "S3_ACCESS_KEY_ID" => "test",
+    "S3_SECRET_ACCESS_KEY" => "test",
+    "FAKE_AWS_BUCKET_DIR" => bucket,
+    "FAKE_AWS_LOG" => log
+  )
+  env["FAKE_AWS_FAIL_PREFIX"] = fail_prefix if fail_prefix
+  out, status = Open3.capture2e(env, "bash", SCRIPT.to_s, export)
+  calls = File.file?(log) ? File.readlines(log).map { |l| JSON.parse(l) } : []
+  [out, status, calls, bucket]
+end
+
+Dir.mktmpdir("pandorga-parallel-publish") do |tmp|
+  out, status, calls, bucket = run_publish(tmp)
+  check!(status.success?, "publish failed (#{status.exitstatus}):\n#{out}")
+
+  objects = Dir.glob(File.join(bucket, "**", "*")).select { |f| File.file?(f) }
+               .map { |f| f.delete_prefix("#{bucket}/") }.sort
+  expected = %w[collections/articles/a.json data/b.json manifest.json media/images/x.png pages/c.json]
+  check!(objects == expected, "bucket objects #{objects.inspect}, expected #{expected.inspect}")
+
+  first_list = %w[collections data pages].to_h do |prefix|
+    call = calls.find { |c| c["argv"][0, 2] == %w[s3api list-objects-v2] && c["argv"].include?("#{prefix}/") }
+    check!(call, "no listing of #{prefix}/ in #{calls.map { |c| c['argv'].first(2) }.inspect}")
+    [prefix, call]
+  end
+  latest_start = first_list.values.map { |c| c["start"] }.max
+  earliest_end = first_list.values.map { |c| c["end"] }.min
+  check!(latest_start < earliest_end,
+         "collections/, data/ and pages/ syncs did not overlap; they must run in parallel")
+
+  manifest = calls.select { |c| c["argv"][0, 2] == %w[s3 cp] && c["argv"][3].to_s.end_with?("/manifest.json") }
+  check!(manifest.length == 1, "expected one manifest upload, got #{manifest.length}")
+  others = calls - manifest
+  check!(others.all? { |c| c["end"] <= manifest.first["start"] },
+         "manifest.json must be uploaded after every other call ended")
+
+  order = ["R2 replace sync collections/", "R2 replace sync data/", "R2 replace sync pages/",
+           "==> Sync media/", "==> Upload manifest.json", "==> Object-store publish complete"]
+  positions = order.map { |marker| out.index(marker) }
+  check!(positions.none?(&:nil?), "missing log markers #{order.zip(positions).select { |_, p| p.nil? }.map(&:first)}:\n#{out}")
+  check!(positions == positions.sort, "job logs must print in the fixed order #{order.inspect}:\n#{out}")
+
+  out, status, calls, = run_publish(tmp, fail_prefix: "data")
+  check!(!status.success?, "publish must fail when a prefix sync fails:\n#{out}")
+  check!(out.include?("listing data/ failed") || out.include?("list-objects-v2 failed for data/"),
+         "the failed job's log must be printed:\n#{out}")
+  check!(out.include?("data"), "the failure must name the failed job")
+  check!(calls.none? { |c| c["argv"][0, 2] == %w[s3 cp] && c["argv"][3].to_s.end_with?("/manifest.json") },
+         "manifest.json must not be uploaded when a job failed")
+end
+
+puts "PASS test-parallel-content-publish"

data/scripts/validate.sh

@@ -62,6 +62,7 @@ TESTS=(
   scripts/test/test-theme-contrast.rb
   scripts/test/test-hydrate-reserve.rb
   scripts/test/test-hero-summary.rb
+  scripts/test/test-listing-entry-titles.rb
   scripts/test/test-lighthouse-policy.rb
   scripts/test/test-home-lists.rb
   scripts/test/test-home-articles.rb
@@ -78,6 +79,7 @@ TESTS=(
   scripts/test/test-v3-layout.rb
   scripts/test/test-studio-section-texts.rb
   scripts/test/test-home-article-cards.rb
+  scripts/test/test-parallel-content-publish.rb
 )
 
 failed=0

data/studio/schema.yml

@@ -823,7 +823,7 @@ content:
     type: group
     items:
       - name: cv_summary_public
-        label: Public
+        label: CV page
         type: file
         path: content/pages/cv/summary.md
         format: raw
@@ -862,6 +862,16 @@ content:
           - name: body
             component: markdown_body
 
+      - name: home_summary
+        label: Hero summary
+        type: file
+        path: content/pages/home/summary.md
+        format: raw
+        description: "Home hero paragraph. Without this file the hero shows the CV page summary; saved empty, the hero shows none."
+        fields:
+          - name: body
+            component: markdown_body
+
       - name: cv_ai_practice
         label: AI practice
         type: file

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pandorga
 version: !ruby/object:Gem::Version
-  version: 2.0.1
+  version: 2.0.3
 platform: ruby
 authors:
 - The Pandorga contributors
@@ -449,11 +449,13 @@ files:
 - scripts/test/test-language-export.rb
 - scripts/test/test-ledger-blocks.rb
 - scripts/test/test-lighthouse-policy.rb
+- scripts/test/test-listing-entry-titles.rb
 - scripts/test/test-listing-filter-contract.rb
 - scripts/test/test-math-row-breaks.mjs
 - scripts/test/test-math-row-breaks.rb
 - scripts/test/test-new.rb
 - scripts/test/test-no-personal-data.rb
+- scripts/test/test-parallel-content-publish.rb
 - scripts/test/test-poster-srcset.mjs
 - scripts/test/test-public-cv-export.rb
 - scripts/test/test-publish-site-root.rb