-
Notifications
You must be signed in to change notification settings - Fork 262
Closed
Closed
Copy link
Labels
Description
Objective
Make noop available as a default safe-output tool in all compiled workflows so agents can always signal a no-op completion without misconfiguration.
Context
From audit discussion #18635 (2026-02-26→2026-02-27):
Note on
noopmissing: Duplicate Code Detector explicitly reported needing to callnoop(no duplicates found) but the tool wasn't in its allowed list. This is a workflow misconfiguration —noopshould be available to all workflows as a no-op completion signal.
Currently, noop must be explicitly added to each workflow's allowed safe-output tools list. When it's missing, agents that find nothing to do (the correct behavior) cannot signal completion properly.
Approach
- Find where safe-output allowed tools are determined during compilation in
pkg/workflow/ - Find where the
(safe-output-tools)system prompt section is constructed (lists available tools to the agent) - Ensure
noopis always included in the allowed tools list, regardless of the workflow's explicit tool configuration - Update compilation logic so
noopis injected as a baseline tool even if not specified in frontmatter - Add or update a test verifying
noopis always present in compiled workflow tool lists
Files to Investigate
pkg/workflow/— compiler and safe-output tool list generation.github/workflows/duplicate-code-detector.md— the affected workflow (addnoopas a fix)
Acceptance Criteria
- Every compiled workflow includes
noopin its safe-output tools list - Adding
noopdoes not break workflows that already list it explicitly (no duplicates) - Duplicate Code Detector workflow explicitly has
noopavailable - Unit test verifies
noopis always present after compilation -
make agent-finishpasses
Generated by Plan Command for issue #discussion #18635
- expires on Mar 1, 2026, 6:37 AM UTC
Reactions are currently unavailable