Skip to content

fix(cli): make logout and update notices clearer - #317

Draft
AmanVarshney01 wants to merge 1 commit into
mainfrom
codex/clean-logout-update-output
Draft

AmanVarshney01 wants to merge 1 commit into
mainfrom
codex/clean-logout-update-output

Conversation

@AmanVarshney01

Copy link
Copy Markdown
Member

What changed

  • Remove the redundant ended: 1 field and completed-work notice from human prisma auth logout output. --json still returns endedCount and workspaceIds.
  • Check the published prisma package for updates and suggest prisma@latest, not @prisma/cli@latest.
  • Show a package-manager command when the local install is identifiable. Otherwise link directly to the official installation section. Ignore cached results from the old package.

Terminal previews

Captured with Termcut from the actual CLI presentation and update-check code, using an in-memory demo session and a synthetic 9.0.0 update. No real credentials were touched.

Known npm install

Logout with npm update command

Install method unknown

Logout with installation instructions link

Verification

  • pnpm --filter @prisma/cli test: 1022 passed, 2 skipped
  • pnpm typecheck and pnpm lint: passed
  • pnpm --filter @prisma/cli test:e2e: 7 passed, 48 skipped without a test API token

Signed-off-by: Aman Varshney <amanvarshney.work@gmail.com>
@coderabbitai

coderabbitai Bot commented Sep 28, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

Comment @coderabbitai help to get the list of available commands.

@pkg-pr-new

pkg-pr-new Bot commented Sep 28, 2026

Copy link
Copy Markdown

Open in StackBlitz

npx https://pkg.pr.new/@prisma/cli@317
npx https://pkg.pr.new/@prisma/cli-engine@317

commit: 347ab78

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