@spec-wave/cli 0.6.0 → 0.7.0

package/bin/spec-wave.mjs

@@ -193,4 +193,41 @@ program
     await implement({ issue, ...options }).catch(err => { console.error(err.message); process.exit(1); });
   });
 
+program
+  .command('order')
+  .description('Ordena as Stories de uma Feature pelas dependências (topológica)')
+  .argument('<feature>', 'Número da issue da Feature, ex.: 12 ou #12')
+  .action(async (feature) => {
+    const { order } = await import('../src/commands/order.mjs');
+    await order({ feature }).catch(err => { console.error(err.message); process.exit(1); });
+  });
+
+program
+  .command('task')
+  .description('Gerencia uma Task no board: start (Status "In Progress") ou done (Done)')
+  .argument('<action>', 'Ação: start ou done')
+  .argument('<n>', 'Número da issue da Task, ex.: 12 ou #12')
+  .action(async (action, n) => {
+    const { task } = await import('../src/commands/task.mjs');
+    await task({ action, issue: n }).catch(err => { console.error(err.message); process.exit(1); });
+  });
+
+program
+  .command('story')
+  .description('Gerencia uma Story no board: review (move para Code Review)')
+  .argument('<action>', 'Ação: review')
+  .argument('<n>', 'Número da issue da Story, ex.: 12 ou #12')
+  .action(async (action, n) => {
+    const { story } = await import('../src/commands/story.mjs');
+    await story({ action, issue: n }).catch(err => { console.error(err.message); process.exit(1); });
+  });
+
+program
+  .command('doctor')
+  .description('Diagnostica a configuração do spec-wave no repositório atual')
+  .action(async () => {
+    const { doctor } = await import('../src/commands/doctor.mjs');
+    await doctor().catch(err => { console.error(err.message); process.exit(1); });
+  });
+
 program.parse();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@spec-wave/cli",
-  "version": "0.6.0",
+  "version": "0.7.0",
   "description": "Setup spec-driven GitHub workflow with Projects v2, labels, issue templates, and AI-powered Actions",
   "type": "module",
   "bin": {
@@ -13,6 +13,9 @@
     "bin",
     "src"
   ],
+  "scripts": {
+    "test": "node --test test/*.test.mjs"
+  },
   "engines": {
     "node": ">=20"
   },

package/src/api/github-rest.mjs

@@ -87,10 +87,12 @@ export async function upsertFile(token, owner, repo, path, content, message) {
   });
 }
 
+// `id` é o database id da issue — exigido pela API de dependências
+// (blocked_by), que não aceita number nem node id.
 export async function createIssue(token, owner, repo, title, body, labels) {
   const octokit = makeOctokit(token);
   const res = await octokit.rest.issues.create({ owner, repo, title, body, labels });
-  return { number: res.data.number, nodeId: res.data.node_id, url: res.data.html_url };
+  return { number: res.data.number, nodeId: res.data.node_id, url: res.data.html_url, id: res.data.id };
 }
 
 export async function getIssue(token, owner, repo, issueNumber) {
@@ -148,6 +150,48 @@ export async function removeLabel(token, owner, repo, issueNumber, labelName) {
   }
 }
 
