@opsee/mcp-server 0.9.54 → 0.9.56

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

@@ -164,6 +164,30 @@ export declare type DeleteCycleRequest = Message<"api.v1.DeleteCycleRequest"> &
  */
 export declare const DeleteCycleRequestSchema: GenMessage<DeleteCycleRequest>;
 
+/**
+ * @generated from message api.v1.CloseCycleRequest
+ */
+export declare type CloseCycleRequest = Message<"api.v1.CloseCycleRequest"> & {
+  /**
+   * @generated from field: uint32 id = 1;
+   */
+  id: number;
+
+  /**
+   * Cycle the incomplete tasks are carried forward into. Must belong to the
+   * same project. Omit (or 0) to close without moving anything.
+   *
+   * @generated from field: optional uint32 target_cycle_id = 2;
+   */
+  targetCycleId?: number;
+};
+
+/**
+ * Describes the message api.v1.CloseCycleRequest.
+ * Use `create(CloseCycleRequestSchema)` to create a new message.
+ */
+export declare const CloseCycleRequestSchema: GenMessage<CloseCycleRequest>;
+
 /**
  * @generated from message api.v1.AddCycleResponse
  */
@@ -233,6 +257,31 @@ export declare type GetCyclesResponse = Message<"api.v1.GetCyclesResponse"> & {
  */
 export declare const GetCyclesResponseSchema: GenMessage<GetCyclesResponse>;
 
+/**
+ * @generated from message api.v1.CloseCycleResponse
+ */
+export declare type CloseCycleResponse = Message<"api.v1.CloseCycleResponse"> & {
+  /**
+   * The closed cycle, with is_active already flipped to false.
+   *
+   * @generated from field: api.v1.Cycle cycle = 1;
+   */
+  cycle?: Cycle;
+
+  /**
+   * How many incomplete tasks were carried into target_cycle_id.
+   *
+   * @generated from field: int32 moved_task_count = 2;
+   */
+  movedTaskCount: number;
+};
+
+/**
+ * Describes the message api.v1.CloseCycleResponse.
+ * Use `create(CloseCycleResponseSchema)` to create a new message.
+ */
+export declare const CloseCycleResponseSchema: GenMessage<CloseCycleResponse>;
+
 /**
  * @generated from service api.v1.CycleService
  */
@@ -277,5 +326,13 @@ export declare const CycleService: GenService<{
     input: typeof DeleteCycleRequestSchema;
     output: typeof EmptySchema;
   },
+  /**
+   * @generated from rpc api.v1.CycleService.CloseCycle
+   */
+  closeCycle: {
+    methodKind: "unary";
+    input: typeof CloseCycleRequestSchema;
+    output: typeof CloseCycleResponseSchema;
+  },
 }>;
 

package/gen/api/v1/cycle_pb.js

@@ -14,7 +14,7 @@ import { file_api_v1_models } from "./models_pb";
  * Describes the file api/v1/cycle.proto.
  */
 export const file_api_v1_cycle = /*@__PURE__*/
