From 9ab29057773ad09bf165f5b61530eb6355728894 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Thu, 12 Feb 2026 16:33:51 +0000 Subject: [PATCH 1/2] Initial plan From 92f28fbe6293f165caf28fa2560137290c3ff5a1 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Thu, 12 Feb 2026 16:48:41 +0000 Subject: [PATCH 2/2] Fix slide-deck-maintainer: add explicit safe output tool requirements Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> --- .github/workflows/slide-deck-maintainer.md | 56 ++++++++++++++++++---- 1 file changed, 46 insertions(+), 10 deletions(-) diff --git a/.github/workflows/slide-deck-maintainer.md b/.github/workflows/slide-deck-maintainer.md index 80796561269..0248b60992b 100644 --- a/.github/workflows/slide-deck-maintainer.md +++ b/.github/workflows/slide-deck-maintainer.md @@ -226,9 +226,42 @@ kill $(cat /tmp/server.pid) 2>/dev/null || true rm -f /tmp/server.pid /tmp/slides-preview.html /tmp/slides-preview-updated.html /tmp/server.log ``` -## Step 9: Create Pull Request (if changes made) +## Step 9: Report Your Actions (REQUIRED) -If you made changes to `docs/slides/index.md`, create a pull request with: +**CRITICAL**: You MUST call one of the safe output tools before completing: + +### If NO changes were made: + +Call the `noop` tool to report completion: + +```json +{ + "message": "Slide deck maintenance complete - no changes needed", + "details": { + "slides_reviewed": 49, + "layout_issues_found": 0, + "content_errors_found": 0, + "sources_checked": ["code", "docs", "workflows"], + "focus_mode": "${{ inputs.focus }}", + "next_recommended_focus": "feature-deep-dive or area to review next" + } +} +``` + +**Why this matters**: The `noop` tool records that you completed your work successfully +even though no code changes were made. Without this, the workflow will be marked as failed. + +### If changes WERE made: + +Proceed to Step 10 to create a pull request. + +**Important Note**: Safe output tools (`noop`, `create_pull_request`, etc.) are MCP tools +available through your standard tool calling interface. Call them directly - do NOT try to +invoke them via bash commands, npm scripts, or curl requests. + +## Step 10: Create Pull Request (if changes made) + +If you made changes to `docs/slides/index.md`, call the `create_pull_request` tool with: **Title**: `[slides] Update slide deck - [brief description]` @@ -257,12 +290,15 @@ ${{ inputs.focus }} **Labels**: `documentation`, `automated`, `slides` -## Summary +## Completion Checklist + +Before finishing, ensure you have: + +- [ ] Built and tested slides (or documented why not possible) +- [ ] Scanned repository content for accuracy +- [ ] Detected and documented any layout issues +- [ ] Made changes if needed +- [ ] **Called `noop` OR `create_pull_request`** ← REQUIRED -After completing your work, provide: -- Number of slides reviewed -- Number of layout issues detected and fixed -- Key content updates made -- Source areas scanned (code/workflows/docs) -- PR link (if created) -- Next recommended focus area for the next run +**Remember**: Safe output tools are MCP tools - call them through your tool interface, +not via bash/shell commands.