baton-issue-tracker 1.9.0 → 1.10.0

package/package.json

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

package/source/cli.js

@@ -29,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';
 
@@ -48,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
@@ -79,6 +81,7 @@ 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
@@ -107,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"
@@ -146,6 +150,7 @@ async function main() {
     update: () => runUpdate(args),
     delete: () => runDelete(args),
     log: () => runLog(args),
+    submit: () => runSubmit(args),
   };
 
   const handler = handlers[command];

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);
 }
 
 /**