Check for process existence before accessing it

[omarestrella]

Problem

On some edge platforms (specifically Cloudflare Workers), process does not exist at all, so checking !process isn't enough and causes a runtime error / crash.

Solution

I added a typeof process check before any process access occurs so that the Pinecone client works on non-Node runtimes.

Type of Change

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• This change requires a documentation update
• Infrastructure change (CI configs, etc)
• Non-code change (docs, etc)
• None of the above: (explain here)

Test Plan

No test changes should be necessary, as this is just a more strict !process check.

[jhamon]

Thanks for the fix. If you run npm run format and commit the diff that will fix the lint/format check. The rest are failing due to environment variables being withheld for security reasons so shouldn't be a concern.

[omarestrella]

Thanks for the fix. If you run npm run format and commit the diff that will fix the lint/format check. The rest are failing due to environment variables being withheld for security reasons so shouldn't be a concern.

Gotcha, formatting should be fixed.

[austin-denoble]

Looks good to me, thanks again for the fix.

[jhamon]

Thanks again!

[austin-denoble]

This fix has been included in the v1.1.2 release: https://github.com/pinecone-io/pinecone-ts-client/releases/tag/v1.1.2.

Thanks again for contributing!