npm - kanna-code - Versions diffs - 0.17.0 → 0.18.0 - Mend

kanna-code 0.17.0 → 0.18.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (16) hide show

package/dist/client/assets/index-BvhI1JYs.css +32 -0
package/dist/client/assets/index-CLqK0AWC.js +606 -0
package/dist/client/index.html +2 -2
package/package.json +2 -1
package/src/server/agent.test.ts +64 -2
package/src/server/agent.ts +45 -3
package/src/server/paths.ts +4 -0
package/src/server/server.ts +178 -1
package/src/server/uploads.test.ts +282 -0
package/src/server/uploads.ts +127 -0
package/src/shared/protocol.ts +2 -0
package/src/shared/tools.test.ts +65 -0
package/src/shared/tools.ts +76 -0
package/src/shared/types.ts +33 -1
package/dist/client/assets/index-CtKZIZdD.js +0 -533
package/dist/client/assets/index-Cwex1U8S.css +0 -32

package/dist/client/index.html CHANGED Viewed

@@ -5,8 +5,8 @@
     <meta name="viewport" content="width=device-width, initial-scale=1.0" />
     <link rel="icon" type="image/png" href="/favicon.png" />
     <title>Kanna</title>
-    <script type="module" crossorigin src="/assets/index-CtKZIZdD.js"></script>
-    <link rel="stylesheet" crossorigin href="/assets/index-Cwex1U8S.css">
+    <script type="module" crossorigin src="/assets/index-CLqK0AWC.js"></script>
+    <link rel="stylesheet" crossorigin href="/assets/index-BvhI1JYs.css">
   </head>
   <body>
     <div id="root"></div>

package/package.json CHANGED Viewed

