@opsee/mcp-server 0.9.33 → 0.9.35

package/README.md

@@ -77,7 +77,7 @@ Ask Claude things like:
 | `opsee_create_cycle` | Create a new cycle |
 | `opsee_update_cycle` | Update a cycle's fields |
 | `opsee_delete_cycle` | Delete a cycle |
-| `opsee_list_doc_spaces` | List doc spaces |
+| `opsee_list_doc_spaces` | List doc spaces in a project |
 | `opsee_list_doc_pages` | List pages in a doc space |
 | `opsee_get_doc_page` | Read a doc page |
 | `opsee_create_doc_page` | Create a new doc page |

package/gen/api/v1/doc_space_pb.d.ts

@@ -152,6 +152,17 @@ export declare type GetDocSpacesRequest = Message<"api.v1.GetDocSpacesRequest">
    * @generated from field: api.v1.FilterOptions filter_options = 2;
    */
   filterOptions?: FilterOptions;
+
+  /**
+   * Doc spaces are project-scoped. This has to be a first-class request field
+   * rather than a FilterOptions entry: scopes.ParseFilterOptions drops a
+   * "ProjectId" filter key as security-sensitive, so a filter-based scope
+   * silently widens the result to every doc space in the company. Mirrors
+   * GetDocPagesRequest.project_id.
+   *
+   * @generated from field: uint32 project_id = 3;
+   */
+  projectId: number;
 };
 
 /**

package/gen/api/v1/doc_space_pb.js

@@ -14,7 +14,7 @@ import { file_api_v1_models } from "./models_pb";
  * Describes the file api/v1/doc_space.proto.
  */
 export const file_api_v1_doc_space = /*@__PURE__*/
