Skip to content

Add smoke-copilot-safe-inputs workflow with gh CLI safe-input tool, Serena, and emoji-only messages#5758

Merged
pelikhan merged 6 commits intomainfrom
copilot/add-smoke-copilot-safe-inputs
Dec 7, 2025
Merged

Add smoke-copilot-safe-inputs workflow with gh CLI safe-input tool, Serena, and emoji-only messages#5758
pelikhan merged 6 commits intomainfrom
copilot/add-smoke-copilot-safe-inputs

Conversation

Copy link
Contributor

Copilot AI commented Dec 7, 2025

Creates a variant of smoke-copilot.md that uses the gh CLI safe-input tool instead of the GitHub MCP server, with message personality set to 100% emojis.

Changes

  • New workflow: .github/workflows/smoke-copilot-safe-inputs.md
    • Disables GitHub MCP tool via github: false (otherwise auto-enabled by default)
    • Imports shared/gh.md to provide gh CLI as safe-input tool
    • Adds Serena MCP server for Go code analysis
    • Messages use only emojis with run URL tracking: 📰🚀🔍👀📡🕵️ [{run_url}]({run_url}) (run-started), 📰✅🎉🏁✨🎤 [{run_url}]({run_url}) (success), 📰⚠️🔥❌🚨🔧 [{run_url}]({run_url}) (failure)

Test Requirements

The workflow validates:

  1. GitHub API access via gh CLI safe-input tool
  2. File writing and bash command execution
  3. Serena's ability to list classes in the project

Technical Details

The github: false directive is required because applyDefaultTools() automatically adds the GitHub MCP server unless explicitly disabled. The safe-input gh tool provides equivalent GitHub API access via CLI instead of MCP.

imports:
  - shared/gh.md
tools:
  github: false  # Disable auto-added GitHub MCP server
  serena: ["go"]  # Enable Serena for Go code analysis
safe-outputs:
  messages:
    footer: "📰🔥📋 [{run_url}]({run_url})"
    run-started: "📰🚀🔍👀📡🕵️ [{run_url}]({run_url})"
Original prompt

Add an agentic workflow "smoke copilot safe inputs" that is a copy of smoke copilot, but disables GitHub tool and instead imports shared/gh.md.

Messages personality should 100% emojis


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


Changeset

  • Type: patch
  • Description: Adds new workflow .github/workflows/smoke-copilot-safe-inputs.md which uses the gh CLI safe-input tool instead of the GitHub MCP server and sets messages to emoji-only.

AI generated by Changeset Generator



Smoke Test Summary

Timestamp: 2025-12-07 13:57 UTC | Status: PASS
All tests passed: GitHub MCP, File Writing, Bash Tool, Playwright MCP

AI generated by Smoke Copilot No Firewall

Copilot AI and others added 2 commits December 7, 2025 13:35
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Add agentic workflow for smoke copilot safe inputs Add smoke-copilot-safe-inputs workflow with gh CLI safe-input tool and emoji-only messages Dec 7, 2025
Copilot AI requested a review from pelikhan December 7, 2025 13:54
@pelikhan pelikhan added the smoke label Dec 7, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Dec 7, 2025

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

@github-actions
Copy link
Contributor

github-actions bot commented Dec 7, 2025

📰 BREAKING: Smoke Copilot Playwright is now investigating this pull request. Sources say the story is developing...

@github-actions
Copy link
Contributor

github-actions bot commented Dec 7, 2025

🤖 SYSTEM_INIT: Smoke Copilot No Firewall ACTIVATED. PROCESSING pull request. ALL SUBSYSTEMS ONLINE.

@github-actions
Copy link
Contributor

github-actions bot commented Dec 7, 2025

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

@github-actions
Copy link
Contributor

github-actions bot commented Dec 7, 2025

📰🚀🔍👀📡🕵️

@github-actions
Copy link
Contributor

github-actions bot commented Dec 7, 2025

