@port60/template-kit 0.9.0 → 0.11.0

package/src/vendor/validator/validate.mjs

@@ -22,6 +22,7 @@ import contextContract from '../contract/v1/context.json' with { type: 'json' };
 import fontCatalogue from '../contract/v1/fonts.json' with { type: 'json' };
 import layoutContract from '../contract/v1/layout.json' with { type: 'json' };
 import behaviourCatalogue from '../contract/v1/behaviours.json' with { type: 'json' };
+import { buildSiteFixture, extractContentFootprint, contentModel } from './site-context.mjs';
 
 const Ajv = Ajv2020.default ?? Ajv2020;
 
@@ -55,6 +56,18 @@ export async function validateArtifact(files) {
   } catch (e) {
     return { errors: [`manifest.json unreadable: ${e.message}`], warnings, manifest: null };
   }
+  // preview-content.json is a DEV-ONLY data override: package excludes it and the intake
+  // refuses it — an artifact must never carry data, only shape.
+  if (has('preview-content.json')) {
+    errors.push('preview-content.json: development preview data never ships in an artifact — remove it (package excludes it automatically)');
+  }
+
+  // Content model v1: the footprint is decidable from the sources (closed dialect); unknown
+  // paths, dynamic indexing and tree aliasing are errors from site-context.mjs.
+  const contentAnalysis = extractContentFootprint(files);
+  errors.push(...contentAnalysis.errors);
+  const siteFx = buildSiteFixture(manifest);
+
   const ajv = new Ajv({ allErrors: true });
   const validateManifest = ajv.compile(manifestSchema);
   if (!validateManifest(manifest)) {
@@ -81,14 +94,12 @@ export async function validateArtifact(files) {
       events: () => sections.has('events') || pages.has('events') || islands.has('events_carousel'),
       articles: () => sections.has('articles') || hasAny(pages, ['articles', 'article']) || islands.has('latest_articles'),
       services: () => sections.has('programmes'),
-      volunteering: () => sections.has('volunteering') || islands.has('volunteer_signup'),
       forms: () => islands.has('form'),
       resources: () => sections.has('resources'),
       locations: () => sections.has('locations'),
       newsletter: () => islands.has('newsletter_signup'),
       i18n: () => islands.has('language_switch'),
-      search: () => islands.has('search'),
-      media: () => sections.has('media')
+      search: () => islands.has('search')
     };
     for (const capability of manifest?.requiresCapabilities ?? []) {
       if (!capabilitySurface[capability]?.()) {
@@ -212,15 +223,18 @@ export async function validateArtifact(files) {
   // the data itself — in which case both directions are proven behaviourally (the worship
   // pattern): the populated fixture's sentinel must appear, and the EMPTY context must render it
   // away (derive or omit — nothing invented, nothing dangling).
+  // Sentinels are DERIVED from the canonical fixtures (the first item's display field), never
+  // hardcoded — the fixture data is free to become richer without touching a proof, and a proof
+  // can never drift from the data it renders.
+  const sentinelOf = (type, dataKey, field) =>
+    (contextContract.fixtures.sections?.[type]?.[dataKey] ?? [])[0]?.[field] ?? null;
   const WIDGET_SECTIONS = {
-    events: { island: 'events_carousel', dataKey: 'events', sentinel: 'Fixture Lantern Gala' },
-    whatsOn: { island: 'whats_on_strip', dataKey: 'infoEvents', sentinel: 'Fixture Open Morning' },
-    articles: { island: 'latest_articles', dataKey: 'latestArticles', sentinel: 'Fixture Winter Diary' },
-    campaigns: { island: null, dataKey: 'campaigns', sentinel: 'Fixture Winter Campaign' },
-    resources: { island: null, dataKey: 'resources', sentinel: 'Fixture annual report.pdf' },
-    locations: { island: null, dataKey: 'locations', sentinel: 'Fixture Community Hall' },
-    volunteering: { island: 'volunteer_signup', dataKey: 'opportunities', sentinel: 'Fixture Garden Volunteer' },
-    media: { island: null, dataKey: 'media', sentinel: 'Fixture Community Talk' }
+    events: { island: 'events_carousel', dataKey: 'events', sentinel: sentinelOf('events', 'events', 'name') },
+    whatsOn: { island: 'whats_on_strip', dataKey: 'infoEvents', sentinel: sentinelOf('whatsOn', 'infoEvents', 'name') },
+    articles: { island: 'latest_articles', dataKey: 'latestArticles', sentinel: sentinelOf('articles', 'latestArticles', 'title') },
+    campaigns: { island: null, dataKey: 'campaigns', sentinel: sentinelOf('campaigns', 'campaigns', 'title') },
+    resources: { island: null, dataKey: 'resources', sentinel: sentinelOf('resources', 'resources', 'title') },
+    locations: { island: null, dataKey: 'locations', sentinel: sentinelOf('locations', 'locations', 'name') }
   };
 
   // 2–4. Sections: catalogue membership, parse, fixture renders.
@@ -251,12 +265,13 @@ export async function validateArtifact(files) {
         const populated = await liquid.render(parsed, {
           section: {},
           brand: contextContract.fixtures.brand,
+          site: siteFx,
           ...dataFixture
         });
         const placesIsland = widget.island !== null
           && splitIslandParts(populated).some((p) => p.island === widget.island);
         if (!placesIsland) {
-          if (!populated.includes(widget.sentinel)) {
+          if (widget.sentinel && !populated.includes(widget.sentinel)) {
             errors.push(
               widget.island
                 ? `section '${type}': neither places the ${widget.island} island nor renders the ${widget.dataKey} context — render the data (the fixture's "${widget.sentinel}" must appear) or place the island`
@@ -266,9 +281,10 @@ export async function validateArtifact(files) {
           const empty = await liquid.render(parsed, {
             section: {},
             brand: contextContract.fixtures.brand,
+            site: { ...siteFx, content: Object.fromEntries(Object.keys(siteFx.content).map((k) => [k, []])) },
             [widget.dataKey]: []
           });
-          if (empty.includes(widget.sentinel)) {
+          if (widget.sentinel && empty.includes(widget.sentinel)) {
             errors.push(`section '${type}': still shows fixture content with an empty ${widget.dataKey} — content must come from the context`);
           }
           if (/\bundefined\b|\bnull\b/.test(empty.replace(/data-[a-z-]+="[^"]*"/g, ''))) {
@@ -285,19 +301,23 @@ export async function validateArtifact(files) {
         const html = await liquid.render(parsed, {
           section: fixture,
           brand: contextContract.fixtures.brand,
+          site: siteFx,
           // Widget data rides the ordinary fixture renders too, so a hand-rendering section
           // doesn't fail the generic pass for want of its context.
           ...(widget ? (contextContract.fixtures.sections?.[type] ?? {}) : {})
         });
         if (type === 'homeHero' && fixtureName === 'sample') {
-          const probe = 'Hero photo one';
+          // The sample's own first image reference is the probe: `p60fixture:` refs are quote-free
+          // and survive HTML escaping, so "does the hero display the photos?" stays a substring check.
+          const probe = (fixture.images ?? [])[0]?.imageUrl ?? 'p60fixture:';
           homeHeroMultiShows = html.includes(probe)
             || splitIslandParts(html).some((p) => p.island === 'hero_carousel');
           // The single-photo path proven separately: same fixture, first photo only.
           try {
             const single = await liquid.render(parsed, {
               section: { ...fixture, images: (fixture.images ?? []).slice(0, 1) },
-              brand: contextContract.fixtures.brand
+              brand: contextContract.fixtures.brand,
+              site: siteFx
             });
             homeHeroSingleShows = single.includes(probe);
           } catch {
@@ -359,6 +379,7 @@ export async function validateArtifact(files) {
       if (parsedLayout) {
         try {
           const html = await liquid.render(parsedLayout, {
+            site: siteFx,
             brand: contextContract.fixtures.brand,
             nav: contextContract.fixtures.layout.nav,
             socials: contextContract.fixtures.layout.socials ?? [],
@@ -394,10 +415,12 @@ export async function validateArtifact(files) {
             }
             if (manifest?.supports?.worship) {
               const nullHtml = await liquid.render(parsedLayout, {
+                site: siteFx,
                 brand: contextContract.fixtures.brand,
                 nav: contextContract.fixtures.layout.nav,
                 socials: contextContract.fixtures.layout.socials ?? [],
                 worship: null,
+                site: { ...siteFx, content: { ...siteFx.content, schedules: [] } },
                 locale: contextContract.fixtures.layout.locale
               });
               if (nullHtml.includes(worshipProbe)) {
@@ -434,7 +457,7 @@ export async function validateArtifact(files) {
       continue;
     }
     try {
-      const html = await liquid.render(parsedPage, { ...fixture, brand: contextContract.fixtures.brand });
+      const html = await liquid.render(parsedPage, { ...fixture, site: siteFx, brand: contextContract.fixtures.brand });
       for (const part of splitIslandParts(html)) {
         if (part.island === CONTENT_SLOT) {
           errors.push(`page template '${pageName}': uses {% content %} — that tag is layout-only`);
@@ -495,5 +518,35 @@ export async function validateArtifact(files) {
     }
   }
 
-  return { errors, warnings, manifest };
+  // Open enums, proven survivable (content model v1 discipline 2): a template whose footprint
+  // reads site.content.events must survive a registration mode it has never heard of — new modes
+  // WILL arrive within the major. No throw, and no undefined/null literal leaking into markup.
+  if (contentAnalysis.footprint.includes('content.events')) {
+    const futureEvent = {
+      ...(siteFx.content.events[0] ?? {}),
+      id: 'ffffffff-ffff-4fff-8fff-ffffffffffff',
+      name: 'Open Enum Probe Event',
+      registrationMode: 'X_FUTURE_MODE',
+      detailHref: '/events?event=ffffffff-ffff-4fff-8fff-ffffffffffff'
+    };
+    const doctored = { ...siteFx, content: { ...siteFx.content, events: [...siteFx.content.events, futureEvent] } };
+    for (const type of manifest?.supports?.sections ?? []) {
+      const file = `sections/${type}.liquid`;
+      if (!has(file) || !/\bsite\s*[.[]/.test(read(file))) continue;
+      try {
+        const html = await liquid.render(liquid.parse(read(file)), {
+          section: catalogueByType.get(type)?.sample ?? {},
+          brand: contextContract.fixtures.brand,
+          site: doctored
+        });
+        if (/\bundefined\b|\bnull\b/.test(html.replace(/data-[a-z-]+="[^"]*"/g, ''))) {
+          errors.push(`section '${type}': renders 'undefined'/'null' literals for an unknown event registrationMode — the enum is OPEN, branch on the modes you style and fall back for the rest`);
+        }
+      } catch (e) {
+        errors.push(`section '${type}': failed rendering an unknown event registrationMode — the enum is OPEN and new modes will arrive (${e.message})`);
+      }
+    }
+  }
+
+  return { errors, warnings, manifest, contentFootprint: contentAnalysis.footprint, minContentVersion: contentAnalysis.minModelVersion };
 }

package/starter/assets/theme.css

@@ -1,4 +1,4 @@
-/* starter theme — a deliberately distinct look (midnight/emerald) proving the artifact owns the skin */
+/* starter theme, a deliberately distinct look (midnight/emerald) proving the artifact owns the skin */
 :root:root {
   --bg: #0e1614;
   --bg-tint: #14201d;
@@ -39,13 +39,13 @@ strong { color: var(--ink); }
 .lq-cta-grid { display: grid; grid-template-columns: 1fr 1fr; gap: 2.5rem; align-items: start; }
 @media (max-width: 900px) { .lq-cta-grid { grid-template-columns: 1fr; } }
 
-/* The one font slot — headings above already follow it; body text too. */
+/* The one font slot, headings above already follow it; body text too. */
 body { font-family: ' IBM Plex Sans Arabic', var(--p60s-siteFont, 'Inter'), ui-sans-serif, system-ui, sans-serif; }
 
-/* ── Home hero (supports.heroImagery — the reference treatment) ─────────────────────────────
+/* ── Home hero (supports.heroImagery, the reference treatment) ─────────────────────────────
    The no-photo state is a DESIGNED gradient (rule: derive or omit, never a placeholder image).
    Photos get a scrim built from the palette itself via color-mix, so uploads tone into the
-   template — and any palette change re-tones them for free. Never place a hero photo raw. */
+   template, and any palette change re-tones them for free. Never place a hero photo raw. */
 .lq-homehero {
   position: relative;
   padding: 4.5rem 0;
@@ -73,7 +73,7 @@ body { font-family: ' IBM Plex Sans Arabic', var(--p60s-siteFont, 'Inter'), ui-s
 .lq-homehero-lead { color: var(--muted); max-width: 46ch; margin: 0.9rem 0 1.4rem; }
 .lq-homehero-actions { display: flex; gap: 0.8rem; flex-wrap: wrap; }
 
-/* The platform hero_carousel island, styled through its class API — the island owns motion
+/* The platform hero_carousel island, styled through its class API, the island owns motion
    (auto-advance, swipe, reduced-motion), this owns the look. Each slide's .hero-slide-scrim is
    the same palette treatment as a single photo. */
 .lq-homehero--carousel { overflow: hidden; }
@@ -114,7 +114,7 @@ body { font-family: ' IBM Plex Sans Arabic', var(--p60s-siteFont, 'Inter'), ui-s
 }
 .lq-homehero--carousel .hero-dot--active { background: var(--accent); }
 
-/* ── Vista framing (photoFraming 'whole') — the hero adapts to the PHOTO's shape ───────────── */
+/* ── Vista framing (photoFraming 'whole'), the hero adapts to the PHOTO's shape ───────────── */
 .lq-homehero--vista { padding-top: 0; }
 .lq-vista { position: relative; overflow: hidden; }
 .lq-vista-sizer { display: block; width: 100%; height: auto; max-height: 74vh; object-fit: cover; }
@@ -153,7 +153,7 @@ body { font-family: ' IBM Plex Sans Arabic', var(--p60s-siteFont, 'Inter'), ui-s
   .lq-vista-sizer { min-height: 160px; }
 }
 
-/* Vista + form (1.5.2): the widget rides the seam — grid pulled up over the band (under the
+/* Vista + form (1.5.2): the widget rides the seam, grid pulled up over the band (under the
    280px floor), gradient filling where the image ends; narrow screens stack back underneath. */
 .lq-homehero--vista-form { padding-bottom: 2.6rem; }
 .lq-vista-overlap { position: relative; z-index: 1; margin-top: -240px; align-items: start; }
@@ -161,7 +161,7 @@ body { font-family: ' IBM Plex Sans Arabic', var(--p60s-siteFont, 'Inter'), ui-s
   .lq-vista-overlap { margin-top: 0; padding-top: 1.6rem; }
 }
 
-/* People / team cards — grid to single column on phones. */
+/* People / team cards, grid to single column on phones. */
 .lq-people {
   display: grid;
   grid-template-columns: repeat(auto-fit, minmax(220px, 1fr));
@@ -188,13 +188,13 @@ body { font-family: ' IBM Plex Sans Arabic', var(--p60s-siteFont, 'Inter'), ui-s
   .lq-people { grid-template-columns: 1fr; }
 }
 
-/* Campaigns — the same quiet rail as values: title, summary, one action. */
+/* Campaigns, the same quiet rail as values: title, summary, one action. */
 .lq-campaigns { padding: 3.5rem 0; }
 .lq-campaign-list { list-style: none; margin: 0; padding: 0; display: grid; gap: 1.6rem; max-width: 700px; }
 .lq-campaign h3 { margin: 0 0 0.25rem; }
 .lq-campaign p { margin: 0 0 0.6rem; }
 
-/* Behaviours (engine-wired via data-p60-* — docs: Motion and behaviour guide). Everything that
+/* Behaviours (engine-wired via data-p60-*, docs: Motion and behaviour guide). Everything that
    hides pre-behaviour content is .p60-js-scoped: the no-JS render stays complete. */
 
 /* Reveal: the values rail rises in with a stagger. */

package/starter/layout.liquid

@@ -1,22 +1,21 @@
 {% comment %}
-  The starter's chrome — header/nav/footer around the {% content %} slot. It reuses the platform
+  The starter's chrome, header/nav/footer around the {% content %} slot. It reuses the platform
   chrome classes (.site-header/.site-nav/…) so the base mechanics (sticky header, burger
   breakpoint, dropdown hover) apply, and emits the documented behaviour attributes
   (data-nav-burger, #site-nav) so the platform tail script wires the mobile burger. Identity is
-  PLACED, never rebuilt: {% island 'member_menu' %}. The footer derives everything from brand/nav —
-  no fabricated copy (derive or omit).
+  PLACED, never rebuilt: {% island 'member_menu' %}. The footer derives everything from brand/nav, no fabricated copy (derive or omit).
 {% endcomment %}
 <header class="site-header">
   <div class="container header-inner">
     <a class="brand-lockup" href="/">
-      {% if brand.logoUrl and brand.logoType != 'icon' %}
-        <img class="brand-logo" src="{{ brand.logoUrl }}" alt="{{ brand.name }}">
-      {% elsif brand.logoUrl %}
-        <img class="brand-logo-icon" src="{{ brand.logoUrl }}" alt="">
-        <strong>{{ brand.name }}</strong>
+      {% if site.brand.logoUrl and site.brand.logoType != 'icon' %}
+        <img class="brand-logo" src="{{ site.brand.logoUrl }}" alt="{{ site.brand.name }}">
+      {% elsif site.brand.logoUrl %}
+        <img class="brand-logo-icon" src="{{ site.brand.logoUrl }}" alt="">
+        <strong>{{ site.brand.name }}</strong>
       {% else %}
         <span class="brand-mark" aria-hidden="true">♥</span>
-        <strong>{{ brand.name }}</strong>
+        <strong>{{ site.brand.name }}</strong>
       {% endif %}
     </a>
 
@@ -27,7 +26,7 @@
     </button>
 
     <nav class="site-nav" id="site-nav" aria-label="Primary">
-      {% for item in nav.items %}
+      {% for item in site.nav.items %}
         {% if item.children %}
           <span class="nav-dropdown">
             <a href="{{ item.href }}" class="nav-dropdown-toggle" aria-haspopup="true">{{ item.label }} ▾</a>
@@ -55,20 +54,20 @@
 <footer class="site-footer">
   <div class="container footer-grid">
     <div>
-      <p class="lq-kicker">{{ brand.name }}</p>
-      {% if brand.tagline %}<h2>{{ brand.tagline }}</h2>{% endif %}
+      <p class="lq-kicker">{{ site.brand.name }}</p>
+      {% if site.brand.tagline %}<h2>{{ site.brand.tagline }}</h2>{% endif %}
     </div>
     <div class="footer-links">
-      {% comment %} A tenant-curated footer menu when they've built one, else the main nav. {% endcomment %}
-      {% if nav.footer.size > 0 %}
-        {% for item in nav.footer %}
+      {% comment %} A tenant-curated footer menu when they've built one, else the main site.nav. {% endcomment %}
+      {% if site.nav.footer.size > 0 %}
+        {% for item in site.nav.footer %}
           <a href="{{ item.href }}"{% if item.external %} target="_blank" rel="noopener"{% endif %}>{{ item.label }}</a>
           {% for child in item.children %}
             <a href="{{ child.href }}"{% if child.external %} target="_blank" rel="noopener"{% endif %}>{{ child.label }}</a>
           {% endfor %}
         {% endfor %}
       {% else %}
-        {% for item in nav.items %}
+        {% for item in site.nav.items %}
           <a href="{{ item.href }}">{{ item.label }}</a>
         {% endfor %}
       {% endif %}

package/starter/manifest.json

@@ -1,9 +1,9 @@
 {
   "name": "starter",
-  "version": "1.8.0",
+  "version": "1.9.0",
   "format": "port60-liquid@1",
   "label": "Starter",
-  "description": "The reference template for the Port60 dialect \u2014 the developer docs' worked example and the base to copy when building your own. 1.1.0 adds the layout slot: the template owns the header, navigation and footer chrome.",
+  "description": "The reference template for the Port60 dialect, the developer docs' worked example and the base to copy when building your own. 1.1.0 adds the layout slot: the template owns the header, navigation and footer chrome.",
   "supports": {
     "pages": [
       "home",

package/starter/sections/campaigns.liquid

@@ -1,16 +1,16 @@
 {% comment %}
-  Live campaigns — the starter's plain rendering of the `campaigns` context, kept simple on
+  Live site.content.campaigns, the starter's plain rendering of the `site.content.campaigns` context, kept simple on
   purpose: this template is the worked example template authors copy from. The context is the
-  platform's published campaigns; hrefs are platform-derived — always link via campaign.href.
+  platform's published site.content.campaigns; hrefs are platform-derived, always link via campaign.href.
   Derive or omit: nothing renders when no campaign is live.
 {% endcomment %}
-{% if campaigns.size > 0 %}
-<section class="lq-campaigns">
+{% if site.content.campaigns.size > 0 %}
+<section class="lq-site.content.campaigns">
   <div class="container">
     <h2>{{ section.heading | default: campaignsLabel }}</h2>
     {% if section.text %}<p>{{ section.text }}</p>{% endif %}
     <ul class="lq-campaign-list">
-      {% for campaign in campaigns %}
+      {% for campaign in site.content.campaigns %}
       <li class="lq-campaign">
         <h3>{{ campaign.title }}</h3>
         {% if campaign.summary %}<p>{{ campaign.summary }}</p>{% endif %}

package/starter/sections/homeHero.liquid

@@ -1,16 +1,16 @@
 {% comment %}
-  The home opener — and the REFERENCE IMPLEMENTATION of supports.heroImagery. Rules worth
+  The home opener, and the REFERENCE IMPLEMENTATION of supports.heroImagery. Rules worth
   copying:
 
   1. EVERY field is optional. Copy derives from brand (| default) or omits; no photos means the
-     designed gradient below — never a placeholder image.
+     designed gradient below, never a placeholder image.
   2. Photos are NEVER placed raw. The treatments in theme.css lay scrims built from the palette
      variables (color-mix over var(--bg)/var(--bg-tint)), so any uploaded photograph is toned
      into the template. Tune the treatment, don't remove it.
   3. Rotation is the PLATFORM's hero_carousel island (templates ship no JavaScript): with two or
      more photos, place it and style its documented classes.
   4. photoFraming is the charity's call, honour BOTH: 'fill' (default) covers your designed hero
-     shape (per-photo focus picks the surviving slice); 'whole' is the VISTA — the hero adapts
+     shape (per-photo focus picks the surviving slice); 'whole' is the VISTA, the hero adapts
      toward the photo's own shape (height-clamped) so a wide landscape stays entirely visible,
      with your copy stacked after the band.
 {% endcomment %}
@@ -38,8 +38,8 @@
     {% if heroCount > 1 %}{% island 'hero_carousel' %}{% else %}<div class="lq-vista-scrim" aria-hidden="true"></div>{% endif %}
     <div class="lq-vista-copy">
       <div class="container">
-        <p class="lq-kicker">{{ section.eyebrow | default: brand.name }}</p>
-        <h1>{{ section.title | default: brand.tagline | default: brand.name }}</h1>
+        <p class="lq-kicker">{{ section.eyebrow | default: site.brand.name }}</p>
+        <h1>{{ section.title | default: site.brand.tagline | default: site.brand.name }}</h1>
         {% if section.lead %}<p class="lq-homehero-lead">{{ section.lead }}</p>{% endif %}
         <div class="lq-homehero-actions">
           <a class="button button-primary" href="{{ section.primaryHref | default: '/donate' }}">{{ section.primaryLabel | default: 'Donate now' }}</a>
@@ -50,7 +50,7 @@
   </div>
 </section>
 {% else %}
-{% comment %} Form mode: the grid pulls up over the band — copy on the photo, the widget riding
+{% comment %} Form mode: the grid pulls up over the band, copy on the photo, the widget riding
     the seam onto the gradient below (never a stacked strip). {% endcomment %}
 <section class="lq-homehero lq-homehero--vista lq-homehero--vista-form{% if heroCount > 1 %} lq-homehero--carousel{% endif %}">
   <div class="lq-vista">
@@ -59,8 +59,8 @@
   </div>
   <div class="container lq-homehero-inner lq-vista-overlap">
     <div>
-      <p class="lq-kicker">{{ section.eyebrow | default: brand.name }}</p>
-      <h1>{{ section.title | default: brand.tagline | default: brand.name }}</h1>
+      <p class="lq-kicker">{{ section.eyebrow | default: site.brand.name }}</p>
+      <h1>{{ section.title | default: site.brand.tagline | default: site.brand.name }}</h1>
       {% if section.lead %}<p class="lq-homehero-lead">{{ section.lead }}</p>{% endif %}
       <div class="lq-homehero-actions">
         <a class="button button-primary" href="#donate">{{ section.primaryLabel | default: 'Donate now' }}</a>
@@ -76,7 +76,7 @@
 {% else %}
 <section class="lq-homehero{% if heroPhoto %} lq-homehero--photo{% endif %}{% if heroCount > 1 %} lq-homehero--carousel{% endif %}"{% if heroPhoto %} style="background-image: url('{{ heroPhoto }}'); background-position: {{ heroFocus }}"{% endif %}>
   {% comment %}
-    Several photos → the BEHAVIOUR carousel (data-p60-carousel, engine-wired — the template ships
+    Several photos → the BEHAVIOUR carousel (data-p60-carousel, engine-wired, the template ships
     no JavaScript): complete static markup, one data-p60-slide per photo, auto-advance opt-in.
     Hiding non-active slides is .p60-js-scoped in theme.css so a no-JS visitor sees the photos
     stacked, never a blank hero. The vista framings above show the other route: the ready-made
@@ -93,8 +93,8 @@
   {% endif %}
   <div class="container lq-homehero-inner">
     <div>
-      <p class="lq-kicker">{{ section.eyebrow | default: brand.name }}</p>
-      <h1>{{ section.title | default: brand.tagline | default: brand.name }}</h1>
+      <p class="lq-kicker">{{ section.eyebrow | default: site.brand.name }}</p>
+      <h1>{{ section.title | default: site.brand.tagline | default: site.brand.name }}</h1>
       {% if section.lead %}<p class="lq-homehero-lead">{{ section.lead }}</p>{% endif %}
       <div class="lq-homehero-actions">
         {% if section.givingStyle == 'button' %}

package/starter/sections/people.liquid

@@ -1,5 +1,5 @@
 {% comment %}
-  People / team — the reference implementation: simple cards (name, role, optional photo and
+  People / team, the reference implementation: simple cards (name, role, optional photo and
   one-line statement), a grid that collapses to one column on phones. No placeholder faces:
   a person without a photo just renders text (derive or omit, never fabricate).
 {% endcomment %}

package/starter/sections/values.liquid

@@ -3,7 +3,7 @@
     <h2>{{ section.heading | default: "What guides us" }}</h2>
     {% comment %}
       data-p60-reveal-group: the engine stamps --p60-reveal-index on each child and adds
-      is-revealed when the rail scrolls into view — the stagger and motion are OURS, in theme.css,
+      is-revealed when the rail scrolls into view, the stagger and motion are OURS, in theme.css,
       and .p60-js-scoped so a no-JS visitor sees the rail complete and still.
     {% endcomment %}
     <ol class="lq-value-rail" data-p60-reveal-group>