devrites 3.2.3 → 3.2.4

package/CHANGELOG.md

@@ -2,6 +2,12 @@
 
 All notable changes to DevRites are documented here. The format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/) and DevRites adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html). Releases are generated automatically by [semantic-release](https://semantic-release.gitbook.io/) from Conventional Commits on `main`.
 
+## [3.2.4](https://github.com/ViktorsBaikers/DevRites/compare/v3.2.3...v3.2.4) (2026-07-25)
+
+### Fixed
+
+* **agents:** handle hidden v2 agent_type schema ([31ece9a](https://github.com/ViktorsBaikers/DevRites/commit/31ece9aad686fa3a3fa0308d5a3ede11f5b58c07))
+
 ## [3.2.3](https://github.com/ViktorsBaikers/DevRites/compare/v3.2.2...v3.2.3) (2026-07-25)
 
 ### Fixed

package/README.md

@@ -25,7 +25,7 @@ final commit, push, and tag, and it requires a typed `GO` confirmation.
 Unattended runs may create local WIP checkpoint commits along the way, but only
 Ship collapses and pushes them.
 
-**Status:** [`v3.2.3`](https://github.com/ViktorsBaikers/DevRites/releases/tag/v3.2.3): see [`CHANGELOG.md`](CHANGELOG.md) for release notes.
+**Status:** [`v3.2.4`](https://github.com/ViktorsBaikers/DevRites/releases/tag/v3.2.4): see [`CHANGELOG.md`](CHANGELOG.md) for release notes.
 
 ## Quick start
 

package/engine/hooks_agent_dispatch.go

@@ -802,7 +802,7 @@ func incompleteDispatchReason(armed map[string]struct{}, attempts []*agentDispat
 		return "DevRites could not determine this skill's required agents because its installed required-agent-roles contract is invalid. Reinstall or repair the DevRites skill pack before continuing."
 	}
 	return fmt.Sprintf(
-		"DevRites dispatch for %s is not complete. On MultiAgent V2 call spawn_agent with agent_type=%s, a unique task_name, and fork_turns=\"none\" so Codex loads .codex/agents/%s.toml natively. On V1 use agent_type=%s with fork_turns=\"none\" and name that role TOML in the message. Wait for the returned child and use its non-empty result. Do not call wait before spawn_agent and do not synthesize the agent result.",
+		"DevRites dispatch for %s is not complete. On MultiAgent V2 call spawn_agent with agent_type=%s, a unique task_name, and fork_turns=\"none\" so Codex loads .codex/agents/%s.toml natively. GPT-5.6 V2 may omit agent_type from the visible tool schema even though the runtime accepts it; send agent_type anyway rather than using a default child. On V1 use agent_type=%s with fork_turns=\"none\" and name that role TOML in the message. Wait for the returned child and use its non-empty result. Do not call wait before spawn_agent and do not synthesize the agent result.",
 		role, role, role, expectedAgentType(role),
 	)
 }

package/engine/tests/hook_test.go

@@ -1218,7 +1218,10 @@ func TestCodexAgentDispatchBlocksFalseWaitAndStop(t *testing.T) {
 	if err := json.Unmarshal([]byte(strings.TrimSpace(out)), &stopDecision); err != nil {
 		t.Fatalf("invalid Stop response: %v\n%s", err, out)
 	}
-	if stopDecision.Decision != "block" || !strings.Contains(stopDecision.Reason, "spawn_agent") {
+	if stopDecision.Decision != "block" ||
+		!strings.Contains(stopDecision.Reason, "spawn_agent") ||
+		!strings.Contains(stopDecision.Reason, "visible tool schema") ||
+		!strings.Contains(stopDecision.Reason, "send agent_type anyway") {
 		t.Fatalf("false completion not blocked: %#v", stopDecision)
 	}
 

package/pack/.claude/skills/devrites-lib/reference/standards/agents.md

@@ -67,11 +67,12 @@ field from the installed skill at `UserPromptSubmit` and arms a fail-closed comp
 receipt for every listed role; the engine derives roles from skill metadata.
 Conditional scouts and reviewers remain owned by their explicit phase triggers.
 
-Claude uses `Agent` (`Task` alias); Codex uses `spawn_agent`. V2 sends the exact named
-role, a unique `task_name`, and `fork_turns="none"`; Codex loads its TOML natively.
-Because V2 lifecycle calls bypass hooks, Stop/reconcile verify the durable rollout's
-role, instructions, wait, completion, and delivered result. V1 uses guarded
-`explorer`/`worker` with injected rules and a lifecycle receipt. Prose isn't evidence.
+Claude: `Agent`; Codex: `spawn_agent`. V2 calls the named `agent_type` with
+unique `task_name` and `fork_turns="none"`. GPT-5.6 may hide `agent_type` from the
+schema, but runtime accepts it and loads the TOML; never use `default`. V2 bypasses
+hooks, so Stop/reconcile verify role, native instructions, wait, completion, and result.
+V1 uses guarded `explorer`/`worker` with injected rules and receipt. Prose
+isn't evidence.
 
 ## File-backed dispatch contract
 

package/pack/generated/claude/skills/devrites-lib/reference/standards/agents.md

@@ -67,11 +67,12 @@ field from the installed skill at `UserPromptSubmit` and arms a fail-closed comp
 receipt for every listed role; the engine derives roles from skill metadata.
 Conditional scouts and reviewers remain owned by their explicit phase triggers.
 
-Claude uses `Agent` (`Task` alias); Codex uses `spawn_agent`. V2 sends the exact named
-role, a unique `task_name`, and `fork_turns="none"`; Codex loads its TOML natively.
-Because V2 lifecycle calls bypass hooks, Stop/reconcile verify the durable rollout's
-role, instructions, wait, completion, and delivered result. V1 uses guarded
-`explorer`/`worker` with injected rules and a lifecycle receipt. Prose isn't evidence.
+Claude: `Agent`; Codex: `spawn_agent`. V2 calls the named `agent_type` with
+unique `task_name` and `fork_turns="none"`. GPT-5.6 may hide `agent_type` from the
+schema, but runtime accepts it and loads the TOML; never use `default`. V2 bypasses
+hooks, so Stop/reconcile verify role, native instructions, wait, completion, and result.
+V1 uses guarded `explorer`/`worker` with injected rules and receipt. Prose
+isn't evidence.
 
 ## File-backed dispatch contract
 

package/pack/generated/codex/skills/devrites-lib/reference/standards/agents.md

@@ -67,11 +67,12 @@ field from the installed skill at `UserPromptSubmit` and arms a fail-closed comp
 receipt for every listed role; the engine derives roles from skill metadata.
 Conditional scouts and reviewers remain owned by their explicit phase triggers.
 
-Claude uses `Agent` (`spawn_agent` alias); Codex uses `spawn_agent`. V2 sends the exact named
-role, a unique `task_name`, and `fork_turns="none"`; Codex loads its TOML natively.
-Because V2 lifecycle calls bypass hooks, Stop/reconcile verify the durable rollout's
-role, instructions, wait, completion, and delivered result. V1 uses guarded
-`explorer`/`worker` with injected rules and a lifecycle receipt. Prose isn't evidence.
+Claude: `Agent`; Codex: `spawn_agent`. V2 calls the named `agent_type` with
+unique `task_name` and `fork_turns="none"`. GPT-5.6 may hide `agent_type` from the
+schema, but runtime accepts it and loads the TOML; never use `default`. V2 bypasses
+hooks, so Stop/reconcile verify role, native instructions, wait, completion, and result.
+V1 uses guarded `explorer`/`worker` with injected rules and receipt. Prose
+isn't evidence.
 
 ## File-backed dispatch contract
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "devrites",
-  "version": "3.2.3",
+  "version": "3.2.4",
   "description": "DevRites: a disciplined senior-engineer workflow pack for Claude Code and Codex",
   "license": "SEE LICENSE IN LICENSE",
   "homepage": "https://github.com/ViktorsBaikers/DevRites#readme",