Feature/copilot memory plugin plan

[jwayong]

Description

Related Issue

Type of Change

• Bug fix (non-breaking change that fixes an issue)
• New feature (non-breaking change that adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Documentation update
• Refactoring (no functional changes)
• Performance improvement
• Test update

Changes Made

Testing

• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes
• I have tested this on the following platforms:
  • Linux
  • macOS
  • Windows

Checklist

• My code follows the project's coding style
• I have performed a self-review of my code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• Any dependent changes have been merged and published

Screenshots (if applicable)

Additional Notes

[github-actions]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 3 🔵🔵🔵⚪⚪

🏅 Score: 85

🧪 PR contains tests

🔒 No security concerns identified

✅ No TODO sections

🔀 No multiple PR themes

⚡ No major issues detected

[github-actions]

PR Code Suggestions ✨

Explore these optional code suggestions:

Category	Suggestion	Impact
Possible issue	Replace blocking sync file read with async read in async function Replace blocking sync file read with async file read in the async runDebugCapture function to avoid blocking the event loop. Update the readFile dependency to return a promise and use fs/promises.readFile as the default implementation. examples/copilot/packages/shared/src/debug/capture-debugger.ts [21-239] -import { readFileSync } from "node:fs"; +import { readFile } from "node:fs/promises"; // ... export interface DebugCaptureDeps { cfg: PluginConfig; - /** Inject for tests; defaults to fs.readFileSync at runtime. */ - readFile?: (path: string) => string; + /** Inject for tests; defaults to fs.promises.readFile at runtime. */ + readFile?: (path: string) => Promise<string>; /** Buffered output sink. Defaults to a string accumulator. */ write?: (chunk: string) => void; } // ... try { const reader = deps.readFile ?? defaultReadFile; - body = reader(args.path); + body = await reader(args.path); } catch (err) { writeLn(` ERROR: ${err instanceof Error ? err.message : String(err)}`); return { exitCode: 2, output: lines.join("") }; } // ... -function defaultReadFile(path: string): string { - return readFileSync(path, "utf-8"); +async function defaultReadFile(path: string): Promise<string> { + return readFile(path, "utf-8"); } Suggestion importance[1-10]: 5 __ Why: Replacing the blocking synchronous file read with an async read in the async runDebugCapture function avoids blocking the event loop, improving the function's behavior in async contexts. The suggestion correctly updates the readFile dependency interface to return a promise, uses fs/promises.readFile as the default implementation, and adjusts the call site to await the result.	Low