Skip to content

chore: sync 5 org-standard workflow stub(s) from petry-projects/.github - #440

Merged
don-petry merged 5 commits into
mainfrom
standards-sync/workflows-20260722
Jul 22, 2026
Merged

don-petry merged 5 commits into
mainfrom
standards-sync/workflows-20260722

Conversation

@don-petry

@don-petry don-petry commented Jul 22, 2026 •

Copy link
Copy Markdown
Contributor

User description

Syncs the following org-standard workflow stub(s) from petry-projects/.github (standards/workflows/), deployed verbatim:

  • dev-lead.yml
  • auto-rebase.yml
  • dependabot-automerge.yml
  • pr-auto-review.yml
  • feature-ideation.yml

Opened by scripts/deploy-standard-workflows.sh. Stubs are thin callers; all behaviour lives in the reusables. See standards/ci-standards.md. Labeled standards-sync and left for the normal review/auto-merge pipeline — the deploy script never merges directly.


CodeAnt-AI Description

Sync workflow stubs to the latest org-standard channels and secret handling

What Changed

  • Updated the shared workflow callers to use the current stable release channels for auto-rebase, dev lead, PR auto-review, dependabot auto-merge, and feature ideation
  • PR auto-review now accepts the canonical org PAT as a fallback, so it can keep running when the preferred secret name is not present
  • Kept the existing permissions and workflow behavior otherwise unchanged

Impact

✅ Fewer workflow failures after org updates
✅ More reliable PR auto-review runs
✅ Safer centralized workflow rollouts

💡 Usage Guide

Checking Your Pull Request

Every time you make a pull request, our system automatically looks through it. We check for security issues, mistakes in how you're setting up your infrastructure, and common code problems. We do this to make sure your changes are solid and won't cause any trouble later.

Talking to CodeAnt AI

Got a question or need a hand with something in your pull request? You can easily get in touch with CodeAnt AI right here. Just type the following in a comment on your pull request, and replace "Your question here" with whatever you want to ask:

@codeant-ai ask: Your question here

This lets you have a chat with CodeAnt AI about your pull request, making it easier to understand and improve your code.

Example

@codeant-ai ask: Can you suggest a safer alternative to storing this secret?

Preserve Org Learnings with CodeAnt

You can record team preferences so CodeAnt AI applies them in future reviews. Reply directly to the specific CodeAnt AI suggestion (in the same thread) and replace "Your feedback here" with your input:

@codeant-ai: Your feedback here

This helps CodeAnt AI learn and adapt to your team's coding style and standards.

Example

@codeant-ai: Do not flag unused imports.

Retrigger review

Ask CodeAnt AI to review the PR again, by typing:

@codeant-ai: review

Check Your Repository Health

To analyze the health of your code repository, visit our dashboard at https://app.codeant.ai. This tool helps you identify potential issues and areas for improvement in your codebase, ensuring your repository maintains high standards of code health.

@don-petry don-petry added the standards-sync Org-standard workflow stub synced from petry-projects/.github label Jul 22, 2026
@don-petry
don-petry requested a review from a team as a code owner July 22, 2026 17:33
@don-petry don-petry added the standards-sync Org-standard workflow stub synced from petry-projects/.github label Jul 22, 2026
@codeant-ai

codeant-ai Bot commented Jul 22, 2026 •

Copy link
Copy Markdown

🤖 CodeAnt AI — Review Status

Status Commit Started (UTC) Finished (UTC)
✅ Reviewed your PR a1de846 Jul 22, 2026 · 17:33 17:34

Updated in place by CodeAnt AI · last 5 reviews

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@coderabbitai

coderabbitai Bot commented Jul 22, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@don-petry, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 55 minutes

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 654df50b-2349-4d89-8867-02b47c839a24

📥 Commits

Reviewing files that changed from the base of the PR and between 259c744 and a1de846.

📒 Files selected for processing (5)
  • .github/workflows/auto-rebase.yml
  • .github/workflows/dependabot-automerge.yml
  • .github/workflows/dev-lead.yml
  • .github/workflows/feature-ideation.yml
  • .github/workflows/pr-auto-review.yml
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch standards-sync/workflows-20260722

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@gemini-code-assist

Copy link
Copy Markdown

Note

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

@codeant-ai codeant-ai Bot added the size:S This PR changes 10-29 lines, ignoring generated files label Jul 22, 2026
@qodo-code-review

Copy link
Copy Markdown

PR Summary by Qodo

Sync org-standard GitHub Actions workflow stubs to latest moving-channel tags

⚙️ Configuration changes 🕐 Less than 10 minutes

Grey Divider

AI Description

• Re-pin org-standard workflow stubs to the current moving-channel refs (v1/v2-stable).
• Add inline NOSONAR justifications for trusted secrets: inherit usage.
• Update PR auto-review to use canonical-first PAT secret fallback while keeping key name stable.
Diagram

