@jant/core 0.3.36 → 0.3.37

package/src/lib/__tests__/sqid.test.ts

@@ -1,65 +0,0 @@
-import { describe, it, expect } from "vitest";
-import { encode, decode, isValidSqid } from "../sqid.js";
-
-describe("encode", () => {
-  it("encodes a numeric ID to a string", () => {
-    const result = encode(1);
-    expect(typeof result).toBe("string");
-    expect(result.length).toBeGreaterThanOrEqual(4);
-  });
-
-  it("produces minimum 4-character strings", () => {
-    expect(encode(0).length).toBeGreaterThanOrEqual(4);
-    expect(encode(1).length).toBeGreaterThanOrEqual(4);
-    expect(encode(100).length).toBeGreaterThanOrEqual(4);
-  });
-
-  it("produces different strings for different IDs", () => {
-    const a = encode(1);
-    const b = encode(2);
-    const c = encode(100);
-    expect(a).not.toBe(b);
-    expect(b).not.toBe(c);
-  });
-
-  it("produces consistent results for the same ID", () => {
-    expect(encode(42)).toBe(encode(42));
-  });
-});
-
-describe("decode", () => {
-  it("decodes an encoded string back to the original ID", () => {
-    for (const id of [0, 1, 42, 100, 999, 10000]) {
-      const encoded = encode(id);
-      expect(decode(encoded)).toBe(id);
-    }
-  });
-
-  it("returns null for empty string", () => {
-    expect(decode("")).toBe(null);
-  });
-
-  it("handles round-trip encoding", () => {
-    const original = 12345;
-    const sqid = encode(original);
-    const decoded = decode(sqid);
-    expect(decoded).toBe(original);
-  });
-});
-
-describe("isValidSqid", () => {
-  it("returns true for valid encoded sqids", () => {
-    const sqid = encode(1);
-    expect(isValidSqid(sqid)).toBe(true);
-  });
-
-  it("returns true for various valid sqids", () => {
-    for (const id of [0, 1, 100, 999]) {
-      expect(isValidSqid(encode(id))).toBe(true);
-    }
-  });
-
-  it("returns false for empty string", () => {
-    expect(isValidSqid("")).toBe(false);
-  });
-});

package/src/lib/sqid.ts

@@ -1,79 +0,0 @@
-/**
- * Sqids - Short unique IDs for URLs
- *
- * Encodes numeric IDs to short strings like "jR3k"
- */
-
-import Sqids from "sqids";
-
-const sqids = new Sqids({
-  minLength: 4,
-});
-
-/**
- * Encodes a numeric database ID to a short, URL-friendly string.
- *
- * Uses the Sqids library to generate short unique identifiers with a minimum length of 4 characters.
- * These are used in URLs (e.g., `/p/jR3k`) to obscure sequential integer IDs while maintaining
- * uniqueness and reversibility.
- *
- * @param id - The numeric database ID to encode
- * @returns A short string representation of the ID (minimum 4 characters)
- *
- * @example
- * ```ts
- * const sqid = encode(123);
- * // Returns: "jR3k" (or similar short string)
- * ```
- */
-export function encode(id: number): string {
-  return sqids.encode([id]);
-}
-
-/**
- * Decodes a sqid string back to the original numeric database ID.
- *
- * Attempts to decode a sqid string generated by the `encode()` function. Returns the original
- * numeric ID if valid, or `null` if the string is not a valid sqid. This is used to extract
- * database IDs from URL parameters.
- *
- * @param str - The sqid string to decode
- * @returns The original numeric ID if valid, or `null` if decoding fails
- *
- * @example
- * ```ts
- * const id = decode("jR3k");
- * // Returns: 123
- *
- * const invalid = decode("invalid");
- * // Returns: null
- * ```
- */
-export function decode(str: string): number | null {
-  try {
-    const ids = sqids.decode(str);
-    return ids[0] ?? null;
-  } catch {
-    return null;
-  }
-}
-
-/**
- * Checks if a string is a valid sqid that can be decoded.
- *
- * Validates whether a string can be successfully decoded to a numeric ID.
- * Useful for route validation and input sanitization.
- *
- * @param str - The string to validate
- * @returns `true` if the string is a valid sqid, `false` otherwise
- *
- * @example
- * ```ts
- * if (isValidSqid("jR3k")) {
- *   // Process the valid sqid
- * }
- * ```
- */
-export function isValidSqid(str: string): boolean {
-  return decode(str) !== null;
-}

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

