pi-weave 0.3.12 → 0.3.14

package/src/web/client/shell/shell.model.ts

@@ -21,9 +21,10 @@ import { kindIcon } from "../tree/tree.model";
 import type { IconName } from "./icons.model";
 export const NOTE_DRAG_TYPE = "application/x-weave-note";
 
-/** Pane drops open only note IDs that still exist in this workspace. */
+/** Pane drops open only note and HTML artifact IDs that still exist in this workspace. */
 export function noteDropId(graph: GraphPayload | null, id: string): string | null {
-  return id.startsWith("note:") && graph?.model.nodes.some((node) => node.id === id && node.kind === "note") ? id : null;
+  const kind = id.startsWith("note:") ? "note" : id.startsWith("artifact:") ? "file" : null;
+  return kind !== null && graph?.model.nodes.some((node) => node.id === id && node.kind === kind) ? id : null;
 }
 
 export type ColumnId = "tree" | "note" | "graph";

package/src/web/client/tree/Tree.tsx

@@ -2,7 +2,7 @@
 
 import { NOTE_DRAG_TYPE } from "../shell/shell.model";
 import { useLayoutEffect, useRef, useState } from "preact/hooks";
-import { createFolder, deleteFolder, deleteNote, moveNote, renameFolder, renameNote } from "../api";
+import { createFolder, deleteArtifact, deleteFolder, deleteNote, moveArtifact, moveNote, renameArtifact, renameFolder, renameNote } from "../api";
 import { fetchJson } from "../api.dom";
 import { isTextEntry, type KeyTarget } from "../shell/keys.model";
 import { ICON_BOX, ICON_STROKE, ICONS } from "../shell/icons.model";
