Skip to content

fix: regenerate catalog with the built-in GitHub token - #117

Draft
chocholous wants to merge 1 commit into
mainfrom
codex/catalog-github-token
Draft

chocholous wants to merge 1 commit into
mainfrom
codex/catalog-github-token

Conversation

@chocholous

Copy link
Copy Markdown
Collaborator

Catalog regeneration currently fails after merges because it requires an unconfigured GitHub App. Use the built-in GITHUB_TOKEN with contents:write only in the regenerate job; other jobs retain contents:read. Remove App credentials and App-specific identity lookup, and use the standard GitHub Actions bot identity. Keep strict validation, serialization and push retries. Include the workflow itself in the push path filter so merging this fix triggers regeneration, and explicitly target main when pushing.

Verified in apify/awesome-skills on an isolated branch with the same generation/authentication steps and branch-specific targets:

  • First run regenerated and pushed exactly marketplace.json, agents/AGENTS.md and the README skills table: 18 stale entries restored to 20; all three remote files match the trusted generated baseline byte for byte.
  • Rerunning checked out the latest branch, reported a fixed point, and created no additional commit. The bot push did not create a recursive workflow run.
  • Main remained unchanged throughout. No repository settings, App installation or PAT were needed.

Functional test, two successful attempts.

The main-branch push has not been exercised by this test; it becomes the production verification after this PR is merged. Current main reports no branch protection or applicable rulesets.

@chocholous chocholous added the maintainer Infra PR approved by a maintainer — lifts the skill/infra split in CI label Sep 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintainer Infra PR approved by a maintainer — lifts the skill/infra split in CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants