-
Notifications
You must be signed in to change notification settings - Fork 42
Externalize threat detection prompt template to runtime file #12930
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…o read at runtime Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
|
|
|
|
📰 BREAKING: Smoke Copilot is now investigating this pull request. Sources say the story is developing... |
|
|
|
|
|
@copilot fix resolution path See https://github.com/githubnext/gh-aw/actions/runs/21546038294/job/62087632348#step:7:1 |
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟 |
|
🎬 THE END — Smoke Claude MISSION: ACCOMPLISHED! The hero saves the day! ✨ |
|
📰 BREAKING: Smoke Copilot is now investigating this pull request. Sources say the story is developing... |
|
🎉 Yo ho ho! Changeset Generator found the treasure and completed successfully! ⚓💰 |
Agent Container Tool Check
Result: 10/12 tools available Missing tools:
Status: PARTIAL - Core tools (bash, git, jq, yq, curl, gh, node, python, go) are all available. Java and .NET runtimes are missing but may not be required for typical workflows.
|
Smoke Test: PASS ✅
@pelikhan
|
|
GitHub MCP merged PRs ✅ 🔧 Improve Agentic Workflow Authoring Structure / Remove noop from handler registry and clarify skip reason logging
|
|
📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤 |
Smoke Test: Claude ✅Last 2 Merged PRs:
Test Results:
Overall Status: PASS ✅
|
Move threat detection prompt to actions/setup and update JavaScript to read at runtime
pkg/workflow/prompts/threat_detection.mdtoactions/setup/md/threat_detection.mdsetup_threat_detection.cjsto read the prompt file from runtime prompts directory at/opt/gh-aw/prompts/threat_detection.mdthreat_detection.goand update to pass file path instead of contentSummary
Successfully moved the threat detection prompt text from an embedded Go string to a file in
actions/setup/md/threat_detection.md. The JavaScript code now reads this file at runtime from/opt/gh-aw/prompts/threat_detection.mdwhere the setup action copies it. All tests pass and linting is clean.Path Resolution Fix
The initial implementation used a relative path
../md/threat_detection.md, but at runtime the JavaScript files are executed from/opt/gh-aw/actions/, not from the source directory. The setup action copies markdown files fromactions/setup/md/to/opt/gh-aw/prompts/, so the correct runtime path is/opt/gh-aw/prompts/threat_detection.md.Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.