@@ -21,6 +21,7 @@ import {
   deleteNeedsConfirmation,
   depthVar,
   dropFolder,
+  movableTreeRow,
   initialTreeView,
   newFolderParent,
   newFolderPath,
@@ -41,6 +42,7 @@ import {
   treeEmptyMessage,
   treeKey,
 } from "./tree.model";
+import type { TreeTarget } from "./tree.model";
 
 export interface TreeProps {
   graph: GraphPayload | null;
@@ -91,7 +93,7 @@ function Row({ view, recentIds, edit, onEdit, onRename, onSelect, onToggle, onMe
       aria-expanded={view.hasKids ? view.expanded : undefined}
       style={depthVar(view.depth)}
       onClick={(event) => onSelect(event.metaKey || event.ctrlKey)}
-      draggable={view.id.startsWith("note:")}
+      draggable={movableTreeRow(view.id) !== null}
       aria-label={view.id === DRAFT_FOLDER_ID ? "New folder" : undefined}
       onDragStart={(event) => { event.dataTransfer?.setData("text/plain", view.id); event.dataTransfer?.setData(NOTE_DRAG_TYPE, view.id); if (event.dataTransfer) event.dataTransfer.effectAllowed = "copyMove"; }}
       onDragOver={folder === undefined ? undefined : (event) => event.preventDefault()}
@@ -210,7 +212,7 @@ export function Tree(props: TreeProps) {
       // Before the request: asking after a delete is asking about a file that
       // is already gone, and "keep editing" would strand the draft on a 404.
       if (!props.onMutate(affectedNoteSlugs(props.graph, target))) return;
-      await run(target.type === "note" ? deleteNote(fetchJson, target.path) : deleteFolder(fetchJson, target.path), deletesSelection(props.selectedId, target) ? "vault" : undefined);
+      await run(target.type === "note" ? deleteNote(fetchJson, target.path) : target.type === "artifact" ? deleteArtifact(fetchJson, target.path) : deleteFolder(fetchJson, target.path), deletesSelection(props.selectedId, target) ? "vault" : undefined);
     }
   };
 
@@ -222,7 +224,7 @@ export function Tree(props: TreeProps) {
    * the row shows no count for, so it takes two — and the second click's label
    * names the folder, which is the chance to notice the wrong row was hit.
    */
-  const clickDelete = (target: { type: "note" | "folder" | "vault" }): void => {
+  const clickDelete = (target: Pick<TreeTarget, "type">): void => {
     if (menu === null) return;
     if (deleteNeedsConfirmation(target) && menu.armed !== true) {
       setMenu({ ...menu, armed: true });
@@ -259,7 +261,7 @@ export function Tree(props: TreeProps) {
     // An open draft is a different question; `onMutate` asks that one.
     if (target !== null && name && name !== current.label) {
       if (!props.onMutate(affectedNoteSlugs(props.graph, target))) return;
-      void run(target.type === "note" ? renameNote(fetchJson, target.path, name) : renameFolder(fetchJson, target.path, name));
+      void run(target.type === "note" ? renameNote(fetchJson, target.path, name) : target.type === "artifact" ? renameArtifact(fetchJson, target.path, name) : renameFolder(fetchJson, target.path, name));
     }
   };
 
@@ -297,7 +299,7 @@ export function Tree(props: TreeProps) {
             setMenu({ id: "vault", label: "Vault", x: event.clientX, y: event.clientY });
           }}
         >
-          {rowViews(rows, props.selectedId, props.now).map((view) => <Row key={view.id} view={view} recentIds={props.recentIds} edit={editing?.id === view.id ? editing.value : null} onEdit={(value) => setEditing((current) => current === null ? null : { ...current, value })} onRename={commitEdit} onSelect={(newTab) => { props.onSelect(view.id, newTab); if (view.hasKids) setState((current) => toggleExpanded(current, view.id)); }} onToggle={() => setState(toggleExpanded(state, view.id))} onMenu={(x, y) => setMenu({ id: view.id, label: view.label, x, y })} onDrop={(dragged) => { const folder = dropFolder(view.id); if (dragged.startsWith("note:") && folder !== undefined) { const target = mutableTreeRow(dragged); if (target !== null && !props.onMutate(affectedNoteSlugs(props.graph, target))) return; void run(moveNote(fetchJson, dragged.slice("note:".length), folder)); } }} />)}
+          {rowViews(rows, props.selectedId, props.now).map((view) => <Row key={view.id} view={view} recentIds={props.recentIds} edit={editing?.id === view.id ? editing.value : null} onEdit={(value) => setEditing((current) => current === null ? null : { ...current, value })} onRename={commitEdit} onSelect={(newTab) => { props.onSelect(view.id, newTab); if (view.hasKids) setState((current) => toggleExpanded(current, view.id)); }} onToggle={() => setState(toggleExpanded(state, view.id))} onMenu={(x, y) => setMenu({ id: view.id, label: view.label, x, y })} onDrop={(dragged) => { const folder = dropFolder(view.id); const moving = movableTreeRow(dragged); if (moving === null || folder === undefined) return; if (moving.type === "artifact") { void run(moveArtifact(fetchJson, moving.slug, folder)); return; } const target = mutableTreeRow(dragged); if (target !== null && !props.onMutate(affectedNoteSlugs(props.graph, target))) return; void run(moveNote(fetchJson, moving.slug, folder)); }} />)}
         </ul>
       ) : <p class="weave-tree-empty">{empty}</p>}
       <p class="weave-tree-count">{rowCountLabel(rows)}</p>

package/src/web/client/tree/tree.model.ts

@@ -131,7 +131,7 @@ export const DRAFT_FOLDER_ID = "vfolder:\u0000draft";
  * it; on a *note*, a sibling of that note — which is what "new folder here"
  * means when "here" is a file. Returns the parent path, `""` for the root.
  */
-export function newFolderParent(target: { type: "note" | "folder" | "vault"; path: string }): string {
+export function newFolderParent(target: TreeTarget): string {
   if (target.type === "vault") return "";
   if (target.type === "folder") return target.path;
   return target.path.split("/").slice(0, -1).join("/");
@@ -225,25 +225,40 @@ export function dropFolder(id: string): string | null | undefined {
   return id.startsWith("vfolder:") ? id.slice("vfolder:".length) : undefined;
 }
 
-export function mutableTreeRow(id: string): { type: "note" | "folder" | "vault"; path: string } | null {
+/** A row the tree can drag into a folder: a vault note or a vault HTML artifact. */
+export function movableTreeRow(id: string): { type: "note" | "artifact"; slug: string } | null {
+  if (id.startsWith("note:")) return { type: "note", slug: id.slice("note:".length) };
+  if (id.startsWith("artifact:")) return { type: "artifact", slug: id.slice("artifact:".length) };
+  return null;
+}
+
+/** A tree row the file actions (rename, delete, new folder) can act on. */
+export interface TreeTarget {
+  type: "note" | "artifact" | "folder" | "vault";
+  path: string;
+}
+
+export function mutableTreeRow(id: string): TreeTarget | null {
   if (id === "vault") return { type: "vault", path: "" };
   if (id.startsWith("note:")) return { type: "note", path: id.slice("note:".length) };
+  if (id.startsWith("artifact:")) return { type: "artifact", path: id.slice("artifact:".length) };
   if (id.startsWith("vfolder:")) return { type: "folder", path: id.slice("vfolder:".length) };
   return null;
 }
 
 /** Note slugs a note or folder mutation can invalidate. */
-export function affectedNoteSlugs(graph: GraphPayload | null, target: { type: "note" | "folder" | "vault"; path: string }): string[] {
+export function affectedNoteSlugs(graph: GraphPayload | null, target: TreeTarget): string[] {
   if (target.type === "note") return [target.path];
   if (target.type !== "folder" || graph === null) return [];
   const prefix = `note:${target.path}/`;
   return graph.model.nodes.filter((node) => node.kind === "note" && node.id.startsWith(prefix)).map((node) => node.id.slice(5));
 }
 
-export function deletesSelection(selectedId: string | null, target: { type: "note" | "folder" | "vault"; path: string }): boolean {
+export function deletesSelection(selectedId: string | null, target: TreeTarget): boolean {
   if (selectedId === null) return false;
   if (target.type === "note") return selectedId === `note:${target.path}`;
-  if (target.type === "folder") return selectedId === `vfolder:${target.path}` || selectedId.startsWith(`vfolder:${target.path}/`) || selectedId.startsWith(`note:${target.path}/`);
+  if (target.type === "artifact") return selectedId === `artifact:${target.path}`;
+  if (target.type === "folder") return selectedId === `vfolder:${target.path}` || ["vfolder:", "note:", "artifact:"].some((kind) => selectedId.startsWith(`${kind}${target.path}/`));
   return false;
 }
 
@@ -260,7 +275,7 @@ export function deletesSelection(selectedId: string | null, target: { type: "not
  * The confirmation is {@link deleteItemLabel}: the same menu item, clicked
  * twice, with the second click's label saying exactly what it will destroy.
  */
-export function deleteNeedsConfirmation(target: { type: "note" | "folder" | "vault" }): boolean {
+export function deleteNeedsConfirmation(target: Pick<TreeTarget, "type">): boolean {
   return target.type === "folder";
 }
 
@@ -274,7 +289,7 @@ export function deleteNeedsConfirmation(target: { type: "note" | "folder" | "vau
  * safety mechanism — it is the user's chance to notice they right-clicked the
  * wrong row.
  */
-export function deleteItemLabel(target: { type: "note" | "folder" | "vault" }, label: string, armed: boolean): string {
+export function deleteItemLabel(target: Pick<TreeTarget, "type">, label: string, armed: boolean): string {
   if (!deleteNeedsConfirmation(target)) return "Delete";
   return armed ? `Delete “${label}” and everything in it` : "Delete…";
 }

package/src/web/server/routes.ts

@@ -21,6 +21,9 @@
  * | POST   | `/api/history-settings`  | turn note history on or off                  |
  * | GET    | `/api/okf/:rel`          | {@link OkfFilePayload}                       |
  * | GET    | `/api/artifact/:rel`     | sandboxed HTML artifact                     |
+ * | POST   | `/api/artifact/:rel/move` | move a vault HTML artifact                 |
+ * | POST   | `/api/artifact/:rel/rename` | rename a vault HTML artifact             |
+ * | DELETE | `/api/artifact/:rel`     | delete a vault HTML artifact                |
  * | GET    | `/api/search?q=`         | {@link SearchPayload}                        |
  * | POST   | `/api/open`              | {@link OpenResult} — hand the note to `$EDITOR` |
  *
@@ -64,7 +67,7 @@ import { openNoteInEditor } from "../../core/openInEditor";
 import { recallTrail, type RecallTrail } from "../../core/trail";
 import type { Note } from "../../core/types";
 import type { VaultMutationResult } from "../../core/vault";
-import { addNote, createFolder, deleteFolder, deleteNote, getNote, moveNote, renameFolder, readNoteHistory, renameNote, resolveHtmlPath, searchNotes, setNoteBody } from "../../core/vault";
+import { addNote, createFolder, deleteFolder, deleteNote, deleteHtmlArtifact, getNote, moveHtmlArtifact, moveNote, renameHtmlArtifact, renameFolder, readNoteHistory, renameNote, resolveHtmlPath, searchNotes, setNoteBody } from "../../core/vault";
 import { deriveTagIndex, type TaggedNote } from "../../core/view/links";
 import type {
   GraphPayload,
@@ -434,6 +437,12 @@ async function route(
   if (method === "GET" && path.startsWith("/api/artifact/")) {
     return sendArtifact(deps, path.slice("/api/artifact/".length), res);
   }
+  if (method === "POST" && path.startsWith("/api/artifact/") && /\/(move|rename)$/.test(path)) {
+    return changeArtifact(deps, path.slice("/api/artifact/".length), req, res);
+  }
+  if (method === "DELETE" && path.startsWith("/api/artifact/")) {
+    return sendMutation(res, await deleteHtmlArtifact(deps.vaultRoot, path.slice("/api/artifact/".length)), "artifact");
+  }
   if (method === "GET" && path === "/api/search") return sendSearch(deps, query, res);
   if (method === "POST" && path === "/api/open") return openNote(deps, req, res);
   sendText(res, 404, "not found\n");
@@ -705,9 +714,22 @@ async function routeFolder(deps: RouteDeps, method: string, target: string, req:
   return false;
 }
 
-function sendMutation(res: ServerResponse, result: VaultMutationResult): void {
+/** `POST /api/artifact/:rel/move` and `/rename`; `rel` still carries the suffix. */
+async function changeArtifact(deps: RouteDeps, target: string, req: IncomingMessage, res: ServerResponse): Promise<void> {
+  const rename = target.endsWith("/rename");
+  const rel = target.slice(0, target.lastIndexOf("/"));
+  const body = await readJsonBody(req);
+  const fields = typeof body === "object" && body !== null ? body as { folder?: unknown; name?: unknown } : {};
+  if (rename) {
+    if (typeof fields.name !== "string") sendJson(res, 400, { error: "expected { name: string }" });
+    else sendMutation(res, await renameHtmlArtifact(deps.vaultRoot, rel, fields.name), "artifact");
+  } else if (fields.folder !== null && typeof fields.folder !== "string") sendJson(res, 400, { error: "expected { folder: string | null }" });
+  else sendMutation(res, await moveHtmlArtifact(deps.vaultRoot, rel, fields.folder), "artifact");
+}
+
+function sendMutation(res: ServerResponse, result: VaultMutationResult, kind: "note" | "artifact" = "note"): void {
   if (result.ok) {
-    const id = result.slug === undefined ? result.path === undefined ? undefined : `vfolder:${result.path}` : `note:${result.slug}`;
+    const id = result.slug === undefined ? result.path === undefined ? undefined : `vfolder:${result.path}` : `${kind}:${result.slug}`;
     sendJson(res, 200, { ok: true, ...(id === undefined ? {} : { id }) });
     return;
   }

package/src/opencode/tools.ts

@@ -1,43 +0,0 @@
-import { executeNoteAction, executeRepoAction, WEAVE_NOTE_DESCRIPTION, WEAVE_REPO_DESCRIPTION, type NoteActionInput, type RepoActionInput } from "../core";
-
-const noteInput = {
-  type: "object",
-  properties: {
-    action: { type: "string", enum: ["list", "get", "add", "append", "finalize", "search", "links", "suggest"] },
-    title: { type: "string", description: "Note title (add)" },
-    text: { type: "string", description: "Markdown body (add), addition (append), or restructured body (finalize)" },
-    tags: { type: "array", items: { type: "string" }, description: "Tags (add)" },
-    slug: { type: "string", description: "Note slug (get, append, finalize)" },
-    raw: { type: "boolean", description: "add/append: preserve text verbatim in the ## Raw tail (use for the user's own words)" },
-    source: { type: "string", enum: ["human", "agent"], description: "Provenance (add; defaults to agent)" },
-    query: { type: "string", description: "Search query (search)" },
-    fix: { type: "boolean", description: "links: apply unambiguous repairs" },
-    limit: { type: "number", description: "suggest: maximum suggestions (default 20)" },
-  },
-  required: ["action"],
-  additionalProperties: false,
-} as const;
-
-const repoInput = {
-  type: "object",
-  properties: {
-    action: { type: "string", enum: ["overview"] },
-  },
-  required: ["action"],
-  additionalProperties: false,
-} as const;
-
-export const weaveTools = [
-  {
-    name: "weave_note",
-    description: WEAVE_NOTE_DESCRIPTION,
-    input: noteInput,
-    execute: (input: unknown, _cwd: string, _progress?: (text: string) => Promise<void>) => executeNoteAction(input as NoteActionInput),
-  },
-  {
-    name: "weave_repo",
-    description: WEAVE_REPO_DESCRIPTION,
-    input: repoInput,
-    execute: (input: unknown, cwd: string) => executeRepoAction(input as RepoActionInput, cwd),
-  },
-];