fix(envvars): fix split for pasting envvars with query params

[waleedlatif1]

Summary

fix split for pasting envvars with query params

Type of Change

• Bug fix

Testing

Fix splitting envvars with query params

Checklist

• Code follows project style guidelines
• Self-reviewed my changes
• Tests added/updated and passing
• No new warnings introduced
• I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Preview	Comments	Updated (UTC)
sim	Ready	Preview	Comment	Aug 28, 2025 2:55am

1 Skipped Deployment

Project	Deployment	Preview	Comments	Updated (UTC)
docs	Skipped			Aug 28, 2025 2:55am

[greptile-apps]

Greptile Summary

This PR fixes a critical bug in the environment variable paste handling functionality within the settings modal. The issue occurred when users pasted environment variables containing URLs with query parameters or other values with multiple = characters. The original implementation used split('=') which would incorrectly break these values into multiple parts, corrupting the data.

The fix introduces two key improvements:

1. Smarter parsing logic: Replaces split('=') with indexOf('=') to find only the first equals sign and treat everything after it as the complete value, preserving URLs and other complex values intact
2. Environment variable name validation: Adds regex validation (/^[A-Za-z_][A-Za-z0-9_]*$/) to ensure only properly formatted environment variable names are treated as keys during paste operations

The changes are contained within the handleKeyValuePaste function in the environment variables component, which is responsible for processing clipboard content when users paste key-value pairs. This component is part of the settings modal's environment management interface, allowing users to efficiently manage their application's environment variables. The fix ensures that legitimate environment variable values containing = characters (such as API endpoints with query parameters) are preserved correctly while maintaining the ability to parse actual key-value pairs from pasted content.

Confidence score: 4/5

• This PR is safe to merge with minimal risk as it addresses a specific parsing bug without affecting other functionality
• Score reflects targeted bug fix with improved validation logic that maintains backward compatibility while fixing the core issue
• Pay close attention to the environment variable parsing logic in the handleKeyValuePaste function

_{1 file reviewed, no comments}

_{Edit Code Review Bot Settings | Greptile}