kanban-lite 1.0.32 → 1.0.33

package/CHANGELOG.md

@@ -8,6 +8,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ## [Unreleased]
 
 ### Added
+- **Attachments subfolder**: attachments for the markdown storage engine are now stored in an `attachments/` subdirectory inside each column folder (e.g. `.kanban/boards/default/backlog/attachments/`) instead of alongside the card `.md` files
+- **Browser-viewable attachments**: PDFs and other binary attachments now open with the OS/browser default viewer in the VS Code extension; the standalone server now serves PDF, JPEG, GIF, WebP, CSV, plain-text, and XML attachments with correct `Content-Type` headers so browsers render them inline in a new tab
+- **KanbanSDK.getAttachmentDir(cardId, boardId?)**: new public SDK method that returns the absolute path to the attachment directory for a card (delegates to the active storage engine)
 - **Pluggable storage engine**: new `StorageEngine` interface (`src/sdk/storage/types.ts`) decouples all card I/O from the SDK business logic
 - **SQLite storage engine**: `SqliteStorageEngine` stores cards and comments in a single `.kanban/kanban.db` file using `better-sqlite3`; config (boards, columns, labels, webhooks) always stays in `.kanban.json`
 - **Markdown storage engine**: `MarkdownStorageEngine` wraps the existing file-based I/O, unchanged default behavior

package/dist/cli.js

