Skip to content

Fix Daily File Diet issue output - #11380

Open
Amaury Levé (Evangelink) wants to merge 1 commit into
mainfrom
dev/amauryleve/fix-daily-file-diet-reporting
Open

Amaury Levé (Evangelink) wants to merge 1 commit into
mainfrom
dev/amauryleve/fix-daily-file-diet-reporting

Conversation

@Evangelink

Copy link
Copy Markdown
Member

Summary

  • route the final create_issue write through the safeoutputs CLI supported by the current Copilot harness
  • require the final title and body to be prepared before the workflow's single issue-write attempt
  • forbid placeholder probes and fallback JSON/scripting paths that can consume the one-per-run quota
  • regenerate the workflow lockfile with the repository-pinned gh-aw v0.88.7 compiler

Validation

  • gh-aw.exe compile daily-file-diet --strict (v0.88.7)
  • python .github/scripts/check_action_pins.py

Fixes #11366

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot AI balanced review requested due to automatic review settings September 18, 2026 10:41
@github-actions github-actions Bot added the state/needs-review Awaiting review from the team. label Sep 18, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟡 Changes recommended

The configured body path has a missing parent directory, causing the file-write step to fail.

Get a fresh assessment by requesting another Copilot review.

Review effort: Balanced
Findings: 1 Medium severity

Open (1)
What changed in this PR

Updates Daily File Diet to submit its finalized issue through the safe-output CLI.

Changes:

  • Requires final title/body preparation before one issue-write attempt.
  • Regenerates the strict gh-aw lockfile.
File Description
.github/​workflows/​daily-file-diet.md Adds safe-output CLI instructions.
.github/​workflows/​daily-file-diet.lock.yml Updates the generated workflow hash.

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment on lines +203 to +208
1. Use the file-editing tool to write the final body to
`/tmp/gh-aw/agent/issue-body.md`.
2. Invoke the safe-output CLI exactly once:

```bash
safeoutputs create_issue --title "<FINAL_TITLE>" --body "$(cat /tmp/gh-aw/agent/issue-body.md)"
@microsoft-github-policy-service microsoft-github-policy-service Bot added state/needs-review Awaiting review from the team. and removed state/needs-review Awaiting review from the team. labels Sep 18, 2026
@github-actions github-actions Bot added state/needs-review Awaiting review from the team. and removed state/needs-review Awaiting review from the team. labels Sep 18, 2026
@microsoft-github-policy-service microsoft-github-policy-service Bot added state/needs-review Awaiting review from the team. and removed state/needs-review Awaiting review from the team. labels Sep 18, 2026
@github-actions github-actions Bot added state/approved Proposal approved; ready for implementation. and removed state/needs-review Awaiting review from the team. labels Sep 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

state/approved Proposal approved; ready for implementation.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[aw] Daily File Diet reported incomplete result

3 participants