[V8] remove direct process.env access, replacing with getEnv helper

[nicknisi]

Description

This pull request refactors how environment variables are accessed in the WorkOS class to improve code clarity and maintainability. The changes replace direct access to process.env with a utility function getEnv.

Refactoring environment variable access:

• src/workos.ts: Introduced the getEnv utility function from ./common/utils/env to centralize and simplify access to environment variables.
• src/workos.ts: Replaced direct usage of process.env.WORKOS_API_KEY with getEnv('WORKOS_API_KEY') for retrieving the API key. This ensures consistent handling of environment variables across environments.
• src/workos.ts: Replaced direct usage of process.env.WORKOS_CLIENT_ID with getEnv('WORKOS_CLIENT_ID') for retrieving the client ID. This further standardizes the approach to accessing environment variables.

Documentation

Does this require changes to the WorkOS Docs? E.g. the API Reference or code snippets need updates.

[ ] Yes

If yes, link a related docs PR and add a docs maintainer as a reviewer. Their approval is required.

[greptile-apps]

Greptile Summary

This PR improves the cross-runtime compatibility of the WorkOS Node.js SDK by replacing direct process.env access with a new getEnv utility function. This change is particularly important as it:

1. Enables the SDK to work in environments where process.env isn't available (like Deno, Bun, Cloudflare Workers, etc.)
2. Centralizes environment variable access through a single utility function
3. Makes the code more maintainable by abstracting environment handling

The changes specifically affect how WORKOS_API_KEY and WORKOS_CLIENT_ID are accessed in the WorkOS class.

Confidence score: 4/5

1. This PR is safe to merge as it maintains existing functionality while improving runtime compatibility
2. High confidence due to the straightforward nature of the changes and their clear benefits for cross-runtime support
3. Files needing attention:
   • src/common/utils/env.ts - Verify the implementation of getEnv utility
   • src/workos.ts - Ensure all process.env accesses are properly replaced

_{1 file reviewed, 1 comment
Edit PR Review Bot Settings | Greptile}