Skip to content

skills: promote databricks-vector-search to stable#87

Open
jamesbroadhead wants to merge 7 commits into
mainfrom
jb/promote-vector-search-to-stable
Open

skills: promote databricks-vector-search to stable#87
jamesbroadhead wants to merge 7 commits into
mainfrom
jb/promote-vector-search-to-stable

Conversation

@jamesbroadhead
Copy link
Copy Markdown
Contributor

Summary

Per Lennart's audit on #73, item #10: Vector Search is no longer considered experimental in Genie Code. Promote experimental/databricks-vector-search/skills/databricks-vector-search/.

Changes

  • git mv experimental/databricks-vector-search skills/databricks-vector-search.
  • Move the 4 top-level reference files into references/ to match the stable-skills layout convention (apps, lakebase, pipelines).
  • SKILL.md frontmatter: add parent: databricks-core and metadata.version: "0.1.0". Body: add the standard "FIRST: Use the parent databricks-core skill" prelude.
  • Rewrite 10 path references in SKILL.md for the new references/<file>.md locations.
  • scripts/skills.py: add databricks-vector-search to SKILL_METADATA.
  • Root README.md "Available Skills" list: add databricks-vector-search.
  • experimental/README.md: remove from the experimental skill list.

Manifest regenerated; python3 scripts/skills.py validate passes.

Cross-repo

  • a-d-k tombstone PR databricks-solutions/ai-dev-kit#546 currently redirects databricks-vector-search with --experimental. I'll drop the flag in that PR to match this promotion.
  • Stacking concern: this PR has zero overlap with the open references-restructure PR #86 since the vector-search files move in this PR too (commit includes the references/ move). Whichever lands first, the other will rebase cleanly with no conflicts in the vector-search subtree.

Test plan

  • python3 scripts/skills.py generate clean.
  • python3 scripts/skills.py validate passes (Everything is up to date.).
  • CI green.
  • Reviewer confirmation that stable classification is correct (@simonfaltum flagged this).

This pull request and its description were written by Claude.

Per Lennart's audit on #73: vector-search is no longer considered
experimental in the Genie Code world. Promote it from
`experimental/databricks-vector-search/` to `skills/databricks-vector-search/`.

Changes:
- `git mv experimental/databricks-vector-search skills/databricks-vector-search`.
- Move the 4 top-level reference files into `references/` for layout
  consistency with other stable skills (apps, lakebase, pipelines).
- SKILL.md: add `parent: databricks-core`, `metadata.version: "0.1.0"`,
  and the standard "FIRST: Use the parent databricks-core skill" prelude.
  Rewrite 10 path references for the new `references/<file>.md` locations.
- scripts/skills.py: add `databricks-vector-search` to `SKILL_METADATA`
  with a stable description.
- Update root README "Available Skills" list to include it; remove from
  experimental/README list.

Manifest regenerated. `python3 scripts/skills.py validate` passes.

Once this lands, the matching change in the a-d-k tombstone PR
(databricks-solutions/ai-dev-kit#546) is to drop the `--experimental`
flag from the vector-search tombstone's install command. I'll update
that PR directly.

Co-authored-by: Isaac
Copy link
Copy Markdown
Collaborator

@dustinvannoy-db dustinvannoy-db left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm good with promoting to stable

…rch-to-stable

# Conflicts:
#	experimental/databricks-vector-search/end-to-end-rag.md
#	experimental/databricks-vector-search/index-types.md
#	experimental/databricks-vector-search/references/end-to-end-rag.md
#	experimental/databricks-vector-search/references/index-types.md
#	experimental/databricks-vector-search/references/search-modes.md
#	experimental/databricks-vector-search/references/troubleshooting-and-operations.md
#	experimental/databricks-vector-search/search-modes.md
#	experimental/databricks-vector-search/troubleshooting-and-operations.md
#	manifest.json
#	skills/databricks-vector-search/references/end-to-end-rag.md
#	skills/databricks-vector-search/references/index-types.md
#	skills/databricks-vector-search/references/search-modes.md
#	skills/databricks-vector-search/references/troubleshooting-and-operations.md
jamesbroadhead added a commit that referenced this pull request May 27, 2026
…tmatter

Adds three frontmatter fields to 17 experimental SKILL.md files that stable
skills already carry but the imported a-d-k snapshot does not:

- `compatibility: Requires databricks CLI (>= v0.294.0)`
- `metadata.version: "0.1.0"` (was the `0.0.1` script floor)
- `parent: databricks-core`

`experimental/databricks-vector-search` is intentionally skipped to avoid
conflict with #87 (promotion to stable, which adds the same fields as part of
the move).

Manifest deltas: every experimental skill's `version` flips from `0.0.1`
(scripts/skills.py fallback floor) to `0.1.0`. No other consumer-visible
fields move — `compatibility` and `parent` are SKILL.md-only and not surfaced
in manifest.json today.

## Test plan

- `python3 scripts/skills.py generate` clean
- `python3 scripts/skills.py validate` passes (`Everything is up to date.`)

This PR was prepared by Claude.
Frontmatter description still carried the old verbose text while the
manifest.json and scripts/skills.py override carried the concise version.
That meant tools reading SKILL.md saw one description; tools reading the
manifest saw another. Update the frontmatter to match.

Co-authored-by: Isaac
…rch-to-stable

# Conflicts:
#	scripts/skills.py
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.

2 participants