@@ -1,7 +1,7 @@
 {
   "name": "kanna-code",
   "type": "module",
-  "version": "0.17.0",
+  "version": "0.18.0",
   "description": "A beautiful web UI for Claude Code",
   "license": "MIT",
   "keywords": [
@@ -54,6 +54,7 @@
     "@xterm/xterm": "^6.0.0",
     "cloudflared": "^0.7.1",
     "default-shell": "^2.2.0",
+    "file-type": "^22.0.0",
     "qrcode": "^1.5.4",
     "react-resizable-panels": "^4.7.3"
   },

package/src/server/agent.test.ts CHANGED Viewed

@@ -1,7 +1,7 @@
 import { describe, expect, test } from "bun:test"
-import { AgentCoordinator, normalizeClaudeStreamMessage } from "./agent"
+import { AgentCoordinator, buildAttachmentHintText, buildPromptText, normalizeClaudeStreamMessage } from "./agent"
 import type { HarnessTurn } from "./harness-types"
-import type { TranscriptEntry } from "../shared/types"
+import type { ChatAttachment, TranscriptEntry } from "../shared/types"
 function timestamped<T extends Omit<TranscriptEntry, "_id" | "createdAt">>(entry: T): TranscriptEntry {
   return {
@@ -63,6 +63,68 @@ describe("normalizeClaudeStreamMessage", () => {
   })
 })
+describe("attachment prompt helpers", () => {
+  test("appends a structured attachment hint block for all attachment kinds", () => {
+    const attachments: ChatAttachment[] = [
+      {
+        id: "image-1",
+        kind: "image",
+        displayName: "shot.png",
+        absolutePath: "/tmp/project/.kanna/uploads/shot.png",
+        relativePath: "./.kanna/uploads/shot.png",
+        contentUrl: "/api/projects/project-1/uploads/shot.png/content",
+        mimeType: "image/png",
+        size: 512,
+      },
+      {
+        id: "file-1",
+        kind: "file",
+        displayName: "spec.pdf",
+        absolutePath: "/tmp/project/.kanna/uploads/spec.pdf",
+        relativePath: "./.kanna/uploads/spec.pdf",
+        contentUrl: "/api/projects/project-1/uploads/spec.pdf/content",
+        mimeType: "application/pdf",
+        size: 1234,
+      },
+    ]
+    const prompt = buildPromptText("Review these", attachments)
+    expect(prompt).toContain("<kanna-attachments>")
+    expect(prompt).toContain('path="/tmp/project/.kanna/uploads/shot.png"')
+    expect(prompt).toContain('project_path="./.kanna/uploads/spec.pdf"')
+  })
+  test("supports attachment-only prompts", () => {
+    const attachments: ChatAttachment[] = [{
+      id: "file-1",
+      kind: "file",
+      displayName: "todo.txt",
+      absolutePath: "/tmp/project/.kanna/uploads/todo.txt",
+      relativePath: "./.kanna/uploads/todo.txt",
+      contentUrl: "/api/projects/project-1/uploads/todo.txt/content",
+      mimeType: "text/plain",
+      size: 32,
+    }]
+    expect(buildPromptText("", attachments)).toContain("Please inspect the attached files.")
+  })
+  test("escapes xml attribute values for attachment hint markup", () => {
+    const hint = buildAttachmentHintText([{
+      id: "file-1",
+      kind: "file",
+      displayName: "\"report\" <draft>.txt",
+      absolutePath: "/tmp/project/.kanna/uploads/report.txt",
+      relativePath: "./.kanna/uploads/report.txt",
+      contentUrl: "/api/projects/project-1/uploads/report.txt/content",
+      mimeType: "text/plain",
+      size: 64,
+    }])
+    expect(hint).toContain("&quot;report&quot; &lt;draft&gt;.txt")
+  })
+})
 describe("AgentCoordinator codex integration", () => {
   test("generates a chat title in the background on the first user message", async () => {
     const fakeCodexManager = {

package/src/server/agent.ts CHANGED Viewed

@@ -1,6 +1,7 @@
 import { query, type CanUseTool, type PermissionResult, type Query } from "@anthropic-ai/claude-agent-sdk"
 import type {
   AgentProvider,
+  ChatAttachment,
   NormalizedToolCall,
   PendingToolSnapshot,
   KannaStatus,
@@ -89,6 +90,41 @@ function stringFromUnknown(value: unknown) {
   }
 }
+function escapeXmlAttribute(value: string) {
+  return value
+    .replaceAll("&", "&amp;")
+    .replaceAll("\"", "&quot;")
+    .replaceAll("<", "&lt;")
+    .replaceAll(">", "&gt;")
+}
+export function buildAttachmentHintText(attachments: ChatAttachment[]) {
+  if (attachments.length === 0) return ""
+  const lines = attachments.map((attachment) => (
+    `<attachment kind="${escapeXmlAttribute(attachment.kind)}" mime_type="${escapeXmlAttribute(attachment.mimeType)}" path="${escapeXmlAttribute(attachment.absolutePath)}" project_path="${escapeXmlAttribute(attachment.relativePath)}" size_bytes="${attachment.size}" display_name="${escapeXmlAttribute(attachment.displayName)}" />`
+  ))
+  return [
+    "<kanna-attachments>",
+    ...lines,
+    "</kanna-attachments>",
+  ].join("\n")
+}
+export function buildPromptText(content: string, attachments: ChatAttachment[]) {
+  const attachmentHint = buildAttachmentHintText(attachments)
+  if (!attachmentHint) {
+    return content.trim()
+  }
+  const trimmed = content.trim()
+  return [
+    trimmed || "Please inspect the attached files.",
+    attachmentHint,
+  ].join("\n\n").trim()
+}
 function discardedToolResult(
   tool: NormalizedToolCall & { toolKind: "ask_user_question" | "exit_plan_mode" }
 ) {
@@ -389,6 +425,7 @@ export class AgentCoordinator {
     chatId: string
     provider: AgentProvider
     content: string
+    attachments: ChatAttachment[]
     model: string
     effort?: string
     serviceTier?: "fast"
@@ -409,7 +446,10 @@ export class AgentCoordinator {
     const shouldGenerateTitle = args.appendUserPrompt && chat.title === "New Chat" && existingMessages.length === 0
     if (args.appendUserPrompt) {
-      await this.store.appendMessage(args.chatId, timestamped({ kind: "user_prompt", content: args.content }, Date.now()))
+      await this.store.appendMessage(
+        args.chatId,
+        timestamped({ kind: "user_prompt", content: args.content, attachments: args.attachments }, Date.now())
+      )
     }
     await this.store.recordTurnStarted(args.chatId)
@@ -443,7 +483,7 @@ export class AgentCoordinator {
     let turn: HarnessTurn
     if (args.provider === "claude") {
       turn = await startClaudeTurn({
-        content: args.content,
+        content: buildPromptText(args.content, args.attachments),
         localPath: project.localPath,
         model: args.model,
         effort: args.effort,
@@ -461,7 +501,7 @@ export class AgentCoordinator {
       })
       turn = await this.codexManager.startTurn({
         chatId: args.chatId,
-        content: args.content,
+        content: buildPromptText(args.content, args.attachments),
         model: args.model,
         effort: args.effort as any,
         serviceTier: args.serviceTier,
@@ -519,6 +559,7 @@ export class AgentCoordinator {
       chatId,
       provider,
       content: command.content,
+      attachments: command.attachments ?? [],
       model: settings.model,
       effort: settings.effort,
       serviceTier: settings.serviceTier,
@@ -600,6 +641,7 @@ export class AgentCoordinator {
             chatId: active.chatId,
             provider: active.provider,
             content: active.postToolFollowUp.content,
+            attachments: [],
             model: active.model,
             effort: active.effort,
             serviceTier: active.serviceTier,

package/src/server/paths.ts CHANGED Viewed

@@ -25,3 +25,7 @@ export async function ensureProjectDirectory(localPath: string) {
     throw new Error("Project path must be a directory")
   }
 }
+export function getProjectUploadDir(localPath: string) {
+  return path.join(resolveLocalPath(localPath), ".kanna", "uploads")
+}

package/src/server/server.ts CHANGED Viewed

@@ -1,5 +1,7 @@
 import path from "node:path"
+import { stat } from "node:fs/promises"
 import { APP_NAME, getRuntimeProfile } from "../shared/branding"
+import type { ChatAttachment } from "../shared/types"
 import { EventStore } from "./event-store"
 import { AgentCoordinator } from "./agent"
 import { discoverProjects, type DiscoveredProject } from "./discovery"
@@ -9,6 +11,45 @@ import { TerminalManager } from "./terminal-manager"
 import { UpdateManager } from "./update-manager"
 import type { UpdateInstallAttemptResult } from "./cli-runtime"
 import { createWsRouter, type ClientState } from "./ws-router"
+import { deleteProjectUpload, inferAttachmentContentType, persistProjectUpload } from "./uploads"
+import { getProjectUploadDir } from "./paths"
+const MAX_UPLOAD_FILES = 10
+const MAX_UPLOAD_SIZE_BYTES = 25 * 1024 * 1024
+export async function persistUploadedFiles(args: {
+  projectId: string
+  localPath: string
+  files: File[]
+  persistUpload?: typeof persistProjectUpload
+}): Promise<ChatAttachment[]> {
+  const persistUpload = args.persistUpload ?? persistProjectUpload
+  const attachments: ChatAttachment[] = []
+  try {
+    for (const file of args.files) {
+      const bytes = new Uint8Array(await file.arrayBuffer())
+      const attachment = await persistUpload({
+        projectId: args.projectId,
+        localPath: args.localPath,
+        fileName: file.name,
+        bytes,
+        fallbackMimeType: file.type || undefined,
+      })
+      attachments.push(attachment)
+    }
+  } catch (error) {
+    await Promise.allSettled(
+      attachments.map((attachment) => deleteProjectUpload({
+        localPath: args.localPath,
+        storedName: path.basename(attachment.absolutePath),
+      }))
+    )
+    throw error
+  }
+  return attachments
+}
 export interface StartKannaServerOptions {
   port?: number
@@ -79,7 +120,7 @@ export async function startKannaServer(options: StartKannaServerOptions = {}) {
       server = Bun.serve<ClientState>({
         port: actualPort,
         hostname,
-        fetch(req, serverInstance) {
+        async fetch(req, serverInstance) {
           const url = new URL(req.url)
           if (url.pathname === "/ws") {
@@ -95,6 +136,21 @@ export async function startKannaServer(options: StartKannaServerOptions = {}) {
             return Response.json({ ok: true, port: actualPort })
           }
+          const uploadResponse = await handleProjectUpload(req, url, store)
+          if (uploadResponse) {
+            return uploadResponse
+          }
+          const deleteUploadResponse = await handleProjectUploadDelete(req, url, store)
+          if (deleteUploadResponse) {
+            return deleteUploadResponse
+          }
+          const attachmentContentResponse = await handleAttachmentContent(req, url, store)
+          if (attachmentContentResponse) {
+            return attachmentContentResponse
+          }
           return serveStatic(distDir, url.pathname)
         },
         websocket: {
@@ -140,6 +196,127 @@ export async function startKannaServer(options: StartKannaServerOptions = {}) {
   }
 }
+async function handleProjectUpload(req: Request, url: URL, store: EventStore) {
+  if (req.method !== "POST") {
+    return null
+  }
+  const match = url.pathname.match(/^\/api\/projects\/([^/]+)\/uploads$/)
+  if (!match) {
+    return null
+  }
+  const project = store.getProject(match[1])
+  if (!project) {
+    return Response.json({ error: "Project not found" }, { status: 404 })
+  }
+  const formData = await req.formData()
+  const files = formData
+    .getAll("files")
+    .filter((value): value is File => value instanceof File)
+  if (files.length === 0) {
+    return Response.json({ error: "No files uploaded" }, { status: 400 })
+  }
+  if (files.length > MAX_UPLOAD_FILES) {
+    return Response.json({ error: `You can upload up to ${MAX_UPLOAD_FILES} files at a time.` }, { status: 400 })
+  }
+  for (const file of files) {
+    if (file.size > MAX_UPLOAD_SIZE_BYTES) {
+      return Response.json(
+        { error: `File "${file.name}" exceeds the ${Math.floor(MAX_UPLOAD_SIZE_BYTES / (1024 * 1024))} MB limit.` },
+        { status: 413 }
+      )
+    }
+  }
+  try {
+    const attachments = await persistUploadedFiles({
+      projectId: project.id,
+      localPath: project.localPath,
+      files,
+    })
+    return Response.json({ attachments })
+  } catch (error) {
+    console.error("[uploads] Upload failed:", error)
+    return Response.json({ error: "Upload failed" }, { status: 500 })
+  }
+}
+async function handleAttachmentContent(req: Request, url: URL, store: EventStore) {
+  const match = url.pathname.match(/^\/api\/projects\/([^/]+)\/uploads\/([^/]+)\/content$/)
+  if (!match) {
+    return null
+  }
+  if (req.method !== "GET") {
+    return new Response(null, {
+      status: 405,
+      headers: {
+        Allow: "GET",
+      },
+    })
+  }
+  const project = store.getProject(match[1])
+  if (!project) {
+    return Response.json({ error: "Project not found" }, { status: 404 })
+  }
+  const storedName = decodeURIComponent(match[2])
+  if (!storedName || storedName.includes("/") || storedName.includes("\\") || storedName === "." || storedName === "..") {
+    return Response.json({ error: "Invalid attachment path" }, { status: 400 })
+  }
+  const filePath = path.join(getProjectUploadDir(project.localPath), storedName)
+  const file = Bun.file(filePath)
+  try {
+    const info = await stat(filePath)
+    if (!info.isFile()) {
+      return Response.json({ error: "Attachment not found" }, { status: 404 })
+    }
+  } catch {
+    return Response.json({ error: "Attachment not found" }, { status: 404 })
+  }
+  return new Response(file, {
+    headers: {
+      "Content-Type": inferAttachmentContentType(storedName, file.type),
+    },
+  })
+}
+async function handleProjectUploadDelete(req: Request, url: URL, store: EventStore) {
+  if (req.method !== "DELETE") {
+    return null
+  }
+  const match = url.pathname.match(/^\/api\/projects\/([^/]+)\/uploads\/([^/]+)$/)
+  if (!match) {
+    return null
+  }
+  const project = store.getProject(match[1])
+  if (!project) {
+    return Response.json({ error: "Project not found" }, { status: 404 })
+  }
+  const storedName = decodeURIComponent(match[2])
+  if (!storedName || storedName.includes("/") || storedName.includes("\\") || storedName === "." || storedName === "..") {
+    return Response.json({ error: "Invalid attachment path" }, { status: 400 })
+  }
+  const deleted = await deleteProjectUpload({
+    localPath: project.localPath,
+    storedName,
+  })
+  return Response.json({ ok: deleted })
+}
 async function serveStatic(distDir: string, pathname: string) {
   const requestedPath = pathname === "/" ? "/index.html" : pathname
   const filePath = path.join(distDir, requestedPath)