Fix(yaml env var): added env var fallback

[aadamgough]

Summary

Follow other uses of SIM_AGENT_API_URL to allow easier local testing.

Testing

Manually

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	Sep 10, 2025 1:13am

1 Skipped Deployment

Project	Deployment	Preview	Comments	Updated (UTC)
docs	Skipped			Sep 10, 2025 1:13am

[greptile-apps]

Greptile Summary

This PR introduces a partial fix for environment variable handling in the YAML workflow conversion route. The change adds proper fallback logic for the SIM_AGENT_API_URL environment variable by importing the SIM_AGENT_API_URL_DEFAULT constant and creating a local variable that falls back to the default value when the environment variable is undefined.

The modification addresses a potential runtime issue where the application could fail when making API calls to the sim-agent service if the SIM_AGENT_API_URL environment variable is not configured. According to the environment configuration, this variable is optional, so providing a fallback ensures the application can gracefully default to 'https://agent.sim.ai' in deployment scenarios where this environment variable isn't set.

The change follows the established pattern in the codebase where critical URLs have default fallbacks, ensuring robustness across different deployment environments. This aligns with the application's architecture of supporting multiple deployment configurations while maintaining core functionality.

PR Description Notes:

• The PR description is incomplete with empty Summary, Type of Change, and Testing sections
• All checklist items remain unchecked

Confidence score: 2/5

• This PR introduces an incomplete fix that creates inconsistent behavior within the same file
• Score reflects the partial implementation where only one of two usages of the same environment variable was updated (line 295 fixed, line 96 still vulnerable)
• Pay close attention to apps/sim/app/api/workflows/[id]/yaml/route.ts as it contains inconsistent environment variable usage that could cause runtime errors

_{1 file reviewed, no comments}

_{Edit Code Review Bot Settings | Greptile}