@firfi/huly-mcp 0.17.1 → 0.17.2

package/dist/index.cjs

@@ -167891,7 +167891,7 @@ var parseDeleteTestResultParams = Schema_exports.decodeUnknown(DeleteTestResultP
 var parseRunTestPlanParams = Schema_exports.decodeUnknown(RunTestPlanParamsSchema);
 
 // src/version.ts
-var VERSION = true ? "0.17.1" : "0.0.0-dev";
+var VERSION = true ? "0.17.2" : "0.0.0-dev";
 
 // src/mcp/error-mapping.ts
 var McpErrorCode = {
@@ -168904,6 +168904,36 @@ var findProject = (projectIdentifier) => Effect_exports.gen(function* () {
   );
   return { client, project: project3 };
 });
+var statusInfoFromDoc = (doc) => {
+  const categoryStr = doc.category ? doc.category : "";
+  return {
+    _id: doc._id,
+    name: doc.name,
+    isDone: categoryStr === task.statusCategory.Won,
+    isCanceled: categoryStr === task.statusCategory.Lost
+  };
+};
+var fallbackStatusInfoFromRef = (statusRef) => {
+  const name = statusRef.includes(":") ? statusRef.slice(statusRef.lastIndexOf(":") + 1) : statusRef;
+  const nameLower = name.toLowerCase();
+  const isDone5 = nameLower.includes("done") || nameLower.includes("complete") || nameLower.includes("finished") || nameLower.includes("resolved") || nameLower.includes("closed");
+  const isCanceled = nameLower.includes("cancel") || nameLower.includes("reject") || nameLower.includes("abort") || nameLower.includes("wontfix") || nameLower.includes("invalid");
+  return {
+    _id: statusRef,
+    name,
+    isDone: isDone5,
+    isCanceled
+  };
+};
+var uniqueStatusRefs = (refs) => refs.reduce(
+  (unique, ref) => unique.includes(ref) ? unique : [...unique, ref],
+  []
+);
+var uniqueStatusDocs = (statuses) => Array.from(statuses).reduce(
+  (unique, status) => unique.some((existing) => existing._id === status._id) ? unique : [...unique, status],
+  []
+);
+var uniqueProjectTypeStatusRefs = (statuses) => uniqueStatusRefs(statuses.map((status) => status._id));
 var findProjectWithStatuses = (projectIdentifier) => Effect_exports.gen(function* () {
   const client = yield* HulyClient;
   const project3 = yield* findOneOrFail(
@@ -168914,45 +168944,25 @@ var findProjectWithStatuses = (projectIdentifier) => Effect_exports.gen(function
     { lookup: { type: task.class.ProjectType } }
   );
   const projectType = project3.$lookup?.type;
-  const statuses = [];
-  if (projectType?.statuses) {
-    const statusRefs = projectType.statuses.map((s) => s._id);
-    if (statusRefs.length > 0) {
-      const statusDocsResult = yield* Effect_exports.either(
-        client.findAll(
-          core.class.Status,
-          hulyQuery({ _id: { $in: statusRefs } })
-        )
-      );
-      if (statusDocsResult._tag === "Right") {
-        for (const doc of statusDocsResult.right) {
-          const categoryStr = doc.category ? doc.category : "";
-          statuses.push({
-            _id: doc._id,
-            name: doc.name,
-            isDone: categoryStr === task.statusCategory.Won,
-            isCanceled: categoryStr === task.statusCategory.Lost
-          });
-        }
-      } else {
-        yield* Effect_exports.logWarning(
-          `Status query failed for project ${projectIdentifier}, using fallback. Category-based filtering (open/done/canceled) will use name heuristics. Error: ${statusDocsResult.left.message}`
-        );
-        for (const ps of projectType.statuses) {
-          const name = ps._id.split(":").pop() ?? "Unknown";
-          const nameLower = name.toLowerCase();
-          const isDone5 = nameLower.includes("done") || nameLower.includes("complete") || nameLower.includes("finished") || nameLower.includes("resolved") || nameLower.includes("closed");
-          const isCanceled = nameLower.includes("cancel") || nameLower.includes("reject") || nameLower.includes("abort") || nameLower.includes("wontfix") || nameLower.includes("invalid");
-          statuses.push({
-            _id: ps._id,
-            name,
-            isDone: isDone5,
-            isCanceled
-          });
-        }
-      }
+  const statuses = projectType?.statuses ? yield* Effect_exports.gen(function* () {
+    const statusRefs = uniqueProjectTypeStatusRefs(projectType.statuses);
+    if (statusRefs.length === 0) {
+      return [];
     }
-  }
+    const statusDocsResult = yield* Effect_exports.either(
+      client.findAll(
+        core.class.Status,
+        hulyQuery({ _id: { $in: statusRefs } })
+      )
+    );
+    if (statusDocsResult._tag === "Right") {
+      return uniqueStatusDocs(statusDocsResult.right).map(statusInfoFromDoc);
+    }
+    yield* Effect_exports.logWarning(
+      `Status query failed for project ${projectIdentifier}, using fallback. Category-based filtering (open/done/canceled) will use name heuristics. Error: ${statusDocsResult.left.message}`
+    );
+    return statusRefs.map(fallbackStatusInfoFromRef);
+  }) : [];
   const defaultStatusId = project3.defaultIssueStatus || statuses[0]?._id;
   return { client, defaultStatusId, project: project3, projectType, statuses };
 });
@@ -177709,9 +177719,15 @@ var encodeOrConnectionError2 = (schema, value3, operation) => Schema_exports.enc
     })
   )
 );
