Merge branch 'lessweb:main' into main

Author: hqwlkj

.lintstagedrc

@@ -1,5 +1,5 @@
 {
-  "*.{ts,tsx,js,mjs,cjs,ejs,jsx}": [
+  "*.{ts,tsx,js,mjs,cjs,jsx}": [
     "eslint --fix",
     "prettier --write"
   ],

AGENTS.md

@@ -0,0 +1,35 @@
+# Repository Guidelines
+
+## Project Structure & Module Organization
+
+- `src/` contains the TypeScript CLI implementation, with tool handlers in `src/tools/`, MCP integration in `src/mcp/`, UI components in `src/ui/`, and shared helpers in `src/common/`.
+- `src/tests/` contains Node test files named `*.test.ts`.
+- `templates/` contains runtime prompt assets: `templates/prompts/` for EJS prompt templates and `templates/tools/` for tool instruction Markdown loaded into the system prompt.
+- `docs/` is reserved for user-facing documentation such as configuration and MCP guides.
+- `resources/` stores static images used by the documentation or UI.
+
+## Build, Test, and Development Commands
+
+- `npm test` runs all test files with `tsx --test`.
+- `npm run test:single -- src/tests/<name>.test.ts` runs one test file.
+- `npm run typecheck` verifies TypeScript types without emitting files.
+- `npm run lint` checks ESLint rules for `src/`.
+- `npm run build` runs checks, bundles `src/cli.tsx` to `dist/cli.js`, and marks the bundle executable.
+
+## Coding Style & Naming Conventions
+
+- Use TypeScript ES modules and keep imports explicit.
+- Prefer small, focused functions; keep filesystem path construction centralized when a path is reused.
+- Use two-space indentation and Prettier-compatible formatting.
+- Respond in standard technical English. Avoid nonstandard phrasing and corporate jargon.
+
+## Testing Guidelines
+
+- Add or update tests in `src/tests/` when changing command behavior, prompt rendering, session flow, tools, or settings.
+- Prefer Node's built-in `node:test` and `node:assert/strict` APIs, matching the existing tests.
+- Keep tests deterministic by using temporary directories and mocked network calls where needed.
+
+## Commit & Pull Request Guidelines
+
+- Keep commits focused on a single change and use concise, imperative commit messages.
+- In pull requests, describe the behavior change, list verification commands, and note any packaging or template path changes.

package.json

@@ -15,8 +15,8 @@
   "main": "./dist/cli.js",
   "files": [
     "dist/cli.js",
-    "docs/tools/**",
-    "docs/prompts/**",
+    "templates/tools/**",
+    "templates/prompts/**",
     "README.md",
     "LICENSE"
   ],

src/prompt.ts

@@ -252,7 +252,7 @@ type PromptToolOptions = {
 };
 
 function readToolDocs(extensionRoot: string, _options: PromptToolOptions = {}): string {
-  const toolsDir = path.join(extensionRoot, "docs", "tools");
+  const toolsDir = path.join(extensionRoot, "templates", "tools");
   if (!fs.existsSync(toolsDir)) {
     return "";
   }

src/session.ts

@@ -1485,7 +1485,7 @@ ${skillMd}
   }
 
   private renderInitCommandPrompt(): string {
-    const templatePath = path.join(getExtensionRoot(), "docs", "prompts", "init_command.md.ejs");
+    const templatePath = path.join(getExtensionRoot(), "templates", "prompts", "init_command.md.ejs");
     const template = fs.readFileSync(templatePath, "utf8");
     return ejs.render(template, {
       agentsMdFile: this.getEffectiveProjectAgentsMdFile(),
@@ -1747,8 +1747,9 @@ ${skillMd}
       }
       const params = Array.isArray(message.contentParams) ? message.contentParams : [message.contentParams];
       for (const param of params) {
-        if (param && typeof param === "object") {
-          contentParts.push(param as ChatCompletionContentPart);
+        const part = param as ChatCompletionContentPart;
+        if (part && part.type !== "image_url") {
+          contentParts.push(part);
         }
       }
       const contentValue: string | ChatCompletionContentPart[] = contentParts.length > 0 ? contentParts : content;

src/tests/prompt.test.ts

@@ -1,7 +1,12 @@
 import { test } from "node:test";
 import assert from "node:assert/strict";
+import * as fs from "fs";
+import * as path from "path";
+import { fileURLToPath } from "url";
 import { getSystemPrompt, getTools } from "../prompt";
 
+const repoRoot = path.resolve(path.dirname(fileURLToPath(import.meta.url)), "../..");
+
 test("getTools always includes WebSearch", () => {
   const names = getTools().map((tool) => tool.function.name);
   assert.equal(names.includes("WebSearch"), true);
@@ -11,3 +16,10 @@ test("getSystemPrompt always includes WebSearch docs", () => {
   const prompt = getSystemPrompt("/tmp/project");
   assert.equal(prompt.includes("## WebSearch"), true);
 });
+
+test("runtime prompt assets live under templates", () => {
+  assert.equal(fs.existsSync(path.join(repoRoot, "templates", "tools", "web-search.md")), true);
+  assert.equal(fs.existsSync(path.join(repoRoot, "templates", "prompts", "init_command.md.ejs")), true);
+  assert.equal(fs.existsSync(path.join(repoRoot, "docs", "tools")), false);
+  assert.equal(fs.existsSync(path.join(repoRoot, "docs", "prompts")), false);
+});