+// Lista todos os comentários de uma issue/PR: [{ author, body, createdAt }].
+// Paginado — traz o histórico completo (usado pela crítica adversarial).
+export async function listIssueComments(token, owner, repo, issueNumber) {
+  const octokit = makeOctokit(token);
+  const comments = await octokit.paginate(octokit.rest.issues.listComments, {
+    owner,
+    repo,
+    issue_number: issueNumber,
+    per_page: 100,
+  });
+  return comments.map(c => ({
+    author: c.user?.login || '',
+    body: c.body || '',
+    createdAt: c.created_at,
+  }));
+}
+
+// Marca uma issue como bloqueada por outra (relação nativa do GitHub).
+// `blockingIssueId` é o DATABASE id da issue bloqueadora (não o number nem o
+// node id — ver createIssue). Erros propagam: o chamador usa como fallback.
+export async function addBlockedBy(token, owner, repo, issueNumber, blockingIssueId) {
+  const octokit = makeOctokit(token);
+  await octokit.request('POST /repos/{owner}/{repo}/issues/{issue_number}/dependencies/blocked_by', {
+    owner,
+    repo,
+    issue_number: issueNumber,
+    issue_id: blockingIssueId,
+  });
+}
+
+// Lista as issues que bloqueiam `issueNumber`: [{ number, title, state, id }].
+export async function listBlockedBy(token, owner, repo, issueNumber) {
+  const octokit = makeOctokit(token);
+  const issues = await octokit.paginate('GET /repos/{owner}/{repo}/issues/{issue_number}/dependencies/blocked_by', {
+    owner,
+    repo,
+    issue_number: issueNumber,
+    per_page: 100,
+  });
+  return issues.map(i => ({ number: i.number, title: i.title, state: i.state, id: i.id }));
+}
+
 export async function commentOnIssue(token, owner, repo, issueNumber, body) {
   const octokit = makeOctokit(token);
   await octokit.rest.issues.createComment({

package/src/commands/code-review.mjs

@@ -2,9 +2,10 @@ import { existsSync, readFileSync } from 'node:fs';
 import path from 'node:path';
 import { resolveToken } from '../api/auth.mjs';
 import { getIssue, getPR, commentOnIssue } from '../api/github-rest.mjs';
-import { addProjectItem, setItemSingleSelect, getSingleSelectField, getIssueParent, listSubIssues, getItemSingleSelectValue } from '../api/github-graphql.mjs';
+import { addProjectItem, getIssueParent, listSubIssues, getItemSingleSelectValue } from '../api/github-graphql.mjs';
 import { detectIssueType } from '../lib/issue-type.mjs';
-import { CONFIG_FILE, STATUS_OPTIONS, STAGE_ORDER, STAGE_DONE, PROGRESS_TODO, PROGRESS_DONE } from '../config.mjs';
+import { loadProjectConfig, resolveField, advanceToStage } from '../lib/board.mjs';
+import { CONFIG_FILE, STATUS_OPTIONS, STAGE_ORDER, STAGE_DONE, PROGRESS_TODO, PROGRESS_DONE, isManualStageType } from '../config.mjs';
 
 // Ao abrir o PR: Stories/Feature → Etapa "👀 Code Review" (Status "Todo");
 // Tasks → Etapa "🎉 Done" (Status "Done"), pois a implementação da task terminou.
@@ -25,15 +26,6 @@ function extractIssueNumbers(body) {
   return [...nums];
 }
 
-// Resolve o campo SINGLE_SELECT pelo nome: usa .spec-wave.json, legado ou API.
-async function resolveField(token, project, name) {
-  if (project.fields?.[name]) return project.fields[name];
-  if (name === 'Etapa' && project.etapaFieldId) {
-    return { id: project.etapaFieldId, options: project.stageOptions || {} };
-  }
-  return await getSingleSelectField(token, project.id, name);
-}
-
 // A partir de qualquer issue (Feature/Story/Task), sobe a hierarquia e retorna a Feature.
 async function resolveFeatureIssue(token, owner, repo, issueNumber) {
   let issue;
@@ -77,25 +69,29 @@ async function collectReviewUnit(token, owner, repo, issueNumber) {
     ? { number: featureIssue.number, nodeId: featureIssue.node_id, title: featureIssue.title }
     : null;
 
+  // Spikes (tipos manuais) nunca são movidos automaticamente — pulados aqui.
+  const isManual = (sub) => isManualStageType(detectIssueType({ title: sub.title, labels: sub.labels }));
+
   if (type === 'Feature') {
     // Referência direta à Feature: toda a subárvore (Stories + Tasks).
     const subs = await listSubIssues(token, issue.node_id).catch(() => []);
     for (const st of subs) {
+      if (isManual(st)) continue;
       addStory(st.number, st.nodeId, st.title);
       const tks = await listSubIssues(token, st.nodeId).catch(() => []);
-      for (const t of tks) addTask(t.number, t.nodeId, t.title);
+      for (const t of tks) { if (isManual(t)) continue; addTask(t.number, t.nodeId, t.title); }
     }
   } else if (type === 'Story') {
     addStory(issue.number, issue.node_id, issue.title);
     const tks = await listSubIssues(token, issue.node_id).catch(() => []);
-    for (const t of tks) addTask(t.number, t.nodeId, t.title);
+    for (const t of tks) { if (isManual(t)) continue; addTask(t.number, t.nodeId, t.title); }
   } else { // Task → inclui a Story pai e as Tasks irmãs.
     addTask(issue.number, issue.node_id, issue.title);
     const parent = await getIssueParent(token, issue.node_id).catch(() => null);
     if (parent && detectIssueType({ title: parent.title }) === 'Story') {
       addStory(parent.number, parent.nodeId, parent.title);
       const tks = await listSubIssues(token, parent.nodeId).catch(() => []);
-      for (const t of tks) addTask(t.number, t.nodeId, t.title);
+      for (const t of tks) { if (isManual(t)) continue; addTask(t.number, t.nodeId, t.title); }
     }
   }
   return { feature, stories, tasks };
@@ -118,30 +114,6 @@ async function allStoriesReadyForReview(readToken, projToken, project, etapaFiel
   return true;
 }
 
-// Avança um item do board para `targetStage` (Etapa) e define o Status para
-// `targetStatus`. Uma issue só AVANÇA: se já estiver em `targetStage` ou em uma
-// etapa posterior, não é tocada (retorna false). Retorna true se avançou.
-async function advanceToStage(token, project, etapaField, statusField, nodeId, targetStage, targetStatus) {
-  const itemId = await addProjectItem(token, project.id, nodeId);
-
-  if (etapaField?.id && targetStage) {
-    // Nunca retroceder: compara a etapa atual com a de destino na ordem canônica.
-    const current = await getItemSingleSelectValue(token, itemId, etapaField.id).catch(() => null);
-    const curIdx = current ? STAGE_ORDER.indexOf(current) : -1;
-    const tgtIdx = STAGE_ORDER.indexOf(targetStage);
-    if (curIdx !== -1 && tgtIdx !== -1 && curIdx >= tgtIdx) {
-      return false; // já está nessa etapa ou adiante — não retrocede
-    }
-    const optionId = etapaField.options?.[targetStage];
-    if (optionId) await setItemSingleSelect(token, project.id, itemId, etapaField.id, optionId);
-  }
-  if (statusField?.id && targetStatus) {
-    const optionId = statusField.options?.[targetStatus];
-    if (optionId) await setItemSingleSelect(token, project.id, itemId, statusField.id, optionId);
-  }
-  return true;
-}
-
 export async function codeReview({ prNumber }) {
   const token = await resolveToken();
   const projectToken = process.env.PROJECT_TOKEN || token;
@@ -168,20 +140,9 @@ export async function codeReview({ prNumber }) {
   }
 
   // Carrega projeto do .spec-wave.json
-  const configPath = path.join(process.cwd(), CONFIG_FILE);
-  if (!existsSync(configPath)) {
-    console.warn(`${CONFIG_FILE} não encontrado — board não atualizado.`);
-    return;
-  }
-  let project;
-  try {
-    project = JSON.parse(readFileSync(configPath, 'utf-8')).project || {};
-  } catch (err) {
-    console.warn(`${CONFIG_FILE} corrompido (${err.message}) — board não atualizado.`);
-    return;
-  }
-  if (!project.id) {
-    console.warn(`Project não configurado em ${CONFIG_FILE} — board não atualizado.`);
+  const { project, error: projectError } = loadProjectConfig();
+  if (projectError) {
+    console.warn(`${projectError} — board não atualizado.`);
     return;
   }