@myvillage/cli 1.82.0 → 1.83.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@myvillage/cli",
-  "version": "1.82.0",
+  "version": "1.83.0",
   "description": "MyVillageOS CLI for community developers",
   "type": "module",
   "bin": {

package/src/agent-runtime/loop.js

@@ -112,6 +112,14 @@ export async function agentLoop(agentName, { signal }) {
     model = anthropic(modelId);
   }
   const maxTokens = config.brain?.max_tokens_per_loop || 8000;
+  // Max agentic steps (one model turn + its tool executions) per loop
+  // iteration. A task like "enumerate N villagers' knowledge, then Slack each
+  // lead" needs a step per discovery wave plus the action steps; 10 was too low
+  // — the model would exhaust the budget mid-task (e.g. right after listing
+  // Slack channels but before sending). Configurable per agent via
+  // brain.max_steps_per_loop; default 25. Cost stays bounded by
+  // max_tokens_per_loop and the rate limiter regardless of this ceiling.
+  const maxSteps = config.brain?.max_steps_per_loop || 25;
 
   // Initialize MCP tools
   let tools;
@@ -293,10 +301,10 @@ Guidelines:
       // AI SDK v6 removed `maxSteps` — the loop now uses `stopWhen` with
       // a StopCondition predicate. The default is `stepCountIs(1)`, so
       // without this the model gets exactly ONE round of tool calls and
-      // returns immediately. We give it up to 10 steps so a task like
-      // "find any README under the workspace" can do
-      // list_allowed_directories → list_directory → search_files → ...
-      // until it's actually done.
+      // returns immediately. We give it up to `maxSteps` steps (default 25,
+      // see above) so a multi-step task like "find the low-activity villagers,
+      // then message each lead" can do discover → reason → act across enough
+      // turns to actually finish, instead of stopping mid-task.
       //
       // Anthropic prompt caching: we mark the system block as ephemeral so
       // everything before it (tool schemas + system prompt) is cached.
@@ -319,7 +327,7 @@ Guidelines:
           { role: 'user', content: context },
         ],
         tools,
-        stopWhen: stepCountIs(10),
+        stopWhen: stepCountIs(maxSteps),
         maxOutputTokens: maxTokens,
       });
 

package/src/commands/agent.js

@@ -92,10 +92,14 @@ export async function agentCreateCommand() {
     // org-chart hierarchy. Failure here is non-fatal — just skip the picker.
     let parentChoices = [];
     try {
-      const existing = await listMyAgents();
+      // excludeSystem: don't offer system personas (Griot et al.) as parents —
+      // admins technically own them, but they aren't part of a villager's org chart.
+      const existing = await listMyAgents({ excludeSystem: true });
       const mine = existing.data || existing;
       if (Array.isArray(mine)) {
-        parentChoices = mine.map(a => ({ name: `@${a.handle} — ${a.displayName}`, value: a.id }));
+        parentChoices = mine
+          .filter(a => !a.isSystemAgent)
+          .map(a => ({ name: `@${a.handle} — ${a.displayName}`, value: a.id }));
       }
     } catch {
       parentChoices = [];
@@ -311,11 +315,12 @@ export async function agentEditCommand(handle) {
     // avoid the trivial cycle; deeper cycles are rejected by the backend).
     let parentChoices = [];
     try {
-      const existing = await listMyAgents();
+      // excludeSystem: system personas (Griot et al.) aren't valid parents.
+      const existing = await listMyAgents({ excludeSystem: true });
       const mine = existing.data || existing;
       if (Array.isArray(mine)) {
         parentChoices = mine
-          .filter(a => a.id !== agent.id)
+          .filter(a => a.id !== agent.id && !a.isSystemAgent)
           .map(a => ({ name: `@${a.handle} — ${a.displayName}`, value: a.id }));
       }
     } catch {