@gmickel/gno 2.5.1 → 2.7.0

package/src/serve/public/hooks/use-api.ts

@@ -1,5 +1,7 @@
 import { useCallback, useState } from "react";
 
+import { writeOutcomeUnknown } from "../lib/request-intent";
+
 interface ApiState<T> {
   data: T | null;
   loading: boolean;
@@ -81,7 +83,17 @@ export function useApi<T>() {
 export async function apiFetch<T>(
   endpoint: string,
   options?: RequestInit
-): Promise<{ data: T | null; error: string | null }> {
+): Promise<{
+  data: T | null;
+  error: string | null;
+  /** `error.details` of the JSON error envelope, when the server sent one. */
+  details?: Record<string, unknown>;
+  /**
+   * A write may or may not have committed: no readable response, a server
+   * error, or the request is still pending under its request ID.
+   */
+  outcomeUnknown?: true;
+}> {
   try {
     const res = await fetch(endpoint, {
       headers: { "Content-Type": "application/json" },
@@ -91,14 +103,24 @@ export async function apiFetch<T>(
     const { json, parseError } = await parseJsonSafe(res);
 
     if (parseError) {
-      return { data: null, error: parseError };
+      return { data: null, error: parseError, outcomeUnknown: true };
     }
 
     if (!res.ok) {
-      const apiError = json as { error?: { message?: string } };
+      const apiError = json as {
+        error?: {
+          code?: string;
+          message?: string;
+          details?: Record<string, unknown>;
+        };
+      };
       return {
         data: null,
         error: apiError.error?.message || `Request failed: ${res.status}`,
+        details: apiError.error?.details,
+        ...(writeOutcomeUnknown(res.status, apiError.error?.code)
+          ? { outcomeUnknown: true as const }
+          : {}),
       };
     }
 
@@ -107,6 +129,7 @@ export async function apiFetch<T>(
     return {
       data: null,
       error: err instanceof Error ? err.message : "Network error",
+      outcomeUnknown: true,
     };
   }
 }

package/src/serve/public/lib/request-intent.ts

@@ -0,0 +1,77 @@
+/**
+ * One request ID per submitted write intent.
+ *
+ * Retrying the same payload (after a lost response, an error, or a page
+ * refresh) reuses the ID, so the server replays the committed outcome instead
+ * of writing twice. A changed payload is a new intent and gets a new ID. The
+ * unresolved intent is kept in sessionStorage so it survives a refresh of
+ * this tab; it is cleared once the write is confirmed.
+ */
+export interface RequestIntent {
+  key: string;
+  requestId: string;
+}
+
+/** Compact payload fingerprint (cyrb53); a collision only yields a conflict error. */
+function fingerprint(text: string): string {
+  let h1 = 0xde_ad_be_ef;
+  let h2 = 0x41_c6_ce_57;
+  for (let index = 0; index < text.length; index += 1) {
+    const code = text.charCodeAt(index);
+    h1 = Math.imul(h1 ^ code, 2_654_435_761);
+    h2 = Math.imul(h2 ^ code, 1_597_334_677);
+  }
+  h1 = Math.imul(h1 ^ (h1 >>> 16), 2_246_822_507);
+  h1 ^= Math.imul(h2 ^ (h2 >>> 13), 3_266_489_909);
+  h2 = Math.imul(h2 ^ (h2 >>> 16), 2_246_822_507);
+  h2 ^= Math.imul(h1 ^ (h1 >>> 13), 3_266_489_909);
+  return (4_294_967_296 * (2_097_151 & h2) + (h1 >>> 0)).toString(36);
+}
+
+function readStored(storageKey: string): RequestIntent | null {
+  try {
+    const raw = sessionStorage.getItem(storageKey);
+    return raw ? (JSON.parse(raw) as RequestIntent) : null;
+  } catch {
+    return null;
+  }
+}
+
+export function requestIdForIntent(
+  slot: { current: RequestIntent | null },
+  payload: string,
+  storageKey: string
+): string {
+  const key = fingerprint(payload);
+  slot.current ??= readStored(storageKey);
+  if (slot.current?.key !== key) {
+    slot.current = { key, requestId: crypto.randomUUID() };
+    try {
+      sessionStorage.setItem(storageKey, JSON.stringify(slot.current));
+    } catch {
+      // Storage unavailable: the ID still covers retries in this page.
+    }
+  }
+  return slot.current.requestId;
+}
+
+/** The intent's write is confirmed: the next submit is a new intent. */
+export function clearRequestIntent(
+  slot: { current: RequestIntent | null },
+  storageKey: string
+): void {
+  slot.current = null;
+  try {
+    sessionStorage.removeItem(storageKey);
+  } catch {
+    // Nothing persisted to clear.
+  }
+}
+
+/**
+ * Whether a failed write's HTTP answer leaves its commit unknown: a server
+ * error, or the request ID's earlier attempt is still in progress.
+ */
+export function writeOutcomeUnknown(status: number, code?: string): boolean {
+  return status >= 500 || code === "REQUEST_PENDING";
+}

package/src/serve/public/lib/snippet.tsx

@@ -0,0 +1,52 @@
+import type { ReactNode } from "react";
+
+const MARK_OPEN = "<mark>";
+const MARK_CLOSE = "</mark>";
+
+/** CommonMark backslash escape: `\` before any ASCII punctuation character. */
+const MARKDOWN_ESCAPE = /\\([!"#$%&'()*+,\-./:;<=>?@[\\\]^_`{|}~])/g;
+
+/** Drop Markdown backslash escapes (`state.db\#` -> `state.db#`). */
+export function unescapeMarkdown(text: string): string {
+  return text.replace(MARKDOWN_ESCAPE, "$1");
+}
+
+/**
+ * Render a search snippet: FTS `<mark>` highlights become real <mark>
+ * elements, everything else stays React text (never parsed as HTML).
+ * Markdown backslash escapes are dropped because indexed text keeps them.
+ */
+export function renderSnippet(snippet: string): ReactNode[] {
+  const parts: ReactNode[] = [];
+  let remaining = snippet;
+  let key = 0;
+
+  while (remaining.length > 0) {
+    const markStart = remaining.indexOf(MARK_OPEN);
+    if (markStart === -1) {
+      parts.push(unescapeMarkdown(remaining));
+      break;
+    }
+    if (markStart > 0) {
+      parts.push(unescapeMarkdown(remaining.slice(0, markStart)));
+    }
+    const markEnd = remaining.indexOf(MARK_CLOSE, markStart);
+    if (markEnd === -1) {
+      parts.push(unescapeMarkdown(remaining.slice(markStart)));
+      break;
+    }
+    parts.push(
+      <mark
+        className="rounded bg-primary/20 px-0.5 font-medium text-primary"
+        key={key++}
+      >
+        {unescapeMarkdown(
+          remaining.slice(markStart + MARK_OPEN.length, markEnd)
+        )}
+      </mark>
+    );
+    remaining = remaining.slice(markEnd + MARK_CLOSE.length);
+  }
+
+  return parts;
+}

package/src/serve/public/lib/workspace-actions.ts

@@ -14,7 +14,8 @@ export type WorkspaceActionId =
   | "go-ask"
   | "go-graph"
   | "go-collections"
-  | "go-connectors";
+  | "go-connectors"
+  | "go-sessions";
 
 export interface WorkspaceAction {
   id: WorkspaceActionId;
@@ -155,6 +156,13 @@ export function getWorkspaceActions(
       keywords: ["connectors", "mcp", "skills", "agents"],
       available: true,
     },
+    {
+      id: "go-sessions",
+      group: "Go To",
+      label: "Agent sessions",
+      keywords: ["sessions", "archive", "transcripts", "import", "agents"],
+      available: true,
+    },
   ];
 }
 
@@ -220,6 +228,9 @@ export function runWorkspaceAction(
     case "go-connectors":
       handlers.navigate("/connectors");
       break;
+    case "go-sessions":
+      handlers.navigate("/sessions");
+      break;
   }
 
   handlers.closePalette();

package/src/serve/public/lib/workspace-tabs.ts

@@ -69,6 +69,8 @@ function getLocationLabel(location: string): string {
       return "Compiled context";
     case "/traces":
       return "Trace history";
+    case "/sessions":
+      return "Agent sessions";
     case "/doc":
     case "/edit": {
       const uri = params.get("uri") ?? "";

package/src/serve/public/pages/Dashboard.tsx

@@ -1,4 +1,5 @@
 import {
+  ArchiveIcon,
   BookOpen,
   CheckCircle2Icon,
   CpuIcon,
@@ -540,6 +541,15 @@ export default function Dashboard({ navigate }: PageProps) {
             <HistoryIcon className="size-4" />
             Trace history
           </Button>
+          <Button
+            className="gap-2"
+            onClick={() => navigate("/sessions")}
+            size="lg"
+            variant="outline"
+          >
+            <ArchiveIcon className="size-4" />
+            Agent sessions
+          </Button>
           <Button
             onClick={() => navigate("/context/compiled")}
             size="lg"
@@ -778,25 +788,28 @@ export default function Dashboard({ navigate }: PageProps) {
                   }
                   style={{ animationDelay: `${0.4 + index * 0.1}s` }}
                 >
-                  <CardContent className="flex items-center justify-between py-4">
-                    <div className="flex items-center gap-3">
+                  <CardContent className="flex flex-wrap items-center justify-between gap-3 py-4">
+                    <div className="flex min-w-0 flex-1 basis-48 items-center gap-3">
                       {syncing ? (
-                        <Loader2Icon className="size-4 animate-spin text-amber-500" />
+                        <Loader2Icon className="size-4 shrink-0 animate-spin text-amber-500" />
                       ) : collection.embeddedCount >= collection.chunkCount ? (
-                        <CheckCircle2Icon className="size-4 text-green-500" />
+                        <CheckCircle2Icon className="size-4 shrink-0 text-green-500" />
                       ) : (
-                        <div className="size-4 rounded-full border-2 border-amber-500" />
+                        <div className="size-4 shrink-0 rounded-full border-2 border-amber-500" />
                       )}
-                      <div>
-                        <div className="font-medium text-lg transition-colors group-hover:text-primary">
+                      <div className="min-w-0">
+                        <div className="truncate font-medium text-lg transition-colors group-hover:text-primary">
                           {collection.name}
                         </div>
-                        <div className="font-mono text-muted-foreground text-sm">
+                        <div
+                          className="truncate font-mono text-muted-foreground text-sm"
+                          title={collection.path}
+                        >
                           {collection.path}
                         </div>
                       </div>
                     </div>
-                    <div className="flex items-center gap-3 text-right">
+                    <div className="flex shrink-0 items-center gap-3 text-right">
                       <Button
                         onClick={(event) => {
                           event.stopPropagation();

package/src/serve/public/pages/DocView.tsx

@@ -37,6 +37,7 @@ import {
   type FileRefactorPreviewPlan,
 } from "../../../core/file-refactor-contract";
 import { extractSections } from "../../../core/sections";
+import { updateFrontmatterTags } from "../../../ingestion/frontmatter";
 import {
   CodeBlock,
   CodeBlockCopyButton,
@@ -83,6 +84,13 @@ import {
   isPdfDocument,
 } from "../lib/doc-asset-url";
 import { waitForDocumentAvailability } from "../lib/document-availability";
+import {
+  clearRequestIntent,
+  type RequestIntent,
+  requestIdForIntent,
+} from "../lib/request-intent";
+
+const TAG_INTENT_KEY = "gno:tag-intent";
 import {
   buildReadableSectionUrl,
   createCitationSectionUrl,
@@ -576,11 +584,10 @@ export default function DocView({ navigate }: PageProps) {
     return buildDocAssetUrl(doc.uri, doc.relPath);
   }, [doc, isPdf]);
 
-  // Remote "Open original" target for any document that has a source file:
-  // /api/doc-asset serves any collection file inline, so this keeps the
-  // previous file:// scope (every read-only source) for remote clients.
+  // Remote "Open original" target: /api/doc-asset serves any collection file
+  // inline by URI, and remote clients never receive the host absPath.
   const sourceAssetUrl = useMemo(() => {
-    if (!doc?.source.absPath) {
+    if (!doc) {
       return null;
     }
     return buildDocAssetUrl(doc.uri, doc.relPath);
@@ -1201,7 +1208,8 @@ export default function DocView({ navigate }: PageProps) {
     setTagSaveError(null);
   }, []);
 
-  // Save tags
+  // Save tags (one request ID per tag-set intent, reused on retry)
+  const tagIntentRef = useRef<RequestIntent | null>(null);
   const handleSaveTags = useCallback(async () => {
     if (!doc) return;
 
@@ -1218,6 +1226,11 @@ export default function DocView({ navigate }: PageProps) {
           expectedSourceHash: doc.source.sourceHash,
           expectedModifiedAt: doc.source.modifiedAt,
           uri: doc.uri,
+          requestId: requestIdForIntent(
+            tagIntentRef,
+            `${doc.uri}\u0000${doc.source.sourceHash}\u0000${editedTags.join(",")}`,
+            TAG_INTENT_KEY
+          ),
         }),
       }
     );
@@ -1229,10 +1242,16 @@ export default function DocView({ navigate }: PageProps) {
       return;
     }
 
-    // Update doc with new tags
+    clearRequestIntent(tagIntentRef, TAG_INTENT_KEY);
+    // Mirror the committed write-back so the frontmatter tag list is current
+    // without waiting for the index to resync.
     setDoc({
       ...doc,
       tags: editedTags,
+      content:
+        data?.writeBack === "applied" && doc.content !== null
+          ? updateFrontmatterTags(doc.content, editedTags)
+          : doc.content,
       source: {
         ...doc.source,
         sourceHash: data?.version.sourceHash ?? doc.source.sourceHash,