Repository navigation
chore(repo-maintain): harden Dependabot cooldown/Node 24 rules and sync docs - #31
Merged
Merged
Conversation
…arity Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
🟢 Approval recommended
No active review issues are reported, and the documented Dependabot, Node compatibility, authentication, and license-notice updates align with the reviewed changes.
No open issues 🎉
What changed in this PR
This maintenance PR hardens Dependabot release timing and Node 24 compatibility while synchronizing documentation with the implemented authentication and license-notice behavior.
Changes:
- Adds seven-day cooldowns for npm and GitHub Actions Dependabot updates.
- Prevents semver-major
@types/nodeupdates while the Action targets Node 24. - Clarifies workflow-token authentication and narrows the license-notice documentation guarantee.
| File | Description |
|---|---|
| README.md | Clarifies that copilot-token supports workflow-token authentication for explicit runs. |
| CONTRIBUTING.md | Removes the unsupported broad guarantee about retaining full license sections from package READMEs. |
| .github/dependabot.yml | Adds seven-day cooldowns and ignores semver-major @types/node updates. |
Review effort: Lite (dynamic)
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
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.
Summary
Consolidated
repo-maintainpass hardening.github/dependabot.ymlrelease-age cooldown and Node 24 runtime parity rules, alongside verified documentation clarifications folded from #25.Phase Breakdown
baaa579cooldown: { default-days: 7 }on bothnpmandgithub-actionsecosystems in.github/dependabot.yml, and ignore@types/nodeversion-update:semver-majorupdates while the Action targetsnode24(engines.node: >=24 <25).0465ebfCONTRIBUTING.md(matchingtooling/license-notices.mjs's MIT-heading check) and clarify inREADME.mdthatcopilot-tokensupports workflow-token authentication in explicitrunmode alongsidecredential-env: COPILOT_GITHUB_TOKEN(matchingsrc/native.ts).Open PR Disposition
CONTRIBUTING.mdandREADME.mdclarifications verified againsttooling/license-notices.mjsandsrc/native.tsand folded with co-author attribution).oxlint1.92.0->1.93.0remains held until it clears the 7-day release-age window).Repo Health Score:
97/100 (A+)->97/100 (A+)Verification
pnpm check(tsc --noEmit,oxlint --deny-warnings src tests,vitest run --coverage180/180,node tooling/build.mjs --check,node --test tooling/packaged.test.mjs6/6)pnpm exec vally lint --strict examples/copilot-smoke --eval-spec=examples/copilot-smokepnpm exec vally lint --strict --eval-spec=tests/fixtures