fix(ci): prevent empty safeoutputs schema probes in smoke-claude workflow#5307
Conversation
✅ Coverage Check PassedOverall Coverage
📁 Per-file Coverage Changes (1 files)
Coverage comparison generated by |
There was a problem hiding this comment.
Pull request overview
This PR updates the Smoke Claude workflow prompt guidance (and its lock metadata) to prevent “schema probe” / empty-argument invocations of safeoutputs tools, and adds a focused Jest assertion to ensure the guardrail stays present in the source workflow.
Changes:
- Add an explicit prompt instruction to never call
add_comment/add_labelswith empty arguments (avoid schema-probing). - Update the smoke workflow Jest test to assert the new instruction exists in
smoke-claude.md. - Regenerate
smoke-claude.lock.ymlmetadata to reflect the updated workflow source.
Show a summary per file
| File | Description |
|---|---|
| scripts/ci/smoke-claude-workflow.test.ts | Adds a test assertion to enforce the new “no empty safeoutputs calls” guardrail in the workflow source. |
| .github/workflows/smoke-claude.md | Adds the explicit instruction to avoid empty-argument add_comment / add_labels calls. |
| .github/workflows/smoke-claude.lock.yml | Updates compiled workflow metadata (body hash) corresponding to the source prompt change. |
Copilot's findings
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 3/3 changed files
- Comments generated: 0
|
@copilot address the review feedback and fix the pr title and summary |
|
@copilot address the review feedback and fix the pr title and summary |
|
✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟 |
|
Chroot tests passed! Smoke Chroot - All security and functionality tests succeeded. |
|
🔑 Smoke Copilot PAT PAT auth validated. All systems operational. ✅ |
|
✅ Smoke Copilot BYOK completed. Copilot BYOK mode operational. 🔓 |
|
✅ Build Test Suite completed successfully! |
|
❌ Smoke Copilot BYOK AOAI (Entra) reports failed. AOAI BYOK (Entra) mode investigation needed... |
|
📡 Smoke OTel Tracing completed. All tracing scenarios validated. ✅ |
|
✅ Smoke Gemini completed. All facets verified. 💎 |
|
❌ Smoke Claude failed |
|
📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤 |
|
🚀 Security Guard has started processing this pull request |
|
❌ Smoke Copilot BYOK AOAI (api-key) reports failed. AOAI BYOK (api-key) mode investigation needed... |
|
🔌 Smoke Services — All services reachable! ✅ |
|
✅ Contribution Check completed successfully! |
🔬 Smoke Test ResultsPR: [WIP] Debug smoke Claude workflow failure
Overall: FAIL (pre-computed data missing due to unresolved
|
Smoke Test: Copilot PAT — PASS ✅
Auth mode: PAT (COPILOT_GITHUB_TOKEN)
|
Smoke Test: Copilot BYOK (Direct) Mode ✅ PASSResults:
Running in direct BYOK mode via COPILOT_PROVIDER_API_KEY. /cc
|
Smoke Test: API Proxy OpenTelemetry Tracing
All 5 scenarios pass. OTEL tracing integration is functioning correctly.
|
Chroot Runtime Version Comparison
Result: 1/3 tests passed — Go matches, but Python and Node.js versions differ between host and chroot environments.
|
Gemini Smoke Test Results
Overall status: PASS Warning Firewall blocked 1 domainThe following domain was blocked by the firewall during workflow execution:
network:
allowed:
- defaults
- "localhost"See Network Configuration for more information.
|
|
[WIP] Debug smoke Claude workflow failure by Copilot
|
🏗️ Build Test Suite Results
Overall: 8/8 ecosystems passed — ✅ PASS
|
Smoke Test Results
Overall: FAIL
|
Updated the PR metadata: title is now |
add_comment/add_labelscallssmoke-claude.lock.ymlfor the source change