Merged
Conversation
ba9b954 to
a1c652a
Compare
a1c652a to
0286f10
Compare
arsenyinfo
approved these changes
Apr 7, 2026
Collaborator
arsenyinfo
left a comment
There was a problem hiding this comment.
A single nitpick: imo a verb is a better choice for a script name, but overall LGTM!
lennartkats-db
approved these changes
Apr 7, 2026
5690f62 to
6ba6c28
Compare
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
Adds Codex UI metadata to every skill and refactors the build tooling so each skill directory is self-contained (assets copied in)
Main changes
Codex marketplace metadata (
skills/*/agents/openai.yaml):Each skill now ships an
agents/openai.yamlwithinterfacefields (display_name,short_description,icon_small/icon_large,brand_color,default_prompt). These are installed as files by the CLI and consumed by Codex for marketplace display.Self-contained skill directories with asset sync (
scripts/skills.py):Renamed
scripts/generate_manifest.pytoscripts/skills.pyand expanded it into a three-mode tool:synccopies shared assets (assets/databricks.{svg,png}) from repo root into each skill'sassets/dir (idempotent, preserves mtime viacopy2);generateruns sync then writesmanifest.jsonwith plain relative paths (no@root:prefix);validatechecks both asset sync state and manifest freshness, printing a fix command on failure.Skill rename:
databrickstodatabricks-core(manifest.json):The old
databricksskill entry (which mapped toskills/databricks/) is replaced bydatabricks-core(mapping toskills/databricks-core/). The orphanskills/databricks/directory is removed.Supporting infrastructure
assets/databricks.svg— Shared Databricks vector logo (source of truth for all skills).assets/databricks.png— Shared Databricks raster logo.skills/*/assets/databricks.{svg,png}— Per-skill copies generated byscripts/skills.py sync.Other changes
.github/workflows/validate-manifest.ymlupdated to referencescripts/skills.py validateand trigger onassets/**changes.CLAUDE.mdandREADME.mdupdated to document the newscripts/skills.pycommands (sync,generate,validate).Preview
Skills installer

In chat

All skills installed

Codex UI will have this skill openai/skills#307 that uses the CLI to install the rest
Documentation safety checklist
ALL PRIVILEGES, admin tokens, or broad scopes)