-  fileDesc("ChZhcGkvdjEvZG9jX3NwYWNlLnByb3RvEgZhcGkudjEi4gEKEkFkZERvY1NwYWNlUmVxdWVzdBIWCgV0aXRsZRgBIAEoCUIH+kIEcgIQARIVCgRzbHVnGAIgASgJQgf6QgRyAhABEhgKC2Rlc2NyaXB0aW9uGAMgASgJSACIAQESEgoKaXNfZGVmYXVsdBgEIAEoCBITCgtpc19hcmNoaXZlZBgFIAEoCBISCgpwcm9qZWN0X2lkGAYgASgNEhoKEmNyZWF0ZWRfYnlfdXNlcl9pZBgHIAEoDRIaChJ1cGRhdGVkX2J5X3VzZXJfaWQYCCABKA1CDgoMX2Rlc2NyaXB0aW9uIvgBChNFZGl0RG9jU3BhY2VSZXF1ZXN0EhMKAmlkGAEgASgNQgf6QgQqAigBEhYKBXRpdGxlGAIgASgJQgf6QgRyAhABEhUKBHNsdWcYAyABKAlCB/pCBHICEAESGAoLZGVzY3JpcHRpb24YBCABKAlIAIgBARISCgppc19kZWZhdWx0GAUgASgIEhMKC2lzX2FyY2hpdmVkGAYgASgIEhIKCnByb2plY3RfaWQYByABKA0SGgoSY3JlYXRlZF9ieV91c2VyX2lkGAggASgNEhoKEnVwZGF0ZWRfYnlfdXNlcl9pZBgJIAEoDUIOCgxfZGVzY3JpcHRpb24iIAoSR2V0RG9jU3BhY2VSZXF1ZXN0EgoKAmlkGAEgASgNInYKE0dldERvY1NwYWNlc1JlcXVlc3QSMAoKcGFnaW5hdGlvbhgBIAEoCzISLmFwaS52MS5QYWdpbmF0aW9uQgj6QgWKAQIQARItCg5maWx0ZXJfb3B0aW9ucxgCIAEoCzIVLmFwaS52MS5GaWx0ZXJPcHRpb25zIiMKFURlbGV0ZURvY1NwYWNlUmVxdWVzdBIKCgJpZBgBIAEoDSJkChRHZXREb2NTcGFjZXNSZXNwb25zZRIkCgpkb2Nfc3BhY2VzGAEgAygLMhAuYXBpLnYxLkRvY1NwYWNlEiYKCnBhZ2luYXRpb24YAiABKAsyEi5hcGkudjEuUGFnaW5hdGlvbiI6ChNBZGREb2NTcGFjZVJlc3BvbnNlEiMKCWRvY19zcGFjZRgBIAEoCzIQLmFwaS52MS5Eb2NTcGFjZSI7ChRFZGl0RG9jU3BhY2VSZXNwb25zZRIjCglkb2Nfc3BhY2UYASABKAsyEC5hcGkudjEuRG9jU3BhY2UiOgoTR2V0RG9jU3BhY2VSZXNwb25zZRIjCglkb2Nfc3BhY2UYASABKAsyEC5hcGkudjEuRG9jU3BhY2UyoQQKD0RvY1NwYWNlU2VydmljZRJlCgtBZGREb2NTcGFjZRIaLmFwaS52MS5BZGREb2NTcGFjZVJlcXVlc3QaGy5hcGkudjEuQWRkRG9jU3BhY2VSZXNwb25zZSIdgtPkkwIXOgEqIhIvYXBpL3YxL2RvY19zcGFjZXMSbQoMRWRpdERvY1NwYWNlEhsuYXBpLnYxLkVkaXREb2NTcGFjZVJlcXVlc3QaHC5hcGkudjEuRWRpdERvY1NwYWNlUmVzcG9uc2UiIoLT5JMCHDoBKhoXL2FwaS92MS9kb2Nfc3BhY2VzL3tpZH0SZwoLR2V0RG9jU3BhY2USGi5hcGkudjEuR2V0RG9jU3BhY2VSZXF1ZXN0GhsuYXBpLnYxLkdldERvY1NwYWNlUmVzcG9uc2UiH4LT5JMCGRIXL2FwaS92MS9kb2Nfc3BhY2VzL3tpZH0SZQoMR2V0RG9jU3BhY2VzEhsuYXBpLnYxLkdldERvY1NwYWNlc1JlcXVlc3QaHC5hcGkudjEuR2V0RG9jU3BhY2VzUmVzcG9uc2UiGoLT5JMCFBISL2FwaS92MS9kb2Nfc3BhY2VzEmgKDkRlbGV0ZURvY1NwYWNlEh0uYXBpLnYxLkRlbGV0ZURvY1NwYWNlUmVxdWVzdBoWLmdvb2dsZS5wcm90b2J1Zi5FbXB0eSIfgtPkkwIZKhcvYXBpL3YxL2RvY19zcGFjZXMve2lkfUJyCgpjb20uYXBpLnYxQg1Eb2NTcGFjZVByb3RvUAFaHG9wc2VlL2JhY2tlbmQvZ2VuL2FwaS92MTtnZW6iAgNBWFiqAgZBcGkuVjHKAgZBcGlcVjHiAhJBcGlcVjFcR1BCTWV0YWRhdGHqAgdBcGk6OlYxYgZwcm90bzM", [file_google_protobuf_empty, file_google_api_annotations, file_validate_validate, file_api_v1_pagination, file_api_v1_filter, file_api_v1_models]);
+  fileDesc("ChZhcGkvdjEvZG9jX3NwYWNlLnByb3RvEgZhcGkudjEi4gEKEkFkZERvY1NwYWNlUmVxdWVzdBIWCgV0aXRsZRgBIAEoCUIH+kIEcgIQARIVCgRzbHVnGAIgASgJQgf6QgRyAhABEhgKC2Rlc2NyaXB0aW9uGAMgASgJSACIAQESEgoKaXNfZGVmYXVsdBgEIAEoCBITCgtpc19hcmNoaXZlZBgFIAEoCBISCgpwcm9qZWN0X2lkGAYgASgNEhoKEmNyZWF0ZWRfYnlfdXNlcl9pZBgHIAEoDRIaChJ1cGRhdGVkX2J5X3VzZXJfaWQYCCABKA1CDgoMX2Rlc2NyaXB0aW9uIvgBChNFZGl0RG9jU3BhY2VSZXF1ZXN0EhMKAmlkGAEgASgNQgf6QgQqAigBEhYKBXRpdGxlGAIgASgJQgf6QgRyAhABEhUKBHNsdWcYAyABKAlCB/pCBHICEAESGAoLZGVzY3JpcHRpb24YBCABKAlIAIgBARISCgppc19kZWZhdWx0GAUgASgIEhMKC2lzX2FyY2hpdmVkGAYgASgIEhIKCnByb2plY3RfaWQYByABKA0SGgoSY3JlYXRlZF9ieV91c2VyX2lkGAggASgNEhoKEnVwZGF0ZWRfYnlfdXNlcl9pZBgJIAEoDUIOCgxfZGVzY3JpcHRpb24iIAoSR2V0RG9jU3BhY2VSZXF1ZXN0EgoKAmlkGAEgASgNIpMBChNHZXREb2NTcGFjZXNSZXF1ZXN0EjAKCnBhZ2luYXRpb24YASABKAsyEi5hcGkudjEuUGFnaW5hdGlvbkII+kIFigECEAESLQoOZmlsdGVyX29wdGlvbnMYAiABKAsyFS5hcGkudjEuRmlsdGVyT3B0aW9ucxIbCgpwcm9qZWN0X2lkGAMgASgNQgf6QgQqAigBIiMKFURlbGV0ZURvY1NwYWNlUmVxdWVzdBIKCgJpZBgBIAEoDSJkChRHZXREb2NTcGFjZXNSZXNwb25zZRIkCgpkb2Nfc3BhY2VzGAEgAygLMhAuYXBpLnYxLkRvY1NwYWNlEiYKCnBhZ2luYXRpb24YAiABKAsyEi5hcGkudjEuUGFnaW5hdGlvbiI6ChNBZGREb2NTcGFjZVJlc3BvbnNlEiMKCWRvY19zcGFjZRgBIAEoCzIQLmFwaS52MS5Eb2NTcGFjZSI7ChRFZGl0RG9jU3BhY2VSZXNwb25zZRIjCglkb2Nfc3BhY2UYASABKAsyEC5hcGkudjEuRG9jU3BhY2UiOgoTR2V0RG9jU3BhY2VSZXNwb25zZRIjCglkb2Nfc3BhY2UYASABKAsyEC5hcGkudjEuRG9jU3BhY2UyoQQKD0RvY1NwYWNlU2VydmljZRJlCgtBZGREb2NTcGFjZRIaLmFwaS52MS5BZGREb2NTcGFjZVJlcXVlc3QaGy5hcGkudjEuQWRkRG9jU3BhY2VSZXNwb25zZSIdgtPkkwIXOgEqIhIvYXBpL3YxL2RvY19zcGFjZXMSbQoMRWRpdERvY1NwYWNlEhsuYXBpLnYxLkVkaXREb2NTcGFjZVJlcXVlc3QaHC5hcGkudjEuRWRpdERvY1NwYWNlUmVzcG9uc2UiIoLT5JMCHDoBKhoXL2FwaS92MS9kb2Nfc3BhY2VzL3tpZH0SZwoLR2V0RG9jU3BhY2USGi5hcGkudjEuR2V0RG9jU3BhY2VSZXF1ZXN0GhsuYXBpLnYxLkdldERvY1NwYWNlUmVzcG9uc2UiH4LT5JMCGRIXL2FwaS92MS9kb2Nfc3BhY2VzL3tpZH0SZQoMR2V0RG9jU3BhY2VzEhsuYXBpLnYxLkdldERvY1NwYWNlc1JlcXVlc3QaHC5hcGkudjEuR2V0RG9jU3BhY2VzUmVzcG9uc2UiGoLT5JMCFBISL2FwaS92MS9kb2Nfc3BhY2VzEmgKDkRlbGV0ZURvY1NwYWNlEh0uYXBpLnYxLkRlbGV0ZURvY1NwYWNlUmVxdWVzdBoWLmdvb2dsZS5wcm90b2J1Zi5FbXB0eSIfgtPkkwIZKhcvYXBpL3YxL2RvY19zcGFjZXMve2lkfUJyCgpjb20uYXBpLnYxQg1Eb2NTcGFjZVByb3RvUAFaHG9wc2VlL2JhY2tlbmQvZ2VuL2FwaS92MTtnZW6iAgNBWFiqAgZBcGkuVjHKAgZBcGlcVjHiAhJBcGlcVjFcR1BCTWV0YWRhdGHqAgdBcGk6OlYxYgZwcm90bzM", [file_google_protobuf_empty, file_google_api_annotations, file_validate_validate, file_api_v1_pagination, file_api_v1_filter, file_api_v1_models]);
 
 /**
  * Describes the message api.v1.AddDocSpaceRequest.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@opsee/mcp-server",
-  "version": "0.9.33",
+  "version": "0.9.35",
   "description": "Opsee MCP server — manage projects, tasks, docs, and cycles from AI coding environments",
   "type": "module",
   "bin": {
@@ -38,6 +38,7 @@
   },
   "files": [
     "src",
+    "!**/__tests__",
     "bin",
     "gen"
   ]