📰 DEVELOPING STORY: Smoke Copilot reports failed. Our correspondents are investigating the incident...

@github-actions
Copy link
Contributor

github-actions bot commented Dec 7, 2025

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

@github-actions
Copy link
Contributor

github-actions bot commented Dec 7, 2025

Smoke Test Results - Claude Engine

Recent PRs

Test Results

✅ GitHub MCP: Fetched 2 recent merged PRs
✅ File Writing: Created test file successfully
✅ Bash Tool: Verified file contents
✅ Playwright MCP: Navigated to GitHub, title verified

Status: PASS

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

@github-actions
Copy link
Contributor

github-actions bot commented Dec 7, 2025

Smoke Test Results - Copilot Engine (No Firewall)

Timestamp: 2025-12-07 13:57 UTC

✅ GitHub MCP: PR #5743 "Fix linting errors and test failures", PR #5745 "Document repo-memory feature"
✅ File Writing: /tmp/gh-aw/agent/smoke-test-copilot-20005264080.txt created
✅ Bash Tool: File verified with cat
✅ Playwright MCP: GitHub.com loaded, title contains "GitHub"

Overall Status: PASS

🤖 DIAGNOSTIC REPORT GENERATED BY Smoke Copilot No Firewall fer issue #5758 🗺️

@github-actions
Copy link
Contributor

github-actions bot commented Dec 7, 2025

PRs reviewed: Fix linting errors and test failures; Document repo-memory feature
Tests: GitHub MCP review ✅; File write/read ✅; Playwright title ✅
Overall: PASS

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

@github-actions
Copy link
Contributor

github-actions bot commented Dec 7, 2025

Smoke Test Results

Playwright MCP: Successfully navigated to https://github.com and verified page title contains "GitHub"

Status: PASS

📰 BREAKING: Report filed by Smoke Copilot Playwright fer issue #5758 🗺️

@pelikhan
Copy link
Contributor

pelikhan commented Dec 7, 2025

@copilot the message should have the run url

@github-actions
Copy link
Contributor

github-actions bot commented Dec 7, 2025

Smoke Test Results

Last 2 Merged PRs:

Test Results:
✅ GitHub MCP Testing - Retrieved merged PRs successfully
✅ File Writing Testing - Created test file (62 bytes)
✅ Bash Tool Testing - Verified file creation
✅ GitHub MCP Default Toolset - Confirmed get_me not in default toolsets (per documentation)

Overall Status: PASS

📰🔥📋 fer issue #5758 🗺️

Comment on lines 52 to 54
2. **File Writing Testing**: Create a test file `/tmp/gh-aw/agent/smoke-test-copilot-${{ github.run_id }}.txt` with content "Smoke test passed for Copilot at $(date)" (create the directory if it doesn't exist)
3. **Bash Tool Testing**: Execute bash commands to verify file creation was successful (use `cat` to read the file back)
4. **GitHub MCP Default Toolset Testing**: Verify that the `get_me` tool is NOT available with default toolsets. Try to use it and confirm it fails with a tool not found error.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot remove this prompt

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed the GitHub MCP Default Toolset Testing requirement (lines 52-54) and replaced with Serena MCP Testing. (684862b)

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

Copilot AI commented Dec 7, 2025

@copilot the message should have the run url

Added run_url to all message fields (footer, run-started, run-success, run-failure) while keeping the emoji personality. The messages now include [{run_url}]({run_url}) for tracking. (c6a9c7a)

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title Add smoke-copilot-safe-inputs workflow with gh CLI safe-input tool and emoji-only messages Add smoke-copilot-safe-inputs workflow with gh CLI safe-input tool, Serena, and emoji-only messages Dec 7, 2025
@pelikhan pelikhan marked this pull request as ready for review December 7, 2025 14:12
@pelikhan pelikhan merged commit a2002c2 into main Dec 7, 2025
5 checks passed
@pelikhan pelikhan deleted the copilot/add-smoke-copilot-safe-inputs branch December 7, 2025 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants