Skip to content

dev-lead: deferred review findings #531

Closed as not planned
Closed as not planned

Description

@don-petry

Single tracker for valid review findings deferred out of scope by dev-lead.

  • PR feat: implement issue #508 — Compliance: stub-surface-drift-pr-auto-review.yml-concurrency #521 (CodeRabbit summary hardening proposals): (1) validate the shared pr-auto-review reusable workflow's interruption/repetition contract (partial dispatch, cleanup, recovery) before treating same-commit cancellation as end-to-end safe; (2) strengthen scripts/test-pr-auto-review-workflow.sh with semantic event-case evaluation of the concurrency expressions rather than token/regex presence.

Activity

  1. don-petry commented on Oct 13, 2026

    @don-petry
    ContributorAuthor

    PR #539 (synced stubs; fixes belong upstream in petry-projects/.github): rate-limit gate counts deferred runs toward cooldown/daily budget (combsmith/.github#1327); gate invoked without --labels so security exemption is lost (combsmith/.github#1326); pr-review.yml trigger set lacks opened / fork runs lack secrets (template-owned trigger set). Comments: IC_kwDORsnIRM8AAAABbVs-gw (CodeRabbit), IC_kwDORsnIRM8AAAABbVxV9g (CodeAnt). Threads: #539

  2. don-petry commented on Oct 21, 2026

    @don-petry
    ContributorAuthor

    Closing after the 2026-10-21 backlog triage.

    Why: Duplicate of upstream .github#1326/#1327; remaining items are speculative bot hardening ideas
    Evidence: combsmith/.github#1326

    Reopen if this still reproduces on main.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions