Skip to content

Update recompile workflow no-change gate to include untracked generated files#1285

Open
Copilot wants to merge 5 commits into
mainfrom
copilot/update-recompile-agentic-workflows
Open

Update recompile workflow no-change gate to include untracked generated files#1285
Copilot wants to merge 5 commits into
mainfrom
copilot/update-recompile-agentic-workflows

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 10, 2026

This updates the agentic workflow recompilation gate to match the behavior introduced in plengauer/plengauer.github.io#139: no-change detection now considers untracked/new files, not just tracked diffs.
Scope is limited to the existing recompile_agentic_workflows.yml workflow in this repository.

  • Workflow gate logic

    • Replaced the early-exit condition in .github/workflows/recompile_agentic_workflows.yml from tracked-diff-only detection to porcelain status detection.
  • Behavior impact

    • Prevents false “no changes” exits when gh aw compile produces newly added workflow files.
    • Leaves behavior unchanged when the working tree is truly clean.
# before
if git diff --quiet HEAD; then exit 0; fi

# after
if [ -z "$(git status --porcelain)" ]; then exit 0; fi
Original prompt

Inspect and update all listed repositories owned by plengauer that contain .github/workflows/recompile_agentic_workflows.yml and still use the old early-exit condition if git diff --quiet HEAD; then exit 0; fi in that workflow.

Reference change: the behavior merged in plengauer/plengauer.github.io#139 changed the no-change gate to treat newly added/untracked generated files as changes.

Required workflow change:

# before
if git diff --quiet HEAD; then exit 0; fi

# after
if [ -z "$(git status --porcelain)" ]; then exit 0; fi

Apply this exact logic only in repositories where the old condition is still present. Do not change repositories that already use the new porcelain-based check. Do not modify repositories that do not contain the workflow.

Repositories to inspect from the provided list:

  • plengauer/Thoth
  • plengauer/plengauer.github.io
  • plengauer/Philbot
  • plengauer/autopuppeteer
  • plengauer/autoversion
  • plengauer/create-github-finegrained-pat
  • plengauer/openai
  • plengauer/bash-ai
  • plengauer/dafuq
  • plengauer/create-package-repository
  • plengauer/Utilities
  • plengauer/opentelemetry-bash-rc
  • plengauer/BoomerCode
  • plengauer/GitAptly
  • plengauer/autoapprove
  • plengauer/abom-http
  • plengauer/Dionysus
  • plengauer/autorerun
  • plengauer/yourtube
  • plengauer/cookie2ghsecret
  • plengauer/debugging_2025_08_15
  • plengauer/api-mcp
  • plengauer/RicherPresence
  • plengauer/DXGIOutputDuplication
  • plengauer/Thoth-test
  • plengauer/lgtm

From prior inspection in this conversation:

  • plengauer/plengauer.github.io already has the fix via PR Update plengauer/opentelemetry-bash action to v4.10.1 #139 and should be left unchanged.
  • The following repositories contain the workflow and appear to still use the old git diff --quiet HEAD condition, so they likely need the update:
    • plengauer/Philbot
    • plengauer/autopuppeteer
    • plengauer/autoversion
    • plengauer/create-github-finegrained-pat
    • plengauer/openai
    • plengauer/bash-ai
    • plengauer/dafuq
    • plengauer/create-package-repository
    • plengauer/Utilities
    • plengauer/opentelemetry-bash-rc
    • plengauer/BoomerCode
    • plengauer/GitAptly
    • plengauer/autoapprove
    • plengauer/Dionysus
    • plengauer/autorerun
    • plengauer/cookie2ghsecret
    • plengauer/debugging_2025_08_15
    • plengauer/api-mcp
    • plengauer/RicherPresence
    • plengauer/DXGIOutputDuplication
  • plengauer/Thoth already uses the new porcelain check and should be left unchanged.
  • plengauer/abom-http, plengauer/yourtube, and plengauer/Thoth-test do not appear to contain this workflow and should be left unchanged.
  • plengauer/lgtm could not be inspected from prior tool results; if accessible, inspect it too and apply the same rule.

Open pull requests for every repository that needs this change. Use concise PR titles indicating that the recompile workflow should include untracked/new files in the no-change gate.

The following is the prior conversation context from the user's chat exploration (may be truncated):

User: Check the changes here. A workflow to recompile agentic workflows exists in almost any of my repositories. Iterate through all (dont forget any) of the repositories owned by me, check if such a workflow exists, check if the changes there @plengauer/plengauer.github.io/pull/139 shall be made in the same manner here, and if so, open a pull request

