@bdsqqq/lnr-cli 1.3.0 → 1.4.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bdsqqq/lnr-cli",
-  "version": "1.3.0",
+  "version": "1.4.0",
   "description": "cli for linear issue tracking",
   "type": "module",
   "private": false,

package/src/router/issues.ts

@@ -21,6 +21,7 @@ import {
   createReaction,
   deleteReaction,
   createIssueRelation,
+  getProject,
   type Issue,
   type ListIssuesFilter,
   type Comment,
@@ -65,6 +66,7 @@ const issueInput = z.object({
   team: z.string().optional().describe("team key (required for new)"),
   title: z.string().optional().describe("issue title (required for new)"),
   description: z.string().optional().describe("issue description"),
+  project: z.string().optional().describe("project name to add issue to"),
   comments: z.boolean().optional().describe("list comments on issue"),
   editComment: z.string().optional().describe("comment id to edit (requires --text)"),
   text: z.string().optional().describe("text for --edit-comment or --reply-to"),
@@ -228,61 +230,27 @@ async function handleUpdateIssue(
       exitWithError(`issue ${identifier} not found`, undefined, EXIT_CODES.NOT_FOUND);
     }
 
-    if (input.comment) {
-      await addComment(client, issue.id, input.comment);
-      console.log(`commented on ${identifier}`);
-      return;
+    // Upfront validation: required flags
+    if (input.editComment && !input.text) {
+      exitWithError("--text is required with --edit-comment");
     }
-
-    if (input.editComment) {
-      if (!input.text) {
-        exitWithError("--text is required with --edit-comment");
-      }
-      await updateComment(client, input.editComment, input.text);
-      console.log(`updated comment ${input.editComment.slice(0, 8)}`);
-      return;
+    if (input.replyTo && !input.text) {
+      exitWithError("--text is required with --reply-to");
     }
-
-    if (input.replyTo) {
-      if (!input.text) {
-        exitWithError("--text is required with --reply-to");
-      }
-      await replyToComment(client, issue.id, input.replyTo, input.text);
-      console.log(`replied to comment ${input.replyTo.slice(0, 8)}`);
-      return;
+    if (input.react && !input.emoji) {
+      exitWithError("--emoji is required with --react");
     }
 
-    if (input.deleteComment) {
-      await deleteComment(client, input.deleteComment);
-      console.log(`deleted comment ${input.deleteComment.slice(0, 8)}`);
-      return;
+    // Upfront validation: mutual exclusivity for comment operations
+    const commentOpCount = [input.comment, input.editComment, input.replyTo, input.deleteComment].filter(Boolean).length;
+    if (commentOpCount > 1) {
+      exitWithError("only one comment operation allowed per invocation", "use --comment, --edit-comment, --reply-to, or --delete-comment separately");
     }
 
-    if (input.archive) {
-      await archiveIssue(client, issue.id);
-      console.log(`archived ${identifier}`);
-      return;
-    }
-
-    if (input.react) {
-      if (!input.emoji) {
-        exitWithError("--emoji is required with --react");
-      }
-      const success = await createReaction(client, input.react, input.emoji);
-      if (!success) {
-        exitWithError(`failed to add reaction to comment ${input.react.slice(0, 8)}`);
-      }
-      console.log(`added reaction ${input.emoji} to comment ${input.react.slice(0, 8)}`);
-      return;
-    }
-
-    if (input.unreact) {
-      const success = await deleteReaction(client, input.unreact);
-      if (!success) {
-        exitWithError(`reaction ${input.unreact.slice(0, 8)} not found`, undefined, EXIT_CODES.NOT_FOUND);
-      }
-      console.log(`removed reaction ${input.unreact.slice(0, 8)}`);
-      return;
+    // Upfront validation: mutual exclusivity for reaction operations
+    const reactionOpCount = [input.react, input.unreact].filter(Boolean).length;
+    if (reactionOpCount > 1) {
+      exitWithError("only one reaction operation allowed per invocation", "use --react or --unreact separately");
     }
 
     const updatePayload: Record<string, unknown> = {};
@@ -421,6 +389,50 @@ async function handleUpdateIssue(
       }
       console.log(`${identifier} now relates to ${input.relatesTo}`);
     }
+
+    // Comment operations (mutually exclusive, validated above)
+    if (input.comment) {
+      await addComment(client, issue.id, input.comment);
+      console.log(`commented on ${identifier}`);
+    }
+
+    if (input.editComment) {
+      await updateComment(client, input.editComment, input.text!);
+      console.log(`updated comment ${input.editComment.slice(0, 8)}`);
+    }
+
+    if (input.replyTo) {
+      await replyToComment(client, issue.id, input.replyTo, input.text!);
+      console.log(`replied to comment ${input.replyTo.slice(0, 8)}`);
+    }
+
+    if (input.deleteComment) {
+      await deleteComment(client, input.deleteComment);
+      console.log(`deleted comment ${input.deleteComment.slice(0, 8)}`);
+    }
+
+    // Reaction operations (mutually exclusive, validated above)
+    if (input.react) {
+      const success = await createReaction(client, input.react, input.emoji!);
+      if (!success) {
+        exitWithError(`failed to add reaction to comment ${input.react.slice(0, 8)}`);
+      }
+      console.log(`added reaction ${input.emoji} to comment ${input.react.slice(0, 8)}`);
+    }
+
+    if (input.unreact) {
+      const success = await deleteReaction(client, input.unreact);
+      if (!success) {
+        exitWithError(`reaction ${input.unreact.slice(0, 8)} not found`, undefined, EXIT_CODES.NOT_FOUND);
+      }
+      console.log(`removed reaction ${input.unreact.slice(0, 8)}`);
+    }
+
+    // Archive last (restricts further edits)
+    if (input.archive) {
+      await archiveIssue(client, issue.id);
+      console.log(`archived ${identifier}`);
+    }
   } catch (error) {
     handleApiError(error);
   }
@@ -452,6 +464,7 @@ async function handleCreateIssue(input: IssueInput): Promise<void> {
       priority?: number;
       labelIds?: string[];
       parentId?: string;
+      projectId?: string;
     } = {
       teamId: team.id,
       title: input.title,
@@ -499,6 +512,14 @@ async function handleCreateIssue(input: IssueInput): Promise<void> {
       createPayload.parentId = parentIssue.id;
     }
 
+    if (input.project) {
+      const project = await getProject(client, input.project);
+      if (!project) {
+        exitWithError(`project "${input.project}" not found`);
+      }
+      createPayload.projectId = project.id;
+    }
+
     const issue = await createIssue(client, createPayload);
     if (issue) {
       console.log(`created ${issue.identifier}: ${issue.title}`);