Skip to content

ci: fix Grok mirror bot push and release-please config - #8

Merged
prathamesh-sonpatki merged 1 commit into
masterfrom
ci/grok-bot-and-release-fixes
Aug 24, 2026
Merged

ci: fix Grok mirror bot push and release-please config#8
prathamesh-sonpatki merged 1 commit into
masterfrom
ci/grok-bot-and-release-fixes

Conversation

@prathamesh-sonpatki

@prathamesh-sonpatki prathamesh-sonpatki commented Aug 24, 2026

Copy link
Copy Markdown
Member

Summary

Two master-push failures from the repo-as-hub landing (#6):

  1. Grok mirror refresh deniedgithub-actions[bot] got 403 pushing plugins/last9/skills because the workflow declares contents: read. Job-scoped contents: write on sync-parity fixes it. The bot's pending change (dropping an orphaned mirror entry) lands on the next run.
  2. release-please crashextra-files entries pointing at root manifests via plugins/last9/../.. are rejected as illegal path traversal. Entries dropped: those two manifest versions become static (marketplace installs resolve via git SHA, so this is display-only metadata).

Test plan

  • Config JSON validated; workflow YAML unchanged structurally except permissions block.
  • Post-merge: watch "Plugin skill sync parity" on master — expect a chore(grok): bot commit removing the orphaned mirror entry, followed by a green re-run.

View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

- The Grok mirror refresh commits to master, so its job needs
  contents: write (workflow default is read; the refresh pushed as
  github-actions[bot] and was denied with 403).
- release-please rejects extra-files paths that escape the package dir
  (plugins/last9/../../.claude-plugin/...). Root plugin manifest versions
  become manually managed — marketplace installs key off git SHAs, not
  these fields.
@prathamesh-sonpatki
prathamesh-sonpatki merged commit be3d2c9 into master Aug 24, 2026
5 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