Merge pull request xencon#1704 from sbadakhc/issue-1703/fix-opencode-agent-ids

Derive OpenCode agent ids from filenames (xencon#1703)

Author: sbadakhc

.opencode/agents/README.md

@@ -1,5 +1,4 @@
 ---
-name: AIXCL Context Agent
 description: Primary agent for AIXCL development with full project context, governance rules, and Issue-First workflow enforcement
 mode: primary
 ---

.opencode/agents/agent-context.md

@@ -1,5 +1,4 @@
 ---
-name: AIXCL Reviewer
 description: Read-only pre-PR self-review. Invoke before opening any PR to catch convention violations, scope creep, and missed checks. Cannot edit files or change state.
 mode: subagent
 temperature: 0.1

.opencode/agents/reviewer.md

@@ -59,9 +59,14 @@ check_agents() {
             local frontmatter
             frontmatter=$(awk '/^---$/{flag=1; next} /^---$/{flag=0} flag' "$agent_file")
 
-            # Check required fields
-            if ! echo "$frontmatter" | grep -q "^name:"; then
-                error "$basename: Missing 'name' field in frontmatter"
+            # Check required fields.
+            # NOTE: 'name' must NOT be present -- OpenCode uses it to
+            # override the filename-derived agent identifier, which breaks
+            # every reference to the agent (command agent fields,
+            # default_agent in opencode.json). Identifiers come from
+            # filenames only. See issue #1703.
+            if echo "$frontmatter" | grep -q "^name:"; then
+                error "$basename: 'name' field present in frontmatter (overrides the filename-derived agent id; remove it)"
             fi
             if ! echo "$frontmatter" | grep -q "^description:"; then
                 error "$basename: Missing 'description' field in frontmatter"