Skip to content

[4/5] Aitools: list command, flat structure, --skills/--agents/--include-experimental flags#4813

Open
simonfaltum wants to merge 3 commits intosimonfaltum/aitools-pr3-lifecyclefrom
simonfaltum/aitools-pr4-restructure
Open

[4/5] Aitools: list command, flat structure, --skills/--agents/--include-experimental flags#4813
simonfaltum wants to merge 3 commits intosimonfaltum/aitools-pr3-lifecyclefrom
simonfaltum/aitools-pr4-restructure

Conversation

@simonfaltum
Copy link
Member

@simonfaltum simonfaltum commented Mar 22, 2026

PR Stack

  1. [1/5] State + release discovery + directory rename ([1/5] Aitools: state tracking, manifest source, directory rename #4810)
  2. [2/5] Install writes state + interactive agent selection ([2/5] Aitools: install writes state, interactive agent selection, idempotent install #4811)
  3. [3/5] Update + uninstall + version commands ([3/5] Aitools: update, uninstall, and version commands #4812)
  4. [4/5] List improvements + command restructuring + flags (this PR)
  5. [5/5] Project scope (--project/--global) ([5/5] Aitools: project scope (--project/--global) #4814)

Manifest v2 PR: databricks/databricks-agent-skills#35

Base: simonfaltum/aitools-pr3-lifecycle (PR 3)

Why

Commands are nested under skills which burns namespace for future component types (hooks, etc.). There's no way to install specific skills, target specific agents from the CLI, or see installed status in the list output.

Changes

New list command: Table output with skill names, versions, installed status, [experimental] tags. Sorted alphabetically. Uses tabwriter for alignment.

Flat command structure: aitools install/update/uninstall/list/version at the top level. Hidden skills install and skills list aliases for backward compat.

Flags:

  • --skills (string, comma-separated) on install, update, uninstall: operate on specific skills
  • --skills (bool) on list, version: show detailed skills view
  • --agents (string, comma-separated) on install: target specific agents, validates against registry, skips interactive prompt
  • --include-experimental on install: include experimental skills

Selective uninstall: --skills on uninstall removes only named skills, preserves state file with remaining.

Test plan

  • install --skills, --agents, --include-experimental flags work
  • --agents validates against registry, errors on unknown
  • uninstall --skills selective removal, state preserved
  • update --skills selective update
  • list shows detailed table
  • skills install and skills list hidden aliases work (Execute path tested)
  • cmd/apps/init.go integration preserved
  • All lint checks pass

Add --skills, --agents, and --include-experimental flags to install command.
Add --skills flag to update, uninstall, and version commands. Create new
list command with detailed table output showing available/installed skills.
Make skills subcommand hidden for backward compat while promoting flat
command structure. Add selective uninstall support via UninstallOptions.

Co-authored-by: Isaac
…ning

- Always call SetArgs in skills install backward-compat alias (fixes
  Execute path inheriting parent args)
- Add cobra.MaximumNArgs(1) to reject extra positional args
- Remove dead installer.ListSkills function (replaced by list command)
- Restore yellow color for "No agents detected" warning in install.go
- Add Execute-path tests for skills install alias

Co-authored-by: Isaac
@simonfaltum simonfaltum force-pushed the simonfaltum/aitools-pr3-lifecycle branch from 14680ad to 2dd4a43 Compare March 22, 2026 22:00
@simonfaltum simonfaltum force-pushed the simonfaltum/aitools-pr4-restructure branch from 1243dc4 to ccf24e9 Compare March 22, 2026 22:00
@simonfaltum simonfaltum marked this pull request as ready for review March 22, 2026 22:01
@simonfaltum simonfaltum requested review from a team and lennartkats-db as code owners March 22, 2026 22:01
@github-actions
Copy link

Suggested reviewers

Based on git history of the changed files, these people are best suited to review:

  • @arsenyinfo -- recent work in experimental/aitools/cmd/, experimental/aitools/lib/installer/

Confidence: high

Eligible reviewers

Based on CODEOWNERS, these people or teams could also review:

@databricks/eng-apps-devex, @lennartkats-db

Suggestions based on git history of 12 changed files (5 scored). See CODEOWNERS for path-specific ownership rules.

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.

1 participant