stagent 0.7.0 → 0.8.0

package/src/components/tables/table-spreadsheet.tsx

@@ -14,17 +14,14 @@ import { Button } from "@/components/ui/button";
 import { Plus } from "lucide-react";
 import { toast } from "sonner";
 import { cn } from "@/lib/utils";
-import { CellEditor, CellDisplay } from "./table-cell-editor";
+import { CellDisplay } from "./table-cell-editor";
 import { SpreadsheetColumnHeader } from "./table-column-header";
 import { TableColumnSheet } from "./table-column-sheet";
 import { TableToolbar } from "./table-toolbar";
 import { TableImportWizard } from "./table-import-wizard";
+import { TableRowSheet } from "./table-row-sheet";
 import { EmptyState } from "@/components/shared/empty-state";
 import { Table2 } from "lucide-react";
-import {
-  useSpreadsheetKeys,
-  type CellPosition,
-} from "./use-spreadsheet-keys";
 import { evaluateComputedColumns } from "@/lib/tables/computed";
 import type { ColumnDef, SortSpec } from "@/lib/tables/types";
 import type { UserTableRowRow } from "@/lib/db/schema";
@@ -58,15 +55,12 @@ export function TableSpreadsheet({
 }: TableSpreadsheetProps) {
   const [columns, setColumns] = useState<ColumnDef[]>(initialColumns);
   const [rows, setRows] = useState<ParsedRow[]>(() => parseRows(initialRows));
-  const [editingCell, setEditingCell] = useState<CellPosition | null>(null);
-  const [editValue, setEditValue] = useState<unknown>(null);
   const [selectedRows, setSelectedRows] = useState<Set<string>>(new Set());
   const [sorts, setSorts] = useState<SortSpec[]>([]);
   const [columnSheetOpen, setColumnSheetOpen] = useState(false);
   const [importOpen, setImportOpen] = useState(false);
-  const [pendingSaves, setPendingSaves] = useState<Set<string>>(new Set());
-
-  const debounceTimers = useRef<Map<string, ReturnType<typeof setTimeout>>>(new Map());
+  const [rowSheetOpen, setRowSheetOpen] = useState(false);
+  const [rowSheetRow, setRowSheetRow] = useState<ParsedRow | null>(null);
 
   // ── Refresh helpers ─────────────────────────────────────────────────
 
@@ -87,116 +81,33 @@ export function TableSpreadsheet({
       const res = await fetch(`/api/tables/${tableId}/rows?limit=500${sortsParam}`);
       if (res.ok) {
         const raw = (await res.json()) as UserTableRowRow[];
-        // Evaluate computed columns client-side after refresh
         const enriched = evaluateComputedColumns(columns, raw);
         setRows(parseRows(enriched));
       }
     } catch { /* silent */ }
   }, [tableId, sorts, columns]);
 
-  // ── Cell operations ─────────────────────────────────────────────────
-
-  const getCellValue = useCallback(
-    (pos: CellPosition): unknown => {
-      const row = rows[pos.rowIndex];
-      const col = columns[pos.colIndex];
-      if (!row || !col) return null;
-      return row.data[col.name] ?? null;
-    },
-    [rows, columns]
-  );
+  // ── Boolean toggle (inline for convenience) ─────────────────────────
 
