@forloop-cc/forloop-opencode-plugin-planner 2.2.4 → 2.2.6

package/README.md

@@ -7,6 +7,8 @@
 
 **AI-powered sprint planning, story management, and task automation for opencode**
 
+[English](README.md) · [中文](README_zh.md) · [日本語](README_ja.md)
+
 [Quick Install](#quick-install) · [How It Works](#how-it-works) · [Capabilities](#capabilities) · [FAQ](#faq)
 
 </div>

package/package.json

@@ -1,7 +1,7 @@
 {
   "$schema": "https://json.schemastore.org/package.json",
   "name": "@forloop-cc/forloop-opencode-plugin-planner",
-  "version": "2.2.4",
+  "version": "2.2.6",
   "description": "ForLoop integration plugin for opencode — Sprint, story, and agent tools for the ForLoop platform",
   "type": "module",
   "license": "MIT",
@@ -24,6 +24,10 @@
     ".": {
       "import": "./plugins/forloop-plugin.ts",
       "types": "./plugins/forloop-plugin.ts"
+    },
+    "./exports": {
+      "import": "./plugins/exports.ts",
+      "types": "./plugins/exports.ts"
     }
   },
   "files": [

package/plugins/exports.ts

@@ -0,0 +1,14 @@
+// Re-exports for @forloop-cc/forloop-cli consumption.
+// These were moved out of forloop-plugin.ts because Bun's TypeScript
+// transpiler in opencode-ai@1.15.6 fails when a file has both
+// `export default async` and additional `export { }` / `export type { }` lines.
+export { ForLoopAPIClient } from './capabilities/api-client';
+export type { APIError, CreateStoryRequest, CreateSprintRequest, ListSprintsParams, SprintOptions } from './capabilities/api-client';
+export { getConfig, isLambdaExecution, PROD_API_URL, DEV_API_URL } from './capabilities/config';
+export type { ForLoopConfig } from './capabilities/config';
+export { getToken, setToken, clearToken } from './capabilities/token-storage';
+export type { TokenProfile } from './capabilities/token-storage';
+export { validateToken, detectTokenType } from './capabilities/auth';
+export type { TokenValidationResult } from './capabilities/auth';
+export { resolveSprintId, getCurrentBranch, resolveSprintFromBranch, getForloopRoot, getManifestPath } from './capabilities/context-resolver';
+export type { ContextResolution } from './capabilities/context-resolver';

package/plugins/forloop-plugin.ts

@@ -35,6 +35,7 @@ import {
   createFileListTool,
   createFileDeleteTool,
   createFileDownloadUrlTool,
+  createFileDownloadTool,
 } from './tools/file-tools';
 import {
   createScheduleTool,
@@ -123,7 +124,8 @@ export default async (ctx: any) => {
       'forloopFileUpload': createFileUploadTool(client),
       'forloopFileList': createFileListTool(client),
       'forloopFileDelete': createFileDeleteTool(client),
-      'forloopFileDownload': createFileDownloadUrlTool(client),
+      'forloopFileDownloadUrl': createFileDownloadUrlTool(client),
+      'forloopFileDownload': createFileDownloadTool(client),
       
       // Schedule Meetings
       'forloopScheduleCreate': createScheduleTool(client),
@@ -174,13 +176,3 @@ export default async (ctx: any) => {
   };
 };
 
-export { ForLoopAPIClient } from './capabilities/api-client';
-export type { APIError, CreateStoryRequest, CreateSprintRequest, ListSprintsParams, SprintOptions } from './capabilities/api-client';
-export { getConfig, isLambdaExecution, PROD_API_URL, DEV_API_URL } from './capabilities/config';
-export type { ForLoopConfig } from './capabilities/config';
-export { getToken, setToken, clearToken } from './capabilities/token-storage';
-export type { TokenProfile } from './capabilities/token-storage';
-export { validateToken, detectTokenType } from './capabilities/auth';
-export type { TokenValidationResult } from './capabilities/auth';
-export { resolveSprintId, getCurrentBranch, resolveSprintFromBranch, getForloopRoot, getManifestPath } from './capabilities/context-resolver';
-export type { ContextResolution } from './capabilities/context-resolver';

package/plugins/tools/file-tools.ts

@@ -176,10 +176,10 @@ export function createFileDeleteTool(client: ForLoopAPIClient) {
 
 export function createFileDownloadUrlTool(client: ForLoopAPIClient) {
   return tool({
-    description: 'Get a download URL for a file',
+    description: 'Get a presigned download URL for a file (does not download the file itself)',
     args: {
       fileId: tool.schema.number()
-        .describe('File ID to download'),
+        .describe('File ID to get download URL for'),
     },
     async execute(args, _context) {
       const tokenResult = await validateToken();
@@ -189,10 +189,9 @@ export function createFileDownloadUrlTool(client: ForLoopAPIClient) {
       client.setToken?.(tokenResult.token);
 
       try {
-        // Log file read (tracks access)
-        await client.logFileRead(args.fileId);
+        // Fire-and-forget audit log — don't block download on audit failure
+        client.logFileRead(args.fileId).catch(() => {});
 
-        // Get download URL
         const download = await client.getFileDownloadUrl(args.fileId);
 
         return [
@@ -209,3 +208,59 @@ export function createFileDownloadUrlTool(client: ForLoopAPIClient) {
     },
   });
 }
+
+export function createFileDownloadTool(client: ForLoopAPIClient) {
+  return tool({
+    description: 'Download a sprint file by ID to the local sandbox',
+    args: {
+      fileId: tool.schema.number()
+        .describe('File ID to download (from forloopFileList)'),
+      destPath: tool.schema.string()
+        .optional()
+        .describe('Destination path relative to sandbox downloads dir (default: <filename>)'),
+    },
+    async execute(args, _context) {
+      const tokenResult = await validateToken();
+      if (!tokenResult.valid) {
+        return tokenResult.error ?? '❌ No API token configured.';
+      }
+      client.setToken?.(tokenResult.token);
+
+      try {
+        const open = await client.getFileOpenUrl(Number(args.fileId));
+        const url = String(open?.openUrl || open?.url || '');
+        if (!url) {
+          return `❌ Could not get download URL for file #${args.fileId}.`;
+        }
+
+        const response = await fetch(url);
+        if (!response.ok) {
+          return `❌ Download failed: HTTP ${response.status}`;
+        }
+
+        const buf = Buffer.from(await response.arrayBuffer());
+        const fileName = open?.originalName || open?.filename || `file-${args.fileId}`;
+        const downloadDir = path.join(process.env.HOME || '/tmp', 'downloads');
+        if (!fs.existsSync(downloadDir)) {
+          fs.mkdirSync(downloadDir, { recursive: true });
+        }
+
+        const destFile = args.destPath
+          ? path.join(downloadDir, args.destPath)
+          : path.join(downloadDir, fileName);
+        const destDir = path.dirname(destFile);
+        if (!fs.existsSync(destDir)) {
+          fs.mkdirSync(destDir, { recursive: true });
+        }
+        fs.writeFileSync(destFile, buf);
+
+        // Log file read (fire-and-forget audit)
+        client.logFileRead(args.fileId).catch(() => {});
+
+        return `✅ Downloaded file #${args.fileId} → ${destFile} (${buf.length} bytes)`;
+      } catch (error: any) {
+        return `❌ File download failed: ${error.message}`;
+      }
+    },
+  });
+}