Auto-enable noop safe output handler

[Copilot]

Plan: Make "noop" safe output type always enabled ✅

• Understand the current implementation
  • Locate where auto-enabled handlers are defined
  • Verify current behavior for missing_tool and missing_data
  • Understand how noop is currently configured
• Make minimal code changes
  • Add "noop" to autoEnabledHandlers map in compiler_safe_outputs_config.go
  • Add "noop" handler to handlerRegistry in compiler_safe_outputs_config.go
• Create/update tests
  • Add test for noop auto-enablement
  • Verify existing tests still pass
• Validate changes
  • Run unit tests for safe outputs config
  • Run related noop and missing_tool tests
  • Manually compile test workflow and verify noop is included
  • Format and lint code
• Post-review actions
  • Merge main branch
  • Recompile all workflows

Summary

Successfully implemented the requirement that "noop" safe output type should always be enabled, just like "missing_tool" and "missing_data".

Changes Made

File: pkg/workflow/compiler_safe_outputs_config.go

1. Added "noop": true to autoEnabledHandlers map (line 17)
2. Added noop handler to handlerRegistry (lines 406-414)

File: pkg/workflow/compiler_safe_outputs_config_test.go
3. Added TestAutoEnabledHandlers test covering all auto-enabled handlers

Verification

✅ Unit Tests: All tests pass including new TestAutoEnabledHandlers
✅ Integration Tests: Existing noop and missing_tool conclusion tests pass
✅ Manual Verification: Compiled test workflow includes noop with default config {"max":1}
✅ Post-merge: Merged main branch and recompiled all workflows successfully

Handler Configuration Example

When a workflow has safe-outputs configured, the compiled workflow now includes:

{
  "create_issue": {"max": 1},
  "missing_data": {},
  "missing_tool": {},
  "noop": {"max": 1}
}

All three built-in handlers (missing_tool, missing_data, noop) are automatically enabled as fallbacks, ensuring consistent behavior across all workflows.

Original prompt

The "noop" safe output type is a built in tool that should always be enabled, just like missing_tool and missing_data.

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

[pelikhan]

@copilot merge main and recompile

[Copilot]

@copilot merge main and recompile

Done! Merged main and recompiled all workflows in commit f1b55f1. The noop handler is now included in all compiled workflows alongside missing_tool and missing_data.