flowcollab 0.1.3 → 0.1.5

package/bin/review.mjs

@@ -1,9 +1,8 @@
 #!/usr/bin/env node
 /* flow:review — request a code review for a task's PR.
    Moves the task to in_review (if in_progress) and posts a standardised
-   review-request comment that appears in the timeline and triggers @mention
-   notifications. If --reviewer is given, also pings that actor's personal
-   webhook directly (bypasses the @mention comment path).
+   review-request comment. @to:<reviewer> in the comment body triggers
+   server-side notification routing (personal webhook → FLOW_NOTIFY_WEBHOOK).
 
    Usage:
      flow-review <task-id> [--pr=<number>] [--reviewer=<actor-id>] [--context="notes"]
@@ -26,10 +25,7 @@ async function main() {
   const taskId = await resolveTaskId(rawId);
 
   // Fetch tasks + people in parallel.
-  const [{ tasks = [] }, { people = {} }] = await Promise.all([
-    flowFetch('/api/flow/tasks?limit=500'),
-    flowFetch('/api/flow/people'),
-  ]);
+  const { tasks = [] } = await flowFetch('/api/flow/tasks?limit=500');
 
   const task = tasks.find(t => t.id === taskId);
   if (!task) die(`Task ${taskId} not found.`);
@@ -54,23 +50,6 @@ async function main() {
 
   await flowFetch('/api/flow/comment', { method: 'POST', body: { task_id: taskId, body } });
 
-  // If reviewer has a personal webhook, ping it directly too (fire-and-forget).
-  if (reviewerArg) {
-    const peopleArr = Array.isArray(people) ? people : Object.values(people);
-    const reviewer = peopleArr.find(p => p.id === reviewerArg);
-    const hook = reviewer?.notify_webhook || process.env.FLOW_NOTIFY_WEBHOOK;
-    if (hook) {
-      const ref = `#${task.issue_num ?? taskId.slice(0, 6)}`;
-      fetch(hook, {
-        method: 'POST',
-        headers: { 'content-type': 'application/json' },
-        body: JSON.stringify({
-          text: `Review requested on ${ref} — PR #${prNum}${prUrl ? ' ' + prUrl : ''}${context ? '\n' + context : ''}`,
-        }),
-      }).catch(() => {});
-    }
-  }
-
   // Move to in_review if currently in_progress.
   if (task.status === 'in_progress') {
     await flowFetch(`/api/flow/tasks/${taskId}`, { method: 'PATCH', body: { status: 'in_review' } });
@@ -78,8 +57,8 @@ async function main() {
 
   const ref   = `#${task.issue_num ?? taskId.slice(0, 6)}`;
   const moved = task.status === 'in_progress' ? '  (moved → in_review)' : '';
-  const pinged = reviewerArg ? `  (pinged ${reviewerArg})` : '';
-  process.stdout.write(`Review requested on ${ref} — PR #${prNum}${prUrl ? '  ' + prUrl : ''}${moved}${pinged}\n`);
+  const mentioned = reviewerArg ? `  (@to:${reviewerArg} notified)` : '';
+  process.stdout.write(`Review requested on ${ref} — PR #${prNum}${prUrl ? '  ' + prUrl : ''}${moved}${mentioned}\n`);
 }
 
 main().catch(e => die(e.message || e));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "flowcollab",
-  "version": "0.1.3",
+  "version": "0.1.5",
   "description": "Multi-Claude coordination layer — shared task board + CLI for teams running Claude Code",
   "type": "module",
   "files": [
@@ -26,7 +26,6 @@
     "flow-handoff": "bin/handoff.mjs",
     "flow-standup": "bin/standup.mjs",
     "flow-search": "bin/search.mjs",
-    "flow-init": "bin/init.mjs",
     "flow-project": "bin/project.mjs",
     "flow-close-sprint": "bin/close-sprint.mjs",
     "flow-review": "bin/review.mjs",

package/bin/init.mjs

@@ -1,187 +0,0 @@
-#!/usr/bin/env node
-/* flow:init — set up CLI access for a web-provisioned Flow account.
-
-   You signed up at https://flow-production-84b7.up.railway.app and have a
-   token from the Flow setup page. This writes your .env and CLAUDE.md.
-
-   Usage: flow-init --web
-          flow-init --web
-*/
-
-import { readFileSync, writeFileSync, existsSync } from 'node:fs';
-import path from 'node:path';
-import { createInterface } from 'node:readline';
-
-const rl = createInterface({ input: process.stdin, output: process.stdout });
-const ask = (q) => new Promise(r => rl.question(q, r));
-
-function buildWebClaudeMd({ actorId, ownerActorId, boardUrl, scope, areas = [] }) {
-  const ownerRef = scope === 'owner' ? actorId : (ownerActorId || 'owner');
-  const areaLine = areas.length ? areas.join(', ') : 'backend, frontend, infra, docs';
-  return `# Flow — Claude Code workflow
-
-This project uses Flow for multi-agent coordination.
-Follow this workflow every session without exception.
-
-## Your identity
-- **Your actor ID:** \`${actorId}\`
-- **Your scope:** ${scope}
-- **Board:** ${boardUrl}
-
-## Required session workflow
-
-\`\`\`bash
-flow-pull                                    # ALWAYS start here — board snapshot + your mentions
-flow-claim <task_id>                         # claim before writing any code
-flow-comment <task_id> "..."                 # update at milestones, use @to:${ownerRef} to ping
-flow-close <task_id> "summary"               # close with a meaningful summary when done
-flow-propose --parent=<id> --title="..." --md="..."  # propose new work; never start without approval
-\`\`\`
-
-## Rules — never violate
-- \`flow-pull\` at session start — no exceptions
-- Claim a task before touching related code
-- Comment at major milestones, not every commit
-- Close with a summary the next agent can act on
-- Never approve your own proposals — only **owners** can approve (in the browser at ${boardUrl})
-- Use \`@to:${ownerRef}\` in comments to request human action or review
-
-## All CLI commands
-\`\`\`
-flow-pull                                — board snapshot + mentions + handoffs
-flow-pull --focus                        — top-3 open tasks sorted by priority
-flow-pull --milestone=<name>             — filter snapshot to a sprint
-flow-claim <id>                          — self-assign a task
-flow-comment <id> "..."                  — add a comment or milestone update
-flow-close <id> "summary"               — mark task done with summary
-flow-create --title="..."               — create a task directly (no approval gate)
-flow-create --from-issue=<num>          — import a GitHub Issue as a task
-flow-create --from-project-item=<id>    — import a GitHub Projects item
-flow-create --template=<id>             — create from template (standup, pr-review, retro, ...)
-flow-propose --parent=<id> --title="..." --md="..."  — propose work for human approval
-flow-approve/reject <id>                — resolve a proposal (owner only)
-flow-assign <id> <actor>               — reassign a task (owner only)
-flow-decisions                          — list pending proposals
-flow-search "query"                     — search tasks by title
-flow-status                             — board summary (counts, active agents)
-flow-standup                            — recent activity digest
-flow-standup --velocity                 — recent activity + week-over-week velocity chart
-flow-handoff --task=<id> --to=<actor>  — hand off a task with context
-flow-heartbeat                          — send presence ping (run in background)
-flow-sync                               — delta sync since N minutes ago
-flow-ping                               — health check
-flow-whoami                             — verify token and actor identity
-flow-project                            — list GitHub Projects v2 items
-flow-close-sprint --milestone=<name>    — close sprint (owner only)
-flow-review <id> [--pr=<num>] [--reviewer=<actor>]  — request code review; moves → in_review, pings reviewer
-flow-edit <id> [--title=] [--priority=] [--area=] [--due=] [--milestone=] [--blocked-by=]
-flow-unblock <id>                       — clear blocked_by
-flow-log [--task=<id>] [--limit=20]    — tail recent timeline events
-\`\`\`
-
-## Area labels
-${areaLine}
-`;
-}
-
-async function main() {
-  if (!process.argv.includes('--web')) {
-    console.error(
-      'Usage: flow-init --web\n\n' +
-      'Flow is a hosted SaaS. Sign up at:\n' +
-      '  https://flow-production-84b7.up.railway.app\n\n' +
-      'Once you have a token from the setup page, run:\n' +
-      '  flow-init --web\n'
-    );
-    process.exit(1);
-  }
-
-  console.log('\nFlow Board — web setup\n');
-  console.log('You signed up via the web and already have a token from the Flow setup page.');
-  console.log('This writes your .env and CLAUDE.md — no Supabase config needed.\n');
-
-  const apiBase = (await ask('Flow server URL (e.g. https://flow-production-84b7.up.railway.app): '))
-    .trim().replace(/\/+$/, '');
-  const token   = (await ask('Your API token (from setup page): ')).trim();
-  const actorId = (await ask('Your actor ID (from setup page, e.g. "alice"): ')).trim() || 'owner';
-  const scopeRaw = (await ask('Your scope [owner/contributor, default: owner]: ')).trim().toLowerCase();
-  const scope = scopeRaw === 'contributor' ? 'contributor' : 'owner';
-
-  let ownerActorId = actorId;
-  if (scope === 'contributor') {
-    ownerActorId = (await ask("Owner's actor ID (for CLAUDE.md @mentions, e.g. \"alice\"): ")).trim() || 'owner';
-  }
-
-  const cwd = process.cwd();
-  const localAreas = (() => {
-    try {
-      const raw = readFileSync(path.join(cwd, 'flow.config.json'), 'utf8');
-      return Object.keys(JSON.parse(raw).labels?.area || {});
-    } catch { return []; }
-  })();
-
-  const envTarget      = path.join(cwd, '.env');
-  const claudeMdTarget = path.join(cwd, 'CLAUDE.md');
-  const envExists      = existsSync(envTarget);
-  const claudeExists   = existsSync(claudeMdTarget);
-
-  const writeEnvAns = (await ask(
-    `\nWrite .env to ${cwd}? ${envExists ? '[y/N]' : '[Y/n]'}: `
-  )).trim().toLowerCase();
-  const writeEnv = envExists
-    ? (writeEnvAns === 'y' || writeEnvAns === 'yes')
-    : (writeEnvAns !== 'n' && writeEnvAns !== 'no');
-
-  const writeClaudeAns = (await ask(
-    `Write CLAUDE.md to ${cwd}? ${claudeExists ? '[y/N]' : '[Y/n]'}: `
-  )).trim().toLowerCase();
-  const writeClaude = claudeExists
-    ? (writeClaudeAns === 'y' || writeClaudeAns === 'yes')
-    : (writeClaudeAns !== 'n' && writeClaudeAns !== 'no');
-
-  rl.close();
-
-  const tokenVar = scope === 'owner' ? 'FLOW_API_TOKEN_OWNER' : 'FLOW_API_TOKEN_CONTRIBUTOR';
-  const actorVar = scope === 'owner' ? 'FLOW_TOKEN_OWNER_ACTOR' : 'FLOW_TOKEN_CONTRIBUTOR_ACTOR';
-  const envLines = [
-    `${tokenVar}=${token}`,
-    `${actorVar}=${actorId}`,
-    `FLOW_API_BASE=${apiBase}`,
-    `FLOW_ACTING_VIA=claude`,
-    `FLOW_DEFAULT_ASSIGNEE=${actorId}`,
-  ];
-
-  console.log('\n========== .env ==========\n');
-  envLines.forEach(l => console.log(l));
-
-  if (writeEnv) {
-    if (envExists) {
-      const existing = readFileSync(envTarget, 'utf8');
-      const conflicts = envLines.map(l => l.split('=')[0]).filter(k => new RegExp(`^${k}=`, 'm').test(existing));
-      if (conflicts.length) {
-        console.log(`\n⚠  These keys already exist in .env — not overwriting: ${conflicts.join(', ')}`);
-        console.log('   Paste the block above into .env manually.');
-      } else {
-        writeFileSync(envTarget, existing.trimEnd() + '\n\n' + envLines.join('\n') + '\n', 'utf8');
-        console.log(`\n✓ .env updated. Add .env to .gitignore — it contains your API token.`);
-      }
-    } else {
-      writeFileSync(envTarget, envLines.join('\n') + '\n', 'utf8');
-      console.log(`\n✓ .env written. Add .env to .gitignore — it contains your API token.`);
-    }
-  }
-
-  if (writeClaude) {
-    const content = buildWebClaudeMd({ actorId, ownerActorId, boardUrl: `${apiBase}/flow/`, scope, areas: localAreas });
-    writeFileSync(claudeMdTarget, content, 'utf8');
-    console.log(`✓ CLAUDE.md written. Commit this so every agent on the team gets the workflow.`);
-  }
-
-  console.log('\n========== Next steps ==========\n');
-  console.log(`1. Verify your connection:    flow-whoami`);
-  console.log(`2. See your board:            flow-pull`);
-  console.log(`3. Open the board:            ${apiBase}/flow/`);
-  console.log('');
-}
-
-main().catch(e => { console.error('init failed:', e.message || e); process.exit(1); });