@jant/core 0.6.14 → 0.6.16

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

@@ -8,6 +8,7 @@ type ComposeHarness = HTMLElement & {
   clearLocalDraftFromStorage?: () => void;
   clearEditDraftFromStorage?: (postId: string) => void;
   openEdit?: (id: string) => Promise<void>;
+  openDraft?: (id: string) => Promise<void>;
   openNew?: (options?: { restoreDraft?: boolean }) => Promise<void>;
   openReply?: (
     id: string,
@@ -42,9 +43,18 @@ async function flushBridgeWork(times = 4) {
   }
 }
 
+function renderPostView(postId: string) {
+  const postView = document.createElement("div");
+  postView.dataset.postView = "";
+  postView.dataset.postViewId = postId;
+  document.body.appendChild(postView);
+  return postView;
+}
+
 describe("compose bridge", () => {
   beforeEach(() => {
     document.body.innerHTML = "";
+    delete document.documentElement.dataset.sitePathPrefix;
     vi.restoreAllMocks();
     globalThis.sessionStorage.clear();
   });
@@ -118,6 +128,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",
@@ -453,7 +527,12 @@ describe("compose bridge", () => {
       undefined,
       "pst_root",
       { kind: "timeline-item", id: "pst_root" },
-      { restoreDraft: true, initialFormat: "quote" },
+      {
+        restoreDraft: true,
+        initialFormat: "quote",
+        restoreToast: false,
+        restoreMedia: [],
+      },
     );
     expect(composeEl.openNew).not.toHaveBeenCalled();
     expect(composeEl.clearLocalDraftFromStorage).not.toHaveBeenCalled();
@@ -652,6 +731,241 @@ describe("compose bridge", () => {
 
     await flushBridgeWork();
 
-    expect(fetchSpy).toHaveBeenCalledTimes(2);
+    expect(fetchSpy).toHaveBeenCalledTimes(1);
+  });
+
+  it("keeps an updated draft on its authenticated preview path", async () => {
+    const composeEl = document.createElement(
+      "jant-compose-dialog",
+    ) as ComposeHarness;
+    composeEl.refreshCollections = vi.fn(async () => true);
+    composeEl.pageMode = false;
+    document.body.appendChild(composeEl);
+    const previewBar = document.createElement("aside");
+    previewBar.dataset.previewStatus = "";
+    document.body.appendChild(previewBar);
+    document.documentElement.dataset.sitePathPrefix = "/blog";
+    globalThis.history.replaceState({}, "", "/blog/preview/draft-post");
+
+    const assignSpy = vi
+      .spyOn(globalThis.location, "assign")
+      .mockImplementation(() => {});
+    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");
+
+      expect(url.pathname).toBe("/api/posts/pst_draft");
+      expect(init?.method).toBe("PUT");
+      expect(JSON.parse(String(init?.body))).toMatchObject({
+        status: "draft",
+      });
+
+      return new Response(
+        JSON.stringify({
+          id: "pst_draft",
+          slug: "draft-post",
+          status: "draft",
+        }),
+        {
+          status: 200,
+          headers: { "Content-Type": "application/json" },
+        },
+      );
+    });
+
+    composeEl.dispatchEvent(
+      new CustomEvent("jant:compose-submit-deferred", {
+        bubbles: true,
+        detail: {
+          format: "note",
+          title: "Draft post",
+          body: "Updated draft body",
+          url: "",
+          quoteText: "",
+          quoteAuthor: "",
+          slug: "draft-post",
+          status: "draft",
+          visibility: "public",
+          rating: 0,
+          collectionIds: [],
+          attachments: [],
+          pendingAttachments: [],
+          editPostId: "pst_draft",
+          draftSourceId: "pst_draft",
+        },
+      }),
+    );
+
+    await flushBridgeWork();
+
+    expect(assignSpy).toHaveBeenCalledWith("/blog/preview/draft-post");
+  });
+
+  it("keeps a draft-panel save in place and signals the updated list", async () => {
+    const composeEl = document.createElement(
+      "jant-compose-dialog",
+    ) as ComposeHarness;
+    const refreshCollections = vi.fn(async () => true);
+    composeEl.refreshCollections = refreshCollections;
+    composeEl.pageMode = false;
+    document.body.appendChild(composeEl);
+
+    const assignSpy = vi
+      .spyOn(globalThis.location, "assign")
+      .mockImplementation(() => {});
+    const completeSpy = vi.fn();
+    document.addEventListener("jant:compose-submit-complete", completeSpy, {
+      once: true,
+    });
+    vi.spyOn(globalThis, "fetch").mockResolvedValue(
+      new Response(
+        JSON.stringify({
+          id: "pst_draft",
+          slug: "draft-post",
+          status: "draft",
+        }),
+        {
+          status: 200,
+          headers: { "Content-Type": "application/json" },
+        },
+      ),
+    );
+
+    composeEl.dispatchEvent(
+      new CustomEvent("jant:compose-submit-deferred", {
+        bubbles: true,
+        detail: {
+          format: "note",
+          title: "Draft post",
+          body: "Updated draft body",
+          url: "",
+          quoteText: "",
+          quoteAuthor: "",
+          slug: "draft-post",
+          status: "draft",
+          visibility: "public",
+          rating: 0,
+          collectionIds: [],
+          attachments: [],
+          pendingAttachments: [],
+          editPostId: "pst_draft",
+          draftSourceId: "pst_draft",
+        },
+      }),
+    );
+
+    await flushBridgeWork();
+
+    expect(assignSpy).not.toHaveBeenCalled();
+    expect(refreshCollections).toHaveBeenCalled();
+    expect(completeSpy).toHaveBeenCalledOnce();
+  });
+
+  it("reopens a failed server draft through the draft loader", async () => {
+    const composeEl = document.createElement(
+      "jant-compose-dialog",
+    ) as ComposeHarness;
+    composeEl.refreshCollections = vi.fn(async () => true);
+    composeEl.pageMode = false;
+    composeEl.openDraft = vi.fn(async () => {});
+    composeEl.openEdit = vi.fn(async () => {});
+    document.body.appendChild(composeEl);
+    vi.spyOn(globalThis, "fetch").mockResolvedValue(
+      new Response(JSON.stringify({ error: "Could not save draft." }), {
+        status: 500,
+        headers: { "Content-Type": "application/json" },
+      }),
+    );
+
+    composeEl.dispatchEvent(
+      new CustomEvent("jant:compose-submit-deferred", {
+        bubbles: true,
+        detail: {
+          format: "note",
+          title: "Draft post",
+          body: "Updated draft body",
+          url: "",
+          quoteText: "",
+          quoteAuthor: "",
+          status: "draft",
+          visibility: "public",
+          rating: 0,
+          collectionIds: [],
+          attachments: [],
+          pendingAttachments: [],
+          editPostId: "pst_draft",
+          draftSourceId: "pst_draft",
+        },
+      }),
+    );
+
+    await flushBridgeWork();
+
+    expect(composeEl.openDraft).toHaveBeenCalledWith("pst_draft");
+    expect(composeEl.openEdit).not.toHaveBeenCalled();
+  });
+
+  it("sends a published draft to its public permalink", async () => {
+    const composeEl = document.createElement(
+      "jant-compose-dialog",
+    ) as ComposeHarness;
+    composeEl.refreshCollections = vi.fn(async () => true);
+    composeEl.pageMode = false;
+    document.body.appendChild(composeEl);
+    renderPostView("pst_draft");
+    const previewBar = document.createElement("aside");
+    previewBar.dataset.previewStatus = "";
+    document.body.appendChild(previewBar);
+    document.documentElement.dataset.sitePathPrefix = "/blog";
+    globalThis.history.replaceState({}, "", "/blog/preview/draft-post");
+
+    const assignSpy = vi
+      .spyOn(globalThis.location, "assign")
+      .mockImplementation(() => {});
+    vi.spyOn(globalThis, "fetch").mockResolvedValue(
+      new Response(
+        JSON.stringify({
+          id: "pst_draft",
+          slug: "published-post",
+          status: "published",
+        }),
+        {
+          status: 200,
+          headers: { "Content-Type": "application/json" },
+        },
+      ),
+    );
+
+    composeEl.dispatchEvent(
+      new CustomEvent("jant:compose-submit-deferred", {
+        bubbles: true,
+        detail: {
+          format: "note",
+          title: "Published post",
+          body: "Ready to publish",
+          url: "",
+          quoteText: "",
+          quoteAuthor: "",
+          slug: "published-post",
+          status: "published",
+          visibility: "public",
+          rating: 0,
+          collectionIds: [],
+          attachments: [],
+          pendingAttachments: [],
+          editPostId: "pst_draft",
+          draftSourceId: "pst_draft",
+        },
+      }),
+    );
+
+    await flushBridgeWork();
+
+    expect(assignSpy).toHaveBeenCalledWith("/blog/published-post");
   });
 });

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

