docs(auto-derive): document cross-repo dispatch trigger + fix dead G6 link#144
Conversation
… link Two updates: * Fix the dead `[G6 matrix](#)` empty-anchor link (rabbit had flagged in openemr#135 review; the original fix targeted README:6 not README:19; replaced with prose since the matrix is in the design doc linked above, not anywhere in this repo). * Rewrite the "Daily reconcile + auto-PR" section as "Triggers + reconcile + auto-PR" — enumerates all 3 fire triggers (daily cron, release-targets-changed repository_dispatch added by openemr#12657, manual workflow_dispatch). Existing reconciliation behavior moved under the "in all three trigger paths" framing. Trimmed the redundant workflow_dispatch paragraph that now duplicates the trigger enumeration.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
📝 WalkthroughWalkthroughUpdates ChangesREADME documentation update
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@tools/auto-derive/README.md`:
- Around line 145-149: The README wording for the workflow trigger section
conflates dry-run with the reconcile path, since `workflow_dispatch` can select
`dry-run` and that mode does not create or update a PR. Update the section
around the `workflow_dispatch` description and the “reconcile job” wording to
explicitly scope the behavior to reconcile mode, or split out a separate dry-run
description so the `reconcile` and `dry-run` paths are clearly distinguished.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro Plus
Run ID: a257de96-f5f5-459a-b81a-fe7a3eb49607
📒 Files selected for processing (1)
tools/auto-derive/README.md
Summary
Two README updates following the G6 work that landed:
Fix dead `G6 matrix` link. Original rabbit comment from PR feat(auto-derive): scaffold ip_map + demoLibrary derivation bot (dry-run) #135 review (id=3485751948) flagged this; the addressed-in-commit fix targeted README:6 but missed this second occurrence at README:19. Replaced with prose since the matrix lives in the design doc linked above, not anywhere in this repo.
Document the cross-repo dispatch trigger. PR feat(auto-derive): atomic flip — retire bump-tag.yml + tools/release/ PHP toolchain #141 added the `repository_dispatch: types: [release-targets-changed]` listener; feat(release): add release-targets-changed dispatch event + workflow openemr#12657 added the openemr-side dispatcher that fires it on push to `.github/release-targets.yml`. README's "Daily reconcile + auto-PR" section is rewritten as "Triggers + reconcile + auto-PR" — enumerates all 3 fire triggers (daily cron, release-targets-changed dispatch, manual workflow_dispatch) with the existing reconciliation behavior under a unified "in all three trigger paths" framing.
Test plan
🤖 Generated with Claude Code
Summary by CodeRabbit