Skip to content

feat: disable automatic triggers on Claude workflows#111

Merged
JacobPEvans merged 2 commits intomainfrom
feature/disable-auto-claude-triggers
Mar 8, 2026
Merged

feat: disable automatic triggers on Claude workflows#111
JacobPEvans merged 2 commits intomainfrom
feature/disable-auto-claude-triggers

Conversation

@JacobPEvans
Copy link
Owner

@JacobPEvans JacobPEvans commented Mar 8, 2026

Summary

  • Disable automatic triggers on all Claude-executing workflows
  • All triggers commented out with DISABLED AUTOMATIC TRIGGERS markers for easy restoration
  • No code deleted - fully reversible

Files changed (6)

  • dogfood-all.yml - commented out pull_request, issue_comment, pull_request_review, push triggers + post-merge-dispatch job
  • dogfood-ci.yml - commented out workflow_run trigger + concurrency group
  • dogfood-best-practices.yml - commented out schedule trigger
  • dogfood-issue-hygiene.yml - commented out schedule trigger
  • dogfood-issue-sweeper.yml - commented out schedule trigger
  • pr-issue-linker.yml - commented out pull_request trigger + job if condition

How to Restore

Search for DISABLED AUTOMATIC TRIGGERS and uncomment the blocks.

🤖 Generated with Claude Code

Greptile Summary

This PR surgically disables all automatic event-driven triggers across the 6 Claude-executing workflows, reducing each to workflow_dispatch-only operation. No code is deleted — everything is commented out with consistent DISABLED AUTOMATIC TRIGGERS / END DISABLED markers, making restoration a simple search-and-uncomment operation (a very git-brained move 🧠).

Key changes:

  • dogfood-all.ymlpull_request, issue_comment, pull_request_review, and push triggers disabled; post-merge-dispatch job commented out; stale if: github.event_name != 'push' guard on the all job correctly removed
  • dogfood-ci.ymlworkflow_run trigger replaced with a freshly-added workflow_dispatch; workflow_run-specific concurrency block commented out to avoid a dangling github.event.workflow_run.* context reference
  • dogfood-best-practices.yml, dogfood-issue-hygiene.yml, dogfood-issue-sweeper.ymlschedule cron triggers commented out cleanly; workflow_dispatch preserved
  • pr-issue-linker.ymlpull_request trigger and its job-level bot/draft guard if condition both commented out; see inline comment for a minor note about the guard

Minor note: In pr-issue-linker.yml, the if condition that filtered out bots and draft PRs is now silently absent for manual dispatch runs. The issue-linker.yml reusable workflow will receive null for all github.event.pull_request.* fields when manually triggered — worth documenting in a comment for future clarity.

Confidence Score: 5/5

  • Safe to merge — purely additive comments, zero deleted code, and all workflows retain at least one active trigger.
  • All changes are comment-outs of existing configuration with zero functional logic altered in the active code paths. Every workflow retains workflow_dispatch so they can still be triggered manually. The dogfood-all.yml guard removal (if: github.event_name != 'push') is correctly coupled to the push-trigger disable. The only minor point is the implicit loss of the bot/draft guard in pr-issue-linker.yml during manual dispatch, but this is a style concern rather than a runtime error.
  • pr-issue-linker.yml deserves a second glance to confirm the bot/draft guard omission is intentional for manual runs.

Important Files Changed

