Repository navigation
feat(ingest): rename the catalog verb connectors → datasources - #216
Merged
Merged
Conversation
'hotdata ingest datasources' now browses the catalog of available datasource types; 'connectors' stays as a hidden alias. Help, error hints, README, and the skill follow.
![claude[bot]](https://avatars.githubusercontent.com/in/1236702?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Clean, well-scoped rename of the catalog verb connectors → datasources with the old name retained as a hidden alias. Enum variant, dispatch arm, alias attribute, and all docs (README, SKILL.md, doc comments, error hints) are consistent; the backend /connectors API route is correctly untouched. No collision with the existing datasource-noun commands.
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
eddietejeda
added a commit
that referenced
this pull request
Jul 10, 2026
'hotdata ingest datasources' now browses the catalog of available datasource types; 'connectors' stays as a hidden alias. Help, error hints, README, and the skill follow. Co-authored-by: Eddie A Tejeda <669988+eddietejeda@users.noreply.github.com>
eddietejeda
added a commit
that referenced
this pull request
Jul 10, 2026
* feat(cli): loading indicators for indexes list and usage Both commands sat silent while fetching. `indexes list` gets one spinner over the whole fetch - the unscoped path is a whole-workspace scan (connections enumeration + managed-DB discovery + per-table index reads) that can take seconds. The database discovery inside the scan (databases::list_database_ids) drops its nested "Loading databases..." wakeup spinner so two indicatif bars never fight over the same line; `databases list` itself keeps the wakeup-hint spinner. `usage` gets a plain spinner around its single request. * feat(databases): show created_at in list and show commands (#217) * feat(databases): show created_at in list and show commands Adds a CREATED column to `databases list` and a created_at row to `databases show`. Uses the updated SDK models from sdk-rust feat/database-timestamps (PR #909 in runtimedb). Cargo.toml temporarily points to local sdk-rust path; update to published version once sdk-rust feat/database-timestamps is released. * chore: update hotdata SDK to v0.8.1 (created_at on databases) --------- Co-authored-by: Eddie A Tejeda <669988+eddietejeda@users.noreply.github.com> * feat(ingest): rename the catalog verb connectors -> datasources (#216) 'hotdata ingest datasources' now browses the catalog of available datasource types; 'connectors' stays as a hidden alias. Help, error hints, README, and the skill follow. Co-authored-by: Eddie A Tejeda <669988+eddietejeda@users.noreply.github.com> * fix(indexes): clear spinner before exiting on API error The fetch helpers called e.exit() (process::exit) internally, leaving the indicatif spinner live on the terminal when any API call failed. Convert list_one_table, list_one_table_scan, collect_tables, connection_lookup, managed_db_connection_ids, workspace_scan_targets, and collect_connection_wide to return Result<_, ApiError>, then finish_and_clear() the spinner in list() before unwrap_or_else(|e| e.exit()) -- matching the usage.rs pattern. * style: cargo fmt --------- Co-authored-by: Eddie A Tejeda <669988+eddietejeda@users.noreply.github.com>
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.
hotdata ingest datasourcesnow browses the catalog of available datasource types (SQL dialects, API services, buckets, iceberg);connectorsremains as a hidden alias. Help text, error hints, README, and the hotdata skill updated to match. Smoke-verified: new verb renders the catalog table, alias still works.