@cfbender/cesium 0.6.1 → 0.7.0

package/src/server/api.ts

@@ -4,13 +4,22 @@
 // Routes:
 //   POST /api/sessions/:projectSlug/:filename/answers/:questionId
 //   GET  /api/sessions/:projectSlug/:filename/state
+//   POST /api/sessions/:projectSlug/:filename/comments
+//   DELETE /api/sessions/:projectSlug/:filename/comments/:commentId
+//   POST /api/sessions/:projectSlug/:filename/verdict
 //
 // Any other /api/* path returns a JSON 404 (rather than falling through to the
 // static file handler, which would return the HTML 404 page).
 
 import { join, resolve, relative } from "node:path";
 import { Hono } from "hono";
-import { submitAnswer, getState } from "../storage/mutate.ts";
+import {
+  submitAnswer,
+  getState,
+  addComment,
+  removeComment,
+  setVerdict,
+} from "../storage/mutate.ts";
 import type { AnswerValue } from "../render/validate.ts";
 
 export interface ApiHandlerOptions {
@@ -136,16 +145,166 @@ export function createApiApp(options: ApiHandlerOptions): Hono {
       return c.json({ ok: false, reason: outcome.reason }, 404);
     }
 
+    if (outcome.kind === "ask") {
+      return c.json(
+        {
+          kind: "ask",
+          status: outcome.status,
+          answers: outcome.answers,
+          remaining: outcome.remaining,
+        },
+        200,
+      );
+    }
+
+    // kind === "annotate"
     return c.json(
       {
+        kind: "annotate",
         status: outcome.status,
-        answers: outcome.answers,
-        remaining: outcome.remaining,
+        comments: outcome.comments,
+        verdict: outcome.verdict,
+        verdictMode: outcome.verdictMode,
       },
       200,
     );
   });
 
