Skip to content

Commit 9cfb43f

Browse files
authored
Require noop call in Multi-Device Docs Tester when no issues found (#14553)
1 parent 5cfbb53 commit 9cfb43f

1 file changed

Lines changed: 22 additions & 2 deletions

File tree

.github/workflows/daily-multi-device-docs-tester.md

Lines changed: 22 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -111,6 +111,22 @@ Organize findings by severity:
111111

112112
## Step 5: Report Results
113113

114+
### If NO Issues Found
115+
116+
**YOU MUST CALL** the `noop` tool to log completion:
117+
118+
```json
119+
{
120+
"noop": {
121+
"message": "Multi-device documentation testing complete. All {device_count} devices tested successfully with no issues found."
122+
}
123+
}
124+
```
125+
126+
**DO NOT just write this message in your output text** - you MUST actually invoke the `noop` tool. The workflow will fail if you don't call it.
127+
128+
### If Issues ARE Found
129+
114130
## 📝 Report Formatting Guidelines
115131

116132
**CRITICAL**: Follow these formatting guidelines to create well-structured, readable reports:
@@ -159,7 +175,7 @@ Create reports that:
159175
- **Create delight**: Use progressive disclosure to reduce overwhelm
160176
- **Maintain consistency**: Follow the same patterns as other reporting workflows
161177

162-
If issues are detected, create a GitHub issue titled "🔍 Multi-Device Docs Testing Report - [Date]" with:
178+
Create a GitHub issue titled "🔍 Multi-Device Docs Testing Report - [Date]" with:
163179

164180
```markdown
165181
### Test Summary
@@ -212,4 +228,8 @@ Follow the shared **Documentation Server Lifecycle Management** instructions for
212228

213229
## Summary
214230

215-
Provide: total devices tested, test results (passed/failed/warnings), key findings, and link to issue (if created).
231+
**Always provide a safe output:**
232+
- **If issues found**: Create GitHub issue with test results, findings, and recommendations
233+
- **If no issues found**: Call `noop` tool with completion message including total devices tested and pass status
234+
235+
The workflow requires explicit safe output (either issue creation or noop) to confirm completion.

0 commit comments

Comments
 (0)