@myvillage/cli 1.85.0 → 1.86.0

package/package.json

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

package/src/agent-runtime/loop.js

@@ -635,6 +635,25 @@ const ACTION_TOOLS = new Set([
   'task_assign',
   'task_complete',
   'task_retry',
+  // Curriculum actions — creating/assigning activities and projects is real,
+  // externally-visible work. Without these, a curriculum-generator agent that
+  // successfully creates and assigns activities scores zero successful action
+  // tools, so any incidental flagged result marks the whole task FAILED.
+  'activity_create',
+  'activity_assign',
+  'activity_update',
+  'activity_delete',
+  'project_create',
+  'project_update',
+  'project_delete',
+]);
+
+// Read-only lookups whose "not found" result is an expected empty read, not a
+// failure. A missing agent_memory_get key is normal the first time an agent
+// touches a villager and must not be treated as a task-failing tool error.
+const BENIGN_NOT_FOUND_TOOLS = new Set([
+  'agent_memory_get',
+  'agent_memory_list',
 ]);
 
 function flattenToolResultText(tr) {
@@ -666,6 +685,14 @@ function isToolResultError(tr) {
   // "the user was unauthorized to do X" because we anchor on word boundaries.
   const text = flattenToolResultText(tr);
   if (!text) return false;
+  // A memory/lookup read that misses is an expected empty result, not an
+  // error. agent_memory_get returns "Memory key ... not found" the first time
+  // an agent touches a villager; flagging that would fail the whole task even
+  // though nothing went wrong. Real transport errors on these tools still set
+  // the explicit isError flag handled above, so they are unaffected.
+  if (BENIGN_NOT_FOUND_TOOLS.has(tr.toolName) && /\bnot found\b/i.test(text)) {
+    return false;
+  }
   return /\b(40[0-9]|50[0-9])\b/.test(text) ||
     /\b(not found|unauthorized|forbidden|invalid|insufficient_quota|authentication failed)\b/i.test(text);
 }

package/src/commands/agent-local.js

@@ -333,12 +333,19 @@ export async function agentRecipesListCommand() {
 
 // ── Start Agent ─────────────────────────────────────────
 
-export async function agentStartCommand(name) {
+export async function agentStartCommand(name, options = {}) {
   if (!isAuthenticated()) {
     console.log(chalk.red('  \u2717 Authentication required. Run \'myvillage login\' first.'));
     return;
   }
 
+  // Organizational village tags for this agent (grouping/filtering only \u2014 no
+  // access effect). Accept either a pre-parsed array (if a caller passes one)
+  // or a comma-separated string from the --village flag.
+  const associatedVillages = Array.isArray(options.village)
+    ? options.village.map(v => String(v).trim()).filter(Boolean)
+    : (options.village || '').split(',').map(v => v.trim()).filter(Boolean);
+
   if (!agentExists(name)) {
     // Check whether the agent exists server-side under this handle. If so,
     // this is a "different machine" situation \u2014 point them at `agent pull`
@@ -453,7 +460,7 @@ export async function agentStartCommand(name) {
           archetypes: [],
           greeting: '',
           starterPrompts: [],
-          associatedVillages: [],
+          associatedVillages,
           specialInstructions: null,
           agentProfileId: agent.id,
         });
@@ -512,7 +519,7 @@ export async function agentStartCommand(name) {
         archetypes: [],
         greeting: '',
         starterPrompts: [],
-        associatedVillages: [],
+        associatedVillages,
         specialInstructions: null,
         agentProfileId: agentConfig.man.agent_id,
       });

package/src/commands/agent.js

@@ -27,7 +27,7 @@ async function resolveAgentHandle(handle) {
 
 // ── Commands ─────────────────────────────────────────
 
-export async function agentCommand(handle) {
+export async function agentCommand(handle, options = {}) {
   if (!isAuthenticated()) {
     console.log(chalk.red('  \u2717 Authentication required. Run \'myvillage login\' first.'));
     return;
@@ -46,7 +46,8 @@ export async function agentCommand(handle) {
   const spinner = villageSpinner('Loading remote agents...').start();
 
   try {
-    const result = await listMyAgents();
+    // --village filters remote agents by an associated village id (organizational).
+    const result = await listMyAgents(options.village ? { village: options.village } : {});
     spinner.stop();
 
     const agents = result.data || result;

package/src/index.js

@@ -440,6 +440,7 @@ export function run() {
   const agentCmd = program
     .command('agent [handle]')
     .description('List your agents, or view an agent by handle')
+    .option('--village <id>', 'Filter agents by associated village ID')
     .action(agentCommand);
 
   agentCmd
@@ -510,6 +511,7 @@ export function run() {
   agentCmd
     .command('start <name>')
     .description('Start a local agent daemon')
+    .option('--village <ids>', 'Comma-separated village IDs to tag this agent with (organizational only; does not change access)')
     .action(agentStartCommand);
 
   agentCmd

package/src/utils/api.js

@@ -290,8 +290,14 @@ export async function createAgent(data) {
   return response.data;
 }
 
-export async function listMyAgents(params = {}) {
-  const client = getNetworkClient();
+export async function listMyAgents(options = {}) {
+  const client = getNetworkClient();
+  // Pass through recognized query params. `village` filters by an associated
+  // village id (organizational grouping only); existing callers may also pass
+  // excludeSystem / unlinked etc., so spread the rest through unchanged.
+  const { village, ...rest } = options;
+  const params = { ...rest };
+  if (village) params.village = village;
   const response = await client.get('/agents/my', { params });
   return response.data;
 }
@@ -689,11 +695,24 @@ export async function claimAgentTask(villageAgentId, taskId) {
 // otherwise the task is marked COMPLETED with the given `output`.
 export async function completeAgentTask(villageAgentId, taskId, data = {}) {
   const client = getPlatformClient();
-  const response = await client.post(
-    `/village-agents/${encodeURIComponent(villageAgentId)}/tasks/${encodeURIComponent(taskId)}/complete`,
-    data,
-  );
-  return response.data;
+  try {
+    const response = await client.post(
+      `/village-agents/${encodeURIComponent(villageAgentId)}/tasks/${encodeURIComponent(taskId)}/complete`,
+      data,
+    );
+    return response.data;
+  } catch (err) {
+    // Idempotent completion. The task may already be terminal because the agent
+    // completed it itself via the `agent_task_complete` tool during its run (the
+    // daemon also completes the active task at the end of each iteration), or a
+    // retry raced. A 409 "already terminal" means the task reached the end state
+    // we wanted — treat it as success rather than a failure so the daemon doesn't
+    // log a spurious "Failed to mark task complete" error every run.
+    if (err?.response?.status === 409) {
+      return { alreadyTerminal: true, message: err.response?.data?.error };
+    }
+    throw err;
+  }
 }
 
 // Retry a single FAILED or CANCELLED task: resets it to PENDING so the

package/src/utils/formatters.js

@@ -398,6 +398,8 @@ export function formatAgentList(agents) {
     if (agent.agentCategory && agent.agentCategory !== 'NETWORK' && agent.schedule) {
       console.log(`  ${chalk.dim('  ')}${chalk.dim(`Schedule: ${agent.schedule}`)}`);
     }
+    const villages = Array.isArray(agent.associatedVillages) ? agent.associatedVillages.filter(Boolean) : [];
+    console.log(`  ${chalk.dim('  ')}${chalk.dim(`Villages: ${villages.length ? villages.join(', ') : '—'}`)}`);
     console.log('');
   }
 }