[Bug Fix] process.env.APP_URL doesn't work in production build

[t-jindai]

WHY are these changes introduced?

UI Extensions can request web API that you defined in web app.

In local development by using shopify app dev, web API host changes every time you launch dev server via cloudflare tunnel. It is very useful to use APP_URL environment variable to refer that tunnel url like this.

UI Extension

const url = `${process.env.APP_URL}/api/awesome-logic`

In production, however, APP_URL cannot be web API url (application_url in shopify.app.toml) through shopify app deploy or shopify app build. We expect that environment variables will change its value the environment basis (local, production) but it don't work now.

WHAT is this pull request doing?

shopify app dev passes configuration.application_url as APP_URL to extension build process.

cli/packages/app/src/cli/services/dev/app-events/app-event-watcher.ts

Line 270 in 7b3ba77

appURL: this.appURL,

otherwise, it isn't passed to extension build process in shopify app build and shopify app deploy command.

cli/packages/app/src/cli/services/deploy/bundle.ts

Line 38 in 7b3ba77

{stderr, stdout, signal, app: options.app, environment: 'production'},

In this PR, I added extra argument as AppURL.

How to test your changes?

Post-release steps

Measuring impact

How do we know this change was effective? Please choose one:

• n/a - this doesn't need measurement, e.g. a linting rule or a bug-fix
• Existing analytics will cater for this addition
• PR includes analytics changes to measure impact

Checklist

• I've considered possible cross-platform impacts (Mac, Linux, Windows)
• I've considered possible documentation changes