@@ -506,11 +506,14 @@ async function moveCardFile(currentPath, kanbanDir, newStatus, attachments) {
   await fs2.rename(currentPath, targetPath);
   if (attachments && attachments.length > 0) {
     const sourceDir = path2.dirname(currentPath);
+    const sourceAttachmentsDir = path2.join(sourceDir, "attachments");
+    const destAttachmentsDir = path2.join(targetDir, "attachments");
     for (const attachment of attachments) {
-      const srcAttach = path2.join(sourceDir, attachment);
-      const destAttach = path2.join(targetDir, attachment);
+      const srcAttach = path2.join(sourceAttachmentsDir, attachment);
+      const destAttach = path2.join(destAttachmentsDir, attachment);
       try {
         await fs2.access(srcAttach);
+        await fs2.mkdir(destAttachmentsDir, { recursive: true });
         await fs2.rename(srcAttach, destAttach);
       } catch {
       }
@@ -3516,15 +3519,16 @@ var init_markdown = __esm({
       }
       // --- Attachments ---
       getCardDir(card) {
-        return path5.dirname(card.filePath);
+        return path5.join(path5.dirname(card.filePath), "attachments");
       }
       async copyAttachment(sourcePath, card) {
         const filename = path5.basename(sourcePath);
-        const cardDir = this.getCardDir(card);
-        const destPath = path5.join(cardDir, filename);
-        const sourceDir = path5.dirname(path5.resolve(sourcePath));
-        if (sourceDir !== cardDir) {
-          await fs4.copyFile(path5.resolve(sourcePath), destPath);
+        const attachmentDir = this.getCardDir(card);
+        await fs4.mkdir(attachmentDir, { recursive: true });
+        const destPath = path5.join(attachmentDir, filename);
+        const resolvedSource = path5.resolve(sourcePath);
+        if (path5.dirname(resolvedSource) !== attachmentDir) {
+          await fs4.copyFile(resolvedSource, destPath);
         }
       }
       // --- Private helpers ---
@@ -5916,6 +5920,28 @@ var init_KanbanSDK = __esm({
           throw new Error(`Card not found: ${cardId}`);
         return card.attachments;
       }
+      /**
+       * Returns the absolute path to the attachment directory for a card.
+       *
+       * For the markdown engine this is `{column_dir}/attachments/`.
+       * For the SQLite engine this is `.kanban/boards/{boardId}/attachments/{cardId}/`.
+       *
+       * @param cardId - The ID of the card.
+       * @param boardId - Optional board ID. Defaults to the workspace's default board.
+       * @returns A promise resolving to the absolute directory path, or `null` if the card is not found.
+       *
+       * @example
+       * ```ts
+       * const dir = await sdk.getAttachmentDir('42')
+       * // '/workspace/.kanban/boards/default/backlog/attachments'
+       * ```
+       */
+      async getAttachmentDir(cardId, boardId) {
+        const card = await this.getCard(cardId, boardId);
+        if (!card)
+          return null;
+        return this._storage.getCardDir(card);
+      }
       // --- Comment management ---
       /**
        * Lists all comments on a card.
@@ -51778,7 +51804,8 @@ function startServer(kanbanDir, port, webviewDir) {
     const card = cards.find((f) => f.id === cardId);
     if (!card)
       return false;
-    const cardDir = path10.dirname(card.filePath);
+    const cardDir = sdk.storageEngine.getCardDir(card);
+    fs7.mkdirSync(cardDir, { recursive: true });
     fs7.writeFileSync(path10.join(cardDir, filename), fileData);
     migrating = true;
     try {
@@ -52525,7 +52552,7 @@ function startServer(kanbanDir, port, webviewDir) {
       const card = cards.find((f) => f.id === id);
       if (!card)
         return jsonError(res, 404, "Task not found");
-      const cardDir = path10.dirname(card.filePath);
+      const cardDir = sdk.storageEngine.getCardDir(card);
       const attachmentPath = path10.resolve(cardDir, attachName);
       if (!attachmentPath.startsWith(absoluteKanbanDir)) {
         res.writeHead(403, { "Content-Type": "text/plain" });
@@ -52844,7 +52871,7 @@ function startServer(kanbanDir, port, webviewDir) {
         res.end("Card not found");
         return;
       }
-      const cardDir = path10.dirname(card.filePath);
+      const cardDir = sdk.storageEngine.getCardDir(card);
       const attachmentPath = path10.resolve(cardDir, filename);
       if (!attachmentPath.startsWith(absoluteKanbanDir)) {
         res.writeHead(403, { "Content-Type": "text/plain" });
@@ -52989,8 +53016,16 @@ var init_server3 = __esm({
       ".css": "text/css",
       ".json": "application/json",
       ".png": "image/png",
+      ".jpg": "image/jpeg",
+      ".jpeg": "image/jpeg",
+      ".gif": "image/gif",
+      ".webp": "image/webp",
       ".svg": "image/svg+xml",
       ".ico": "image/x-icon",
+      ".pdf": "application/pdf",
+      ".txt": "text/plain",
+      ".xml": "text/xml",
+      ".csv": "text/csv",
       ".map": "application/json"
     };
   }

package/dist/extension.js

@@ -318,11 +318,14 @@ async function moveCardFile(currentPath, kanbanDir, newStatus, attachments) {
   await fs2.rename(currentPath, targetPath);
   if (attachments && attachments.length > 0) {
     const sourceDir = path2.dirname(currentPath);
+    const sourceAttachmentsDir = path2.join(sourceDir, "attachments");
+    const destAttachmentsDir = path2.join(targetDir, "attachments");
     for (const attachment of attachments) {
-      const srcAttach = path2.join(sourceDir, attachment);
-      const destAttach = path2.join(targetDir, attachment);
+      const srcAttach = path2.join(sourceAttachmentsDir, attachment);
+      const destAttach = path2.join(destAttachmentsDir, attachment);
       try {
         await fs2.access(srcAttach);
+        await fs2.mkdir(destAttachmentsDir, { recursive: true });
         await fs2.rename(srcAttach, destAttach);
       } catch {
       }
@@ -3328,15 +3331,16 @@ var init_markdown = __esm({
       }
       // --- Attachments ---
       getCardDir(card) {
-        return path5.dirname(card.filePath);
+        return path5.join(path5.dirname(card.filePath), "attachments");
       }
       async copyAttachment(sourcePath, card) {
         const filename = path5.basename(sourcePath);
-        const cardDir = this.getCardDir(card);
-        const destPath = path5.join(cardDir, filename);
-        const sourceDir = path5.dirname(path5.resolve(sourcePath));
-        if (sourceDir !== cardDir) {
-          await fs4.copyFile(path5.resolve(sourcePath), destPath);
+        const attachmentDir = this.getCardDir(card);
+        await fs4.mkdir(attachmentDir, { recursive: true });
+        const destPath = path5.join(attachmentDir, filename);
+        const resolvedSource = path5.resolve(sourcePath);
+        if (path5.dirname(resolvedSource) !== attachmentDir) {
+          await fs4.copyFile(resolvedSource, destPath);
         }
       }
       // --- Private helpers ---
@@ -9640,6 +9644,28 @@ var KanbanSDK = class {
       throw new Error(`Card not found: ${cardId}`);
     return card.attachments;
   }
+  /**
+   * Returns the absolute path to the attachment directory for a card.
+   *
+   * For the markdown engine this is `{column_dir}/attachments/`.
+   * For the SQLite engine this is `.kanban/boards/{boardId}/attachments/{cardId}/`.
+   *
+   * @param cardId - The ID of the card.
+   * @param boardId - Optional board ID. Defaults to the workspace's default board.
+   * @returns A promise resolving to the absolute directory path, or `null` if the card is not found.
+   *
+   * @example
+   * ```ts
+   * const dir = await sdk.getAttachmentDir('42')
+   * // '/workspace/.kanban/boards/default/backlog/attachments'
+   * ```
+   */
+  async getAttachmentDir(cardId, boardId) {
+    const card = await this.getCard(cardId, boardId);
+    if (!card)
+      return null;
+    return this._storage.getCardDir(card);
+  }
   // --- Comment management ---
   /**
    * Lists all comments on a card.
@@ -11960,8 +11986,16 @@ var MIME_TYPES = {
   ".css": "text/css",
   ".json": "application/json",
   ".png": "image/png",
+  ".jpg": "image/jpeg",
+  ".jpeg": "image/jpeg",
+  ".gif": "image/gif",
+  ".webp": "image/webp",
   ".svg": "image/svg+xml",
   ".ico": "image/x-icon",
+  ".pdf": "application/pdf",
+  ".txt": "text/plain",
+  ".xml": "text/xml",
+  ".csv": "text/csv",
   ".map": "application/json"
 };
 function startServer(kanbanDir, port, webviewDir) {
@@ -12228,7 +12262,8 @@ function startServer(kanbanDir, port, webviewDir) {
     const card = cards.find((f) => f.id === cardId);
     if (!card)
       return false;
-    const cardDir = path9.dirname(card.filePath);
+    const cardDir = sdk.storageEngine.getCardDir(card);
+    fs6.mkdirSync(cardDir, { recursive: true });
     fs6.writeFileSync(path9.join(cardDir, filename), fileData);
     migrating = true;
     try {
@@ -12975,7 +13010,7 @@ function startServer(kanbanDir, port, webviewDir) {
       const card = cards.find((f) => f.id === id);
       if (!card)
         return jsonError(res, 404, "Task not found");
-      const cardDir = path9.dirname(card.filePath);
+      const cardDir = sdk.storageEngine.getCardDir(card);
       const attachmentPath = path9.resolve(cardDir, attachName);
       if (!attachmentPath.startsWith(absoluteKanbanDir)) {
         res.writeHead(403, { "Content-Type": "text/plain" });
@@ -13294,7 +13329,7 @@ function startServer(kanbanDir, port, webviewDir) {
         res.end("Card not found");
         return;
       }
-      const cardDir = path9.dirname(card.filePath);
+      const cardDir = sdk.storageEngine.getCardDir(card);
       const attachmentPath = path9.resolve(cardDir, filename);
       if (!attachmentPath.startsWith(absoluteKanbanDir)) {
         res.writeHead(403, { "Content-Type": "text/plain" });
@@ -14076,15 +14111,22 @@ var KanbanPanel = class _KanbanPanel {
     }
   }
   async _openAttachment(cardId, attachment) {
+    const sdk = this._getSDK();
     const card = this._cards.find((f) => f.id === cardId);
     if (!card)
       return;
-    const featureDir = path10.dirname(card.filePath);
-    const attachmentPath = path10.resolve(featureDir, attachment);
+    const attachmentDir = sdk ? await sdk.getAttachmentDir(cardId, this._currentBoardId) ?? path10.join(path10.dirname(card.filePath), "attachments") : path10.join(path10.dirname(card.filePath), "attachments");
+    const attachmentPath = path10.resolve(attachmentDir, attachment);
+    const ext = path10.extname(attachment).toLowerCase();
+    const isTextFile = [".json", ".txt", ".md", ".html", ".xml", ".csv", ".ts", ".js", ".py", ".yaml", ".yml", ".toml", ".log"].includes(ext);
     try {
       await vscode.workspace.fs.stat(vscode.Uri.file(attachmentPath));
-      const doc = await vscode.workspace.openTextDocument(vscode.Uri.file(attachmentPath));
-      await vscode.window.showTextDocument(doc, { viewColumn: vscode.ViewColumn.Beside });
+      if (isTextFile) {
+        const doc = await vscode.workspace.openTextDocument(vscode.Uri.file(attachmentPath));
+        await vscode.window.showTextDocument(doc, { viewColumn: vscode.ViewColumn.Beside });
+      } else {
+        await vscode.env.openExternal(vscode.Uri.file(attachmentPath));
+      }
     } catch {
       await vscode.env.openExternal(vscode.Uri.file(attachmentPath));
     }

package/dist/mcp-server.js

@@ -291,11 +291,14 @@ async function moveCardFile(currentPath, kanbanDir, newStatus, attachments) {
   await fs2.rename(currentPath, targetPath);
   if (attachments && attachments.length > 0) {
     const sourceDir = path2.dirname(currentPath);
+    const sourceAttachmentsDir = path2.join(sourceDir, "attachments");
+    const destAttachmentsDir = path2.join(targetDir, "attachments");
     for (const attachment of attachments) {
-      const srcAttach = path2.join(sourceDir, attachment);
-      const destAttach = path2.join(targetDir, attachment);
+      const srcAttach = path2.join(sourceAttachmentsDir, attachment);
+      const destAttach = path2.join(destAttachmentsDir, attachment);
       try {
         await fs2.access(srcAttach);
+        await fs2.mkdir(destAttachmentsDir, { recursive: true });
         await fs2.rename(srcAttach, destAttach);
       } catch {
       }
@@ -3301,15 +3304,16 @@ var init_markdown = __esm({
       }
       // --- Attachments ---
       getCardDir(card) {
-        return path5.dirname(card.filePath);
+        return path5.join(path5.dirname(card.filePath), "attachments");
       }
       async copyAttachment(sourcePath, card) {
         const filename = path5.basename(sourcePath);
-        const cardDir = this.getCardDir(card);
-        const destPath = path5.join(cardDir, filename);
-        const sourceDir = path5.dirname(path5.resolve(sourcePath));
-        if (sourceDir !== cardDir) {
-          await fs4.copyFile(path5.resolve(sourcePath), destPath);
+        const attachmentDir = this.getCardDir(card);
+        await fs4.mkdir(attachmentDir, { recursive: true });
+        const destPath = path5.join(attachmentDir, filename);
+        const resolvedSource = path5.resolve(sourcePath);
+        if (path5.dirname(resolvedSource) !== attachmentDir) {
+          await fs4.copyFile(resolvedSource, destPath);
         }
       }
       // --- Private helpers ---
@@ -5899,6 +5903,28 @@ var KanbanSDK = class {
       throw new Error(`Card not found: ${cardId}`);
     return card.attachments;
   }
+  /**
+   * Returns the absolute path to the attachment directory for a card.
+   *
+   * For the markdown engine this is `{column_dir}/attachments/`.
+   * For the SQLite engine this is `.kanban/boards/{boardId}/attachments/{cardId}/`.
+   *
+   * @param cardId - The ID of the card.
+   * @param boardId - Optional board ID. Defaults to the workspace's default board.
+   * @returns A promise resolving to the absolute directory path, or `null` if the card is not found.
+   *
+   * @example
+   * ```ts
+   * const dir = await sdk.getAttachmentDir('42')
+   * // '/workspace/.kanban/boards/default/backlog/attachments'
+   * ```
+   */
+  async getAttachmentDir(cardId, boardId) {
+    const card = await this.getCard(cardId, boardId);
+    if (!card)
+      return null;
+    return this._storage.getCardDir(card);
+  }
   // --- Comment management ---
   /**
    * Lists all comments on a card.

package/dist/sdk/index.cjs

@@ -295,11 +295,14 @@ async function moveCardFile(currentPath, kanbanDir, newStatus, attachments) {
   await fs2.rename(currentPath, targetPath);
   if (attachments && attachments.length > 0) {
     const sourceDir = path2.dirname(currentPath);
+    const sourceAttachmentsDir = path2.join(sourceDir, "attachments");
+    const destAttachmentsDir = path2.join(targetDir, "attachments");
     for (const attachment of attachments) {
-      const srcAttach = path2.join(sourceDir, attachment);
-      const destAttach = path2.join(targetDir, attachment);
+      const srcAttach = path2.join(sourceAttachmentsDir, attachment);
+      const destAttach = path2.join(destAttachmentsDir, attachment);
       try {
         await fs2.access(srcAttach);
+        await fs2.mkdir(destAttachmentsDir, { recursive: true });
         await fs2.rename(srcAttach, destAttach);
       } catch {
       }
@@ -3305,15 +3308,16 @@ var init_markdown = __esm({
       }
       // --- Attachments ---
       getCardDir(card) {
-        return path5.dirname(card.filePath);
+        return path5.join(path5.dirname(card.filePath), "attachments");
       }
       async copyAttachment(sourcePath, card) {
         const filename = path5.basename(sourcePath);
-        const cardDir = this.getCardDir(card);
-        const destPath = path5.join(cardDir, filename);
-        const sourceDir = path5.dirname(path5.resolve(sourcePath));
-        if (sourceDir !== cardDir) {
-          await fs4.copyFile(path5.resolve(sourcePath), destPath);
+        const attachmentDir = this.getCardDir(card);
+        await fs4.mkdir(attachmentDir, { recursive: true });
+        const destPath = path5.join(attachmentDir, filename);
+        const resolvedSource = path5.resolve(sourcePath);
+        if (path5.dirname(resolvedSource) !== attachmentDir) {
+          await fs4.copyFile(resolvedSource, destPath);
         }
       }
       // --- Private helpers ---
@@ -5922,6 +5926,28 @@ var KanbanSDK = class {
       throw new Error(`Card not found: ${cardId}`);
     return card.attachments;
   }
+  /**
+   * Returns the absolute path to the attachment directory for a card.
+   *
+   * For the markdown engine this is `{column_dir}/attachments/`.
+   * For the SQLite engine this is `.kanban/boards/{boardId}/attachments/{cardId}/`.
+   *
+   * @param cardId - The ID of the card.
+   * @param boardId - Optional board ID. Defaults to the workspace's default board.
+   * @returns A promise resolving to the absolute directory path, or `null` if the card is not found.
+   *
+   * @example
+   * ```ts
+   * const dir = await sdk.getAttachmentDir('42')
+   * // '/workspace/.kanban/boards/default/backlog/attachments'
+   * ```
+   */
+  async getAttachmentDir(cardId, boardId) {
+    const card = await this.getCard(cardId, boardId);
+    if (!card)
+      return null;
+    return this._storage.getCardDir(card);
+  }
   // --- Comment management ---
   /**
    * Lists all comments on a card.

package/dist/sdk/index.mjs

@@ -302,11 +302,14 @@ async function moveCardFile(currentPath, kanbanDir, newStatus, attachments) {
   await fs2.rename(currentPath, targetPath);
   if (attachments && attachments.length > 0) {
     const sourceDir = path2.dirname(currentPath);
+    const sourceAttachmentsDir = path2.join(sourceDir, "attachments");
+    const destAttachmentsDir = path2.join(targetDir, "attachments");
     for (const attachment of attachments) {
-      const srcAttach = path2.join(sourceDir, attachment);
-      const destAttach = path2.join(targetDir, attachment);
+      const srcAttach = path2.join(sourceAttachmentsDir, attachment);
+      const destAttach = path2.join(destAttachmentsDir, attachment);
       try {
         await fs2.access(srcAttach);
+        await fs2.mkdir(destAttachmentsDir, { recursive: true });
         await fs2.rename(srcAttach, destAttach);
       } catch {
       }
@@ -3307,15 +3310,16 @@ var init_markdown = __esm({
       }
       // --- Attachments ---
       getCardDir(card) {
-        return path5.dirname(card.filePath);
+        return path5.join(path5.dirname(card.filePath), "attachments");
       }
       async copyAttachment(sourcePath, card) {
         const filename = path5.basename(sourcePath);
-        const cardDir = this.getCardDir(card);
-        const destPath = path5.join(cardDir, filename);
-        const sourceDir = path5.dirname(path5.resolve(sourcePath));
-        if (sourceDir !== cardDir) {
-          await fs4.copyFile(path5.resolve(sourcePath), destPath);
+        const attachmentDir = this.getCardDir(card);
+        await fs4.mkdir(attachmentDir, { recursive: true });
+        const destPath = path5.join(attachmentDir, filename);
+        const resolvedSource = path5.resolve(sourcePath);
+        if (path5.dirname(resolvedSource) !== attachmentDir) {
+          await fs4.copyFile(resolvedSource, destPath);
         }
       }
       // --- Private helpers ---
@@ -5898,6 +5902,28 @@ var KanbanSDK = class {
       throw new Error(`Card not found: ${cardId}`);
     return card.attachments;
   }
+  /**
+   * Returns the absolute path to the attachment directory for a card.
+   *
+   * For the markdown engine this is `{column_dir}/attachments/`.
+   * For the SQLite engine this is `.kanban/boards/{boardId}/attachments/{cardId}/`.
+   *
+   * @param cardId - The ID of the card.
+   * @param boardId - Optional board ID. Defaults to the workspace's default board.
+   * @returns A promise resolving to the absolute directory path, or `null` if the card is not found.
+   *
+   * @example
+   * ```ts
+   * const dir = await sdk.getAttachmentDir('42')
+   * // '/workspace/.kanban/boards/default/backlog/attachments'
+   * ```
+   */
+  async getAttachmentDir(cardId, boardId) {
+    const card = await this.getCard(cardId, boardId);
+    if (!card)
+      return null;
+    return this._storage.getCardDir(card);
+  }
   // --- Comment management ---
   /**
    * Lists all comments on a card.

package/dist/sdk/sdk/KanbanSDK.d.ts

@@ -619,6 +619,23 @@ export declare class KanbanSDK {
      * ```
      */
     listAttachments(cardId: string, boardId?: string): Promise<string[]>;
+    /**
+     * Returns the absolute path to the attachment directory for a card.
+     *
+     * For the markdown engine this is `{column_dir}/attachments/`.
+     * For the SQLite engine this is `.kanban/boards/{boardId}/attachments/{cardId}/`.
+     *
+     * @param cardId - The ID of the card.
+     * @param boardId - Optional board ID. Defaults to the workspace's default board.
+     * @returns A promise resolving to the absolute directory path, or `null` if the card is not found.
+     *
+     * @example
+     * ```ts
+     * const dir = await sdk.getAttachmentDir('42')
+     * // '/workspace/.kanban/boards/default/backlog/attachments'
+     * ```
+     */
+    getAttachmentDir(cardId: string, boardId?: string): Promise<string | null>;
     /**
      * Lists all comments on a card.
      *

package/package.json

@@ -2,7 +2,7 @@
   "name": "kanban-lite",
   "displayName": "Kanban Lite",
   "description": "A kanban board for your codebase. Cards stored as markdown.",
-  "version": "1.0.32",
+  "version": "1.0.33",
   "publisher": "borgius",
   "license": "MIT",
   "repository": {

package/src/extension/KanbanPanel.ts

@@ -744,18 +744,31 @@ export class KanbanPanel {
   }
 
   private async _openAttachment(cardId: string, attachment: string): Promise<void> {
+    const sdk = this._getSDK()
     const card = this._cards.find(f => f.id === cardId)
     if (!card) return
 
-    const featureDir = path.dirname(card.filePath)
-    const attachmentPath = path.resolve(featureDir, attachment)
+    // Resolve attachment directory via SDK (handles both markdown and SQLite paths)
+    const attachmentDir = sdk
+      ? (await sdk.getAttachmentDir(cardId, this._currentBoardId) ?? path.join(path.dirname(card.filePath), 'attachments'))
+      : path.join(path.dirname(card.filePath), 'attachments')
+
+    const attachmentPath = path.resolve(attachmentDir, attachment)
+
+    const ext = path.extname(attachment).toLowerCase()
+    // Text-based files open as VS Code editor tabs; everything else opens externally
+    // (PDF, images, etc. open via the OS default viewer or browser)
+    const isTextFile = ['.json', '.txt', '.md', '.html', '.xml', '.csv', '.ts', '.js', '.py', '.yaml', '.yml', '.toml', '.log'].includes(ext)
 
     try {
       await vscode.workspace.fs.stat(vscode.Uri.file(attachmentPath))
-      const doc = await vscode.workspace.openTextDocument(vscode.Uri.file(attachmentPath))
-      await vscode.window.showTextDocument(doc, { viewColumn: vscode.ViewColumn.Beside })
+      if (isTextFile) {
+        const doc = await vscode.workspace.openTextDocument(vscode.Uri.file(attachmentPath))
+        await vscode.window.showTextDocument(doc, { viewColumn: vscode.ViewColumn.Beside })
+      } else {
+        await vscode.env.openExternal(vscode.Uri.file(attachmentPath))
+      }
     } catch {
-      // For binary files or files that can't be opened as text, reveal in OS
       await vscode.env.openExternal(vscode.Uri.file(attachmentPath))
     }
   }

package/src/sdk/KanbanSDK.ts

@@ -1155,6 +1155,28 @@ export class KanbanSDK {
     return card.attachments
   }
 
+  /**
+   * Returns the absolute path to the attachment directory for a card.
+   *
+   * For the markdown engine this is `{column_dir}/attachments/`.
+   * For the SQLite engine this is `.kanban/boards/{boardId}/attachments/{cardId}/`.
+   *
+   * @param cardId - The ID of the card.
+   * @param boardId - Optional board ID. Defaults to the workspace's default board.
+   * @returns A promise resolving to the absolute directory path, or `null` if the card is not found.
+   *
+   * @example
+   * ```ts
+   * const dir = await sdk.getAttachmentDir('42')
+   * // '/workspace/.kanban/boards/default/backlog/attachments'
+   * ```
+   */
+  async getAttachmentDir(cardId: string, boardId?: string): Promise<string | null> {
+    const card = await this.getCard(cardId, boardId)
+    if (!card) return null
+    return this._storage.getCardDir(card)
+  }
+
   // --- Comment management ---
 
   /**

package/src/sdk/__tests__/KanbanSDK.test.ts

@@ -470,8 +470,8 @@ describe('KanbanSDK', () => {
       const updated = await sdk.addAttachment('card', srcFile)
       expect(updated.attachments).toContain('test-attach.txt')
 
-      // Verify file was copied to the status subfolder
-      const destPath = path.join(tempDir, 'boards', 'default', 'backlog', 'test-attach.txt')
+      // Verify file was copied to the attachments subfolder inside the status dir
+      const destPath = path.join(tempDir, 'boards', 'default', 'backlog', 'attachments', 'test-attach.txt')
       expect(fs.existsSync(destPath)).toBe(true)
 
       fs.unlinkSync(srcFile)

package/src/sdk/__tests__/storage-markdown.test.ts

@@ -167,7 +167,7 @@ describe('MarkdownStorageEngine', () => {
       const card = makeCard({
         filePath: path.join(boardDir, 'backlog', '1-test-card.md'),
       })
-      expect(engine.getCardDir(card)).toBe(path.join(boardDir, 'backlog'))
+      expect(engine.getCardDir(card)).toBe(path.join(boardDir, 'backlog', 'attachments'))
     })
   })
 })

package/src/sdk/fileUtils.ts

@@ -72,11 +72,14 @@ export async function moveCardFile(
 
   if (attachments && attachments.length > 0) {
     const sourceDir = path.dirname(currentPath)
+    const sourceAttachmentsDir = path.join(sourceDir, 'attachments')
+    const destAttachmentsDir = path.join(targetDir, 'attachments')
     for (const attachment of attachments) {
-      const srcAttach = path.join(sourceDir, attachment)
-      const destAttach = path.join(targetDir, attachment)
+      const srcAttach = path.join(sourceAttachmentsDir, attachment)
+      const destAttach = path.join(destAttachmentsDir, attachment)
       try {
         await fs.access(srcAttach)
+        await fs.mkdir(destAttachmentsDir, { recursive: true })
         await fs.rename(srcAttach, destAttach)
       } catch {
         // Best effort -- skip failed attachment moves

package/src/sdk/storage/markdown.ts

@@ -161,16 +161,17 @@ export class MarkdownStorageEngine implements StorageEngine {
   // --- Attachments ---
 
   getCardDir(card: Card): string {
-    return path.dirname(card.filePath)
+    return path.join(path.dirname(card.filePath), 'attachments')
   }
 
   async copyAttachment(sourcePath: string, card: Card): Promise<void> {
     const filename = path.basename(sourcePath)
-    const cardDir = this.getCardDir(card)
-    const destPath = path.join(cardDir, filename)
-    const sourceDir = path.dirname(path.resolve(sourcePath))
-    if (sourceDir !== cardDir) {
-      await fs.copyFile(path.resolve(sourcePath), destPath)
+    const attachmentDir = this.getCardDir(card)
+    await fs.mkdir(attachmentDir, { recursive: true })
+    const destPath = path.join(attachmentDir, filename)
+    const resolvedSource = path.resolve(sourcePath)
+    if (path.dirname(resolvedSource) !== attachmentDir) {
+      await fs.copyFile(resolvedSource, destPath)
     }
   }
 

package/src/standalone/server.ts

@@ -28,8 +28,16 @@ const MIME_TYPES: Record<string, string> = {
   '.css': 'text/css',
   '.json': 'application/json',
   '.png': 'image/png',
+  '.jpg': 'image/jpeg',
+  '.jpeg': 'image/jpeg',
+  '.gif': 'image/gif',
+  '.webp': 'image/webp',
   '.svg': 'image/svg+xml',
   '.ico': 'image/x-icon',
+  '.pdf': 'application/pdf',
+  '.txt': 'text/plain',
+  '.xml': 'text/xml',
+  '.csv': 'text/csv',
   '.map': 'application/json'
 }
 
@@ -334,8 +342,9 @@ export function startServer(kanbanDir: string, port: number, webviewDir?: string
     const card = cards.find(f => f.id === cardId)
     if (!card) return false
 
-    // Write file data to the card's directory
-    const cardDir = path.dirname(card.filePath)
+    // Write file data to the card's attachment directory
+    const cardDir = sdk.storageEngine.getCardDir(card)
+    fs.mkdirSync(cardDir, { recursive: true })
     fs.writeFileSync(path.join(cardDir, filename), fileData)
 
     // Register attachment via SDK (skips copy since file is already in place)
@@ -1100,7 +1109,7 @@ export function startServer(kanbanDir: string, port: number, webviewDir?: string
       const { id, filename: attachName } = params
       const card = cards.find(f => f.id === id)
       if (!card) return jsonError(res, 404, 'Task not found')
-      const cardDir = path.dirname(card.filePath)
+      const cardDir = sdk.storageEngine.getCardDir(card)
       const attachmentPath = path.resolve(cardDir, attachName)
       if (!attachmentPath.startsWith(absoluteKanbanDir)) {
         res.writeHead(403, { 'Content-Type': 'text/plain' })
@@ -1426,7 +1435,7 @@ export function startServer(kanbanDir: string, port: number, webviewDir?: string
       }
       const card = cards.find(f => f.id === cardId)
       if (!card) { res.writeHead(404, { 'Content-Type': 'text/plain' }); res.end('Card not found'); return }
-      const cardDir = path.dirname(card.filePath)
+      const cardDir = sdk.storageEngine.getCardDir(card)
       const attachmentPath = path.resolve(cardDir, filename)
       if (!attachmentPath.startsWith(absoluteKanbanDir)) {
         res.writeHead(403, { 'Content-Type': 'text/plain' }); res.end('Forbidden'); return