pandorga 1.7.1 → 1.7.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: aa9e02cc789f512401bb4e9bdbc28c732dff0f3849bbf055d0cd546678cbdd1d
-  data.tar.gz: 8a17699ce008fa34e5114f9b4e8e643d0d754f7620ed3b0357aeb2cc9a4c201e
+  metadata.gz: 13bf0047d336a1cfa896a02c88b8bcb172175c564d922474bf65136b113db106
+  data.tar.gz: e146ecadc0250a2315ab17b887a8b2425d9c7ded47cfc0f45c6a3ee1bc0edc6d
 SHA512:
-  metadata.gz: 7c6e5413b160b64e546f4aa7c80f1e45bb8c5716336593984e4f1f14e642ed444adba3dc5261d08d31257367060dab631777f744f6126c38df1c264e7259d139
-  data.tar.gz: 21588e084fe1f417a4f9fa55e532c8ebca01930ea95ad740ecafece15578df4c237461e0213a9328526b982847dcb4bff07d47f1a182a50a36a9fb5348f9cb74
+  metadata.gz: e5ccb2966deff68f8c3292f997cd39980024e47eeaab2e756f74d193e03ff6b40fd5f235c51fc30a69277057c31350246451004b25f1086df24a163efd45c777
+  data.tar.gz: df67d8c8be5fd84ffca3abde105d4b05e6c76030e9e1d45f9e26c6dfdb1722a2ecd82ba55a4b9d699a73f1dacd5bc5999402d1ae7f60f727bee5a8058f7f1c16

data/CHANGELOG.md

@@ -1,5 +1,28 @@
 # Changelog
 
+## [1.7.2] — 2026-10-11
+
+### Fixed
+
+- A published article, post, project, or product is a static page at its
+  public URL. The sitemap lists that URL. The static canonical and
+  `og:url` are the detail path, and `data-listing-path` stays the
+  listing. Articles emit `Article` JSON-LD. Posts emit `BlogPosting`.
+- The build writes `_redirects` into the output and drops the splat 200
+  lines that sent those URLs back to the listing shell. The source file
+  is unchanged. Publish the built file.
+- A listing page's static `<h1>` is the title from
+  `content/pages/headers.yml`, or the page title. It is not the word
+  Loading.
+- The top bar is a `<header>`. The section links are one
+  `<nav aria-label="Sections">` inside it.
+- Dark theme text on the tertiary container is `#ebe7c3` on `#6a684f`
+  (4.52:1). The container colour is unchanged.
+- The hero portrait gate crops the runtime through `pickHeroPortrait`.
+
+No `pandorga:` config keys were added or renamed. Font roles and the
+ledger fallbacks are unchanged.
+
 ## [1.7.1] — 2026-10-11
 
 ### Fixed

data/_data/themes/dark.yml

@@ -13,7 +13,8 @@ colors:
   tertiary_container: "#6a684f"
   on_primary_container: "#c0d8e6"
   on_secondary_container: "#b5d9c2"
-  on_tertiary_container: "#e8e4c0"
+  # #e8e4c0 on #6a684f was 4.40:1. Three steps lighter clears 4.5:1.
+  on_tertiary_container: "#ebe7c3"
   chart_project: "#98c8ac"
 
 neutrals:

data/_includes/content-runtime/60-fragments.html