@@ -1,218 +0,0 @@
-import { describe, it, expect } from "vitest";
-import { createTestApp } from "../../../__tests__/helpers/app.js";
-import { pagesApiRoutes } from "../pages.js";
-
-describe("Pages API Routes", () => {
-  describe("GET /api/pages", () => {
-    it("returns empty list when no pages exist", async () => {
-      const { app } = createTestApp();
-      app.route("/api/pages", pagesApiRoutes);
-
-      const res = await app.request("/api/pages");
-      expect(res.status).toBe(200);
-
-      const body = await res.json();
-      expect(body.pages).toEqual([]);
-    });
-
-    it("returns pages list", async () => {
-      const { app, services } = createTestApp();
-      app.route("/api/pages", pagesApiRoutes);
-
-      await services.pages.create({ slug: "about", title: "About" });
-      await services.pages.create({ slug: "contact", title: "Contact" });
-
-      const res = await app.request("/api/pages");
-      const body = await res.json();
-
-      expect(body.pages).toHaveLength(2);
-    });
-  });
-
-  describe("GET /api/pages/:id", () => {
-    it("returns a page by id", async () => {
-      const { app, services } = createTestApp();
-      app.route("/api/pages", pagesApiRoutes);
-
-      const page = await services.pages.create({
-        slug: "about",
-        title: "About Us",
-      });
-
-      const res = await app.request(`/api/pages/${page.id}`);
-      expect(res.status).toBe(200);
-
-      const body = await res.json();
-      expect(body.title).toBe("About Us");
-      expect(body.slug).toBe("about");
-    });
-
-    it("returns 400 for invalid id", async () => {
-      const { app } = createTestApp();
-      app.route("/api/pages", pagesApiRoutes);
-
-      const res = await app.request("/api/pages/abc");
-      expect(res.status).toBe(400);
-    });
-
-    it("returns 404 for non-existent page", async () => {
-      const { app } = createTestApp();
-      app.route("/api/pages", pagesApiRoutes);
-
-      const res = await app.request("/api/pages/9999");
-      expect(res.status).toBe(404);
-    });
-  });
-
-  describe("POST /api/pages", () => {
-    it("returns 401 when not authenticated", async () => {
-      const { app } = createTestApp({ authenticated: false });
-      app.route("/api/pages", pagesApiRoutes);
-
-      const res = await app.request("/api/pages", {
-        method: "POST",
-        headers: { "Content-Type": "application/json" },
-        body: JSON.stringify({ slug: "about", title: "About" }),
-      });
-
-      expect(res.status).toBe(401);
-    });
-
-    it("creates a page when authenticated", async () => {
-      const { app } = createTestApp({ authenticated: true });
-      app.route("/api/pages", pagesApiRoutes);
-
-      const res = await app.request("/api/pages", {
-        method: "POST",
-        headers: { "Content-Type": "application/json" },
-        body: JSON.stringify({
-          slug: "about",
-          title: "About Us",
-          body: "We are Jant.",
-          status: "published",
-        }),
-      });
-
-      expect(res.status).toBe(201);
-      const body = await res.json();
-      expect(body.slug).toBe("about");
-      expect(body.title).toBe("About Us");
-    });
-
-    it("returns 400 for missing slug", async () => {
-      const { app } = createTestApp({ authenticated: true });
-      app.route("/api/pages", pagesApiRoutes);
-
-      const res = await app.request("/api/pages", {
-        method: "POST",
-        headers: { "Content-Type": "application/json" },
-        body: JSON.stringify({ title: "No Slug" }),
-      });
-
-      expect(res.status).toBe(400);
-    });
-  });
-
-  describe("PUT /api/pages/:id", () => {
-    it("returns 401 when not authenticated", async () => {
-      const { app, services } = createTestApp({ authenticated: false });
-      app.route("/api/pages", pagesApiRoutes);
-
-      const page = await services.pages.create({
-        slug: "about",
-        title: "About",
-      });
-
-      const res = await app.request(`/api/pages/${page.id}`, {
-        method: "PUT",
-        headers: { "Content-Type": "application/json" },
-        body: JSON.stringify({ title: "Updated" }),
-      });
-
-      expect(res.status).toBe(401);
-    });
-
-    it("updates a page when authenticated", async () => {
-      const { app, services } = createTestApp({ authenticated: true });
-      app.route("/api/pages", pagesApiRoutes);
-
-      const page = await services.pages.create({
-        slug: "about",
-        title: "About",
-      });
-
-      const res = await app.request(`/api/pages/${page.id}`, {
-        method: "PUT",
-        headers: { "Content-Type": "application/json" },
-        body: JSON.stringify({ title: "Updated About" }),
-      });
-
-      expect(res.status).toBe(200);
-      const body = await res.json();
-      expect(body.title).toBe("Updated About");
-    });
-
-    it("returns 404 for non-existent page", async () => {
-      const { app } = createTestApp({ authenticated: true });
-      app.route("/api/pages", pagesApiRoutes);
-
-      const res = await app.request("/api/pages/9999", {
-        method: "PUT",
-        headers: { "Content-Type": "application/json" },
-        body: JSON.stringify({ title: "test" }),
-      });
-
-      expect(res.status).toBe(404);
-    });
-  });
-
-  describe("DELETE /api/pages/:id", () => {
-    it("returns 401 when not authenticated", async () => {
-      const { app, services } = createTestApp({ authenticated: false });
-      app.route("/api/pages", pagesApiRoutes);
-
-      const page = await services.pages.create({
-        slug: "about",
-        title: "About",
-      });
-
-      const res = await app.request(`/api/pages/${page.id}`, {
-        method: "DELETE",
-      });
-
-      expect(res.status).toBe(401);
-    });
-
-    it("deletes a page when authenticated", async () => {
-      const { app, services } = createTestApp({ authenticated: true });
-      app.route("/api/pages", pagesApiRoutes);
-
-      const page = await services.pages.create({
-        slug: "about",
-        title: "About",
-      });
-
-      const res = await app.request(`/api/pages/${page.id}`, {
-        method: "DELETE",
-      });
-
-      expect(res.status).toBe(200);
-      const body = await res.json();
-      expect(body.success).toBe(true);
-
-      const found = await services.pages.getById(page.id);
-      expect(found).toBeNull();
-    });
-
-    it("returns 404 for non-existent page", async () => {
-      const { app } = createTestApp({ authenticated: true });
-      app.route("/api/pages", pagesApiRoutes);
-
-      const res = await app.request("/api/pages/9999", {
-        method: "DELETE",
-      });
-
-      expect(res.status).toBe(404);
-    });
-  });
-});

