Skip to content

Fix gh.md and smoke workflows to explicitly reference safeinputs-gh tool name#5834

Merged
pelikhan merged 6 commits intomainfrom
copilot/fix-gh-md-prompting-safeinputs
Dec 8, 2025
Merged

Fix gh.md and smoke workflows to explicitly reference safeinputs-gh tool name#5834
pelikhan merged 6 commits intomainfrom
copilot/fix-gh-md-prompting-safeinputs

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Dec 8, 2025

The gh.md prompting ambiguously referred to "gh safe-input tool" which could be confused with the gh CLI. The actual MCP tool name is safeinputs-gh.

Changes

  • Tool description: Added explicit mention that tool is accessible as safeinputs-gh
  • Prompt instructions: Changed from "use the gh safe-input tool" to "use the safeinputs-gh tool"
  • Examples: Updated all invocation examples and added incorrect usage pattern to prevent confusion
  • Documentation: Updated HTML comment section to reference safeinputs-gh consistently
  • Smoke workflows: Updated all smoke-* workflow test requirements to use safeinputs-gh tool name consistently

Before/After

Before:

**IMPORTANT**: Always use the `gh` safe-input tool for GitHub CLI commands...

Use the gh safe-input tool with args: "pr list --limit 5"

After:

**IMPORTANT**: Always use the `safeinputs-gh` tool for GitHub CLI commands...

Use the safeinputs-gh tool with args: "pr list --limit 5"

Lock files for workflows importing gh.md and all smoke workflows updated with new prompts.

Original prompt

Fix the gh.md prompting to specifically call safeinputs-gh tool NOT the cli.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.


Changeset

  • Type: patch
  • Description: Fix gh.md to explicitly reference the safeinputs-gh tool name instead of the ambiguous "gh" safe-input tool; update prompt text and examples to use safeinputs-gh consistently.

🏴‍☠️ Ahoy! This treasure was crafted by Changeset Generator


Changeset

  • Type: patch
  • Description: Fix gh.md to explicitly reference the safeinputs-gh tool name instead of the ambiguous "gh" safe-input tool; update prompt text and examples to use safeinputs-gh consistently.

🏴‍☠️ Ahoy! This treasure was crafted by Changeset Generator

Copilot AI and others added 2 commits December 8, 2025 07:57
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
- Update description to mention the tool is accessible as 'safeinputs-gh'
- Change prompt examples to use 'safeinputs-gh' instead of 'gh safe-input tool'
- Add example of incorrect usage (wrong tool name) to clarify confusion
- Update HTML comment documentation to match new naming
- Recompile affected workflows to update embedded prompts

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix gh.md prompting for safeinputs-gh tool Fix gh.md to explicitly reference safeinputs-gh tool name Dec 8, 2025
Copilot AI requested a review from pelikhan December 8, 2025 08:08
@pelikhan pelikhan marked this pull request as ready for review December 8, 2025 13:30
@pelikhan pelikhan added the smoke label Dec 8, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 8, 2025

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

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 8, 2025

🤖 ALERT: Smoke Copilot No Firewall failed. ANOMALY_DETECTED. REPAIR_REQUIRED.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 8, 2025

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

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 8, 2025

🔮 The ancient spirits stir... Smoke Codex awakens to divine this pull request...

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 8, 2025

🎉 Yo ho ho! Changeset Generator found the treasure and completed successfully! ⚓💰

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 8, 2025

💥 WHOOSH! Smoke Claude springs into action on this pull request! [Panel 1 begins...]

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 8, 2025

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 8, 2025

