flowcollab 0.2.5 → 0.2.7

package/bin/_client.mjs

@@ -54,7 +54,10 @@ function loadGlobalConfig() {
 }
 const _gc = loadGlobalConfig();
 
-const DEFAULT_BASE = process.env.FLOW_API_BASE || _gc.apiBase || 'https://flow-production-84b7.up.railway.app';
+// flow-login writes apiBase to ~/.flow/config.json — that takes precedence over env so
+// a stale FLOW_API_BASE doesn't silently override the user's server selection.
+// FLOW_API_BASE is still honoured as an explicit override when config.json has no apiBase.
+const DEFAULT_BASE = _gc.apiBase || process.env.FLOW_API_BASE || 'https://flow-production-84b7.up.railway.app';
 
 function envToken() {
   return process.env.FLOW_API_TOKEN_OWNER || process.env.FLOW_API_TOKEN_CONTRIBUTOR || _gc.token || '';
@@ -76,6 +79,8 @@ export async function flowFetch(path, { method = 'GET', body } = {}) {
     'content-type': 'application/json',
   };
   if (actingViaClaude()) headers['x-flow-acting-via'] = 'claude';
+  const org = process.env.FLOW_ORG_ID || _gc.orgId || '';
+  if (org) headers['x-flow-org'] = org;
   const proj = process.env.FLOW_PROJECT_ID || _gc.projectId || '';
   if (proj) headers['x-flow-project'] = proj;
 
@@ -97,7 +102,9 @@ export async function flowFetch(path, { method = 'GET', body } = {}) {
 }
 
 export const RESOLVED_BASE = DEFAULT_BASE;
-export const RESOLVED_ACTOR = process.env.FLOW_DEFAULT_ASSIGNEE || _gc.actorId || '';
+// Prefer config.json actorId (written by flow-login) over env vars.
+// FLOW_DEFAULT_ASSIGNEE is the legacy env name; kept for backwards compat.
+export const RESOLVED_ACTOR = _gc.actorId || process.env.FLOW_DEFAULT_ASSIGNEE || process.env.FLOW_ACTOR || '';
 export const RESOLVED_PROJECT = process.env.FLOW_PROJECT_ID || _gc.projectId || '';
 export function resolvedTokenDisplay() {
   const raw = envToken();

package/bin/close.mjs

@@ -16,7 +16,8 @@ async function main() {
     body: { task_id: id, ...(summary ? { summary } : {}) },
   });
 
-  process.stdout.write(`Closed #${r.task.id.slice(0, 6)} — "${r.task.title}"\n`);
+  const ref = r.task.issue_num != null ? r.task.issue_num : r.task.id.slice(0, 6);
+  process.stdout.write(`Closed #${ref} — "${r.task.title}"\n`);
 }
 
 main().catch(e => die(e.message || e));

package/bin/create.mjs

@@ -93,7 +93,7 @@ async function main() {
     ...(arg('priority')   ? { priority:          arg('priority')   } : {}),
     ...(arg('status')     ? { status:            arg('status')     } : {}),
     ...(arg('assignee')   ? { assignee_id:       arg('assignee')   } : {}),
-    ...(arg('due')        ? { due_at: new Date(arg('due')).toISOString() } : {}),
+    ...(arg('due')        ? (() => { const d = new Date(arg('due')); if (isNaN(d.getTime())) die(`Invalid --due date: "${arg('due')}". Use YYYY-MM-DD format.`); return { due_at: d.toISOString() }; })() : {}),
     ...(arg('blocked-by') ? { blocked_by: arg('blocked-by').split(',').map(s => s.trim()).filter(Boolean) } : {}),
     ...(arg('milestone')  ? { milestone:         arg('milestone')  } : {}),
   };

package/bin/handoff.mjs

@@ -9,10 +9,11 @@
    flow_files are auto-read from the task and included in handoff_data.
 */
 
-import { flowFetch, arg } from './_client.mjs';
+import { flowFetch, resolveTaskId, arg } from './_client.mjs';
 
 async function main() {
-  const task_id = arg('task');
+  const rawTask = arg('task');
+  const task_id = rawTask ? await resolveTaskId(rawTask) : undefined;
   const to = arg('to');
   const context = arg('context');
   const branch = arg('branch') || null;

package/bin/login.mjs

@@ -61,8 +61,9 @@ async function main() {
       process.stdout.write('\n');
       throw new Error(`Unexpected server response: ${poll.status}. Run flow-login again.`);
     }
-    if (poll.ok) {
+    if (poll.status === 200) {
       const { token, actor_id, org_id, api_base } = await poll.json();
+      if (!token) throw new Error('Server returned no token. Run flow-login again.');
       const dir = join(homedir(), '.flow');
       mkdirSync(dir, { recursive: true });
       writeFileSync(

package/bin/review.mjs

@@ -24,8 +24,15 @@ async function main() {
 
   const taskId = await resolveTaskId(rawId);
 
-  // Fetch tasks + people in parallel.
-  const { tasks = [] } = await flowFetch('/api/flow/tasks?limit=500');
+  // Fetch all tasks (paginated to handle boards with >500 tasks).
+  let tasks = [], cursor = null;
+  do {
+    const url = `/api/flow/tasks?limit=200${cursor ? '&cursor=' + encodeURIComponent(cursor) : ''}`;
+    const page = await flowFetch(url);
+    const pageItems = Array.isArray(page) ? page : (page.tasks ?? []);
+    tasks = tasks.concat(pageItems);
+    cursor = page.has_more ? page.next_cursor : null;
+  } while (cursor);
 
   const task = tasks.find(t => t.id === taskId);
   if (!task) die(`Task ${taskId} not found.`);

package/bin/scan.mjs

@@ -176,10 +176,11 @@ function printSection(heading, suggestions) {
 
 // ── Main ───────────────────────────────────────────────────────────────────────
 async function main() {
-  const doTodos    = !!arg('todos');
-  const doIssues   = !!arg('issues');
-  const doPRs      = !!arg('prs');
-  const doSecurity = !!arg('security');
+  const argv = process.argv;
+  const doTodos    = argv.includes('--todos');
+  const doIssues   = argv.includes('--issues');
+  const doPRs      = argv.includes('--prs');
+  const doSecurity = argv.includes('--security');
   const all = !doTodos && !doIssues && !doPRs && !doSecurity;
 
   const dir  = arg('dir') || process.cwd();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "flowcollab",
-  "version": "0.2.5",
+  "version": "0.2.7",
   "description": "Multi-Claude coordination layer — shared task board + CLI for teams running Claude Code",
   "type": "module",
   "files": [