Skip to content

Add AIO_DEV env flag to detect running local #151

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
purplecabbage merged 3 commits into from
May 20, 2025
Merged

Add AIO_DEV env flag to detect running local #151

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
d6b673d
add AIO_DEV env flag to detect running local
purplecabbage May 15, 2025
f035428
use somewhat more correct 'true' string and don't rely on .env type c…
purplecabbage May 20, 2025
c1e1699
explicity state that this is an example and not commented code .. for…
purplecabbage May 20, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions src/lib/run-dev.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -81,6 +81,10 @@ async function runDev (runOptions, config, _inprocHookRunner) {
process.env.__OW_API_KEY = process.env.AIO_RUNTIME_AUTH
process.env.__OW_NAMESPACE = process.env.AIO_RUNTIME_NAMESPACE
process.env.__OW_API_HOST = process.env.AIO_RUNTIME_APIHOST
// set up environment variables for aio
// this can be read as truthy, it will not exist in Runtime
// ex. console.log('AIO_DEV ', process.env.AIO_DEV ? 'dev' : 'prod')
process.env.AIO_DEV = 'true'
Copy link

Copilot AI May 20, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Consider checking if process.env.AIO_DEV is already set before overwriting it. This helps preserve any external overrides and makes the intent of the flag clearer.

Suggested change
process.env.AIO_DEV = 'true'
if (process.env.AIO_DEV === undefined) {
process.env.AIO_DEV = 'true'
}

Copilot uses AI. Check for mistakes.

const serverPortToUse = parseInt(process.env.PORT) || SERVER_DEFAULT_PORT
const serverPort = await getPort({ port: serverPortToUse })
Expand Down