@aaronshaf/plane 0.1.2

package/tests/issue-commands.test.ts

@@ -0,0 +1,322 @@
+import { afterAll, afterEach, beforeAll, beforeEach, describe, expect, it } from "bun:test"
+import { Effect } from "effect"
+import { http, HttpResponse } from "msw"
+import { setupServer } from "msw/node"
+import { _clearProjectCache } from "@/resolve"
+
+const BASE = "http://issue-cmd-test.local"
+const WS = "testws"
+
+const PROJECTS = [{ id: "proj-acme", identifier: "ACME", name: "Acme Project" }]
+const ISSUES = [
+  { id: "i1", sequence_id: 29, name: "Migrate Button", priority: "high", state: "s1" },
+  { id: "i2", sequence_id: 30, name: "Migrate Input", priority: "medium", state: "s2" },
+]
+const STATES = [
+  { id: "s-done", name: "Done", group: "completed" },
+  { id: "s-todo", name: "Todo", group: "unstarted" },
+]
+
+const server = setupServer(
+  http.get(`${BASE}/api/v1/workspaces/${WS}/projects/`, () =>
+    HttpResponse.json({ results: PROJECTS }),
+  ),
+  http.get(`${BASE}/api/v1/workspaces/${WS}/projects/proj-acme/issues/`, () =>
+    HttpResponse.json({ results: ISSUES }),
+  ),
+  http.get(`${BASE}/api/v1/workspaces/${WS}/projects/proj-acme/states/`, () =>
+    HttpResponse.json({ results: STATES }),
+  ),
+)
+
+beforeAll(() => server.listen({ onUnhandledRequest: "error" }))
+afterAll(() => server.close())
+
+beforeEach(() => {
+  _clearProjectCache()
+  process.env["PLANE_HOST"] = BASE
+  process.env["PLANE_WORKSPACE"] = WS
+  process.env["PLANE_API_TOKEN"] = "test-token"
+})
+
+afterEach(() => {
+  server.resetHandlers()
+  delete process.env["PLANE_HOST"]
+  delete process.env["PLANE_WORKSPACE"]
+  delete process.env["PLANE_API_TOKEN"]
+})
+
+describe("issueGet", () => {
+  it("prints full JSON for an issue", async () => {
+    const { issueGet } = await import("@/commands/issue")
+    const logs: string[] = []
+    const orig = console.log
+    console.log = (...args: unknown[]) => logs.push(args.join(" "))
+
+    try {
+      await Effect.runPromise((issueGet as any).handler({ ref: "ACME-29" }))
+    } finally {
+      console.log = orig
+    }
+
+    const output = logs.join("\n")
+    const parsed = JSON.parse(output)
+    expect(parsed.id).toBe("i1")
+    expect(parsed.name).toBe("Migrate Button")
+  })
+})
+
+describe("issueUpdate", () => {
+  it("updates state", async () => {
+    server.use(
+      http.patch(
+        `${BASE}/api/v1/workspaces/${WS}/projects/proj-acme/issues/i1/`,
+        async ({ request }) => {
+          const body = (await request.json()) as any
+          return HttpResponse.json({
+            id: "i1",
+            sequence_id: 29,
+            name: "Migrate Button",
+            priority: "high",
+            state: body.state ?? "s1",
+          })
+        },
+      ),
+    )
+
+    const { issueUpdate } = await import("@/commands/issue")
+    const logs: string[] = []
+    const orig = console.log
+    console.log = (...args: unknown[]) => logs.push(args.join(" "))
+
+    try {
+      await Effect.runPromise(
+        (issueUpdate as any).handler({
+          ref: "ACME-29",
+          state: { _tag: "Some", value: "completed" },
+          priority: { _tag: "None" },
+        }),
+      )
+    } finally {
+      console.log = orig
+    }
+
+    expect(logs.join("\n")).toContain("Updated ACME-29")
+  })
+
+  it("updates priority", async () => {
+    server.use(
+      http.patch(
+        `${BASE}/api/v1/workspaces/${WS}/projects/proj-acme/issues/i1/`,
+        async ({ request }) => {
+          const body = (await request.json()) as any
+          return HttpResponse.json({
+            id: "i1",
+            sequence_id: 29,
+            name: "Migrate Button",
+            priority: body.priority,
+            state: "s1",
+          })
+        },
+      ),
+    )
+
+    const { issueUpdate } = await import("@/commands/issue")
+    const logs: string[] = []
+    const orig = console.log
+    console.log = (...args: unknown[]) => logs.push(args.join(" "))
+
+    try {
+      await Effect.runPromise(
+        (issueUpdate as any).handler({
+          ref: "ACME-29",
+          state: { _tag: "None" },
+          priority: { _tag: "Some", value: "urgent" },
+        }),
+      )
+    } finally {
+      console.log = orig
+    }
+
+    expect(logs.join("\n")).toContain("urgent")
+  })
+
+  it("fails when nothing to update", async () => {
+    const { issueUpdate } = await import("@/commands/issue")
+    const result = await Effect.runPromise(
+      Effect.either(
+        (issueUpdate as any).handler({
+          ref: "ACME-29",
+          state: { _tag: "None" },
+          priority: { _tag: "None" },
+        }),
+      ),
+    )
+    expect(result._tag).toBe("Left")
+    if (result._tag === "Left") {
+      expect(result.left.message).toContain("Nothing to update")
+    }
+  })
+})
+
+describe("issueComment", () => {
+  it("adds a comment", async () => {
+    let postedBody: unknown
+    server.use(
+      http.post(
+        `${BASE}/api/v1/workspaces/${WS}/projects/proj-acme/issues/i1/comments/`,
+        async ({ request }) => {
+          postedBody = await request.json()
+          return HttpResponse.json({ id: "c1" }, { status: 201 })
+        },
+      ),
+    )
+
+    const { issueComment } = await import("@/commands/issue")
+    const logs: string[] = []
+    const orig = console.log
+    console.log = (...args: unknown[]) => logs.push(args.join(" "))
+
+    try {
+      await Effect.runPromise(
+        (issueComment as any).handler({ ref: "ACME-29", text: "Fixed in latest build" }),
+      )
+    } finally {
+      console.log = orig
+    }
+
+    expect((postedBody as any).comment_html).toContain("Fixed in latest build")
+    expect(logs.join("\n")).toContain("Comment added to ACME-29")
+  })
+
+  it("HTML-escapes angle brackets in comment text", async () => {
+    let postedBody: unknown
+    server.use(
+      http.post(
+        `${BASE}/api/v1/workspaces/${WS}/projects/proj-acme/issues/i1/comments/`,
+        async ({ request }) => {
+          postedBody = await request.json()
+          return HttpResponse.json({ id: "c2" }, { status: 201 })
+        },
+      ),
+    )
+
+    const { issueComment } = await import("@/commands/issue")
+    try {
+      await Effect.runPromise(
+        (issueComment as any).handler({ ref: "ACME-29", text: "<script>alert(1)</script>" }),
+      )
+    } finally {}
+
+    expect((postedBody as any).comment_html).toContain("&lt;script&gt;")
+    expect((postedBody as any).comment_html).not.toContain("<script>")
+  })
+})
+
+describe("issueCreate", () => {
+  it("creates an issue with just a title", async () => {
+    server.use(
+      http.post(
+        `${BASE}/api/v1/workspaces/${WS}/projects/proj-acme/issues/`,
+        async ({ request }) => {
+          const body = (await request.json()) as any
+          return HttpResponse.json({
+            id: "new-i",
+            sequence_id: 99,
+            name: body.name,
+            priority: "none",
+            state: "s1",
+          })
+        },
+      ),
+    )
+
+    const { issueCreate } = await import("@/commands/issue")
+    const logs: string[] = []
+    const orig = console.log
+    console.log = (...args: unknown[]) => logs.push(args.join(" "))
+
+    try {
+      await Effect.runPromise(
+        (issueCreate as any).handler({
+          project: "ACME",
+          title: "New issue",
+          priority: { _tag: "None" },
+          state: { _tag: "None" },
+        }),
+      )
+    } finally {
+      console.log = orig
+    }
+
+    expect(logs.join("\n")).toContain("ACME-99")
+    expect(logs.join("\n")).toContain("New issue")
+  })
+
+  it("creates an issue with priority and state", async () => {
+    server.use(
+      http.post(
+        `${BASE}/api/v1/workspaces/${WS}/projects/proj-acme/issues/`,
+        async ({ request }) => {
+          const body = (await request.json()) as any
+          return HttpResponse.json({
+            id: "new-i2",
+            sequence_id: 100,
+            name: body.name,
+            priority: body.priority,
+            state: body.state ?? "s1",
+          })
+        },
+      ),
+    )
+
+    const { issueCreate } = await import("@/commands/issue")
+    const logs: string[] = []
+    const orig = console.log
+    console.log = (...args: unknown[]) => logs.push(args.join(" "))
+
+    try {
+      await Effect.runPromise(
+        (issueCreate as any).handler({
+          project: "ACME",
+          title: "High priority issue",
+          priority: { _tag: "Some", value: "high" },
+          state: { _tag: "Some", value: "completed" },
+        }),
+      )
+    } finally {
+      console.log = orig
+    }
+
+    expect(logs.join("\n")).toContain("ACME-100")
+  })
+})
+
+describe("issueDelete", () => {
+  it("deletes an issue", async () => {
+    let deleted = false
+    server.use(
+      http.delete(
+        `${BASE}/api/v1/workspaces/${WS}/projects/proj-acme/issues/i1/`,
+        () => {
+          deleted = true
+          return new HttpResponse(null, { status: 204 })
+        },
+      ),
+    )
+
+    const { issueDelete } = await import("@/commands/issue")
+    const logs: string[] = []
+    const orig = console.log
+    console.log = (...args: unknown[]) => logs.push(args.join(" "))
+
+    try {
+      await Effect.runPromise((issueDelete as any).handler({ ref: "ACME-29" }))
+    } finally {
+      console.log = orig
+    }
+
+    expect(deleted).toBe(true)
+    expect(logs.join("\n")).toContain("Deleted ACME-29")
+  })
+})

