Skip to content

docs: track aitools rename to top-level command#60

Merged
pkosiec merged 2 commits into
mainfrom
jbroadhead/aitools-cmd-rename
May 26, 2026
Merged

docs: track aitools rename to top-level command#60
pkosiec merged 2 commits into
mainfrom
jbroadhead/aitools-cmd-rename

Conversation

@jamesbroadhead
Copy link
Copy Markdown
Contributor

@jamesbroadhead jamesbroadhead commented Apr 29, 2026

Summary

Mirrors databricks/cli#4917 which promotes the aitools skills-management surface from databricks experimental aitools to top-level databricks aitools. Updates 3 install-command references so user-facing copy points at the canonical path:

  • README.md
  • skills/databricks-jobs/SKILL.md
  • skills/databricks-pipelines/SKILL.md

Why now

Purely cosmetic — there is no deprecation warning to avoid. The CLI keeps databricks experimental aitools install working as a silent backward-compat alias (it forwards to the same code, prints nothing). This PR is just keeping the docs aligned with the canonical command path so users meet the right name on first read.

Intentionally left alone

The 37 databricks experimental aitools tools … references in skills/databricks-core/, skills/databricks-apps/, and elsewhere are not updated. The tools subtree (query, discover-schema, get-default-warehouse, statement) keeps its "no stability guarantees" disclaimer in #4917 and stays under experimental aitools tools. The docs continue to point at the live, canonical path.

Compatibility

Lands in any order relative to #4917 — the old paths work either way, with or without the CLI change.

Test plan

  • README setup steps point at the new command
  • databricks-jobs and databricks-pipelines skill markdown points at the new command
  • No accidental rewrites of tools-subtree references

This pull request was AI-assisted by Isaac.

@pkosiec pkosiec force-pushed the jbroadhead/aitools-cmd-rename branch from 8fe4978 to 1bdfc0c Compare May 21, 2026 10:13
Copy link
Copy Markdown
Member

@pkosiec pkosiec left a comment

Choose a reason for hiding this comment

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

LGTM

Databricks CLI 1.0 has been released: https://github.com/databricks/cli/releases/tag/v1.0.0

I've just verified that the aitools command group is no longer under experimental, so we can merge this 👍 I just pushed the prerequisite versions update for the changed skills.

@pkosiec pkosiec enabled auto-merge (squash) May 21, 2026 10:14
@pkosiec
Copy link
Copy Markdown
Member

pkosiec commented May 21, 2026

@lennartkats-db @simonfaltum please take a look 🙏 Thank you!

@pkosiec pkosiec force-pushed the jbroadhead/aitools-cmd-rename branch from 1bdfc0c to 0f77c39 Compare May 25, 2026 16:03
jamesbroadhead and others added 2 commits May 25, 2026 19:28
…s install'

Mirrors the CLI change in databricks/cli#4917 which promotes the
aitools skills-management surface (install/update/uninstall/list/
version/skills) from `databricks experimental aitools` to top-level
`databricks aitools`.

Updates 3 install-command references:
- README.md
- skills/databricks-jobs/SKILL.md
- skills/databricks-pipelines/SKILL.md

The 37 `databricks experimental aitools tools …` references in
skills/databricks-core/, skills/databricks-apps/, and elsewhere are
intentionally left as-is — that subtree (query, discover-schema,
get-default-warehouse, statement) still has "no stability guarantees"
per its own help text and is NOT being promoted to the top level in
#4917. It stays under `experimental aitools tools`.

Co-authored-by: Isaac
The non-experimental `databricks aitools install` command requires
CLI >= v1.0.0. Bump compatibility and skill versions accordingly.

Co-authored-by: Isaac
@pkosiec pkosiec force-pushed the jbroadhead/aitools-cmd-rename branch from 0f77c39 to d639b22 Compare May 25, 2026 17:28
@pkosiec pkosiec merged commit 33641d9 into main May 26, 2026
1 check passed
@pkosiec pkosiec deleted the jbroadhead/aitools-cmd-rename branch May 26, 2026 07:53
pkosiec added a commit that referenced this pull request May 26, 2026
## Summary

Addresses 4 friction clusters identified in the CAO (Coding Agent
Optimization) benchmark run 2 (May 21-22, 2026; Opus 4.7 + Codex 5.5, 25
tasks).

### Changes

**P2: OAuth auth guidance (Cluster A -- 16/25 tasks affected)**
- `databricks-core/SKILL.md`: Add troubleshooting row for `"OAuth Token
not supported for current auth type pat"` directing to `databricks auth
login`
- `databricks-apps/SKILL.md`: Note that `databricks apps logs` requires
OAuth in Post-Deploy Verification

**P4: Model Serving endpoint readiness (Cluster F -- regression on both
models)**
- `databricks-model-serving/SKILL.md`: Add "Endpoint Readiness"
subsection with state transitions, provisioning times, and
poll-before-query guidance. Enhance troubleshooting for `NOT_READY`
state.

**P5: Off-platform anti-pattern (Drizzle task -- Codex scores 35)**
- `databricks-lakebase/references/off-platform.md`: Add callout that
off-platform apps are NOT Databricks Apps. Add "Running Locally" section
with standard Node.js commands.
- `databricks-lakebase/SKILL.md`: Strengthen off-platform reference
description.

**P6: Lakehouse Sync UI-only limitation (Cluster G)**
- `databricks-lakebase/references/lakehouse-sync.md`: Strengthen
warning: "Do NOT attempt to automate this step."
- `databricks-lakebase/SKILL.md`: Add "(UI-only)" annotation to
Lakehouse Sync reference entry.

### Context

The benchmark also identified these clusters that are addressed
elsewhere:
- **Cluster B/D** (npm proxy ETIMEDOUT / apps init failure): Fixed in
newer AppKit version (wrong `package-lock.json` entry)
- **Cluster C** (CLI `experimental aitools install` deprecation):
Addressed in PR #60

### Test plan

- [x] `python3 scripts/skills.py validate` passes

This pull request and its description were written by Isaac.
jamesbroadhead added a commit that referenced this pull request May 27, 2026
Follow-up to #60, which only touched README/manifest and the jobs +
pipelines SKILL.md. This sweeps the remaining `databricks experimental
aitools` references across the core/apps skills, the experimental
skills, and reference docs.

The old paths still work as deprecated aliases (see databricks/cli#4917),
but new readers should be pointed at the supported command.

Co-authored-by: Isaac
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.

4 participants