@jant/core 0.6.6 → 0.6.8

package/dist/{github-api-UD4u_7fa.js → github-api-BgSiE71w.js}

@@ -1,4 +1,4 @@
-import { y as __exportAll } from "./url-XF0GbKGO.js";
+import { b as __exportAll } from "./url-BMYO-Zlt.js";
 //#region src/lib/github-api.ts
 var github_api_exports = /* @__PURE__ */ __exportAll({
 	GitHubApiError: () => GitHubApiError,

package/dist/{github-app-DeX6Td1O.js → github-app-BbklkFmU.js}

@@ -1,4 +1,4 @@
-import { y as __exportAll } from "./url-XF0GbKGO.js";
+import { b as __exportAll } from "./url-BMYO-Zlt.js";
 //#region src/lib/github-app.ts
 var github_app_exports = /* @__PURE__ */ __exportAll({
 	buildInstallUrl: () => buildInstallUrl,

package/dist/{github-sync-BtHY2AST.js → github-sync-D1Cw8mOY.js}

@@ -1,6 +1,6 @@
-import { c as parseMarkdownDocument, r as parseFrontMatter, t as createExportService } from "./export-DLukCOO3.js";
-import { r as getInstallationToken } from "./github-app-DeX6Td1O.js";
-import { r as parseRepoSlug, t as createGitHubClient } from "./github-api-UD4u_7fa.js";
+import { c as parseMarkdownDocument, r as parseFrontMatter, t as createExportService } from "./export-Be082J0n.js";
+import { r as getInstallationToken } from "./github-app-BbklkFmU.js";
+import { r as parseRepoSlug, t as createGitHubClient } from "./github-api-BgSiE71w.js";
 //#region src/lib/markdown-to-tiptap.ts
 /**
 * Markdown → TipTap JSON Conversion

package/dist/{github-sync-BeDecPen.js → github-sync-_kPWM4m9.js}

@@ -1,4 +1,4 @@
-import "./url-XF0GbKGO.js";
-import "./export-DLukCOO3.js";
-import { i as classifyRepoForSync, o as createGitHubSyncService } from "./github-sync-BtHY2AST.js";
+import "./url-BMYO-Zlt.js";
+import "./export-Be082J0n.js";
+import { i as classifyRepoForSync, o as createGitHubSyncService } from "./github-sync-D1Cw8mOY.js";
 export { classifyRepoForSync, createGitHubSyncService };

package/dist/index.js

@@ -1,6 +1,6 @@
-import { v as url_exports } from "./url-XF0GbKGO.js";
-import { A as MAX_MEDIA_ATTACHMENTS, C as toMediaView, D as toPostViews, E as toPostView, F as STATUSES, I as TEXT_ATTACHMENT_CONTENT_FORMATS, M as MEDIA_KINDS, N as NAV_ITEM_TYPES, O as toSearchResultView, P as SORT_ORDERS, S as toArchiveGroupsWithMedia, T as toNavItemViews, b as createMediaContext, h as defaultFeedRenderer, j as MAX_PINNED_POSTS, k as FORMATS, t as createApp, w as toNavItemView, x as toArchiveGroups } from "./app-BJkOcMbZ.js";
-import { T as time_exports, a as markdown_exports } from "./export-DLukCOO3.js";
-import "./env-CoSe-1y4.js";
-import "./github-sync-BtHY2AST.js";
+import { y as url_exports } from "./url-BMYO-Zlt.js";
+import { A as MAX_MEDIA_ATTACHMENTS, C as toMediaView, D as toPostViews, E as toPostView, F as STATUSES, I as TEXT_ATTACHMENT_CONTENT_FORMATS, M as MEDIA_KINDS, N as NAV_ITEM_TYPES, O as toSearchResultView, P as SORT_ORDERS, S as toArchiveGroupsWithMedia, T as toNavItemViews, b as createMediaContext, h as defaultFeedRenderer, j as MAX_PINNED_POSTS, k as FORMATS, t as createApp, w as toNavItemView, x as toArchiveGroups } from "./app-9P4rVCe2.js";
+import { T as time_exports, a as markdown_exports } from "./export-Be082J0n.js";
+import "./env-OHRKGcMj.js";
+import "./github-sync-D1Cw8mOY.js";
 export { FORMATS, MAX_MEDIA_ATTACHMENTS, MAX_PINNED_POSTS, MEDIA_KINDS, NAV_ITEM_TYPES, SORT_ORDERS, STATUSES, TEXT_ATTACHMENT_CONTENT_FORMATS, createApp, createMediaContext, defaultFeedRenderer, markdown_exports as markdown, time_exports as time, toArchiveGroups, toArchiveGroupsWithMedia, toMediaView, toNavItemView, toNavItemViews, toPostView, toPostViews, toSearchResultView, url_exports as url };

package/dist/node.js

@@ -1,8 +1,8 @@
-import "./url-XF0GbKGO.js";
-import { B as isAssetPath, L as buildThemeStyle, R as BUILTIN_COLOR_THEMES, _ as sqliteSchemaBundle, a as resolveDatabaseDialect, c as resolveConfig, d as setWebhook, f as BUILTIN_FONT_THEMES, g as pgSchemaBundle, i as createSiteService, l as getWebhookUrl, m as getFontThemeCssVariables, n as createNodeCliRuntime, o as getHostBasedStartupConfigurationIssues, p as getCjkSerifCssVariables, r as createNodeRequestRuntime, s as createStorageDriver, t as createApp, u as setMyCommands, v as createNodeDatabase, y as schema_exports, z as getPublicAssetBasePath } from "./app-BJkOcMbZ.js";
-import { t as createExportService } from "./export-DLukCOO3.js";
-import { C as shouldTrustProxy, S as getTelegramWebhookSecret, b as getSiteResolutionMode, d as getHostedControlPlaneBaseUrl, i as getConfiguredSingleSitePathPrefix, l as getEnvString, r as getConfiguredSingleSiteOrigin, x as getTelegramBotPool, y as getPort } from "./env-CoSe-1y4.js";
-import "./github-sync-BtHY2AST.js";
+import "./url-BMYO-Zlt.js";
+import { B as isAssetPath, L as buildThemeStyle, R as BUILTIN_COLOR_THEMES, _ as sqliteSchemaBundle, a as resolveDatabaseDialect, c as resolveConfig, d as setWebhook, f as BUILTIN_FONT_THEMES, g as pgSchemaBundle, i as createSiteService, l as getWebhookUrl, m as getFontThemeCssVariables, n as createNodeCliRuntime, o as getHostBasedStartupConfigurationIssues, p as getCjkSerifCssVariables, r as createNodeRequestRuntime, s as createStorageDriver, t as createApp, u as setMyCommands, v as createNodeDatabase, y as schema_exports, z as getPublicAssetBasePath } from "./app-9P4rVCe2.js";
+import { t as createExportService } from "./export-Be082J0n.js";
+import { C as shouldTrustProxy, S as getTelegramWebhookSecret, b as getSiteResolutionMode, d as getHostedControlPlaneBaseUrl, i as getConfiguredSingleSitePathPrefix, l as getEnvString, r as getConfiguredSingleSiteOrigin, x as getTelegramBotPool, y as getPort } from "./env-OHRKGcMj.js";
+import "./github-sync-D1Cw8mOY.js";
 import { drizzle } from "drizzle-orm/better-sqlite3";
 import { serve } from "@hono/node-server";
 import Database from "better-sqlite3";
@@ -529,7 +529,7 @@ async function createNodeRequestHandler(options) {
 async function start(env = process.env, app) {
 	const handler = await createNodeRequestHandler({
 		env,
-		app: async () => app ?? (await import("./app-CL2PC1Fl.js")).createApp()
+		app: async () => app ?? (await import("./app-DaxS_Cz-.js")).createApp()
 	});
 	const hostname = resolveHost(env);
 	const port = resolvePort(env);

package/dist/{url-XF0GbKGO.js → url-BMYO-Zlt.js}

@@ -32,7 +32,8 @@ var __exportAll = (all, no_symbols) => {
 	toAbsoluteAssetUrl: () => toAbsoluteAssetUrl,
 	toAbsoluteSiteUrl: () => toAbsoluteSiteUrl,
 	toPublicHref: () => toPublicHref,
-	toPublicPath: () => toPublicPath
+	toPublicPath: () => toPublicPath,
+	toSameSitePath: () => toSameSitePath
 });
 function normalizeSitePathname(pathname) {
 	if (pathname === "/" || pathname === "") return "";
@@ -109,6 +110,45 @@ function normalizeSitePathname(pathname) {
 	return str.startsWith("http://") || str.startsWith("https://");
 }
 /**
+* If a full URL points at the site's own host, return its same-site path
+* (`pathname` + `search` + `hash`). Returns `null` when the input is not a full
+* URL, is unparseable, or points at a different host.
+*
+* Self-referential absolute links — e.g. a nav item set to
+* `https://example.com/about` on `example.com` — should behave like the
+* internal path `/about`: no external-link icon, no `target="_blank"`.
+*
+* Matching is by **hostname**, not full origin: scheme and port differences are
+* treated as same-site. This keeps the check intuitive ("same domain") and
+* robust in dev, where the site is often served over `http://host:<port>` while
+* a nav link stores the canonical `https://host` URL. In production (canonical
+* https + default port) hostname match is equivalent to origin match.
+*
+* @param url - Candidate URL (full URL or relative path)
+* @param siteOrigin - The site's own origin, e.g. `https://example.com`
+* @returns The same-site path, or `null` when the URL is external/non-absolute
+*
+* @example
+* ```ts
+* toSameSitePath("https://example.com/about", "https://example.com");    // "/about"
+* toSameSitePath("https://example.com/about", "http://example.com:8787"); // "/about"
+* toSameSitePath("https://other.com/about", "https://example.com");      // null
+* toSameSitePath("/about", "https://example.com");                       // null
+* ```
+*/ function toSameSitePath(url, siteOrigin) {
+	if (!siteOrigin || !isFullUrl(url)) return null;
+	let parsed;
+	let reference;
+	try {
+		parsed = new URL(url);
+		reference = new URL(siteOrigin);
+	} catch {
+		return null;
+	}
+	if (parsed.hostname !== reference.hostname) return null;
+	return `${parsed.pathname}${parsed.search}${parsed.hash}` || "/";
+}
+/**
 * Converts text to a URL-friendly slug.
 *
 * Transforms text into a lowercase, hyphen-separated slug using limax for
@@ -352,4 +392,4 @@ var SAFE_URL_PROTOCOLS = new Set([
 	return toAbsoluteSiteUrl(url, siteUrl, sitePathPrefix);
 }
 //#endregion
-export { toPublicPath as _, getSitePathPrefix as a, normalizePath as c, sanitizeUrl as d, slugify as f, toPublicHref as g, toAbsoluteSiteUrl as h, getSiteOrigin as i, normalizeSitePathPrefix as l, toAbsoluteAssetUrl as m, extractDisplayDomain as n, isFullUrl as o, stripSitePathPrefix as p, extractDomain as r, isSafeInternalRedirect as s, buildSiteUrl as t, normalizeSiteUrl as u, url_exports as v, __exportAll as y };
+export { toPublicPath as _, getSitePathPrefix as a, __exportAll as b, normalizePath as c, sanitizeUrl as d, slugify as f, toPublicHref as g, toAbsoluteSiteUrl as h, getSiteOrigin as i, normalizeSitePathPrefix as l, toAbsoluteAssetUrl as m, extractDisplayDomain as n, isFullUrl as o, stripSitePathPrefix as p, extractDomain as r, isSafeInternalRedirect as s, buildSiteUrl as t, normalizeSiteUrl as u, toSameSitePath as v, url_exports as y };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jant/core",
-  "version": "0.6.6",
+  "version": "0.6.8",
   "description": "A modern, open-source microblogging platform built on Cloudflare Workers",
   "type": "module",
   "exports": {

package/src/__tests__/bin/uploads-cleanup.test.ts

@@ -41,6 +41,7 @@ describe("jant uploads cleanup", () => {
         JSON.stringify({
           abortedMultipartUploads: 1,
           deletedSessions: 3,
+          deletedOrphanMedia: 2,
         }),
         {
           status: 200,
@@ -73,5 +74,6 @@ describe("jant uploads cleanup", () => {
     );
     expect(logSpy).toHaveBeenNthCalledWith(2, "Deleted sessions: 3");
     expect(logSpy).toHaveBeenNthCalledWith(3, "Aborted multipart uploads: 1");
+    expect(logSpy).toHaveBeenNthCalledWith(4, "Orphaned media deleted: 2");
   });
 });

package/src/client/__tests__/compose-bridge.test.ts

@@ -459,6 +459,111 @@ describe("compose bridge", () => {
     expect(composeEl.clearLocalDraftFromStorage).not.toHaveBeenCalled();
   });
 
+  it("re-binds the thread-context Show more toggle after a reply swaps in a thread preview", async () => {
+    document.body.innerHTML = `
+      <div data-timeline-item data-thread-root-id="pst_root">
+        <div data-timeline-item-content>
+          <article data-post data-post-id="pst_root">Root</article>
+        </div>
+      </div>
+    `;
+
+    const composeEl = document.createElement(
+      "jant-compose-dialog",
+    ) as ComposeHarness;
+    composeEl.refreshCollections = vi.fn(async () => true);
+    composeEl.pageMode = false;
+    document.body.appendChild(composeEl);
+
+    // The freshly rendered thread preview carries the collapsed ancestor shell
+    // plus its "Show more" toggle — the markup the bug left inert until reload.
+    const threadPreviewHtml = `
+      <div class="thread-group thread-group-preview">
+        <div class="thread-context-shell" data-thread-context data-collapsed="">
+          <div class="thread-item"><article data-post>Root</article></div>
+        </div>
+        <button
+          type="button"
+          class="thread-context-toggle"
+          data-thread-context-toggle
+          data-label-more="Show more"
+          data-label-less="Show less"
+          aria-expanded="false"
+        >
+          <span class="thread-context-toggle-label">Show more</span>
+        </button>
+        <div class="thread-item thread-item-hero"><article data-post>Reply</article></div>
+      </div>
+    `;
+
+    vi.spyOn(globalThis, "fetch").mockImplementation(async (input) => {
+      const raw =
+        typeof input === "string"
+          ? input
+          : input instanceof URL
+            ? input.toString()
+            : input.url;
+      const url = new URL(raw, "http://localhost");
+
+      if (url.pathname === "/compose") {
+        return new Response(
+          JSON.stringify({ status: "published", permalink: "/the-reply" }),
+          { status: 200, headers: { "Content-Type": "application/json" } },
+        );
+      }
+
+      if (url.pathname === "/_/timeline-item/pst_root") {
+        return new Response(threadPreviewHtml, {
+          status: 200,
+          headers: { "Content-Type": "text/html" },
+        });
+      }
+
+      throw new Error(`Unexpected fetch: ${url.pathname}`);
+    });
+
+    composeEl.dispatchEvent(
+      new CustomEvent("jant:compose-submit-deferred", {
+        bubbles: true,
+        detail: {
+          format: "note",
+          title: "",
+          body: "A reply",
+          url: "",
+          quoteText: "",
+          quoteAuthor: "",
+          slug: "",
+          status: "published",
+          visibility: "public",
+          rating: 0,
+          collectionIds: [],
+          attachments: [],
+          pendingAttachments: [],
+          replyToId: "pst_parent",
+          replyThreadRootId: "pst_root",
+          replyRefreshKind: "timeline-item",
+          replyRefreshId: "pst_root",
+        },
+      }),
+    );
+
+    await flushBridgeWork();
+
+    const toggle = document.querySelector<HTMLElement>(
+      "[data-thread-context-toggle]",
+    );
+    const shell = document.querySelector<HTMLElement>("[data-thread-context]");
+    expect(toggle).not.toBeNull();
+    expect(shell).not.toBeNull();
+    // setupThreadContexts ran on the swapped-in markup.
+    expect(toggle?.dataset.threadContextToggleBound).toBe("1");
+
+    // And the bound listener actually toggles the collapsed state.
+    toggle?.dispatchEvent(new MouseEvent("click", { bubbles: true }));
+    expect(shell?.dataset.collapsed).toBeUndefined();
+    expect(toggle?.getAttribute("aria-expanded")).toBe("true");
+  });
+
   it("sends nulls for cleared quote attribution fields when editing", async () => {
     const composeEl = document.createElement(
       "jant-compose-dialog",

package/src/client/__tests__/hydrate-partial.test.ts

@@ -0,0 +1,27 @@
+// @vitest-environment happy-dom
+
+import { describe, expect, it, vi } from "vitest";
+
+// Stub the initializer modules so importing hydrate-partial doesn't run their
+// module-load side effects (DOMContentLoaded registration, observers), and so we
+// can assert hydratePartial forwards the root to each one.
+vi.mock("../thread-context.js", () => ({ setupThreadContexts: vi.fn() }));
+vi.mock("../feed-video-player.js", () => ({ initFeedVideoPlayer: vi.fn() }));
+vi.mock("../audio-player.js", () => ({ initPrecomputedWaveforms: vi.fn() }));
+
+import { hydratePartial } from "../hydrate-partial.js";
+import { setupThreadContexts } from "../thread-context.js";
+import { initFeedVideoPlayer } from "../feed-video-player.js";
+import { initPrecomputedWaveforms } from "../audio-player.js";
+
+describe("hydratePartial", () => {
+  it("re-initializes per-element behaviors scoped to the swapped root", () => {
+    const root = document.createElement("div");
+
+    hydratePartial(root);
+
+    expect(setupThreadContexts).toHaveBeenCalledWith(root);
+    expect(initFeedVideoPlayer).toHaveBeenCalledWith(root);
+    expect(initPrecomputedWaveforms).toHaveBeenCalledWith(root);
+  });
+});

package/src/client/__tests__/json.test.ts

@@ -0,0 +1,94 @@
+import { describe, expect, it } from "vitest";
+
+import {
+  readErrorMessage,
+  readErrorMessageFromText,
+  readJsonObject,
+} from "../json.js";
+
+function jsonResponse(body: unknown, status = 200): Response {
+  return new Response(JSON.stringify(body), {
+    status,
+    headers: { "Content-Type": "application/json" },
+  });
+}
+
+function textResponse(body: string, status: number): Response {
+  return new Response(body, {
+    status,
+    headers: { "Content-Type": "text/plain" },
+  });
+}
+
+describe("readJsonObject", () => {
+  it("parses a JSON object body", async () => {
+    const res = jsonResponse({ id: "abc", count: 2 });
+    expect(await readJsonObject(res)).toEqual({ id: "abc", count: 2 });
+  });
+
+  it("returns empty object for empty body", async () => {
+    const res = new Response("", { status: 200 });
+    expect(await readJsonObject(res)).toEqual({});
+  });
+
+  it("returns empty object for a JSON primitive", async () => {
+    expect(await readJsonObject(jsonResponse("hi"))).toEqual({});
+  });
+
+  it("throws an informative error when body is not JSON", async () => {
+    const res = textResponse("Not Found", 404);
+    await expect(readJsonObject(res)).rejects.toThrow(
+      /Expected JSON \(HTTP 404\) but got: Not Found/,
+    );
+  });
+});
+
+describe("readErrorMessage", () => {
+  it("returns the JSON `error` field when present", async () => {
+    const res = jsonResponse({ error: "Quota exceeded" }, 400);
+    expect(await readErrorMessage(res, "Default")).toBe("Quota exceeded");
+  });
+
+  it("surfaces plain-text body when not JSON (the Not Found case)", async () => {
+    const res = textResponse("Not Found", 404);
+    expect(await readErrorMessage(res, "Failed to start upload")).toBe(
+      "Not Found",
+    );
+  });
+
+  it("falls back when body is empty", async () => {
+    const res = new Response("", { status: 500 });
+    expect(await readErrorMessage(res, "Default")).toBe("Default");
+  });
+
+  it("truncates very long bodies (e.g. error HTML pages)", async () => {
+    const long = "x".repeat(500);
+    const res = textResponse(long, 502);
+    const result = await readErrorMessage(res, "Default");
+    expect(result.length).toBeLessThanOrEqual(201);
+    expect(result.endsWith("…")).toBe(true);
+  });
+
+  it("falls back when JSON has no `error` field", async () => {
+    const res = jsonResponse({ status: "bad" }, 400);
+    expect(await readErrorMessage(res, "Default")).toBe('{"status":"bad"}');
+  });
+});
+
+describe("readErrorMessageFromText", () => {
+  it("extracts error from JSON text", () => {
+    expect(
+      readErrorMessageFromText('{"error":"part too small"}', "Default"),
+    ).toBe("part too small");
+  });
+
+  it("returns plain text when not JSON", () => {
+    expect(readErrorMessageFromText("Internal Error", "Default")).toBe(
+      "Internal Error",
+    );
+  });
+
+  it("falls back when text is blank", () => {
+    expect(readErrorMessageFromText("   ", "Default")).toBe("Default");
+  });
+});

package/src/client/__tests__/note-expand.test.ts

@@ -0,0 +1,130 @@
+// @vitest-environment happy-dom
+
+import { beforeEach, describe, expect, it, vi } from "vitest";
+import "../note-expand.js";
+
+interface CardOptions {
+  clamped?: boolean;
+  withMarker?: boolean;
+}
+
+function buildCard(options: CardOptions = {}) {
+  const { clamped = true, withMarker = true } = options;
+
+  const article = document.createElement("article");
+  article.setAttribute("data-post", "");
+
+  const body = document.createElement("div");
+  body.setAttribute("data-post-body", "");
+  if (clamped) body.setAttribute("data-note-clamp", "");
+  body.innerHTML = withMarker
+    ? "<p>Summary</p><span data-note-break></span><p>Rest</p>"
+    : "<p>Summary</p>";
+
+  const control = document.createElement("a");
+  control.setAttribute("data-note-expand", "");
+  control.setAttribute("aria-expanded", "false");
+  control.setAttribute("href", "/post-1");
+  control.dataset.labelMore = "Read more";
+  control.dataset.labelLess = "Read less";
+  control.textContent = "Read more";
+
+  article.appendChild(body);
+  article.appendChild(control);
+  document.body.appendChild(article);
+  return { article, body, control };
+}
+
+/** Dispatch a click; returns false when a handler called preventDefault. */
+function click(
+  el: HTMLElement,
+  init: { metaKey?: boolean; button?: number } = {},
+): boolean {
+  return el.dispatchEvent(
+    new MouseEvent("click", { bubbles: true, cancelable: true, ...init }),
+  );
+}
+
+describe("note expand", () => {
+  beforeEach(() => {
+    document.body.innerHTML = "";
+    vi.restoreAllMocks();
+  });
+
+  it("reveals the tail by removing the clamp on expand", () => {
+    const { body, control } = buildCard();
+
+    const notCancelled = click(control);
+
+    expect(notCancelled).toBe(false);
+    expect(body.hasAttribute("data-note-clamp")).toBe(false);
+    expect(control.getAttribute("aria-expanded")).toBe("true");
+    expect(control.textContent).toBe("Read less");
+  });
+
+  it("re-clamps the tail on collapse", () => {
+    const { body, control } = buildCard();
+
+    click(control);
+    click(control);
+
+    expect(body.hasAttribute("data-note-clamp")).toBe(true);
+    expect(control.getAttribute("aria-expanded")).toBe("false");
+    expect(control.textContent).toBe("Read more");
+  });
+
+  it("ignores modified clicks so the link opens normally", () => {
+    const { body, control } = buildCard();
+
+    const notCancelled = click(control, { metaKey: true });
+
+    expect(notCancelled).toBe(true);
+    expect(body.hasAttribute("data-note-clamp")).toBe(true);
+  });
+
+  it("leaves the link alone when the body has no break marker", () => {
+    const { body, control } = buildCard({ clamped: false, withMarker: false });
+
+    const notCancelled = click(control);
+
+    expect(notCancelled).toBe(true);
+    expect(body.hasAttribute("data-note-clamp")).toBe(false);
+  });
+
+  it("only toggles the clicked card", () => {
+    const a = buildCard();
+    const b = buildCard();
+
+    click(a.control);
+
+    expect(a.body.hasAttribute("data-note-clamp")).toBe(false);
+    expect(b.body.hasAttribute("data-note-clamp")).toBe(true);
+  });
+
+  it("scrolls the note top into view on collapse when scrolled past it", () => {
+    const { article, control } = buildCard();
+    click(control); // expand
+
+    const scrollIntoView = vi.fn();
+    article.scrollIntoView = scrollIntoView;
+    vi.spyOn(article, "getBoundingClientRect").mockReturnValue({
+      top: -50,
+    } as unknown as ReturnType<HTMLElement["getBoundingClientRect"]>);
+
+    click(control); // collapse
+
+    expect(scrollIntoView).toHaveBeenCalledWith({ block: "start" });
+  });
+
+  it("keeps working after the card DOM is replaced", () => {
+    buildCard();
+
+    // Simulate compose-bridge replacing the card with a fresh collapsed render.
+    document.body.innerHTML = "";
+    const { body, control } = buildCard();
+
+    click(control);
+
+    expect(body.hasAttribute("data-note-clamp")).toBe(false);
+  });
+});

package/src/client/archive-nav.js

@@ -65,7 +65,8 @@ document.querySelectorAll(".archive-chip-dropdown").forEach((chip) => {
         } else {
           url.searchParams.delete(filterKey);
         }
-        // Clear hasMedia when toggling kinds
+        // Kind toggles overwrite media=any/none via the shared param; also
+        // drop the legacy hasMedia param when acting on an old URL.
         url.searchParams.delete("hasMedia");
         url.searchParams.delete("page");
         window.location.href = url.pathname + (url.search || "");

package/src/client/audio-player.ts

@@ -99,9 +99,11 @@ async function extractPeaks(url: string, count: number): Promise<number[]> {
  * time to lay out the canvas (which starts as display:none and only
  * becomes visible when the `has-waveform` class is added).
  */
-function initPrecomputedWaveforms() {
+export function initPrecomputedWaveforms(
+  root: globalThis.Document | globalThis.Element = document,
+) {
   const canvases =
-    document.querySelectorAll<HTMLCanvasElement>("[data-audio-peaks]");
+    root.querySelectorAll<HTMLCanvasElement>("[data-audio-peaks]");
   const cardsToRender: HTMLElement[] = [];
 
   for (const canvas of canvases) {
@@ -472,7 +474,9 @@ document.addEventListener(
 // --- Initialize pre-computed waveforms on page load ---
 
 if (document.readyState === "loading") {
-  document.addEventListener("DOMContentLoaded", initPrecomputedWaveforms);
+  document.addEventListener("DOMContentLoaded", () =>
+    initPrecomputedWaveforms(),
+  );
 } else {
   initPrecomputedWaveforms();
 }