@jant/core 0.6.14 → 0.6.15

package/src/__tests__/helpers/export-fixtures.ts

@@ -88,6 +88,8 @@ export function makeSiteConfig(over: Partial<SiteConfig> = {}): SiteConfig {
     siteDescription: "Export fixture",
     siteLanguage: "en",
     showJantBrandingOnHome: true,
+    publicApiEnabled: true,
+    rssFeedsEnabled: true,
     mainRssFeed: "latest",
     siteFooter: "",
     showHeaderAvatar: false,

package/src/__tests__/import-site-command.test.ts

@@ -17,8 +17,10 @@ import { __test__ } from "../../bin/commands/import-site.js";
 const {
   walkHugoContent,
   loadSiteConfig,
+  buildSettingsUpdatesFromConfig,
   mediaSpecFromJantMedia,
   resolveCollectionMemberships,
+  resolveThreadCollectionMemberships,
   buildPostPayloadFromBundle,
   getRootAliasPathsForImport,
   uploadMediaList,
@@ -101,6 +103,8 @@ describe("Hugo import CLI helpers", () => {
         "show_jant_branding_on_home = true",
         "show_header_avatar = false",
         "noindex = false",
+        "public_api_enabled = false",
+        "rss_feeds_enabled = false",
         'site_avatar_mode = "none"',
         'favicon_mode = "default"',
         'apple_touch_mode = "default"',
@@ -135,6 +139,12 @@ describe("Hugo import CLI helpers", () => {
     expect(siteConfig.title).toBe("Example Site");
     expect(siteConfig.base_url).toBe("https://example.com/");
     expect(siteConfig.extra.jant.theme_id).toBe("paper");
+    expect(siteConfig.extra.jant.public_api_enabled).toBe(false);
+    expect(siteConfig.extra.jant.rss_feeds_enabled).toBe(false);
+    expect(buildSettingsUpdatesFromConfig(siteConfig)).toMatchObject({
+      PUBLIC_API_ENABLED: "false",
+      RSS_FEEDS_ENABLED: "false",
+    });
     expect(siteConfig.extra.jant).not.toHaveProperty("home_default_view");
     expect(siteConfig.extra.jant.nav_exported).toBe(true);
     expect(siteConfig.extra.jant.nav).toHaveLength(1);
@@ -297,6 +307,61 @@ describe("Hugo import CLI helpers", () => {
     ]);
   });
 
+  it("unions legacy per-post Collection metadata across a Thread", () => {
+    const slugToId = new Map([
+      ["ideas", "col_ideas"],
+      ["walks", "col_walks"],
+    ]);
+    const rootBundle = {
+      frontMatter: {
+        collections: [
+          {
+            slug: "ideas",
+            collected_at: "2026-03-01T00:00:00Z",
+            position: 5,
+            pinned_at: "2026-03-05T00:00:00Z",
+          },
+        ],
+      },
+      children: [
+        {
+          frontMatter: {
+            collections: [
+              {
+                slug: "ideas",
+                collected_at: "2026-03-03T00:00:00Z",
+                position: 2,
+                pinned_at: "2026-03-04T00:00:00Z",
+              },
+              {
+                slug: "walks",
+                collected_at: "2026-03-02T00:00:00Z",
+                position: 7,
+              },
+            ],
+          },
+        },
+      ],
+    };
+
+    expect(resolveThreadCollectionMemberships(rootBundle, slugToId)).toEqual({
+      entries: [
+        {
+          collectionId: "col_ideas",
+          createdAt: Date.parse("2026-03-03T00:00:00Z") / 1000,
+          position: 2,
+          pinnedAt: Date.parse("2026-03-05T00:00:00Z") / 1000,
+        },
+        {
+          collectionId: "col_walks",
+          createdAt: Date.parse("2026-03-02T00:00:00Z") / 1000,
+          position: 7,
+        },
+      ],
+      ids: ["col_ideas", "col_walks"],
+    });
+  });
+
   it("buildPostPayloadFromBundle translates front matter into createPost input", () => {
     const bundle = {
       slug: "hello",

package/src/__tests__/vite-config.test.ts

@@ -0,0 +1,45 @@
+import { mkdtempSync, rmSync, writeFileSync } from "node:fs";
+import { tmpdir } from "node:os";
+import { join } from "node:path";
+import { afterEach, describe, expect, it, vi } from "vitest";
+import { resolveWorkerDevProcessOptions } from "../../vite.config.js";
+
+describe("worker Vite config", () => {
+  const tempDirs: string[] = [];
+
+  afterEach(() => {
+    vi.unstubAllEnvs();
+    for (const dir of tempDirs.splice(0)) {
+      rmSync(dir, { recursive: true, force: true });
+    }
+  });
+
+  function createEnvDir(files: Record<string, string>): string {
+    const dir = mkdtempSync(join(tmpdir(), "jant-vite-env-"));
+    tempDirs.push(dir);
+
+    for (const [name, contents] of Object.entries(files)) {
+      writeFileSync(join(dir, name), contents);
+    }
+
+    return dir;
+  }
+
+  it("loads the worker dev port from .env", () => {
+    vi.stubEnv("PORT", undefined);
+    const envDir = createEnvDir({ ".env": "PORT=3012\n" });
+
+    expect(resolveWorkerDevProcessOptions("development", envDir).port).toBe(
+      3012,
+    );
+  });
+
+  it("lets the shell environment override .env", () => {
+    const envDir = createEnvDir({ ".env": "PORT=3012\n" });
+    vi.stubEnv("PORT", "4012");
+
+    expect(resolveWorkerDevProcessOptions("development", envDir).port).toBe(
+      4012,
+    );
+  });
+});

package/src/app.tsx

@@ -55,6 +55,7 @@ import {
 import { telegramWebhookRoutes } from "./routes/api/telegram.js";
 import { internalTextAttachmentsRoutes } from "./routes/api/internal/text-attachments.js";
 import { internalSearchReindexRoutes } from "./routes/api/internal/search-reindex.js";
+import { internalPostBodyHtmlRoutes } from "./routes/api/internal/post-body-html.js";
 import { internalUploadsRoutes } from "./routes/api/internal/uploads.js";
 import { publicArchiveApiRoutes } from "./routes/api/public/archive.js";
 import { publicPostsApiRoutes } from "./routes/api/public/posts.js";
@@ -65,11 +66,13 @@ import { composeRoutes } from "./routes/compose.js";
 import { feedRoutes } from "./routes/feed/feed.js";
 import { sitemapRoutes } from "./routes/feed/sitemap.js";
 import { manifestRoutes } from "./routes/feed/manifest.js";
+import { siteSkillRoutes } from "./routes/site-skill.js";
 
 // Middleware
 import { requireAuth } from "./middleware/auth.js";
 import { attachSession } from "./middleware/session.js";
 import { defaultCacheControl } from "./middleware/cache-control.js";
+import { requireRssFeedsEnabled } from "./middleware/public-content-access.js";
 import { requireOnboarding } from "./middleware/onboarding.js";
 import { errorHandler } from "./middleware/error-handler.js";
 import { withConfig } from "./middleware/config.js";
@@ -355,6 +358,7 @@ export function createApp(): App {
   app.route("/api/internal/sites", internalSitesRoutes);
   app.route("/api/internal/text-attachments", internalTextAttachmentsRoutes);
   app.route("/api/internal/search/reindex", internalSearchReindexRoutes);
+  app.route("/api/internal/posts/body-html", internalPostBodyHtmlRoutes);
   app.route("/api/internal/uploads", internalUploadsRoutes);
   app.route("/api/github-sync", githubSyncWebhookRoutes);
   app.route("/api/telegram", telegramWebhookRoutes);
@@ -382,7 +386,9 @@ export function createApp(): App {
       if (!object) return c.notFound();
 
       const markdown = await new Response(object.body).text();
-      const html = renderTiptapJson(markdownToTiptapJson(markdown));
+      const html = renderTiptapJson(markdownToTiptapJson(markdown), {
+        namespace: media.id,
+      });
 
       return c.json({ html, markdown }, 200, {
         "Cache-Control": "public, no-cache",
@@ -485,8 +491,8 @@ export function createApp(): App {
   // Redirect middleware — only handles redirect-type custom URLs
   app.use("*", async (c, next) => {
     const path = new URL(c.req.url).pathname;
-    // Skip redirect check for API routes and static assets
-    if (path.startsWith("/api/") || isAssetPath(path)) {
+    // Skip redirect lookup for fixed machine routes and static assets.
+    if (path.startsWith("/api/") || path === "/skill.md" || isAssetPath(path)) {
       return next();
     }
 
@@ -511,8 +517,15 @@ export function createApp(): App {
   app.use("*", withConfig());
   app.use("*", i18nMiddleware());
 
+  // Feed URLs are spread across page route groups (including dynamic
+  // collection aliases), so one config-aware guard owns the URL policy.
+  app.use("*", requireRssFeedsEnabled());
+
   // --- Routes that need config ---
 
+  // Public machine-readable site guide
+  app.route("/", siteSkillRoutes);
+
   // API Routes
   app.route("/api/public/posts", publicPostsApiRoutes);
   app.route("/api/public/archive", publicArchiveApiRoutes);

package/src/client/__tests__/collection-created-notice.test.ts

@@ -0,0 +1,36 @@
+// @vitest-environment happy-dom
+
+import { beforeEach, describe, expect, it } from "vitest";
+import {
+  consumeCollectionCreatedNotice,
+  PENDING_COLLECTION_CREATED_STORAGE_KEY,
+  queueCollectionCreatedNotice,
+} from "../collection-created-notice.js";
+
+describe("collection created notice", () => {
+  beforeEach(() => {
+    globalThis.sessionStorage.clear();
+  });
+
+  it("queues and consumes the newly created Collection ID once", () => {
+    queueCollectionCreatedNotice("col_books");
+
+    expect(
+      globalThis.sessionStorage.getItem(PENDING_COLLECTION_CREATED_STORAGE_KEY),
+    ).toBe('{"collectionId":"col_books"}');
+    expect(consumeCollectionCreatedNotice()).toBe("col_books");
+    expect(consumeCollectionCreatedNotice()).toBeNull();
+  });
+
+  it("clears malformed pending state", () => {
+    globalThis.sessionStorage.setItem(
+      PENDING_COLLECTION_CREATED_STORAGE_KEY,
+      '{"collectionId":false}',
+    );
+
+    expect(consumeCollectionCreatedNotice()).toBeNull();
+    expect(
+      globalThis.sessionStorage.getItem(PENDING_COLLECTION_CREATED_STORAGE_KEY),
+    ).toBeNull();
+  });
+});

package/src/client/__tests__/collection-form-bridge.test.ts

@@ -6,6 +6,10 @@ vi.mock("../toast.js", () => ({
   showToast: vi.fn(),
 }));
 
+vi.mock("../collection-created-notice.js", () => ({
+  queueCollectionCreatedNotice: vi.fn(),
+}));
+
 import "../collection-form-bridge.js";
 
 type CollectionFormHarness = HTMLElement & {
@@ -50,6 +54,7 @@ async function flushAsyncWork() {
 describe("collection form bridge", () => {
   beforeEach(() => {
     document.body.innerHTML = "";
+    vi.clearAllMocks();
     vi.restoreAllMocks();
     window.location.href = "http://localhost/collections/new";
   });
@@ -57,7 +62,7 @@ describe("collection form bridge", () => {
   it("redirects new collections back to the configured return URL", async () => {
     const formEl = createPageForm("/collections");
     const fetchSpy = vi.spyOn(globalThis, "fetch").mockResolvedValue(
-      new Response(JSON.stringify({ slug: "books" }), {
+      new Response(JSON.stringify({ id: "col_books", slug: "books" }), {
         status: 201,
         headers: { "Content-Type": "application/json" },
       }),
@@ -79,6 +84,9 @@ describe("collection form bridge", () => {
 
     await flushAsyncWork();
 
+    const { queueCollectionCreatedNotice } =
+      await import("../collection-created-notice.js");
+
     expect(fetchSpy).toHaveBeenCalledWith("/api/collections", {
       method: "POST",
       headers: {
@@ -91,6 +99,7 @@ describe("collection form bridge", () => {
       }),
     });
     expect(window.location.pathname).toBe("/collections");
+    expect(queueCollectionCreatedNotice).toHaveBeenCalledWith("col_books");
   });
 
   it("redirects edited collections back to the collections page when launched from the list", async () => {

package/src/client/__tests__/collection-page-actions.test.ts

@@ -4,6 +4,7 @@ import { beforeEach, describe, expect, it, vi } from "vitest";
 
 vi.mock("../toast.js", () => ({
   showToast: vi.fn(),
+  showToastWithAction: vi.fn(),
 }));
 
 vi.mock("../confirm.js", () => ({
@@ -16,7 +17,8 @@ function createMarkup() {
     <div
       data-collection-page-actions
       data-collection-id="collection-1"
-      data-collection-page-labels='{"edit":"Edit","moreActions":"More actions","deleteCollection":"Delete","confirmDelete":"Delete this collection permanently? Posts inside won\\u0027t be removed.","cancel":"Cancel","saveFailed":"Couldn\\u0027t save. Try again in a moment.","deleted":"Deleted"}'
+      data-collection-page-labels='{"edit":"Edit","addToNavigation":"Add to Navigation","addingToNavigation":"Adding…","addedToNavigation":"Collection added to navigation.","editNavigation":"Edit Navigation","addToNavigationFailed":"Couldn\\u0027t add this collection to navigation. Try again.","moreActions":"More actions","deleteCollection":"Delete","confirmDelete":"Delete this collection permanently? Threads inside won\\u0027t be removed.","cancel":"Cancel","saveFailed":"Couldn\\u0027t save. Try again in a moment.","deleted":"Deleted"}'
+      data-collection-in-navigation="false"
       data-collection-page-redirect-url="/collections"
     >
       <button
@@ -30,6 +32,21 @@ function createMarkup() {
         <a href="/collections/original-slug/edit?returnTo=%2Foriginal-slug" role="menuitem">
           Edit
         </a>
+        <button
+          type="button"
+          role="menuitem"
+          data-collection-page-action="add-to-navigation"
+        >
+          <span class="collections-page-menu-item-label">Add to Navigation</span>
+        </button>
+        <a
+          href="/settings/navigation"
+          role="menuitem"
+          data-collection-page-edit-navigation
+          hidden
+        >
+          Edit Navigation
+        </a>
         <button
           type="button"
           role="menuitem"
@@ -42,6 +59,11 @@ function createMarkup() {
   `;
 }
 
+async function flushAsyncWork() {
+  await new Promise((resolve) => setTimeout(resolve, 0));
+  await new Promise((resolve) => setTimeout(resolve, 0));
+}
+
 describe("collection detail page actions", () => {
   beforeEach(() => {
     vi.resetModules();
@@ -98,7 +120,7 @@ describe("collection detail page actions", () => {
 
     expect(showConfirmDialog).toHaveBeenCalledWith({
       message:
-        "Delete this collection permanently? Posts inside won't be removed.",
+        "Delete this collection permanently? Threads inside won't be removed.",
       confirmLabel: "Delete",
       cancelLabel: "Cancel",
       tone: "danger",
@@ -109,4 +131,51 @@ describe("collection detail page actions", () => {
     expect(showToast).toHaveBeenCalledWith("Deleted");
     expect(window.location.pathname).toBe("/collections");
   });
+
+  it("adds the collection to navigation and reveals the settings action", async () => {
+    createMarkup();
+    const fetchMock = vi.fn().mockResolvedValue(
+      new Response(JSON.stringify({ id: "nav-1" }), {
+        status: 201,
+        headers: { "Content-Type": "application/json" },
+      }),
+    );
+    vi.stubGlobal("fetch", fetchMock);
+
+    const { showToastWithAction } = await import("../toast.js");
+    await import("../collection-page-actions.js");
+
+    const addButton = document.querySelector<HTMLButtonElement>(
+      "[data-collection-page-action='add-to-navigation']",
+    );
+    const editNavigationLink = document.querySelector<HTMLAnchorElement>(
+      "[data-collection-page-edit-navigation]",
+    );
+
+    addButton?.click();
+    await flushAsyncWork();
+
+    expect(fetchMock).toHaveBeenCalledWith("/api/nav-items", {
+      method: "POST",
+      headers: {
+        "Content-Type": "application/json",
+        Accept: "application/json",
+        "X-Jant-Site-Header": "include",
+      },
+      body: JSON.stringify({
+        type: "collection",
+        collectionId: "collection-1",
+        placement: "header",
+      }),
+    });
+    expect(addButton?.hidden).toBe(true);
+    expect(editNavigationLink?.hidden).toBe(false);
+    expect(showToastWithAction).toHaveBeenCalledWith(
+      "Collection added to navigation.",
+      {
+        label: "Edit Navigation",
+        href: "/settings/navigation",
+      },
+    );
+  });
 });

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

@@ -118,6 +118,70 @@ describe("compose bridge", () => {
     expect(refreshCollections).toHaveBeenCalledTimes(1);
   });
 
+  it("omits stale Collection state from Reply requests", async () => {
+    const composeEl = document.createElement(
+      "jant-compose-dialog",
+    ) as ComposeHarness;
+    composeEl.refreshCollections = vi.fn(async () => true);
+    composeEl.pageMode = false;
+    document.body.appendChild(composeEl);
+
+    const fetchSpy = vi
+      .spyOn(globalThis, "fetch")
+      .mockImplementation(async (input, init) => {
+        const raw =
+          typeof input === "string"
+            ? input
+            : input instanceof URL
+              ? input.toString()
+              : input.url;
+        const url = new URL(raw, "http://localhost");
+
+        if (url.pathname === "/compose") {
+          const body = JSON.parse(String(init?.body)) as Record<
+            string,
+            unknown
+          >;
+          expect(body.replyToId).toBe("pst_parent");
+          expect(body).not.toHaveProperty("collectionIds");
+
+          return new Response(
+            JSON.stringify({ status: "draft", toast: "Draft saved." }),
+            {
+              status: 200,
+              headers: { "Content-Type": "application/json" },
+            },
+          );
+        }
+
+        throw new Error(`Unexpected fetch: ${url.pathname}`);
+      });
+
+    composeEl.dispatchEvent(
+      new CustomEvent("jant:compose-submit-deferred", {
+        bubbles: true,
+        detail: {
+          format: "note",
+          title: "",
+          body: "Reply draft",
+          url: "",
+          quoteText: "",
+          quoteAuthor: "",
+          status: "draft",
+          rating: 0,
+          collectionIds: ["col-stale"],
+          attachments: [],
+          pendingAttachments: [],
+          replyToId: "pst_parent",
+        },
+      }),
+    );
+
+    await flushBridgeWork();
+
+    expect(fetchSpy).toHaveBeenCalledOnce();
+  });
+
   it("stays on page and resets form after page-mode publish", async () => {
     const composeEl = document.createElement(
       "jant-compose-dialog",

package/src/client/__tests__/feed-video-player.test.ts

@@ -1,7 +1,11 @@
 // @vitest-environment happy-dom
 
 import { describe, expect, it } from "vitest";
-import { chooseAutoplayVideo } from "../feed-video-player.js";
+import {
+  chooseAutoplayVideo,
+  shouldSuppressFeedVideoAutoplay,
+} from "../feed-video-player.js";
+import { setMediaVideoPlaybackPaused } from "../media-lightbox-events.js";
 
 describe("chooseAutoplayVideo", () => {
   it("prefers the candidate with the largest visible area", () => {
@@ -41,4 +45,15 @@ describe("chooseAutoplayVideo", () => {
 
     expect(winner?.video).toBe("second");
   });
+
+  it("suppresses only media the user explicitly paused", () => {
+    setMediaVideoPlaybackPaused("media-paused", true);
+
+    expect(shouldSuppressFeedVideoAutoplay("media-paused")).toBe(true);
+    expect(shouldSuppressFeedVideoAutoplay("media-playing")).toBe(false);
+    expect(shouldSuppressFeedVideoAutoplay(undefined)).toBe(false);
+
+    setMediaVideoPlaybackPaused("media-paused", false);
+    expect(shouldSuppressFeedVideoAutoplay("media-paused")).toBe(false);
+  });
 });