Filename Overview
.github/workflows/dogfood-all.yml Commented out pull_request, issue_comment, pull_request_review, and push triggers + the post-merge-dispatch job. Also correctly removed the now-stale if: github.event_name != 'push' guard from the all job.
.github/workflows/dogfood-ci.yml workflow_run trigger replaced with workflow_dispatch (added fresh since it didn't exist before), and the workflow_run-specific concurrency block correctly commented out to avoid a dangling context reference.
.github/workflows/pr-issue-linker.yml pull_request trigger and the job-level if-condition (which guards against bots/drafts and checks event action) both commented out; job now runs unconditionally on workflow_dispatch, which may surprise callers if they expect the bot/draft guard to still be active.
.github/workflows/dogfood-best-practices.yml schedule cron trigger commented out cleanly; workflow_dispatch preserved for manual runs.
.github/workflows/dogfood-issue-hygiene.yml schedule cron trigger commented out cleanly; workflow_dispatch preserved for manual runs.
.github/workflows/dogfood-issue-sweeper.yml schedule cron trigger commented out cleanly; workflow_dispatch preserved for manual runs.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    subgraph BEFORE["Before — Automatic Triggers Active"]
        A1[pull_request opened/sync] --> W1[dogfood-all.yml]
        A2[issue_comment created] --> W1
        A3[pull_request_review submitted] --> W1
        A4[push to main] --> J1[post-merge-dispatch job] --> W1
        A5[workflow_run: Tests completed] --> W2[dogfood-ci.yml]
        A6[pull_request opened/closed] --> W3[pr-issue-linker.yml]
        A7[schedule: 0 3 * * 2,5] --> W4[dogfood-best-practices.yml]
        A8[schedule: 0 7 * * 0] --> W5[dogfood-issue-hygiene.yml]
        A9[schedule: 0 6 * * 0] --> W6[dogfood-issue-sweeper.yml]
    end

    subgraph AFTER["After — Manual Dispatch Only"]
        B1[workflow_dispatch] --> W1B[dogfood-all.yml]
        B2[workflow_dispatch] --> W2B[dogfood-ci.yml]
        B3[workflow_dispatch] --> W3B[pr-issue-linker.yml]
        B4[workflow_dispatch] --> W4B[dogfood-best-practices.yml]
        B5[workflow_dispatch] --> W5B[dogfood-issue-hygiene.yml]
        B6[workflow_dispatch] --> W6B[dogfood-issue-sweeper.yml]
    end

    style BEFORE fill:#ffe5e5,stroke:#cc0000
    style AFTER fill:#e5ffe5,stroke:#009900
Loading

Last reviewed commit: 3c967c5

Greptile also left 1 inline comment on this PR.

Convert all workflows that execute Claude from automatic triggers
to workflow_dispatch only to control usage costs. All triggers
commented out with DISABLED AUTOMATIC TRIGGERS markers for easy
restoration.

(claude)
Copilot AI review requested due to automatic review settings March 8, 2026 12:20
@gemini-code-assist
Copy link

Note

Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported.

Copy link

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

Disables automatic triggers for internal “dogfood” / wrapper workflows that execute Claude, leaving only workflow_dispatch entrypoints and preserving the original trigger blocks as commented sections for restoration.

Changes:

  • Commented out automatic event triggers (pull_request, push, workflow_run, schedule) across the affected workflows with DISABLED AUTOMATIC TRIGGERS markers.
  • Commented out push-based re-dispatch logic in dogfood-all.yml.
  • Commented out the PR-event gating if: in pr-issue-linker.yml.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
.github/workflows/dogfood-all.yml Disables PR/comment/review/push triggers and the push re-dispatch job; leaves manual dispatch.
.github/workflows/dogfood-ci.yml Disables workflow_run trigger (Tests completion) and comments out concurrency; leaves manual dispatch.
.github/workflows/dogfood-best-practices.yml Disables scheduled trigger; leaves manual dispatch.
.github/workflows/dogfood-issue-hygiene.yml Disables scheduled trigger; leaves manual dispatch.
.github/workflows/dogfood-issue-sweeper.yml Disables scheduled trigger; leaves manual dispatch.
.github/workflows/pr-issue-linker.yml Disables pull_request trigger and comments out PR-event gating if:; leaves manual dispatch.

@JacobPEvans JacobPEvans merged commit 912eede into main Mar 8, 2026
3 checks passed
@JacobPEvans JacobPEvans deleted the feature/disable-auto-claude-triggers branch March 8, 2026 15:39
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.

2 participants