@yemi33/minions 0.1.2442 → 0.1.2444

package/docs/copilot-cli-schema.md

@@ -116,12 +116,12 @@ warns when required permission-bypass flags are absent.
 ```powershell
 # Test A: stdin without -p — works.
 "Say only the word: pong" |
-  copilot --output-format json -s --allow-all --no-ask-user --autopilot --log-level error
+  copilot --output-format json -s --allow-all --no-ask-user --autopilot
 # EXIT=0; user.message.data.content = "Say only the word: pong\r\n"; assistant replied "pong".
 
 # Test B: -p "<40_000-char string>" — Windows OS rejects spawn.
 $big = "x" * 40000
-copilot -p $big --output-format json -s --allow-all --no-ask-user --autopilot --log-level error
+copilot -p $big --output-format json -s --allow-all --no-ask-user --autopilot
 # Program 'copilot.exe' failed to run:
 #   The filename or extension is too long.
 #   (CreateProcess ARG_MAX limit, ~32 KB on Windows)
@@ -176,7 +176,7 @@ Empirically confirmed flags for non-interactive Copilot invocations:
 | `--allow-all` | **required** | Equivalent to `--allow-all-tools --allow-all-paths --allow-all-urls`. Without this the CLI prompts for every tool/path use, which deadlocks in stdin/stdout mode. |
 | `--no-ask-user` | **required** | Removes the `ask_user` tool. Without it the agent can stall waiting for human input. |
 | `--autopilot` | for multi-turn agency | Enables `task_complete`-driven multi-turn loop. **Without it** the session ends after one assistant response (see §3.1). |
-| `--log-level error` | recommended | Suppresses INFO/DEBUG diagnostics that aren't part of the JSONL stream. |
+| `--log-level <level>` | **do not emit** | Copilot CLI 1.0.76-1 exits silently with code 1 for explicit non-default levels. Omit the flag and use the CLI default; `--output-format json` still keeps stdout machine-readable. |
 | `--disable-builtin-mcps` | gated by config | Disables `github-mcp-server`. Default-on for Minions (`copilotDisableBuiltinMcps: true`) to prevent split-brain PR creation. |
 | `--no-color` | always emitted | Keeps any incidental CLI rendering free of ANSI color. |
 | `--plain-diff` | always emitted | Keeps incidental diff rendering plain even though normal output is JSONL. |
@@ -499,7 +499,7 @@ Constructed test: created `AGENTS.md` in cwd with content
 
 ```text
 # A) Default behavior — AGENTS.md is loaded
-PS> "Just say hello." | copilot --output-format json -s --allow-all --no-ask-user --autopilot --log-level error
+PS> "Just say hello." | copilot --output-format json -s --allow-all --no-ask-user --autopilot
 {"type":"assistant.message", ..., "content": "Hello. __AGENTS_LOADED__"}      ← marker present
 
 ```
@@ -602,11 +602,11 @@ When implementing `engine/runtimes/copilot.js`:
    - Recognize npm shims returned by PATH and resolve the package's JavaScript
      entry; no separate npm-prefix scan is needed.
 3. `buildArgs(opts)` always emits:
-   `--output-format json -s --no-color --plain-diff --allow-all --no-ask-user --autopilot --log-level error`
+   `--output-format json -s --no-color --plain-diff --allow-all --no-ask-user --autopilot`
    plus the conditional flags from §3, plus `--disable-builtin-mcps` per
    `opts.disableBuiltinMcps`,
    plus `--attachment <path>` (repeatable) for each image in `opts.images` (§10a).
-   **Never** emit `--verbose`.
+   **Never** emit `--verbose` or `--log-level`.
 4. `buildPrompt()` injects `<system>...</system>\n\n` when sysprompt is
    non-empty on a fresh call; a resumed call (`opts.sessionId`) passes through
    only the new user prompt (§2).

package/engine/runtimes/copilot.js

@@ -434,12 +434,15 @@ function _mapEffort(level) {
 //
 // Always-on baseline (per docs/copilot-cli-schema.md §3 and the PRD spec):
 //   --output-format json -s --no-color --plain-diff --autopilot
-//   --allow-all --no-ask-user --log-level error
+//   --allow-all --no-ask-user
 //
 // Conditional flags only emitted when their corresponding opt is set/truthy.
 // Copilot has no --verbose flag — never emit it. The `bare` / `maxBudget` /
 // `fallbackModel` opts are silently ignored (their capability flags are false
 // so engine code shouldn't pass them, but we tolerate them gracefully).
+// Do not emit --log-level: Copilot CLI 1.0.76-1 exits silently with code 1
+// when given an explicit non-default level. Omitting it preserves the CLI's
+// compatible default while --output-format json keeps stdout machine-readable.
 
 // Map MIME type to file extension for --attachment temp file naming.
 const _MIME_EXT = {
@@ -475,7 +478,6 @@ function buildArgs(opts = {}) {
     '--autopilot',
     '--allow-all',
     '--no-ask-user',
-    '--log-level', 'error',
   ];
 
   if (Array.isArray(addDirs)) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yemi33/minions",
-  "version": "0.1.2442",
+  "version": "0.1.2444",
   "description": "Multi-agent AI dev team that runs from ~/.minions/ — five autonomous agents share a single engine, dashboard, and knowledge base",
   "bin": {
     "minions": "bin/minions.js"