package/src/routes/api/pages.ts

@@ -1,73 +0,0 @@
-/**
- * Pages API Routes
- */
-
-import { Hono } from "hono";
-import { z } from "zod";
-import type { Bindings } from "../../types.js";
-import type { AppVariables } from "../../types/app-context.js";
-import { requireAuthApi } from "../../middleware/auth.js";
-import {
-  CreatePageSchema,
-  StatusSchema,
-  parseValidated,
-} from "../../lib/schemas.js";
-import { assertFound, parseIntParam, NotFoundError } from "../../lib/errors.js";
-
-type Env = { Bindings: Bindings; Variables: AppVariables };
-
-export const pagesApiRoutes = new Hono<Env>();
-
-// API update schema extends shared schema with nullable fields for explicit clearing
-const UpdatePageSchema = CreatePageSchema.partial().extend({
-  title: z.string().nullable().optional(),
-  body: z.string().nullable().optional(),
-  status: StatusSchema.optional(),
-});
-
-// List pages
-pagesApiRoutes.get("/", async (c) => {
-  const pages = await c.var.services.pages.list();
-  return c.json({ pages });
-});
-
-// Get single page
-pagesApiRoutes.get("/:id", async (c) => {
-  const id = parseIntParam(c.req.param("id"));
-  const page = assertFound(await c.var.services.pages.getById(id), "Page");
-  return c.json(page);
-});
-
-// Create page (requires auth)
-pagesApiRoutes.post("/", requireAuthApi(), async (c) => {
-  const body = parseValidated(CreatePageSchema, await c.req.json());
-
-  const page = await c.var.services.pages.create({
-    slug: body.slug,
-    title: body.title,
-    body: body.body,
-    status: body.status,
-  });
-
-  return c.json(page, 201);
-});
-
-// Update page (requires auth)
-pagesApiRoutes.put("/:id", requireAuthApi(), async (c) => {
-  const id = parseIntParam(c.req.param("id"));
-  const body = parseValidated(UpdatePageSchema, await c.req.json());
-
-  const page = assertFound(await c.var.services.pages.update(id, body), "Page");
-
-  return c.json(page);
-});
-
-// Delete page (requires auth)
-pagesApiRoutes.delete("/:id", requireAuthApi(), async (c) => {
-  const id = parseIntParam(c.req.param("id"));
-
-  const success = await c.var.services.pages.delete(id);
-  if (!success) throw new NotFoundError("Page");
-
-  return c.json({ success: true });
-});

