Adds an --immediate flag to ext:uninstall which can be run outside a firebase project dir and immediately tears down GCP resources

[Berlioz]

No description provided.

[gemini-code-assist]

Code Review

This pull request introduces an --immediate option to the ext:uninstall command, enabling immediate deletion of an extension instance and its associated GCP resources. The review feedback suggests several improvements: throwing a FirebaseError on deletion failure to ensure non-zero exit codes in CI environments, importing FirebaseError, adding a description to the new --immediate option, avoiding the use of any in catch clauses, and checking if the extension instance exists in the manifest before attempting removal to prevent blocking the teardown process.

[gemini-code-assist]

1. Correctness: Logging the error and returning undefined allows the command to exit with code 0 (success) even if the deletion failed. To ensure scripts and CI environments can detect failures, we should throw a FirebaseError instead.
2. Style Guide Adherence: Use unknown instead of any in the catch clause to maintain type safety.

      try {
        await deleteInstance(projectId, instanceId);
      } catch (err: unknown) {
        throw new FirebaseError(
          `Error when attempting deletion: ${err instanceof Error ? err.message : String(err)}`,
          { original: err instanceof Error ? err : undefined }
        );
      }

References

1. Never use any or unknown as an escape hatch. Define proper interfaces/types or use type guards. ^(link)
2. Throw FirebaseError (src/error.ts) for expected, user-facing errors. ^(link)

[gemini-code-assist]

To support throwing a FirebaseError when deletion fails, we need to import FirebaseError from ../error.

Suggested change

	import { Options } from "../options";
	import { needProjectId } from "../projectUtils";
	import { confirm } from "../prompt";
	import { Options } from "../options";
	import { needProjectId } from "../projectUtils";
	import { confirm } from "../prompt";
	import { FirebaseError } from "../error";

[Berlioz]

Ack

[inlined]

LGTM after you run the formatter.

[inlined]

What the heck does return after throw mean?

[Berlioz]

it means i'm an idiot