@@ -264,7 +264,7 @@
       var standfirstEl = root.querySelector('[data-content-header-standfirst]');
       return loadPageHeaders().then(function (headers) {
         var data = headers[pageKey];
-        if (titleEl) {
+        if (titleEl && !titleEl.hasAttribute('data-content-static')) {
           var title = data && data.title;
           if (title) {
             titleEl.innerHTML = stripSingleParagraph(parseInlineMarkdown(String(title)));
@@ -299,7 +299,7 @@
         }
       }).catch(function (error) {
         console.error(error);
-        if (titleEl) {
+        if (titleEl && !titleEl.hasAttribute('data-content-static')) {
           titleEl.textContent = 'Page header could not be loaded right now.';
         }
       });

data/_includes/page/page-header.html

@@ -1,4 +1,6 @@
 {% assign _header_key = include.key | default: page.key %}
+{% assign _header_record = site.data.page_headers[_header_key] %}
+{% assign _header_title = _header_record.title | default: page.title | default: _header_key %}
 
 <header class="{{ include.header_class | default: 'mb-10 md:mb-12' }}" data-content-page-header="{{ _header_key }}">
   <div class="page-header-row{% if include.listing_filter %} page-header-row--with-filter{% endif %}">
@@ -6,7 +8,7 @@
       <span data-content-header-eyebrow class="{{ include.eyebrow_class | default: 'font-label text-sm text-secondary block mb-4 hidden' }}" hidden></span>
 
       <h1 class="page-header-title font-title{% if include.title_class %} {{ include.title_class }}{% endif %}">
-        <span data-content-header-title class="italic">Loading...</span>
+        <span data-content-header-title class="italic"{% if _header_title != blank %} data-content-static{% endif %}>{% if _header_title contains "*" %}{% assign _header_bits = _header_title | split: "*" %}{% for _header_bit in _header_bits %}{% assign _header_em = forloop.index0 | modulo: 2 %}{% if _header_em == 1 %}<em>{{ _header_bit | escape }}</em>{% else %}{{ _header_bit | escape }}{% endif %}{% endfor %}{% else %}{{ _header_title | escape }}{% endif %}</span>
       </h1>
 
       <span data-content-header-tags class="{{ include.tags_class | default: 'page-header-tags font-label text-tertiary mt-4 block hidden' }}" hidden></span>

data/_includes/theme/json-ld.html

@@ -1,6 +1,8 @@
 {%- comment -%}
-  WebSite and Person from identity. No new required keys: name is already
-  required, and url is the existing optional canonical origin.
+  WebSite and Person from identity. A detail page with schema_type
+  Article or BlogPosting also emits that type. No new required keys:
+  name is already required, and url is the existing optional canonical
+  origin.
 {%- endcomment -%}
 {%- assign ld_name = site.pandorga.identity.name | default: site.author | default: site.title -%}
 {%- assign ld_url = site.pandorga.identity.url | default: site.url -%}
@@ -9,6 +11,37 @@
 {%- else -%}
   {%- assign ld_website = "/" | absolute_url -%}
 {%- endif -%}
+{%- assign ld_canonical = canonical_path | default: page.canonical_url | default: page.url -%}
+{%- if page.schema_type == "Article" or page.schema_type == "BlogPosting" -%}
+{%- if page.title != blank -%}
+<script type="application/ld+json">
+{
+  "@context": "https://schema.org",
+  "@type": {{ page.schema_type | jsonify }},
+  "headline": {{ page.title | jsonify }},
+  "url": {{ ld_canonical | absolute_url | jsonify }}
+  {%- if page.description != blank -%},
+  "description": {{ page.description | jsonify }}
+  {%- endif -%}
+  {%- if page.date -%},
+  "datePublished": {{ page.date | date_to_xmlschema | jsonify }}
+  {%- endif -%}
+  {%- if page.last_updated -%},
+  "dateModified": {{ page.last_updated | date_to_xmlschema | jsonify }}
+  {%- endif -%}
+  {%- if page.lang -%},
+  "inLanguage": {{ page.lang | jsonify }}
+  {%- endif -%}
+  {%- if ld_name != blank -%},
+  "author": {
+    "@type": "Person",
+    "name": {{ ld_name | jsonify }}
+  }
+  {%- endif -%}
+}
+</script>
+{%- endif -%}
+{%- endif -%}
 {%- if ld_name != blank -%}
 <script type="application/ld+json">
 {

data/_layouts/shell.html

@@ -211,8 +211,8 @@
 
   {% assign _all_pages = site.pages %}
 
-  <!-- TopNavBar -->
-  <nav class="sticky top-0 z-50 bg-surface">
+  <!-- TopNavBar. The bar is the banner landmark. The section links are the only nav inside it. -->
+  <header class="sticky top-0 z-50 bg-surface">
     <div class="flex justify-between items-center w-full px-6 sm:px-8 md:px-10 pt-5 pb-3.5 leading-none max-w-7xl mx-auto">
       <div class="flex items-center gap-3 sm:gap-4 min-w-0">
         <a href="/" aria-label="{% if _home_label != blank %}{{ _home_label }} – Home{% else %}Home{% endif %}" class="shrink-0 inline-flex items-center gap-3 site-wordmark-link">
@@ -279,7 +279,7 @@
     </div>
     <!-- Tonal separator — "No-Line" design rule: background-color shift instead of a CSS border -->
     <div class="bg-surface-container-low h-[1px]"></div>
-  </nav>
+  </header>
   <script>
     (function () {
       var wrap = document.getElementById('site-menu');

data/_plugins/local_detail_rewrites.rb

@@ -6,6 +6,14 @@ require "net/http"
 require "uri"
 require_relative "content_detail_paths"
 
+# The gem copy and the _plugins copy stay the same file. Either path can
+# resolve the shared helper.
+detail_routes = [
+  File.expand_path("../../../detail_routes", __dir__),
+  File.expand_path("../lib/pandorga/detail_routes", __dir__)
+].find { |path| File.file?("#{path}.rb") }
+require detail_routes if detail_routes
+
 # Mirrors Cloudflare _redirects detail rewrites during `jekyll serve` so
 # /posts/:slug/ and /articles/:slug/ work locally without per-slug HTML files.
 # Also handles the local dev re-export endpoint and proxies /api/studio/* to
@@ -115,7 +123,8 @@ module Jekyll
           end
 
           target = ContentDetailPaths.rewrite_path(req.path)
-          if target
+          root = instance_variable_get(:@root)
+          if target && !Pandorga::DetailRoutes.static_index?(root, req.path)
             req.instance_variable_set(:@path, target)
             req.instance_variable_set(:@path_info, target)
           end

data/lib/pandorga/detail_routes.rb

@@ -0,0 +1,29 @@
+# frozen_string_literal: true
+
+module Pandorga
+  # Detail URLs that used to be 200-rewritten onto a listing shell.
+  # Cloudflare follows `_redirects` even when a static file exists, so the
+  # built file must drop those splat lines or the generated page is never
+  # served. The source file is left alone.
+  module DetailRoutes
+    SPLAT_LINE = %r{\A/(?:articles|posts|blog|projects|products)/(?::slug|\*)/?\s+\S+\s+200\s*\z}
+
+    module_function
+
+    def strip_splat_rewrites(text)
+      text.to_s.lines.reject { |line| SPLAT_LINE.match?(line.strip) }.join
+    end
+
+    def static_index?(root, path)
+      root = root.to_s
+      return false if root.empty?
+
+      clean = path.to_s.split("?").first.to_s
+      relative = clean.sub(%r{\A/}, "").sub(%r{/\z}, "")
+      return false if relative.empty? || relative.include?("..")
+
+      File.file?(File.join(root, relative, "index.html")) ||
+        File.file?(File.join(root, "#{relative}.html"))
+    end
+  end
+end

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

@@ -0,0 +1,154 @@
+# frozen_string_literal: true
+
+require "jekyll"
+require "pathname"
+
+require_relative "../../detail_routes"
+require_relative "lib/writing_entry"
+
+module Pandorga
+  module Jekyll
+    # One static page per published article, post, project, and product.
+    # The listing layout stays, so the client still opens the entry from
+    # the URL. `listing_path` stays the listing URL. The permalink is the
+    # detail URL, which is what the sitemap and the static canonical use.
+    class DetailDocumentsGenerator < ::Jekyll::Generator
+      safe true
+      priority :low
+
+      COLLECTION_FOR_TYPE = {
+        "article" => "articles",
+        "post" => "posts",
+        "project" => "projects",
+        "product" => "products"
+      }.freeze
+
+      DETAIL_PATTERN = {
+        "article" => "/articles/:slug/",
+        "post" => "/posts/:slug/",
+        "project" => "/projects/:slug/",
+        "product" => "/products/:slug/"
+      }.freeze
+
+      SCHEMA = {
+        "article" => "Article",
+        "post" => "BlogPosting"
+      }.freeze
+
+      def generate(site)
+        return if site.config["shell_content_agnostic"]
+
+        root = Pathname.new(site.source)
+        COLLECTION_FOR_TYPE.each do |type, collection|
+          listing = listing_page(site, type)
+          next unless listing
+          next if listing.data["listing_path"].to_s.empty?
+
+          directory = root.join(ContentKeys::CONTENT_PATHS.fetch(type))
+          next unless directory.directory?
+
+          Dir.glob(directory.join("*.{md,markdown}")).sort.each do |absolute|
+            emit(site, listing, type, collection, absolute, root)
+          end
+        end
+      end
+
+      private
+
+      def emit(site, listing, type, collection, absolute, root)
+        relative = Pathname.new(absolute).relative_path_from(root.join("content"))
+        repo_relative = Pathname.new("content").join(relative).to_s
+        record = WritingEntry.build(absolute, collection, repo_relative)
+        return if record.nil?
+
+        detail_path = record["url"].to_s
+        return if detail_path.empty?
+        return if taken?(site, detail_path)
+
+        front_matter, = ContentKeys.split_front_matter(File.read(absolute))
+        dir = detail_path.sub(%r{\A/}, "").sub(%r{/\z}, "")
+        page = ::Jekyll::PageWithoutAFile.new(site, site.source, dir, "index.html")
+        page.data["layout"] = listing.data["layout"]
+        page.data["permalink"] = detail_path
+        page.data["canonical_url"] = detail_path
+        page.data["listing_path"] = listing.data["listing_path"]
+        page.data["collection"] = collection
+        page.data["detail"] = listing.data["detail"].to_s
+        page.data["detail"] = DETAIL_PATTERN.fetch(type) if page.data["detail"].empty?
+        page.data["detail_document"] = true
+        page.data["accent"] = listing.data["accent"] if listing.data["accent"]
+        title = record["title"].to_s.strip
+        title = record["key"].to_s if title.empty?
+        page.data["title"] = title unless title.empty?
+        description = record["description"].to_s.strip
+        description = record["excerpt"].to_s.strip if description.empty?
+        page.data["description"] = description unless description.empty?
+        page.data["date"] = record["date"] unless record["date"].to_s.empty?
+        page.data["last_updated"] = record["last_updated"] unless record["last_updated"].to_s.empty?
+        thumbnail = record["thumbnail"].to_s.strip
+        page.data["thumbnail"] = thumbnail unless thumbnail.empty?
+        lang = front_matter["language"] || front_matter["lang"] || listing.data["lang"]
+        page.data["lang"] = lang if lang
+        if SCHEMA.key?(type)
+          page.data["og_type"] = "article"
+          page.data["schema_type"] = SCHEMA.fetch(type)
+        end
+        page.content = ""
+        site.pages << page
+      end
+
+      def taken?(site, detail_path)
+        site.pages.any? do |page|
+          page.data["permalink"].to_s == detail_path || page.url.to_s == detail_path
+        end
+      end
+
+      def listing_page(site, type)
+        entry = registry_entry(site, type)
+        return nil unless entry
+
+        site.pages.find do |page|
+          page.data["detail_document"] != true && page.data["key"].to_s == entry["key"].to_s
+        end
+      end
+
+      def registry_entry(site, type)
+        pages = site.data["pandorga_pages"]
+        return nil unless pages.is_a?(Array)
+
+        pages.find { |entry| entry.is_a?(Hash) && registry_match?(entry, type) }
+      end
+
+      def registry_match?(entry, type)
+        case type
+        when "article"
+          entry["template"].to_s == "articles" || entry["collection"].to_s == "articles"
+        when "post"
+          entry["template"].to_s == "blog" || entry["collection"].to_s == "posts"
+        when "project"
+          entry["template"].to_s == "portfolio" || entry["collection"].to_s == "projects"
+        when "product"
+          cols = entry["collections"]
+          entry["template"].to_s == "cv" ||
+            (cols.is_a?(Hash) && !cols["products"].to_s.empty?)
+        else
+          false
+        end
+      end
+    end
+  end
+end
+
+Jekyll::Hooks.register :site, :post_write do |site|
+  # Jekyll skips underscore files, so a source `_redirects` never reaches
+  # the output unless the site lists it under `include`. The published
+  # file has to be the one Cloudflare reads.
+  dest = File.join(site.dest, "_redirects")
+  source = File.join(site.source, "_redirects")
+  File.write(dest, File.read(source)) if !File.file?(dest) && File.file?(source)
+  next unless File.file?(dest)
+
+  original = File.read(dest)
+  stripped = Pandorga::DetailRoutes.strip_splat_rewrites(original)
+  File.write(dest, stripped) if stripped != original
+end

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

@@ -6,6 +6,14 @@ require "net/http"
 require "uri"
 require_relative "content_detail_paths"
 
+# The gem copy and the _plugins copy stay the same file. Either path can
+# resolve the shared helper.
+detail_routes = [
+  File.expand_path("../../../detail_routes", __dir__),
+  File.expand_path("../lib/pandorga/detail_routes", __dir__)
+].find { |path| File.file?("#{path}.rb") }
+require detail_routes if detail_routes
+
 # Mirrors Cloudflare _redirects detail rewrites during `jekyll serve` so
 # /posts/:slug/ and /articles/:slug/ work locally without per-slug HTML files.
 # Also handles the local dev re-export endpoint and proxies /api/studio/* to
@@ -115,7 +123,8 @@ module Jekyll
           end
 
           target = ContentDetailPaths.rewrite_path(req.path)
-          if target
+          root = instance_variable_get(:@root)
+          if target && !Pandorga::DetailRoutes.static_index?(root, req.path)
             req.instance_variable_set(:@path, target)
             req.instance_variable_set(:@path_info, target)
           end

data/lib/pandorga/version.rb

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

data/scripts/test/test-discovery.rb

@@ -3,9 +3,13 @@
 
 # Portrait sources and discovery documents (PLT-AC-20, PLT-AC-21).
 
+require "fileutils"
 require "pathname"
 
+require "tmpdir"
+
 require_relative "../../lib/pandorga/discovery"
+require_relative "../../lib/pandorga/detail_routes"
 
 ROOT = Pathname.new(__dir__).join("..", "..").expand_path
 
@@ -47,5 +51,32 @@ image_meta = shell[/<meta name="image"[^>]*>/]
 fail!("shell missing name=image") unless image_meta
 fail!("name=image also sets property=og:image") if image_meta.include?("property=")
 fail!("discovery generator is not loaded from the plugin dir") unless ROOT.join("lib/pandorga/jekyll/plugins/discovery.rb").file?
+fail!("detail pages are not generated from the plugin dir") unless ROOT.join("lib/pandorga/jekyll/plugins/detail_documents.rb").file?
+
+redirects = <<~TXT
+  /legacy /articles/2026-01-01-welcome/ 301
+  /articles/:slug /pages/articles/ 200
+  /posts/:slug/ /pages/blog/ 200
+  /blog/:slug /pages/blog/ 200
+  /projects/:slug /pages/projects/ 200
+  /products/* /pages/cv/ 200
+  /keep /pages/articles/ 301
+TXT
+stripped = Pandorga::DetailRoutes.strip_splat_rewrites(redirects)
+fail!("splat 200 rewrite survived") if stripped.match?(%r{/(?:articles|posts|blog|projects|products)/(?::slug|\*)})
+fail!("a 301 was stripped with the splats") unless stripped.include?("/legacy /articles/2026-01-01-welcome/ 301")
+fail!("a listing 301 was stripped") unless stripped.include?("/keep /pages/articles/ 301")
+
+Dir.mktmpdir("detail-static") do |dir|
+  FileUtils.mkdir_p(File.join(dir, "articles", "welcome"))
+  File.write(File.join(dir, "articles", "welcome", "index.html"), "<p>welcome</p>")
+  fail!("a built detail page must not be rewritten") unless Pandorga::DetailRoutes.static_index?(dir, "/articles/welcome/")
+  fail!("a missing detail page must still be rewritten") if Pandorga::DetailRoutes.static_index?(dir, "/articles/missing/")
+end
+
+json_ld = ROOT.join("_includes/theme/json-ld.html").read
+fail!("detail pages have no Article schema") unless json_ld.include?('page.schema_type == "Article"')
+fail!("detail pages have no BlogPosting schema") unless json_ld.include?('page.schema_type == "BlogPosting"')
+fail!("detail schema omits datePublished") unless json_ld.include?("datePublished")
 
 puts "PASS test-discovery"

data/scripts/test/test-examples-build.rb

@@ -33,8 +33,23 @@ end
   FileUtils.rm_rf(example.join("_site"))
   FileUtils.rm_rf(example.join("_content_json"))
 
+  redirects_path = example.join("_redirects")
+  original_redirects = redirects_path.file? ? redirects_path.read : nil
+
+  begin
   run!("bundle", "install", "--quiet", chdir: example) if example.join("Gemfile").file?
   run!("ruby", EXE.to_s, "export", "content", "_content_json", chdir: example)
+  if name == "full" && redirects_path.file?
+    splat = <<~TXT
+
+      /articles/:slug /pages/articles/ 200
+      /posts/:slug /pages/blog/ 200
+      /blog/:slug /pages/blog/ 200
+      /projects/:slug /pages/projects/ 200
+      /products/:slug /pages/cv/ 200
+    TXT
+    redirects_path.write("#{redirects_path.read.rstrip}\n#{splat}")
+  end
   run!("bundle", "exec", "jekyll", "build", "--quiet", chdir: example)
 
   index = example.join("_site/index.html")
@@ -65,6 +80,18 @@ end
       warn "FAIL test-examples-build: full home still hydrates the summary"
       exit 1
     end
+    {
+      "pages/articles/index.html" => "Articles",
+      "pages/blog/index.html" => "Rascunhos",
+      "pages/projects/index.html" => "Projects"
+    }.each do |rel, title|
+      listing = example.join("_site", rel).read
+      heading = listing[/<h1 class="page-header-title.*?<\/h1>/m].to_s
+      unless heading.include?(title) && heading.include?("data-content-static") && !heading.include?("Loading...")
+        warn "FAIL test-examples-build: #{rel} static h1 is #{heading.inspect}"
+        exit 1
+      end
+    end
   else
     unless html.include?('id="home-cv-summary" class="home-hero-summary font-body" data-content-fragment="cv/summary" data-content-quiet')
       warn "FAIL test-examples-build: minimal home dropped the summary fallback"
@@ -86,6 +113,66 @@ end
     warn "FAIL test-examples-build: #{name} robots missing the sitemap line\n#{robots}"
     exit 1
   end
+  if name == "full"
+    {
+      "/articles/2026-01-01-welcome/" => "Article",
+      "/posts/2026-01-02-note/" => "BlogPosting"
+    }.each do |path, schema|
+      loc = "#{origin}#{path}"
+      unless sitemap.include?("<loc>#{loc}</loc>")
+        warn "FAIL test-examples-build: sitemap missing #{loc}\n#{sitemap}"
+        exit 1
+      end
+      page = example.join("_site#{path}index.html")
+      unless page.file?
+        warn "FAIL test-examples-build: missing static detail #{page}"
+        exit 1
+      end
+      body = page.read
+      canonical = body[/rel="canonical" href="([^"]+)"/, 1].to_s
+      og_url = body[/property="og:url" content="([^"]+)"/, 1].to_s
+      unless canonical.include?(path) && !canonical.include?("/pages/")
+        warn "FAIL test-examples-build: canonical for #{path} is #{canonical}"
+        exit 1
+      end
+      unless og_url.include?(path) && !og_url.include?("/pages/")
+        warn "FAIL test-examples-build: og:url for #{path} is #{og_url}"
+        exit 1
+      end
+      unless body.include?(%("@type": "#{schema}"))
+        warn "FAIL test-examples-build: #{path} missing #{schema} JSON-LD"
+        exit 1
+      end
+      listing_attr = path.include?("/posts/") ? "/pages/blog/" : "/pages/articles/"
+      unless body.include?(%(data-listing-path="#{listing_attr}"))
+        warn "FAIL test-examples-build: #{path} lost the listing path"
+        exit 1
+      end
+    end
+    ["/projects/demo-kit/", "/products/ledger-tool/"].each do |path|
+      loc = "#{origin}#{path}"
+      unless sitemap.include?("<loc>#{loc}</loc>")
+        warn "FAIL test-examples-build: sitemap missing #{loc}\n#{sitemap}"
+        exit 1
+      end
+      page = example.join("_site#{path}index.html")
+      body = page.file? ? page.read : ""
+      canonical = body[/rel="canonical" href="([^"]+)"/, 1].to_s
+      unless canonical.include?(path)
+        warn "FAIL test-examples-build: canonical for #{path} is #{canonical}"
+        exit 1
+      end
+    end
+    built_redirects = example.join("_site/_redirects").read
+    if built_redirects.match?(%r{/(?:articles|posts|blog|projects|products)/:slug})
+      warn "FAIL test-examples-build: built _redirects still rewrites detail URLs\n#{built_redirects}"
+      exit 1
+    end
+    unless redirects_path.read.match?(%r{/articles/:slug\s+/pages/articles/\s+200})
+      warn "FAIL test-examples-build: the build edited the source _redirects"
+      exit 1
+    end
+  end
   unless html.include?('application/ld+json') && html.include?('"@type": "Person"')
     warn "FAIL test-examples-build: #{name} home is missing JSON-LD"
     exit 1
@@ -99,6 +186,9 @@ end
     warn "FAIL test-examples-build: #{name} home dropped the title face"
     exit 1
   end
+ensure
+  redirects_path.write(original_redirects) if original_redirects
+end
 end
 
 puts "PASS test-examples-build"

data/scripts/test/test-hero-portraits.mjs

@@ -13,14 +13,24 @@ const source = fs.readFileSync(
 );
 
 const start = source.indexOf('function heroPortraitEntries');
-const end = source.indexOf('function applyHeroPortrait(');
+// applyHeroPortraitSources is defined before pickHeroPortrait. Ending at
+// applyHeroPortrait( still works today, and it is one character away from
+// stopping on the Sources function. showHeroPortrait is the next function
+// after the picker, so the slice always includes pickHeroPortrait.
+const end = source.indexOf('function showHeroPortrait');
 if (start < 0 || end < start) {
   console.error('hero portrait functions missing from 60-fragments.html');
   process.exit(1);
 }
 
+const slice = source.slice(start, end);
+if (!slice.includes('function pickHeroPortrait')) {
+  console.error('hero portrait crop stopped before pickHeroPortrait');
+  process.exit(1);
+}
+
 const api = new Function(
-  `${source.slice(start, end)}\nreturn { heroPortraitEntries, pickHeroPortrait };`
+  `${slice}\nreturn { heroPortraitEntries, pickHeroPortrait };`
 )();
 
 const { heroPortraitEntries, pickHeroPortrait } = api;

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

@@ -37,13 +37,19 @@ puts "Testing home hero portraits..."
 
 # ── AC-HERO-01 ──────────────────────────────────────────────────────────────
 
-check!(POOL.directory?, "missing #{POOL}")
 check!(!ROOT.join("content/pages/home/portraits.yml").exist?,
        "the author does not keep a portrait list")
 
-pool_names = POOL.children.select(&:file?).map { |path| path.basename.to_s }.sort
-check!(pool_names == %w[DSC00571.JPG DSC00632.JPG DSC00721.JPG DSC00727.JPG],
-       "the hero folder must hold the four uploaded photographs, got #{pool_names.inspect}")
+# The four photographs live with the site, not in the gem. When they are
+# here, the folder must be exactly that set. When they are not, the picker
+# extracted from the runtime still has to run.
+personal_pool = %w[DSC00571.JPG DSC00632.JPG DSC00721.JPG DSC00727.JPG]
+pool_names = POOL.directory? ? POOL.children.select(&:file?).map { |path| path.basename.to_s }.sort : []
+if pool_names.empty?
+  puts "AC-HERO-01: no hero photographs in this checkout"
+elsif pool_names != personal_pool
+  check!(false, "the hero folder must hold the four uploaded photographs, got #{pool_names.inspect}")
+end
 
 Dir.mktmpdir("hero-pool") do |dir|
   root = Pathname.new(dir)
@@ -135,6 +141,7 @@ check!(status.success?, "picker cases failed")
 
 # ── AC-HERO-02 ──────────────────────────────────────────────────────────────
 
+if pool_names == personal_pool
 Dir.mktmpdir("hero-portraits") do |dir|
   out = Pathname.new(dir).join("export")
   env = { "CONTENT_API_BASE_URL" => "", "CONTENT_EXPORT_SKIP_VALIDATE" => "1" }
@@ -158,5 +165,6 @@ Dir.mktmpdir("hero-portraits") do |dir|
   check!(listed.include?("pages/home/portraits.json"),
          "manifest page_headers must include the portrait list, got #{listed.inspect}")
 end
+end
 
 puts "All home hero portrait checks passed."

data/scripts/test/test-lighthouse-policy.rb

@@ -101,6 +101,10 @@ fail!("band note is still painted by script") unless band.include?("page_headers
 fail!("project inventory has a name and no role") unless band.include?('role="region"') && band.include?('aria-label="Project inventory"')
 headers = File.read(File.expand_path("../../_plugins/content_headers.rb", __dir__))
 fail!("page headers are not loaded at build time") unless headers.include?('site.data["page_headers"]')
+page_header = File.read(File.expand_path("../../_includes/page/page-header.html", __dir__))
+fail!("listing h1 is still Loading...") if page_header.include?("Loading...")
+fail!("listing h1 does not read page headers") unless page_header.include?("page_headers")
+fail!("listing h1 is not marked static") unless page_header.include?("data-content-static")
 home = File.read(File.expand_path("../../_layouts/home.html", __dir__))
 fail!("hero subtitle is still painted by script") unless home.include?("page_headers.home") && home.include?("data-content-static")
 

data/scripts/test/test-shell-a11y.rb

@@ -27,6 +27,12 @@ fail!("footer home link still uses site.author alone") if FOOTER.include?(bare)
 fail!("shell home link needs an identity.name fallback") unless SHELL.include?("site.pandorga.identity.name")
 fail!("footer home link needs an identity.name fallback") unless FOOTER.include?("site.pandorga.identity.name")
 
+fail!("top bar is still a nav") if SHELL.include?('<nav class="sticky')
+fail!("top bar is not a header") unless SHELL.include?('<header class="sticky top-0 z-50 bg-surface">')
+fail!("section links lost their name") unless SHELL.include?('aria-label="Sections"')
+navs = SHELL.scan(/<nav\b/).size
+fail!("shell should have one nav landmark, found #{navs}") unless navs == 1
+
 # Decorative Material Symbols sit beside a text label. Without aria-hidden
 # the ligature ("arrow_left_alt", "shuffle", an icon name) is read aloud.
 ICON_FILES = [

data/scripts/test/test-theme-contrast.rb

@@ -0,0 +1,43 @@
+#!/usr/bin/env ruby
+# frozen_string_literal: true
+
+# Dark tertiary container text must clear WCAG AA for normal text.
+# PLT-AC-33
+
+require "pathname"
+require "yaml"
+
+ROOT = Pathname.new(__dir__).join("..", "..").expand_path
+
+def fail!(message)
+  warn "FAIL test-theme-contrast: #{message}"
+  exit 1
+end
+
+def channel(value)
+  value = value / 255.0
+  value <= 0.04045 ? value / 12.92 : ((value + 0.055) / 1.055)**2.4
+end
+
+def luminance(hex)
+  rgb = hex.delete("#").scan(/../).map { |part| part.to_i(16) }
+  fail!("not a hex colour: #{hex}") unless rgb.length == 3
+
+  (0.2126 * channel(rgb[0])) + (0.7152 * channel(rgb[1])) + (0.0722 * channel(rgb[2]))
+end
+
+def contrast(foreground, background)
+  lighter, darker = [luminance(foreground), luminance(background)].sort.reverse
+  (lighter + 0.05) / (darker + 0.05)
+end
+
+dark = YAML.safe_load(ROOT.join("_data/themes/dark.yml").read) || {}
+colors = dark["colors"] || {}
+background = colors["tertiary_container"].to_s
+foreground = colors["on_tertiary_container"].to_s
+fail!("dark tertiary container moved") unless background.casecmp("#6a684f").zero?
+
+ratio = contrast(foreground, background)
+fail!("#{foreground} on #{background} is #{ratio.round(2)}:1") unless ratio >= 4.5
+
+puts "PASS test-theme-contrast (#{foreground} on #{background} = #{ratio.round(2)}:1)"

data/scripts/validate.sh

@@ -57,6 +57,7 @@ TESTS=(
   scripts/test/test-examples-build.rb
   scripts/test/test-validate-allowlist.rb
   scripts/test/test-shell-a11y.rb
+  scripts/test/test-theme-contrast.rb
   scripts/test/test-hydrate-reserve.rb
   scripts/test/test-hero-summary.rb
   scripts/test/test-lighthouse-policy.rb
@@ -82,6 +83,11 @@ for test in "${TESTS[@]}"; do
   fi
 done
 
+echo "-- scripts/test/test-hero-portraits.mjs"
+if ! node scripts/test/test-hero-portraits.mjs; then
+  failed=1
+fi
+
 echo "-- scripts/test/test-studio-pipeline-config.mjs"
 if ! node scripts/test/test-studio-pipeline-config.mjs; then
   failed=1

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pandorga
 version: !ruby/object:Gem::Version
-  version: 1.7.1
+  version: 1.7.2
 platform: ruby
 authors:
 - The Pandorga contributors
@@ -287,6 +287,7 @@ files:
 - lib/pandorga/commands/publish.rb
 - lib/pandorga/commands/serve.rb
 - lib/pandorga/commands/studio_schema.rb
+- lib/pandorga/detail_routes.rb
 - lib/pandorga/discovery.rb
 - lib/pandorga/fonts.rb
 - lib/pandorga/hero_summary.rb
@@ -303,6 +304,7 @@ files:
 - lib/pandorga/jekyll/plugins/content_validators_hook.rb
 - lib/pandorga/jekyll/plugins/content_writing_index.rb
 - lib/pandorga/jekyll/plugins/content_writing_paths.rb
+- lib/pandorga/jekyll/plugins/detail_documents.rb
 - lib/pandorga/jekyll/plugins/discovery.rb
 - lib/pandorga/jekyll/plugins/fonts.rb
 - lib/pandorga/jekyll/plugins/force_utf8_encoding.rb
@@ -466,6 +468,7 @@ files:
 - scripts/test/test-studio-slug-config.rb
 - scripts/test/test-template-packages.rb
 - scripts/test/test-text-excerpt.rb
+- scripts/test/test-theme-contrast.rb
 - scripts/test/test-unwrap-media-src.mjs
 - scripts/test/test-unwrap-media-src.rb
 - scripts/test/test-validate-allowlist.rb