Skip to content

Comments

fix(taskctl): add taskctl permission to adversarial-pipeline and fix verdict prompt syntax#261

Merged
randomm merged 1 commit intodevfrom
fix/issue-259-260-adversarial-taskctl-permission
Feb 20, 2026
Merged

fix(taskctl): add taskctl permission to adversarial-pipeline and fix verdict prompt syntax#261
randomm merged 1 commit intodevfrom
fix/issue-259-260-adversarial-taskctl-permission

Conversation

@randomm
Copy link
Owner

@randomm randomm commented Feb 20, 2026

Fixes #259, #260

Problems Fixed

Bug 1: Missing taskctl permission (#259)

adversarial-pipeline had "*": "deny" with no taskctl exception. The taskctl MCP tool was completely blocked — the agent could never submit a verdict regardless of prompt instructions.

Fix: Added taskctl: "allow" to adversarial-pipeline permissions.

Bug 2: Wrong prompt syntax (#259)

Prompt instructed the agent to call taskctl verdict <taskId> --verdict APPROVED (CLI bash syntax). The agent correctly identified it couldn't find the binary and gave up.

Fix: Updated prompt to describe taskctl as an MCP tool with correct parameter names (command, taskId, verdict, verdictSummary, verdictIssues).

Tests Added (#260)

New test/tasks/agent-permissions.test.ts:

  • adversarial-pipeline has taskctl permission
  • adversarial-pipeline has bash permission
  • developer-pipeline denies taskctl (separation of concerns)
  • adversarial-pipeline denies everything else (minimal permissions)
  • developer-pipeline denies task tool

@randomm randomm merged commit 5879706 into dev Feb 20, 2026
1 check passed
@randomm randomm deleted the fix/issue-259-260-adversarial-taskctl-permission branch February 20, 2026 11:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(taskctl): adversarial-pipeline agent missing taskctl permission and prompt uses wrong invocation syntax

1 participant