-  fileDesc("ChJhcGkvdjEvY3ljbGUucHJvdG8SBmFwaS52MSKIAgoPQWRkQ3ljbGVSZXF1ZXN0EhcKBG5hbWUYASABKAlCCfpCBnIEEAEYZBIbCgRnb2FsGAIgASgJQgj6QgVyAxj0A0gAiAEBEi4KCnN0YXJ0X2RhdGUYAyABKAsyGi5nb29nbGUucHJvdG9idWYuVGltZXN0YW1wEiwKCGVuZF9kYXRlGAQgASgLMhouZ29vZ2xlLnByb3RvYnVmLlRpbWVzdGFtcBIbCgpwcm9qZWN0X2lkGAUgASgNQgf6QgQqAigBEhEKCWlzX2FjdGl2ZRgGIAEoCBIYCgtkZXNjcmlwdGlvbhgHIAEoCUgBiAEBQgcKBV9nb2FsQg4KDF9kZXNjcmlwdGlvbiKBAgoQRWRpdEN5Y2xlUmVxdWVzdBITCgJpZBgBIAEoDUIH+kIEKgIoARIXCgRuYW1lGAIgASgJQgn6QgZyBBABGGQSGwoEZ29hbBgDIAEoCUII+kIFcgMY9ANIAIgBARIuCgpzdGFydF9kYXRlGAQgASgLMhouZ29vZ2xlLnByb3RvYnVmLlRpbWVzdGFtcBIsCghlbmRfZGF0ZRgFIAEoCzIaLmdvb2dsZS5wcm90b2J1Zi5UaW1lc3RhbXASEQoJaXNfYWN0aXZlGAYgASgIEhgKC2Rlc2NyaXB0aW9uGAcgASgJSAGIAQFCBwoFX2dvYWxCDgoMX2Rlc2NyaXB0aW9uIh0KD0dldEN5Y2xlUmVxdWVzdBIKCgJpZBgBIAEoDSKQAQoQR2V0Q3ljbGVzUmVxdWVzdBIbCgpwcm9qZWN0X2lkGAEgASgNQgf6QgQqAigBEjAKCnBhZ2luYXRpb24YAiABKAsyEi5hcGkudjEuUGFnaW5hdGlvbkII+kIFigECEAESLQoOZmlsdGVyX29wdGlvbnMYAyABKAsyFS5hcGkudjEuRmlsdGVyT3B0aW9ucyIgChJEZWxldGVDeWNsZVJlcXVlc3QSCgoCaWQYASABKA0iMAoQQWRkQ3ljbGVSZXNwb25zZRIcCgVjeWNsZRgBIAEoCzINLmFwaS52MS5DeWNsZSIxChFFZGl0Q3ljbGVSZXNwb25zZRIcCgVjeWNsZRgBIAEoCzINLmFwaS52MS5DeWNsZSIwChBHZXRDeWNsZVJlc3BvbnNlEhwKBWN5Y2xlGAEgASgLMg0uYXBpLnYxLkN5Y2xlIloKEUdldEN5Y2xlc1Jlc3BvbnNlEh0KBmN5Y2xlcxgBIAMoCzINLmFwaS52MS5DeWNsZRImCgpwYWdpbmF0aW9uGAIgASgLMhIuYXBpLnYxLlBhZ2luYXRpb24y4AMKDEN5Y2xlU2VydmljZRJYCghBZGRDeWNsZRIXLmFwaS52MS5BZGRDeWNsZVJlcXVlc3QaGC5hcGkudjEuQWRkQ3ljbGVSZXNwb25zZSIZgtPkkwITOgEqIg4vYXBpL3YxL2N5Y2xlcxJgCglFZGl0Q3ljbGUSGC5hcGkudjEuRWRpdEN5Y2xlUmVxdWVzdBoZLmFwaS52MS5FZGl0Q3ljbGVSZXNwb25zZSIegtPkkwIYOgEqGhMvYXBpL3YxL2N5Y2xlcy97aWR9EloKCEdldEN5Y2xlEhcuYXBpLnYxLkdldEN5Y2xlUmVxdWVzdBoYLmFwaS52MS5HZXRDeWNsZVJlc3BvbnNlIhuC0+STAhUSEy9hcGkvdjEvY3ljbGVzL3tpZH0SWAoJR2V0Q3ljbGVzEhguYXBpLnYxLkdldEN5Y2xlc1JlcXVlc3QaGS5hcGkudjEuR2V0Q3ljbGVzUmVzcG9uc2UiFoLT5JMCEBIOL2FwaS92MS9jeWNsZXMSXgoLRGVsZXRlQ3ljbGUSGi5hcGkudjEuRGVsZXRlQ3ljbGVSZXF1ZXN0GhYuZ29vZ2xlLnByb3RvYnVmLkVtcHR5IhuC0+STAhUqEy9hcGkvdjEvY3ljbGVzL3tpZH1CbwoKY29tLmFwaS52MUIKQ3ljbGVQcm90b1ABWhxvcHNlZS9iYWNrZW5kL2dlbi9hcGkvdjE7Z2VuogIDQVhYqgIGQXBpLlYxygIGQXBpXFYx4gISQXBpXFYxXEdQQk1ldGFkYXRh6gIHQXBpOjpWMWIGcHJvdG8z", [file_google_protobuf_timestamp, file_google_protobuf_empty, file_google_api_annotations, file_validate_validate, file_api_v1_pagination, file_api_v1_filter, file_api_v1_models]);
+  fileDesc("ChJhcGkvdjEvY3ljbGUucHJvdG8SBmFwaS52MSKIAgoPQWRkQ3ljbGVSZXF1ZXN0EhcKBG5hbWUYASABKAlCCfpCBnIEEAEYZBIbCgRnb2FsGAIgASgJQgj6QgVyAxj0A0gAiAEBEi4KCnN0YXJ0X2RhdGUYAyABKAsyGi5nb29nbGUucHJvdG9idWYuVGltZXN0YW1wEiwKCGVuZF9kYXRlGAQgASgLMhouZ29vZ2xlLnByb3RvYnVmLlRpbWVzdGFtcBIbCgpwcm9qZWN0X2lkGAUgASgNQgf6QgQqAigBEhEKCWlzX2FjdGl2ZRgGIAEoCBIYCgtkZXNjcmlwdGlvbhgHIAEoCUgBiAEBQgcKBV9nb2FsQg4KDF9kZXNjcmlwdGlvbiKBAgoQRWRpdEN5Y2xlUmVxdWVzdBITCgJpZBgBIAEoDUIH+kIEKgIoARIXCgRuYW1lGAIgASgJQgn6QgZyBBABGGQSGwoEZ29hbBgDIAEoCUII+kIFcgMY9ANIAIgBARIuCgpzdGFydF9kYXRlGAQgASgLMhouZ29vZ2xlLnByb3RvYnVmLlRpbWVzdGFtcBIsCghlbmRfZGF0ZRgFIAEoCzIaLmdvb2dsZS5wcm90b2J1Zi5UaW1lc3RhbXASEQoJaXNfYWN0aXZlGAYgASgIEhgKC2Rlc2NyaXB0aW9uGAcgASgJSAGIAQFCBwoFX2dvYWxCDgoMX2Rlc2NyaXB0aW9uIh0KD0dldEN5Y2xlUmVxdWVzdBIKCgJpZBgBIAEoDSKQAQoQR2V0Q3ljbGVzUmVxdWVzdBIbCgpwcm9qZWN0X2lkGAEgASgNQgf6QgQqAigBEjAKCnBhZ2luYXRpb24YAiABKAsyEi5hcGkudjEuUGFnaW5hdGlvbkII+kIFigECEAESLQoOZmlsdGVyX29wdGlvbnMYAyABKAsyFS5hcGkudjEuRmlsdGVyT3B0aW9ucyIgChJEZWxldGVDeWNsZVJlcXVlc3QSCgoCaWQYASABKA0iWgoRQ2xvc2VDeWNsZVJlcXVlc3QSEwoCaWQYASABKA1CB/pCBCoCKAESHAoPdGFyZ2V0X2N5Y2xlX2lkGAIgASgNSACIAQFCEgoQX3RhcmdldF9jeWNsZV9pZCIwChBBZGRDeWNsZVJlc3BvbnNlEhwKBWN5Y2xlGAEgASgLMg0uYXBpLnYxLkN5Y2xlIjEKEUVkaXRDeWNsZVJlc3BvbnNlEhwKBWN5Y2xlGAEgASgLMg0uYXBpLnYxLkN5Y2xlIjAKEEdldEN5Y2xlUmVzcG9uc2USHAoFY3ljbGUYASABKAsyDS5hcGkudjEuQ3ljbGUiWgoRR2V0Q3ljbGVzUmVzcG9uc2USHQoGY3ljbGVzGAEgAygLMg0uYXBpLnYxLkN5Y2xlEiYKCnBhZ2luYXRpb24YAiABKAsyEi5hcGkudjEuUGFnaW5hdGlvbiJMChJDbG9zZUN5Y2xlUmVzcG9uc2USHAoFY3ljbGUYASABKAsyDS5hcGkudjEuQ3ljbGUSGAoQbW92ZWRfdGFza19jb3VudBgCIAEoBTLLBAoMQ3ljbGVTZXJ2aWNlElgKCEFkZEN5Y2xlEhcuYXBpLnYxLkFkZEN5Y2xlUmVxdWVzdBoYLmFwaS52MS5BZGRDeWNsZVJlc3BvbnNlIhmC0+STAhM6ASoiDi9hcGkvdjEvY3ljbGVzEmAKCUVkaXRDeWNsZRIYLmFwaS52MS5FZGl0Q3ljbGVSZXF1ZXN0GhkuYXBpLnYxLkVkaXRDeWNsZVJlc3BvbnNlIh6C0+STAhg6ASoaEy9hcGkvdjEvY3ljbGVzL3tpZH0SWgoIR2V0Q3ljbGUSFy5hcGkudjEuR2V0Q3ljbGVSZXF1ZXN0GhguYXBpLnYxLkdldEN5Y2xlUmVzcG9uc2UiG4LT5JMCFRITL2FwaS92MS9jeWNsZXMve2lkfRJYCglHZXRDeWNsZXMSGC5hcGkudjEuR2V0Q3ljbGVzUmVxdWVzdBoZLmFwaS52MS5HZXRDeWNsZXNSZXNwb25zZSIWgtPkkwIQEg4vYXBpL3YxL2N5Y2xlcxJeCgtEZWxldGVDeWNsZRIaLmFwaS52MS5EZWxldGVDeWNsZVJlcXVlc3QaFi5nb29nbGUucHJvdG9idWYuRW1wdHkiG4LT5JMCFSoTL2FwaS92MS9jeWNsZXMve2lkfRJpCgpDbG9zZUN5Y2xlEhkuYXBpLnYxLkNsb3NlQ3ljbGVSZXF1ZXN0GhouYXBpLnYxLkNsb3NlQ3ljbGVSZXNwb25zZSIkgtPkkwIeOgEqIhkvYXBpL3YxL2N5Y2xlcy97aWR9L2Nsb3NlQm8KCmNvbS5hcGkudjFCCkN5Y2xlUHJvdG9QAVocb3BzZWUvYmFja2VuZC9nZW4vYXBpL3YxO2dlbqICA0FYWKoCBkFwaS5WMcoCBkFwaVxWMeICEkFwaVxWMVxHUEJNZXRhZGF0YeoCB0FwaTo6VjFiBnByb3RvMw", [file_google_protobuf_timestamp, 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.AddCycleRequest.
@@ -51,33 +51,47 @@ export const GetCyclesRequestSchema = /*@__PURE__*/
 export const DeleteCycleRequestSchema = /*@__PURE__*/
   messageDesc(file_api_v1_cycle, 4);
 
+/**
+ * Describes the message api.v1.CloseCycleRequest.
+ * Use `create(CloseCycleRequestSchema)` to create a new message.
+ */
+export const CloseCycleRequestSchema = /*@__PURE__*/
+  messageDesc(file_api_v1_cycle, 5);
+
 /**
  * Describes the message api.v1.AddCycleResponse.
  * Use `create(AddCycleResponseSchema)` to create a new message.
  */
 export const AddCycleResponseSchema = /*@__PURE__*/
-  messageDesc(file_api_v1_cycle, 5);
+  messageDesc(file_api_v1_cycle, 6);
 
 /**
  * Describes the message api.v1.EditCycleResponse.
  * Use `create(EditCycleResponseSchema)` to create a new message.
  */
 export const EditCycleResponseSchema = /*@__PURE__*/
-  messageDesc(file_api_v1_cycle, 6);
+  messageDesc(file_api_v1_cycle, 7);
 
 /**
  * Describes the message api.v1.GetCycleResponse.
  * Use `create(GetCycleResponseSchema)` to create a new message.
  */
 export const GetCycleResponseSchema = /*@__PURE__*/
-  messageDesc(file_api_v1_cycle, 7);
+  messageDesc(file_api_v1_cycle, 8);
 
 /**
  * Describes the message api.v1.GetCyclesResponse.
  * Use `create(GetCyclesResponseSchema)` to create a new message.
  */
 export const GetCyclesResponseSchema = /*@__PURE__*/
-  messageDesc(file_api_v1_cycle, 8);
+  messageDesc(file_api_v1_cycle, 9);
+
+/**
+ * Describes the message api.v1.CloseCycleResponse.
+ * Use `create(CloseCycleResponseSchema)` to create a new message.
+ */
+export const CloseCycleResponseSchema = /*@__PURE__*/
+  messageDesc(file_api_v1_cycle, 10);
 
 /**
  * @generated from service api.v1.CycleService

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@opsee/mcp-server",
-  "version": "0.9.54",
+  "version": "0.9.56",
   "description": "Opsee MCP server — manage projects, tasks, docs, and cycles from AI coding environments",
   "type": "module",
   "bin": {

package/src/tools/cycles.ts

@@ -1,14 +1,8 @@
 import { z } from "zod";
-import { create } from "@bufbuild/protobuf";
 import { timestampFromDate } from "@bufbuild/protobuf/wkt";
 import type { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
 import type { ApiClients } from "../client/api.js";
-import { defaultPagination } from "../client/api.js";
 import { formatCycleList, formatCycle, formatError } from "../utils/format.js";
-import {
-  FilterOptionsSchema,
-  FilterSchema,
-} from "../../gen/api/v1/filter_pb.js";
 
 export function registerCycleTools(
   server: McpServer,
@@ -132,10 +126,10 @@ export function registerCycleTools(
 
   server.tool(
     "opsee_close_cycle",
-    "Close a cycle (mark inactive) and optionally carry every remaining task forward to a target cycle. Composes a single bulk_move_to_cycle + EditCycle, so /tpm cycle close becomes one call instead of a loop. Pass createNextCycle:true with name/startDate/endDate to also create the next cycle and carry forward into it in the same call.",
+    "Close a cycle (mark inactive) and carry its UNFINISHED tasks forward to a target cycle. Tasks in a done/archived column stay behind so the closed cycle keeps an accurate completion record. Pass createNextCycle:true with name/startDate/endDate to also create the next cycle and carry forward into it in the same call. Omit both target options to close without moving anything.",
     {
       cycleId: z.number().int().min(1).describe("ID of the cycle to close"),
-      nextCycleId: z.number().int().min(1).optional().describe("Existing cycle to carry remaining tasks into. Mutually exclusive with createNextCycle."),
+      nextCycleId: z.number().int().min(1).optional().describe("Existing cycle to carry unfinished tasks into. Must be in the same project. Mutually exclusive with createNextCycle."),
       createNextCycle: z.boolean().optional().describe("If true, create the next cycle in this project (requires name/startDate/endDate) and carry tasks into it"),
       nextCycleName: z.string().optional().describe("Name of the cycle to create when createNextCycle:true"),
       nextCycleStartDate: z.string().optional().describe("ISO 8601 start date when createNextCycle:true"),
@@ -146,14 +140,9 @@ export function registerCycleTools(
       try {
         const clients = getClients();
 
-        const cycleRes = await clients.cycles.getCycle({ id: cycleId });
-        const existing = cycleRes.cycle;
-        if (!existing) {
-          return { content: [{ type: "text", text: "Cycle not found." }] };
-        }
-        const projectId = existing.projectId;
-
-        // Resolve the target cycle for carry-forward.
+        // Resolve the target cycle for carry-forward. Creating one still needs a
+        // round-trip; the close itself is a single backend call that filters the
+        // unfinished tasks, moves them, and deactivates the cycle in one place.
         let targetCycleId: number | undefined = nextCycleId;
         let createdNextCycle: { id: number; name: string } | undefined;
         if (createNextCycle) {
@@ -163,8 +152,15 @@ export function registerCycleTools(
           if (!nextCycleName || !nextCycleStartDate || !nextCycleEndDate) {
             return { content: [{ type: "text", text: "createNextCycle requires nextCycleName, nextCycleStartDate, nextCycleEndDate." }], isError: true };
           }
+
+          const cycleRes = await clients.cycles.getCycle({ id: cycleId });
+          const existing = cycleRes.cycle;
+          if (!existing) {
+            return { content: [{ type: "text", text: "Cycle not found." }] };
+          }
+
           const addRes = await clients.cycles.addCycle({
-            projectId,
+            projectId: existing.projectId,
             name: nextCycleName,
             goal: "",
             startDate: timestampFromDate(new Date(nextCycleStartDate)),
@@ -179,49 +175,18 @@ export function registerCycleTools(
           targetCycleId = addRes.cycle.id;
         }
 
-        // Carry forward all tasks currently in the cycle, if a target was provided.
-        let carriedForward = 0;
-        if (targetCycleId) {
-          const tasksRes = await clients.tasks.getTasks({
-            projectId,
-            pagination: defaultPagination(1, 500),
-            filterOptions: create(FilterOptionsSchema, {
-              filters: [create(FilterSchema, { key: "CycleId", value: String(cycleId) })],
-            }),
-            labelIds: [],
-          });
-          const taskIds = (tasksRes.tasks ?? []).map((t: { id: number }) => t.id);
-          if (taskIds.length > 0) {
-            const bulkRes = await clients.tasks.bulkEditTasks({
-              projectId,
-              taskIds,
-              cycleId: targetCycleId,
-              repositoryIds: [],
-              labelIds: [],
-            });
-            carriedForward = bulkRes.updatedCount;
-          }
-        }
-
-        // Close the cycle by marking it inactive.
-        const editRes = await clients.cycles.editCycle({
-          id: existing.id,
-          name: existing.name,
-          goal: existing.goal,
-          startDate: existing.startDate,
-          endDate: existing.endDate,
-          isActive: false,
-          description: existing.description,
-        });
-        if (!editRes.cycle) {
-          return { content: [{ type: "text", text: "Failed to close cycle. Carry-forward may have partially applied; check opsee_list_tasks." }], isError: true };
+        const res = await clients.cycles.closeCycle({ id: cycleId, targetCycleId });
+        if (!res.cycle) {
+          return { content: [{ type: "text", text: "Failed to close cycle. The backend returned an empty response (no error). Retry; if it persists, surface the call to the Opsee team." }], isError: true };
         }
 
-        const lines = [`Cycle "${existing.name}" (ID ${existing.id}) closed.`];
-        if (carriedForward > 0 && targetCycleId) {
-          lines.push(`Carried ${carriedForward} task(s) forward to cycle ${targetCycleId}${createdNextCycle ? ` ("${createdNextCycle.name}")` : ""}.`);
-        } else if (targetCycleId) {
-          lines.push("No tasks to carry forward.");
+        const lines = [`Cycle "${res.cycle.name}" (ID ${res.cycle.id}) closed.`];
+        if (targetCycleId) {
+          lines.push(
+            res.movedTaskCount > 0
+              ? `Carried ${res.movedTaskCount} unfinished task(s) forward to cycle ${targetCycleId}${createdNextCycle ? ` ("${createdNextCycle.name}")` : ""}.`
+              : "No unfinished tasks to carry forward.",
+          );
         }
         return { content: [{ type: "text", text: lines.join("\n") }] };
       } catch (error) {