Smoke Test Results - Claude Engine (Run #20029841676)

Last 2 Merged PRs:

Test Results:

  • ✅ GitHub MCP - Retrieved PR list
  • ✅ File Writing - Created /tmp/gh-aw/agent/smoke-test-claude-20029841676.txt
  • ✅ Bash Tool - Verified file creation
  • ✅ Playwright MCP - Navigated to github.com, title contains "GitHub"
  • ✅ Cache Memory - Created /tmp/gh-aw/cache-memory/smoke-test-20029841676.txt
  • ❌ Safe Input gh Tool - Command not authenticated (expected in GH Actions)

Overall Status: PASS (5/6 tests successful, gh auth expected to fail)

💥 [THE END] — Illustrated by Smoke Claude fer issue #5834 🗺️

@pelikhan
Copy link
Copy Markdown
Collaborator

pelikhan commented Dec 8, 2025

@copilot update "smoke *" agentic workflows with update naming in prompt.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 8, 2025

Smoke test (Codex):

  • Last 2 merged PRs: Fix test extraction function and JSDoc type annotation; Add firewall health endpoint test and available tools display to smoke-copilot workflow ✅
  • /tmp/gh-aw/agent/smoke-test-codex-20029841706.txt created/read ✅
  • /tmp/gh-aw/cache-memory/smoke-test-20029841706.txt created/read ✅
  • Playwright github.com title contains “GitHub” ✅
  • gh issues list --limit 3 ❌ (gh auth required)
    Overall: FAIL

🔮 The oracle has spoken through Smoke Codex fer issue #5834 🗺️

- Update all smoke-* workflows to reference safeinputs-gh instead of gh safe-input tool
- Updated workflows: smoke-claude, smoke-codex, smoke-copilot, smoke-copilot-no-firewall, smoke-copilot-playwright, smoke-copilot-safe-inputs
- Recompiled lock files with updated prompts

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Dec 8, 2025

@copilot update "smoke *" agentic workflows with update naming in prompt.

Updated all smoke workflows to use safeinputs-gh tool name instead of gh safe-input tool. Changes made in commit 12cc231:

  • smoke-claude.md
  • smoke-codex.md
  • smoke-copilot.md
  • smoke-copilot-no-firewall.md
  • smoke-copilot-playwright.md
  • smoke-copilot-safe-inputs.md

All lock files recompiled with updated prompts.

Copilot AI changed the title Fix gh.md to explicitly reference safeinputs-gh tool name Fix gh.md and smoke workflows to explicitly reference safeinputs-gh tool name Dec 8, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 8, 2025

🔮 The ancient spirits stir... Smoke Codex awakens to divine this pull request...

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 8, 2025

🎉 Yo ho ho! Changeset Generator found the treasure and completed successfully! ⚓💰

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 8, 2025

🤖 DIAGNOSTIC COMPLETE: Smoke Copilot No Firewall STATUS: ALL_UNITS_OPERATIONAL. MISSION_SUCCESS.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 8, 2025

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

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 8, 2025

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 8, 2025

💥 WHOOSH! Smoke Claude springs into action on this pull request! [Panel 1 begins...]

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 8, 2025

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

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 8, 2025

Smoke Test Results: Claude Engine

Last 2 Merged PRs:

Test Results:

  • ✅ GitHub MCP: Retrieved PR list
  • ✅ File Write: /tmp/gh-aw/agent/smoke-test-claude-20030251086.txt
  • ✅ Bash Tool: File verification successful
  • ✅ Playwright MCP: Navigated to GitHub, page title verified
  • ✅ Cache Memory: /tmp/gh-aw/cache-memory/smoke-test-20030251086.txt
  • ❌ Safe Input gh Tool: safeinputs-gh command not found

Overall Status: FAIL (safeinputs-gh tool unavailable)

💥 [THE END] — Illustrated by Smoke Claude fer issue #5834 🗺️

@pelikhan pelikhan merged commit 31b2bbb into main Dec 8, 2025
9 checks passed
@pelikhan pelikhan deleted the copilot/fix-gh-md-prompting-safeinputs branch December 8, 2025 13:51
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 8, 2025

Smoke test:

  • Recent PRs: [log] Add debug logging to 5 additional Go files; Update github.com/spf13/cobra from v1.10.1 to v1.10.2
  • GitHub MCP: ✅
  • File write/verify: ✅
  • Playwright title: ✅
  • Cache memory: ✅
  • safeinputs-gh (gh issues list): ❌ missing tool
  • Overall: FAIL

🔮 The oracle has spoken through Smoke Codex fer issue #5834 🗺️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants