Skip to content

Commit 7e28775

Browse files
authored
Clarify direct safe-output usage in Daily Compiler Quality Check (#40575)
1 parent 9f631ce commit 7e28775

3 files changed

Lines changed: 22 additions & 6 deletions

File tree

.github/workflows/daily-compiler-quality.lock.yml

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

.github/workflows/daily-compiler-quality.md

Lines changed: 20 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -366,10 +366,26 @@ Generate a comprehensive discussion report with findings.
366366

367367
### Output Contract (Required)
368368

369-
1. Emit **exactly one** `create_discussion` safe-output item.
370-
2. Do **not** emit placeholder or draft bodies (for example: `test`, `.`, `todo`, or similar short placeholders).
371-
3. Only emit `create_discussion` after the final report body is complete and fully rendered.
372-
4. The workflow enforces a **minimum 200-character body length**, so very short outputs (placeholder or otherwise) will fail safe-outputs.
369+
1. Call safe-output tools **directly**. Use the `create_discussion` or `noop` tool itself — **never** use `bash`, `shell`, or `safeoutputs ...` CLI commands to invoke safe outputs.
370+
2. If the final report body is complete and valid, emit **exactly one** direct `create_discussion` tool call.
371+
3. Do **not** emit placeholder or draft bodies (for example: `test`, `.`, `todo`, or similar short placeholders).
372+
4. Only call `create_discussion` after the final report body is complete and fully rendered.
373+
5. If you cannot produce a valid discussion body, or decide no discussion should be created, call `noop` directly with a brief explanation instead of returning plain text.
374+
6. The workflow enforces a **minimum 200-character body length**, so very short outputs (placeholder or otherwise) will fail safe-outputs.
375+
376+
### Direct Tool Call Examples
377+
378+
Use the `create_discussion` tool directly:
379+
380+
```json
381+
{"title":"Daily Compiler Code Quality Report - YYYY-MM-DD","category":"audits","body":"...full markdown report..."}
382+
```
383+
384+
If you do not create a discussion, use `noop` directly:
385+
386+
```json
387+
{"message":"No discussion created: [brief explanation]"}
388+
```
373389

374390
### Discussion Title
375391

.github/workflows/smoke-copilot.lock.yml

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)