Skip to content

ci: complete daily bot update PRs and automatic validation - #546

Merged
Sunrisepeak merged 3 commits into
mainfrom
fix/daily-bot-update-loop
Oct 19, 2026
Merged

Sunrisepeak merged 3 commits into
mainfrom
fix/daily-bot-update-loop

Conversation

@Sunrisepeak

@Sunrisepeak Sunrisepeak commented Oct 19, 2026 •

Copy link
Copy Markdown
Member

Daily update scanning previously required a local CLI to create a PR, so the scheduled bot workflow was incomplete. Run the scan every day at 02:17 UTC and consume its exact plan inside a default-branch proposal job using GITHUB_TOKEN. The bot opens at most three new PRs per run and explicitly dispatches candidate, consumer, site and openkal checks on their branches. Existing PRs resume missing dispatches without duplicates.

Consumer/openkal dispatches verify an open same-repository bot PR, exact head SHA, default base and single descriptor before selecting affected members. Ordinary manual consumer runs remain full sweeps; PR/branch validation never creates more PRs. Only the proposal job has write permissions, and no persistent secret, automatic approval or merge is added. Organization/repository PR creation is now enabled by the maintainer; default workflow permissions remain read-only.

Document eligibility and append-only behavior in English/Chinese and the package skill. Ordinary package PRs must add eligible update declarations and leave root READMEs/site configuration alone. Simplify both root READMEs and link the detailed guides instead of appending package examples.

Validation: 8 local integration tests, Actionlint on all three changed workflows, declaration coverage for all 82 direct GitHub descriptors, and historical preservation for all 254 descriptors passed. Real bot PR creation/CI dispatch will be verified after this workflow reaches main.

Self-review: explicit read-only PR permissions are granted only where identity validation needs them. Dispatches validate the PR head and reject human/fork/closed/wrong-base/workflow-changing PRs. Duplicate detection also respects manual PRs already editing a package, such as httplib #542; their ordinary PR CI is retained. The updater is pinned to the squash-merged #2 commit 6c0bee2941e36132bc2ac3d1f7edbef8b5790320 (61 tests and all seven CI checks passed). Both root READMEs are 37 lines and their local links pass validation. Historical preservation still covers all 254 descriptors.

@Sunrisepeak
Sunrisepeak merged commit f20ab0f into main Oct 19, 2026
57 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant