docs(skill): align SKILL and WORKFLOWS with CLI (flags, sandbox, datasets)#54
Merged
eddietejeda merged 3 commits intomainfrom Apr 24, 2026
Merged
docs(skill): align SKILL and WORKFLOWS with CLI (flags, sandbox, datasets)#54eddietejeda merged 3 commits intomainfrom
eddietejeda merged 3 commits intomainfrom
Conversation
- Document -o/--output instead of nonexistent --format - Add datasets create --upload-id and --format - Document HOTDATA_WORKSPACE lock, queries without -w, jobs list pagination - Mention global --debug; clarify workspaces list default marker
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
![claude[bot]](https://avatars.githubusercontent.com/in/1236702?s=60&v=4){kind=small}
- SKILL: document sandbox run vs active sandbox, datasets schema (datasets.<sandbox_id> vs main), full_name/FULL NAME, list scope, sandbox context for queries, PostgreSQL quoted identifiers - SKILL: use long CLI flags (--workspace-id, --output, --connection-id) - WORKFLOWS: new Sandboxes and datasets section; align Chain, Model refresh, History, Indexes examples; cross-cutting pointers
eddietejeda
changed the title
| @@ -60,21 +68,21 @@ Full step-by-step procedures: [references/WORKFLOWS.md](references/WORKFLOWS.md) | |||
|
|
|||
| ### List Workspaces | |||
| ``` | |||
Contributor
There was a problem hiding this comment.
nit: This Chain summary bullet still says datasets.main.<table>, but the PR adds content below clarifying the schema is not always main—it can be a sandbox id (e.g. datasets.s_ufmblmvq.tac_csat). Suggest updating to datasets.<schema>.<table> to be consistent with the rest of the PR. (not blocking)
Align the Chain workflow bullet and Querying datasets section on the qualified name pattern; keep full_name guidance and workspace example.
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
1 participant
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.
Summary
Updates the bundled Hotdata agent skill and workflow reference so they match current CLI behavior and common pitfalls.
SKILL.md
--workspace-id,--output,--connection-id).--outputvs nonexistent--format,datasets create --upload-id/--format, workspace lock (HOTDATA_WORKSPACE,sandbox runancestor),querieswithout--workspace-id, jobs list pagination,--debug.sandbox runalways creates a new sandbox when no id is given; use direct commands orsandbox <id> runwhen reusing an active sandbox.datasets.<sandbox_id>.<table>vsdatasets.main;full_name/FULL NAME;datasets listis unfiltered; sandbox context for queries; PostgreSQL double-quoted identifiers for mixed-case columns.WORKFLOWS.md
PR history
Earlier commits on this branch aligned SKILL output flags and workspace/query notes; this commit adds sandbox/dataset guidance and WORKFLOWS parity.