atris 3.60.0 → 3.62.0

package/lib/functional-owner.js

@@ -3,6 +3,7 @@
 const fs = require('fs');
 const path = require('path');
 const escapeRegExp = require('./escape-regexp');
+const { isMemberParked } = require('./member-park');
 
 const ENGINE_OWNER_IDS = new Set([
   'atris-2-fast',
@@ -87,11 +88,17 @@ function listWorkspaceMemberSlugs(root) {
   }
 }
 
-function firstAvailableFunctionalOwner(candidates, existingMembers) {
+// Members owner inference may pick: every member minus parked ones. An
+// explicit --owner still reaches a parked member.
+function assignableMemberSlugs(root, existingMembers) {
+  return new Set([...existingMembers].filter(slug => !isMemberParked(root || process.cwd(), slug)));
+}
+
+function firstAvailableFunctionalOwner(candidates, existingMembers, assignable = existingMembers) {
   for (const candidate of candidates) {
     const owner = normalizeOwnerSlug(candidate);
     if (!owner) continue;
-    if (existingMembers.size === 0 || existingMembers.has(owner)) return owner;
+    if (existingMembers.size === 0 || assignable.has(owner)) return owner;
   }
   return null;
 }
@@ -120,6 +127,7 @@ function proposedMemberSlugFromTask({ title, tag }) {
 function resolveFunctionalOwner({ requestedOwner, title, tag, note, goal, root, fallbackOwners } = {}) {
   const requested = normalizeOwnerSlug(requestedOwner);
   const existingMembers = listWorkspaceMemberSlugs(root);
+  const assignable = assignableMemberSlugs(root, existingMembers);
   const highSignalText = [title, note, goal].filter(Boolean).join(' ').toLowerCase();
   const tagText = tag && tag !== true ? String(tag).toLowerCase() : '';
   const text = [highSignalText, tagText].filter(Boolean).join(' ');
@@ -144,7 +152,8 @@ function resolveFunctionalOwner({ requestedOwner, title, tag, note, goal, root,
 
   const exact = firstAvailableFunctionalOwner(
     FUNCTIONAL_MEMBER_TOPICS.filter(topic => text.includes(topic.owner)).map(topic => topic.owner),
-    existingMembers
+    existingMembers,
+    assignable
   );
   if (exact) {
     const remapped = Boolean(requested && isEngineOwner(requested) && requested !== exact);
@@ -170,7 +179,7 @@ function resolveFunctionalOwner({ requestedOwner, title, tag, note, goal, root,
     }))
     .filter(row => row.score > 0)
     .sort((a, b) => b.score - a.score);
-  const inferred = firstAvailableFunctionalOwner(scored.map(row => row.owner), existingMembers);
+  const inferred = firstAvailableFunctionalOwner(scored.map(row => row.owner), existingMembers, assignable);
   if (inferred) {
     const remapped = Boolean(requested && isEngineOwner(requested) && requested !== inferred);
     return {
@@ -184,8 +193,8 @@ function resolveFunctionalOwner({ requestedOwner, title, tag, note, goal, root,
     };
   }
 
-  const fallback = firstAvailableFunctionalOwner(fallbackOwners || ['architect', 'task-planner', 'mission-lead', 'validator'], existingMembers)
-    || Array.from(existingMembers).find(owner => !isEngineOwner(owner))
+  const fallback = firstAvailableFunctionalOwner(fallbackOwners || ['architect', 'task-planner', 'mission-lead', 'validator'], existingMembers, assignable)
+    || Array.from(assignable).find(owner => !isEngineOwner(owner))
     || 'architect';
   const remapped = Boolean(requested && requested !== fallback);
   return {