feat: Phase 4 — reusable "subscribe" workflow (+ dogfood it) (#8)
Build & push image / build-and-push (push) Successful in 9s
Build & push image / build-and-push (push) Successful in 9s
Centralizes the consumer stub into a reusable Gitea workflow (.gitea/workflows/review-reusable.yml, workflow_call + defaulted inputs + secrets: inherit); gadfly's own dogfood is now a thin caller of it, which proved end-to-end that github.event context propagates into the reusable on this act_runner. Adds the slim examples/reusable.yml stub + docs. Folded in the swarm's findings: timeout_minutes default 30->45, map GADFLY_API_KEY, explicit permissions block, drop the dead specialist_suite input, and harden the example's actor gate. ~70 findings graded. Completes the gadfly-games build (Phases 1-4 + quality fixes). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Co-authored-by: Steve Dudenhoeffer <steve@stevedudenhoeffer.com> Co-committed-by: Steve Dudenhoeffer <steve@stevedudenhoeffer.com>
This commit was merged in pull request #8.
This commit is contained in:
@@ -1,11 +1,10 @@
|
||||
# Gadfly — agentic adversarial PR reviewer (https://gitea.stevedudenhoeffer.com/steve/gadfly).
|
||||
# Gadfly reviewing its OWN PRs — now a thin CALLER of the reusable workflow
|
||||
# (.gitea/workflows/review-reusable.yml), dogfooding the Phase-4 "subscribe"
|
||||
# path. The reusable holds the image pin + env plumbing; this file holds only
|
||||
# the triggers, the actor gate, and gadfly's specific swarm config.
|
||||
#
|
||||
# Runs the published Gadfly image (pinned to an immutable :sha- tag — act_runner
|
||||
# caches :latest, and this build is what carries foreman provider-type support)
|
||||
# as a specialist swarm and posts
|
||||
# ONE consolidated review comment as gitea-actions. Advisory only — never blocks a
|
||||
# merge. Gadfly reviewing its OWN PRs — dogfooding: 6 cloud models + the Claude
|
||||
# Code engine (sonnet) as a competitor. Local Macs and weak cloud models dropped.
|
||||
# Advisory only — never blocks a merge. Fleet: 6 cloud + Claude Code
|
||||
# (sonnet, opus, opus:max) competitors.
|
||||
|
||||
name: Adversarial Review (Gadfly)
|
||||
|
||||
@@ -33,57 +32,21 @@ jobs:
|
||||
review:
|
||||
# Security: only trusted users may trigger a secret-bearing run via a PR
|
||||
# comment (pull_request + workflow_dispatch are already trusted). Mirrors
|
||||
# GADFLY_ALLOWED_USERS, the in-container belt-and-suspenders check.
|
||||
# the allowed_users input below, the in-container belt-and-suspenders check.
|
||||
if: >-
|
||||
github.event_name != 'issue_comment'
|
||||
|| (github.event.issue.pull_request
|
||||
&& (github.actor == 'steve'
|
||||
|| github.actor == 'fizi'
|
||||
|| github.actor == 'dazed'))
|
||||
runs-on: ubuntu-latest
|
||||
# Fleet (6 cloud + claude-code/sonnet, all concurrent) reviewing every PR
|
||||
# with the 3-lens suite. All cloud now, so runs are fast.
|
||||
timeout-minutes: 90
|
||||
steps:
|
||||
- uses: docker://gitea.stevedudenhoeffer.com/steve/gadfly:sha-c342bdb
|
||||
env:
|
||||
GITEA_API: ${{ github.server_url }}/api/v1/repos/${{ github.repository }}
|
||||
GITEA_TOKEN: ${{ secrets.GITEA_TOKEN }}
|
||||
OLLAMA_CLOUD_API_KEY: ${{ secrets.OLLAMA_CLOUD_API_KEY }}
|
||||
# Claude Code engine auth (for the claude-code/* entries in GADFLY_MODELS
|
||||
# below): Pro/Max subscription token. Dogfoods the Phase-1 engine on
|
||||
# gadfly's own PRs as a competitor alongside the Ollama models.
|
||||
CLAUDE_CODE_OAUTH_TOKEN: ${{ secrets.CLAUDE_CODE_OAUTH_TOKEN }}
|
||||
# Fleet: 6 cloud (3 at a time) + Claude Code (sonnet, opus, and opus
|
||||
# with max extended thinking) — one
|
||||
# consolidated comment each, all cloud now. The local Macs (m1/m5) and
|
||||
# the weaker cloud models (gemma4, gpt-oss:120b, kimi-k2.7-code) were
|
||||
# dropped as low-signal. The claude-code/* entries run the Phase-1
|
||||
# engine as competitors in their own lane (need CLAUDE_CODE_OAUTH_TOKEN).
|
||||
GADFLY_MODELS: "minimax-m3:cloud,glm-5.2:cloud,glm-5.1:cloud,deepseek-v4-pro:cloud,nemotron-3-super:cloud,qwen3-coder:480b-cloud,claude-code/sonnet,claude-code/opus,claude-code/opus:max"
|
||||
# cloud runs 3 at once; claude-code 2 at a time; both lanes parallel.
|
||||
GADFLY_PROVIDER_CONCURRENCY: "ollama-cloud=3,claude-code=2"
|
||||
# 3 cloud models x 3 lenses = 9 concurrent ollama-cloud queries (under the 10 budget).
|
||||
GADFLY_PROVIDER_LENS_CONCURRENCY: "ollama-cloud=3"
|
||||
# Default => the 3-lens suite (security, correctness, error-handling).
|
||||
# Set the repo var GADFLY_SPECIALISTS to override (csv / "all" / "auto").
|
||||
GADFLY_SPECIALISTS: ${{ vars.GADFLY_SPECIALISTS || 'security,correctness,error-handling' }}
|
||||
# Per-lens deadline + bounded steps (also bounds the claude-code lane).
|
||||
GADFLY_TIMEOUT_SECS: "600"
|
||||
GADFLY_MAX_STEPS: "14"
|
||||
# Allow-list for the comment trigger (mirrors the job-level if: guard).
|
||||
GADFLY_ALLOWED_USERS: "steve,fizi,dazed"
|
||||
# --- findings telemetry: POST runs + findings to the gadfly-reports store ---
|
||||
# Advisory & off unless GADFLY_FINDINGS_URL is set; failures only log to
|
||||
# stderr and never affect the review. GADFLY_REPO / GADFLY_PR are derived
|
||||
# in-container; the URL + token are user-scope secrets.
|
||||
GADFLY_FINDINGS_URL: ${{ secrets.GADFLY_FINDINGS_URL }}
|
||||
GADFLY_FINDINGS_TOKEN: ${{ secrets.GADFLY_FINDINGS_TOKEN }}
|
||||
# --- event context (leave as-is) ---
|
||||
EVENT_NAME: ${{ github.event_name }}
|
||||
PR: ${{ github.event.pull_request.number || github.event.issue.number || github.event.inputs.pr_number }}
|
||||
PR_BRANCH: ${{ github.head_ref }}
|
||||
IS_DRAFT: ${{ github.event.pull_request.draft }}
|
||||
COMMENT_BODY: ${{ github.event.comment.body }}
|
||||
COMMENT_ID: ${{ github.event.comment.id }}
|
||||
ACTOR: ${{ github.actor }}
|
||||
uses: ./.gitea/workflows/review-reusable.yml
|
||||
secrets: inherit
|
||||
with:
|
||||
models: "minimax-m3:cloud,glm-5.2:cloud,glm-5.1:cloud,deepseek-v4-pro:cloud,nemotron-3-super:cloud,qwen3-coder:480b-cloud,claude-code/sonnet,claude-code/opus,claude-code/opus:max"
|
||||
specialists: "security,correctness,error-handling"
|
||||
provider_concurrency: "ollama-cloud=3,claude-code=2"
|
||||
provider_lens_concurrency: "ollama-cloud=3"
|
||||
timeout_secs: "600"
|
||||
max_steps: "14"
|
||||
allowed_users: "steve,fizi,dazed"
|
||||
timeout_minutes: 90
|
||||
|
||||
Reference in New Issue
Block a user