Skip to content

Commit 4f5748b

Browse files
authored
Remove experimental/aitools (#4217)
Removes the experimental/aitools directory and all references to it throughout the codebase. Changes: - Delete experimental/aitools/ directory (35 files) - Remove aitools from TEST_PACKAGES and test-exp-aitools target from Makefile - Remove test-exp-aitools CI job from GitHub workflow - Remove aitools command registration from cmd/experimental - Update CODEOWNERS, testmask targets and tests All tests pass and the code builds successfully.
1 parent 2f9022d commit 4f5748b

42 files changed

Lines changed: 4 additions & 2916 deletions

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.github/CODEOWNERS

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,5 +7,4 @@
77
/cmd/workspace/apps/ @databricks/eng-app-devex
88
/libs/apps/ @databricks/eng-app-devex
99
/acceptance/apps/ @databricks/eng-app-devex
10-
/experimental/aitools/ @databricks/eng-app-devex @lennartkats-db
1110
/experimental/apps-mcp/ @databricks/eng-app-devex @lennartkats-db

.github/workflows/push.yml

Lines changed: 0 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -145,37 +145,6 @@ jobs:
145145
- name: Analyze slow tests
146146
run: make slowest
147147

148-
test-exp-aitools:
149-
needs:
150-
- cleanups
151-
- testmask
152-
153-
# Only run if the target is in the list of targets from testmask
154-
if: ${{ contains(fromJSON(needs.testmask.outputs.targets), 'test-exp-aitools') }}
155-
name: "make test-exp-aitools"
156-
runs-on: ${{ matrix.os }}
157-
158-
strategy:
159-
fail-fast: false
160-
matrix:
161-
os:
162-
- macos-latest
163-
- ubuntu-latest
164-
- windows-latest
165-
166-
steps:
167-
- name: Checkout repository and submodules
168-
uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
169-
170-
- name: Setup build environment
171-
uses: ./.github/actions/setup-build-environment
172-
with:
173-
cache-key: test-exp-aitools
174-
175-
- name: Run tests
176-
run: |
177-
make test-exp-aitools
178-
179148
test-exp-apps-mcp:
180149
needs:
181150
- cleanups
@@ -282,7 +251,6 @@ jobs:
282251
test-result:
283252
needs:
284253
- test
285-
- test-exp-aitools
286254
- test-exp-apps-mcp
287255
- test-exp-ssh
288256
- test-pipelines

Makefile

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
default: checks fmt lint
22

33
# Default packages to test (all)
4-
TEST_PACKAGES = ./acceptance/internal ./libs/... ./internal/... ./cmd/... ./bundle/... ./experimental/aitools/... ./experimental/ssh/... .
4+
TEST_PACKAGES = ./acceptance/internal ./libs/... ./internal/... ./cmd/... ./bundle/... ./experimental/ssh/... .
55

66
# Default acceptance test filter (all)
77
ACCEPTANCE_TEST_FILTER = ""
@@ -173,9 +173,6 @@ generate:
173173

174174
.PHONY: lint lintfull tidy lintcheck fmt fmtfull test test-unit test-acc test-slow test-slow-unit test-slow-acc cover showcover build snapshot snapshot-release schema integration integration-short acc-cover acc-showcover docs ws wsfix links checks test-update test-update-templates generate-out-test-toml test-update-aws test-update-all generate-validation
175175

176-
test-exp-aitools:
177-
make test TEST_PACKAGES="./experimental/aitools/..." ACCEPTANCE_TEST_FILTER="TestAccept/idontexistyet/aitools"
178-
179176
test-exp-apps-mcp:
180177
make test TEST_PACKAGES="./experimental/apps-mcp/..." ACCEPTANCE_TEST_FILTER="TestAccept/idontexistyet/apps-mcp"
181178

cmd/experimental/experimental.go

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
package experimental
22

33
import (
4-
"github.com/databricks/cli/experimental/aitools"
54
mcp "github.com/databricks/cli/experimental/apps-mcp/cmd"
65
"github.com/spf13/cobra"
76
)
@@ -21,7 +20,6 @@ These commands provide early access to new features that are still under
2120
development. They may change or be removed in future versions without notice.`,
2221
}
2322

24-
cmd.AddCommand(aitools.New())
2523
cmd.AddCommand(mcp.NewMcpCmd())
2624

2725
return cmd

experimental/aitools/DESIGN.md

Lines changed: 0 additions & 227 deletions
This file was deleted.

experimental/aitools/agents/claude.go

Lines changed: 0 additions & 44 deletions
This file was deleted.

experimental/aitools/agents/cursor.go

Lines changed: 0 additions & 96 deletions
This file was deleted.

experimental/aitools/agents/custom.go

Lines changed: 0 additions & 36 deletions
This file was deleted.

experimental/aitools/aitools.go

Lines changed: 0 additions & 34 deletions
This file was deleted.

experimental/aitools/auth/check.go

Lines changed: 0 additions & 67 deletions
This file was deleted.

0 commit comments

Comments
 (0)