+  // POST /api/sessions/:projectSlug/:filename/comments
+  app.post("/api/sessions/:projectSlug/:filename/comments", async (c) => {
+    const { projectSlug, filename } = c.req.param();
+
+    const resolved = resolveArtifact(stateDir, projectSlug, filename);
+    if (resolved instanceof Response) return resolved;
+
+    let body: unknown;
+    try {
+      body = await c.req.json();
+    } catch {
+      return c.json({ ok: false, error: "invalid JSON body" }, 400);
+    }
+
+    if (body === null || typeof body !== "object" || Array.isArray(body)) {
+      return c.json({ ok: false, error: "body must be an object" }, 400);
+    }
+
+    const raw = body as Record<string, unknown>;
+    if (
+      typeof raw["anchor"] !== "string" ||
+      typeof raw["selectedText"] !== "string" ||
+      typeof raw["comment"] !== "string"
+    ) {
+      return c.json(
+        { ok: false, error: 'body must contain "anchor", "selectedText", and "comment" fields' },
+        400,
+      );
+    }
+
+    const outcome = await addComment({
+      artifactPath: resolved.artifactPath,
+      anchor: raw["anchor"],
+      selectedText: raw["selectedText"],
+      comment: raw["comment"],
+    });
+
+    if (outcome.ok) {
+      return c.json({ ok: true, comment: outcome.comment }, 200);
+    }
+
+    switch (outcome.reason) {
+      case "not-found":
+      case "not-interactive":
+        return c.json({ ok: false, reason: outcome.reason }, 404);
+      case "session-ended":
+        return c.json({ ok: false, status: outcome.status }, 410);
+      case "expired":
+        return c.json({ ok: false, status: "expired" }, 410);
+      case "invalid-value":
+        return c.json({ ok: false, message: outcome.message }, 422);
+    }
+
+    return c.json({ ok: false, error: "internal error" }, 500);
+  });
+
+  // DELETE /api/sessions/:projectSlug/:filename/comments/:commentId
+  app.delete("/api/sessions/:projectSlug/:filename/comments/:commentId", async (c) => {
+    const { projectSlug, filename, commentId } = c.req.param();
+
+    const resolved = resolveArtifact(stateDir, projectSlug, filename);
+    if (resolved instanceof Response) return resolved;
+
+    const outcome = await removeComment({
+      artifactPath: resolved.artifactPath,
+      commentId,
+    });
+
+    if (outcome.ok) {
+      return c.json({ ok: true }, 200);
+    }
+
+    switch (outcome.reason) {
+      case "not-found":
+      case "not-interactive":
+        return c.json({ ok: false, reason: outcome.reason }, 404);
+      case "comment-not-found":
+        return c.json({ ok: false, reason: "comment-not-found" }, 404);
+      case "session-ended":
+        return c.json({ ok: false, status: outcome.status }, 410);
+      case "expired":
+        return c.json({ ok: false, status: "expired" }, 410);
+    }
+
+    return c.json({ ok: false, error: "internal error" }, 500);
+  });
+
+  // POST /api/sessions/:projectSlug/:filename/verdict
+  app.post("/api/sessions/:projectSlug/:filename/verdict", async (c) => {
+    const { projectSlug, filename } = c.req.param();
+
+    const resolved = resolveArtifact(stateDir, projectSlug, filename);
+    if (resolved instanceof Response) return resolved;
+
+    let body: unknown;
+    try {
+      body = await c.req.json();
+    } catch {
+      return c.json({ ok: false, error: "invalid JSON body" }, 400);
+    }
+
+    if (body === null || typeof body !== "object" || Array.isArray(body)) {
+      return c.json({ ok: false, error: "body must be an object" }, 400);
+    }
+
+    const raw = body as Record<string, unknown>;
+    if (typeof raw["verdict"] !== "string") {
+      return c.json({ ok: false, error: 'body must contain a string "verdict" field' }, 400);
+    }
+
+    // Pass verdict as-is to setVerdict; mode-checking happens internally
+    const outcome = await setVerdict({
+      artifactPath: resolved.artifactPath,
+      verdict: raw["verdict"] as import("../render/validate.ts").Verdict,
+    });
+
+    if (outcome.ok) {
+      return c.json({ ok: true, status: outcome.status, verdict: outcome.verdict }, 200);
+    }
+
+    switch (outcome.reason) {
+      case "not-found":
+      case "not-interactive":
+        return c.json({ ok: false, reason: outcome.reason }, 404);
+      case "session-ended":
+        return c.json({ ok: false, status: outcome.status }, 410);
+      case "expired":
+        return c.json({ ok: false, status: "expired" }, 410);
+      case "invalid-value":
+        return c.json({ ok: false, message: outcome.message }, 422);
+    }
+
+    return c.json({ ok: false, error: "internal error" }, 500);
+  });
+
   // Catch-all under /api/* — keeps unmatched API paths as JSON 404 instead of
   // falling through to the static file handler.
   app.all("/api/*", (c) => c.json({ ok: false, error: "not found" }, 404));

package/src/storage/index-gen.ts

@@ -94,6 +94,8 @@ function indexCss(): string {
   /* the eyebrow text is uppercased + tracked; the emblem sits flush left */
 }
 .cesium-eyebrow svg { display: block; }
+a.cesium-eyebrow { color: inherit; text-decoration: none; transition: opacity 0.15s; }
+a.cesium-eyebrow:hover { opacity: 0.7; }
 .filter-row { display: flex; flex-wrap: wrap; gap: 8px; margin-bottom: 24px; align-items: center; }
 .filter-chip {
   display: inline-block; font-family: var(--sans); font-size: 0.8em; font-weight: 500;
@@ -365,7 +367,7 @@ ${cardsHtml}
 </head>
 <body>
 <div class="page">
-  <p class="eyebrow cesium-eyebrow">${faviconEmblemSvg(18)}<span>cesium · project</span></p>
+  <a class="eyebrow cesium-eyebrow" href="../../index.html">${faviconEmblemSvg(18)}<span>cesium · project</span></a>
   <h1 class="h-display">${esc(projectName)}</h1>
   ${subhead}
   ${filterRow}
@@ -454,7 +456,7 @@ export function renderGlobalIndex(args: RenderGlobalIndexArgs): string {
 </head>
 <body>
 <div class="page">
-  <p class="eyebrow cesium-eyebrow">${faviconEmblemSvg(18)}<span>cesium</span></p>
+  <a class="eyebrow cesium-eyebrow" href="index.html">${faviconEmblemSvg(18)}<span>cesium</span></a>
   <h1 class="h-display">All projects</h1>
   ${subhead}
   ${bodyContent}