package/src/routes/dash/__tests__/pages.test.ts

@@ -1,226 +0,0 @@
-/**
- * Tests for the page/nav management logic used by dashboard pages routes.
- *
- * Note: Route handler tests that import JSX components with @lingui/react/macro
- * cannot run in vitest (requires SWC plugin). These tests verify the service
- * layer operations that the routes orchestrate.
- */
-
-import { describe, it, expect, beforeEach } from "vitest";
-import { createTestDatabase } from "../../../__tests__/helpers/db.js";
-import { createPageService } from "../../../services/page.js";
-import { createNavItemService } from "../../../services/navigation.js";
-import { createPathRegistryService } from "../../../services/path-registry.js";
-import type { Database } from "../../../db/index.js";
-
-describe("Dashboard Pages - Nav Management Logic", () => {
-  let db: Database;
-  let pageService: ReturnType<typeof createPageService>;
-  let navItemService: ReturnType<typeof createNavItemService>;
-
-  beforeEach(() => {
-    const testDb = createTestDatabase();
-    db = testDb.db as unknown as Database;
-    pageService = createPageService(db, createPathRegistryService(db));
-    navItemService = createNavItemService(db);
-  });
-
-  describe("add page to nav", () => {
-    it("creates a page-type nav item for the page", async () => {
-      const page = await pageService.create({
-        slug: "about",
-        title: "About Us",
-      });
-
-      // Simulate what the route handler does
-      await navItemService.create({
-        type: "page",
-        label: page.title || page.slug,
-        url: `/${page.slug}`,
-        pageId: page.id,
-      });
-
-      const navItems = await navItemService.list();
-      expect(navItems).toHaveLength(1);
-      expect(navItems[0]?.type).toBe("page");
-      expect(navItems[0]?.label).toBe("About Us");
-      expect(navItems[0]?.url).toBe("/about");
-      expect(navItems[0]?.pageId).toBe(page.id);
-    });
-
-    it("uses slug as label when page has no title", async () => {
-      const page = await pageService.create({ slug: "about" });
-
-      await navItemService.create({
-        type: "page",
-        label: page.title || page.slug,
-        url: `/${page.slug}`,
-        pageId: page.id,
-      });
-
-      const navItems = await navItemService.list();
-      expect(navItems[0]?.label).toBe("about");
-    });
-
-    it("page appears in nav and not in listNotInNav after adding", async () => {
-      const page = await pageService.create({
-        slug: "about",
-        title: "About",
-      });
-
-      // Before adding to nav
-      let notInNav = await pageService.listNotInNav();
-      expect(notInNav).toHaveLength(1);
-
-      // Add to nav
-      await navItemService.create({
-        type: "page",
-        label: page.title || page.slug,
-        url: `/${page.slug}`,
-        pageId: page.id,
-      });
-
-      // After adding to nav
-      notInNav = await pageService.listNotInNav();
-      expect(notInNav).toHaveLength(0);
-
-      const navItems = await navItemService.list();
-      expect(navItems).toHaveLength(1);
-    });
-  });
-
-  describe("remove page from nav", () => {
-    it("removes the nav item but keeps the page", async () => {
-      const page = await pageService.create({
-        slug: "about",
-        title: "About",
-      });
-      await navItemService.create({
-        type: "page",
-        label: "About",
-        url: "/about",
-        pageId: page.id,
-      });
-
-      // Simulate what the route handler does: find and delete nav item by pageId
-      const allNavItems = await navItemService.list();
-      const found = allNavItems.find((item) => item.pageId === page.id);
-      expect(found).toBeDefined();
-      await navItemService.delete(found?.id as number);
-
-      // Nav item should be gone
-      const navItems = await navItemService.list();
-      expect(navItems).toHaveLength(0);
-
-      // Page should still exist
-      const foundPage = await pageService.getById(page.id);
-      expect(foundPage).not.toBeNull();
-
-      // Page should appear in "not in nav" list
-      const notInNav = await pageService.listNotInNav();
-      expect(notInNav).toHaveLength(1);
-      expect(notInNav[0]?.slug).toBe("about");
-    });
-  });
-
-  describe("reorder nav items", () => {
-    it("reorders nav items by position", async () => {
-      const a = await navItemService.create({
-        type: "link",
-        label: "A",
-        url: "/a",
-      });
-      const b = await navItemService.create({
-        type: "link",
-        label: "B",
-        url: "/b",
-      });
-
-      // Reverse order
-      await navItemService.reorder([b.id, a.id]);
-
-      const items = await navItemService.list();
-      expect(items[0]?.label).toBe("B");
-      expect(items[1]?.label).toBe("A");
-    });
-  });
-
-  describe("link CRUD", () => {
-    it("creates a link nav item", async () => {
-      await navItemService.create({
-        type: "link",
-        label: "Blog",
-        url: "/blog",
-      });
-
-      const navItems = await navItemService.list();
-      expect(navItems).toHaveLength(1);
-      expect(navItems[0]?.type).toBe("link");
-      expect(navItems[0]?.label).toBe("Blog");
-      expect(navItems[0]?.url).toBe("/blog");
-    });
-
-    it("updates a link nav item", async () => {
-      const item = await navItemService.create({
-        type: "link",
-        label: "Blog",
-        url: "/blog",
-      });
-
-      await navItemService.update(item.id, {
-        label: "Posts",
-        url: "/posts",
-      });
-
-      const updated = await navItemService.getById(item.id);
-      expect(updated?.label).toBe("Posts");
-      expect(updated?.url).toBe("/posts");
-    });
-
-    it("deletes a link nav item", async () => {
-      const item = await navItemService.create({
-        type: "link",
-        label: "Blog",
-        url: "/blog",
-      });
-
-      await navItemService.delete(item.id);
-
-      const found = await navItemService.getById(item.id);
-      expect(found).toBeNull();
-    });
-  });
-
-  describe("unified page listing", () => {
-    it("separates pages into nav and non-nav groups", async () => {
-      const aboutPage = await pageService.create({
-        slug: "about",
-        title: "About",
-      });
-      await pageService.create({ slug: "contact", title: "Contact" });
-
-      // Add about to nav
-      await navItemService.create({
-        type: "page",
-        label: "About",
-        url: "/about",
-        pageId: aboutPage.id,
-      });
-
-      // Also add a link nav item
-      await navItemService.create({
-        type: "link",
-        label: "External",
-        url: "https://example.com",
-      });
-
-      // Simulate the unified page view data fetch
-      const navItems = await navItemService.list();
-      const otherPages = await pageService.listNotInNav();
-
-      expect(navItems).toHaveLength(2); // page + link
-      expect(otherPages).toHaveLength(1); // only contact
-      expect(otherPages[0]?.slug).toBe("contact");
-    });
-  });
-});