@jant/core 0.3.36 → 0.3.37

package/src/routes/api/__tests__/posts.test.ts

@@ -1,12 +1,19 @@
 import { describe, it, expect } from "vitest";
 import { createTestApp } from "../../../__tests__/helpers/app.js";
 import { postsApiRoutes } from "../posts.js";
-import * as sqid from "../../../lib/sqid.js";
 
 describe("Posts API Routes", () => {
   describe("GET /api/posts", () => {
+    it("returns 401 when not authenticated", async () => {
+      const { app } = createTestApp({ authenticated: false });
+      app.route("/api/posts", postsApiRoutes);
+
+      const res = await app.request("/api/posts");
+      expect(res.status).toBe(401);
+    });
+
     it("returns empty list when no posts exist", async () => {
-      const { app } = createTestApp();
+      const { app } = createTestApp({ authenticated: true });
       app.route("/api/posts", postsApiRoutes);
 
       const res = await app.request("/api/posts");
@@ -17,30 +24,30 @@ describe("Posts API Routes", () => {
       expect(body.nextCursor).toBeNull();
     });
 
-    it("returns posts with sqids", async () => {
-      const { app, services } = createTestApp();
+    it("returns posts with IDs", async () => {
+      const { app, services } = createTestApp({ authenticated: true });
       app.route("/api/posts", postsApiRoutes);
 
       await services.posts.create({
         format: "note",
-        body: "Hello world",
+        bodyMarkdown: "Hello world",
       });
 
       const res = await app.request("/api/posts");
       const body = await res.json();
 
       expect(body.posts).toHaveLength(1);
-      expect(body.posts[0].body).toBe("Hello world");
-      expect(body.posts[0].sqid).toBeTruthy();
+      expect(body.posts[0].bodyText).toBe("Hello world");
+      expect(body.posts[0].id).toBeTruthy();
     });
 
     it("includes mediaAttachments in list response", async () => {
-      const { app, services } = createTestApp();
+      const { app, services } = createTestApp({ authenticated: true });
       app.route("/api/posts", postsApiRoutes);
 
       const post = await services.posts.create({
         format: "note",
-        body: "with media",
+        bodyMarkdown: "with media",
       });
 
       const media = await services.media.create({
@@ -63,20 +70,20 @@ describe("Posts API Routes", () => {
       expect(body.posts[0].mediaAttachments[0].mimeType).toBe("image/jpeg");
       expect(body.posts[0].mediaAttachments[0].url).toBeTruthy();
       expect(body.posts[0].mediaAttachments[0].previewUrl).toBeTruthy();
-      expect(body.posts[0].mediaAttachments[0].position).toBe(0);
+      expect(body.posts[0].mediaAttachments[0].position).toBe("a0");
     });
 
     it("filters by status", async () => {
-      const { app, services } = createTestApp();
+      const { app, services } = createTestApp({ authenticated: true });
       app.route("/api/posts", postsApiRoutes);
 
       await services.posts.create({
         format: "note",
-        body: "published post",
+        bodyMarkdown: "published post",
       });
       await services.posts.create({
         format: "note",
-        body: "draft post",
+        bodyMarkdown: "draft post",
         status: "draft",
       });
 
@@ -88,13 +95,13 @@ describe("Posts API Routes", () => {
     });
 
     it("supports limit parameter", async () => {
-      const { app, services } = createTestApp();
+      const { app, services } = createTestApp({ authenticated: true });
       app.route("/api/posts", postsApiRoutes);
 
       for (let i = 0; i < 5; i++) {
         await services.posts.create({
           format: "note",
-          body: `post ${i}`,
+          bodyMarkdown: `post ${i}`,
         });
       }
 
@@ -107,31 +114,41 @@ describe("Posts API Routes", () => {
   });
 
   describe("GET /api/posts/:id", () => {
-    it("returns a post by sqid", async () => {
-      const { app, services } = createTestApp();
+    it("returns 401 when not authenticated", async () => {
+      const { app, services } = createTestApp({ authenticated: false });
       app.route("/api/posts", postsApiRoutes);
 
       const post = await services.posts.create({
         format: "note",
-        body: "test post",
+        bodyMarkdown: "test post",
       });
-      const id = sqid.encode(post.id);
+      const res = await app.request(`/api/posts/${post.id}`);
+      expect(res.status).toBe(401);
+    });
+
+    it("returns a post by ID", async () => {
+      const { app, services } = createTestApp({ authenticated: true });
+      app.route("/api/posts", postsApiRoutes);
 
-      const res = await app.request(`/api/posts/${id}`);
+      const post = await services.posts.create({
+        format: "note",
+        bodyMarkdown: "test post",
+      });
+      const res = await app.request(`/api/posts/${post.id}`);
       expect(res.status).toBe(200);
 
       const body = await res.json();
-      expect(body.body).toBe("test post");
-      expect(body.sqid).toBe(id);
+      expect(body.bodyText).toBe("test post");
+      expect(body.id).toBe(post.id);
     });
 
     it("includes mediaAttachments in single post response", async () => {
-      const { app, services } = createTestApp();
+      const { app, services } = createTestApp({ authenticated: true });
       app.route("/api/posts", postsApiRoutes);
 
       const post = await services.posts.create({
         format: "note",
-        body: "with media",
+        bodyMarkdown: "with media",
       });
 
       const media = await services.media.create({
@@ -144,15 +161,15 @@ describe("Posts API Routes", () => {
 
       await services.media.attachToPost(post.id, [media.id]);
 
-      const res = await app.request(`/api/posts/${sqid.encode(post.id)}`);
+      const res = await app.request(`/api/posts/${post.id}`);
       const body = await res.json();
 
       expect(body.mediaAttachments).toHaveLength(1);
       expect(body.mediaAttachments[0].id).toBe(media.id);
     });
 
-    it("returns 400 for invalid sqid", async () => {
-      const { app } = createTestApp();
+    it("returns 400 for invalid ID", async () => {
+      const { app } = createTestApp({ authenticated: true });
       app.route("/api/posts", postsApiRoutes);
 
       const res = await app.request("/api/posts/!!invalid!!");
@@ -160,10 +177,12 @@ describe("Posts API Routes", () => {
     });
 
     it("returns 404 for non-existent post", async () => {
-      const { app } = createTestApp();
+      const { app } = createTestApp({ authenticated: true });
       app.route("/api/posts", postsApiRoutes);
 
-      const res = await app.request(`/api/posts/${sqid.encode(9999)}`);
+      const res = await app.request(
+        "/api/posts/00000000-0000-0000-0000-000000009999",
+      );
       expect(res.status).toBe(404);
     });
   });
@@ -178,7 +197,7 @@ describe("Posts API Routes", () => {
         headers: { "Content-Type": "application/json" },
         body: JSON.stringify({
           format: "note",
-          body: "test",
+          bodyMarkdown: "test",
         }),
       });
 
@@ -194,18 +213,57 @@ describe("Posts API Routes", () => {
         headers: { "Content-Type": "application/json" },
         body: JSON.stringify({
           format: "note",
-          body: "Hello from API",
+          bodyMarkdown: "Hello from API",
         }),
       });
 
       expect(res.status).toBe(201);
 
       const body = await res.json();
-      expect(body.body).toBe("Hello from API");
-      expect(body.sqid).toBeTruthy();
+      expect(body.bodyText).toBe("Hello from API");
+      expect(body.id).toBeTruthy();
       expect(body.mediaAttachments).toEqual([]);
     });
 
+    it("creates a post with bodyMarkdown", async () => {
+      const { app } = createTestApp({ authenticated: true });
+      app.route("/api/posts", postsApiRoutes);
+
+      const res = await app.request("/api/posts", {
+        method: "POST",
+        headers: { "Content-Type": "application/json" },
+        body: JSON.stringify({
+          format: "note",
+          bodyMarkdown: "Hello **bold** world",
+        }),
+      });
+
+      expect(res.status).toBe(201);
+
+      const body = await res.json();
+      expect(body.bodyText).toContain("Hello");
+      expect(body.bodyHtml).toContain("<strong>bold</strong>");
+    });
+
+    it("returns 400 when both body and bodyMarkdown are provided", async () => {
+      const { app } = createTestApp({ authenticated: true });
+      app.route("/api/posts", postsApiRoutes);
+
+      const res = await app.request("/api/posts", {
+        method: "POST",
+        headers: { "Content-Type": "application/json" },
+        body: JSON.stringify({
+          format: "note",
+          body: '{"type":"doc","content":[]}',
+          bodyMarkdown: "Hello",
+        }),
+      });
+
+      expect(res.status).toBe(400);
+      const body = await res.json();
+      expect(body.error).toContain("Provide either body or bodyMarkdown");
+    });
+
     it("creates a post with mediaIds and attaches them", async () => {
       const { app, services } = createTestApp({ authenticated: true });
       app.route("/api/posts", postsApiRoutes);
@@ -230,7 +288,7 @@ describe("Posts API Routes", () => {
         headers: { "Content-Type": "application/json" },
         body: JSON.stringify({
           format: "note",
-          body: "with images",
+          bodyMarkdown: "with images",
           mediaIds: [m1.id, m2.id],
         }),
       });
@@ -239,9 +297,9 @@ describe("Posts API Routes", () => {
       const body = await res.json();
       expect(body.mediaAttachments).toHaveLength(2);
       expect(body.mediaAttachments[0].id).toBe(m1.id);
-      expect(body.mediaAttachments[0].position).toBe(0);
+      expect(body.mediaAttachments[0].position).toBe("a0");
       expect(body.mediaAttachments[1].id).toBe(m2.id);
-      expect(body.mediaAttachments[1].position).toBe(1);
+      expect(body.mediaAttachments[1].position).toBe("a1");
     });
 
     it("returns 400 for invalid media IDs", async () => {
@@ -253,7 +311,7 @@ describe("Posts API Routes", () => {
         headers: { "Content-Type": "application/json" },
         body: JSON.stringify({
           format: "note",
-          body: "test",
+          bodyMarkdown: "test",
           mediaIds: ["nonexistent-id"],
         }),
       });
@@ -300,13 +358,13 @@ describe("Posts API Routes", () => {
 
       const post = await services.posts.create({
         format: "note",
-        body: "original",
+        bodyMarkdown: "original",
       });
 
-      const res = await app.request(`/api/posts/${sqid.encode(post.id)}`, {
+      const res = await app.request(`/api/posts/${post.id}`, {
         method: "PUT",
         headers: { "Content-Type": "application/json" },
-        body: JSON.stringify({ body: "updated" }),
+        body: JSON.stringify({ bodyMarkdown: "updated" }),
       });
 
       expect(res.status).toBe(401);
@@ -318,18 +376,18 @@ describe("Posts API Routes", () => {
 
       const post = await services.posts.create({
         format: "note",
-        body: "original",
+        bodyMarkdown: "original",
       });
 
-      const res = await app.request(`/api/posts/${sqid.encode(post.id)}`, {
+      const res = await app.request(`/api/posts/${post.id}`, {
         method: "PUT",
         headers: { "Content-Type": "application/json" },
-        body: JSON.stringify({ body: "updated" }),
+        body: JSON.stringify({ bodyMarkdown: "updated" }),
       });
 
       expect(res.status).toBe(200);
       const body = await res.json();
-      expect(body.body).toBe("updated");
+      expect(body.bodyText).toBe("updated");
       expect(body.mediaAttachments).toEqual([]);
     });
 
@@ -339,7 +397,7 @@ describe("Posts API Routes", () => {
 
       const post = await services.posts.create({
         format: "note",
-        body: "test",
+        bodyMarkdown: "test",
       });
 
       const m1 = await services.media.create({
@@ -360,7 +418,7 @@ describe("Posts API Routes", () => {
         storageKey: "media/2025/01/b.jpg",
       });
 
-      const res = await app.request(`/api/posts/${sqid.encode(post.id)}`, {
+      const res = await app.request(`/api/posts/${post.id}`, {
         method: "PUT",
         headers: { "Content-Type": "application/json" },
         body: JSON.stringify({ mediaIds: [m2.id] }),
@@ -378,7 +436,7 @@ describe("Posts API Routes", () => {
 
       const post = await services.posts.create({
         format: "note",
-        body: "test",
+        bodyMarkdown: "test",
       });
 
       const m1 = await services.media.create({
@@ -391,10 +449,10 @@ describe("Posts API Routes", () => {
 
       await services.media.attachToPost(post.id, [m1.id]);
 
-      const res = await app.request(`/api/posts/${sqid.encode(post.id)}`, {
+      const res = await app.request(`/api/posts/${post.id}`, {
         method: "PUT",
         headers: { "Content-Type": "application/json" },
-        body: JSON.stringify({ body: "updated content" }),
+        body: JSON.stringify({ bodyMarkdown: "updated content" }),
       });
 
       expect(res.status).toBe(200);
@@ -407,11 +465,14 @@ describe("Posts API Routes", () => {
       const { app } = createTestApp({ authenticated: true });
       app.route("/api/posts", postsApiRoutes);
 
-      const res = await app.request(`/api/posts/${sqid.encode(9999)}`, {
-        method: "PUT",
-        headers: { "Content-Type": "application/json" },
-        body: JSON.stringify({ body: "test" }),
-      });
+      const res = await app.request(
+        "/api/posts/00000000-0000-0000-0000-000000009999",
+        {
+          method: "PUT",
+          headers: { "Content-Type": "application/json" },
+          body: JSON.stringify({ bodyMarkdown: "test" }),
+        },
+      );
 
       expect(res.status).toBe(404);
     });
@@ -422,10 +483,10 @@ describe("Posts API Routes", () => {
 
       const post = await services.posts.create({
         format: "note",
-        body: "test",
+        bodyMarkdown: "test",
       });
 
-      const res = await app.request(`/api/posts/${sqid.encode(post.id)}`, {
+      const res = await app.request(`/api/posts/${post.id}`, {
         method: "PUT",
         headers: { "Content-Type": "application/json" },
         body: JSON.stringify({ format: "invalid-type" }),
@@ -442,10 +503,10 @@ describe("Posts API Routes", () => {
 
       const post = await services.posts.create({
         format: "note",
-        body: "test",
+        bodyMarkdown: "test",
       });
 
-      const res = await app.request(`/api/posts/${sqid.encode(post.id)}`, {
+      const res = await app.request(`/api/posts/${post.id}`, {
         method: "DELETE",
       });
 
@@ -458,10 +519,10 @@ describe("Posts API Routes", () => {
 
       const post = await services.posts.create({
         format: "note",
-        body: "to be deleted",
+        bodyMarkdown: "to be deleted",
       });
 
-      const res = await app.request(`/api/posts/${sqid.encode(post.id)}`, {
+      const res = await app.request(`/api/posts/${post.id}`, {
         method: "DELETE",
       });
 
@@ -478,9 +539,12 @@ describe("Posts API Routes", () => {
       const { app } = createTestApp({ authenticated: true });
       app.route("/api/posts", postsApiRoutes);
 
-      const res = await app.request(`/api/posts/${sqid.encode(9999)}`, {
-        method: "DELETE",
-      });
+      const res = await app.request(
+        "/api/posts/00000000-0000-0000-0000-000000009999",
+        {
+          method: "DELETE",
+        },
+      );
 
       expect(res.status).toBe(404);
     });
@@ -491,7 +555,7 @@ describe("Posts API Routes", () => {
 
       const post = await services.posts.create({
         format: "note",
-        body: "with media",
+        bodyMarkdown: "with media",
       });
 
       const m1 = await services.media.create({
@@ -511,7 +575,7 @@ describe("Posts API Routes", () => {
 
       await services.media.attachToPost(post.id, [m1.id, m2.id]);
 
-      const res = await app.request(`/api/posts/${sqid.encode(post.id)}`, {
+      const res = await app.request(`/api/posts/${post.id}`, {
         method: "DELETE",
       });
 
@@ -528,11 +592,11 @@ describe("Posts API Routes", () => {
 
       const root = await services.posts.create({
         format: "note",
-        body: "thread root",
+        bodyMarkdown: "thread root",
       });
       const reply = await services.posts.create({
         format: "note",
-        body: "reply",
+        bodyMarkdown: "reply",
         replyToId: root.id,
       });
 
@@ -554,7 +618,7 @@ describe("Posts API Routes", () => {
       await services.media.attachToPost(root.id, [rootMedia.id]);
       await services.media.attachToPost(reply.id, [replyMedia.id]);
 
-      const res = await app.request(`/api/posts/${sqid.encode(root.id)}`, {
+      const res = await app.request(`/api/posts/${root.id}`, {
         method: "DELETE",
       });
 

package/src/routes/api/__tests__/search.test.ts

@@ -2,6 +2,19 @@ import { describe, it, expect } from "vitest";
 import { createTestApp } from "../../../__tests__/helpers/app.js";
 import { searchApiRoutes } from "../search.js";
 
+/** Wraps plain text in a minimal valid TipTap JSON document. */
+function tiptapDoc(text: string): string {
+  return JSON.stringify({
+    type: "doc",
+    content: [
+      {
+        type: "paragraph",
+        content: [{ type: "text", text }],
+      },
+    ],
+  });
+}
+
 describe("Search API Routes", () => {
   it("returns 400 when query is missing", async () => {
     const { app } = createTestApp({ fts: true });
@@ -40,7 +53,7 @@ describe("Search API Routes", () => {
 
     await services.posts.create({
       format: "note",
-      body: "Testing search functionality in jant",
+      body: tiptapDoc("Testing search functionality in jant"),
     });
 
     const res = await app.request("/api/search?q=jant");
@@ -50,7 +63,7 @@ describe("Search API Routes", () => {
     expect(body.query).toBe("jant");
     expect(body.results.length).toBeGreaterThanOrEqual(1);
     expect(body.count).toBeGreaterThanOrEqual(1);
-    expect(body.results[0].url).toMatch(/^\/p\//);
+    expect(body.results[0].url).toMatch(/^\/[a-z0-9]/);
   });
 
   it("returns empty results for non-matching query", async () => {