package/src/tools/docs.ts

@@ -1,18 +1,15 @@
 import { z } from "zod";
-import { create } from "@bufbuild/protobuf";
 import type { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
 import type { ApiClients } from "../client/api.js";
+import { overridePagination } from "../client/api.js";
 import { authManager } from "../auth/manager.js";
 import {
   formatDocSpaceList,
   formatDocPageList,
   formatDocPage,
   formatError,
+  detectDocContentType,
 } from "../utils/format.js";
-import {
-  FilterOptionsSchema,
-  FilterSchema,
-} from "../../gen/api/v1/filter_pb.js";
 
 function toKebabCase(str: string): string {
   return str
@@ -33,15 +30,14 @@ export function registerDocTools(
     async ({ projectId }) => {
       try {
         const clients = getClients();
+        // project_id goes on the request itself, as with doc pages: the backend
+        // scopes the list with GetDocSpacesRequest.project_id, and a generic
+        // ProjectId filter cannot stand in for it -- scopes.ParseFilterOptions
+        // drops that key as security-sensitive, which returned every doc space
+        // in the company.
         const res = await clients.docSpaces.getDocSpaces({
-          filterOptions: create(FilterOptionsSchema, {
-            filters: [
-              create(FilterSchema, {
-                key: "ProjectId",
-                value: String(projectId),
-              }),
-            ],
-          }),
+          projectId,
+          pagination: overridePagination(),
         });
         return {
           content: [{ type: "text", text: formatDocSpaceList(res.docSpaces) }],
@@ -54,21 +50,20 @@ export function registerDocTools(
 
   server.tool(
     "opsee_list_doc_pages",
-    "List documentation pages in a doc space.",
-    { docSpaceId: z.number().int().min(1).describe("The doc space ID") },
+    "List documentation pages in an Opsee project.",
+    { projectId: z.number().int().min(1).describe("The project ID") },
     { readOnlyHint: true, destructiveHint: false },
-    async ({ docSpaceId }) => {
+    async ({ projectId }) => {
       try {
         const clients = getClients();
+        // Doc pages hang off the project, not off a doc space: model.DocPage has
+        // no doc_space_id column (only the legacy proto field), and the backend
+        // scopes the list with the request's own project_id, which the validator
+        // requires to be >= 1. A generic ProjectId filter cannot substitute for
+        // it -- scopes.ParseFilterOptions drops that key as security-sensitive.
         const res = await clients.docPages.getDocPages({
-          filterOptions: create(FilterOptionsSchema, {
-            filters: [
-              create(FilterSchema, {
-                key: "DocSpaceId",
-                value: String(docSpaceId),
-              }),
-            ],
-          }),
+          projectId,
+          pagination: overridePagination(),
         });
         return {
           content: [{ type: "text", text: formatDocPageList(res.docPages) }],
@@ -103,11 +98,17 @@ export function registerDocTools(
     {
       projectId: z.number().int().min(1).describe("The project ID"),
       title: z.string().describe("Page title"),
-      content: z.string().describe("Page content (text or JSON)"),
+      content: z.string().describe("Page content: markdown/plain text, or a BlockNote JSON array"),
+      contentType: z
+        .enum(["text", "json"])
+        .optional()
+        .describe(
+          "How the reader should interpret content: 'text' (markdown/plain) or 'json' (BlockNote array). Leave unset to detect it from the content.",
+        ),
       parentPageId: z.number().int().min(1).optional().describe("Parent page ID for nested pages"),
     },
     { readOnlyHint: false, destructiveHint: false },
-    async ({ projectId, title, content, parentPageId }) => {
+    async ({ projectId, title, content, contentType, parentPageId }) => {
       try {
         const clients = getClients();
 
@@ -123,7 +124,7 @@ export function registerDocTools(
           title,
           slug: toKebabCase(title),
           currentContent: content,
-          contentType: "json",
+          contentType: contentType ?? detectDocContentType(content),
           currentVersionNumber: 1,
           isRoot: !parentPageId,
           isArchived: false,

package/src/tools/initiatives.ts

@@ -23,6 +23,7 @@ const ANCHOR_TYPES = ["none", "task", "milestone", "cycle"] as const;
 const MEMORY_KINDS = ["decision", "outcome", "learning", "blocker", "context"] as const;
 const MEMORY_SOURCE_TYPES = ["none", "task", "pull_request", "doc", "workflow"] as const;
 const EDGE_TYPES = ["blocks", "blocked_by", "duplicates", "relates_to"] as const;
+const MEMORY_CONTENT_TYPES = ["text", "json"] as const;
 
 export function registerInitiativeTools(
   server: McpServer,
@@ -345,7 +346,7 @@ export function registerInitiativeTools(
       initiativeId: z.number().int().min(1).describe("The initiative ID"),
       kind: z.enum(MEMORY_KINDS).describe("Entry kind: decision | outcome | learning | blocker | context"),
       body: z.string().describe("The memory entry body text"),
-      contentType: z.string().optional().describe("Content type of body (e.g. 'text'). Defaults to 'text'"),
+      contentType: z.enum(MEMORY_CONTENT_TYPES).optional().describe("Content type of body: 'text' (markdown — the default) or 'json' (BlockNote). Defaults to 'text'"),
       sourceType: z.enum(MEMORY_SOURCE_TYPES).optional().describe("Source type: none | task | pull_request | doc | workflow. Defaults to 'none'"),
       sourceId: z.number().int().min(1).optional().describe("ID of the source entity (task ID, PR ID, etc.)"),
       sourceUrl: z.string().optional().describe("URL linking to the source"),

package/src/tools/milestones.ts

@@ -11,7 +11,9 @@ import {
   formatError,
 } from "../utils/format.js";
 
-const MILESTONE_STATUSES = ["not_started", "in_progress", "at_risk", "completed"] as const;
+// Must match the values AddMilestoneRequest/EditMilestoneRequest accept in
+// proto/api/v1/milestone.proto; the backend rejects anything else outright.
+const MILESTONE_STATUSES = ["Planned", "InProgress", "Completed", "Cancelled"] as const;
 
 export function registerMilestoneTools(
   server: McpServer,
@@ -59,7 +61,7 @@ export function registerMilestoneTools(
     {
       projectId: z.number().int().min(1).describe("The project ID"),
       name: z.string().describe("Milestone name"),
-      status: z.enum(MILESTONE_STATUSES).optional().describe("Milestone status. Defaults to 'not_started'"),
+      status: z.enum(MILESTONE_STATUSES).optional().describe("Milestone status. Defaults to 'Planned'"),
       description: z.string().optional().describe("Milestone description"),
       startDate: z.string().optional().describe("Start date (ISO 8601, e.g. 2026-04-01)"),
       endDate: z.string().optional().describe("End date (ISO 8601, e.g. 2026-04-08)"),
@@ -71,7 +73,7 @@ export function registerMilestoneTools(
         const res = await clients.milestones.addMilestone({
           projectId,
           name,
-          status: status ?? "not_started",
+          status: status ?? "Planned",
           description,
           startDate: startDate ? timestampFromDate(new Date(startDate)) : undefined,
           endDate: endDate ? timestampFromDate(new Date(endDate)) : undefined,

package/src/utils/format/blocknote.ts

@@ -94,6 +94,27 @@ function renderBlock(block: BNBlock, indent: string): string[] {
   return lines;
 }
 
+// Mirrors the two existing product-side detectors so a page written through MCP
+// is labelled exactly as the same bytes would be from the web UI:
+// frontend/src/pages/Docs/hooks/useDocTreeMutations.js (JSON.parse + Array.isArray)
+// and backend/service/mentions.go IsBlockNoteJSON (unmarshal into []interface{}).
+// A JSON array is the only shape BlockNote persists, so an object or a bare
+// scalar is content, not a document body.
+export function isBlockNoteJSON(content: string): boolean {
+  try {
+    return Array.isArray(JSON.parse(content));
+  } catch {
+    return false;
+  }
+}
+
+// The reader (frontend/src/pages/Docs/DocPage.jsx) only JSON-parses when
+// contentType is "json" and otherwise wraps the raw string in a paragraph, so
+// mislabelling markdown as "json" renders an empty page.
+export function detectDocContentType(content: string): "text" | "json" {
+  return isBlockNoteJSON(content) ? "json" : "text";
+}
+
 export function blockNoteToMarkdown(content: string): string {
   try {
     const parsed = JSON.parse(content);

package/src/utils/format/index.ts

@@ -17,7 +17,7 @@ export {
 export { formatUser, formatUserList } from "./user.js";
 export { formatLabel, formatLabelList, formatTaskLabelList } from "./label.js";
 export { formatComment, formatCommentList } from "./comment.js";
-export { blockNoteToMarkdown } from "./blocknote.js";
+export { blockNoteToMarkdown, isBlockNoteJSON, detectDocContentType } from "./blocknote.js";
 export {
   WORKING_AGREEMENT,
   formatInitiative,

package/src/__tests__/blocknote-fixtures.test.ts

@@ -1,58 +0,0 @@
-import { describe, test, expect } from "vitest";
-import { readdirSync, readFileSync } from "node:fs";
-import { dirname, join } from "node:path";
-import { fileURLToPath } from "node:url";
-import { blockNoteToMarkdown } from "../utils/format.js";
-
-// The BlockNote <-> markdown mapping is implemented three times against no
-// shared contract: this reader, orchestrator/internal/util/blocknote.go, and
-// the frontend BlockEditor via @blocknote/core. testdata/blocknote is the
-// language-neutral fixture set all three should eventually assert against;
-// this suite pins the TypeScript side of it.
-const FIXTURE_DIR = join(dirname(fileURLToPath(import.meta.url)), "../../../testdata/blocknote");
-
-interface Fixture {
-  name: string;
-  description: string;
-  /** Stored value when it is BlockNote JSON. Serialized before rendering. */
-  blocknote?: unknown[];
-  /** Stored value when it is not BlockNote JSON, passed through verbatim. */
-  input?: string;
-  markdown: string;
-}
-
-const fixtures = readdirSync(FIXTURE_DIR)
-  .filter((f) => f.endsWith(".json"))
-  .sort()
-  .map((f) => JSON.parse(readFileSync(join(FIXTURE_DIR, f), "utf8")) as Fixture);
-
-describe("blockNoteToMarkdown golden fixtures", () => {
-  // Naming the cases rather than counting them: a suite whose whole job is to
-  // catch drift must fail, not vacuously pass, if the fixtures go missing.
-  test("covers every construct the task-description contract promises", () => {
-    expect(fixtures.map((f) => f.name).sort()).toEqual([
-      "block-separation",
-      "checklist",
-      "code-fence",
-      "empty-document",
-      "heading-level-clamp",
-      "headings",
-      "inline-styles",
-      "links",
-      "nested-bullets",
-      "passthrough-malformed-block",
-      "passthrough-non-array-json",
-      "passthrough-plain-text",
-      "task-description",
-    ]);
-  });
-
-  for (const fixture of fixtures) {
-    test(`${fixture.name}: ${fixture.description}`, () => {
-      const isBlockNote = fixture.blocknote !== undefined;
-      expect(isBlockNote || typeof fixture.input === "string").toBe(true);
-      const stored = isBlockNote ? JSON.stringify(fixture.blocknote) : fixture.input!;
-      expect(blockNoteToMarkdown(stored)).toBe(fixture.markdown);
-    });
-  }
-});

package/src/__tests__/blocknote-markdown.test.ts

@@ -1,92 +0,0 @@
-import { describe, test, expect } from "vitest";
-import { blockNoteToMarkdown } from "../utils/format.js";
-
-const bn = (blocks: unknown[]) => JSON.stringify(blocks);
-
-const text = (t: string, styles?: Record<string, boolean>) => ({
-  type: "text",
-  text: t,
-  ...(styles ? { styles } : {}),
-});
-
-describe("blockNoteToMarkdown", () => {
-  test("renders paragraphs", () => {
-    expect(blockNoteToMarkdown(bn([{ type: "paragraph", content: [text("hello")] }]))).toBe("hello");
-  });
-
-  test("renders headings, bullets and checkboxes", () => {
-    const md = blockNoteToMarkdown(
-      bn([
-        { type: "heading", props: { level: 2 }, content: [text("Acceptance criteria")] },
-        { type: "checkListItem", props: { checked: true }, content: [text("done")] },
-        { type: "checkListItem", props: { checked: false }, content: [text("todo")] },
-      ]),
-    );
-    expect(md).toContain("## Acceptance criteria");
-    expect(md).toContain("- [x] done");
-    expect(md).toContain("- [ ] todo");
-  });
-
-  test("renders bold, italic and code inline styles", () => {
-    expect(blockNoteToMarkdown(bn([{ type: "paragraph", content: [text("a", { bold: true })] }]))).toBe("**a**");
-    expect(blockNoteToMarkdown(bn([{ type: "paragraph", content: [text("a", { italic: true })] }]))).toBe("*a*");
-    expect(blockNoteToMarkdown(bn([{ type: "paragraph", content: [text("a", { code: true })] }]))).toBe("`a`");
-  });
-
-  test("renders links", () => {
-    const md = blockNoteToMarkdown(
-      bn([{ type: "paragraph", content: [{ type: "link", href: "https://x.test", content: [text("x")] }] }]),
-    );
-    expect(md).toBe("[x](https://x.test)");
-  });
-
-  test("returns the raw string for non-BlockNote input", () => {
-    expect(blockNoteToMarkdown("just markdown")).toBe("just markdown");
-  });
-
-  // Both Go writers persist strikethrough under the BlockNote style key
-  // "strike": backend/service/task.go parseInlineMarkdown and
-  // orchestrator/internal/util/blocknote.go. This reader is what renders an
-  // initiative's coreIdea for get-initiative-context, so dropping the markers
-  // makes an agent read struck-out (abandoned) plan items as live ones.
-  test("preserves strikethrough written by the Go markdown parser", () => {
-    const md = blockNoteToMarkdown(
-      bn([
-        {
-          type: "paragraph",
-          content: [text("Ship "), text("by Friday", { strike: true }), text(" - moved to next cycle")],
-        },
-      ]),
-    );
-    expect(md).toBe("Ship ~~by Friday~~ - moved to next cycle");
-  });
-
-  // The local BNInlineStyle type declares `strikethrough`, so cover that
-  // spelling too - whichever key the pipeline uses, the marker must survive.
-  test("preserves strikethrough under the declared style key", () => {
-    const md = blockNoteToMarkdown(bn([{ type: "paragraph", content: [text("gone", { strikethrough: true })] }]));
-    expect(md).toContain("~~gone~~");
-  });
-
-  test("preserves underline as declared in BNInlineStyle", () => {
-    const md = blockNoteToMarkdown(bn([{ type: "paragraph", content: [text("u", { underline: true })] }]));
-    expect(md).not.toBe("u");
-  });
-
-  test("nests a code block inside a list item on its own indented lines", () => {
-    const md = blockNoteToMarkdown(
-      bn([
-        {
-          type: "bulletListItem",
-          content: [text("run this")],
-          children: [{ type: "codeBlock", props: { language: "sh" }, content: [text("make test\nmake lint")] }],
-        },
-      ]),
-    );
-    const lines = md.split("\n");
-    // Every line of the fenced block body must carry the nesting indent,
-    // otherwise the second command escapes the list item.
-    const bodyLines = lines.filter((l) => l.includes("make "));
-    expect(bodyLines.every((l) => l.startsWith("  "))).toBe(true);
-  });
-});

package/src/__tests__/comments.test.ts

@@ -1,193 +0,0 @@
-import { describe, test, expect, beforeAll, afterAll, beforeEach } from "vitest";
-import { Client } from "@modelcontextprotocol/sdk/client/index.js";
-import { InMemoryTransport } from "@modelcontextprotocol/sdk/inMemory.js";
-import { createServer } from "../server.js";
-import type { ApiClients } from "../client/api.js";
-
-const mockUser = { id: 5, fullName: "Alice Test" };
-
-const mockComment = {
-  id: 11,
-  content: "old text",
-  isInternal: false,
-  taskId: 10,
-  userId: 5,
-  user: mockUser,
-};
-
-// Records the last request each mock method received so tests can assert pass-through.
-let lastAddReq: any = null;
-let lastEditReq: any = null;
-let lastListReq: any = null;
-
-function createMockClients(): ApiClients {
-  lastAddReq = null;
-  lastEditReq = null;
-  lastListReq = null;
-  return {
-    users: {
-      getMe: async () => ({ user: mockUser }),
-    },
-    comments: {
-      getComments: async (req: any) => {
-        lastListReq = req;
-        return { comments: [mockComment] };
-      },
-      getComment: async () => ({ comment: mockComment }),
-      addComment: async (req: any) => {
-        lastAddReq = req;
-        return { comment: { ...mockComment, id: 12, content: req.content, isInternal: req.isInternal, userId: req.userId } };
-      },
-      editComment: async (req: any) => {
-        lastEditReq = req;
-        return { comment: { ...mockComment, content: req.content, isInternal: req.isInternal } };
-      },
-      deleteComment: async () => ({}),
-    },
-  } as unknown as ApiClients;
-}
-
-describe("Comment tools", () => {
-  let client: Client;
-  let cleanup: () => Promise<void>;
-
-  beforeAll(async () => {
-    const [ct, st] = InMemoryTransport.createLinkedPair();
-    const server = createServer(() => createMockClients());
-    await server.connect(st);
-    client = new Client({ name: "comments-test", version: "1.0.0" });
-    await client.connect(ct);
-    cleanup = async () => {
-      await client.close();
-      await server.close();
-    };
-  });
-
-  afterAll(async () => {
-    await cleanup();
-  });
-
-  beforeEach(() => {
-    lastAddReq = null;
-    lastEditReq = null;
-    lastListReq = null;
-  });
-
-  test("opsee_list_comments returns the formatted list", async () => {
-    const result = await client.callTool({
-      name: "opsee_list_comments",
-      arguments: { taskId: 10 },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("Found 1 comment(s)");
-    expect(text).toContain("old text");
-    expect(text).toContain("Alice Test");
-    expect(lastListReq.taskId).toBe(10);
-  });
-
-  test("opsee_list_comments reports none for an empty result", async () => {
-    const emptyClients = createMockClients();
-    (emptyClients.comments as any).getComments = async () => ({ comments: [] });
-
-    const [ct, st] = InMemoryTransport.createLinkedPair();
-    const server = createServer(() => emptyClients);
-    await server.connect(st);
-    const c = new Client({ name: "empty-comments-test", version: "1.0.0" });
-    await c.connect(ct);
-
-    const result = await c.callTool({
-      name: "opsee_list_comments",
-      arguments: { initiativeId: 2 },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("No comments found.");
-
-    await c.close();
-    await server.close();
-  });
-
-  test("opsee_add_comment resolves the current user via getMe and posts the comment", async () => {
-    const result = await client.callTool({
-      name: "opsee_add_comment",
-      arguments: { taskId: 10, content: "new comment" },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("Comment added");
-    expect(text).toContain("new comment");
-    expect(lastAddReq.userId).toBe(5);
-    expect(lastAddReq.isInternal).toBe(false);
-  });
-
-  test("opsee_add_comment passes isInternal through", async () => {
-    await client.callTool({
-      name: "opsee_add_comment",
-      arguments: { taskId: 10, content: "internal note", isInternal: true },
-    });
-    expect(lastAddReq.isInternal).toBe(true);
-  });
-
-  test("opsee_add_comment errors when the current user cannot be resolved", async () => {
-    const noUserClients = createMockClients();
-    (noUserClients.users as any).getMe = async () => ({});
-
-    const [ct, st] = InMemoryTransport.createLinkedPair();
-    const server = createServer(() => noUserClients);
-    await server.connect(st);
-    const c = new Client({ name: "no-user-test", version: "1.0.0" });
-    await c.connect(ct);
-
-    const result = await c.callTool({
-      name: "opsee_add_comment",
-      arguments: { taskId: 10, content: "orphan comment" },
-    });
-    expect(result.isError).toBe(true);
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("Could not resolve current user");
-
-    await c.close();
-    await server.close();
-  });
-
-  test("opsee_edit_comment looks up the existing comment before editing", async () => {
-    const result = await client.callTool({
-      name: "opsee_edit_comment",
-      arguments: { commentId: 11, content: "edited text" },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("Comment updated");
-    expect(text).toContain("edited text");
-    expect(lastEditReq.id).toBe(11);
-    expect(lastEditReq.taskId).toBe(10);
-    expect(lastEditReq.isInternal).toBe(false);
-  });
-
-  test("opsee_edit_comment reports not found when the comment does not exist", async () => {
-    const missingClients = createMockClients();
-    (missingClients.comments as any).getComment = async () => ({});
-
-    const [ct, st] = InMemoryTransport.createLinkedPair();
-    const server = createServer(() => missingClients);
-    await server.connect(st);
-    const c = new Client({ name: "missing-comment-test", version: "1.0.0" });
-    await c.connect(ct);
-
-    const result = await c.callTool({
-      name: "opsee_edit_comment",
-      arguments: { commentId: 999, content: "won't apply" },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("Comment not found.");
-
-    await c.close();
-    await server.close();
-  });
-
-  test("opsee_delete_comment succeeds", async () => {
-    const result = await client.callTool({
-      name: "opsee_delete_comment",
-      arguments: { commentId: 11 },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("Comment deleted.");
-  });
-});