chore(devex): lint pull request event fanout - #72668
Merged
Merged
Conversation
rnegron
force-pushed
the
chore/ci-dispatch-fanout-lint
branch
from
July 28, 2026 14:24
8258b50 to
21ef9cb
Compare
|
😎 Merged successfully - details. |
…anout-lint # Conflicts: # .agents/skills/authoring-ci-workflows/SKILL.md # tools/hogli-commands/hogli_commands/tests/test_workflow_lint.py # tools/hogli-commands/hogli_commands/workflow_lint/checks/__init__.py
rnegron
marked this pull request as ready for review
August 5, 2026 15:28
Contributor
|
Reviews (1): Last reviewed commit: "chore(ci): exempt label triggers from th..." | Re-trigger Greptile |
![graphite-app[bot]](https://avatars.githubusercontent.com/in/158384?s=60&v=4){kind=small}
|
Note 🤖 stamphog review failed before producing a verdict Check the workflow run and re-apply the label to retry. Updated in place from an earlier stamphog review. |
![stamphog[bot]](https://avatars.githubusercontent.com/in/3000332?s=60&v=4){kind=small}
There was a problem hiding this comment.
This is a well-tested addition to the internal workflow linter (author owns the area with STRONG familiarity), and the diff shows the Graphite-flagged empty-types logic bug was correctly fixed before merge, with no other unresolved concerns.
- Author wrote 100% of the modified lines and has 38 merged PRs in these paths (familiarity STRONG).
- 👍 on the PR from greptile-apps[bot], hex-security-app[bot].
Gate mechanics and policy version
| Gate | Result | |
|---|---|---|
| prerequisites | ✓ | all clear |
| deny-list | ✓ | no deny categories matched |
| size | ✓ | 110L, 2F substantive, 234L/4F incl. docs/generated/snapshots — within ceiling |
| tier | ✓ | T1-agent / T1c-medium (234L, 4F, two-areas, chore) |
| stamphog 2.0.0b4 | .stamphog/policy.yml @ 5d75423 · reviewed head 7980c95 |
Asheesh7298
pushed a commit
to Asheesh7298/posthog
that referenced
this pull request
Aug 6, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
pull_requestwithout path filters dispatches a run on every PR event, silently growing per-event fanout toward GitHub's 500 runs / 10 s dispatch cap.Changes
WF008-pr-event-fanoutinhogli lint:workflowspull_request/pull_request_targetsubscription (one dispatch each)paths:allowlist.paths-ignorestill counts, since it usually excludes a narrow slicepaths:filter, or raise the budget explicitly in review.labeled/unlabeledare not counted. A label arrives one PR at a time, so it cannot produce the burst this guards, and all 7 remaining subscribers want the trigger.Note
The budget is a repo-wide sum, not a per-workflow rule. It caps total fanout; it does not stop one workflow from re-acquiring an action another just gave up. The merge-gate label ban stays with AGENTS.md and the inline
on:comments from stop label events re-running CI, which a fungible sum cannot express.How did you test this code?
paths:exclusion, thatpaths-ignorestill counts, and that labels are not counted.hogli lint:workflowsover 124 workflows: clean, so master and untouched PRs are unaffected.hogli ci:preflight --strict: 0 failures.Automatic notifications
Docs update
🤖 Agent context
Autonomy: Human-driven (agent-assisted)
Skills:
/writing-tests,/writing-pr-descriptions,/code-review,/authoring-ci-workflows,/writing-skills.paths:is.