graph TD
  A["Repo workflow stubs (.github/workflows)"] --> B["Reusable workflows (petry-projects/.github)"] --> C["Moving-channel tags (v1/v2-stable)"]
  A --> D["Reusable workflows (petry-projects/.github-private)"] --> C
  A --> E["Repo secrets"]
  B --> F["Repo checks / PR gates"]
  D --> F
Loading
High-Level Assessment

This is the expected org-standard approach: keep repository workflows as thin stubs and pin to centrally managed moving-channel tags (v1/v2-stable) to enable rapid org-wide rollout/rollback. Alternatives like pinning to SHA/@main or in-repo duplication would reduce central control and conflict with the stated compliance/standards model.

Files changed (5) +24 / -21

Other (5) +24 / -21
auto-rebase.ymlRe-pin auto-rebase stub to v2-stable and annotate secrets inheritance +2/-2

Re-pin auto-rebase stub to v2-stable and annotate secrets inheritance

• Updates the documented/pinned moving-channel ref from 'auto-rebase/stable' to 'auto-rebase/v2-stable'. Adds a NOSONAR justification comment to the trusted 'secrets: inherit' usage.

.github/workflows/auto-rebase.yml

dependabot-automerge.ymlAnnotate dependabot-automerge secrets inheritance as trusted reusable usage +1/-1

Annotate dependabot-automerge secrets inheritance as trusted reusable usage

• Keeps the reusable pinned to 'dependabot-automerge/v2-stable' and adds a NOSONAR comment clarifying that 'secrets: inherit' is used with a first-party trusted reusable.

.github/workflows/dependabot-automerge.yml

dev-lead.ymlMove dev-lead stub to v1-stable channel and align agent_ref +5/-5

Move dev-lead stub to v1-stable channel and align agent_ref

• Switches the reusable ref from 'dev-lead/stable' to 'dev-lead/v1-stable' and updates 'agent_ref' to match. Adds a NOSONAR justification comment to the trusted 'secrets: inherit' usage.

.github/workflows/dev-lead.yml

feature-ideation.ymlRe-pin feature-ideation stub to v1-stable channel +1/-1

Re-pin feature-ideation stub to v1-stable channel

• Updates the reusable workflow reference from 'feature-ideation/stable' to 'feature-ideation/v1-stable', keeping the rest of the stub intact.

.github/workflows/feature-ideation.yml

pr-auto-review.ymlRe-pin pr-auto-review to v1-stable and add canonical-first PAT secret fallback +15/-12

Re-pin pr-auto-review to v1-stable and add canonical-first PAT secret fallback

• Updates header guidance to enforce the moving-channel model and changes the reusable ref from 'pr-auto-review/stable' to 'pr-auto-review/v1-stable'. Updates the 'GH_PAT_WORKFLOWS' secret value to prefer 'GH_PAT_DON_PETRY' with fallback to the existing secret, preserving the passed key name for reusable compatibility.

.github/workflows/pr-auto-review.yml

@don-petry

Copy link
Copy Markdown
Contributor Author

Dev-Lead — review-changes (no-changes)

No changes were needed for this PR.

@don-petry
don-petry enabled auto-merge (squash) July 22, 2026 17:36
@qodo-code-review

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (1) 📘 Rule violations (0) 📜 Skill insights (0)

Context used
✅ Compliance rules (platform): 124 rules
✅ Skills: 5 invoked
  bmad-testarch-ci
  bmad-testarch-test-review
  bmad-testarch-atdd
  bmad-testarch-framework
  bmad-testarch-automate

Grey Divider


Informational

1. PAT secret docs mismatch 🐞 Bug ⚙ Maintainability
Description
.github/workflows/pr-auto-review.yml now prefers secrets.GH_PAT_DON_PETRY (falling back to
secrets.GH_PAT_WORKFLOWS), but the header still states that GH_PAT_WORKFLOWS is the required org
secret, which can mislead adopters configuring secrets.
Code

.github/workflows/pr-auto-review.yml[R55-58]

+      # Canonical-first fallback (.github-private#1326). The reusable resolves this
+      # secret BY NAME, so only the VALUE changes — the passed key stays named
+      # GH_PAT_WORKFLOWS.
+      GH_PAT_WORKFLOWS: ${{ secrets.GH_PAT_DON_PETRY || secrets.GH_PAT_WORKFLOWS }}
Relevance

⭐⭐⭐ High

Team previously accepted updating workflow headers to match actual uses:/config (doc-reality
mismatch) in PR #170.

PR-#170

ⓘ Recommendations generated based on similar findings in past PRs

Evidence
The workflow header documents only GH_PAT_WORKFLOWS as required, while the job now explicitly
prefers GH_PAT_DON_PETRY via an expression fallback, making the header incomplete/out-of-date.

.github/workflows/pr-auto-review.yml[24-24]
.github/workflows/pr-auto-review.yml[53-58]

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

### Issue description
`pr-auto-review.yml` now supports two secret sources (preferred `GH_PAT_DON_PETRY`, fallback `GH_PAT_WORKFLOWS`), but the file header still documents only `GH_PAT_WORKFLOWS` as required.