@@ -8,6 +8,7 @@ type ComposeHarness = HTMLElement & {
   openNew: (options?: unknown) => Promise<void>;
   openReply: (...args: unknown[]) => Promise<void>;
   openEdit: (id: string) => Promise<void>;
+  openDraft: (id: string) => Promise<void>;
 };
 
 function dispatchShortcut(
@@ -30,6 +31,7 @@ function createComposeHarness(): ComposeHarness {
   composeEl.openNew = vi.fn(async () => {});
   composeEl.openReply = vi.fn(async () => {});
   composeEl.openEdit = vi.fn(async () => {});
+  composeEl.openDraft = vi.fn(async () => {});
   document.body.appendChild(composeEl);
   return composeEl;
 }
@@ -284,6 +286,24 @@ describe("compose shortcuts", () => {
     expect(window.location.search).toBe("");
   });
 
+  it("loads the current draft through the drafts workflow on preview pages", async () => {
+    const composeEl = createComposeHarness();
+    document.body.setAttribute("data-authenticated", "true");
+    const previewBar = document.createElement("aside");
+    previewBar.dataset.previewStatus = "";
+    document.body.appendChild(previewBar);
+    renderThreadDetailPage();
+    globalThis.history.pushState({}, "", "/preview/draft-post?edit=1");
+
+    composeShortcutsTestOnly.openEditFromQueryParam();
+    await Promise.resolve();
+
+    expect(composeEl.openDraft).toHaveBeenCalledWith("post-current");
+    expect(composeEl.openEdit).not.toHaveBeenCalled();
+    expect(window.location.pathname).toBe("/preview/draft-post");
+    expect(window.location.search).toBe("");
+  });
+
   it("keeps using the hovered post for reply shortcuts in the timeline", () => {
     const composeEl = createComposeHarness();
     const article = document.createElement("article");

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);
+  });
 });