The credential manager says what it decides instead of calling it a pin

[wmadden-electric]

The operator's ask, twice over: first the banned word #repin, then the whole "pin" vocabulary — nobody knows what a pin is. The concept is which credential this process acts as, decided once at the first activeCredential() read. The private surface now says that: type ActingAs, field #actingAs, resolver #resolveActingAs, and #actAs for the mutation-time change that also discards storage built for the previous credential. Test descriptions use the same words. No behaviour change; all 978 CLI tests pass.

🤖 Generated with Claude Code

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 4e0ef64d-83d6-407f-8cc5-23aa52047ed3

📥 Commits

Reviewing files that changed from the base of the PR and between fef263e and 3d668a4.

📒 Files selected for processing (3)

• packages/cli/src/auth/credential-manager.ts
• packages/cli/tests/credential-manager-processes.test.ts
• packages/cli/tests/credential-manager.test.ts

Included review availability: Your plan includes up to 1 review per rolling hour; 0 remain after this review.

---

Summary by CodeRabbit

• Refactor
  • Improved credential selection so the process consistently retains the credential it is acting as.
  • Credential changes made by the process are applied immediately while external changes are handled predictably.
  • Environment-provided credentials remain available during credential updates.
  • Session and activity messages now use clearer acting-as terminology.

Walkthrough

The credential manager replaces Pin state with ActingAs state. Resolution caches the environment, session, or none state directly. Session mutations update acting-as state and clear cached active storage. Active credentials and file-backed storage use the acting session. Tests and process descriptions now use acting-as terminology.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	The title clearly summarizes the terminology change from “pin” to acting as a credential.
Description check	✅ Passed	The description directly explains the terminology changes, intended behavior, implementation details, and test results.

✨ Finishing Touches 💡 1

🛠️ Fix failing CI checks 💡

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch claude/rename-repin

✨ Simplify code

• Create PR with simplified code
• Commit simplified code in branch claude/rename-repin

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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

[pkg-pr-new]

Open in StackBlitz

npx https://pkg.pr.new/@prisma/cli@201

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

commit: 6d51f3d