@foundation0/api 1.1.8 → 1.1.10

package/mcp/server.test.ts

@@ -204,7 +204,8 @@ describe("createExampleMcpServer request handling", () => {
 		}
 	});
 
-	it("accepts repoName for projects tools (preferred over legacy repoRoot/processRoot)", async () => {
+	it("accepts repoName selectors in short and owner/repo format", async () => {
+		const originalCwd = process.cwd();
 		const tempDir = await fs.mkdtemp(
 			path.join(os.tmpdir(), "f0-mcp-server-reponame-"),
 		);
@@ -216,114 +217,67 @@ describe("createExampleMcpServer request handling", () => {
 			await fs.mkdir(path.join(tempDir, "projects", "beta", "docs"), {
 				recursive: true,
 			});
-
-			const instance = createExampleMcpServer({ repoRoot: tempDir, repoName: "test" });
-			const handler = getToolHandler(instance);
-
-			const result = await handler(
-				{
-					method: "tools/call",
-					params: {
-						name: "projects.listProjects",
-						arguments: {
-							repoName: "test",
-						},
-					},
-				},
-				{},
+			await fs.mkdir(path.join(tempDir, ".git"), { recursive: true });
+			await fs.writeFile(
+				path.join(tempDir, ".git", "config"),
+				[
+					'[remote "origin"]',
+					"\turl = https://example.com/F0/adl.git",
+					"",
+				].join("\n"),
 			);
 
-			expect(result.isError).toBe(false);
-			const payload = JSON.parse(result.content?.[0]?.text ?? "{}");
-			expect(payload.ok).toBe(true);
-			expect(payload.result).toContain("adl");
-			expect(payload.result).toContain("beta");
-		} finally {
-			await fs.rm(tempDir, { recursive: true, force: true });
-		}
-	});
-
-	it("auto-detects repoRoot from process.cwd() when repoName is omitted", async () => {
-		const tempDir = await fs.mkdtemp(
-			path.join(os.tmpdir(), "f0-mcp-server-autoreporoot-"),
-		);
-		const originalCwd = process.cwd();
-
-		try {
-			await fs.mkdir(path.join(tempDir, "api"), { recursive: true });
-			await fs.mkdir(path.join(tempDir, "projects", "adl", "docs"), {
-				recursive: true,
-			});
-			await fs.mkdir(path.join(tempDir, "projects", "beta", "docs"), {
-				recursive: true,
-			});
-
 			process.chdir(tempDir);
-
 			const instance = createExampleMcpServer();
 			const handler = getToolHandler(instance);
 
-			const result = await handler(
+			const shortSelector = await handler(
 				{
 					method: "tools/call",
 					params: {
 						name: "projects.listProjects",
-						arguments: {},
+						arguments: {
+							repoName: "adl",
+						},
 					},
 				},
 				{},
 			);
+			expect(shortSelector.isError).toBe(false);
+			const shortPayload = JSON.parse(shortSelector.content?.[0]?.text ?? "{}");
+			expect(shortPayload.ok).toBe(true);
+			expect(shortPayload.result).toContain("adl");
+			expect(shortPayload.result).toContain("beta");
 
-			expect(result.isError).toBe(false);
-			const payload = JSON.parse(result.content?.[0]?.text ?? "{}");
-			expect(payload.ok).toBe(true);
-			expect(payload.result).toContain("adl");
-			expect(payload.result).toContain("beta");
-		} finally {
-			process.chdir(originalCwd);
-			await fs.rm(tempDir, { recursive: true, force: true });
-		}
-	});
-
-	it("still accepts legacy processRoot for backwards compatibility", async () => {
-		const tempDir = await fs.mkdtemp(
-			path.join(os.tmpdir(), "f0-mcp-server-processroot-"),
-		);
-		try {
-			await fs.mkdir(path.join(tempDir, "api"), { recursive: true });
-			await fs.mkdir(path.join(tempDir, "projects", "adl", "docs"), {
-				recursive: true,
-			});
-
-			const instance = createExampleMcpServer();
-			const handler = getToolHandler(instance);
-
-			const result = await handler(
+			const fullSelector = await handler(
 				{
 					method: "tools/call",
 					params: {
 						name: "projects.listProjects",
 						arguments: {
-							processRoot: tempDir,
+							repoName: "F0/adl",
 						},
 					},
 				},
 				{},
 			);
-
-			expect(result.isError).toBe(false);
-			const payload = JSON.parse(result.content?.[0]?.text ?? "{}");
-			expect(payload.ok).toBe(true);
-			expect(payload.result).toContain("adl");
+			expect(fullSelector.isError).toBe(false);
+			const fullPayload = JSON.parse(fullSelector.content?.[0]?.text ?? "{}");
+			expect(fullPayload.ok).toBe(true);
+			expect(fullPayload.result).toContain("adl");
+			expect(fullPayload.result).toContain("beta");
 		} finally {
+			process.chdir(originalCwd);
 			await fs.rm(tempDir, { recursive: true, force: true });
 		}
 	});
 
-	it("normalizes legacy repoRoot when caller accidentally passes a project root", async () => {
+	it("auto-detects Git workspace from process.cwd() when repoName is omitted", async () => {
 		const tempDir = await fs.mkdtemp(
-			path.join(os.tmpdir(), "f0-mcp-server-reporoot-normalize-"),
+			path.join(os.tmpdir(), "f0-mcp-server-autoworkspace-"),
 		);
+		const originalCwd = process.cwd();
+
 		try {
 			await fs.mkdir(path.join(tempDir, "api"), { recursive: true });
 			await fs.mkdir(path.join(tempDir, "projects", "adl", "docs"), {
@@ -332,6 +286,17 @@ describe("createExampleMcpServer request handling", () => {
 			await fs.mkdir(path.join(tempDir, "projects", "beta", "docs"), {
 				recursive: true,
 			});
+			await fs.mkdir(path.join(tempDir, ".git"), { recursive: true });
+			await fs.writeFile(
+				path.join(tempDir, ".git", "config"),
+				[
+					'[remote "origin"]',
+					"\turl = https://example.com/F0/adl.git",
+					"",
+				].join("\n"),
+			);
+
+			process.chdir(tempDir);
 
 			const instance = createExampleMcpServer();
 			const handler = getToolHandler(instance);
@@ -341,9 +306,7 @@ describe("createExampleMcpServer request handling", () => {
 					method: "tools/call",
 					params: {
 						name: "projects.listProjects",
-						arguments: {
-							repoRoot: path.join(tempDir, "projects", "adl"),
-						},
+						arguments: {},
 					},
 				},
 				{},
@@ -355,24 +318,41 @@ describe("createExampleMcpServer request handling", () => {
 			expect(payload.result).toContain("adl");
 			expect(payload.result).toContain("beta");
 		} finally {
+			process.chdir(originalCwd);
 			await fs.rm(tempDir, { recursive: true, force: true });
 		}
 	});
 
-	it("uses server default repoRoot when repoName is omitted", async () => {
-		const tempDir = await fs.mkdtemp(
-			path.join(os.tmpdir(), "f0-mcp-server-default-reporoot-"),
+	it("ignores legacy processRoot/repoRoot payload fields and uses Git workspace", async () => {
+		const originalCwd = process.cwd();
+		const workspaceA = await fs.mkdtemp(
+			path.join(os.tmpdir(), "f0-mcp-server-legacy-fields-a-"),
+		);
+		const workspaceB = await fs.mkdtemp(
+			path.join(os.tmpdir(), "f0-mcp-server-legacy-fields-b-"),
 		);
 		try {
-			await fs.mkdir(path.join(tempDir, "api"), { recursive: true });
-			await fs.mkdir(path.join(tempDir, "projects", "adl", "docs"), {
+			await fs.mkdir(path.join(workspaceA, "api"), { recursive: true });
+			await fs.mkdir(path.join(workspaceA, "projects", "adl", "docs"), {
 				recursive: true,
 			});
-			await fs.mkdir(path.join(tempDir, "projects", "beta", "docs"), {
+			await fs.mkdir(path.join(workspaceA, ".git"), { recursive: true });
+			await fs.writeFile(
+				path.join(workspaceA, ".git", "config"),
+				[
+					'[remote "origin"]',
+					"\turl = https://example.com/F0/adl.git",
+					"",
+				].join("\n"),
+			);
+
+			await fs.mkdir(path.join(workspaceB, "api"), { recursive: true });
+			await fs.mkdir(path.join(workspaceB, "projects", "beta", "docs"), {
 				recursive: true,
 			});
 
-			const instance = createExampleMcpServer({ repoRoot: tempDir });
+			process.chdir(workspaceA);
+			const instance = createExampleMcpServer();
 			const handler = getToolHandler(instance);
 
 			const result = await handler(
@@ -380,7 +360,10 @@ describe("createExampleMcpServer request handling", () => {
 					method: "tools/call",
 					params: {
 						name: "projects.listProjects",
-						arguments: {},
+						arguments: {
+							processRoot: workspaceB,
+							repoRoot: workspaceB,
+						},
 					},
 				},
 				{},
@@ -390,13 +373,16 @@ describe("createExampleMcpServer request handling", () => {
 			const payload = JSON.parse(result.content?.[0]?.text ?? "{}");
 			expect(payload.ok).toBe(true);
 			expect(payload.result).toContain("adl");
-			expect(payload.result).toContain("beta");
+			expect(payload.result).not.toContain("beta");
 		} finally {
-			await fs.rm(tempDir, { recursive: true, force: true });
+			process.chdir(originalCwd);
+			await fs.rm(workspaceA, { recursive: true, force: true });
+			await fs.rm(workspaceB, { recursive: true, force: true });
 		}
 	});
 
-	it("exposes mcp.workspace to explain server filesystem context", async () => {
+	it("exposes mcp.workspace with Git workspace context", async () => {
+		const originalCwd = process.cwd();
 		const tempDir = await fs.mkdtemp(
 			path.join(os.tmpdir(), "f0-mcp-server-workspace-"),
 		);
@@ -405,8 +391,18 @@ describe("createExampleMcpServer request handling", () => {
 			await fs.mkdir(path.join(tempDir, "projects", "adl", "docs"), {
 				recursive: true,
 			});
+			await fs.mkdir(path.join(tempDir, ".git"), { recursive: true });
+			await fs.writeFile(
+				path.join(tempDir, ".git", "config"),
+				[
+					'[remote "origin"]',
+					"\turl = https://example.com/F0/adl.git",
+					"",
+				].join("\n"),
+			);
 
-			const instance = createExampleMcpServer({ repoRoot: tempDir });
+			process.chdir(tempDir);
+			const instance = createExampleMcpServer();
 			const handler = getToolHandler(instance);
 
 			const result = await handler(
@@ -423,16 +419,21 @@ describe("createExampleMcpServer request handling", () => {
 			expect(result.isError).toBe(false);
 			const payload = JSON.parse(result.content?.[0]?.text ?? "{}");
 			expect(payload.ok).toBe(true);
-			expect(payload.result.repoRoot).toBe(path.resolve(tempDir));
-			expect(payload.result.defaultRepoRoot).toBe(path.resolve(tempDir));
+			expect(payload.result.workspaceRoot).toBe(path.resolve(tempDir));
+			expect(payload.result.processRoot).toBe(path.resolve(tempDir));
+			expect(payload.result.defaultRepoName).toBe("adl");
+			expect(payload.result.defaultRepoFullName).toBe("F0/adl");
+			expect(payload.result.availableRepoNames).toContain("adl");
+			expect(payload.result.availableRepoNames).toContain("F0/adl");
 			expect(payload.result.hasProjectsDir).toBe(true);
 			expect(Array.isArray(payload.result.projects)).toBe(true);
 		} finally {
+			process.chdir(originalCwd);
 			await fs.rm(tempDir, { recursive: true, force: true });
 		}
 	});
 
-	it("auto-detects repo root/name from .git/config when no repoRoot/repoName are provided", async () => {
+	it("auto-detects repo identity from .git/config when no repoName is provided", async () => {
 		const originalCwd = process.cwd();
 		const tempDir = await fs.mkdtemp(
 			path.join(os.tmpdir(), "f0-mcp-server-git-detect-"),
@@ -470,11 +471,13 @@ describe("createExampleMcpServer request handling", () => {
 			expect(workspace.isError).toBe(false);
 			const payload = JSON.parse(workspace.content?.[0]?.text ?? "{}");
 			expect(payload.ok).toBe(true);
-			expect(payload.result.defaultRepoRoot).toBe(path.resolve(tempDir));
-			expect(payload.result.repoRoot).toBe(path.resolve(tempDir));
+			expect(payload.result.workspaceRoot).toBe(path.resolve(tempDir));
+			expect(payload.result.processRoot).toBe(path.resolve(tempDir));
 			expect(payload.result.defaultRepoName).toBe("adl");
+			expect(payload.result.defaultRepoFullName).toBe("F0/adl");
 			expect(payload.result.repoName).toBe("adl");
 			expect(payload.result.availableRepoNames).toContain("adl");
+			expect(payload.result.availableRepoNames).toContain("F0/adl");
 
 			const list = await handler(
 				{
@@ -538,6 +541,8 @@ describe("createExampleMcpServer request handling", () => {
 			const payload = JSON.parse(workspace.content?.[0]?.text ?? "{}");
 			expect(payload.ok).toBe(true);
 			expect(payload.result.defaultRepoName).toBe("adl");
+			expect(payload.result.defaultRepoFullName).toBe("F0/adl");
+			expect(payload.result.availableRepoNames).toContain("F0/adl");
 		} finally {
 			process.chdir(originalCwd);
 			await fs.rm(tempDir, { recursive: true, force: true });