@jant/core 0.6.16 → 0.7.0

package/src/client-auth.ts

@@ -1,44 +1,25 @@
 /**
- * Authenticated client entry point.
+ * Authenticated client entry point: the shell every signed-in page loads.
  *
- * Extends the public bundle with editor, upload, and settings interactions
- * that should not be shipped to anonymous page views.
+ * Extends the public bundle with what an author uses on any page — the post
+ * menu, the command palette, keyboard shortcuts, the buttons that open the
+ * composer — and leaves the rest to `client/lazy-entries.ts`, which loads the
+ * editor, settings, and management entries by what the page contains.
  */
 
 import "./client.js";
-import "./client/image-processor.js";
-import "./client/avatar-upload.js";
 import { ensureConfirmDialog } from "./client/confirm.js";
-import "./client/components/jant-compose-dialog.js";
-import "./client/components/jant-compose-editor.js";
-import "./client/components/jant-compose-fullscreen.js";
-import "./client/compose-bridge.js";
-import "./client/compose-discovery-bridge.js";
-import "./client/slash-discovery-bridge.js";
-import "./client/compose-shortcuts.js";
-import "./client/components/jant-settings-general.js";
-import "./client/components/jant-settings-avatar.js";
-import "./client/components/jant-config-editor.js";
-import "./client/settings-bridge.js";
-import "./client/components/jant-repo-picker.js";
-import "./client/components/jant-collection-form.js";
-import "./client/components/jant-collection-directory.js";
-import "./client/collection-form-bridge.js";
-import "./client/components/jant-post-form.js";
-import "./client/post-form-bridge.js";
-import "./client/components/jant-nav-manager.js";
 import "./client/components/jant-post-menu.js";
-import "./client/collection-page-actions.js";
-import "./client/custom-url-menu.js";
+// The post menu renders the collection form inline when a post needs a new
+// collection, so the form's definition travels with the menu.
+import "./client/components/jant-collection-form.js";
 import "./client/components/jant-command-palette.js";
 import "./client/palette-shortcuts.js";
 import "./client/palette-search-trigger.js";
-
-// Mount fullscreen overlay at body level to escape the dialog's containing
-// block (dialog animation creates a containing block that traps fixed children).
-if (!document.querySelector("jant-compose-fullscreen")) {
-  document.body.appendChild(document.createElement("jant-compose-fullscreen"));
-}
+import "./client/compose-triggers.js";
+import "./client/compose-shortcuts.js";
+import "./client/compose-discovery-bridge.js";
+import { loadEntriesForPage } from "./client/lazy-entries.js";
 
 ensureConfirmDialog();
 
@@ -46,3 +27,5 @@ ensureConfirmDialog();
 if (!document.querySelector("jant-command-palette")) {
   document.body.appendChild(document.createElement("jant-command-palette"));
 }
+
+loadEntriesForPage();

package/src/client-compose.ts

@@ -0,0 +1,19 @@
+/**
+ * Composer entry: the editor and everything that only runs while writing.
+ *
+ * Loaded by `client/lazy-entries.ts` — on first open through
+ * `compose-launch.ts`, at once on the compose page — and preloaded by the
+ * layout on every signed-in page. Never referenced from a script tag.
+ */
+
+import "./client/components/jant-compose-dialog.js";
+import "./client/components/jant-compose-editor.js";
+import "./client/components/jant-compose-fullscreen.js";
+import "./client/compose-bridge.js";
+import "./client/slash-discovery-bridge.js";
+
+// Mount the fullscreen overlay at body level to escape the dialog's containing
+// block (dialog animation creates a containing block that traps fixed children).
+if (!document.querySelector("jant-compose-fullscreen")) {
+  document.body.appendChild(document.createElement("jant-compose-fullscreen"));
+}

package/src/client-manage.ts

@@ -0,0 +1,13 @@
+/**
+ * Management entry: navigation manager, collection directory, and the menus
+ * on collection and custom URL pages.
+ *
+ * Loaded by `client/lazy-entries.ts` when the page carries one of their
+ * elements or hooks. Never referenced from a script tag.
+ */
+
+import "./client/components/jant-nav-manager.js";
+import "./client/components/jant-collection-directory.js";
+import "./client/collection-page-actions.js";
+import "./client/smart-collection-page-actions.js";
+import "./client/custom-url-menu.js";

package/src/client-settings.ts

