feat(datasets): Add datasets commands to create, list, and view datasets in CLI

[pthurlow]

No description provided.

[claude]

The nix crate is only used inside a #[cfg(target_os = "macos")] block (stdin_redirect_filename), but it's declared as an unconditional dependency. This means it compiles and links on Linux and Windows even though it's never called there.

Consider a platform-scoped dependency:

Suggested change

	comfy-table = "7"
	nix = { version = "0.29", features = ["fs"], optional = true }

and activate it only for macOS in [target.'cfg(target_os = "macos")'.dependencies], or use a target-specific dependency table directly:

[target.'cfg(target_os = "macos")'.dependencies]
nix = { version = "0.29", features = ["fs"] }

[claude]

Review

Issues

• src/datasets.rs line 399–402 — body.count is used both to display the page size and to compute the next --offset. If count is the total number of records (the common REST API convention), the suggested --offset will be wrong — pointing past all data rather than to the next page. Use body.datasets.len() instead, which is always correct.

Action Required

Clarify or fix the count field semantics in ListResponse. If it is page-scoped, add a comment stating that. If it is total-scoped (or unknown), switch to body.datasets.len() for both the display and the offset hint before merging.

review reason is invalid