docs(skills): document databases lineage, forked_from, and retired job types - #287
Merged
Merged
Conversation
…b types Catch the skills up to the v0.29/v0.30 surface: - add 'databases lineage' to the skill synopsis, command bullets, and README command table (landed in #281, undocumented since) - note forked_from provenance on 'databases show' / 'fork' output, and point the fork-before-risky-changes workflow at lineage for recovering a lost source/fork id pair - drop the retired data_refresh_table / data_refresh_connection job types from the jobs section (removed server-side with #281) - surface 'query --dialect' (#266) in the main skill's query synopsis with a pointer to the analytics subskill
eddietejeda
requested review from
anoop-narang
and removed request for
a team
September 3, 2026 18:39
| - `use` — saves the database **id** as the active database. Subsequent `databases tables` and `databases context` commands use it automatically. Note that a successful `fork` also updates this: the fork becomes the active database. | ||
| - `unset` — clears the active database from config. | ||
| - `<id>` — inspect one database (returns id, catalog, name, expires_at). | ||
| - `<id>` — inspect one database (returns id, catalog, name, expires_at; a fork also shows its `forked_from` record). |
Contributor
There was a problem hiding this comment.
super nit: expires_at is not in the table output. databases::get prints id, name, catalog, created_at, catalog id, forked_from, sql_prefix and attachments (src/commands/databases.rs:1191-1244); expires_at appears only under -o json/yaml. A reader following this bullet looks for a field the default output never shows. Consider listing created_at instead of expires_at. (not blocking)
true
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
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
Audit of the bundled skills against everything that landed since they were last touched (v0.28, #275/#276). Three staleness fixes and one small improvement:
databases lineagewas missing entirely (added in feat(databases): lineage command and forked_from provenance #281, v0.29). Added to thedatabasessynopsis inskills/hotdata/SKILL.mdwith a bullet covering the fork-tree rendering, direct-forks-only semantics, deleted-generation marking, and--forks-limitpaging. Also added the missing row to the README command table.jobs list --job-typelisted retired types.data_refresh_table/data_refresh_connectionwere dropped server-side in feat(databases): lineage command and forked_from provenance #281; the skill now lists onlycreate_indexandmanaged_load.forked_fromprovenance was undocumented. Noted on theforkanddatabases <id>bullets, and the fork-before-risky-changes workflow inreferences/WORKFLOWS.mdnow points atlineagefor recovering a lost source/fork id pair.query --dialect(feat(query): add --dialect to transpile DuckDB/Postgres/Snowflake SQL #266, v0.27) was documented only in the analytics subskill; the main skill's query synopsis now shows the flag with a pointer tohotdata-analyticsfor details.Verified against the clap definitions (
src/commands/databases.rs,src/commands/jobs.rs) and the built binary's--helpoutput — flags, defaults, and the--job-typevalue set all match. The v0.30 / post-release commits (#283, #285, #286) are rendering/dependency fixes with no doc impact; "managed table" wording is intentionally kept per #275.Docs only — no code changes.