-  const setCellValue = useCallback(
-    (rowId: string, colName: string, value: unknown) => {
+  const handleToggleBoolean = useCallback(
+    (rowId: string, colName: string, newValue: boolean) => {
       setRows((prev) =>
         prev.map((r) =>
           r.id === rowId
-            ? { ...r, data: { ...r.data, [colName]: value } }
+            ? { ...r, data: { ...r.data, [colName]: newValue } }
             : r
         )
       );
-    },
-    []
-  );
-
-  const debounceSave = useCallback(
-    (rowId: string, colName: string, value: unknown) => {
-      const key = `${rowId}:${colName}`;
-
-      // Clear previous timer
-      const existing = debounceTimers.current.get(key);
-      if (existing) clearTimeout(existing);
-
-      setPendingSaves((prev) => new Set(prev).add(key));
-
-      const timer = setTimeout(async () => {
-        try {
-          const res = await fetch(`/api/tables/${tableId}/rows/${rowId}`, {
-            method: "PATCH",
-            headers: { "Content-Type": "application/json" },
-            body: JSON.stringify({ data: { [colName]: value } }),
-          });
-          if (!res.ok) {
-            toast.error("Failed to save cell");
-            // Revert would require storing previous value — for now just notify
-          }
-        } catch {
-          toast.error("Failed to save cell");
-        } finally {
-          setPendingSaves((prev) => {
-            const next = new Set(prev);
-            next.delete(key);
-            return next;
-          });
-          debounceTimers.current.delete(key);
-        }
-      }, 300);
-
-      debounceTimers.current.set(key, timer);
+      // Save immediately
+      fetch(`/api/tables/${tableId}/rows/${rowId}`, {
+        method: "PATCH",
+        headers: { "Content-Type": "application/json" },
+        body: JSON.stringify({ data: { [colName]: newValue } }),
+      }).catch(() => toast.error("Failed to save"));
     },
     [tableId]
   );
 
-  // ── Keyboard hook callbacks ─────────────────────────────────────────
-
-  const handleStartEdit = useCallback(
-    (pos: CellPosition) => {
-      setEditingCell(pos);
-      setEditValue(getCellValue(pos));
-    },
-    [getCellValue]
-  );
-
-  const handleConfirmEdit = useCallback(() => {
-    if (!editingCell) return;
-    const row = rows[editingCell.rowIndex];
-    const col = columns[editingCell.colIndex];
-    if (!row || !col) return;
-
-    setCellValue(row.id, col.name, editValue);
-    debounceSave(row.id, col.name, editValue);
-    setEditingCell(null);
-    setEditValue(null);
-  }, [editingCell, editValue, rows, columns, setCellValue, debounceSave]);
-
-  const handleCancelEdit = useCallback(() => {
-    setEditingCell(null);
-    setEditValue(null);
-  }, []);
-
-  const handleClearCell = useCallback(
-    (pos: CellPosition) => {
-      const row = rows[pos.rowIndex];
-      const col = columns[pos.colIndex];
-      if (!row || !col) return;
-      if (col.dataType === "computed") return;
-
-      setCellValue(row.id, col.name, null);
-      debounceSave(row.id, col.name, null);
-    },
-    [rows, columns, setCellValue, debounceSave]
-  );
-
   // ── Row operations ──────────────────────────────────────────────────
 
   const handleAddRow = useCallback(async () => {
@@ -238,6 +149,19 @@ export function TableSpreadsheet({
     }
   }, [tableId, selectedRows, refreshRows]);
 
+  // ── Row sheet ──────────────────────────────────────────────────────
+
+  const handleOpenRowSheet = useCallback((row: ParsedRow) => {
+    setRowSheetRow(row);
+    setRowSheetOpen(true);
+  }, []);
+
+  const handleRowUpdated = useCallback((rowId: string, data: Record<string, unknown>) => {
+    setRows((prev) =>
+      prev.map((r) => (r.id === rowId ? { ...r, data } : r))
+    );
+  }, []);
+
   // ── Column operations ───────────────────────────────────────────────
 
   const handleSort = useCallback(
@@ -261,18 +185,6 @@ export function TableSpreadsheet({
     refreshRows();
   }
 
-  // ── Keyboard navigation ─────────────────────────────────────────────
-
-  const keyboard = useSpreadsheetKeys({
-    rowCount: rows.length,
-    colCount: columns.length,
-    onStartEdit: handleStartEdit,
-    onConfirmEdit: handleConfirmEdit,
-    onCancelEdit: handleCancelEdit,
-    onClearCell: handleClearCell,
-    onAddRow: handleAddRow,
-  });
-
   // ── Selection helpers ───────────────────────────────────────────────
 
   function toggleRowSelect(rowId: string) {
@@ -328,7 +240,7 @@ export function TableSpreadsheet({
   }
 
   return (
-    <div className="space-y-2" data-spreadsheet>
+    <div className="space-y-2">
       <TableToolbar
         tableId={tableId}
         rowCount={rows.length}
@@ -392,12 +304,14 @@ export function TableSpreadsheet({
                 </TableCell>
               </TableRow>
             ) : (
-              rows.map((row, rowIndex) => (
+              rows.map((row) => (
                 <TableRow
                   key={row.id}
                   className={cn(
+                    "cursor-pointer",
                     selectedRows.has(row.id) && "bg-muted/50"
                   )}
+                  onClick={() => handleOpenRowSheet(row)}
                 >
                   <TableCell onClick={(e) => e.stopPropagation()}>
                     <Checkbox
@@ -405,56 +319,26 @@ export function TableSpreadsheet({
                       onCheckedChange={() => toggleRowSelect(row.id)}
                     />
                   </TableCell>
-                  {columns.map((col, colIndex) => {
-                    const isActive =
-                      keyboard.activeCell?.rowIndex === rowIndex &&
-                      keyboard.activeCell?.colIndex === colIndex;
-                    const isEditing =
-                      isActive &&
-                      editingCell?.rowIndex === rowIndex &&
-                      editingCell?.colIndex === colIndex;
+                  {columns.map((col) => {
                     const value = row.data[col.name];
-                    const saveKey = `${row.id}:${col.name}`;
-                    const isSaving = pendingSaves.has(saveKey);
 
                     return (
                       <TableCell
                         key={col.name}
-                        className={cn(
-                          "p-0 cursor-cell relative min-w-[120px]",
-                          isActive && "ring-2 ring-primary ring-inset",
-                          isSaving && "bg-primary/5"
-                        )}
-                        onClick={() =>
-                          keyboard.handleCellClick({ rowIndex, colIndex })
-                        }
-                        onDoubleClick={() =>
-                          keyboard.handleCellDoubleClick({ rowIndex, colIndex })
-                        }
+                        className="min-w-[120px]"
                       >
                         <div className="px-2 py-1 min-h-[32px] flex items-center">
-                          {isEditing && col.dataType !== "computed" ? (
-                            <CellEditor
-                              column={col}
-                              value={editValue}
-                              onChange={setEditValue}
-                              onConfirm={handleConfirmEdit}
-                              onCancel={handleCancelEdit}
-                            />
-                          ) : (
-                            <CellDisplay
-                              column={col}
-                              value={value}
-                              onToggleBoolean={
-                                col.dataType === "boolean"
-                                  ? (newVal) => {
-                                      setCellValue(row.id, col.name, newVal);
-                                      debounceSave(row.id, col.name, newVal);
-                                    }
-                                  : undefined
-                              }
-                            />
-                          )}
+                          <CellDisplay
+                            column={col}
+                            value={value}
+                            onToggleBoolean={
+                              col.dataType === "boolean"
+                                ? (newVal) => {
+                                    handleToggleBoolean(row.id, col.name, newVal);
+                                  }
+                                : undefined
+                            }
+                          />
                         </div>
                       </TableCell>
                     );
@@ -494,6 +378,17 @@ export function TableSpreadsheet({
         onOpenChange={setImportOpen}
         onImported={() => { refreshTable(); refreshRows(); }}
       />
+
+      {rowSheetRow && (
+        <TableRowSheet
+          tableId={tableId}
+          columns={columns}
+          row={rowSheetRow}
+          open={rowSheetOpen}
+          onOpenChange={setRowSheetOpen}
+          onRowUpdated={handleRowUpdated}
+        />
+      )}
     </div>
   );
 }

package/src/components/workflows/workflow-form-view.tsx

@@ -155,14 +155,14 @@ function normalizeParallelSteps(
 
   const normalizedBranches = branches.map((branch, index) => ({
     ...branch,
-    id: options?.cloneIds ? crypto.randomUUID() : branch.id,
+    id: options?.cloneIds ? crypto.randomUUID() : (branch.id || crypto.randomUUID()),
     name: branch.name || `Research Branch ${index + 1}`,
   }));
 
   const normalizedSynthesis = rawSynthesis
     ? {
         ...rawSynthesis,
-        id: options?.cloneIds ? crypto.randomUUID() : rawSynthesis.id,
+        id: options?.cloneIds ? crypto.randomUUID() : (rawSynthesis.id || crypto.randomUUID()),
         name: rawSynthesis.name || "Synthesize findings",
       }
     : undefined;
@@ -257,7 +257,7 @@ function normalizeSwarmSteps(
     id:
       options?.cloneIds && mayorStep
         ? crypto.randomUUID()
-        : (mayorStep?.id ?? crypto.randomUUID()),
+        : (mayorStep?.id || crypto.randomUUID()),
     name: mayorStep?.name || "Mayor plan",
   };
 
@@ -268,7 +268,7 @@ function normalizeSwarmSteps(
 
   const normalizedWorkers = nextWorkers.map((worker, index) => ({
     ...worker,
-    id: options?.cloneIds ? crypto.randomUUID() : worker.id,
+    id: options?.cloneIds ? crypto.randomUUID() : (worker.id || crypto.randomUUID()),
     name: worker.name || `Worker ${index + 1}`,
   }));
 
@@ -277,7 +277,7 @@ function normalizeSwarmSteps(
     id:
       options?.cloneIds && refineryStep
         ? crypto.randomUUID()
-        : (refineryStep?.id ?? crypto.randomUUID()),
+        : (refineryStep?.id || crypto.randomUUID()),
     name: refineryStep?.name || "Refine and merge",
   };
 
@@ -472,7 +472,7 @@ export function WorkflowFormView({
               ? normalizeParallelSteps(def.steps)
               : def.pattern === "swarm"
                 ? normalizeSwarmSteps(def.steps)
-              : def.steps
+              : def.steps.map((s) => ({ ...s, id: s.id || crypto.randomUUID() }))
         );
       }
     }

package/src/lib/agents/profiles/builtins/document-writer/SKILL.md

@@ -14,3 +14,26 @@ You are a technical writer producing clear, well-structured documents.
 - Include a table of contents for documents with 3+ sections
 - Highlight action items or decisions needed in bold
 - If writing from a template, preserve the template's style and structure
+
+## Book Chapter Conventions
+
+When generating AI Native book chapters:
+
+- Preserve existing `> [!authors-note]` blocks unchanged during regeneration
+- Preserve existing `> [!case-study]` blocks — update content only if source material changed
+- Include "Building with Stagent" TypeScript code examples with realistic values
+- Include both "Stagent Today" and "Roadmap Vision" sections
+- Use `> [!case-study]` callout format: name the company, describe their pattern, draw parallel to Stagent
+- Follow the Problem → Solution → Implementation → Lessons narrative arc
+- Target the reading time specified in chapter frontmatter (~250 words/min)
+
+## Originality and Attribution Rules
+
+When writing chapters that reference external case studies (ai-native-notes/ articles):
+
+- **Never copy phrases verbatim** from source articles without quotation marks and explicit attribution
+- **Always credit authors by name** in case-study callouts (e.g., "Geoffrey Huntley" not just "Ralph Wiggum", "Dorsey and Botha" not just "Sequoia")
+- **When structuring content around an external framework** (e.g., 8090's five stations, Block's four pillars), explicitly acknowledge the source: "As [Author] describes in [Work]..." before elaborating
+- **Synthesize from multiple sources** rather than mirroring a single article's structure. If one source dominates a section, bring in at least one additional perspective
+- **Make it Stagent's own**: Every external concept should connect to Stagent's concrete implementation or roadmap. Don't just restate what Stripe/Ramp/Harvey built — explain what Stagent builds differently and why
+- **Use direct quotes sparingly** and only for memorable, well-attributed phrases. The majority of prose should be original analysis

package/src/lib/agents/profiles/builtins/technical-writer/SKILL.md

@@ -29,3 +29,13 @@ Structure documentation according to the type requested:
 - **ADR**: Title, status (proposed/accepted/deprecated/superseded), context, decision, consequences
 - **README**: Title, description, prerequisites, installation, usage, configuration, contributing, license
 - **Changelog**: Version header, date, categorized entries with PR/issue references where available
+- **Book Chapter Review**: Quality pass on AI Native book chapters — terminology consistency, API accuracy, case study attribution, code example quality, section presence, grammar/style. Edit in-place; do not rewrite.
+
+## Originality Checks (Book Chapter Review)
+
+During book chapter quality reviews, specifically check for:
+
+- **Verbatim copying**: Flag any passage >15 words matching source articles in `ai-native-notes/` that lacks quotation marks
+- **Author attribution**: Every case-study callout must name the author (e.g., "Geoffrey Huntley" not "Ralph Wiggum", "Dorsey and Botha" not "Sequoia")
+- **Structural mirroring**: If a chapter follows a source article's exact progression without acknowledgment, flag it and suggest an explicit framing line ("As [Author] traces in [Work]...")
+- **Single-source dependency**: If >50% of a section's ideas come from one source, flag it and suggest multi-source synthesis

package/src/lib/agents/profiles/builtins/technical-writer/profile.yaml

@@ -2,7 +2,7 @@ id: technical-writer
 name: Technical Writer
 version: "1.0.0"
 domain: work
-tags: [documentation, api-docs, adr, readme, changelog, technical-writing]
+tags: [documentation, api-docs, adr, readme, changelog, technical-writing, book-chapter-review]
 supportedRuntimes: [claude-code, openai-codex-app-server, anthropic-direct, openai-direct, ollama]
 preferredRuntime: anthropic-direct
 allowedTools:

package/src/lib/book/chapter-generator.ts

@@ -18,6 +18,7 @@ interface ChapterContext {
   relatedJourney: string | undefined;
   currentMarkdown: string | null;
   sourceContents: string[];
+  caseStudyContents: string[];
   strategy: string | null;
 }
 
@@ -50,6 +51,17 @@ export function gatherChapterContext(chapterId: string): ChapterContext {
     }
   }
 
+  // Read case study articles from ai-native-notes/
+  const caseStudySlugs = mapping?.caseStudies ?? [];
+  const caseStudyContents: string[] = [];
+  for (const csSlug of caseStudySlugs) {
+    const csPath = join(appRoot, "ai-native-notes", `${csSlug}.md`);
+    if (existsSync(csPath)) {
+      const content = readFileSync(csPath, "utf-8");
+      caseStudyContents.push(`### Case Study: ${csSlug}\n${content}`);
+    }
+  }
+
   // Read the book strategy document
   const strategyPath = join(appRoot, "ai-native-notes", "ai-native-book-strategy.md");
   const strategy = existsSync(strategyPath)
@@ -69,6 +81,7 @@ export function gatherChapterContext(chapterId: string): ChapterContext {
     relatedJourney: chapter.relatedJourney,
     currentMarkdown,
     sourceContents,
+    caseStudyContents,
     strategy,
   };
 }
@@ -96,10 +109,14 @@ export function buildChapterRegenerationPrompt(chapterId: string): string {
     `${isNew ? "Write" : "Regenerate"} this book chapter following these rules:`,
     "1. **Narrative voice**: Write in first-person plural ('we') with a technical-but-approachable tone",
     "2. **Structure**: Follow the Problem → Solution → Implementation → Lessons pattern",
-    "3. **Code examples**: Include real code snippets from the Stagent codebase with filename comments",
-    "4. **Markdown format**: Use the conventions below for content blocks",
-    "5. **Reading time**: Target approximately " + ctx.readingTime + " minutes",
-    ...(isNew ? [] : ["6. **Preserve Author's Notes**: Keep any existing `> [!authors-note]` blocks unchanged"]),
+    "3. **Code examples**: Include 'Building with Stagent' TypeScript API examples showing how developers use the feature (realistic UUIDs, timestamps, async/await, comments on non-obvious lines)",
+    "4. **Case studies**: Weave 2-4 case study references as `> [!case-study]` callout blocks. Name the company, describe their pattern, draw a parallel to Stagent.",
+    "5. **Stagent sections**: Each chapter must include a 'Stagent Today' section (current implementation) and a 'Roadmap Vision' section (future direction informed by case studies).",
+    "6. **Narrative thread**: Reference the 'machine that builds machines' thesis — how Stagent uses its own capabilities to build/maintain itself.",
+    "7. **Originality**: Never copy phrases verbatim from case study material without quotation marks. Always credit authors by name in callouts. Synthesize from multiple sources rather than mirroring one article's structure. Make content Stagent's own — connect every external concept to Stagent's implementation.",
+    "8. **Markdown format**: Use the conventions below for content blocks",
+    "9. **Reading time**: Target approximately " + ctx.readingTime + " minutes",
+    ...(isNew ? [] : ["10. **Preserve Author's Notes**: Keep any existing `> [!authors-note]` blocks unchanged"]),
     "",
     "## Tools Available",
     "",
@@ -111,7 +128,7 @@ export function buildChapterRegenerationPrompt(chapterId: string): string {
     "",
     "- Sections: `## Section Title`",
     "- Code blocks: ` ```language ` with `<!-- filename: path -->` before the block",
-    "- Callouts: `> [!tip]`, `> [!warning]`, `> [!info]`, `> [!lesson]`, `> [!authors-note]`",
+    "- Callouts: `> [!tip]`, `> [!warning]`, `> [!info]`, `> [!lesson]`, `> [!authors-note]`, `> [!case-study]`",
     "- Interactive links: `[Try: label](href)`",
     "- Images: `![alt](src \"caption\")`",
     "",
@@ -141,6 +158,19 @@ export function buildChapterRegenerationPrompt(chapterId: string): string {
     );
   }
 
+  if (ctx.caseStudyContents.length > 0) {
+    sections.push(
+      "## Case Study Material",
+      "",
+      "Weave these case studies into the narrative as `> [!case-study]` callout blocks.",
+      "Each callout should: name the company, describe their specific pattern, and draw a parallel to the Stagent feature being discussed.",
+      "Do NOT create a separate 'Case Studies' section — integrate them inline where relevant.",
+      "",
+      ...ctx.caseStudyContents.map(c => c.slice(0, 3000)), // Truncate each to manage context
+      ""
+    );
+  }
+
   if (ctx.strategy) {
     sections.push(
       "## Book Strategy Reference",
@@ -179,3 +209,49 @@ export function buildChapterRegenerationPrompt(chapterId: string): string {
   return sections.join("\n");
 }
 
+/**
+ * Build an agent prompt for the technical-writer profile to quality-review
+ * a generated book chapter. This is a post-generation pass for consistency,
+ * accuracy, and style.
+ */
+export function buildChapterQualityPrompt(chapterId: string): string {
+  const ctx = gatherChapterContext(chapterId);
+  if (!ctx.currentMarkdown) {
+    throw new Error(`No chapter content found for ${chapterId} — generate it first`);
+  }
+
+  const sections: string[] = [
+    `# Chapter Quality Review: ${ctx.title}`,
+    "",
+    `Review Chapter ${ctx.chapterNumber}: "${ctx.title}" — ${ctx.subtitle}`,
+    "",
+    "## Review Checklist",
+    "",
+    "1. **Terminology consistency** — verify consistent use of terms across the chapter (e.g., 'agent profile' not 'agent persona', 'workflow' not 'pipeline' when referring to Stagent workflows)",
+    "2. **API accuracy** — read referenced source files to confirm code examples use correct API paths, parameter names, and response shapes",
+    "3. **Case study attribution** — every `> [!case-study]` callout must name the company, author (if known), and approximate date",
+    "4. **Code example quality** — 'Building with Stagent' examples should use realistic values (UUIDs, timestamps), async/await, and brief comments on non-obvious lines",
+    "5. **Section presence** — verify the chapter contains 'Stagent Today' and 'Roadmap Vision' sections",
+    "6. **Grammar and style** — first-person plural voice ('we'), technical but approachable, no marketing fluff",
+    "7. **Reading time** — content should match the target of ~" + ctx.readingTime + " minutes (~250 words/min)",
+    "",
+    "## Tools Available",
+    "",
+    "You have access to **Read**, **Grep**, **Glob**, **Write**, and **Edit** tools.",
+    "- Use **Read** and **Grep** to verify API paths and source code references",
+    "- Use **Edit** to make corrections directly in the chapter file",
+    "",
+    "## Chapter Content",
+    "",
+    "```markdown",
+    ctx.currentMarkdown,
+    "```",
+    "",
+    "## Output",
+    "",
+    `Edit the chapter file at: \`book/chapters/${ctx.slug}.md\``,
+    "Make corrections in-place. Do not rewrite the entire chapter — only fix issues found in the review checklist.",
+  ];
+
+  return sections.join("\n");
+}