fix(credential-manager): ensure unique name and id when creating credentials

[jarekdanielak]

Problem

If credentials are created with the default name and ID combo, the next time I open the create modal, I see errors, cause the defaults are the same.

Not a great UX.

Screen.Recording.2026-08-24.at.13.18.28.mov

Proposed Changes

Ensure the default values for name and ID are unique, by adding a number incrementally:

Screen.Recording.2026-08-24.at.13.16.53.mov

Steps to try out

1. Create credentials with the default name and ID.
2. Open Create modal again.
3. Before, you see errors on name and ID.
4. After, you see no errors.

Checklist

Ensure you provide everything we need to review your contribution:

• Contribution meets our definition of done
• Pull request establishes context
  • Link to related issue(s), i.e. Closes {LINK_TO_ISSUE} or Related to {LINK_TO_ISSUE}
  • Brief textual description of the changes
  • Screenshots or short videos showing UI/UX changes
  • Steps to try out, i.e. using the @bpmn-io/sr tool

[github-actions]

This Pull Request targets develop branch, but contains fix commits.

Consider targeting main instead.

[Copilot]

Pull request overview

This PR improves the credential creation UX in the cloud credential manager by ensuring the default “Credential name” (display name) and “Credential ID” suggested in the modal are unique relative to existing credentials, preventing immediate validation errors when reopening the create modal.

Changes:

• Add getUniqueCredentialIdentity(...) to derive a non-colliding (displayName, credentialId) pair when opening the modal in create mode.
• Update CredentialModal to use the new helper for initial state in create mode.
• Add a unit test asserting unique name/ID suggestions when collisions already exist.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File	Description
client/src/app/modals/credentials/CredentialModal.js	Uses a new helper to initialize unique default display name + ID in create mode.
client/src/app/modals/credentials/credentialId.js	Adds helper that finds a unique credential identity based on existing credentials.
client/src/app/modals/credentials/tests/CredentialModalSpec.js	Adds test coverage for the unique suggestion behavior.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[nikku]

What is our strategy? Try 1, 2, 3, 4 until one "does not exist"?

[nikku]

(We could instead append a generated ID in case of a clash).

[jarekdanielak]

What is our strategy? Try 1, 2, 3, 4 until one "does not exist"?

Pretty much.

(We could instead append a generated ID in case of a clash).

We could. I chose an incremented number simply cause it looks better on the display name.

[barmac]

That aligns with the connection manager; I think it's fine:

[barmac]

You will rename it anyway.

[barmac]

This will crash the editor if ever reached - do we want that?

[jarekdanielak]

Yea, shouldn't happen, but let's not blow up the modeler.

Fixed in 98fbf68 - we log error and return something to display.