baton-issue-tracker 1.8.0 → 1.10.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "baton-issue-tracker",
-  "version": "1.8.0",
+  "version": "1.10.0",
   "description": "A CLI issue tracker for AI agents",
   "type": "module",
   "bin": {

package/source/cli.js

@@ -18,6 +18,7 @@ import { run as runNext } from './commands/next.js';
 import { run as runLoop } from './commands/loop.js';
 import { run as runStatus } from './commands/status.js';
 import { run as runApprove } from './commands/approve.js';
+import { run as runReject } from './commands/reject.js';
 import { wantsHelp } from './util.js';
 import { run as runView} from './commands/view.js';
 import { run as runSearch } from './commands/search.js';
@@ -28,6 +29,7 @@ import { run as runDelete } from './commands/delete.js';
 import { run as runPriority } from './commands/priority.js';
 import { run as runLog } from './commands/log.js';
 import { run as runRegister } from './commands/register.js';
+import { run as runSubmit } from './commands/submit.js';
 
 import { authenticateContext } from './services/authService.js';
 
@@ -47,6 +49,7 @@ Commands:
   list     Lists issues filtered by status and priority
   create   Creates an issue with specified fields
   approve  Move an issue from in-review to closed
+  submit   Submit finished work for human review
   priority Set an issue's priority level
   update   Updates an issue's specified fields
   delete   Deletes an issue
@@ -78,6 +81,8 @@ Options:
   create --token-limit <n>        Optional token budget for this issue
   create --json                   Output as JSON (for AI agents)
   approve <id> [--json]
+  submit <id> [--json]
+  reject <id> --reason <text>     Reject an issue with a given reason
   priority <id> <level> [--json]  low | medium | high
   update --title <text>           New title
   update --description <text>     New description
@@ -105,6 +110,7 @@ Examples:
   baton create --title "Fix login bug" --priority high
   baton create --title "Refactor auth" --description "Clean up JWT logic" --token-limit 4000
   baton approve 5
+  baton submit 14
   baton priority 5 high
   baton priority 3 low
   baton update 3 --title "Revised title"
@@ -138,11 +144,13 @@ async function main() {
     search: () => runSearch(args),
     list: () => runList(args),
     approve: () => runApprove(args),
+    reject: () => runReject(args),
     priority: () => runPriority(args),
     create: () => runCreate(args),
     update: () => runUpdate(args),
     delete: () => runDelete(args),
     log: () => runLog(args),
+    submit: () => runSubmit(args),
   };
 
   const handler = handlers[command];

package/source/commands/reject.js

@@ -0,0 +1,99 @@
+// reject.js
+// supports rejecting an issue.
+// 
+// AI Was used in generating this file.
+// Usage:
+//   baton reject <id> [options]
+//
+// Options:
+//   --reason <text>        Reason for rejection (required)
+//   --json                 Output in JSON format
+//   -h, --help             Show this help
+//
+// Examples:
+//   baton reject 5 --reason "Needs more detail"
+
+import { rejectIssue, getIssue } from '../services/issuesService.js';
+import { Status } from '../models/issue.js';
+import { hasFlag, getFlagValue, wantsHelp, renderOutput, renderError, serializeIssue } from '../util.js';
+
+const USAGE = "Usage:\n  baton reject <id> [options]\n\nOptions:\n  --reason <text>        Reason for rejection (required)\n  --json                 Output in JSON format\n  -h, --help             Show this help";
+
+/**
+ * Rejects an issue for a specified ID.
+ * 
+ * @param {string[]} args - The command line arguments
+ * @returns {Promise<number>} The exit code: 0 is success, 1 is error.
+ */
+export async function run(args) {
+    const isJson = hasFlag(args, '--json');
+
+    // (0) Help check
+    if (wantsHelp(args)) {
+        console.log(USAGE);
+        return 0;
+    }
+
+    // (1) Parse arguments
+    const idArgs = args.filter(arg => !arg.startsWith('-'));
+    if (idArgs.length === 0) {
+        renderError(isJson, `No ID provided.\n${USAGE}`, 'MISSING_ID');
+        return 1;
+    }
+
+    const id = Number(idArgs[0]);
+    if (!Number.isInteger(id)) {
+        renderError(isJson, `Invalid ID "${idArgs[0]}". ID must be an integer.`, 'INVALID_ID');
+        return 1;
+    }
+
+    if (!hasFlag(args, "--reason")) {
+        renderError(isJson, `Missing reason for reject.\n${USAGE}`, 'MISSING_REASON');
+        return 1;
+    }
+
+    let reasonText;
+    try {
+        reasonText = getFlagValue(args, "--reason");
+    } catch (error) {
+        renderError(isJson, error.message, 'INVALID_REASON');
+        return 1;
+    }
+
+    if (!reasonText || reasonText.trim() === "") {
+        renderError(isJson, "Reason for rejection cannot be empty.", 'EMPTY_REASON');
+        return 1;
+    }
+    
+    let issue;
+    try {
+        issue = await getIssue(id);
+    } catch (error) {
+        if (error.message.includes("not found")) {
+            renderError(isJson, error.message, 'NOT_FOUND');
+        } else {
+            renderError(isJson, error.message);
+        }
+        return 1;
+    }
+
+    if (issue.status !== Status.IN_REVIEW) {
+        renderError(isJson, `Issue #${id} is currently "${issue.status}". Only issues in "${Status.IN_REVIEW}" status can be rejected.`, 'INVALID_STATE');
+        return 1;
+    }
+
+    try {
+        // (3) Perform rejection
+        const updatedIssue = await rejectIssue(id, reasonText);
+        const envelope = { status: 'success', issue: serializeIssue(updatedIssue) };
+
+        renderOutput(isJson, envelope, () => {
+            console.log(`Issue #${id} rejected successfully and moved back to "${updatedIssue.status}".`);
+        });
+
+        return 0;
+    } catch (error) {
+        renderError(isJson, error.message);
+        return 1;
+    }
+}

package/source/commands/submit.js

@@ -0,0 +1,76 @@
+// submit.js
+// AI was consulted to generate the contents of the file. It was then reviewed and edited by a human.
+// Allows an agent to submit "finished" work for human review.
+// The issue must be in the in-progress status to be submitted for review.
+// Usage: baton submit <id> [--json]
+//
+// Options:
+//   --json                 Output as JSON (for AI agents)
+//
+// Examples:
+//   baton submit 14
+
+import { getIssue, submitForReview } from '../services/issuesService.js';
+import { Status } from '../models/issue.js';
+import {
+  hasFlag,
+  renderOutput,
+  renderError,
+  serializeIssue,
+} from '../util.js';
+
+/**
+ * Submits an in-progress issue for human review.
+ * @param {string[]} args - The command line arguments
+ * @returns {Promise<number>} The exit code: 0 is success, 1 is error
+ */
+export async function run(args) {
+  const isJson = hasFlag(args, '--json');
+
+  const idArgs = args.filter((arg) => arg !== '--json');
+  if (idArgs.length === 0) {
+    renderError(isJson, 'Missing issue ID.\nUsage: baton submit <id>', 'MISSING_ID');
+    return 1;
+  }
+
+  const id = Number(idArgs[0]);
+  if (!Number.isInteger(id)) {
+    renderError(isJson, `Invalid ID "${idArgs[0]}". ID must be an integer.`, 'INVALID_ID');
+    return 1;
+  }
+
+  let issue;
+  try {
+    issue = getIssue(id);
+  } catch (error) {
+    if (error.message.includes('not found')) {
+      renderError(isJson, error.message, 'NOT_FOUND');
+    } else {
+      renderError(isJson, error.message);
+    }
+    return 1;
+  }
+
+  if (issue.status !== Status.IN_PROGRESS) {
+    renderError(
+      isJson,
+      `Issue #${id} is currently "${issue.status}". Only issues in "${Status.IN_PROGRESS}" can be submitted for review.`,
+      'INVALID_STATE',
+    );
+    return 1;
+  }
+
+  try {
+    const updatedIssue = submitForReview(id);
+    const envelope = { status: 'success', issue: serializeIssue(updatedIssue) };
+
+    renderOutput(isJson, envelope, () => {
+      console.log(`Success: Issue #${id} submitted for review.`);
+    });
+
+    return 0;
+  } catch (error) {
+    renderError(isJson, error.message);
+    return 1;
+  }
+}

package/source/services/issuesService.js

@@ -33,6 +33,14 @@ function logActivity(db, issueId, action, details = null) {
     .run();
 }
 
+/**
+ * Returns the active actor ID set during authentication.
+ * @returns {number|null}
+ */
+export function getActiveActor() {
+  return currentActorId;
+}
+
 /**
  * Convert a raw database row to an Issue instance.
  * @private
@@ -280,16 +288,37 @@ export function rejectIssue(id, reason) {
 }
 
 /**
- * Change the status of an issue to in-review.
- * Logs a state_change_event.
- * @param {number} id
+ * Change the status of an issue from in-progress to in-review.
+ * Logs a state_change event attributed to the current actor.
+ * @param {number} issueId
  * @returns {Issue}
+ * @throws {Error} If issueId is invalid, issue is not found, or status is not in-progress
  */
-export function submitForReview(id) {
+export function submitForReview(issueId) {
+  if (!Number.isInteger(issueId)) {
+    throw new Error('issueId must be an integer');
+  }
+
   const db = getDB();
-  db.update(issuesTable).set({ status: Status.IN_REVIEW }).where(eq(issuesTable.id, id)).run();
-  logActivity(db, id, Action.STATE_CHANGE, `Issue #${id} was submitted for review.`);
-  return getIssue(id);
+  const existing = findById(db, issueId);
+
+  if (existing.status !== Status.IN_PROGRESS) {
+    throw new Error(
+      `Issue #${issueId} is currently "${existing.status}". Only issues in "${Status.IN_PROGRESS}" can be submitted for review.`,
+    );
+  }
+
+  db.update(issuesTable)
+    .set({ status: Status.IN_REVIEW })
+    .where(eq(issuesTable.id, issueId))
+    .run();
+  logActivity(
+    db,
+    issueId,
+    Action.STATE_CHANGE,
+    `Issue #${issueId} was submitted for review.`,
+  );
+  return getIssue(issueId);
 }
 
 /**