-
Notifications
You must be signed in to change notification settings - Fork 34
Refactor threat detection parsing step to separate .cjs file #8691
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
- Created parse_threat_detection_results.cjs with proper JSDoc - Updated buildParsingStep to use require pattern - Added buildResultsParsingScriptRequire function - Deprecated buildResultsParsingScript inline implementation - Updated detection_success_test.go to check for require pattern - Recompiled all workflow lock files - All tests passing Co-authored-by: pelikhan <[email protected]>
- Removed buildResultsParsingScript() (replaced by buildResultsParsingScriptRequire) - Removed buildSetupScript() (replaced by buildSetupScriptRequire) - Updated TestSetupScriptReferencesPromptFile to test new require pattern - All tests passing, linter clean Co-authored-by: pelikhan <[email protected]>
|
🤖 DIAGNOSTIC COMPLETE: Smoke Copilot No Firewall STATUS: ALL_UNITS_OPERATIONAL. MISSION_SUCCESS. |
|
📰 VERDICT: Smoke Copilot Playwright has concluded. All systems operational. This is a developing story. 🎤 |
|
🎉 Yo ho ho! Smoke Copilot Safe Inputs found the treasure and completed successfully! ⚓💰 |
|
📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤 |
|
🎬 THE END — Smoke Claude MISSION: ACCOMPLISHED! The hero saves the day! ✨ |
|
✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟 |
|
🎉 Yo ho ho! Changeset Generator found the treasure and completed successfully! ⚓💰 |
|
✅ Firewall validation complete... Smoke Codex Firewall confirmed network sandboxing is operational. 🛡️ |
|
Copilot Engine Smoke Test ResultsLast 2 Merged PRs:
Test Results:
Overall Status: PASS @pelikhan
|
Smoke Test Results
Overall Status: FAIL (2/3 tests failed - required tools missing)
|
Smoke Test Results: Copilot Engine (No Firewall)Test Status: Results:
Issue Identified: The GitHub MCP server is not properly authenticated. All GitHub API calls return 401 Unauthorized errors. Note: The Playwright tool and file system operations worked correctly, but GitHub API access requires authentication configuration.
|
Smoke Test Results (Claude)Recent PRs:
Test Results:
Overall Status: PASS (5/6 tests passed)
|
|
PRs: Add --dir flag to fix command; Reduce setup.sh log verbosity by moving file copy messages to debug level
|
The "Parse threat detection results" step was inline JavaScript embedded in Go code. This refactors it to follow the established pattern used by
setup_threat_detection.cjsand other modules.Changes
Created
actions/setup/js/parse_threat_detection_results.cjsbuildResultsParsingScript()Go functionerror_helpers.cjsfor consistent error handlingUpdated
pkg/workflow/threat_detection.gobuildResultsParsingScriptRequire()to generate require-based scriptbuildParsingStep()to use new functionUpdated tests
Example
Before (inline JavaScript in Go):
After (require pattern):
Compiled workflow step:
Original prompt
💡 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.