Skip to content

Add Pi to databricks aitools install - #6198

Closed
lennartkats-db wants to merge 1 commit into
databricks:mainfrom
lennartkats-db:add-agent-pi
Closed

Add Pi to databricks aitools install#6198
lennartkats-db wants to merge 1 commit into
databricks:mainfrom
lennartkats-db:add-agent-pi

Conversation

@lennartkats-db

Copy link
Copy Markdown
Contributor

Changes

  • Register Pi as a skills-only agent (it reads SKILL.md but has no databricks plugin). install/update discover Pi via ~/.pi/agent (honoring PI_CODING_AGENT_DIR) and write Databricks skills into its skills directory, globally and project-scoped (.pi/skills).
  • Pi is the first non-plugin agent with project scope, so update --scope project now also picks up agents whose skills live in the project via DetectProjectInstalled (home-based config-dir detection alone misses them).
  • Derive the "Supported agents" / skills-only lists in the help text from the registry (SupportedNames/SkillsOnlyNames) so they can't drift as agents are added.
  • Add a PI telemetry agent type.

Why

Pi speaks the SKILL.md agent-skills standard, so it can use Databricks skills once they're installed: we just weren't installing for it.

Tests

  • Unit tests for Pi's config dir and override (TestPiConfigDir), its global/project skill paths and project detection (TestSkillAgentRegistryPaths, TestDetectProjectInstalled), the registry-derived help (TestSupportedNamesMatchesRegistry, TestSkillsOnlyNamesMatchesRegistry), and update --scope project picking up .pi/skills (TestUpdateProjectIncludesProjectSkillAgents).
  • task lint, go build ./..., and the aitools acceptance tests pass.
  • Verified end to end against a live model: after databricks aitools install, Pi loads the databricks-core skill in a real session.

Add Pi as a skills-only coding agent: install and update now discover Pi
via ~/.pi/agent (honoring PI_CODING_AGENT_DIR) and write Databricks agent
skills into its skills directory, both globally and project-scoped (.pi/skills).

This also lands the shared project-scope plumbing the Goose and Gemini agents
build on: SupportedNames/SkillsOnlyNames derive the help text from the registry,
and update --scope project picks up project-local skill agents via
DetectProjectInstalled.

Co-authored-by: Isaac
@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

An authorized user can trigger integration tests manually by following the instructions below:

Trigger:
go/deco-tests-run/cli

Inputs:

  • PR number: 6198
  • Commit SHA: 6e89d7fd31d2a20d43b0b33a7674f735a58adaf6

Checks will be approved automatically on success.

@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Approval status: pending

/cmd/aitools/ - needs approval

6 files changed
Suggested: @simonfaltum
Also eligible: @parthban-db, @renaudhartert-db, @fjakobs, @Shridhad, @atilafassina, @keugenek, @igrekun, @pkosiec, @MarioCadenas, @pffigueiredo, @ditadi, @calvarjorge, @hectorcast-db, @tanmay-db, @Divyansh-db, @tejaskochar-db, @mihaimitrea-db, @chrisst, @rauchy

/libs/aitools/ - needs approval

6 files changed
Suggested: @simonfaltum
Also eligible: @parthban-db, @renaudhartert-db, @fjakobs, @Shridhad, @atilafassina, @keugenek, @igrekun, @pkosiec, @MarioCadenas, @pffigueiredo, @ditadi, @calvarjorge, @hectorcast-db, @tanmay-db, @Divyansh-db, @tejaskochar-db, @mihaimitrea-db, @chrisst, @rauchy

/libs/telemetry/ - needs approval

Files: libs/telemetry/protos/aitools_install.go
Suggested: @simonfaltum
Also eligible: @parthban-db, @renaudhartert-db, @hectorcast-db, @tanmay-db, @Divyansh-db, @tejaskochar-db, @mihaimitrea-db, @chrisst, @rauchy

General files (require maintainer)

Files: .nextchanges/cli/aitools-pi.md
Based on git history:

  • @simonfaltum -- recent work in cmd/aitools/, libs/aitools/agents/, libs/aitools/installer/

Any maintainer (@andrewnester, @anton-107, @denik, @pietern, @shreyas-goenka, @simonfaltum, @renaudhartert-db, @janniklasrose) can approve all areas.
See OWNERS for ownership rules.

@lennartkats-db

Copy link
Copy Markdown
Contributor Author

Closing in favor of #6199; PR should not be on a fork.

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