chore: remove uuid package

[AnastasiyaHladina]

Description

This PR removes uuid package which can be replaced with crypto module which comes from the standard web API.

Automation

/ok-to-test tags=""

🔍 Cypress test results

Caution

If you modify the content in this section, you are likely to disrupt the CI result for your PR.

Communication

Should the DevRel and Marketing teams inform users about this change?

• Yes
• No

Summary by CodeRabbit

• Refactor
  • Replaced ID generation with a native browser API across the app, maintaining existing behavior and signatures.
• Tests
  • Updated Cypress helpers and commands to align with the new ID generation approach.
• Chores
  • Removed an external UUID dependency and its type definitions to reduce dependencies and simplify maintenance.
• Impact
  • No user-facing behavior changes. Slightly smaller bundle and improved consistency by using built-in capabilities across the app.

[coderabbitai]

Walkthrough

Replaced usages of the external uuid library with the Web Crypto API’s crypto.randomUUID() across Cypress helpers, runtime components, instrumentation, and constants. Removed uuid and @types/uuid from package.json. No public APIs changed; control flow and data shapes remain the same.

Changes

Cohort / File(s)	Summary
Cypress test utilities app/client/cypress/support/Pages/AggregateHelper.ts, app/client/cypress/support/commands.js	Switched UUID generation from uuid.v4() to crypto.randomUUID(); removed uuid import; retained public command/signatures.
Client form config app/client/src/components/formControls/FunctionCallingConfigControl/components/FunctionCallingConfigForm.tsx	Replaced uuid() with crypto.randomUUID() for new entry IDs; no other logic changes.
Instrumentation event IDs app/client/src/instrumentation/index.ts	Generated eventId via crypto.randomUUID() instead of uuidv4; logging flow unchanged.
Layout constants app/client/src/layoutSystems/common/AnalyticsWrapper/constants.ts	Updated LAYOUT_WRAPPER_ID initializer to crypto.randomUUID(); removed uuid import.
Dependency cleanup app/client/package.json	Removed uuid from dependencies and @types/uuid from devDependencies.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

A spark of ID from crypto’s stream,
No more uuid in our scheme.
Tests and forms keep stepping true,
Instrumentation logs anew.
Package trim, a cleaner view—
Random dreams, but built-in too.

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description Check	⚠️ Warning	The PR description follows the template sections for Description, Automation, and Communication but omits the required “Fixes #Issue” line under the Description header, which is mandatory per the template.	Please add a “Fixes #” or “Fixes ” line under the Description section to reference the issue that this change resolves, ensuring alignment with the repository’s PR template.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title Check	✅ Passed	The title “chore: remove uuid package” accurately and concisely summarizes the primary change of replacing uuid with the built-in crypto API and dropping the uuid dependency.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

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 and usage tips.}

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (1)

app/client/cypress/support/Pages/AggregateHelper.ts (1)

1175-1179: LGTM! UUID generation updated correctly.

The change preserves the "Cy" prefix behavior and Cypress alias storage.

Consider consolidating UUID generation logic with the generateUUID command in commands.js (line 326-329) to reduce duplication. Both implementations split the UUID and take the first segment, though this method adds a "Cy" prefix.

📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 336d318 and dd05ad9.

⛔ Files ignored due to path filters (1)

• app/client/yarn.lock is excluded by !**/yarn.lock, !**/*.lock

📒 Files selected for processing (6)

• app/client/cypress/support/Pages/AggregateHelper.ts (1 hunks)
• app/client/cypress/support/commands.js (1 hunks)
• app/client/package.json (0 hunks)
• app/client/src/components/formControls/FunctionCallingConfigControl/components/FunctionCallingConfigForm.tsx (1 hunks)
• app/client/src/instrumentation/index.ts (1 hunks)
• app/client/src/layoutSystems/common/AnalyticsWrapper/constants.ts (1 hunks)

💤 Files with no reviewable changes (1)

• app/client/package.json

🧰 Additional context used

📓 Path-based instructions (1)

app/client/cypress/**/**.*

⚙️ CodeRabbit configuration file

app/client/cypress/**/**.*: Review the following e2e test code written using the Cypress test library. Ensure that:

• Follow best practices for Cypress code and e2e automation.
• Avoid using cy.wait in code.
• Avoid using cy.pause in code.
• Avoid using agHelper.sleep().
• Use locator variables for locators and do not use plain strings.
• Use data-* attributes for selectors.
• Avoid Xpaths, Attributes and CSS path.
• Avoid selectors like .btn.submit or button[type=submit].
• Perform logins via API with LoginFromAPI.
• Perform logout via API with LogOutviaAPI.
• Perform signup via API with SignupFromAPI.
• Avoid using it.only.
• Avoid using after and aftereach in test cases.
• Use multiple assertions for expect statements.
• Avoid using strings for assertions.
• Do not use duplicate filenames even with different paths.
• Avoid using agHelper.Sleep, this.Sleep in any file in code.

Files:

• app/client/cypress/support/Pages/AggregateHelper.ts
• app/client/cypress/support/commands.js

🧬 Code graph analysis (5)

app/client/cypress/support/Pages/AggregateHelper.ts (2)

app/client/cypress/support/commands.js (1)

• id (327-327)

app/client/config/getHttpsConfig.js (1)

• crypto (5-5)

app/client/cypress/support/commands.js (1)

app/client/config/getHttpsConfig.js (1)

• crypto (5-5)

app/client/src/instrumentation/index.ts (1)

app/client/config/getHttpsConfig.js (1)

• crypto (5-5)

app/client/src/components/formControls/FunctionCallingConfigControl/components/FunctionCallingConfigForm.tsx (2)

app/client/cypress/support/commands.js (1)

• id (327-327)

app/client/config/getHttpsConfig.js (1)

• crypto (5-5)

app/client/src/layoutSystems/common/AnalyticsWrapper/constants.ts (1)

app/client/config/getHttpsConfig.js (1)

• crypto (5-5)

🔇 Additional comments (4)

app/client/src/instrumentation/index.ts (1)

130-130: LGTM! Event ID generation updated correctly.

The migration preserves the method's return type and behavior. Both uuidv4() and crypto.randomUUID() return RFC 4122 v4 UUID strings.

app/client/src/components/formControls/FunctionCallingConfigControl/components/FunctionCallingConfigForm.tsx (1)

92-92: LGTM! ID generation updated correctly.

The change preserves functionality. The generated UUID is used as a unique identifier for new entries in the function calling configuration.

app/client/cypress/support/commands.js (1)

326-329: LGTM! Cypress command updated correctly.

The migration preserves the command's behavior. Cypress runs tests in modern browsers where crypto.randomUUID() is natively supported.

app/client/src/layoutSystems/common/AnalyticsWrapper/constants.ts (1)

1-1: Approve crypto.randomUUID() usage
TypeScript tsconfig includes the DOM lib, so the global crypto.randomUUID() is recognized without extra type declarations.