@dyyz1993/create-agent 2.1.0 → 2.1.2

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@dyyz1993/create-agent",
-	"version": "2.1.0",
+	"version": "2.1.2",
 	"description": "Create Agent project scaffolding CLI",
 	"repository": {
 		"type": "git",
@@ -14,9 +14,10 @@
 		"templates/"
 	],
 	"scripts": {
-		"sync-templates": "rm -rf templates/general templates/chat templates/agent && rsync -a --exclude='node_modules' --exclude='dist' --exclude='build' --exclude='.server-port' --exclude='logs' --exclude='bun.lock' ../../templates/general/ templates/general/ && rsync -a --exclude='node_modules' --exclude='dist' --exclude='build' --exclude='.server-port' --exclude='logs' --exclude='bun.lock' ../../templates/chat/ templates/chat/ && rsync -a --exclude='node_modules' --exclude='dist' --exclude='build' --exclude='.server-port' --exclude='logs' --exclude='bun.lock' ../../templates/agent/ templates/agent/ && rsync -a ../../packages/eslint-plugin-rpc/ templates/agent/eslint-plugin-rpc/ && rsync -a ../../packages/eslint-plugin-rpc/ templates/chat/eslint-plugin-rpc/ && rsync -a ../../packages/eslint-plugin-rpc/ templates/general/eslint-plugin-rpc/ && node scripts/post-sync-templates.mjs",
-		"test": "bun test src/__tests__/",
-		"prepublishOnly": "test -d templates/general/src && test -d templates/chat/src && test -d templates/agent/src"
+		"sync-templates": "node scripts/sync-templates.mjs",
+		"test": "bun test src/__tests__/cli.test.ts src/__tests__/copy.test.ts src/__tests__/templates.test.ts src/__tests__/create-args.test.ts src/__tests__/update.test.ts",
+		"verify-templates": "node scripts/verify-templates.mjs",
+		"prepublishOnly": "node scripts/verify-templates.mjs && bun test src/__tests__/cli.test.ts src/__tests__/copy.test.ts src/__tests__/templates.test.ts src/__tests__/create-args.test.ts src/__tests__/update.test.ts"
 	},
 	"engines": {
 		"bun": ">=1.0.0"

package/src/__tests__/cli.test.ts

@@ -64,6 +64,8 @@ describe('CLI argument parsing (list)', () => {
     expect(output).toContain('general');
     expect(output).toContain('chat');
     expect(output).toContain('agent');
+    expect(output).toContain('browser-agent');
+    expect(output).toContain('cowork');
     expect(output).toContain('Available templates');
 
     mockLog.mockRestore();

package/src/__tests__/copy.test.ts

@@ -1,168 +1,156 @@
-import { describe, test, expect, afterAll } from 'bun:test';
-import { mkdtempSync, mkdirSync, writeFileSync, existsSync, readFileSync, rmSync } from 'fs';
-import { join } from 'path';
-import { tmpdir } from 'os';
-import { copyAndReplace, deriveNames } from '../lib/copy';
+import { describe, test, expect, afterAll } from "bun:test";
+import { mkdtempSync, mkdirSync, writeFileSync, existsSync, readFileSync, rmSync } from "fs";
+import { join } from "path";
+import { tmpdir } from "os";
+import { copyAndReplace, deriveNames } from "../lib/copy";
 
 const TMP_DIRS: string[] = [];
 
 afterAll(() => {
-  for (const d of TMP_DIRS) {
-    if (existsSync(d)) rmSync(d, { recursive: true, force: true });
-  }
+	for (const d of TMP_DIRS) {
+		if (existsSync(d)) rmSync(d, { recursive: true, force: true });
+	}
 });
 
-function makeTmp(prefix = 'pi-cli-test-'): string {
-  const d = mkdtempSync(join(tmpdir(), prefix));
-  TMP_DIRS.push(d);
-  return d;
+function makeTmp(prefix = "pi-cli-test-"): string {
+	const d = mkdtempSync(join(tmpdir(), prefix));
+	TMP_DIRS.push(d);
+	return d;
 }
 
-describe('deriveNames', () => {
-  test('simple name', () => {
-    const names = deriveNames('my-app');
-    expect(names.pascalName).toBe('MyApp');
-    expect(names.identifier).toBe('com.myapp.app');
-    expect(names.shortId).toBe('com.myapp');
-  });
-
-  test('single word', () => {
-    const names = deriveNames('hello');
-    expect(names.pascalName).toBe('Hello');
-    expect(names.identifier).toBe('com.hello.app');
-    expect(names.shortId).toBe('com.hello');
-  });
-
-  test('multi-segment name', () => {
-    const names = deriveNames('my-cool-project');
-    expect(names.pascalName).toBe('MyCoolProject');
-    expect(names.identifier).toBe('com.mycoolproject.app');
-    expect(names.shortId).toBe('com.mycoolproject');
-  });
+describe("deriveNames", () => {
+	test("simple name", () => {
+		const names = deriveNames("my-app");
+		expect(names.pascalName).toBe("MyApp");
+		expect(names.identifier).toBe("com.myapp.app");
+		expect(names.shortId).toBe("com.myapp");
+	});
+
+	test("single word", () => {
+		const names = deriveNames("hello");
+		expect(names.pascalName).toBe("Hello");
+		expect(names.identifier).toBe("com.hello.app");
+		expect(names.shortId).toBe("com.hello");
+	});
+
+	test("multi-segment name", () => {
+		const names = deriveNames("my-cool-project");
+		expect(names.pascalName).toBe("MyCoolProject");
+		expect(names.identifier).toBe("com.mycoolproject.app");
+		expect(names.shortId).toBe("com.mycoolproject");
+	});
 });
 
-describe('copyAndReplace', () => {
-  test('copies files from src to dest', () => {
-    const srcDir = makeTmp('pi-src-');
-    const destDir = makeTmp('pi-dest-');
+describe("copyAndReplace", () => {
+	test("copies files from src to dest", () => {
+		const srcDir = makeTmp("pi-src-");
+		const destDir = makeTmp("pi-dest-");
 
-    writeFileSync(join(srcDir, 'hello.txt'), 'hello world');
+		writeFileSync(join(srcDir, "hello.txt"), "hello world");
 
-    copyAndReplace(srcDir, destDir, 'test-project');
+		copyAndReplace(srcDir, destDir, "test-project");
 
-    expect(existsSync(join(destDir, 'hello.txt'))).toBe(true);
-    expect(readFileSync(join(destDir, 'hello.txt'), 'utf-8')).toBe('hello world');
-  });
+		expect(existsSync(join(destDir, "hello.txt"))).toBe(true);
+		expect(readFileSync(join(destDir, "hello.txt"), "utf-8")).toBe("hello world");
+	});
 
-  test('replaces pi-agent-template with project name', () => {
-    const srcDir = makeTmp('pi-src-');
-    const destDir = makeTmp('pi-dest-');
+	test("replaces pi-agent-template with project name", () => {
+		const srcDir = makeTmp("pi-src-");
+		const destDir = makeTmp("pi-dest-");
 
-    writeFileSync(join(srcDir, 'config.json'), '{"name": "pi-agent-template"}');
+		writeFileSync(join(srcDir, "config.json"), '{"name": "pi-agent-template"}');
 
-    copyAndReplace(srcDir, destDir, 'my-app');
+		copyAndReplace(srcDir, destDir, "my-app");
 
-    const content = readFileSync(join(destDir, 'config.json'), 'utf-8');
-    expect(content).toBe('{"name": "my-app"}');
-  });
+		const content = readFileSync(join(destDir, "config.json"), "utf-8");
+		expect(content).toBe('{"name": "my-app"}');
+	});
 
-  test('replaces Pi Agent Template with pascal name', () => {
-    const srcDir = makeTmp('pi-src-');
-    const destDir = makeTmp('pi-dest-');
+	test("replaces Pi Agent Template with pascal name", () => {
+		const srcDir = makeTmp("pi-src-");
+		const destDir = makeTmp("pi-dest-");
 
-    writeFileSync(join(srcDir, 'readme.md'), '# Pi Agent Template');
+		writeFileSync(join(srcDir, "readme.md"), "# Pi Agent Template");
 
-    copyAndReplace(srcDir, destDir, 'my-app');
+		copyAndReplace(srcDir, destDir, "my-app");
 
-    const content = readFileSync(join(destDir, 'readme.md'), 'utf-8');
-    expect(content).toBe('# MyApp');
-  });
+		const content = readFileSync(join(destDir, "readme.md"), "utf-8");
+		expect(content).toBe("# MyApp");
+	});
 
-  test('replaces Pi Agent with pascal name', () => {
-    const srcDir = makeTmp('pi-src-');
-    const destDir = makeTmp('pi-dest-');
+	test("replaces Pi Agent with pascal name", () => {
+		const srcDir = makeTmp("pi-src-");
+		const destDir = makeTmp("pi-dest-");
 
-    writeFileSync(join(srcDir, 'app.ts'), 'export const Pi Agent = true');
+		writeFileSync(join(srcDir, "app.ts"), "export const Pi Agent = true");
 
-    copyAndReplace(srcDir, destDir, 'cool-project');
+		copyAndReplace(srcDir, destDir, "cool-project");
 
-    const content = readFileSync(join(destDir, 'app.ts'), 'utf-8');
-    expect(content).toBe('export const CoolProject = true');
-  });
+		const content = readFileSync(join(destDir, "app.ts"), "utf-8");
+		expect(content).toBe("export const CoolProject = true");
+	});
 
-  test('replaces com.piagent.template with identifier', () => {
-    const srcDir = makeTmp('pi-src-');
-    const destDir = makeTmp('pi-dest-');
+	test("replaces com.piagent.template with identifier", () => {
+		const srcDir = makeTmp("pi-src-");
+		const destDir = makeTmp("pi-dest-");
 
-    writeFileSync(join(srcDir, 'plist.xml'), '<string>com.piagent.template</string>');
+		writeFileSync(join(srcDir, "plist.xml"), "<string>com.piagent.template</string>");
 
-    copyAndReplace(srcDir, destDir, 'my-app');
+		copyAndReplace(srcDir, destDir, "my-app");
 
-    const content = readFileSync(join(destDir, 'plist.xml'), 'utf-8');
-    expect(content).toBe('<string>com.myapp.app</string>');
-  });
+		const content = readFileSync(join(destDir, "plist.xml"), "utf-8");
+		expect(content).toBe("<string>com.myapp.app</string>");
+	});
 
-  test('replaces com.piagent with shortId', () => {
-    const srcDir = makeTmp('pi-src-');
-    const destDir = makeTmp('pi-dest-');
+	test("replaces com.piagent with shortId", () => {
+		const srcDir = makeTmp("pi-src-");
+		const destDir = makeTmp("pi-dest-");
 
-    writeFileSync(join(srcDir, 'config.xml'), '<id>com.piagent.something</id>');
+		writeFileSync(join(srcDir, "config.xml"), "<id>com.piagent.something</id>");
 
-    copyAndReplace(srcDir, destDir, 'my-app');
+		copyAndReplace(srcDir, destDir, "my-app");
 
-    const content = readFileSync(join(destDir, 'config.xml'), 'utf-8');
-    expect(content).toBe('<id>com.myapp.something</id>');
-  });
+		const content = readFileSync(join(destDir, "config.xml"), "utf-8");
+		expect(content).toBe("<id>com.myapp.something</id>");
+	});
 
-  test('replaces @pi-agent/ scope', () => {
-    const srcDir = makeTmp('pi-src-');
-    const destDir = makeTmp('pi-dest-');
+	test("skips node_modules directory", () => {
+		const srcDir = makeTmp("pi-src-");
+		const destDir = makeTmp("pi-dest-");
 
-    writeFileSync(join(srcDir, 'pkg.json'), '"@pi-agent/rpc-core"');
+		mkdirSync(join(srcDir, "node_modules", "pkg"), { recursive: true });
+		writeFileSync(join(srcDir, "node_modules", "pkg", "index.js"), "module.exports = 1;");
+		writeFileSync(join(srcDir, "root.txt"), "root");
 
-    copyAndReplace(srcDir, destDir, 'my-app');
+		copyAndReplace(srcDir, destDir, "test");
 
-    const content = readFileSync(join(destDir, 'pkg.json'), 'utf-8');
-    expect(content).toBe('"@my-app/rpc-core"');
-  });
+		expect(existsSync(join(destDir, "root.txt"))).toBe(true);
+		expect(existsSync(join(destDir, "node_modules"))).toBe(false);
+	});
 
-  test('skips node_modules directory', () => {
-    const srcDir = makeTmp('pi-src-');
-    const destDir = makeTmp('pi-dest-');
+	test("skips bun.lock file", () => {
+		const srcDir = makeTmp("pi-src-");
+		const destDir = makeTmp("pi-dest-");
 
-    mkdirSync(join(srcDir, 'node_modules', 'pkg'), { recursive: true });
-    writeFileSync(join(srcDir, 'node_modules', 'pkg', 'index.js'), 'module.exports = 1;');
-    writeFileSync(join(srcDir, 'root.txt'), 'root');
+		writeFileSync(join(srcDir, "bun.lock"), "lockfile content");
+		writeFileSync(join(srcDir, "package.json"), "{}");
 
-    copyAndReplace(srcDir, destDir, 'test');
+		copyAndReplace(srcDir, destDir, "test");
 
-    expect(existsSync(join(destDir, 'root.txt'))).toBe(true);
-    expect(existsSync(join(destDir, 'node_modules'))).toBe(false);
-  });
+		expect(existsSync(join(destDir, "package.json"))).toBe(true);
+		expect(existsSync(join(destDir, "bun.lock"))).toBe(false);
+	});
 
-  test('skips bun.lock file', () => {
-    const srcDir = makeTmp('pi-src-');
-    const destDir = makeTmp('pi-dest-');
+	test("copies nested directories recursively", () => {
+		const srcDir = makeTmp("pi-src-");
+		const destDir = makeTmp("pi-dest-");
 
-    writeFileSync(join(srcDir, 'bun.lock'), 'lockfile content');
-    writeFileSync(join(srcDir, 'package.json'), '{}');
+		mkdirSync(join(srcDir, "src", "lib"), { recursive: true });
+		writeFileSync(join(srcDir, "src", "lib", "util.ts"), "// pi-agent-template util");
 
-    copyAndReplace(srcDir, destDir, 'test');
+		copyAndReplace(srcDir, destDir, "my-proj");
 
-    expect(existsSync(join(destDir, 'package.json'))).toBe(true);
-    expect(existsSync(join(destDir, 'bun.lock'))).toBe(false);
-  });
-
-  test('copies nested directories recursively', () => {
-    const srcDir = makeTmp('pi-src-');
-    const destDir = makeTmp('pi-dest-');
-
-    mkdirSync(join(srcDir, 'src', 'lib'), { recursive: true });
-    writeFileSync(join(srcDir, 'src', 'lib', 'util.ts'), '// pi-agent-template util');
-
-    copyAndReplace(srcDir, destDir, 'my-proj');
-
-    const content = readFileSync(join(destDir, 'src', 'lib', 'util.ts'), 'utf-8');
-    expect(content).toBe('// my-proj util');
-  });
+		const content = readFileSync(join(destDir, "src", "lib", "util.ts"), "utf-8");
+		expect(content).toBe("// my-proj util");
+	});
 });

package/src/__tests__/create-args.test.ts

@@ -0,0 +1,113 @@
+import { describe, test, expect } from "bun:test";
+import { mkdtempSync, writeFileSync, readFileSync, rmSync } from "fs";
+import { join } from "path";
+import { tmpdir } from "os";
+import { parseArgs } from "../commands/create";
+import { validateWorkspaceName } from "../commands/workspace";
+import { copyAndReplace, expandTilde } from "../lib/copy";
+import { homedir } from "os";
+
+describe("create parseArgs", () => {
+	test("parses --type and --dir with separate values", () => {
+		const result = parseArgs(["my-app", "--type", "chat", "--dir", "/tmp/x"]);
+		expect(result.error).toBeUndefined();
+		expect(result.positional).toEqual(["my-app"]);
+		expect(result.options.type).toBe("chat");
+		expect(result.options.dir).toBe("/tmp/x");
+	});
+
+	test("supports --key=value syntax", () => {
+		const result = parseArgs(["my-app", "--type=chat"]);
+		expect(result.error).toBeUndefined();
+		expect(result.options.type).toBe("chat");
+	});
+
+	test("unknown flag returns an error instead of silently ignoring", () => {
+		const result = parseArgs(["my-app", "--typo", "chat"]);
+		expect(result.error).toContain("Unknown option: --typo");
+	});
+
+	test("missing --type value returns an error instead of falling back to general", () => {
+		const result = parseArgs(["my-app", "--type"]);
+		expect(result.error).toContain("--type requires a value");
+	});
+
+	test("--type followed by another flag returns an error", () => {
+		const result = parseArgs(["my-app", "--type", "--dir", "/tmp"]);
+		expect(result.error).toContain("--type requires a value");
+	});
+
+	test("-- separates remaining args as positional", () => {
+		const result = parseArgs(["my-app", "--", "not-a-flag"]);
+		expect(result.error).toBeUndefined();
+		expect(result.positional).toEqual(["my-app", "not-a-flag"]);
+	});
+});
+
+describe("expandTilde", () => {
+	test("~ expands to home dir", () => {
+		expect(expandTilde("~")).toBe(homedir());
+	});
+
+	test("~/x expands under home dir", () => {
+		expect(expandTilde("~/projects/app")).toBe(join(homedir(), "projects/app"));
+	});
+
+	test("other paths untouched", () => {
+		expect(expandTilde("/tmp/x")).toBe("/tmp/x");
+		expect(expandTilde("~bin/x")).toBe("~bin/x");
+	});
+});
+
+describe("validateWorkspaceName", () => {
+	test("accepts word chars, dots and dashes", () => {
+		expect(validateWorkspaceName("feature-chat-ui")).toBeNull();
+		expect(validateWorkspaceName("fix_1.2")).toBeNull();
+	});
+
+	test("rejects spaces, slashes, shell metacharacters and empty names", () => {
+		expect(validateWorkspaceName("my workspace")).not.toBeNull();
+		expect(validateWorkspaceName("a/b")).not.toBeNull();
+		expect(validateWorkspaceName("x`cmd`")).not.toBeNull();
+		expect(validateWorkspaceName("..")).not.toBeNull();
+		expect(validateWorkspaceName("")).not.toBeNull();
+	});
+});
+
+describe("copyAndReplace binary safety", () => {
+	test("binary files are copied byte-for-byte, not utf-8 re-encoded", () => {
+		const src = mkdtempSync(join(tmpdir(), "pi-bin-src-"));
+		const dest = mkdtempSync(join(tmpdir(), "pi-bin-dest-"));
+		try {
+			// PNG magic bytes + NUL bytes that utf-8 round-trip would corrupt
+			const binary = Buffer.from([
+				0x89, 0x50, 0x4e, 0x47, 0x0d, 0x0a, 0x1a, 0x0a, 0x00, 0xff, 0xfe, 0x01,
+			]);
+			writeFileSync(join(src, "icon.png"), binary);
+			writeFileSync(join(src, "data.bin"), Buffer.from([0x00, 0x01, 0x02, 0x03]));
+
+			copyAndReplace(src, dest, "my-app");
+
+			expect(readFileSync(join(dest, "icon.png")).equals(binary)).toBe(true);
+			expect(
+				readFileSync(join(dest, "data.bin")).equals(Buffer.from([0x00, 0x01, 0x02, 0x03]))
+			).toBe(true);
+		} finally {
+			rmSync(src, { recursive: true, force: true });
+			rmSync(dest, { recursive: true, force: true });
+		}
+	});
+
+	test("text files still get placeholder replacement", () => {
+		const src = mkdtempSync(join(tmpdir(), "pi-txt-src-"));
+		const dest = mkdtempSync(join(tmpdir(), "pi-txt-dest-"));
+		try {
+			writeFileSync(join(src, "notes.md"), "project pi-agent-template rocks");
+			copyAndReplace(src, dest, "my-app");
+			expect(readFileSync(join(dest, "notes.md"), "utf-8")).toBe("project my-app rocks");
+		} finally {
+			rmSync(src, { recursive: true, force: true });
+			rmSync(dest, { recursive: true, force: true });
+		}
+	});
+});

package/src/__tests__/templates.test.ts

@@ -2,17 +2,19 @@ import { describe, test, expect } from 'bun:test';
 import { getAllTemplates, getTemplateInfo, resolveTemplateDir } from '../lib/templates';
 
 describe('getAllTemplates', () => {
-  test('returns 3 templates', () => {
+  test('returns 5 templates', () => {
     const templates = getAllTemplates();
-    expect(templates).toHaveLength(3);
+    expect(templates).toHaveLength(5);
   });
 
-  test('includes general, chat, agent types', () => {
+  test('includes general, chat, agent, browser-agent, cowork types', () => {
     const templates = getAllTemplates();
     const types = templates.map((t) => t.type);
     expect(types).toContain('general');
     expect(types).toContain('chat');
     expect(types).toContain('agent');
+    expect(types).toContain('browser-agent');
+    expect(types).toContain('cowork');
   });
 
   test('all templates are available', () => {
@@ -53,6 +55,20 @@ describe('getTemplateInfo', () => {
     expect(info!.type).toBe('agent');
   });
 
+  test('returns info for "browser-agent"', () => {
+    const info = getTemplateInfo('browser-agent');
+    expect(info).toBeDefined();
+    expect(info!.type).toBe('browser-agent');
+    expect(info!.dir).toBe('templates/browser-agent');
+  });
+
+  test('returns info for "cowork"', () => {
+    const info = getTemplateInfo('cowork');
+    expect(info).toBeDefined();
+    expect(info!.type).toBe('cowork');
+    expect(info!.dir).toBe('templates/cowork');
+  });
+
   test('returns undefined for unknown type', () => {
     expect(getTemplateInfo('nonexistent')).toBeUndefined();
     expect(getTemplateInfo('')).toBeUndefined();
@@ -75,6 +91,16 @@ describe('resolveTemplateDir', () => {
     expect(dir).toBe('/fake/root/templates/agent');
   });
 
+  test('resolves browser-agent template path', () => {
+    const dir = resolveTemplateDir('/fake/root', 'browser-agent');
+    expect(dir).toBe('/fake/root/templates/browser-agent');
+  });
+
+  test('resolves cowork template path', () => {
+    const dir = resolveTemplateDir('/fake/root', 'cowork');
+    expect(dir).toBe('/fake/root/templates/cowork');
+  });
+
   test('throws for invalid template type', () => {
     expect(() => resolveTemplateDir('/root', 'invalid')).toThrow(
       'Unknown template type: "invalid"',

package/src/__tests__/update.test.ts

@@ -0,0 +1,129 @@
+import { describe, test, expect, afterAll } from "bun:test";
+import { mkdtempSync, mkdirSync, writeFileSync, readFileSync, rmSync, existsSync } from "fs";
+import { join } from "path";
+import { tmpdir } from "os";
+import { detectTemplateMeta } from "../commands/update";
+import { postProcessTemplate } from "../lib/copy";
+
+const TMP_DIRS: string[] = [];
+
+afterAll(() => {
+	for (const d of TMP_DIRS) {
+		if (existsSync(d)) rmSync(d, { recursive: true, force: true });
+	}
+});
+
+function makeTmp(prefix = "pi-update-test-"): string {
+	const d = mkdtempSync(join(tmpdir(), prefix));
+	TMP_DIRS.push(d);
+	return d;
+}
+
+function makeProject(name: string, modules: string[]): string {
+	const dir = makeTmp();
+	writeFileSync(join(dir, "package.json"), JSON.stringify({ name, version: "1.0.0" }, null, "\t"));
+	const modulesDir = join(dir, "src", "shared", "modules");
+	mkdirSync(modulesDir, { recursive: true });
+	for (const m of modules) {
+		mkdirSync(join(modulesDir, m), { recursive: true });
+	}
+	return dir;
+}
+
+describe("detectTemplateMeta", () => {
+	test("agent: bash + todo + rules", () => {
+		const dir = makeProject("my-agent", ["bash", "todo", "rules", "chat"]);
+		expect(detectTemplateMeta(dir)?.templateType).toBe("agent");
+	});
+
+	test("browser-agent: browser module", () => {
+		const dir = makeProject("my-browser", ["browser", "chat"]);
+		expect(detectTemplateMeta(dir)?.templateType).toBe("browser-agent");
+	});
+
+	test("cowork: task + context + output", () => {
+		const dir = makeProject("my-cowork", ["task", "context", "output"]);
+		expect(detectTemplateMeta(dir)?.templateType).toBe("cowork");
+	});
+
+	test("chat: chat without file/git", () => {
+		const dir = makeProject("my-chat", ["chat"]);
+		expect(detectTemplateMeta(dir)?.templateType).toBe("chat");
+	});
+
+	test("general: everything else", () => {
+		const dir = makeProject("my-general", ["chat", "file", "git"]);
+		expect(detectTemplateMeta(dir)?.templateType).toBe("general");
+	});
+
+	test("returns project name from package.json", () => {
+		const dir = makeProject("custom-name", ["chat"]);
+		expect(detectTemplateMeta(dir)?.projectName).toBe("custom-name");
+	});
+
+	test("null when no modules dir", () => {
+		const dir = makeTmp();
+		writeFileSync(join(dir, "package.json"), JSON.stringify({ name: "x" }));
+		expect(detectTemplateMeta(dir)).toBeNull();
+	});
+});
+
+describe("postProcessTemplate", () => {
+	test("resolves workspace deps to npm ranges and rewrites package name", () => {
+		const dir = makeTmp();
+		writeFileSync(
+			join(dir, "package.json"),
+			JSON.stringify(
+				{
+					name: "pi-general-app",
+					workspaces: ["packages/*"],
+					dependencies: { "@dyyz1993/rpc-core": "workspace:*" },
+					devDependencies: { "@dyyz1993/eslint-plugin-rpc": "workspace:*" },
+				},
+				null,
+				"\t"
+			)
+		);
+		writeFileSync(
+			join(dir, "eslint.config.mjs"),
+			[
+				"import rpcPlugin from '@dyyz1993/eslint-plugin-rpc';",
+				"export default [",
+				"  {",
+				"    plugins: { rpc: rpcPlugin },",
+				"    rules: {",
+				"      'rpc/no-bare-method': 'error',",
+				"    },",
+				"  },",
+				"];",
+				"",
+			].join("\n")
+		);
+
+		postProcessTemplate(dir, "my-app");
+
+		const pkg = JSON.parse(readFileSync(join(dir, "package.json"), "utf-8"));
+		// workspace:* 必须被解析成 npm 版本 range，且不允许写回（update --force 的回归测试）
+		expect(JSON.stringify(pkg)).not.toContain("workspace:");
+		expect(pkg.workspaces).toBeUndefined();
+		expect(pkg.dependencies?.["@dyyz1993/rpc-core"]).toMatch(/^\^?\d+\.\d+\.\d+/);
+		// 插件现在从 npm 安装，依赖保留而不是删除
+		expect(pkg.devDependencies?.["@dyyz1993/eslint-plugin-rpc"]).toMatch(/^\^?\d+\.\d+\.\d+/);
+		// 项目名写回（此前模板 name 永远不会被替换）
+		expect(pkg.name).toBe("my-app");
+
+		// RPC lint 规则在生成项目中保持启用
+		const eslint = readFileSync(join(dir, "eslint.config.mjs"), "utf-8");
+		expect(eslint).toContain("import rpcPlugin from '@dyyz1993/eslint-plugin-rpc'");
+		expect(eslint).toContain("'rpc/no-bare-method': 'error'");
+	});
+
+	test("copies shared/ into the project", () => {
+		const dir = makeTmp();
+		postProcessTemplate(dir, "my-app");
+		expect(existsSync(join(dir, "shared", "vite-base.config.ts"))).toBe(true);
+		// 开发期专用文件不应进入用户项目
+		expect(existsSync(join(dir, "shared", "http-routes.ts"))).toBe(false);
+		expect(existsSync(join(dir, "shared", "logger.ts"))).toBe(false);
+	});
+});