You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Thanks for adding the Android quickstart. I think this should be brought up to parity with the existing Next.js /
Python / Go scaffolding before merge, rather than shipping it as clone-only.
Right now the PR describes Android as “clone-only” and disables init support, but that creates an inconsistent
CLI experience and also leaves a bug in the current create flow: if the user already has a current project
selected, quickstart create --template android can still try env seeding after clone, fail with QUICKSTART_TEMPLATE_ENV_UNSUPPORTED, and remove the cloned directory.
Requested changes:
Make Android a full supported quickstart, not clone-only.
So the template should be wired like the other full quickstarts:
SupportsInit: true
env example path should point at the Android repo’s backend env example
env target path should point at the backend .env
install/run commands should describe both Android frontend and Python backend setup
remove “clone-only” wording from README, docs, and EnvDocsSummary
Use the correct default branch/ref.
The PR currently sets Ref: "rest-api", but the backend path above is on main. Please confirm the intended
branch. If main is the supported quickstart, do not force rest-api; let the default branch clone or set
the template ref to main only if we need the JSON envelope to report it explicitly.
Wire env seeding for the Python backend.
Android should receive the same Agora project credentials flow as the Python quickstart, written into the
backend server env file. This should work for:
Please add integration coverage using AGORA_QUICKSTART_ANDROID_REPO_URL pointed at a local git repo fixture
with the expected Android + server layout. The test should verify:
Android appears in quickstart list
quickstart create --template android succeeds without a project
quickstart env write --template android updates the backend env
init --template android works if Android is marked SupportsInit: true
Update docs/contracts.
Since this adds a new public template, please update:
CHANGELOG.md
docs/automation.md quickstart template docs
README examples
docs/llms.txt
Once Android has the same project binding, env writing, and init behavior as the other supported quickstarts,
this will fit the CLI’s current scaffolding model much better.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Type of change
error.coderename/removal)Public-contract impact
error.code— added todocs/error-codes.md.error.code— flagged as breaking, included in CHANGELOG.docs/automation.md.Test plan
go test ./...passes locally.make lintpasses locally (gofmt,golangci-lint, error-code coverage audit).internal/cli/integration_test.go.internal/cli/app_test.go(where applicable).Documentation
CHANGELOG.mdupdated under## Unreleased(Added / Changed / Deprecated / Removed / Fixed / Security).docs/automation.mdupdated for any user-facing JSON shape, env var, or flag change.docs/error-codes.mdupdated for any newerror.code(or N/A).README.mdupdated if the command tree, install path, or quickstart changed.AGENTS.mdupdated if engineering or release process changed.Security checklist
$HOMEwithout0o600perms when it can contain credentials (e.g. session, config).unsafeimport.Additional notes