Make item_number optional for add_comment safe output tool

[Copilot]

The add_comment tool required item_number even when the target could be resolved from workflow context (triggering issue/PR/discussion), causing validation errors.

Changes

• Schema updates: Removed item_number from required fields in safe_outputs_tools.json
• Resolution logic: Added explicit check for item_number before falling back to context resolution
• Test coverage: Added tests for explicit item_number with triggering target and context-based resolution

Resolution Priority

// 1. Explicit item_number (if provided)
if (item.item_number !== undefined) {
  itemNumber = item.item_number;
}
// 2. Discussion context
else if (isDiscussionContext) {
  itemNumber = context.payload?.discussion?.number;
}
// 3. Target resolution (issues/PRs)
else {
  const result = resolveTarget({...});
  itemNumber = result.number;
}

Backward compatible - existing usage with explicit item_number continues to work.

Original prompt

Update safe output tool add_comment:

The item_number parameter must be optional (not required).

See error:

└ calling "tools/call": Invalid arguments: missing or empty 'item_number'

If the item_number is missing, empty, apply the resolution to determine the item_number from the current context.

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[github-actions]

📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤

[github-actions]

✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟

[github-actions]

🎬 THE END — Smoke Claude MISSION: ACCOMPLISHED! The hero saves the day! ✨

[github-actions]

💀 Blimey! Changeset Generator failed and walked the plank! No treasure today, matey! ☠️

[github-actions]

Smoke Test Results - Claude Engine (Run #20891531083)

Last 2 Merged PRs:

• Make tool parameter optional in missing_tool safe output #9600: Make tool parameter optional in missing_tool safe output
• Skip runtime detection for containerized MCP servers #9587: Skip runtime detection for containerized MCP servers

Test Results:

• ✅ GitHub MCP: Retrieved PRs successfully
• ✅ Serena: Tool active (v0.1.4, Go project detected)
• ✅ Playwright: Navigated to github.com (title verified)
• ❌ safeinputs-gh: Tool not found in system
• ✅ File Writing: Test file created successfully
• ✅ Bash Tool: File verified with cat command

Overall Status: PARTIAL PASS (5/6 tests passed)

Note: safeinputs-gh tool doesn't exist - may need creation or is referenced incorrectly.

AI generated by Smoke Claude