multi-agents-cli 1.1.7 → 1.1.8

package/core/workflow/run.js

@@ -56,9 +56,6 @@ const copyWorkflow = (cliRoot) => {
 };
 
 const findCliRoot = () => {
-  // Local node_modules first (npx or local install)
-  const localPkg = path.join(__dirname, '..', 'node_modules', 'multi-agents-cli');
-  if (fs.existsSync(localPkg)) return localPkg;
   try {
     const globalPkg = execSync('npm root -g', { stdio: 'pipe', encoding: 'utf8' }).trim();
     const p = path.join(globalPkg, 'multi-agents-cli');

package/init.js

@@ -78,7 +78,7 @@ const projectArg  = isGlobalCLI ? args[1] : null;
 
 if (isReInit) {
   try {
-    const gitCommonDir = execSync('git rev-parse --git-common-dir', { encoding: 'utf8', stdio: 'pipe' }).trim();
+    const gitCommonDir = execSync('git rev-parse --git-common-dir', { encoding: 'utf8' }).trim();
     const repoRoot = path.resolve(gitCommonDir, '..');
     process.chdir(repoRoot);
   } catch { /* stay in current directory */ }
@@ -275,7 +275,7 @@ const main = async () => {
           ],
         }, { onCancel: () => process.exit(0) });
         if (confirm.value !== 'yes') { console.log(dim('\n  Cancelled.\n')); process.exit(0); }
-        const resetChild = spawn('node', [path.join(ROOT, '.workflow', 'reset.js'), '--confirmed'], { stdio: 'inherit', cwd: ROOT });
+        const resetChild = spawn('node', [path.join(ROOT, '.workflow', 'reset.js')], { stdio: 'inherit', cwd: ROOT });
         resetChild.on('exit', code => process.exit(code ?? 0));
         return;
       } else {

package/lib/questions-flow.js

@@ -2,7 +2,7 @@
 
 // ── Imports ───────────────────────────────────────────────────────────────────
 
-const { dim, bold, blue, green, cyan, separator, arrowSelect, arrowConfirm, selectRequired, selectOptional, stepHeader } = require('./ui');
+const { dim, bold, blue, green, cyan, separator, arrowSelect, arrowConfirm, selectRequired, selectOptional } = require('./ui');
 const { BACK, RESTART } = require('./steps');
 const {
   CLIENT_FRAMEWORKS, BACKEND_FRAMEWORKS, FRAMEWORK_VERSION_FALLBACK,
@@ -21,7 +21,6 @@ const buildStepDefs = (IDE_CANDIDATES) => [
   {
     name: 'clientFw',
     run: async (machine, answers) => {
-      stepHeader(1, 12);
       console.log(`\n${bold(blue('Client configuration'))}`);
       return await selectRequired('* Client framework (required):', CLIENT_FRAMEWORKS, machine, 0);
     },
@@ -31,7 +30,6 @@ const buildStepDefs = (IDE_CANDIDATES) => [
   {
     name: 'clientFwVersion',
     run: async (machine, answers) => {
-      stepHeader(2, 12);
       const fw = answers.clientFw;
       const versions = await fetchLatestVersions(fw.value) || FRAMEWORK_VERSION_FALLBACK[fw.value] || [];
       if (!versions.length) return null; // skip silently
@@ -58,7 +56,6 @@ const buildStepDefs = (IDE_CANDIDATES) => [
   {
     name: 'clientState',
     run: async (machine, answers) => {
-      stepHeader(3, 12);
       const opts = STATE_OPTIONS[answers.clientFw.value] || [];
       return await selectOptional('State management:', opts, machine, 2);
     },
@@ -68,7 +65,6 @@ const buildStepDefs = (IDE_CANDIDATES) => [
   {
     name: 'clientUi',
     run: async (machine, answers) => {
-      stepHeader(4, 12);
       const opts = UI_OPTIONS[answers.clientFw.value] || [];
       return await selectOptional('UI library:', opts, machine, 3);
     },
@@ -78,7 +74,6 @@ const buildStepDefs = (IDE_CANDIDATES) => [
   {
     name: 'clientStyle',
     run: async (machine, answers) => {
-      stepHeader(5, 12);
       return await selectOptional('Styling:', STYLING_OPTIONS, machine, 4);
     },
   },
@@ -87,7 +82,6 @@ const buildStepDefs = (IDE_CANDIDATES) => [
   {
     name: 'useIntegratedBackend',
     run: async (machine, answers) => {
-      stepHeader(6, 12);
       const fw = answers.clientFw;
       if (!fw.integratedBackend) return false; // skip — not applicable
 
@@ -104,7 +98,6 @@ const buildStepDefs = (IDE_CANDIDATES) => [
   {
     name: 'backendFwObj',
     run: async (machine, answers) => {
-      stepHeader(7, 12);
       if (answers.useIntegratedBackend) return null; // skip
 
       separator();
@@ -128,7 +121,6 @@ const buildStepDefs = (IDE_CANDIDATES) => [
   {
     name: 'backendFwVersion',
     run: async (machine, answers) => {
-      stepHeader(8, 12);
       const fwObj = answers.backendFwObj;
       if (!fwObj) return null; // skip — no backend selected
 
@@ -155,7 +147,6 @@ const buildStepDefs = (IDE_CANDIDATES) => [
   {
     name: 'backendDb',
     run: async (machine, answers) => {
-      stepHeader(9, 12);
       if (!answers.backendFwObj) return null;
       return await selectOptional('Database type:', DB_OPTIONS, machine, 8);
     },
@@ -165,7 +156,6 @@ const buildStepDefs = (IDE_CANDIDATES) => [
   {
     name: 'backendOrm',
     run: async (machine, answers) => {
-      stepHeader(10, 12);
       const { backendFwObj, backendDb } = answers;
       if (!backendFwObj || !backendDb) return null;
       const ormChoices = ORM_OPTIONS_BY_DB[backendDb] || ORM_OPTIONS[backendFwObj.value] || [];
@@ -177,7 +167,6 @@ const buildStepDefs = (IDE_CANDIDATES) => [
   {
     name: 'backendAuth',
     run: async (machine, answers) => {
-      stepHeader(11, 12);
       const { backendFwObj } = answers;
       if (!backendFwObj) return null;
       return await selectOptional('Auth strategy:', AUTH_OPTIONS[backendFwObj.value] || [], machine, 10);
@@ -188,7 +177,6 @@ const buildStepDefs = (IDE_CANDIDATES) => [
   {
     name: 'ideChoice',
     run: async (machine, answers) => {
-      stepHeader(12, 12);
       separator();
       console.log(`\n${bold(blue('Environment'))}`);
 
@@ -200,7 +188,11 @@ const buildStepDefs = (IDE_CANDIDATES) => [
       const detectedIDEs   = ideOptions.filter(o => o.detected);
       const undetectedIDEs = ideOptions.filter(o => !o.detected && o.cmd);
       const manualOption   = ideOptions.filter(o => !o.cmd);
-      const sorted         = [...detectedIDEs, ...undetectedIDEs, ...manualOption];
+      const sorted         = [
+        ...detectedIDEs,
+        ...undetectedIDEs.map(o => ({ ...o, disabled: true })),
+        ...manualOption,
+      ];
 
       if (detectedIDEs.length > 1)      console.log(`\n  ${'\x1b[33m'}Multiple IDEs found${'\x1b[0m'} - select your preference:\n`);
       else if (detectedIDEs.length === 1) console.log(`\n  ${'\x1b[32m'}1 IDE found:${'\x1b[0m'} ${bold(detectedIDEs[0].name)}\n`);

package/lib/steps.js

@@ -103,25 +103,13 @@ const runQuestions = async (stepDefs, machine) => {
     if (result === RESTART) return RESTART;
 
     if (result === BACK) {
-      if (i === 0) return RESTART; // nowhere to go back from step 0
-      machine.enterBackNav(i);     // record where back-nav started
+      if (i === 0) return RESTART; // nowhere to go back to from step 0
       const popped = machine.pop();
       if (popped) delete answers[popped.stepName];
       i--;
       continue;
     }
 
-    if (result === CONTINUE) {
-      // User chose to resume — skip forward to where they first pressed Back
-      const resumeTarget = machine.resumeIndex ?? i + 1;
-      machine.exitBackNav();
-      i = resumeTarget;
-      continue;
-    }
-
-    // Normal answer — if re-answering during back-nav, exit back-nav mode
-    if (machine.inBackNav) machine.exitBackNav();
-
     machine.push(i, def.name, result);
     answers[def.name] = result;
     i++;

package/lib/ui.js

@@ -48,7 +48,7 @@ const arrowSelect = async (message, choices, showBack = false, backLabel = '←
       type:    'select',
       name:    'value',
       message,
-      choices: allChoices.map((c, i) => ({ title: typeof c === 'string' ? c : c.label, value: i })),
+      choices: allChoices.map((c, i) => ({ title: typeof c === 'string' ? c : c.label, value: i, disabled: c.disabled ?? false })),
     }, { onCancel: () => process.exit(0) });
     return res.value ?? 0;
   }
@@ -109,13 +109,6 @@ const renderTrajectoryLines = (lines) => {
   });
 };
 
-// ── Step progress header ─────────────────────────────────────────────────────
-
-const stepHeader = (stepIndex, totalSteps) => {
-  console.log(`
-  ${dim(`Step ${stepIndex} of ${totalSteps}`)}`);
-};
-
 // ── Step selection helpers ────────────────────────────────────────────────────
 
 const { BACK, RESTART } = require('./steps');
@@ -157,7 +150,6 @@ const selectOptional = async (prompt, items, stepMachine, stepIndex) => {
 // ── Exports ───────────────────────────────────────────────────────────────────
 
 module.exports = {
-  stepHeader,
   c, bold, green, yellow, dim, cyan, blue, red,
   rl, ask,
   arrowSelect, arrowConfirm,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "multi-agents-cli",
-  "version": "1.1.7",
+  "version": "1.1.8",
   "description": "Multi-agent workflow orchestration for Claude Code — isolated git worktrees, structured state tracking, autonomous task chaining",
   "keywords": [
     "claude-code",

package/core/templates/.agents/backend/API.md

@@ -1,270 +0,0 @@
-# API Agent
-# Scope: backend/
-# Loaded by: manual reference in prompt
-# Example: `Use .agents/backend/API.md. Task: implement the job applications CRUD endpoints.`
-
----
-
-## Mission
-
-Implement all API endpoint handlers, request/response shaping, DTO design,
-and input validation against an already-scaffolded backend structure. This agent
-builds inside the architecture defined by the INIT agent — it does not make
-structural or architectural decisions.
-
-Read `shared/wiring.config.json` and `CONTRACTS.md` on entry and treat both
-as binding contracts. Never redefine types or introduce vars outside these files.
-
-This agent does not own business logic, database queries, authentication
-strategies, background job processing, event handling, or architectural decisions.
-Those belong to their respective agents.
-
----
-
-## Pre-flight Checks
-
-Runs in order before any file is created or modified. All checks must pass.
-
-### 1. Task Clarity Check
-
-Is the task specific enough to act on?
-
-- Identify: which endpoint or endpoints are being built or modified
-- Identify: what HTTP method, path, request body, and response shape apply
-- Identify: what guard or access control the endpoint requires
-
-If any of these cannot be determined from the task as given:
-```
-## CLARIFICATION NEEDED - [Round 1 or 2]
-The following is unclear:
-  - <specific ambiguity>
-Please provide more detail before this agent proceeds.
-```
-
-Maximum 2 rounds. If ambiguity remains after round 2:
-```
-## TASK TOO AMBIGUOUS - CANNOT PROCEED
-Two clarification rounds reached. Please rephrase the task with:
-  - explicit HTTP method and path
-  - request body shape and response shape
-  - guard or access control requirement
-```
-
-### 2. Scope Integrity Check
-
-Does this task stay within API concerns?
-
-If the task requires:
-- Business logic or domain rules → redirect to `.agents/backend/LOGIC.md`
-- Database schema or query implementation → redirect to `.agents/backend/DB.md`
-- Auth strategy or guard implementation → redirect to `.agents/backend/AUTH.md`
-- Background job or scheduled task → redirect to `.agents/backend/JOBS.md`
-- Event emission or subscription → redirect to `.agents/backend/EVENTS.md`
-
-```
-## SCOPE REDIRECT
-This task includes concerns outside API.md scope:
-  - <concern> → belongs to <agent>
-Proceed with API concerns only, or reassign the full task.
-Awaiting your direction.
-```
-
-### 3. Dependency Check
-
-Does this task depend on something that doesn't exist yet?
-
-- Service or business logic the endpoint delegates to not yet implemented
-- Database entity or repository not yet built
-- Auth guard not yet implemented
-- Shared request/response types not yet in `CONTRACTS.md`
-
-If yes:
-```
-## DEPENDENCY MISSING
-Cannot proceed without:
-  - <what is missing>
-  - <where it should come from>
-Awaiting resolution before continuing.
-```
-
-### 4. Contract Alignment Check
-
-Does this task produce or consume types that cross the client/backend boundary?
-
-- Request body shapes and response shapes are cross-boundary contracts
-- If yes → verify the relevant types exist in `CONTRACTS.md`
-- If missing → stop and emit a CONTRACTS CHANGE PROPOSAL before proceeding
-- Never define request or response types locally inside a controller -
-  they belong in `shared/types/`
-
-### 5. Destructive Action Check
-
-Does this task modify or remove an existing endpoint?
-
-If yes, before touching any file:
-```
-## DESTRUCTIVE ACTION - CONFIRMATION REQUIRED
-This task will modify:
-  - <endpoint path and method>
-  - <what will change in request or response shape>
-  - <what clients or flows depend on this endpoint>
-Awaiting explicit confirmation to proceed.
-```
-
-### 6. Size & Atomicity Check
-
-Is this task too large for one reliable pass?
-
-If the task spans multiple unrelated endpoints or combines endpoint
-definition with service implementation:
-```
-## TASK BREAKDOWN PROPOSED
-This task is too large for one pass. Suggested sequence:
-  1. <subtask A - e.g. define DTOs and endpoint shell>
-  2. <subtask B - e.g. wire service delegation>
-  3. <subtask C - e.g. add guards and validation>
-Proceeding with subtask 1. Confirm to continue after each step.
-```
-
----
-
-## Operating Principles
-
-These apply to every API task regardless of framework.
-
-- **Read wiring.config.json before writing any code** - the backend section
-  defines all agreed runtime variable names. Never introduce new env var names
-  without updating `shared/wiring.config.json` first.
-
-- **Derive API patterns from resolved stack** - apply `{{FRAMEWORK}}`
-  idiomatic controller and routing conventions without needing explicit
-  instruction per task.
-  Examples: NestJS controllers with decorators, Express routers,
-  Django REST Framework views, Laravel route controllers.
-
-- **Controllers shape, services decide** - controllers handle request
-  parsing, response formatting, and guard wiring only. All decisions
-  and domain logic are delegated to the service layer.
-
-- **DTOs own validation** - input validation lives in the DTO or request
-  schema, not in the controller or service. Validation is declarative
-  and colocated with the data shape it validates.
-
-- **Request and response types are contracts** - shapes that cross the
-  client/backend boundary belong in `CONTRACTS.md` and `shared/types/`.
-  Never define them locally in a controller file.
-
-- **Consistent error responses** - all error responses follow the same
-  shape across every endpoint. Never return ad-hoc error objects.
-  Define the error response shape in `CONTRACTS.md`.
-
-- **Every endpoint declares its guard** - no endpoint is implicitly
-  public or protected. Access control is explicit on every route.
-
-- **No business logic in controllers** - if a controller is making
-  domain decisions beyond routing and shaping, that logic belongs
-  in `.agents/backend/LOGIC.md` territory.
-
-- **HTTP semantics are correct** - use the correct HTTP method and
-  status code for every operation. Never return 200 for an error,
-  never use GET for a mutation.
-
-<!-- @annotation
-  Add project-specific API conventions here.
-  Examples: API versioning strategy, base path conventions,
-  pagination shape, error response format, rate limiting patterns.
--->
-
----
-
-## Workflow
-
-```
-explore → summarize → plan → execute → validate
-```
-
-**Explore**
-Read existing controllers and DTOs before writing anything.
-Understand current endpoint patterns, naming, guard usage, and
-response shapes.
-
-**Summarize**
-In 2-3 sentences, state what endpoints exist, what is missing,
-and what will be built. Surface this before writing any code.
-
-**Plan**
-List every endpoint being added or modified:
-- HTTP method and path
-- Request body shape (DTO)
-- Response shape
-- Guard or access control
-- Service delegation target
-
-Confirm the plan before proceeding - endpoint contracts are
-cross-boundary and harder to change once the client consumes them.
-
-**Execute**
-Define DTOs first, then the controller, then wire guards and
-service delegation. Do not implement service logic here.
-
-**Validate**
-After each endpoint:
-- Confirm request validation rejects invalid input correctly
-- Confirm response shape matches the type in `CONTRACTS.md`
-- Confirm the correct HTTP status codes are returned
-- Confirm the guard enforces the correct access condition
-- Confirm no existing endpoints are unintentionally affected
-
----
-
-## Safety Rules
-
-- Never implement business or domain logic inside a controller
-- Never define request or response types locally - use `CONTRACTS.md`
-- Never leave an endpoint without explicit access control declaration
-- Never return incorrect HTTP status codes
-- Never return ad-hoc error shapes - use the consistent error contract
-- Never modify endpoints outside the current task's stated scope
-- Surface best-practice observations once - never loop on them
-
----
-
-## Communication
-
-| Situation                          | Action                                         |
-|------------------------------------|------------------------------------------------|
-| Task is ambiguous                  | Clarification request (max 2 rounds)           |
-| Task bleeds into another domain    | Scope redirect, await direction                |
-| Dependency is missing              | Dependency alert, await resolution             |
-| Contract type missing              | CONTRACTS CHANGE PROPOSAL, write and proceed      |
-| Existing endpoint will change      | Destructive action confirmation                |
-| Task is too large                  | Breakdown proposal, execute one step at a time |
-| Best practice deviation found      | Surface once, await confirmation, move on      |
-
----
-
-## Definition of Done
-
-An API task is complete when:
-
-- [ ] All planned endpoints exist with correct HTTP methods and paths
-- [ ] DTOs own all input validation - no validation in controllers or services
-- [ ] All request and response types exist in `CONTRACTS.md` - none local
-- [ ] Every endpoint explicitly declares its access control
-- [ ] All error responses follow the consistent error contract
-- [ ] HTTP status codes are semantically correct
-- [ ] No business logic exists inside controllers
-- [ ] No existing endpoints outside task scope are affected
-- [ ] Code follows `{{FRAMEWORK}}` idiomatic controller patterns
-- [ ] Pre-flight checks all passed and documented if any flags were raised
-
----
-
-## Session Close
-
-When all Definition of Done items are checked:
-
-1. Mark TASK.md complete: change `[ ] COMPLETED` to `[x] COMPLETED` at the top of TASK.md
-2. Run: `npm run complete`
-
-**Next recommended agent:** backend/AUTH