* fix(#3004): update Codex adapter collaboration-tool vocabulary The generated Codex skill adapter documented stale tool vocabulary: - wait(ids) → collaboration.wait_agent(timeout_ms=...) (the real tool), with explicit disambiguation from the unrelated exec-cell functions.wait - close_agent(id) unconditional → gated on tool visibility (same schema- detection pattern already used for spawn_agent's agent_type field) - Missing required task_name field and fork_turns parameter → added alongside the existing fork_context guidance (coexist, not replace) Updated the regression test to assert the new vocabulary (wait_agent not wait(ids), functions.wait disambiguation, task_name, fork_turns, tool_search gate on close_agent). * chore(#3004): backfill changeset PR number 3104 --------- Co-authored-by: sim <sim@local>
This commit is contained in:
@@ -194,8 +194,14 @@ describe('getCodexSkillAdapterHeader', () => {
|
||||
);
|
||||
assert.ok(result.includes('do not invent one-off effort literals'), 'keeps effort policy centralized');
|
||||
assert.ok(result.includes('fork_context'), 'documents fork_context default');
|
||||
assert.ok(result.includes('wait(ids)'), 'documents parallel wait pattern');
|
||||
// #3004: collaboration tool vocabulary must match Codex's actual schema.
|
||||
assert.ok(result.includes('wait_agent'), 'documents the real collaboration wait tool (wait_agent, not wait(ids))');
|
||||
assert.ok(!result.includes('wait(ids)'), 'must NOT contain the obsolete wait(ids) spelling');
|
||||
assert.ok(result.includes('functions.wait'), 'disambiguates from the unrelated exec-cell functions.wait tool');
|
||||
assert.ok(result.includes('task_name'), 'documents the required task_name field');
|
||||
assert.ok(result.includes('fork_turns'), 'documents the fork_turns parameter');
|
||||
assert.ok(result.includes('close_agent'), 'documents close_agent cleanup');
|
||||
assert.ok(result.includes('tool_search'), 'gates close_agent on tool visibility (schema detection)');
|
||||
assert.ok(result.includes('CHECKPOINT'), 'documents result markers');
|
||||
});
|
||||
});
|
||||
|
||||
Reference in New Issue
Block a user