### Issue Context
This is a thin workflow stub intended to be copied verbatim into other repos; stale “required secret” guidance can cause incorrect setup assumptions.

### Fix Focus Areas
- .github/workflows/pr-auto-review.yml[24-24]
- .github/workflows/pr-auto-review.yml[55-58]

### Suggested change
Update the header line to something like:
- `Requires: GH_PAT_DON_PETRY (preferred) or GH_PAT_WORKFLOWS (fallback) org secret.`

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools


Grey Divider

Qodo Logo

Comment thread .github/workflows/pr-auto-review.yml
@sonarqubecloud

Copy link
Copy Markdown

@donpetry-bot donpetry-bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Automated review — APPROVED ✓

Risk: MEDIUM
Reviewed commit: a1de84668a032443ec39c80e88ba556d491f9db6
Cascade: triage → audit (triage: haiku 4.5 → deep: opus 4.8 + duck: o4-mini → audit: fable 5)

Summary

Standards-sync fanout of 5 thin caller stubs; I independently verified that 4/5 files (including pr-auto-review.yml, the only substantive change) are byte-identical to the org templates in petry-projects/.github at PR head. The secret change (GH_PAT_DON_PETRY || GH_PAT_WORKFLOWS) is a documented, org-owner-authored migration (.github-private#1326) using a safe secrets-context-only fallback expression: value-only change, secret name and job permissions unchanged, first-party reusable refs, and all security scanners (CodeQL incl. actions, gitleaks, AgentShield, SonarCloud) green. The personal-PAT-as-canonical-credential concern is a real governance tradeoff, but the decision point is the upstream template and issue #1326, not this verbatim fanout, and it is explicitly tracked with gated cleanup criteria.

Findings

  • minor: Confirmed (accepted documented risk): GH_PAT_DON_PETRY, a personal-account PAT, becomes the preferred credential for org PR-review automation. Ties automation to an individual account (rotation/offboarding/scope breadth vs. a GitHub App or dedicated machine credential). Documented and gated in .github-private#1326; recommend eventually migrating to a GitHub App installation token. Non-blocking here because this PR is a verbatim fanout of the already-adopted org template.
  • minor: Stub header still documents GH_PAT_WORKFLOWS as the required org secret while the job now prefers GH_PAT_DON_PETRY. Fix belongs upstream in the template (standards/workflows/pr-auto-review.yml) since stubs are deployed verbatim; adopting repos could otherwise provision only the fallback secret.
  • info: Sync-completeness gap: feature-ideation.yml at PR head lags the org template — template lines 110/118 already use the GH_PAT_DON_PETRY-first fallback for the prep job's GH_TOKEN, but this PR only updated the uses: channel tag. Not a security regression (head retains the older GH_PAT_WORKFLOWS-only behavior, valid during the migration window), but the next sync should reconcile it.
  • info: Moving-channel tags (v1/v2-stable) on reusable workflow refs are contrary to generic SHA-pinning guidance but are the org's mandated versioning model per ci-standards.md, and all refs target first-party petry-projects repos. Tag renames in this PR are cosmetic channel-name changes with no permission or trigger changes.

Reviewed by the PR-review cascade (triage: haiku 4.5 → deep: opus 4.8 + duck: o4-mini → audit: fable 5). Reply if you need a human review.

@don-petry
don-petry merged commit 3a52a4f into main Jul 22, 2026
33 of 35 checks passed
@don-petry
don-petry deleted the standards-sync/workflows-20260722 branch July 22, 2026 20:23
don-petry added a commit that referenced this pull request Aug 4, 2026
…ub (#440)

* chore: sync 5 org-standard workflow stub(s) from petry-projects/.github

* chore: sync 5 org-standard workflow stub(s) from petry-projects/.github

* chore: sync 5 org-standard workflow stub(s) from petry-projects/.github

* chore: sync 5 org-standard workflow stub(s) from petry-projects/.github

* chore: sync 5 org-standard workflow stub(s) from petry-projects/.github
don-petry added a commit that referenced this pull request Aug 4, 2026
…uire_code_owner_review (#433)

* Add BMAD method workspace and resume beekeeping brainstorming session

* Add product brief for bmad-method outlining core vision, user journeys, and success metrics

* Remove _bmad-output (moved to separate PR)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* Remove duplicate .github/skills content

Skills already exist under .claude/skills.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* Add beekeeping mentor app planning artifacts (_bmad-output)

Includes brainstorming session, market and domain research reports,
product brief, PRD, architecture, implementation readiness report,
and UX design directions and specification.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* Add zero-tap beeyard experience — core differentiator

MAJOR: Broodly's key differentiator is now a zero-tap beeyard session.
Once inspection starts, the user NEVER needs to tap the phone in the field.

New FRs:
- FR19c: Continuous multi-hive voice session (zero-tap, voice navigation)
- FR19d: Per-hive context during multi-hive session (auto-save, TTS announce)
- FR30c: Post-session Evening Review (tap-friendly at home)
- FR19 updated: session-based not per-hive
- FR27 updated: hive navigation commands (next/move to/which/how many/end)

New stories: 8.8 (multi-hive voice session), 8.9 (evening review)
Total: 64 stories across 12 epics

Prototype: 30 screens (+3 new)
- inspection-next-hive: voice transition between hives with context
- inspection-session-end: session summary with stats
- evening-review: tap-friendly per-hive correction UI
- Homepage: "Review today's session" card added

CLAUDE.md: Zero-Tap Beeyard section added as key differentiator
UX spec: Continuous session + evening review patterns added

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* Goal slider, scale weights, card variants, voice-top plan, materials list

1. Goal Selection: replaced multi-select chips with Honey↔Splits
   sliding scale (competing priorities). Removed Learning/Growth.
   Colony health always prioritized. FR2b updated.
2. Homepage: added regional scale weight card (beecounted.org)
   showing daily weight trend for user's area. New FR10a, FR11c.
3. Design system: actionable cards (amber left border + chevron +
   hover) vs informational cards (no affordance). Applied across
   all prototype screens. CLAUDE.md + PRD updated.
4. Weekly Plan: Live Discussion Mode banner moved to top —
   users can start talking immediately. FR13 updated.
5. Weekly Plan: Required Materials checklist before apiary
   accordions (treatments, equipment, syrup). New FR13a.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* Add coding standards, Live Mode icon, notification types, and UX refinements

Update CLAUDE.md with coding standards section (org standards + Broodly-specific
guidelines for TypeScript, Go, and monorepo). Update PRD with FR12b2 (persistent
Live Mode EQ icon), FR30c (photos/videos in Evening Review), FR40/FR40a2
(per-category notification toggles and types screen). Update UX spec with icon
conventions, notification types screen, and Live Mode activation from app header.
Add Live Mode icon exploration HTML files and test framework spec.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix: add project copyright notice to LICENSE (#7)

* fix: add project copyright notice to LICENSE

Add project-specific copyright notice for Don Petry at the top of the
LICENSE file, before the AGPL-3.0 license text.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix: revert LICENSE to verbatim AGPL-3.0 text

Remove the copyright line from the LICENSE file as it makes the file
non-verbatim, violating the FSF's "changing it is not allowed" clause.
The project copyright is already properly declared in the NOTICE file.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Don Petry <don@petry.com>
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* Add Claude Code GitHub Action (#13)

* Add Claude Code GitHub Action for PR reviews

* fix: address review feedback on Claude Code workflow

- Restrict issue_comment trigger to PR comments only
- Add author-association check (OWNER/MEMBER/COLLABORATOR)
- Add pull_request_review_comment trigger
- Add timeout-minutes to prevent runaway jobs

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix: use CLAUDE_CODE_OAUTH_TOKEN org secret

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix: add id-token: write permission for OAuth auth

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix: address remaining review comments

- Pin claude-code-action to commit SHA for supply-chain safety
- Add fork PR guard (secrets unavailable for fork PRs)
- Scope pull_request trigger to main branch
- Use >- folded scalar for if expression

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: DJ <dj@Rachels-MacBook-Air.local>
Co-authored-by: DJ <dj@Rachels-Air.localdomain>
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix: address OpenSSF Scorecard findings

- Add SECURITY.md (#16)
- Scope workflow token permissions to least privilege (#17)
- Pin all GitHub Action dependencies to commit SHAs (#18)
- Add CodeQL SAST workflow for all commits (#19)
- Broaden CI pull_request trigger to cover all branches (#20)

Closes #16, #17, #18, #19, #20

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix: use 'actions' language for CodeQL since repo has no JS/TS source

The repo contains HTML and Groovy, not JavaScript. CodeQL supports
analyzing GitHub Actions workflows via the 'actions' language.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix: address PR review comments

- Replace permissions: read-all with permissions: {} (deny-by-default)
- Reduce dependabot-automerge job token to read-only (app token handles writes)
- Add actions: read to CodeQL job permissions
- Add concrete security contact email and version target to SECURITY.md

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix: use claude_code_oauth_token instead of anthropic_api_key

The action has separate inputs for API keys vs OAuth tokens.
CLAUDE_CODE_OAUTH_TOKEN is an OAuth token, not an API key.

* Add AGENTS.md with full project rules, slim down CLAUDE.md

Follows the org convention where AGENTS.md is the comprehensive
agent-agnostic file and CLAUDE.md is a Claude Code-specific
summary with @import for org-wide standards.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* Bump github/codeql-action from 3.35.1 to 4.35.1 (#47)

Bumps [github/codeql-action](https://github.com/github/codeql-action) from 3.35.1 to 4.35.1.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](github/codeql-action@5c8a8a6...c10b806)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-version: 4.35.1
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump actions/checkout from 4.3.1 to 6.0.2 (#40)

Bumps [actions/checkout](https://github.com/actions/checkout) from 4.3.1 to 6.0.2.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@34e1148...de0fac2)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-version: 6.0.2
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: don-petry <36422719+don-petry@users.noreply.github.com>

* ci: skip Claude Code reviewer on Dependabot PRs (#50)

* ci: skip Claude Code reviewer on Dependabot PRs

The claude workflow fails on Dependabot PRs because secrets
(CLAUDE_CODE_OAUTH_TOKEN) are not available to the dependabot actor.
This blocks the dependabot auto-merge automation when claude is a
required status check.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* ci: use PR author login instead of github.actor for Dependabot check

github.actor reflects who triggered the workflow run (e.g. a maintainer
reopening), not the PR author. Use github.event.pull_request.user.login
for reliable Dependabot detection, consistent with dependabot-automerge.yml.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: DJ <dj@Rachels-Air.localdomain>
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* ci: move Dependabot exclusion to step-level in Claude workflow (#52)

* ci: move Dependabot exclusion to step-level in Claude workflow

Move the dependabot[bot] check from job-level `if` to step-level `if`
so the claude job runs and reports SUCCESS (with a skipped step) instead
of being skipped entirely. A skipped job doesn't satisfy required status
checks in branch protection, but a successful job with a skipped step does.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* ci: guard step-level Dependabot check for pull_request events only

The step-level if needs to handle issue_comment and
pull_request_review_comment events where github.event.pull_request
is not present. Use event_name guard to avoid null dereference.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: DJ <dj@Rachels-Air.localdomain>
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* Bump anthropics/claude-code-action from 1.0.80 to 1.0.82 (#39)

Bumps [anthropics/claude-code-action](https://github.com/anthropics/claude-code-action) from 1.0.80 to 1.0.82.
- [Release notes](https://github.com/anthropics/claude-code-action/releases)
- [Commits](anthropics/claude-code-action@094bd24...88c168b)

---
updated-dependencies:
- dependency-name: anthropics/claude-code-action
  dependency-version: 1.0.82
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* chore(deps): bump anthropics/claude-code-action from 1.0.83 to 1.0.88 (#55)

Bumps [anthropics/claude-code-action](https://github.com/anthropics/claude-code-action) from 1.0.83 to 1.0.88.
- [Release notes](https://github.com/anthropics/claude-code-action/releases)
- [Commits](anthropics/claude-code-action@bee87b3...1eddb33)

---
updated-dependencies:
- dependency-name: anthropics/claude-code-action
  dependency-version: 1.0.88
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* chore: enable Claude issue trigger per org CI standard (#69)

* chore: enable Claude issue trigger per org CI standard

Add issues:[labeled] event trigger and claude label support so Claude
can work issues like a human contributor — reading the issue, creating
a branch, implementing the fix, and opening a PR.

Changes:
- Add issues:[labeled] trigger to on: block
- Add issue label condition to job if: guard
- Upgrade contents permission to write (needed for branch creation)
- Pin claude-code-action to v1.0.89 (6e2bd528)
- Add label_trigger: "claude" input

Matches the standard configuration defined in
combsmith/.github#24 (standards/ci-standards.md § Claude Code).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* chore: add permission comment per CodeRabbit review

Document why contents: write is needed (issue-triggered branch creation).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix: add checkout step to Claude workflow for issue-triggered mode (#70)

The claude-code-action runs git fetch/checkout internally during branch
setup but requires the repository to already be cloned on the runner.
Without actions/checkout, issue-triggered runs fail with:
  fatal: not a git repository

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* feat: split Claude workflow into interactive + issue automation jobs (#79)

* feat: split Claude workflow into interactive + issue automation jobs

Align with org CI standard for Claude Code workflows. The single
monolithic job is now two:

- `claude` — interactive PR reviews and @claude mentions
- `claude-issue` — issue-labeled automation that implements, opens a PR,
  self-reviews, checks CI, and notifies code owners

Adds `actions: read` and `checks: read` permissions plus explicit
allowed-tools and prompt for the issue automation job.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix: add concurrency guard and comment tools to claude-issue job

- Add concurrency group keyed on issue number to prevent duplicate runs
- Add gh pr comment and gh issue comment to allowedTools for review
  replies, thread resolution, and code owner tagging
- Remove Bash(cat:*) since the Read tool already covers file reads

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: DJ <dj@Rachels-MacBook-Air.local>
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* feat: switch to org-level reusable Claude Code workflow (#80)

* chore: add CODEOWNERS file for code owner review enforcement

Adds .github/CODEOWNERS assigning @don-petry as the default code owner
for all repository paths, resolving the missing-codeowners compliance finding.

Closes #68

Co-authored-by: don-petry <don-petry@users.noreply.github.com>

* chore(workflows): adopt centralized stubs from petry-projects/.github (#93)

Replace inline copies of standardized workflows with the canonical
thin caller stubs from petry-projects/.github/standards/workflows/.
Each stub delegates to a versioned reusable workflow at
petry-projects/.github/.github/workflows/<name>-reusable.yml@v1, so
future updates to the standard propagate automatically and drift is
caught by the org-wide compliance audit.

See combsmith/.github#87, #88, #89 for context.

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* chore(deps): bump github/codeql-action from 4.35.1 to 4.35.2 (#156)

Bumps [github/codeql-action](https://github.com/github/codeql-action) from 4.35.1 to 4.35.2.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](github/codeql-action@c10b806...95e58e9)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-version: 4.35.2
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* fix: correct reusable workflow path (remove duplicate .github/) (#162)

fix: correct reusable workflow path (remove duplicate .github/ segment)

Changed: petry-projects/.github/.github/workflows/...
To:      petry-projects/.github/workflows/...

Co-authored-by: Claude Haiku 4.5 <noreply@anthropic.com>

* Revert "fix: correct reusable workflow path (remove duplicate .github/) (#162)"

This reverts commit 7d2897e.

* ci: add auto-rebase workflow and check_run trigger to claude.yml

* add check_run trigger to claude.yml

* add auto-rebase.yml workflow

* chore: add bot accounts to CODEOWNERS for auto-merge support

* fix: use explicit secrets and write permissions in dependabot-rebase workflow (#170)

* fix: use explicit secrets and write permissions in dependabot-rebase workflow

secrets: inherit + permissions: read causes startup_failure on reusable
workflows. Use explicit APP_ID/APP_PRIVATE_KEY secrets and write permissions
so the reusable workflow can update branches and approve PRs.

* fix: pin reusable SHA, update header guidance, fix secrets comment

Address Copilot review comments:
- Pin uses: to commit SHA instead of mutable @v1 tag
- Update header: 'SHA' → 'ref', remove ban on workflow_dispatch trigger
- Update header: '(inherited)' → '(passed explicitly)' for secrets

* fix: apply prettier formatting (single space before inline comments)

* fix: apply prettier formatting to auto-rebase.yml (single space before inline comments)

* chore: standardize CODEOWNERS on @petry-projects/org-leads (#172)

Per the org-wide standard defined in petry-projects/.github
(standards/codeowners-standard.md), replace individual user/bot
listings with the @petry-projects/org-leads team.

Closes the CODEOWNERS gap from pr-review-agent#27.

Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>

* chore(deps): bump github/codeql-action from 4.35.2 to 4.35.3 (#174)

Bumps [github/codeql-action](https://github.com/github/codeql-action) from 4.35.2 to 4.35.3.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](github/codeql-action@95e58e9...e46ed2c)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-version: 4.35.3
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: dependabot-automerge-petry[bot] <270452309+dependabot-automerge-petry[bot]@users.noreply.github.com>

* chore(deps): bump github/codeql-action from 4.35.3 to 4.35.4 (#190)

Bumps [github/codeql-action](https://github.com/github/codeql-action) from 4.35.3 to 4.35.4.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](github/codeql-action@e46ed2c...68bde55)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-version: 4.35.4
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* ci: remove drift codeql.yml, enable GitHub-managed default setup (#126)

ci: remove drift codeql.yml and enable GitHub-managed default setup

Per org standard §2, CodeQL must use GitHub-managed default setup
(Settings → Code security → Code scanning), not a per-repo workflow
file. The existing codeql.yml was identified as drift by the compliance
audit (issue #109).

CodeQL default setup has been configured via the GitHub API:
  gh api -X PATCH repos/petry-projects/broodly/code-scanning/default-setup \
    -F state=configured -F query_suite=default

This removes the drift file and brings the repo into compliance with
the codeql-default-setup-not-configured finding.

Closes #109

Co-authored-by: claude[bot] <41898282+claude[bot]@users.noreply.github.com>
Co-authored-by: don-petry <don-petry@users.noreply.github.com>

* chore(dev-lead): remove claude.yml — replaced by dev-lead.yml (#195)

* fix: add *.pem to .gitignore for push-protection compliance (#141)

Adds `*.pem` to .gitignore alongside existing key/cert patterns
(`*.jks`, `*.p8`, `*.p12`, `*.key`) to satisfy the
gitignore_secrets_block compliance check from the org push-protection
standard.

Closes #113

Co-authored-by: claude[bot] <41898282+claude[bot]@users.noreply.github.com>
Co-authored-by: don-petry <don-petry@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* feat: implement issue #193 — Compliance: unpinned-actions-dev-lead.yml (#208)

Co-authored-by: donpetry-bot <281750570+donpetry-bot@users.noreply.github.com>

* feat: implement issue #189 — Compliance: dependabot_security_updates (#210)

Co-authored-by: donpetry-bot <281750570+donpetry-bot@users.noreply.github.com>

* feat: implement issue #178 — Compliance: codeowners-no-catchall (#209)

Co-authored-by: donpetry-bot <281750570+donpetry-bot@users.noreply.github.com>

* feat: implement issue #188 — Compliance: secret_scanning_non_provider_patterns (#212)

Co-authored-by: donpetry-bot <281750570+donpetry-bot@users.noreply.github.com>

* feat: implement issue #103 — Compliance: unpinned-actions-dependency-audit.yml (#221)

Co-authored-by: donpetry-bot <281750570+donpetry-bot@users.noreply.github.com>

* feat: implement issue #177 — Compliance: codeowners-org-leads-not-first (#215)

Co-authored-by: donpetry-bot <281750570+donpetry-bot@users.noreply.github.com>

* feat: implement issue #181 — Compliance: check-suite-auto-trigger-347564 (#231)

Co-authored-by: donpetry-bot <281750570+donpetry-bot@users.noreply.github.com>

* feat: implement issue #100 — Compliance: unpinned-actions-claude.yml (#236)

Co-authored-by: donpetry-bot <281750570+donpetry-bot@users.noreply.github.com>

* rollout: deploy pr-review-mention standard workflow (#282)

* rollout: deploy pr-review-mention standard workflow

* fix(bot): address bot feedback [skip ci-relay]

---------

Co-authored-by: donpetry-bot <281750570+donpetry-bot@users.noreply.github.com>

* deploy: add pr-review.yml workflow

* feat: implement issue #287 — [Fleet Monitor] petry-projects/broodly — sonarcloud.yml (#288)

Co-authored-by: donpetry-bot <281750570+donpetry-bot@users.noreply.github.com>

* feat: implement issue #72 — fix: broodly CI 'Push API Image' job failing on main (#290)

* feat: implement issue #72 — fix: broodly CI 'Push API Image' job failing on main

* fix(reviews): address review comments [skip ci-relay]

---------

Co-authored-by: donpetry-bot <281750570+donpetry-bot@users.noreply.github.com>
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

* feat: implement issue #262 — Compliance: secret_scanning_non_provider_patterns (#294)

* feat: implement issue #262 — Compliance: secret_scanning_non_provider_patterns

* chore: apply manual instructions [skip ci-relay]

* fix(bot): address bot feedback [skip ci-relay]

---------

Co-authored-by: donpetry-bot <281750570+donpetry-bot@users.noreply.github.com>

* feat: implement issue #177 — Compliance: codeowners-org-leads-not-first (#305)

* feat: implement issue #177 — Compliance: codeowners-org-leads-not-first

* chore: apply manual instructions [skip ci-relay]

* fix: remove duplicate shell-quote override from pnpm-lock.yaml

The pnpm-lock.yaml contained both 'shell-quote' and 'shell-quote@<1.8.4'
overrides, but package.json only specified the versioned variant. This
mismatch caused pnpm install --frozen-lockfile to fail with
ERR_PNPM_LOCKFILE_CONFIG_MISMATCH. Removed the duplicate unversioned entry.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

---------

Co-authored-by: donpetry-bot <281750570+donpetry-bot@users.noreply.github.com>
Co-authored-by: Claude Haiku 4.5 <noreply@anthropic.com>
Co-authored-by: Don Petry Bot <donpetry+bot@gmail.com>

* feat: implement issue #243 — [Fleet Monitor] petry-projects/broodly — ci-failure-analyst.yml (#297)

* fix(deps): pin shell-quote to >=1.8.4 to patch GHSA-w7jw-789q-3m8p

Critical CVE in shell-quote <=1.8.3 allows newline injection via
object .op values. Added pnpm override to force >=1.8.4 and
regenerated pnpm-lock.yaml with the patched version.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* chore: apply manual instructions [skip ci-relay]

* fix(bot): address bot feedback [skip ci-relay]

* fix(deps): broaden dependency overrides to patch nested vulnerabilities

Tightened pnpm dependency overrides from pattern-based constraints (e.g.
tmp@<0.2.6) to unconditional versions (tmp, uuid) to ensure deeply nested
vulnerable packages (e.g., tmp@0.0.33 via patch-package, js-yaml@3.14.2 via
Babel/Jest) are properly patched. This resolves 5 pnpm audit violations
(1 high, 3 moderate, 1 low) and regenerates pnpm-lock.yaml accordingly.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

---------

Co-authored-by: donpetry-bot <281750570+donpetry-bot@users.noreply.github.com>
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

* chore: sync org-standard pr-review-mention.yml stub from petry-projects/.github (#333)

* chore: sync org-standard auto-rebase.yml stub from petry-projects/.github (#335)

* ci: inline NOSONAR(S7637) markers on first-party caller stubs (#549 canonical migration) (#359)

* ci: inline NOSONAR(S7637) marker on add-to-project.yml channel ref (#549)

* ci: inline NOSONAR(S7637) marker on agent-shield.yml channel ref (#549)

* ci: inline NOSONAR(S7637) marker on auto-rebase.yml channel ref (#549)

* ci: inline NOSONAR(S7637) marker on dependabot-automerge.yml channel ref (#549)

* ci: inline NOSONAR(S7637) marker on dependabot-rebase.yml channel ref (#549)

* ci: inline NOSONAR(S7637) marker on dependency-audit.yml channel ref (#549)

* ci: inline NOSONAR(S7637) marker on dev-lead.yml channel ref (#549)

* ci: inline NOSONAR(S7637) marker on pr-auto-review.yml channel ref (#549)

* ci: inline NOSONAR(S7637) marker on pr-review-mention.yml channel ref (#549)

* ci: inline NOSONAR(S7637) marker on pr-review.yml channel ref (#549)

* chore: sync 8 org-standard workflow stub(s) from petry-projects/.github (#387)

* chore: sync 8 org-standard workflow stub(s) from petry-projects/.github

* chore: sync 8 org-standard workflow stub(s) from petry-projects/.github

* chore: sync 8 org-standard workflow stub(s) from petry-projects/.github

* chore: sync 8 org-standard workflow stub(s) from petry-projects/.github

* chore: sync 8 org-standard workflow stub(s) from petry-projects/.github

* chore: sync 8 org-standard workflow stub(s) from petry-projects/.github

* chore: sync 8 org-standard workflow stub(s) from petry-projects/.github

* chore: sync 8 org-standard workflow stub(s) from petry-projects/.github

* chore: sync 8 org-standard workflow stub(s) from petry-projects/.github

* chore: sync 8 org-standard workflow stub(s) from petry-projects/.github

* chore: sync 8 org-standard workflow stub(s) from petry-projects/.github

* chore: sync 8 org-standard workflow stub(s) from petry-projects/.github

* chore: sync 8 org-standard workflow stub(s) from petry-projects/.github

* fix: re-pin agent_ref inputs to v-form (missed by #657 uses:-only re-pin) (#391)

fix: re-pin agent_ref to v-form for add-to-project.yml [#657]

* feat: implement issue #368 — Compliance: non-stub-feature-ideation.yml (#397)

* feat: implement issue #368 — Compliance: non-stub-feature-ideation.yml

* chore: dev-lead update (review-changes) [skip ci-relay]

* chore: dev-lead update (review-changes) [skip ci-relay]

* chore: fix pnpm v11 lockfile mismatch and add @types/jest [skip ci-relay]

Move overrides authority from the now-ignored package.json pnpm.overrides
field to pnpm-workspace.yaml (aligned with lockfile keys). Add @types/jest
as an explicit devDependency to apps/mobile so TypeScript can resolve test
globals under pnpm's strict node_modules isolation. Add "types": ["jest"] to
apps/mobile/tsconfig.json. Regenerate lockfile.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* fix: remove unused test scripts from workspace packages

Packages config, domain-types, graphql-types, test-utils, and ui have
test scripts that invoke jest but contain no test files and jest is not
in their devDependencies. This causes test failure when pnpm run test
is executed recursively. Remove the unused scripts since these packages
are type/domain libraries with no unit test coverage.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

---------

Co-authored-by: donpetry-bot <281750570+donpetry-bot@users.noreply.github.com>
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

* feat: implement issue #410 — Compliance: dev-lead-stub-pin (#412)

Co-authored-by: donpetry-bot <281750570+donpetry-bot@users.noreply.github.com>

* feat: implement issue #411 — Compliance: dev-lead-stub-agent-ref (#413)

Co-authored-by: donpetry-bot <281750570+donpetry-bot@users.noreply.github.com>

* chore: insert org secrets baseline in .gitignore (#420)

* chore: insert org secrets baseline in .gitignore

* chore: dev-lead update (review-changes) [skip ci-relay]

---------

Co-authored-by: donpetry-bot <281750570+donpetry-bot@users.noreply.github.com>

* chore: dev-lead update (review-changes) [skip ci-relay]

* chore: sync 1 org-standard workflow stub(s) from petry-projects/.github (#437)

* chore: sync 5 org-standard workflow stub(s) from petry-projects/.github (#440)

* chore: sync 5 org-standard workflow stub(s) from petry-projects/.github

* chore: sync 5 org-standard workflow stub(s) from petry-projects/.github

* chore: sync 5 org-standard workflow stub(s) from petry-projects/.github

* chore: sync 5 org-standard workflow stub(s) from petry-projects/.github

* chore: sync 5 org-standard workflow stub(s) from petry-projects/.github

* fix(reviews): address review comments [skip ci-relay]

* fix(bot): address bot feedback [skip ci-relay]

* fix(reviews): address review comments [skip ci-relay]

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-authored-by: DJ <dj@Rachels-Air.localdomain>
Co-authored-by: Don Petry <don@petry.com>
Co-authored-by: DJ <dj@Rachels-MacBook-Air.local>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: claude[bot] <41898282+claude[bot]@users.noreply.github.com>
Co-authored-by: don-petry <don-petry@users.noreply.github.com>
Co-authored-by: dependabot-automerge-petry[bot] <270452309+dependabot-automerge-petry[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: donpetry-bot <281750570+donpetry-bot@users.noreply.github.com>
Co-authored-by: Don Petry Bot <donpetry+bot@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S This PR changes 10-29 lines, ignoring generated files standards-sync Org-standard workflow stub synced from petry-projects/.github

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants