@yemi33/minions 0.1.2261 → 0.1.2263

package/engine/cli.js

@@ -609,28 +609,6 @@ const commands = {
     try { shared.applyLegacyCcModelMigration(config, { logger: e.log }); }
     catch (err) { e.log('warn', `legacy ccModel migration failed: ${err.message}`); }
 
-    // One-time force-on of the CC worker pool. The pool has been the resolved
-    // default for copilot CC since PR #2492, but configs still carrying an
-    // explicit `ccUseWorkerPool: false` (set before the default flipped) stay
-    // opted out forever. Flip those to ON once, persisting the
-    // `engine._ccPoolForcedOnV1` marker so a deliberate later opt-out sticks.
-    // Disk-side re-derives from the on-disk copy so a concurrent dashboard
-    // write isn't clobbered; skipWriteIfUnchanged makes it a no-op once marked.
-    try {
-      const forced = shared.applyCcWorkerPoolForceOnMigration(config);
-      if (forced.changed) {
-        const configPath = path.join(shared.MINIONS_DIR, 'config.json');
-        shared.mutateJsonFileLocked(configPath, (onDisk) => {
-          shared.applyCcWorkerPoolForceOnMigration(onDisk);
-          return onDisk;
-        }, { defaultValue: {}, skipWriteIfUnchanged: true });
-        if (forced.flipped.length) {
-          e.log('info', `Forced CC worker pool ON — flipped explicit opt-out in: ${forced.flipped.join(', ')}`);
-          console.log(`  Forced CC worker pool ON (was explicitly off in: ${forced.flipped.join(', ')}).`);
-        }
-      }
-    } catch (err) { e.log('warn', `cc worker pool force-on migration failed: ${err.message}`); }
-
     // Repair work-item note links that broke before the rewriter covered
     // completion-report artifact links (W-mq1j85cj00055a8f). Notes the engine
     // auto-archived back then left the WI pill pointing at the gone

package/engine/create-pr-worktree.js

@@ -121,7 +121,7 @@ async function prepareCreatePrWorktree({
   try {
     await git(['-C', localPath, 'checkout', '--', '.']);
     for (const rel of untracked) {
-      try { fsm.rmSync(path.join(localPath, rel.replace(/\/$/, '')), { recursive: true, force: true }); } catch { /* ignore */ }
+      try { fsm.rmSync(path.join(localPath, rel.replace(/\/$/, '')), { recursive: true, force: true }); } catch (rmErr) { log('warn', `[cc-create-pr] could not remove untracked ${rel} from live checkout (will remain dirty): ${rmErr.message}`); }
     }
   } catch (e) {
     try { await git(['-C', localPath, 'worktree', 'remove', '--force', wtPath]); } catch { /* leak rather than double-throw */ }

package/engine/shared.js

@@ -3523,51 +3523,6 @@ function _resetLegacyCcModelMigrationFlag() {
   _legacyCcModelMigrationLogged = false;
 }
 
-/**
- * One-time force-on for the CC worker pool.
- *
- * The pool has been the resolved default for copilot CC since PR #2492
- * (`resolveCcUseWorkerPool` returns true when the config has no explicit
- * value). But configs that carried an explicit `ccUseWorkerPool: false` —
- * set before the default flipped, or copied from an old template — stay
- * opted out forever. This migration flips those explicit-false opt-outs to
- * `true` ONCE, on both surfaces the value lives in (`engine.ccUseWorkerPool`
- * which the resolver reads, and `features.ccUseWorkerPool` which drives the
- * Settings toggle / `isFeatureOn`), so the two stay consistent.
- *
- * Idempotency: records `engine._ccPoolForcedOnV1` after running. Once that
- * marker is set the migration never touches the value again — so an operator
- * who *deliberately* turns the pool off afterward is never re-forced. The
- * marker is persisted even when nothing needed flipping, which is what lets a
- * later opt-out be distinguished from a never-migrated config.
- *
- * Pure + idempotent: safe to call in-memory then re-apply to the on-disk copy
- * under a lock (mirrors backfillProjectWorkSourceDefaults). Returns
- * `{ changed, flipped }` — `changed` true means the config was mutated (marker
- * set and/or values flipped) and should be persisted; `flipped` lists which
- * surfaces ('engine'/'features') were actually turned on.
- */
-function applyCcWorkerPoolForceOnMigration(config) {
-  const result = { changed: false, flipped: [] };
-  if (!config || typeof config !== 'object') return result;
-  const engine = (config.engine && typeof config.engine === 'object') ? config.engine : null;
-  if (!engine) return result; // no engine section — retry next start, harmless
-  if (engine._ccPoolForcedOnV1) return result; // already forced once; respect later opt-out
-
-  if (engine.ccUseWorkerPool === false) {
-    engine.ccUseWorkerPool = true;
-    result.flipped.push('engine');
-  }
-  const features = (config.features && typeof config.features === 'object') ? config.features : null;
-  if (features && features.ccUseWorkerPool === false) {
-    features.ccUseWorkerPool = true;
-    result.flipped.push('features');
-  }
-  engine._ccPoolForcedOnV1 = true; // marker — never force again
-  result.changed = true; // marker always needs persisting on first run
-  return result;
-}
-
 // ─── Runtime Config Preflight Warnings ──────────────────────────────────────
 //
 // Emit non-fatal warnings about runtime/CLI configuration drift. Consumed by
@@ -4362,13 +4317,6 @@ const FAILURE_CLASS = {
   VERIFY_MISSING_PR: 'verify-missing-pr', // W-mqsk1ip00006cbae: a verify WI exited done but no PR was attached (neither _prUrl/_pr on the item nor a matching pull-requests.json entry for the plan). Flipped to failed so the plan doesn't silently advance without an E2E PR. Retryable — agent may have phantom-crashed before pushing the branch.
   UNKNOWN: 'unknown',                     // Unclassified failure
 };
-const ESCALATION_POLICY = {
-  NO_RETRY: 'no-retry',         // CONFIG_ERROR, PERMISSION_BLOCKED — never retry
-  RETRY_SAME: 'retry-same',     // MERGE_CONFLICT, BUILD_FAILURE, MAX_TURNS — retry same agent
-  RETRY_FRESH: 'retry-fresh',   // TIMEOUT, SPAWN_ERROR — retry with fresh session
-  HUMAN_REVIEW: 'human-review', // EMPTY_OUTPUT, OUT_OF_CONTEXT — flag for human
-  AUTO: 'auto',                 // UNKNOWN, NETWORK_ERROR — use default retry logic
-};
 
 // Structured completion protocol — fields agents must produce in ```completion blocks
 const COMPLETION_FIELDS = ['status', 'summary', 'files_changed', 'tests', 'pr', 'not_changed', 'failure_class', 'retryable', 'needs_rerun', 'verdict', 'artifacts', 'nonce', 'securityFlags'];
@@ -4724,51 +4672,6 @@ function agentCanUseRepo(agent, repoTarget) {
   return false;
 }
 
-/**
- * Check whether `agent` may invoke `toolName`. Case-sensitive comparison —
- * tool names are canonical (e.g. `Edit`, `Read`, `Bash`).
- *
- * Null/missing `allowed_tools` → permissive.
- * Empty array → denies everything.
- */
-function agentCanUseTool(agent, toolName) {
-  const manifest = resolveAgentManifest(agent);
-  if (manifest.allowed_tools == null) return true;
-  if (typeof toolName !== 'string' || toolName.length === 0) return false;
-  return manifest.allowed_tools.includes(toolName);
-}
-
-/**
- * Check whether `agent` may fetch `urlString`. Wildcard entries of the form
- * `*.example.com` match any subdomain AND the apex (`example.com`). Bare
- * `example.com` matches only the exact host.
- *
- * Null/missing `allowed_external_urls` → permissive.
- * Empty array → denies everything.
- * Malformed URL → denied (cannot extract host to compare).
- */
-function agentCanFetchUrl(agent, urlString) {
-  const manifest = resolveAgentManifest(agent);
-  if (manifest.allowed_external_urls == null) return true;
-  let host = '';
-  try {
-    host = new URL(String(urlString)).hostname.toLowerCase();
-  } catch { return false; }
-  if (!host) return false;
-  if (manifest.allowed_external_urls.length === 0) return false;
-  for (const raw of manifest.allowed_external_urls) {
-    const entry = String(raw).trim().toLowerCase();
-    if (!entry) continue;
-    if (entry.startsWith('*.')) {
-      const suffix = entry.slice(2);
-      if (host === suffix || host.endsWith('.' + suffix)) return true;
-    } else if (host === entry) {
-      return true;
-    }
-  }
-  return false;
-}
-
 /** Return the agent's effective memory scope. Unknown values fall back to 'shared'. */
 function agentMemoryScope(agent) {
   const manifest = resolveAgentManifest(agent);
@@ -8854,7 +8757,6 @@ module.exports = {
   resolveCopilotAgentDisabledMcpServers,
   resolveAgentHermeticHarness,
   applyLegacyCcModelMigration, _resetLegacyCcModelMigrationFlag,
-  applyCcWorkerPoolForceOnMigration,
   runtimeConfigWarnings,
   projectWorkSourceWarnings,
   backfillProjectWorkSourceDefaults,
@@ -8862,7 +8764,7 @@ module.exports = {
   WATCH_STATUS, WATCH_TARGET_TYPE, WATCH_CONDITION, WATCH_ABSOLUTE_CONDITIONS, WATCH_ACTION_TYPE,
   WATCH_STALLED_DEFAULT_TICKS, WATCH_STUCK_STAGE_DEFAULT_TICKS,
   PIPELINE_STATUS, STAGE_TYPE, MEETING_STATUS, AGENT_STATUS,
-  FAILURE_CLASS, ESCALATION_POLICY, COMPLETION_FIELDS,
+  FAILURE_CLASS, COMPLETION_FIELDS,
   HARNESS_USED_KINDS, HARNESS_USED_MAX_ENTRIES, HARNESS_USED_MAX_FIELD_LEN, normalizeHarnessUsed, groundHarnessUsed,
   DEFAULT_AGENT_METRICS,
   DEFAULT_AGENTS,
@@ -8874,8 +8776,6 @@ module.exports = {
   validateWorkspaceManifest,
   resolveAgentManifest,
   agentCanUseRepo,
-  agentCanUseTool,
-  agentCanFetchUrl,
   agentMemoryScope,
   mergeManifestAllowedTools,
   formatManifestRejection,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yemi33/minions",
-  "version": "0.1.2261",
+  "version": "0.1.2263",
   "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"