@@ -0,0 +1,14 @@
+/**
+ * Settings entry: the components behind the settings pages.
+ *
+ * Loaded by `client/lazy-entries.ts` when one of these elements is on the
+ * page. Never referenced from a script tag.
+ */
+
+import "./client/avatar-upload.js";
+import "./client/components/jant-settings-general.js";
+import "./client/components/jant-settings-language.js";
+import "./client/components/jant-settings-avatar.js";
+import "./client/components/jant-config-editor.js";
+import "./client/components/jant-repo-picker.js";
+import "./client/settings-bridge.js";

package/src/client-site.ts

@@ -2,14 +2,21 @@
  * Static site client entry point (Hugo theme).
  *
  * Loaded on exported Hugo sites to enable reading-surface interactions —
- * media lightbox, feed video autoplay, audio waveform, and gallery scroll
- * hints. Does NOT include Datastar, auth, toast, or form plumbing; those
- * are runtime-only concerns for the authenticated app.
+ * media lightbox, feed video autoplay, audio waveform, gallery scroll hints,
+ * and the thread-context shell. Does NOT include Datastar, auth, toast, or
+ * form plumbing; those are runtime-only concerns for the authenticated app.
  */
 
 import "./client/audio-player.js";
 import "./client/feed-video-player.js";
 import "./client/media-scroll-hint.js";
 import "./client/site-header-nav.js";
+// A thread preview's ancestor context is server-rendered collapsed, and its
+// "Show more" is the only way back out of that. Off-site pages render the same
+// shell — the Hugo export's own thread-preview partial had to leave the shell
+// out entirely because this was missing — so the reading surface owes them the
+// script that makes it work. It binds on DOMContentLoaded to
+// `[data-thread-context-toggle]` and does nothing on a page carrying none.
+import "./client/thread-context.js";
 import "./client/components/jant-media-lightbox.js";
 import "./styles/site-media.css";

package/src/client.ts

@@ -11,8 +11,12 @@ import "./client/audio-player.js";
 import "./client/feed-video-player.js";
 import "./client/media-scroll-hint.js";
 import "./client/form-enter-submit.js";
+import "./client/copy-field.js";
 import "./client/components/jant-media-lightbox.js";
 import "./client/components/jant-text-preview.js";
+// First-run setup is a signed-out page, so the language picker it shares with
+// the settings page has to ship in the public bundle.
+import "./client/components/jant-locale-picker.js";
 import "./client/toast.js";
 import "./client/thread-context.js";
 import "./client/note-expand.js";

package/src/db/__tests__/backfill-nav-label-mirrors.test.ts

