feat: implement issue #301 — Compliance: non-stub-feature-ideation.yml - #302
feat: implement issue #301 — Compliance: non-stub-feature-ideation.yml#302don-petry wants to merge 1 commit into
Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
Warning Review limit reached
Next review available in: 9 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the 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 configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
|
Note Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported. |
|
There was a problem hiding this comment.
Pull request overview
This PR aims to address compliance issue #301 by updating the repo’s feature-ideation.yml workflow stub that delegates to the org-wide reusable feature ideation workflow.
Changes:
- Added additional
workflow_dispatchinputs (enhance_backlog,target_discussion) and adiscussion→workflow_dispatch“redispatch” bridge job. - Updated the reusable-workflow
uses:pin to a different commit SHA (still labeled# v1).
| enhance_backlog: | ||
| description: 'Backlog enhancement sweep: enhance this repo''s open, human-authored, not-yet-enhanced Ideas (one comment each, idempotent). Combine with dry_run=true to preview.' | ||
| required: false | ||
| default: false | ||
| type: boolean | ||
| target_discussion: | ||
| description: 'Internal — set by `redispatch` when bridging a `discussion:created` event to workflow_dispatch (claude-code-action does not support discussion contexts). Not for manual use.' | ||
| required: false | ||
| default: '' | ||
| type: string |
There was a problem hiding this comment.
No change needed here. The current file is byte-for-byte identical to the canonical stub at petry-projects/.github/standards/workflows/feature-ideation.yml (confirmed via diff). The canonical standard was updated to include the enhance_backlog and target_discussion inputs as well as the redispatch bridge job — so those additions are upstream, not deviations from it.
| id-token: write | ||
| actions: read | ||
| uses: petry-projects/.github/.github/workflows/feature-ideation-reusable.yml@c2beceb1c3ffcaf7981db22d168de669c9fdf816 # v1 | ||
| uses: petry-projects/.github/.github/workflows/feature-ideation-reusable.yml@e20a8fac1b6a10bd6ad0999258e88a423f890ba6 # v1 |
There was a problem hiding this comment.
No change needed here. The canonical stub at petry-projects/.github/standards/workflows/feature-ideation.yml itself uses @e20a8fac1b6a10bd6ad0999258e88a423f890ba6 # v1 (SHA pin with a v1 comment), and the current file is byte-for-byte identical to that template. SHA pinning is also intentional — it follows GitHub's supply-chain security guidance to prevent a compromised tag from injecting malicious code. Switching to a floating @v1 tag would diverge from the canonical template (contradicting Thread 1) and reduce security posture. If the canonical needs updating to a newer SHA, that change belongs in petry-projects/.github and will propagate to all callers on next sync.
|
Closing this conflicted (DIRTY) PR — main has evolved since it was generated (2026-07-03/07), so a hand-merge would drag stale code over main's newer version. Driving issue #301 keeps its |
Pull request was closed



Closes #301
Implemented by dev-lead agent. Please review.