Skip to content

docs: point Dataset CLI to lmnr-cli (LAM-1786)#179

Open
laminar-coding-agent[bot] wants to merge 2 commits into
mainfrom
docs/lam-1786-dataset-cli-lmnr-cli
Open

docs: point Dataset CLI to lmnr-cli (LAM-1786)#179
laminar-coding-agent[bot] wants to merge 2 commits into
mainfrom
docs/lam-1786-dataset-cli-lmnr-cli

Conversation

@laminar-coding-agent

@laminar-coding-agent laminar-coding-agent Bot commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Rewrite datasets/cli.mdx to document the standalone lmnr-cli dataset commands instead of the deprecated SDK-bundled lmnr datasets CLI.
  • Swap the auth model from --project-api-key to user login (lmnr-cli login) plus directory-scoped projects, matching how lmnr-cli actually authenticates.
  • Update argument order and option lists to match the lmnr-cli source in lmnr-ts (push <paths...> -n, pull [output-path] -n, create <name> <paths...> -o).
  • Add a deprecation <Note> explaining lmnr-cli replaces the old lmnr datasets CLI.

Notes

  • Verified command surface and flags against lmnr-ts/packages/lmnr-cli/src/index.ts and commands/dataset/index.ts.
  • npx mintlify broken-links passes; no em dashes or banned marketing words.

🤖 Generated with Claude Code


Note

Low Risk
Documentation-only changes with no runtime or security impact.

Overview
Rewrites datasets/cli.mdx so dataset workflows use the standalone lmnr-cli dataset commands instead of the deprecated SDK lmnr datasets flow, including an install section and a deprecation note.

Authentication is documented as lmnr-cli setup plus .lmnr/project.json (or --project-id), replacing LMNR_PROJECT_API_KEY / --project-api-key. Examples drop the old JS/Python tabs and use updated syntax such as push <file> -n <name> and pull <file> -n <name>.

The reference section matches lmnr-cli flags (--project-id, --json, env defaults for self-hosted --base-url / --port).

Reviewed by Cursor Bugbot for commit 7d48221. Bugbot is set up for automated code reviews on this repo. Configure here.

@mintlify

mintlify Bot commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
laminarai 🟢 Ready View Preview Jun 17, 2026, 10:14 PM

💡 Tip: Enable Workflows to automatically generate PRs for you.

@kolbeyang

Copy link
Copy Markdown
Collaborator

@laminar-coding-agent

Is this PR still necessary or have changes already been made on a different PR?

If necessary, rebase onto latest main branch.

kolbeyang and others added 2 commits July 13, 2026 13:18
Rewrite datasets/cli.mdx to use the standalone lmnr-cli dataset
commands instead of the deprecated SDK-bundled lmnr datasets CLI.
Switches the auth model from --project-api-key to user login plus
directory-scoped projects, and updates argument order to match the
lmnr-cli source.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@laminar-coding-agent laminar-coding-agent Bot force-pushed the docs/lam-1786-dataset-cli-lmnr-cli branch from a89cbbe to 7d48221 Compare July 13, 2026 13:19
@laminar-coding-agent

Copy link
Copy Markdown
Contributor Author

@kolbeyang Yes, this PR is still necessary — datasets/cli.mdx on main still documents the deprecated SDK-bundled lmnr datasets CLI with LMNR_PROJECT_API_KEY auth; no other PR has updated it to lmnr-cli.

Rebased onto latest main (clean rebase, no conflicts) and force-pushed. npx mintlify broken-links passes on the rebased branch.

@laminar-coding-agent

Copy link
Copy Markdown
Contributor Author

Done. PR #179 is still necessary — main still documents the deprecated lmnr datasets SDK CLI, and no other PR has migrated it to lmnr-cli. I rebased the branch onto latest main (clean, no conflicts), verified mintlify broken-links passes, force-pushed to the same branch (new head 7d48221, PR mergeable), and replied to @kolbeyang on the PR confirming the status.

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