Skip to content

Prompt for ABIU runtime settings when creating a backend from the "deploy" flow as well#10406

Open
falahat wants to merge 4 commits intonextfrom
abiuFix
Open

Prompt for ABIU runtime settings when creating a backend from the "deploy" flow as well#10406
falahat wants to merge 4 commits intonextfrom
abiuFix

Conversation

@falahat
Copy link
Copy Markdown
Contributor

@falahat falahat commented Apr 22, 2026

Description

The user can create a backend in 3 ways:

  1. firebase apphosting:backends:create
  2. firebase init
  3. firebase --deploy when it detects that a backend doesn't exist and prompts you to create it

This PR adds support for #3.

Scenarios Tested

firebase deploy with a backend that doesn't exist yet. I verified that i was asked the ABIU runtime questions and my backend was created correctly with the correct runtime and ABIU settings.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request centralizes runtime resolution logic into a new resolveRuntime helper function, ensuring consistent behavior across backend setup and deployment. It also adds comprehensive unit tests for this logic. The review feedback suggests enhancing doSetupSourceDeploy to accept and propagate a rootDir parameter instead of using a hardcoded value, which is necessary for supporting monorepos or projects where the application resides in a subdirectory.

Comment thread src/apphosting/backend.ts
Comment thread src/apphosting/backend.ts Outdated
Comment thread src/deploy/apphosting/prepare.ts
falahat and others added 3 commits April 22, 2026 17:12
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@falahat falahat marked this pull request as ready for review April 22, 2026 21:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants