From 5c527a809d252a45eca7028338442260b54b5d64 Mon Sep 17 00:00:00 2001
From: Cyborg Viktor <307551610+vpetersson-bot@users.noreply.github.com>
Date: Wed, 5 Aug 2026 18:45:04 +0000
Subject: [PATCH] Correct the deploy docs and drop the remaining em dashes

Two doc/copy cleanups, no behaviour change.

README.md already described the tag-driven deploy correctly (#18), but
CLAUDE.md still said "Pushing to master runs deploy-pages.yml" and the
workflow's own header comment still said it publishes "on every push to
master". Neither is true: the trigger is a CalVer tag (or a manual
workflow_dispatch). Both now match the workflow, and README's CalVer
example is spelled YYYY.M.PATCH to match the tags actually in use.

Also removes the remaining em dashes (55 across CLAUDE.md, README.md,
main.css, build.ts, dev-server.ts, src/clocks.ts, src/main.ts and
sync-fonts.ts), rewriting each to a colon, semicolon, comma or
parenthetical as the sentence needs rather than swapping the character
blindly. In the README parameter table the em dash was doing duty as the
"no default" value, so it now reads "none". The manifest and meta
description were already done in #19.

lint, typecheck, test (35 pass) and build all clean.
---
 .github/workflows/deploy-pages.yml |  8 +++---
 CLAUDE.md                          | 27 +++++++++++----------
 README.md                          | 18 +++++++-------
 assets/styles/main.css             | 39 +++++++++++++++---------------
 build.ts                           |  2 +-
 dev-server.ts                      |  6 ++---
 src/clocks.ts                      | 12 ++++-----
 src/main.ts                        | 16 ++++++------
 sync-fonts.ts                      |  2 +-
 9 files changed, 66 insertions(+), 64 deletions(-)

diff --git a/.github/workflows/deploy-pages.yml b/.github/workflows/deploy-pages.yml
index 6141c4e..ef1975b 100644
--- a/.github/workflows/deploy-pages.yml
+++ b/.github/workflows/deploy-pages.yml
@@ -1,9 +1,9 @@
 ---
 name: Deploy to GitHub Pages
 
-# Builds the static site and publishes ./dist to GitHub Pages on every push to
-# master. One-time setup: repo Settings → Pages → Build and deployment → Source
-# = "GitHub Actions".
+# Builds the static site and publishes ./dist to GitHub Pages when a CalVer tag
+# (e.g. 2026.8.0) is pushed; a push to master alone does not deploy. One-time
+# setup: repo Settings → Pages → Build and deployment → Source = "GitHub Actions".
 
 on:
   push:
@@ -32,7 +32,7 @@ jobs:
         with:
           bun-version: 1.3.14
       - run: bun install --frozen-lockfile
-      # Gate the deploy on the same checks CI runs, so master never ships red.
+      # Gate the deploy on the same checks CI runs, so a tag never ships red.
       - run: bun run lint
       - run: bun run typecheck
       - run: bun test
diff --git a/CLAUDE.md b/CLAUDE.md
index 90221fb..b255a2a 100644
--- a/CLAUDE.md
+++ b/CLAUDE.md
@@ -6,11 +6,11 @@ This file provides guidance to Claude Code (claude.ai/code) when working with co
 
 A **fully static** world clock for digital signage: a board of cities, each
 showing its own local time, date, GMT offset and a day/night accent. Everything
-on screen is driven by **URL query parameters** — there is no server and no
+on screen is driven by **URL query parameters**: there is no server and no
 backend. The site builds to `./dist` and is published to **GitHub Pages**.
 
 It is modelled loosely on the sibling [`../clock-app`](../clock-app) (the
-"Observatory" aesthetic — Fraunces serif over a night sky), but where clock-app
+"Observatory" aesthetic: Fraunces serif over a night sky), but where clock-app
 is a single edge-rendered Cloudflare Worker clock, this is a self-contained,
 multi-zone, client-only static site. All client code is **TypeScript**.
 
@@ -32,21 +32,22 @@ bun run sync-fonts # vendor webfonts from @fontsource only
 ```
 
 CI (`.github/workflows/ci.yml`) runs lint → typecheck → test → build on every PR.
-Pushing to `master` runs `deploy-pages.yml`, which re-runs those checks and
-publishes `./dist` to GitHub Pages.
+Deploys are tag-driven: pushing a CalVer tag (e.g. `2026.8.0`) runs
+`deploy-pages.yml`, which re-runs those checks and publishes `./dist` to GitHub
+Pages. Pushing to `master` on its own does not deploy.
 
 ## Architecture
 
 ### Pure logic vs. DOM (the split to preserve)
 
-- **`src/clocks.ts`** — pure, side-effect-free helpers: URL parsing
+- **`src/clocks.ts`** holds the pure, side-effect-free helpers: URL parsing
   (`parseClocks`), `Intl` formatter construction (`buildFormatters`), and the
   formatting/banding helpers. Every function takes an explicit `Date`, so tests
   are independent of the machine clock and timezone. This is the **only** file
   with unit tests (`src/clocks.test.ts`). Keep testable logic here.
-- **`src/main.ts`** — the DOM driver: reads `location.search`, builds one card
+- **`src/main.ts`** is the DOM driver: reads `location.search`, builds one card
   per clock, and repaints them on a shared timer. No `Intl`/parsing logic of its
-  own — it imports it from `clocks.ts`. Loaded as a `type="module"` script.
+  own; it imports it from `clocks.ts`. Loaded as a `type="module"` script.
 
 This mirrors clock-app's `locale.js` / `main.js` split, but here `main.ts` is a
 normal ES module (the static page uses `<script type="module">`), so there is no
@@ -67,7 +68,7 @@ project subpath (`https://<org>.github.io/world-clock/`) unchanged.
 
 `sync-fonts.ts` vendors the Bun-managed `@fontsource` woff2 files into
 `assets/fonts/` (run automatically by `build.ts`). **Only Latin fonts are
-vendored** — non-Latin locales (CJK, Arabic, …) fall back to the device's own
+vendored**; non-Latin locales (CJK, Arabic, …) fall back to the device's own
 fonts. The time digits are always Latin, so the hero always renders in Fraunces.
 
 ## URL parameters (the configuration surface)
@@ -81,7 +82,7 @@ fonts. The time digits are always Latin, so the hero always renders in Fraunces.
 | `seconds` | no         | Present (or `=1`) → tick every second; otherwise tick by the minute     |
 | `title`   | no         | Replaces the "World Clock" masthead label                              |
 
-**Locale is global, not per-clock** — a world clock shows every city in one
+**Locale is global, not per-clock**: a world clock shows every city in one
 language; only the timezone differs per card. Invalid IANA zones are dropped
 (not thrown), and an unsupported `locale` falls back to `en-GB`. With no clock
 params, `DEFAULT_CLOCKS` (a spread of major cities) is shown. See the
