fix(cli): skip settings in push and pull all when only an API key is configured

[ChiragAgg5k]

What does this PR do?

appwrite push all and appwrite pull all stopped at their first step when the CLI was configured with only an API key (appwrite client --endpoint ... --project-id ... --key ...), which is the usual CI setup. Both fan-outs start with project settings, and settings go through console endpoints: consoleClient() returns ErrConsoleNeedsSession ("session not found...") for a key-only session, and the fan-out returned that error before touching tables, functions, sites, buckets, teams, webhooks, topics or analytics.

The fan-outs now recognise ErrConsoleNeedsSession, print a one-line warning and carry on with the remaining resources:

ℹ Warning: Skipping settings: it needs a signed-in session, and only an API key is configured. Run `appwrite login` to push settings too.

The check reuses the existing auth detection in consoleClient() (session.go) rather than adding a second one, so a signed-in session behaves exactly as before. Any other error still stops the fan-out, and an explicit push settings / pull settings still fails with the same message.

Test Plan

New tests in templates/cli/internal/cmd/fanout_test.go:

• TestEverythingSkipsConsoleOnlyActionsUnderAnAPIKey: for both pull and push, a settings action returning ErrConsoleNeedsSession is skipped with the warning and later actions still run.
• TestEverythingStillStopsOnOtherErrors: other errors still abort the fan-out.
• TestAllUnderAnAPIKeyLeavesOutOnlySettings: end to end through the root command with a key-only prefs.json, a project config and a mock server; pull all --force and push all --force both succeed and report the skipped settings.

Red/green: with the two skipsConsoleOnly checks removed from a scratch copy of the generated CLI, the first and third tests fail with session not found. Run appwrite login...; with the fix they pass. php example.php cli, gofmt -l, go build ./..., go vet ./..., go test ./..., composer lint-twig, composer refactor:check and the twig line-length check all pass.

Related PRs and Issues

None.

Closes #2083

[hansi-codes]

🟢 Tier S · Ready to merge

The change matches the targeted API-key-only case, preserves failure behavior for other errors, and tests both fan-outs.

This change lets push all and pull all continue past console-only settings when the CLI is configured with an API key, and prints an actionable warning. Other errors remain fatal, with tests covering the skip behavior and key-only runs.

Verdict	New comments	Fixed	Still open
✅ Approved	0	0	0

📂 Walkthrough · 4

File	Change
templates/cli/internal/cmd/fanout_test.go	Adds unit and end-to-end coverage for skipping settings under an API key while retaining other error handling.
templates/cli/internal/cmd/pullall.go	Continues the pull fan-out when a console-only action returns ErrConsoleNeedsSession.
templates/cli/internal/cmd/pushall.go	Continues the push fan-out when a console-only action returns ErrConsoleNeedsSession.
templates/cli/internal/cmd/session.go	Adds a shared console-only skip check that prints an actionable warning.

---

_{Reviewed 43af547 · Details · Comment @hansi-codes review to re-run, or mention @hansi-codes with a question.}

[hansi-codes]

🟢 Tier S · Looks good to merge. Summary