docs: Add migrations command to README#161
Conversation
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Original prompt from jonatas
|
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
|
Warning Review limit reached
More reviews will be available in 34 minutes and 44 seconds. Learn how PR review limits work. Your organization has run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available. Please see our Fair Usage Limits Policy for further information. 📝 WalkthroughWalkthroughAdds documentation for a new Migrations resource group in the CLI Resource Management command list and a section describing migration workflows with example subcommands for wizard, template export, and user import. ChangesMigrations Feature Documentation
Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Greptile SummaryThis PR documents the
Confidence Score: 5/5Documentation-only change with no code modifications; safe to merge. The change is limited to README additions. The export-auth0 flags were corrected in a prior commit, and the spec file confirms that wizard, import --csv, and export-auth0 --client-id/--client-secret are all exercised in tests. No runtime behavior is altered. README.md — the export-template subcommand examples (lines 211-212) were previously flagged as potentially non-existent in @workos/migrations; worth a manual workos migrations --help check before merging. Important Files Changed
|
|
|
||
| # Import users from CSV |
There was a problem hiding this comment.
Incorrect flags for
export-auth0
The --token flag does not match the actual @workos/migrations CLI. According to the official WorkOS docs and the @workos/migrations GitHub repo, export-auth0 requires --client-id and --client-secret (M2M app credentials), not --token. Using the documented command as-is will fail. The spec file (migrations.spec.ts line 40) also uses --client-id/--client-secret rather than --token, confirming the flag name is wrong.
There was a problem hiding this comment.
Good catch — fixed in the latest commit. Changed to --client-id <id> --client-secret <secret>.
| # Export a blank CSV template | ||
| workos migrations export-template saml_connections --output saml_connections.csv | ||
| workos migrations export-template oidc_connections --output oidc_connections.csv | ||
|
|
There was a problem hiding this comment.
export-template is not a documented subcommand of @workos/migrations
The @workos/migrations package lists its subcommands as: wizard, export-auth0, export-cognito, merge-passwords, transform-clerk, transform-firebase, validate, import, import-package, analyze, enroll-totp, and process-role-definitions. export-template does not appear in that list, nor does it appear in the spec file. Running workos migrations export-template ... as documented would likely result in an unknown command error.
There was a problem hiding this comment.
export-template does exist — it was added in workos-migrations#90 (merged). See src/cli/commands/export-template.ts and src/cli/index.ts in the @workos/migrations repo. Greptile is likely looking at a stale snapshot of the codebase.
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Summary
Adds the
migrationscommand/namespace to the CLI README:### Migrationssection with usage examples (wizard,export-template,export-auth0,import)Review & Testing Checklist for Human
workos migrations --helpactually showsNotes
Docs-only change — no code modified.
Link to Devin session: https://app.devin.ai/sessions/f5a65dfe04c34a4c804b7c1d158c1ee0
Requested by: @jonatascastro12
Summary by CodeRabbit
migrations --help