@rebasepro/cli 0.12.1-canary.gf5f1d39 → 0.13.0

package/bin/rebase.js

@@ -88,4 +88,30 @@ if (!existsSync(distEntry)) {
 
 const { entry } = await import("../dist/index.es.js");
 
-entry(process.argv);
+/**
+ * The CLI's last line of defence.
+ *
+ * `entry()` returns a promise and nothing was awaiting it, so anything a
+ * command threw surfaced as an unhandled rejection: Node's own stack trace,
+ * rooted in `dist/index.es.js`, with the CLI's bundled line numbers and no
+ * exit code of its own. "Collections directory not found" is a sentence a
+ * developer can act on; the same sentence under ten frames of bundle internals
+ * reads as a crash in Rebase.
+ *
+ * The message is the error's own — commands that already print something
+ * friendly and exit never reach here. The stack is available behind
+ * `--debug`/`REBASE_DEBUG`, because when the message is *not* enough that is
+ * the only thing that helps.
+ */
+const wantsStack = process.argv.includes("--debug") || process.env.REBASE_DEBUG === "1";
+
+entry(process.argv).catch((error) => {
+    const message = error instanceof Error ? error.message : String(error);
+    process.stderr.write(`\x1b[31m✗ ${message}\x1b[0m\n`);
+    if (wantsStack && error instanceof Error && error.stack) {
+        process.stderr.write(`\n${error.stack}\n`);
+    } else {
+        process.stderr.write("\x1b[90m  Re-run with --debug for the stack trace.\x1b[0m\n");
+    }
+    process.exit(1);
+});

package/dist/bundle.d.ts

@@ -68,6 +68,41 @@ export declare function detectNativeDependencies(projectRoot: string, declared:
  * own config package already travels inside the bundle.
  */
 export declare function collectDeclaredDependencies(projectRoot: string): Record<string, string>;
+/** One `@rebasepro/*` dependency as some package.json in the project declares it. */
+export interface DeclaredFrameworkDep {
+    name: string;
+    range: string;
+    /** Project-relative package.json it was declared in. */
+    file: string;
+}
+export interface FrameworkDepDrift {
+    /** Declared at a version that can never reach the CLI's own. */
+    behind: DeclaredFrameworkDep[];
+    /**
+     * The distinct lower bounds found across all declared `@rebasepro/*`, when
+     * there is more than one — the project is pinning mixed-era framework
+     * packages against each other.
+     */
+    disagreeing: string[];
+}
+/**
+ * Find `@rebasepro/*` dependencies pinned to a version older than this CLI.
+ *
+ * This is the only place a developer can be told. In development, every
+ * `@rebasepro/*` resolves through pnpm's `link:`/`workspace:` overrides to the
+ * checkout, so the version STRINGS in package.json are never exercised — the
+ * project runs fine locally on whatever is on disk, and the declared numbers are
+ * first honoured when the runtime npm-installs them from a bundle in the cloud.
+ * A project scaffolded at 0.10.0 therefore keeps working on a developer's
+ * machine indefinitely while being, in the cloud, a 0.10.0 driver.
+ *
+ * That matters because the image supplies only `@rebasepro/server`; the database
+ * driver comes from these declarations and a newer runtime never updates it.
+ * Every package.json is scanned, `dependencies` and `devDependencies` both,
+ * because they have to be bumped together and the one that gets forgotten is the
+ * one nobody looks at.
+ */
+export declare function detectFrameworkDepDrift(projectRoot: string, cliVersion: string): FrameworkDepDrift;
 /**
  * Rewrite relative import specifiers in emitted JavaScript so Node can resolve them.
  *
@@ -182,3 +217,4 @@ export declare function buildStaticBundle(options: {
     manifest: RebaseBundleManifest;
     fileCount: number;
 };
+export declare function resolveCliVersion(): string;

package/dist/commands/cloud/context.d.ts

@@ -24,7 +24,32 @@ export declare function requireClient(rawArgs: string[]): Promise<{
     client: CloudClient;
     url: string;
 }>;
+/** One place a deploy can actually land, as the control plane describes it. */
+export interface DeployTarget {
+    clusterId?: string | null;
+    provider: string;
+    region?: string;
+    label?: string;
+    baseDomain?: string;
+}
+export interface PlatformConfig {
+    tenantBaseDomain?: string;
+    deployTargets?: DeployTarget[];
+}
+export declare function fetchPlatformConfig(client: CloudClient, url: string): Promise<PlatformConfig | undefined>;
+/**
+ * The base domain tenant projects are served at, derived from the same
+ * TENANT_BASE_DOMAIN the ingress and the console read (see
+ * saas/backend/src/utils/tenant-domain.ts).
+ */
 export declare function fetchTenantBaseDomain(client: CloudClient, url: string): Promise<string | undefined>;
+/**
+ * The infrastructure a deploy for this control plane would ACTUALLY use, in the
+ * resolver's own preference order (saas/backend/src/k8s/resolve.ts).
+ *
+ * @returns the targets, or `undefined` when the control plane cannot say.
+ */
+export declare function fetchDeployTargets(client: CloudClient, url: string): Promise<DeployTarget[] | undefined>;
 /**
  * Public host for a project — `<subdomain>.<base>`, or the bare subdomain when
  * the base domain is unknown.
@@ -84,6 +109,39 @@ export interface ProjectLink {
 export declare function readLink(cwd?: string): ProjectLink | null;
 export declare function writeLink(link: ProjectLink, cwd?: string): void;
 export declare function removeLink(cwd?: string): boolean;
+/**
+ * Flags that may appear anywhere on a `rebase cloud` line, including *before*
+ * the resource group.
+ *
+ * They have to be declared wherever positionals are resolved, because `arg`'s
+ * `permissive: true` does not merely tolerate an undeclared flag — it pushes it
+ * into `_` alongside the positionals, and for a flag that takes a value it
+ * pushes the value in too. So `cloud --project acme storage create` parsed
+ * without this spec yields `_` of `["--project", "acme", "storage", "create"]`,
+ * and the group reads as `"acme"`: a real project name, in the group position,
+ * dispatching to nothing. Skipping tokens that start with `-` does not save you
+ * there — the damage is the orphaned value, which looks exactly like a
+ * positional.
+ *
+ * Only genuinely global flags belong here. Group-specific ones (`--bucket`,
+ * `--region`, …) are declared by the handler that owns them and always follow
+ * the group, so they cannot shift the group or action.
+ *
+ * `-p` is `--project` in eighteen places and `--password` in `login`. That
+ * ambiguity does not matter to the one caller that reads this spec: it resolves
+ * positionals and never looks at a flag's value, so all it needs to know is
+ * that `-p` takes one. Anything that wants the value must keep declaring it
+ * itself, with the meaning its own command gives it.
+ */
+export declare const GLOBAL_CLOUD_FLAGS: {
+    readonly "--json": BooleanConstructor;
+    readonly "--yes": BooleanConstructor;
+    readonly "--help": BooleanConstructor;
+    readonly "--project": StringConstructor;
+    readonly "-p": "--project";
+    readonly "-y": "--yes";
+    readonly "-h": "--help";
+};
 /**
  * The raw project reference to operate on: explicit `--project` flag wins,
  * otherwise the linked project. Exits with guidance when neither is present.
@@ -134,6 +192,26 @@ export declare function printJson(value: unknown): void;
  * call is what guarantees a command can never print a table AND a JSON blob.
  */
 export declare function emit(human: () => void, json: unknown): void;
+/**
+ * Print a warning (+ optional hint) — in every output mode, always to stderr.
+ *
+ * `emit` is for a command's *result*, and JSON mode legitimately replaces the
+ * human rendering of one. A warning is not a result: it says the command is
+ * about to do something the caller may not have meant, and that is exactly as
+ * true when the output is piped. Gating one on `!isJsonMode()` deleted it
+ * precisely where nobody was watching the terminal — a `--source` deploy ejected
+ * a live project off the managed runtime and said so only to a TTY that wasn't
+ * there.
+ *
+ * stdout carries the JSON value and nothing else, so warnings go to stderr:
+ * a machine parser reading stdout cannot be corrupted by one. Only the
+ * *formatting* may depend on the mode — colour and indentation for a terminal,
+ * plain ASCII otherwise. Whether a warning is emitted at all may not.
+ *
+ * Anything a caller might branch on belongs in the JSON payload as well; stderr
+ * is for whoever reads the transcript afterwards.
+ */
+export declare function warn(message: string, hint?: string): void;
 /** Print an error (+ optional hint) and exit non-zero. Never returns. */
 export declare function fail(message: string, hint?: string, code?: string): never;
 /**

package/dist/commands/cloud/deploy.d.ts

@@ -46,5 +46,69 @@ export declare function timeAgo(value: string | Date | undefined, now: Date): st
 export declare function isManagedProject(project: DeployProjectRow | undefined, latest: DeploySourceRow | undefined): boolean;
 /** What a `deploy` with nothing attached will build, in the words to print. */
 export declare function planBareDeploy(project: DeployProjectRow | undefined, latest: DeploySourceRow | undefined, now: Date): BareDeployPlan;
+/**
+ * A warning attached to a deploy: printed for the human, carried in the JSON.
+ *
+ * `code` is the stable half — the message is prose and will be reworded, so it
+ * is the code that CI or an agent branches on.
+ */
+export interface DeployWarning {
+    code: string;
+    message: string;
+    hint?: string;
+}
+/** `code` of the warning below, and the field name it sets in the payload. */
+export declare const EJECTS_MANAGED_RUNTIME = "ejects_managed_runtime";
+/** The one sentence that says a source build undoes `runtimeMode: managed`. */
+export declare function ejectWarning(projectRef: string): DeployWarning;
+/** How a container-image deploy was asked for — the input to both rules below. */
+export interface EjectContext {
+    /** The project currently runs on the managed runtime. */
+    managed: boolean;
+    /** `--source` was passed: build this directory. */
+    source: boolean;
+    /** `--force` was passed: eject on purpose. */
+    force: boolean;
+}
+/**
+ * Why a container-image deploy of a managed project is refused — or `undefined`
+ * to let it through.
+ *
+ * Every path below this point builds a container image, and a successful one
+ * sets `runtimeMode: "custom"` server-side. So the question is never "which flag
+ * was used" but "did the caller ask to leave the managed runtime", and only
+ * `--force` answers it.
+ *
+ * `--source` used to be read as answering it too, on the theory that uploading a
+ * build context is self-evidently a deliberate eject. It is not: `--source`
+ * picks *which source* gets built — this directory, rather than the stale
+ * archive the control plane is holding — and the eject is a side effect of the
+ * answer. That is exactly how a live project got flipped to `custom` by someone
+ * whose actual intent was "deploy what I have here", and it is the same
+ * ignorance the bare form is refused for. Same ignorance, same refusal.
+ */
+export declare function ejectRefusal(opts: EjectContext, projectRef: string): {
+    message: string;
+    hint: string;
+    code: string;
+} | undefined;
+/**
+ * Which warnings a container-image deploy has earned.
+ *
+ * Pure, and separate from the printing, because the printing is what went
+ * wrong: the eject warning used to be written inline behind `!isJsonMode()`, so
+ * the fact that a deploy ejects a managed project existed only as a side effect
+ * of a TTY being attached. Deciding here, emitting once at the call site, means
+ * the decision cannot be output-mode-dependent again.
+ *
+ * The condition is just `managed`: anything reaching this point is a container
+ * image build that `ejectRefusal` has already let through, and on a managed
+ * project that is an eject however it was spelled. A caller who passed `--force`
+ * knows — the warning is for the transcript and the payload, which is what
+ * anyone reviewing the deploy afterwards actually reads.
+ */
+export declare function deployWarnings(opts: EjectContext, projectRef: string): DeployWarning[];
+/** The warning half of a deploy's JSON payload — merged into whatever it emits. */
+export declare function warningPayload(warnings: DeployWarning[]): Record<string, unknown>;
 export declare function deployCommand(rawArgs: string[], projectRef: string): Promise<void>;
 export declare function logsCommand(rawArgs: string[], projectRef: string): Promise<void>;

package/dist/commands/cloud/index.d.ts

@@ -1 +1,24 @@
+/**
+ * Positional tokens after `rebase cloud` (group, action, …).
+ *
+ * Two things stop a flag being mistaken for the group. `GLOBAL_CLOUD_FLAGS` is
+ * declared so `arg` *consumes* the flags that may precede it — critically
+ * together with their values, which is the half that filtering cannot do. The
+ * leading-`-` skip then covers a flag nobody declared, so an unrecognised
+ * boolean shifts nothing.
+ *
+ * Only leading tokens are skipped: past the group and action, an undeclared
+ * flag and its value are somebody else's positionals and none of our business.
+ * A flag this file has never heard of, that takes a value, placed before the
+ * group, is the one shape still unresolvable here — there is no way to know
+ * whether the token after it is its value or the group, and guessing either way
+ * is worse than the handler reporting an unknown group.
+ *
+ * Exported so its tests can drive the real thing. The dispatch test used to
+ * re-implement it locally as `slice(3).filter(a => !a.startsWith("-"))` — which
+ * filtered flags, while this function did not — so the test asserted the
+ * behaviour we wanted against a copy that had it, and stayed green for as long
+ * as the real dispatcher was broken.
+ */
+export declare function positionals(rawArgs: string[]): string[];
 export declare function cloudCommand(subcommand: string | undefined, rawArgs: string[]): Promise<void>;

package/dist/commands/cloud/projects.d.ts

@@ -1,5 +1,36 @@
-import { type CloudClient } from "./context";
+import { type DeployTarget, type CloudClient } from "./context";
 export declare function listProjects(rawArgs: string[]): Promise<void>;
+/**
+ * Where this project says it runs.
+ *
+ * `provider`/`region` are a *request*: no code downstream reads them to pick a
+ * deploy target — that comes from the project's cluster record or the ambient
+ * in-cluster context (saas/backend/src/k8s/resolve.ts). So a wrong value here is
+ * never contradicted by a failure; it just sits in the record. The CLI used to
+ * default to `hetzner`/`nbg1` unconditionally, which is how projects running on
+ * our GKE cluster came to describe themselves as Hetzner in the console — and
+ * `provider` is half the Stripe compute lookup key (`compute_<provider>_<vmSize>`),
+ * so that is a mispricing, not a cosmetic slip.
+ *
+ * The control plane already publishes the infrastructure that actually exists,
+ * and the console's create wizard reads it. Ask the same question here.
+ *
+ * Exported for tests: the decision is pure, so it can be pinned without a
+ * control plane. The fetching and the exit live in `resolveRequestedTarget`.
+ *
+ * @param requested `--provider`, if the caller named one. An explicit flag wins:
+ *   it is the caller stating intent, and `deploy` corrects the record anyway.
+ * @param targets What the control plane says exists, or `undefined` when it
+ *   cannot say — an older deployment with no `platform-config`, or a failed
+ *   request. That is different from an empty list, which is a control plane
+ *   stating it has no infrastructure at all.
+ * @returns the target to record, or `null` when the control plane answered that
+ *   there is none.
+ */
+export declare function chooseRequestedTarget(requested: string | undefined, targets: DeployTarget[] | undefined): {
+    provider: string;
+    region?: string;
+} | null;
 export declare function createProject(rawArgs: string[]): Promise<void>;
 export declare function projectInfo(rawArgs: string[], projectRef: string): Promise<void>;
 export declare function deleteProject(rawArgs: string[], projectRef: string): Promise<void>;

package/dist/commands/dev.d.ts

@@ -1 +1,17 @@
+/** Well-known filename the backend writes its actual port to. */
+export declare const DEV_PORT_FILENAME = ".rebase-dev-port";
+/**
+ * Compute a deterministic port from the project root path.
+ * Range: 3001–3999 (avoids privileged ports and common services).
+ * Two different project directories will almost always get different ports.
+ */
+export declare function getProjectPort(projectRoot: string): number;
+/**
+ * Resolve the best starting port for this project:
+ * 1. Explicit --port flag (highest priority)
+ * 2. PORT env var
+ * 3. Previously used port from .rebase-dev-port (port affinity across restarts)
+ * 4. Deterministic hash from project path (unique per project)
+ */
+export declare function resolveStartPort(projectRoot: string, explicitPort?: number): number;
 export declare function devCommand(rawArgs: string[]): Promise<void>;

package/dist/commands/telemetry.d.ts

@@ -0,0 +1,9 @@
+/**
+ * `rebase telemetry` — the command that makes the rest of it inspectable.
+ *
+ * The whole subsystem asks for trust it cannot otherwise earn, and the cheapest
+ * way to earn it is to stop describing the payload and print it. `show` runs
+ * the same builder the sender uses, so what appears here is what would go, not
+ * a documentation comment that quietly fell out of date two releases ago.
+ */
+export declare function telemetryCommand(rawArgs: string[]): Promise<void>;