@@ -92,7 +93,7 @@ table-driven tests in `src/clocks.test.ts` for the exact parsing contract.
 This is a [Screenly Playground](https://github.com/Screenly/Playground) app, so a
 quiet Screenly lockup sits in the footer (`assets/images/screenly-logo.svg`,
 copied to `dist/images/` by `build.ts`) and the masthead pip carries the brand
-gradient. Card accents are reserved for the data-driven day/night bands — keep
+gradient. Card accents are reserved for the data-driven day/night bands, so keep
 the brand colour in the chrome, not on the clocks.
 
 ## Conventions
@@ -100,7 +101,7 @@ the brand colour in the chrome, not on the clocks.
 - Biome: single quotes, no semicolons, no trailing commas, 2-space indent, 100
   cols. Run `bun run lint:fix` before committing.
 - TypeScript is `strict` with `noUncheckedIndexedAccess` and
-  `verbatimModuleSyntax` — use `import type` for type-only imports and keep
+  `verbatimModuleSyntax`: use `import type` for type-only imports and keep
   array access guarded.
 
 ### Signature design element (don't flatten it)
@@ -109,8 +110,8 @@ Each card's accent (the pip, the tinted top "sky band", and the glow) is set fro
 **that city's own local hour** via `data-period` (`dawn`/`day`/`dusk`/`night`),
 so a wall of cities reads at a glance as who is awake. The grid uses
 `grid-auto-rows: 1fr` to give every card a definite size, and `.card-time` is
-sized with `min(24cqw, 40cqh)` so the time fits both the card's width and height
-— this is what keeps any number of cities from overflowing on any resolution
+sized with `min(24cqw, 40cqh)` so the time fits both the card's width and height,
+which is what keeps any number of cities from overflowing on any resolution
 (480×800 portrait through 4K). Test layout changes across the resolutions in
 [`Playground/docs/resolutions.md`](../../Playground/docs/resolutions.md).
 
diff --git a/README.md b/README.md
index 246a23b..ee4b2ae 100644
--- a/README.md
+++ b/README.md
@@ -16,20 +16,20 @@ Screenly lockup in the footer.
 
 ## Settings (URL parameters)
 
-Everything on screen is driven by the query string — no build step or config
+Everything on screen is driven by the query string: no build step or config
 file needed to change the board. All settings are optional; with **no parameters
 at all**, a default board of major world cities is shown in 24-hour `en-GB`.
 
 | Setting   | Values / syntax                                   | Default        | Repeatable | What it does |
 | --------- | ------------------------------------------------- | -------------- | ---------- | ------------ |
-| `tz`      | `Area/City` or `Area/City\|Label`                 | —              | yes        | Adds one clock. Repeat the parameter for more cities. |
-| `clocks`  | comma-separated list of the `tz` syntax           | —              | yes        | Adds several clocks in a single parameter. |
+| `tz`      | `Area/City` or `Area/City\|Label`                 | none           | yes        | Adds one clock. Repeat the parameter for more cities. |
+| `clocks`  | comma-separated list of the `tz` syntax           | none           | yes        | Adds several clocks in a single parameter. |
 | `locale`  | a [BCP-47 tag](https://en.wikipedia.org/wiki/IETF_language_tag) (`en-US`, `de-DE`, `fr-FR`, `ja-JP`) | `en-GB` | no | **Global** display language for every card's date and time, and the default 12/24h cycle. |
 | `format`  | `12` or `24`                                      | the locale's default | no | Forces the hour cycle on every card, overriding the locale. |
 | `seconds` | present, or `=1` / `=0`                            | off            | no | Shows seconds and ticks every second (otherwise ticks each minute). |
 | `title`   | any text                                          | `World Clock`  | no | Replaces the masthead label. |
 
-### `tz` and `clocks` — choosing the cities
+### `tz` and `clocks`: choosing the cities
 
 A clock is an [IANA timezone name](https://en.wikipedia.org/wiki/List_of_tz_database_time_zones)
 (`Europe/London`, `America/New_York`, `Asia/Kolkata`) with an optional label
@@ -38,7 +38,7 @@ after a `|`:
 - `tz=Asia/Tokyo` → labelled "Tokyo" (derived from the zone)
 - `tz=Asia/Tokyo|HQ Tokyo` → labelled "HQ Tokyo"
 
-Add as many as you like, either by repeating `tz` or with one `clocks` list — the
+Add as many as you like, either by repeating `tz` or with one `clocks` list; the
 two can be combined and the order is preserved. **Unknown timezones are skipped**
 rather than shown broken, so a typo drops one card instead of breaking the board.
 
@@ -49,13 +49,13 @@ rather than shown broken, so a typo drops one card instead of breaking the board
 
 ### `locale` is global, not per-city
 
-A world clock shows every city in **one** language and time format — the
+A world clock shows every city in **one** language and time format: the
 viewer's, set once with `locale`. The timezone is the only thing that differs
 per card. (Localizing each city into its own local language would make a board of
 clocks unreadable, so it is deliberately not supported.)
 
 > Non-Latin locales (CJK, Arabic, …) render their text using the **device's own
-> fonts** — only Latin webfonts are vendored to keep the site small and CDN-free.
+> fonts**: only Latin webfonts are vendored to keep the site small and CDN-free.
 > The time digits are always Latin, so they render in Fraunces everywhere.
 
 ### Examples
@@ -74,7 +74,7 @@ clocks unreadable, so it is deliberately not supported.)
 ## Resolutions
 
 Responsive from a Raspberry Pi touch display (480×800) up to 4K (4096×2160), in
-both orientations — see [`Playground/docs/resolutions.md`](../../Playground/docs/resolutions.md).
+both orientations, see [`Playground/docs/resolutions.md`](../../Playground/docs/resolutions.md).
 The grid divides the screen into equal cells, so any number of cities fills the
 board without scrolling or clipping.
 
@@ -100,7 +100,7 @@ and copies the HTML shell + vendored fonts into `./dist`.
 
 ## Deployment (GitHub Pages)
 
-Deploys are **tag-driven**. Pushing a CalVer tag (`YYYY.N`, e.g. `2026.8.0`)
+Deploys are **tag-driven**. Pushing a CalVer tag (`YYYY.M.PATCH`, e.g. `2026.8.0`)
 runs [`.github/workflows/deploy-pages.yml`](.github/workflows/deploy-pages.yml),
 which lints, typechecks, tests, builds `./dist`, and publishes it to Pages; it
 also accepts `workflow_dispatch`. Pushing to `master` on its own does **not**
diff --git a/assets/styles/main.css b/assets/styles/main.css
index 30b5af7..d78ecf6 100644
--- a/assets/styles/main.css
+++ b/assets/styles/main.css
@@ -1,9 +1,9 @@
 /* =========================================================================
-   World Clock — "Observatory" theme
+   World Clock: "Observatory" theme
    Editorial almanac aesthetic borrowed from the Screenly clock-app: a Fraunces
    display serif over a deep night sky, atmospheric grain + vignette, and a
-   time-of-day reactive accent. Here the accent is PER CARD — every city glows
-   with its own dawn/day/dusk/night band — so a wall of zones reads at a glance.
+   time-of-day reactive accent. Here the accent is PER CARD: every city glows
+   with its own dawn/day/dusk/night band, so a wall of zones reads at a glance.
    Tuned for unattended digital signage: legible at distance, no interaction,
    responsive from a 480x800 Pi touch display up to 4096x2160.
    ========================================================================= */
@@ -48,7 +48,7 @@
   --ink-dim: rgb(247 243 236 / 0.74);
   --hairline: rgb(247 243 236 / 0.16);
 
-  /* Single fluid root size — symmetric across orientation (vw + vh) so the
+  /* Single fluid root size, symmetric across orientation (vw + vh) so the
      whole layout scales smoothly from a Pi touch display up to 4K. */
   --root: clamp(15px, 0.82vw + 0.82vh, 54px);
 }
@@ -78,7 +78,7 @@ body {
   font-family: var(--font-meta);
   line-height: 1.3;
   /* A deep, neutral observatory sky. Unlike the single-zone clock-app, the page
-     can't belong to one time of day — many zones share it — so the backdrop
+     can't belong to one time of day (many zones share it), so the backdrop
      stays a constant night and each card carries its own period glow. */
   background-color: #06070f;
   background-image:
@@ -95,7 +95,7 @@ body {
 }
 
 /* =========================================================================
-   Board shell — masthead / grid, plus vignette, grain and ambient glow
+   Board shell: masthead / grid, plus vignette, grain and ambient glow
    ========================================================================= */
 .board {
   position: relative;
@@ -157,7 +157,7 @@ body {
 }
 
 /* =========================================================================
-   Masthead — title eyebrow + live UTC reference
+   Masthead: title eyebrow + live UTC reference
    ========================================================================= */
 .masthead {
   display: flex;
@@ -206,8 +206,8 @@ body {
 /* =========================================================================
    Grid of city cards
    The column count (--cols) is computed in main.ts from the city count AND the
-   board's aspect ratio, so the board is a balanced block — 4 cities → 2x2,
-   7 → 4+3 — instead of whatever happens to fit the width. A partial last row is
+   board's aspect ratio, so the board is a balanced block (4 cities → 2x2,
+   7 → 4+3) instead of whatever happens to fit the width. A partial last row is
    centred, not left-aligned: the grid is laid out in TWICE as many tracks as
    there are columns (each card spans 2), so main.ts can offset the last row by
    a half-column and centre an odd remainder exactly. align-content stretches
@@ -233,14 +233,14 @@ body {
 }
 
 /* =========================================================================
-   City card — the hero unit. Each is its own container so the time can size
+   City card: the hero unit. Each is its own container so the time can size
    to the CARD width (container queries), staying balanced no matter how many
    cities share the board.
    ========================================================================= */
 .card {
   --accent: #9aa6f2;
   /* container-type: size (not just inline-size) so the time can query the card
-     HEIGHT as well as its width — the grid gives every card a definite size. */
+     HEIGHT as well as its width; the grid gives every card a definite size. */
   container-type: size;
   position: relative;
   overflow: hidden;
@@ -262,7 +262,7 @@ body {
   transition: --accent 1.1s ease;
 }
 
-/* Period-tinted glow bleeding in from the top — the card's "sky". */
+/* Period-tinted glow bleeding in from the top, the card's "sky". */
 .card-glow {
   position: absolute;
   inset: 0;
@@ -365,7 +365,7 @@ body {
   margin-inline-start: 0;
 }
 
-/* ko / zh-Hant print the period before the time — reorder the marker. */
+/* ko / zh-Hant print the period before the time, so reorder the marker. */
 .card-clock.period-first .card-period {
   order: -1;
   margin-inline-start: 0;
@@ -403,7 +403,7 @@ body {
 }
 
 /* =========================================================================
-   Entrance choreography — one staggered reveal on load
+   Entrance choreography: one staggered reveal on load
    ========================================================================= */
 .anim {
   opacity: 0;
@@ -456,14 +456,15 @@ body {
 }
 
 /* =========================================================================
-   Degraded mode — container-query fallback for older browsers.
+   Degraded mode: container-query fallback for older browsers.
    The card sizes its text AND spacing with cqw/cqh units (container queries:
    Chrome 105 / Safari 16). Where they're unsupported those declarations drop, so
    the text collapses to an inherited size and the card loses its gap/padding
    (leaving the time jammed against the edge, unstyled and misaligned). Re-derive
-   everything off the viewport (vmin) instead — grid-count-agnostic, so it stays
-   legible and laid-out whether the board shows one city or twelve. This block
-   must cover EVERY cqw/cqh declaration in the card, not just the font sizes.
+   everything off the viewport (vmin) instead, which is grid-count-agnostic, so
+   it stays legible and laid-out whether the board shows one city or twelve. This
+   block must cover EVERY cqw/cqh declaration in the card, not just the font
+   sizes.
    ========================================================================= */
 @supports not (container-type: size) {
   .card {
@@ -497,7 +498,7 @@ body {
 }
 
 /* =========================================================================
-   Degraded mode — app-specific resting state (the generic html.legacy kill-switch
+   Degraded mode: app-specific resting state (the generic html.legacy kill-switch
    is prepended at build by @screenly-labs/signage-kit). .anim has a base
    opacity:0 and only appears via the entrance animation, so reveal it here
    (the kill-switch cancels the animation itself), mirroring reduced-motion.
diff --git a/build.ts b/build.ts
index b15261b..27c4ff8 100644
--- a/build.ts
+++ b/build.ts
@@ -90,4 +90,4 @@ console.log(`✓ Manifest: ${DIST}/.well-known/signage-app.json`)
 await Bun.write(`${DIST}/.nojekyll`, '')
 console.log(`✓ ${DIST}/.nojekyll`)
 
-console.log('Build complete — static site written to ./dist')
+console.log('Build complete: static site written to ./dist')
diff --git a/dev-server.ts b/dev-server.ts
index cb5c34c..7643943 100644
--- a/dev-server.ts
+++ b/dev-server.ts
@@ -1,7 +1,7 @@
 #!/usr/bin/env bun
 // Local dev server for testing the static build on the LAN. It builds the site
 // once, serves ./dist on 0.0.0.0 (so other devices on the network can reach it),
-// and rebuilds whenever a source file changes. This is a dev convenience only —
+// and rebuilds whenever a source file changes. This is a dev convenience only,
 // production is the static ./dist published to GitHub Pages, never this server.
 
 import { watch } from 'node:fs'
@@ -27,7 +27,7 @@ let pending: ReturnType<typeof setTimeout> | undefined
 const scheduleBuild = (): void => {
   if (pending) clearTimeout(pending)
   pending = setTimeout(() => {
-    console.log('↻ change detected — rebuilding…')
+    console.log('↻ change detected, rebuilding…')
     void build()
   }, 120)
 }
@@ -68,5 +68,5 @@ console.log(`\nWorld Clock dev server`)
 console.log(`  local:   http://localhost:${server.port}/`)
 console.log(`  network: http://0.0.0.0:${server.port}/  (reachable on your LAN IP)`)
 console.log(
-  `Watching ${[...WATCH_DIRS, ...WATCH_FILES].join(', ')} — edits rebuild automatically.\n`
+  `Watching ${[...WATCH_DIRS, ...WATCH_FILES].join(', ')}, edits rebuild automatically.\n`
 )
diff --git a/src/clocks.ts b/src/clocks.ts
index bdc6ad3..6a85b6c 100644
--- a/src/clocks.ts
+++ b/src/clocks.ts
@@ -7,7 +7,7 @@
 // timezone. main.ts owns the DOM, the ticking timer and `window`.
 
 // A single clock to render: which IANA zone, and what to call it. The display
-// language (locale) is NOT per-clock — a world clock shows every city in the
+// language (locale) is NOT per-clock: a world clock shows every city in the
 // viewer's one chosen language and 12/24h format; only the timezone differs.
 export interface ClockSpec {
   timeZone: string
@@ -33,7 +33,7 @@ export interface ClockConfig {
 }
 
 // Neutral default locale: en-GB gives 24h time and plain English month/weekday
-// names — a sane, predictable baseline for signage when no locale is requested.
+// names, a sane, predictable baseline for signage when no locale is requested.
 export const FALLBACK_LOCALE = 'en-GB'
 
 // Shown when the URL carries no clocks: a spread of major cities across the
@@ -53,7 +53,7 @@ export const DEFAULT_CLOCKS: ClockSpec[] = [
 const RTL_LANGUAGES = new Set(['ar', 'fa', 'he', 'ps', 'dv', 'ur', 'ckb', 'sd', 'yi'])
 
 // BCP-47 subtags are case-insensitive, so lowercase the primary language before
-// the lookup — otherwise "AR-SA" would be mis-classified as LTR.
+// the lookup; otherwise "AR-SA" would be mis-classified as LTR.
 export const isRtlLocale = (locale: string): boolean =>
   RTL_LANGUAGES.has((locale.split('-')[0] ?? '').toLowerCase())
 
@@ -140,7 +140,7 @@ export const parseClocks = (search: string): ClockConfig => {
 }
 
 // The cached Intl formatters for one clock. Built once per spec and reused on
-// every tick — constructing a DateTimeFormat per render is the costly part.
+// every tick; constructing a DateTimeFormat per render is the costly part.
 // `offset` is undefined on engines that lack the 'shortOffset' option (see
 // buildFormatters); the card simply omits the GMT label there.
 export interface ClockFormatters {
@@ -193,7 +193,7 @@ export const buildFormatters = (spec: ClockSpec, config: ClockConfig): ClockForm
   }
 }
 
-// Short GMT offset for the zone, e.g. "GMT+9" — useful orientation on a board of
+// Short GMT offset for the zone, e.g. "GMT+9", useful orientation on a board of
 // many cities. 'shortOffset' only landed in ~Chrome 91 and throws where it is
 // unsupported; since isValidTimeZone validates with a plain formatter, an old
 // signage webview would otherwise crash the whole page here. Degrade to no offset
@@ -232,7 +232,7 @@ export const formatTimeParts = (
 export const formatDate = (formatter: Intl.DateTimeFormat, date: Date): string =>
   formatter.format(date)
 
-// Hour (0-23) of an absolute instant in the clock's zone — feeds getDayPeriod.
+// Hour (0-23) of an absolute instant in the clock's zone; feeds getDayPeriod.
 export const getZonedHour = (formatter: Intl.DateTimeFormat, date: Date): number =>
   Number.parseInt(formatter.format(date), 10)
 
diff --git a/src/main.ts b/src/main.ts
index 8c1cdcf..7525aa6 100644
--- a/src/main.ts
+++ b/src/main.ts
@@ -2,7 +2,7 @@
 // formatting, day-period banding) lives in ./clocks.ts and is unit-tested; this
 // file only reads the query string, builds one card per clock, and re-renders
 // them on a shared timer. Loaded as a `type="module"` script from the static
-// index.html — no framework, no server.
+// index.html: no framework, no server.
 
 // Side-effect import: installs the replaceChildren shim for the older-browser
 // degraded mode (shared across all apps). Must stay first.
@@ -30,13 +30,13 @@ declare global {
   }
 }
 
-// Report how the board is configured — the whole app is URL-driven, so the
+// Report how the board is configured; the whole app is URL-driven, so the
 // query string *is* the usage. Fired once at load so we can see, in aggregate,
 // how many cities people show, which zones, and which locale/format options
 // they pick. No free-text (the custom title) is sent, only whether one is set.
 const reportUsage = (config: ClockConfig): void => {
   if (typeof window.gtag !== 'function') return
-  // GA4 params should be strings or numbers — booleans can coerce
+  // GA4 params should be strings or numbers; booleans can coerce
   // inconsistently or get dropped in reports, so flags are sent as 0/1.
   window.gtag('event', 'clock_config', {
     clock_count: config.clocks.length,
@@ -52,8 +52,8 @@ const reportUsage = (config: ClockConfig): void => {
 }
 
 // Comma-joined IANA zones (deduped, sorted) for the analytics dimension. GA4
-// caps a param value at 100 chars, so drop WHOLE zones — never a mid-name slice
-// like "America/Los_A…" — until the list plus a trailing "…" marker fits. The
+// caps a param value at 100 chars, so drop WHOLE zones, never a mid-name slice
+// like "America/Los_A…", until the list plus a trailing "…" marker fits. The
 // marker keeps a truncated board distinguishable from an exact list.
 const ZONES_MAX = 100
 const encodeZones = (clocks: ClockSpec[]): string => {
@@ -69,7 +69,7 @@ const encodeZones = (clocks: ClockSpec[]): string => {
 
 // One rendered clock: its root element plus an update(date) that repaints it
 // from an absolute instant. Formatters are built once and captured in the
-// closure — rebuilding a DateTimeFormat every tick is the expensive part.
+// closure; rebuilding a DateTimeFormat every tick is the expensive part.
 interface ClockCard {
   el: HTMLElement
   update(now: Date): void
@@ -81,7 +81,7 @@ const setText = (el: Element, value: string): void => {
 
 const createCard = (spec: ClockSpec, config: ClockConfig, index: number): ClockCard => {
   const fmt = buildFormatters(spec, config)
-  // Direction comes from the single global locale — every card shares it.
+  // Direction comes from the single global locale; every card shares it.
   const dir = isRtlLocale(config.locale) ? 'rtl' : 'ltr'
 
   const el = document.createElement('article')
@@ -141,7 +141,7 @@ const createCard = (spec: ClockSpec, config: ClockConfig, index: number): ClockC
 // Choose a balanced grid shape for `count` cards on a board of the current
 // aspect ratio, and centre a partial last row. The board is landscape signage,
 // so we derive the ROW count from the aspect (a wide board wants few rows) and
-// let the columns follow — 4 cities → 2x2, 7 → 4+3, 6 → 3x2. The last row is
+// let the columns follow: 4 cities → 2x2, 7 → 4+3, 6 → 3x2. The last row is
 // centred via the 2x sub-column grid: offsetting its first card by half the
 // empty span lands an odd remainder (e.g. 4+3) exactly in the middle.
 const layoutGrid = (grid: HTMLElement, cards: ClockCard[]): void => {
diff --git a/sync-fonts.ts b/sync-fonts.ts
index 5d028cc..762f80f 100644
--- a/sync-fonts.ts
+++ b/sync-fonts.ts
@@ -1,6 +1,6 @@
 #!/usr/bin/env bun
 // Vendor this app's webfonts into ./assets/fonts. The files, versions, and copy
-// logic all live in @screenly-labs/signage-kit — this just names the families
+// logic all live in @screenly-labs/signage-kit; this just names the families
 // the board uses (Fraunces time digits + Hanken Grotesk chrome) and the dest
 // dir (this app keeps a relative ../fonts/ layout, not /static/fonts).