package/tests/issue-comments-worklogs.test.ts

@@ -0,0 +1,291 @@
+import { afterAll, afterEach, beforeAll, beforeEach, describe, expect, it } from "bun:test"
+import { Effect } from "effect"
+import { http, HttpResponse } from "msw"
+import { setupServer } from "msw/node"
+import { _clearProjectCache } from "@/resolve"
+
+const BASE = "http://cw-test.local"
+const WS = "testws"
+
+const PROJECTS = [{ id: "proj-acme", identifier: "ACME", name: "Acme Project" }]
+const ISSUES = [{ id: "i1", sequence_id: 29, name: "Migrate Button", priority: "high", state: "s1" }]
+const COMMENTS = [
+  {
+    id: "c1",
+    comment_html: "<p>Fixed in v2</p>",
+    actor_detail: { display_name: "Aaron" },
+    created_at: "2025-01-15T10:30:00Z",
+  },
+  {
+    id: "c2",
+    comment_html: "<p>LGTM</p>",
+    actor_detail: { display_name: "Bea" },
+    created_at: "2025-01-16T09:00:00Z",
+  },
+]
+const WORKLOGS = [
+  {
+    id: "w1",
+    description: "Code review",
+    duration: 90,
+    logged_by_detail: { display_name: "Aaron" },
+    created_at: "2025-01-15T10:00:00Z",
+  },
+]
+
+const server = setupServer(
+  http.get(`${BASE}/api/v1/workspaces/${WS}/projects/`, () =>
+    HttpResponse.json({ results: PROJECTS }),
+  ),
+  http.get(`${BASE}/api/v1/workspaces/${WS}/projects/proj-acme/issues/`, () =>
+    HttpResponse.json({ results: ISSUES }),
+  ),
+  http.get(`${BASE}/api/v1/workspaces/${WS}/projects/proj-acme/issues/i1/comments/`, () =>
+    HttpResponse.json({ results: COMMENTS }),
+  ),
+  http.get(`${BASE}/api/v1/workspaces/${WS}/projects/proj-acme/issues/i1/worklogs/`, () =>
+    HttpResponse.json({ results: WORKLOGS }),
+  ),
+)
+
+beforeAll(() => server.listen({ onUnhandledRequest: "error" }))
+afterAll(() => server.close())
+
+beforeEach(() => {
+  _clearProjectCache()
+  process.env["PLANE_HOST"] = BASE
+  process.env["PLANE_WORKSPACE"] = WS
+  process.env["PLANE_API_TOKEN"] = "test-token"
+})
+
+afterEach(() => {
+  server.resetHandlers()
+  delete process.env["PLANE_HOST"]
+  delete process.env["PLANE_WORKSPACE"]
+  delete process.env["PLANE_API_TOKEN"]
+})
+
+describe("issueCommentsList", () => {
+  it("lists comments with author and stripped HTML", async () => {
+    const { issueCommentsList } = await import("@/commands/issue")
+    const logs: string[] = []
+    const orig = console.log
+    console.log = (...args: unknown[]) => logs.push(args.join(" "))
+    try {
+      await Effect.runPromise((issueCommentsList as any).handler({ ref: "ACME-29" }))
+    } finally {
+      console.log = orig
+    }
+    const output = logs.join("\n")
+    expect(output).toContain("c1")
+    expect(output).toContain("Aaron")
+    expect(output).toContain("Fixed in v2")
+    expect(output).not.toContain("<p>")
+  })
+
+  it("shows 'No comments' when empty", async () => {
+    server.use(
+      http.get(`${BASE}/api/v1/workspaces/${WS}/projects/proj-acme/issues/i1/comments/`, () =>
+        HttpResponse.json({ results: [] }),
+      ),
+    )
+    const { issueCommentsList } = await import("@/commands/issue")
+    const logs: string[] = []
+    const orig = console.log
+    console.log = (...args: unknown[]) => logs.push(args.join(" "))
+    try {
+      await Effect.runPromise((issueCommentsList as any).handler({ ref: "ACME-29" }))
+    } finally {
+      console.log = orig
+    }
+    expect(logs.join("\n")).toBe("No comments")
+  })
+
+  it("shows '?' for missing actor", async () => {
+    server.use(
+      http.get(`${BASE}/api/v1/workspaces/${WS}/projects/proj-acme/issues/i1/comments/`, () =>
+        HttpResponse.json({ results: [{ id: "c3", comment_html: "<p>hi</p>", created_at: "2025-01-17T10:00:00Z" }] }),
+      ),
+    )
+    const { issueCommentsList } = await import("@/commands/issue")
+    const logs: string[] = []
+    const orig = console.log
+    console.log = (...args: unknown[]) => logs.push(args.join(" "))
+    try {
+      await Effect.runPromise((issueCommentsList as any).handler({ ref: "ACME-29" }))
+    } finally {
+      console.log = orig
+    }
+    expect(logs.join("\n")).toContain("?")
+  })
+})
+
+describe("issueCommentUpdate", () => {
+  it("updates a comment", async () => {
+    let patchedBody: unknown
+    server.use(
+      http.patch(
+        `${BASE}/api/v1/workspaces/${WS}/projects/proj-acme/issues/i1/comments/c1/`,
+        async ({ request }) => {
+          patchedBody = await request.json()
+          return HttpResponse.json({ id: "c1", comment_html: "<p>Updated</p>", created_at: "2025-01-15T10:30:00Z" })
+        },
+      ),
+    )
+    const { issueCommentUpdate } = await import("@/commands/issue")
+    const logs: string[] = []
+    const orig = console.log
+    console.log = (...args: unknown[]) => logs.push(args.join(" "))
+    try {
+      await Effect.runPromise(
+        (issueCommentUpdate as any).handler({ ref: "ACME-29", commentId: "c1", text: "Updated text" }),
+      )
+    } finally {
+      console.log = orig
+    }
+    expect((patchedBody as any).comment_html).toContain("Updated text")
+    expect(logs.join("\n")).toContain("c1")
+    expect(logs.join("\n")).toContain("updated")
+  })
+})
+
+describe("issueCommentDelete", () => {
+  it("deletes a comment", async () => {
+    let deleted = false
+    server.use(
+      http.delete(
+        `${BASE}/api/v1/workspaces/${WS}/projects/proj-acme/issues/i1/comments/c1/`,
+        () => {
+          deleted = true
+          return new HttpResponse(null, { status: 204 })
+        },
+      ),
+    )
+    const { issueCommentDelete } = await import("@/commands/issue")
+    const logs: string[] = []
+    const orig = console.log
+    console.log = (...args: unknown[]) => logs.push(args.join(" "))
+    try {
+      await Effect.runPromise(
+        (issueCommentDelete as any).handler({ ref: "ACME-29", commentId: "c1" }),
+      )
+    } finally {
+      console.log = orig
+    }
+    expect(deleted).toBe(true)
+    expect(logs.join("\n")).toContain("c1")
+  })
+})
+
+describe("issueWorklogsList", () => {
+  it("lists worklogs with hours", async () => {
+    const { issueWorklogsList } = await import("@/commands/issue")
+    const logs: string[] = []
+    const orig = console.log
+    console.log = (...args: unknown[]) => logs.push(args.join(" "))
+    try {
+      await Effect.runPromise((issueWorklogsList as any).handler({ ref: "ACME-29" }))
+    } finally {
+      console.log = orig
+    }
+    const output = logs.join("\n")
+    expect(output).toContain("w1")
+    expect(output).toContain("1.5h")
+    expect(output).toContain("Aaron")
+    expect(output).toContain("Code review")
+  })
+
+  it("shows 'No worklogs' when empty", async () => {
+    server.use(
+      http.get(`${BASE}/api/v1/workspaces/${WS}/projects/proj-acme/issues/i1/worklogs/`, () =>
+        HttpResponse.json({ results: [] }),
+      ),
+    )
+    const { issueWorklogsList } = await import("@/commands/issue")
+    const logs: string[] = []
+    const orig = console.log
+    console.log = (...args: unknown[]) => logs.push(args.join(" "))
+    try {
+      await Effect.runPromise((issueWorklogsList as any).handler({ ref: "ACME-29" }))
+    } finally {
+      console.log = orig
+    }
+    expect(logs.join("\n")).toBe("No worklogs")
+  })
+})
+
+describe("issueWorklogsAdd", () => {
+  it("logs time without description", async () => {
+    server.use(
+      http.post(
+        `${BASE}/api/v1/workspaces/${WS}/projects/proj-acme/issues/i1/worklogs/`,
+        async ({ request }) => {
+          const body = (await request.json()) as any
+          return HttpResponse.json({ id: "w-new", duration: body.duration, created_at: "2025-01-15T10:00:00Z" })
+        },
+      ),
+    )
+    const { issueWorklogsAdd } = await import("@/commands/issue")
+    const logs: string[] = []
+    const orig = console.log
+    console.log = (...args: unknown[]) => logs.push(args.join(" "))
+    try {
+      await Effect.runPromise(
+        (issueWorklogsAdd as any).handler({ ref: "ACME-29", duration: 60, description: { _tag: "None" } }),
+      )
+    } finally {
+      console.log = orig
+    }
+    expect(logs.join("\n")).toContain("1.0h")
+    expect(logs.join("\n")).toContain("w-new")
+  })
+
+  it("logs time with description", async () => {
+    server.use(
+      http.post(
+        `${BASE}/api/v1/workspaces/${WS}/projects/proj-acme/issues/i1/worklogs/`,
+        async ({ request }) => {
+          const body = (await request.json()) as any
+          return HttpResponse.json({ id: "w-new2", duration: body.duration, description: body.description, created_at: "2025-01-15T10:00:00Z" })
+        },
+      ),
+    )
+    const { issueWorklogsAdd } = await import("@/commands/issue")
+    const logs: string[] = []
+    const orig = console.log
+    console.log = (...args: unknown[]) => logs.push(args.join(" "))
+    try {
+      await Effect.runPromise(
+        (issueWorklogsAdd as any).handler({
+          ref: "ACME-29",
+          duration: 30,
+          description: { _tag: "Some", value: "standup" },
+        }),
+      )
+    } finally {
+      console.log = orig
+    }
+    expect(logs.join("\n")).toContain("0.5h")
+  })
+
+  it("handles missing logged_by_detail in worklogs list", async () => {
+    server.use(
+      http.get(`${BASE}/api/v1/workspaces/${WS}/projects/proj-acme/issues/i1/worklogs/`, () =>
+        HttpResponse.json({
+          results: [{ id: "w2", duration: 45, created_at: "2025-01-15T10:00:00Z" }],
+        }),
+      ),
+    )
+    const { issueWorklogsList } = await import("@/commands/issue")
+    const logs: string[] = []
+    const orig = console.log
+    console.log = (...args: unknown[]) => logs.push(args.join(" "))
+    try {
+      await Effect.runPromise((issueWorklogsList as any).handler({ ref: "ACME-29" }))
+    } finally {
+      console.log = orig
+    }
+    expect(logs.join("\n")).toContain("?")
+    expect(logs.join("\n")).toContain("0.8h")
+  })
+})