@@ -0,0 +1,149 @@
+/**
+ * Executes the real 0006 backfill SQL against snapshot-shaped nav rows.
+ *
+ * The backfill runs on user data through `jant migrate`, and the same file is
+ * executed on both SQLite and Postgres, so it has to be portable and safe to
+ * rerun. These tests cover the SQLite side; the SQL avoids dialect-specific
+ * syntax (no `UPDATE ... FROM`, only correlated subqueries).
+ */
+
+import { readFileSync } from "fs";
+import { resolve } from "path";
+import type BetterSqlite3 from "better-sqlite3";
+import { beforeEach, describe, expect, it } from "vitest";
+import {
+  createTestDatabase,
+  DEFAULT_TEST_SITE_ID,
+} from "../../__tests__/helpers/db.js";
+
+const BACKFILL_SQL = readFileSync(
+  resolve(
+    import.meta.dirname,
+    "../backfills/0006_clear_mirrored_page_nav_labels.sql",
+  ),
+  "utf-8",
+);
+
+const POST_ID = "pst_backfill00000000000000001";
+const COLLECTION_ID = "col_backfill00000000000000001";
+
+describe("0006 backfill — clear nav labels that mirror their target", () => {
+  let sqlite: BetterSqlite3.Database;
+
+  function runBackfill() {
+    sqlite.exec(BACKFILL_SQL);
+  }
+
+  function labelOf(id: string): string {
+    const row = sqlite
+      .prepare(`SELECT label FROM nav_item WHERE id = ?`)
+      .get(id) as { label: string };
+    return row.label;
+  }
+
+  function insertNavItem(row: {
+    id: string;
+    type: "page" | "collection" | "link";
+    label: string;
+    position?: string;
+    postId?: string;
+    collectionId?: string;
+  }) {
+    sqlite
+      .prepare(
+        `INSERT INTO nav_item (
+           id, site_id, type, post_id, collection_id, label, url,
+           placement, position, created_at, updated_at
+         ) VALUES (?, ?, ?, ?, ?, ?, '/x', 'header', ?, 1, 1)`,
+      )
+      .run(
+        row.id,
+        DEFAULT_TEST_SITE_ID,
+        row.type,
+        row.postId ?? null,
+        row.collectionId ?? null,
+        row.label,
+        row.position ?? "a0",
+      );
+  }
+
+  beforeEach(() => {
+    sqlite = createTestDatabase().sqlite;
+    sqlite
+      .prepare(
+        `INSERT INTO post (
+           id, site_id, format, status, visibility, title, thread_id,
+           published_at, last_activity_at, created_at, updated_at
+         ) VALUES (?, ?, 'note', 'published', 'latest_hidden', 'About me', ?, 1, 1, 1, 1)`,
+      )
+      .run(POST_ID, DEFAULT_TEST_SITE_ID, POST_ID);
+    sqlite
+      .prepare(
+        `INSERT INTO collection (id, site_id, title, sort_order, created_at, updated_at)
+         VALUES (?, ?, 'Design Notes', 'newest', 1, 1)`,
+      )
+      .run(COLLECTION_ID, DEFAULT_TEST_SITE_ID);
+  });
+
+  it("clears a page label that only mirrors the post title", () => {
+    insertNavItem({
+      id: "nav_mirror0000000000000000001",
+      type: "page",
+      label: "About me",
+      postId: POST_ID,
+    });
+
+    runBackfill();
+
+    expect(labelOf("nav_mirror0000000000000000001")).toBe("");
+  });
+
+  it("clears a collection label that only mirrors the collection title", () => {
+    insertNavItem({
+      id: "nav_mirror0000000000000000002",
+      type: "collection",
+      label: "Design Notes",
+      collectionId: COLLECTION_ID,
+    });
+
+    runBackfill();
+
+    expect(labelOf("nav_mirror0000000000000000002")).toBe("");
+  });
+
+  it("keeps a label the author typed", () => {
+    insertNavItem({
+      id: "nav_custom0000000000000000001",
+      type: "page",
+      label: "Colophon",
+      postId: POST_ID,
+    });
+    insertNavItem({
+      id: "nav_custom0000000000000000002",
+      type: "link",
+      label: "About me",
+      position: "a1",
+    });
+
+    runBackfill();
+
+    expect(labelOf("nav_custom0000000000000000001")).toBe("Colophon");
+    // A free-form link has no target to follow, so its label is never cleared
+    // even when it happens to read like one.
+    expect(labelOf("nav_custom0000000000000000002")).toBe("About me");
+  });
+
+  it("is safe to rerun", () => {
+    insertNavItem({
+      id: "nav_mirror0000000000000000003",
+      type: "page",
+      label: "About me",
+      postId: POST_ID,
+    });
+
+    runBackfill();
+    runBackfill();
+
+    expect(labelOf("nav_mirror0000000000000000003")).toBe("");
+  });
+});

package/src/db/__tests__/schema-enum-sources.test.ts

@@ -0,0 +1,33 @@
+import { readFileSync } from "node:fs";
+import { resolve } from "node:path";
+import { describe, expect, it } from "vitest";
+
+/**
+ * The schema files must not keep private copies of the value lists that
+ * `types/constants.ts` already owns.
+ *
+ * This is not a style rule. A CHECK constraint is generated from whatever list
+ * the schema file can see, so a private copy that stops matching the shared one
+ * produces a constraint that rejects rows the application considers valid — and
+ * the schema files are the one place where nothing else would catch it, because
+ * `drizzle-kit generate` happily emits the wrong constraint and the types still
+ * line up on both sides.
+ */
+const SCHEMA_FILES = ["../schema.ts", "../pg/schema.ts"];
+
+/** `const NAME = [...] as const`, the shape a value list is declared in. */
+const LOCAL_VALUE_LIST =
+  /const\s+([A-Z][A-Z0-9_]*)\s*=\s*\[[^\]]*\]\s*as const/g;
+
+describe("schema value lists", () => {
+  for (const file of SCHEMA_FILES) {
+    it(`${file} declares no value list of its own`, () => {
+      const source = readFileSync(resolve(__dirname, file), "utf8");
+      const declared = [...source.matchAll(LOCAL_VALUE_LIST)].map(
+        (match) => match[1],
+      );
+
+      expect(declared).toEqual([]);
+    });
+  }
+});