Repository navigation
Add data actions support to CLI - #66
Merged
Merged
Conversation
Member
|
@ranquild actually let's do |
ranquild
force-pushed
the
query-actions
branch
from
October 2, 2026 21:45
4038058 to
b80a79e
Compare
mb data-action list|get|create|update|archive|delete|execute over /api/action, query-type data actions only, a data-action skill, and e2e coverage on servers that accept data actions without a model. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
ranquild
force-pushed
the
query-actions
branch
from
October 2, 2026 21:46
b80a79e to
3db9997
Compare
ranquild
changed the base branch from
fix/transform-test-ee-path
to
release-65-integration-branch
October 2, 2026 21:46
Closed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
mb data-action list | get | create | update | archive | delete | executeover/api/action, plus adata-actionskill. The CLI calls Metabase actions data actions; the server API keeps theactionname.A data action is a parameterized native SQL write (
INSERT,UPDATEorDELETE) filed in a collection and run with parameter values.createacceptstype: "query"withdatabase_idand an MBQL 5 nativedataset_query, pre-flight-validated like card queries. The body is closed, somodel_idandtype: "implicit"are refused client-side. Model-bound and implicit data actions are legacy: they still list, get, run and archive, but the CLI never authors them.collection_idis read on servers that file data actions in collections (dataActionCollections, 65). On older servers it reads asnull.createis gated ondataActionsWithoutModel(65), the first version that accepts a data action without a model (metabase#83435). On older servers it fails before any request.executeposts parameter values keyed by parameter id and reportsrows-affected.data-actionskill (requires: [dataActionsWithoutModel]) covers the body, one raw-variable parameter per template tag, running, and the lifecycle. It states that data actions are off by default and need an admin to enable them on the database.native-sqlpoints writes to it.Testing
bun run checkpasses.Wire-level unit tests in
packages/client/src/resources/data-action.test.tscover every method, the 64-vs-65 collection read, and the client-side create refusal on 64.tests/e2e/data-action.e2e.test.tsruns on servers withdataActionsWithoutModel, which today means the head lanes. Each test turns actions on for the warehouse database through the API. The suite checks that create is refused while actions are off, then covers a model-less create filed in a collection, list and get, an execute that reportsrows-affected, a refused execute missing its required parameter, and update, archive and delete. On 58–64 lanes it checks that create is refused with a capability error before any request. The skills suite expectsdata-actionunderunavailableon a v58 profile. All e2e lanes, 58 through head on both editions, pass.🤖 Generated with Claude Code