fix: remove required inputs from smoke-* workflows, always provide defaults

.github/workflows/smoke-claude.md

@@ -102,11 +102,13 @@ safe-outputs:
         inputs:
           channel:
             description: Slack channel name to post to
-            required: true
+            required: false
+            default: "#general"
             type: string
           message:
             description: Message text to post
-            required: true
+            required: false
+            default: ""
             type: string
         script: |
           const targetChannel = item.channel || "#general";

.github/workflows/smoke-copilot-arm.md

@@ -81,7 +81,8 @@ safe-outputs:
         inputs:
           message:
             description: "The message to send"
-            required: true
+            required: false
+            default: ""
             type: string
         permissions:
           contents: read

.github/workflows/smoke-copilot.md

@@ -89,7 +89,8 @@ safe-outputs:
         inputs:
           message:
             description: "The message to send"
-            required: true
+            required: false
+            default: ""
             type: string
         permissions:
           contents: read

pkg/workflow/label_trigger_parser.go

@@ -117,12 +117,15 @@ func expandLabelTriggerShorthand(entityType string, labelNames []string) map[str
 	}
 	triggerConfig["names"] = namesAny
 
-	// Create workflow_dispatch with item_number input
+	// Create workflow_dispatch with item_number input (not required so the workflow can be
+	// triggered manually without providing a value; the activation job will fall back to
+	// the event payload when item_number is not supplied).
 	workflowDispatchConfig := map[string]any{
 		"inputs": map[string]any{
 			"item_number": map[string]any{
 				"description": "The number of the " + getItemTypeName(entityType),
-				"required":    true,
+				"required":    false,
+				"default":     "",
 				"type":        "string",
 			},
 		},

pkg/workflow/label_trigger_parser_test.go

@@ -426,8 +426,13 @@ func TestExpandLabelTriggerShorthand(t *testing.T) {
 			}
 
 			required, ok := itemNumber["required"].(bool)
-			if !ok || !required {
-				t.Errorf("expandLabelTriggerShorthand() required = %v, want true", required)
+			if !ok || required {
+				t.Errorf("expandLabelTriggerShorthand() required = %v, want false", required)
+			}
+
+			defaultVal, ok := itemNumber["default"].(string)
+			if !ok || defaultVal != "" {
+				t.Errorf("expandLabelTriggerShorthand() default = %v, want empty string", defaultVal)
 			}
 
 			inputType, ok := itemNumber["type"].(string)

pkg/workflow/testdata/wasm_golden/fixtures/smoke-copilot.md

@@ -78,7 +78,8 @@ safe-outputs:
         inputs:
           message:
             description: "The message to send"
-            required: true
+            required: false
+            default: ""
             type: string
         permissions:
           contents: read

pkg/workflow/tools.go

@@ -170,12 +170,15 @@ func (c *Compiler) applyDefaults(data *WorkflowData, markdownPath string) error
 				}
 			}
 
-			// Add workflow_dispatch with item_number input for manual testing
+			// Add workflow_dispatch with item_number input for manual testing.
+			// Not required so the workflow can be triggered without providing a value;
+			// the activation job falls back to the event payload when item_number is omitted.
 			labelEventsMap["workflow_dispatch"] = map[string]any{
 				"inputs": map[string]any{
 					"item_number": map[string]any{
 						"description": "The number of the issue, pull request, or discussion",
-						"required":    true,
+						"required":    false,
+						"default":     "",
 						"type":        "string",
 					},
 				},