-var uniqueStatusIds = (projectType) => Array.from(new Set(projectType.statuses.map((status) => status._id)));
+var uniqueTaskTypeRefs = (refs) => refs.reduce((unique, ref) => unique.includes(ref) ? unique : [...unique, ref], []);
+var uniqueStatusIds = (projectType) => uniqueStatusRefs(projectType.statuses.map((status) => status._id));
+var sameProjectStatus = (left3, right3) => left3._id === right3._id && left3.taskType === right3.taskType;
+var uniqueProjectStatuses = (statuses) => statuses.reduce(
+  (unique, status) => unique.some((existing) => sameProjectStatus(existing, status)) ? unique : [...unique, status],
+  []
+);
 var getStatusDocs = (client, statusIds) => statusIds.length === 0 ? Effect_exports.succeed([]) : client.findAll(core.class.Status, hulyQuery({ _id: { $in: [...statusIds] } })).pipe(
-  Effect_exports.map((result) => [...result])
+  Effect_exports.map(uniqueStatusDocs)
 );
 var getTaskTypes = (client, taskTypeIds) => taskTypeIds.length === 0 ? Effect_exports.succeed([]) : client.findAll(task.class.TaskType, hulyQuery({ _id: { $in: [...taskTypeIds] } })).pipe(
   Effect_exports.map((result) => [...result])
@@ -177743,7 +177759,11 @@ var projectTypeSummary = (data) => ({
   statusCount: uniqueStatusIds(data.projectType).length,
   isDefaultClassic: isDefaultClassicProjectType(data.projectType)
 });
-var statusTaskTypeIds = (projectType, statusId) => projectType.statuses.filter((status) => status._id === statusId).map((status) => status.taskType);
+var statusTaskTypeIds = (projectType, statusId) => uniqueTaskTypeRefs(
+  uniqueProjectStatuses(projectType.statuses).filter((status) => status._id === statusId).map(
+    (status) => status.taskType
+  )
+);
 var statusSummary = (projectType, status) => ({
   id: IssueStatusId.make(status._id),
   name: status.name,
@@ -177757,7 +177777,7 @@ var taskTypeSummary = (projectType, taskType) => ({
   projectTypeName: projectType.name,
   kind: taskType.kind,
   issueClass: taskType.ofClass,
-  statusCount: taskType.statuses.length
+  statusCount: uniqueStatusRefs(taskType.statuses).length
 });
 var projectTypeDetail = (data) => ({
   ...projectTypeSummary(data),
@@ -177769,7 +177789,7 @@ var projectTypeDetail = (data) => ({
   taskTypeStatuses: data.taskTypes.map((taskType) => ({
     taskTypeId: TaskTypeId.make(taskType._id),
     taskTypeName: taskType.name,
-    statusIds: taskType.statuses.map((statusId) => IssueStatusId.make(statusId))
+    statusIds: uniqueStatusRefs(taskType.statuses).map((statusId) => IssueStatusId.make(statusId))
   }))
 });
 var listAllProjectTypes = (client) => client.findAll(
@@ -177809,7 +177829,12 @@ var resolveStatusClass = (taskTypes) => {
   const statusClass = statusClasses.at(0);
   return statusClasses.length === 1 && statusClass !== void 0 ? Effect_exports.succeed(statusClass) : Effect_exports.fail(new HulyError({ message: "Target task types do not share one issue status class." }));
 };
-var replaceOrAppendProjectStatus = (statuses, statusId, taskTypeId) => statuses.some((status) => status._id === statusId && status.taskType === taskTypeId) ? statuses : [...statuses, { _id: statusId, taskType: taskTypeId }];
+var replaceOrAppendProjectStatus = (statuses, statusId, taskTypeId) => statuses.some((status) => status._id === statusId && status.taskType === taskTypeId) ? [...statuses] : [...statuses, { _id: statusId, taskType: taskTypeId }];
+var sameStatusRefList = (left3, right3) => left3.length === right3.length && left3.every((value3, index) => value3 === right3[index]);
+var sameProjectStatusList = (left3, right3) => left3.length === right3.length && left3.every((value3, index) => {
+  const rightValue = right3[index];
+  return sameProjectStatus(value3, rightValue);
+});
 var listProjectTypes = (_params) => Effect_exports.gen(function* () {
   const client = yield* HulyClient;
   const projectTypes = yield* listAllProjectTypes(client);
@@ -177842,29 +177867,42 @@ var createTaskType = (params) => Effect_exports.gen(function* () {
   const workflowData = yield* loadWorkflowData(client, projectType);
   const allProjectTaskTypes = yield* getTaskTypesByProjectType(client, projectType._id);
   const existing = existingTaskTypeByName(allProjectTaskTypes, params.name);
+  const normalizedProjectStatuses = uniqueProjectStatuses(projectType.statuses);
   if (existing !== void 0) {
+    const existingTaskTypeStatuses = uniqueStatusRefs(existing.statuses);
+    const taskTypeChanged = !sameStatusRefList(existingTaskTypeStatuses, existing.statuses);
     const projectTypeTasks = projectType.tasks.includes(existing._id) ? projectType.tasks : [...projectType.tasks, existing._id];
-    const existingProjectStatuses = existing.statuses.reduce(
+    const existingProjectStatuses = existingTaskTypeStatuses.reduce(
       (statuses, statusId) => replaceOrAppendProjectStatus(statuses, statusId, existing._id),
-      [...projectType.statuses]
+      normalizedProjectStatuses
     );
-    const projectTypeChanged = projectTypeTasks.length !== projectType.tasks.length || existingProjectStatuses.length !== projectType.statuses.length;
+    const projectTypeChanged = projectTypeTasks.length !== projectType.tasks.length || !sameProjectStatusList(existingProjectStatuses, projectType.statuses);
+    if (taskTypeChanged) {
+      yield* client.updateDoc(
+        task.class.TaskType,
+        core.space.Model,
+        existing._id,
+        { statuses: [...existingTaskTypeStatuses] }
+      );
+    }
     if (projectTypeChanged) {
       yield* client.updateDoc(
         task.class.ProjectType,
         core.space.Model,
         projectType._id,
-        { tasks: projectTypeTasks, statuses: existingProjectStatuses }
+        { tasks: projectTypeTasks, statuses: [...existingProjectStatuses] }
       );
     }
     const result2 = {
-      created: projectTypeChanged,
+      created: taskTypeChanged || projectTypeChanged,
       projectType: projectTypeSummary({
         projectType: { ...projectType, tasks: projectTypeTasks, statuses: existingProjectStatuses },
-        taskTypes: workflowData.taskTypes.some((taskType) => taskType._id === existing._id) ? workflowData.taskTypes : [...workflowData.taskTypes, existing],
+        taskTypes: workflowData.taskTypes.some((taskType) => taskType._id === existing._id) ? workflowData.taskTypes.map(
+          (taskType) => taskType._id === existing._id ? { ...taskType, statuses: existingTaskTypeStatuses } : taskType
+        ) : [...workflowData.taskTypes, { ...existing, statuses: existingTaskTypeStatuses }],
         statuses: workflowData.statuses
       }),
-      taskType: taskTypeSummary(projectType, existing),
+      taskType: taskTypeSummary(projectType, { ...existing, statuses: existingTaskTypeStatuses }),
       affectedTaskTypeIds: [TaskTypeId.make(existing._id)],
       warning: WORKFLOW_WARNING
     };
@@ -177879,6 +177917,7 @@ var createTaskType = (params) => Effect_exports.gen(function* () {
   const taskTypeId = (0, import_core38.generateId)();
   const targetClassId = `${taskTypeId}:type:mixin`;
   const targetClassRef = toRef(targetClassId);
+  const templateStatusIds = uniqueStatusRefs(template.statuses);
   yield* client.createDoc(
     core.class.Mixin,
     core.space.Model,
@@ -177907,7 +177946,7 @@ var createTaskType = (params) => Effect_exports.gen(function* () {
     kind: template.kind,
     ofClass: template.ofClass,
     targetClass: targetClassRef,
-    statuses: [...template.statuses],
+    statuses: templateStatusIds,
     statusClass: template.statusClass,
     statusCategories: [...template.statusCategories],
     ...template.allowedAsChildOf === void 0 ? {} : { allowedAsChildOf: template.allowedAsChildOf },
@@ -177922,18 +177961,23 @@ var createTaskType = (params) => Effect_exports.gen(function* () {
     {
       tasks: [...projectType.tasks, taskTypeId],
       statuses: [
-        ...projectType.statuses,
-        ...template.statuses.map((statusId) => ({ _id: statusId, taskType: taskTypeId }))
+        ...normalizedProjectStatuses,
+        ...templateStatusIds.map((statusId) => ({ _id: statusId, taskType: taskTypeId }))
       ]
     }
   );
+  const createdProjectStatuses = [
+    ...normalizedProjectStatuses,
+    ...templateStatusIds.map((statusId) => ({ _id: statusId, taskType: taskTypeId }))
+  ];
   const createdTaskType = {
     ...template,
     _id: taskTypeId,
     parent: projectType._id,
     name: params.name,
     kind: template.kind,
-    targetClass: targetClassRef
+    targetClass: targetClassRef,
+    statuses: templateStatusIds
   };
   const result = {
     created: true,
@@ -177941,10 +177985,7 @@ var createTaskType = (params) => Effect_exports.gen(function* () {
       projectType: {
         ...projectType,
         tasks: [...projectType.tasks, taskTypeId],
-        statuses: [
-          ...projectType.statuses,
-          ...template.statuses.map((statusId) => ({ _id: statusId, taskType: taskTypeId }))
-        ]
+        statuses: createdProjectStatuses
       },
       taskTypes: [...workflowData.taskTypes, createdTaskType],
       statuses: workflowData.statuses
@@ -177959,6 +178000,7 @@ var createIssueStatus = (params) => Effect_exports.gen(function* () {
   const client = yield* HulyClient;
   const projectType = yield* resolveProjectType(client, params.projectType);
   const workflowData = yield* loadWorkflowData(client, projectType);
+  const normalizedProjectStatuses = uniqueProjectStatuses(projectType.statuses);
   const targetTaskTypes = params.taskType === void 0 ? workflowData.taskTypes : [yield* resolveTaskType(workflowData.taskTypes, params.taskType)];
   const statusClass = yield* resolveStatusClass(targetTaskTypes);
   const statusesByName = yield* getRecoverableStatusesByName(client, params.name);
@@ -177982,28 +178024,33 @@ var createIssueStatus = (params) => Effect_exports.gen(function* () {
       statusId
     );
   }
-  const taskTypesMissingStatus = targetTaskTypes.filter((taskType) => !taskType.statuses.includes(statusId));
+  const taskTypesNeedingStatusUpdate = targetTaskTypes.filter((taskType) => {
+    const normalizedStatuses = uniqueStatusRefs(taskType.statuses);
+    return !normalizedStatuses.includes(statusId) || !sameStatusRefList(normalizedStatuses, taskType.statuses);
+  });
   yield* Effect_exports.all(
-    taskTypesMissingStatus.map(
-      (taskType) => client.updateDoc(
+    taskTypesNeedingStatusUpdate.map((taskType) => {
+      const normalizedStatuses = uniqueStatusRefs(taskType.statuses);
+      const updatedStatuses = normalizedStatuses.includes(statusId) ? normalizedStatuses : [...normalizedStatuses, statusId];
+      return client.updateDoc(
         task.class.TaskType,
         core.space.Model,
         taskType._id,
-        { statuses: [...taskType.statuses, statusId] }
-      )
-    )
+        { statuses: [...updatedStatuses] }
+      );
+    })
   );
   const updatedProjectStatuses = targetTaskTypes.reduce(
     (statuses, taskType) => replaceOrAppendProjectStatus(statuses, statusId, taskType._id),
-    [...projectType.statuses]
+    normalizedProjectStatuses
   );
-  const projectTypeChanged = updatedProjectStatuses.length !== projectType.statuses.length;
+  const projectTypeChanged = !sameProjectStatusList(updatedProjectStatuses, projectType.statuses);
   if (projectTypeChanged) {
     yield* client.updateDoc(
       task.class.ProjectType,
       core.space.Model,
       projectType._id,
-      { statuses: updatedProjectStatuses }
+      { statuses: [...updatedProjectStatuses] }
     );
   }
   const statusDoc = existingStatus ?? {
@@ -178017,7 +178064,7 @@ var createIssueStatus = (params) => Effect_exports.gen(function* () {
     category: CATEGORY_TO_REF[params.category]
   };
   const result = {
-    created: existingStatus === void 0 || taskTypesMissingStatus.length > 0 || projectTypeChanged,
+    created: existingStatus === void 0 || taskTypesNeedingStatusUpdate.length > 0 || projectTypeChanged,
     projectType: projectTypeSummary({
       projectType: { ...projectType, statuses: updatedProjectStatuses },
       taskTypes: workflowData.taskTypes,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@firfi/huly-mcp",
-  "version": "0.17.1",
+  "version": "0.17.2",
   "description": "MCP server for Huly integration",
   "mcpName": "io.github.dearlordylord/huly-mcp",
   "type": "module",