Skip to content

ci(alert): Pullfrog, an on-request agent, needs no alert - #7428

Merged
kixelated merged 1 commit into
mainfrom
ci/alert-pullfrog
Oct 31, 2026
Merged

kixelated merged 1 commit into
mainfrom
ci/alert-pullfrog

Conversation

@kixelated

Copy link
Copy Markdown
Collaborator

pullfrog.yml landed in 7fc637e without an alert.yml entry, so check gh (sh/gh/alert.sh check-coverage) fails on main. The host land gate's first moq batch failed on it, along with a real failure from #7029.

Pullfrog is a workflow_dispatch agent workflow, not a cron or publish job, so it joins CI_ONLY rather than alert.yml.

Landed with the admin bypass, at the maintainer's direction, as an emergency fix for a break on main that blocks the gate. #7427 carries the same change and drops Land from the list.

Checks

sh/gh/alert.sh check-coverage (33 alertable workflows) and shellcheck pass.

Review

Skipped: a one-entry allowlist change that the coverage check verifies directly.

🤖 Generated with Claude Code

(written by Claude Opus 5.5)

pullfrog.yml landed without an alert.yml entry, so check gh fails on main and
in every land batch that selects it.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@kixelated
kixelated merged commit 9eea0dd into main Oct 31, 2026
8 checks passed
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 31, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-31T02:42:16.972792Z ffc63f9 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@pullfrog pullfrog Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

✅ No new issues found.

Reviewed changes

  • Add Pullfrog to CI_ONLY — classifies the manual workflow_dispatch agent workflow as CI-only so check-coverage no longer expects an alert.yml entry. The comment is updated to match.

sh/gh/alert.sh check-coverage passes here (alert.yml watches all 33 alertable workflows), Pullfrog matches .github/workflows/pullfrog.yml:2 exactly, and the workflow is workflow_dispatch-only, so the CI_ONLY skip at sh/gh/alert.sh:143 is the correct path.

Pullfrog  | View workflow run | Using deepseek-v4.1-flash (free via Pullfrog for OSS) | 𝕏

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.

1 participant