Skip to content

fix(databases): rename --description to --name in databases run#122

Merged
eddietejeda merged 1 commit into
mainfrom
fix/databases-run-use-name
Jun 4, 2026
Merged

fix(databases): rename --description to --name in databases run#122
eddietejeda merged 1 commit into
mainfrom
fix/databases-run-use-name

Conversation

@eddietejeda
Copy link
Copy Markdown
Contributor

Summary

  • Renames --description to --name in databases run to be consistent with databases create, which already uses --name for the catalog alias
  • Updates the field name through command.rs, main.rs, and databases.rs (run and create_and_return_id)

Test plan

  • hotdata databases run --name my-db -- <cmd> creates a named auto-database and runs the command
  • hotdata databases run --database <id> -- <cmd> still works (unaffected path)
  • hotdata databases create --name my-db still works (unchanged)

Align `databases run` with `databases create`, which already uses --name
for the catalog alias of the auto-created database.
Copy link
Copy Markdown
Contributor

@claude claude Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Clean rename — databases run now matches databases create on --name, and the value still flows into the "name" JSON field via create_database_request, so behavior is unchanged.

nit: the docs still reference the old --description flag for databases run and should be updated to --name (not blocking):

  • README.md:140 and README.md:151
  • skills/hotdata/SKILL.md:194 and skills/hotdata/SKILL.md:214

@sentry
Copy link
Copy Markdown

sentry Bot commented Jun 4, 2026

Codecov Report

❌ Patch coverage is 33.33333% with 4 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/databases.rs 50.00% 2 Missing ⚠️
src/main.rs 0.00% 2 Missing ⚠️

📢 Thoughts on this report? Let us know!

@eddietejeda eddietejeda merged commit 66ca52a into main Jun 4, 2026
11 checks passed
eddietejeda added a commit that referenced this pull request Jun 4, 2026
* fix(databases): rename --description to --name in databases run

Align `databases run` with `databases create`, which already uses --name
for the catalog alias of the auto-created database.

* fix(skills): update --description to --name in databases commands

Align skill docs with the CLI change in #122: databases create and
databases run both use --name, not --description.

---------

Co-authored-by: Eddie A Tejeda <669988+eddietejeda@users.noreply.github.com>
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