Assistant: I can do this, but I’m missing one thing I need before I can safely proceed: the full list of repositories owned by plengauer.

I was able to inspect the reference PR:

  • PR: plengauer/plengauer.github.io#139
  • Change: update the recompile workflow no-change check from:
    • git diff --quiet HEAD
    • to if [ -z "$(git status --porcelain)" ]; then exit 0; fi
  • Rationale: treat newly added/untracked generated files as changes too.

What I cannot reliably do with the available tools right now is enumerate all repositories owned by plengauer; the GitHub API endpoint for listing user repos is not available in this environment. Since you explicitly said “don’t forget any,” I shouldn’t guess or proceed with a partial set.

If you send me either:

  1. the exact list of your repositories, or
  2. a link/page export containing all repos under plengauer,

then I can:

  • check every repo for a recompile_agentic_workflows workflow,
  • determine whether the same fix should be applied,
  • and open PRs where needed.

Reference PR checked:

data:
- url: "https://github.com/plengauer/plengauer.github.io/pull/139"
  repository: "plengauer/plengauer.github.io"
  state: "closed"
  draft: false
  title: "Include new/untracked files in recompile workflow no-change gate"
  number: 139
  created_at: ""
  closed_at: "2026-05-10T14:45:57Z"
  merged_at: "2026-05-10T14:45:57Z"
  labels: []
  author: "Copilot"
  comments: 0
  assignees_avatar_urls:
  - "https://avatars.githubusercontent.com/u/100447901?v=4"
  - "https://avatars.githubusercontent.com/in/1143301?v=4"

If you want, paste the repo list and I’ll start checking them one by one immed...

This pull request was created from Copilot chat.

Copilot AI changed the title [WIP] Inspect and update workflows in plengauer repositories Update recompile workflow no-change gate to include untracked generated files May 10, 2026
Copilot AI requested a review from plengauer May 10, 2026 17:35
@plengauer plengauer marked this pull request as ready for review May 10, 2026 17:43
@plengauer plengauer requested a review from moflwi as a code owner May 10, 2026 17:43
Copilot AI review requested due to automatic review settings May 10, 2026 17:43
@plengauer plengauer enabled auto-merge (squash) May 10, 2026 17:43
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the “no changes” early-exit gate in the agentic workflow recompilation GitHub Action so that newly generated untracked workflow files are treated as changes (preventing false “no changes” exits after gh aw compile).

Changes:

  • Replace tracked-diff-only change detection (git diff --quiet HEAD) with a clean-working-tree check based on git status --porcelain.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

plengauer and others added 2 commits May 10, 2026 19:46
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings May 10, 2026 17:48
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Comment thread .github/workflows/opentelemetry.yml Outdated
on:
workflow_run:
workflows: [.github/workflows/copilot-setup-steps.yml, Automatic Rerun, Automatic Version Bump, CodeQL, Copilot cloud agent, Copilot code review, Copilot coding agent, Greetings, Publish Debian Baremetal, Publish Debian Configuration, Publish Debian Containerized, Publish Debian Recommended, Publish Debian Root, Publish Image Backend, Publish Image Collector, Publish Image Discord Gateway 2 HTTP, Publish Image Discord Gateway 2 HTTP Master, Publish Image Scheduler, Publish Image Voice, Publish Package Backend, Publish Package Discord Gateway 2 HTTP, Publish Package Scheduler, Publish Package Voice, Recompile Agentic Workflows, Renovate, Test Integration, Update Package Repository]
workflows: [.github/workflows/copilot-setup-steps.yml, Automatic Rerun, Automatic Version Bump, CodeQL, Copilot cloud agent, Copilot code review, Copilot coding agent, Dependency Graph, Greetings, Publish Debian Baremetal, Publish Debian Configuration, Publish Debian Containerized, Publish Debian Recommended, Publish Debian Root, Publish Image Backend, Publish Image Collector, Publish Image Discord Gateway 2 HTTP, Publish Image Discord Gateway 2 HTTP Master, Publish Image Scheduler, Publish Image Voice, Publish Package Backend, Publish Package Discord Gateway 2 HTTP, Publish Package Scheduler, Publish Package Voice, Recompile Agentic Workflows, Renovate, Test Integration, Update Package Repository]
Comment thread .gitattributes
@@ -0,0 +1 @@
.github/workflows/*.lock.yml linguist-generated=true merge=ours No newline at end of file
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants