-
Notifications
You must be signed in to change notification settings - Fork 36
Add core.info logging to MCP gateway log parser #9645
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
- Add generatePlainTextGatewaySummary to convert gateway.md to plain text - Add generatePlainTextLegacySummary for legacy log files - Update main() to call core.info with plain text before writing step summary - Add comprehensive tests for new functions - All existing tests continue to pass Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
🎬 THE END — Smoke Claude MISSION: ACCOMPLISHED! The hero saves the day! ✨ |
|
🎉 Yo ho ho! Changeset Generator found the treasure and completed successfully! ⚓💰 |
|
📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤 |
|
✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟 |
Smoke Test ResultsLast 2 merged PRs:
Test Results:
Overall Status: PARTIAL PASS (4/6 tests passed)
|
The MCP gateway log parser only wrote to step summary, while safe inputs log parser correctly logged to both step summary and
core.info. This made MCP gateway logs invisible in console output.Changes
Added plain text generators:
generatePlainTextGatewaySummary()- strips markdown/HTML from gateway.mdgeneratePlainTextLegacySummary()- formats legacy gateway.log/stderr.logUpdated main(): calls
core.info()with plain text before writing step summaryExample
Before:
After:
Console output now includes full log content (tables converted to plain text, code blocks preserved) instead of just status messages.
Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.