Repository navigation
Enable the gated pm-github issue sync - #152
Merged
Merged
Conversation
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info
Summary by CodeRabbit
WalkthroughAdds a GitHub Actions workflow that runs the pinned ChangesGitHub Issue Sync
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Change: Feature Merge Risk: ⚪ Minimal · up to
|
| Check name | Status | Explanation |
|---|---|---|
| Title check |  |
The title clearly and concisely describes the main change: enabling the gated pm-github issue synchronization workflow. |
| Description check | |
The description directly explains the scheduled and manually triggered issue-sync workflow, its gating behavior, pinned reusable workflow, validation, and unchanged dependencies. |
| Docstring Coverage | |
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0… |
| Linked Issues check | |
Check skipped because no linked issues were found for this pull request. |
| Out of Scope Changes check | |
Check skipped because no linked issues were found for this pull request. |
✨ Finishing Touches
-
🧪 Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
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 @coderabbitai help to get the list of available commands.
Reviewer's guide (collapsed on small PRs)Reviewer's GuideEnables this repository’s gated GitHub issue synchronization through a commit-pinned reusable workflow, with a daily schedule and manual dispatch while preserving the existing dependency, merge-driver, and threshold configuration. The reusable pipeline performs preview, gated import, idempotency/privacy checks, health validation, and automation-branch PR creation without pushing directly to main; release checks and packed-consumer acceptance passed under both npm and bun. Sequence diagram for the gated GitHub issue syncsequenceDiagram
participant Scheduler as Schedule or manual dispatch
participant Caller as Repository sync caller
participant Pipeline as Pinned pm-github reusable workflow
participant Tracker as Package pm tracker
participant GitHub as GitHub repository
Scheduler->>Caller: Trigger workflow
Caller->>Pipeline: Call pm-github-sync.yml@a0880e9645097363824c9a560fed00fc286bbfc3
Pipeline->>Pipeline: Validate caller identity and access
Pipeline->>GitHub: Preview complete sync plan
Pipeline->>Tracker: Import with --gate
Pipeline->>Tracker: Verify repeat-import idempotency and privacy gate
Pipeline->>Pipeline: Check strict pm health
Pipeline->>GitHub: Push automation branch and open review PR
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
…under Changed The item title names the braces-chain removal, which pm-changelog's title fallback filed under Removed although the entry enables a workflow (CodeRabbit on pm-review#11). The changed tag is the author-controlled strong signal.
The reusable workflow at 2026.10.15 (tag commit 3015930) regenerates the caller's changelog, dispatches the caller's required CI on the sync branch (bot PRs park at action_required), and restricts the sync commit to an explicit scope (pm-github#122). The caller grants actions: write for that dispatch. ci.yml workflow_dispatch present: yes.
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
Owner
Author
|
@coderabbitai full review Please review head 1c580d9, including outside-diff findings. |
✅ Action performedFull review finished. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add the scheduled GitHub issue sync caller so this package's own tracker receives issue changes through the audited reusable pipeline. Before this change, the package had no active gated caller; after merge, its daily schedule is
17 3 * * *UTC, with manual dispatch available.The caller pins
unbraind/pm-github/.github/workflows/pm-github-sync.ymltoa0880e9645097363824c9a560fed00fc286bbfc3and installs the matching published package version2026.10.13. The called workflow validates the caller identity and access, previews the complete plan, imports with--gate, verifies repeat-import idempotency and the privacy gate, then checks strict pm health before committing, pushing an automation branch, and opening a review PR. Findings or unreadable scan inputs fail closed. It does not push main.Dependency manifests, merge-driver scripts, and thresholds are unchanged; the package has no direct fast-glob or jscpd4 devDependency. All dependencies: 0 before → 0 after. Production: 0 before → 0 after.
npm ls bracesis empty.Validation:
npm run changelog:fullandnpm run changelog:check: passed after final tracker writes.Exact release-gate commands:
Tracker: pm-brief-gp5o (proposed). The item remains in progress and its claim has been released for orchestrator verification.
Actions PR creation permission is already enabled. No workflow was dispatched, no PR was merged, and no bots were requested.
Packed consumer acceptance (
accept:packed) passed in both complete release checks.Summary by Sourcery
Enable gated synchronization of repository GitHub issues into the package’s pm tracker without changing dependencies or existing release configuration.
New Features:
CI:
Chores:
Summary by cubic
Adds the scheduled, SHA-pinned GitHub issue sync caller so this package's tracker receives issue changes through the audited gated pipeline. The workflow runs daily at 17:03 UTC (with manual dispatch), calls
pm-github's reusablepm-github-sync.ymlpinned to3015930b351a3d9c1c03ee4a26f8404025287df7(release 2026.10.15), and imports issues with--gateso findings fail closed.actions: writefor that dispatch.changedso pm-changelog files the entry under Changed rather than Removed.Written for commit 1c580d9. Summary will update on new commits.