test(agents): add CLI-level pull tests with fixture and file assertions

- Add `with-agents-for-pull` fixture with a JSONC-commented agent file
- Add tests verifying agent files are written to disk with correct content
- Add test verifying unchanged agents are skipped (JSONC comment preserved)
- Add test verifying changed agents are updated in-place at existing path

Co-authored-by: paveltarno <Paveltarno@users.noreply.github.com>

Author: github-actions[bot]

tests/cli/agents_pull.spec.ts

@@ -1,4 +1,4 @@
-import { describe, it } from "vitest";
+import { describe, expect, it } from "vitest";
 import { fixture, setupCLITests } from "./testkit/index.js";
 
 describe("agents pull command", () => {
@@ -52,4 +52,85 @@ describe("agents pull command", () => {
 
     t.expectResult(result).toFail();
   });
+
+  it("writes agent files to disk after pull", async () => {
+    await t.givenLoggedInWithProject(fixture("basic"));
+    t.api.mockAgentsFetch({
+      items: [
+        {
+          name: "support_agent",
+          description: "Helps users",
+          instructions: "Be helpful",
+        },
+      ],
+      total: 1,
+    });
+
+    const result = await t.run("agents", "pull");
+
+    t.expectResult(result).toSucceed();
+
+    expect(await t.fileExists("base44/agents/support_agent.jsonc")).toBe(true);
+
+    const fileContent = await t.readProjectFile(
+      "base44/agents/support_agent.jsonc",
+    );
+    const parsed = JSON.parse(fileContent!);
+    expect(parsed.name).toBe("support_agent");
+    expect(parsed.description).toBe("Helps users");
+    expect(parsed.instructions).toBe("Be helpful");
+  });
+
+  it("skips unchanged agents and preserves file content", async () => {
+    await t.givenLoggedInWithProject(fixture("with-agents-for-pull"));
+    t.api.mockAgentsFetch({
+      items: [
+        {
+          name: "support_agent",
+          description: "Helps users",
+          instructions: "Be helpful",
+        },
+      ],
+      total: 1,
+    });
+
+    const result = await t.run("agents", "pull");
+
+    t.expectResult(result).toSucceed();
+    t.expectResult(result).toContain("All agents are already up to date");
+
+    // The file should not have been rewritten — JSONC comment must still be present
+    const fileContent = await t.readProjectFile(
+      "base44/agents/support_agent.jsonc",
+    );
+    expect(fileContent).toContain("// My support agent");
+  });
+
+  it("updates agent file in-place when remote data changes", async () => {
+    await t.givenLoggedInWithProject(fixture("with-agents-for-pull"));
+    t.api.mockAgentsFetch({
+      items: [
+        {
+          name: "support_agent",
+          description: "Updated description",
+          instructions: "Updated instructions",
+        },
+      ],
+      total: 1,
+    });
+
+    const result = await t.run("agents", "pull");
+
+    t.expectResult(result).toSucceed();
+
+    // File should be updated at the same path (in-place)
+    expect(await t.fileExists("base44/agents/support_agent.jsonc")).toBe(true);
+
+    const fileContent = await t.readProjectFile(
+      "base44/agents/support_agent.jsonc",
+    );
+    const parsed = JSON.parse(fileContent!);
+    expect(parsed.description).toBe("Updated description");
+    expect(parsed.instructions).toBe("Updated instructions");
+  });
 });

tests/fixtures/with-agents-for-pull/base44/.app.jsonc

@@ -0,0 +1,3 @@
+{
+  "id": "test-app-id"
+}

tests/fixtures/with-agents-for-pull/base44/agents/support_agent.jsonc

@@ -0,0 +1,6 @@
+// My support agent
+{
+  "name": "support_agent",
+  "description": "Helps users",
+  "instructions": "Be helpful"
+}

tests/fixtures/with-agents-for-pull/base44/config.jsonc

@@ -0,0 +1,3 @@
+{
+  "name": "Agents Pull Test Project"
+}