Skip to content

ci: auto-generate skills on toolbox version updates#177

Open
twishabansal wants to merge 4 commits into
mainfrom
ci/auto-generate-skills
Open

ci: auto-generate skills on toolbox version updates#177
twishabansal wants to merge 4 commits into
mainfrom
ci/auto-generate-skills

Conversation

@twishabansal

@twishabansal twishabansal commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Adds a GitHub Actions workflow (.github/workflows/skills-generate.yml) that regenerates the cloud-sql-postgres-* skills when a PR bumps toolbox_version.txt, and commits them back to the PR branch via the built-in GITHUB_TOKEN.
  • Adds .github/scripts/generate_skills.sh, which validates upstream toolsets against the SKILLS array, then regenerates each skill offline (no DB; requires toolbox ≥ 1.5.0).
  • Fixes the toolset-validation parser for the current kind: toolset config format (toolbox 1.2.0+).

Test plan

Adds a Cloud Build pipeline and script to regenerate skills when toolbox_version.txt changes. Skill generation reads only the upstream prebuilt config, so no database connection is required.
Replaces the Cloud Build pipeline (which needed SSH keys and Secret Manager tokens) with a GitHub Actions workflow. On PRs that bump toolbox_version.txt, it regenerates skills and commits them back to the PR branch using the built-in GITHUB_TOKEN. Moves the generator script under .github/scripts/.
@twishabansal twishabansal changed the title ci: auto generate skills on toolbox version update ci: auto-generate skills on toolbox version bump Jun 19, 2026
@twishabansal twishabansal added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Jun 19, 2026
twishabansal and others added 2 commits June 19, 2026 12:29
The upstream prebuilt config moved from a top-level toolsets: block to per-toolset YAML documents (kind: toolset / name:) in toolbox v1.2.0. The old awk parser found zero toolsets on v1.2.0+, so the validation passed vacuously. Parse the new format so unsupported upstream toolsets are detected.
@twishabansal twishabansal changed the title ci: auto-generate skills on toolbox version bump ci: auto-generate skills on toolbox version updates Jun 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not merge Indicates a pull request not ready for merge, due to either quality or timing.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants