@levr-one/cli 0.7.7 → 0.8.0

package/dist/cli.js

@@ -2,6 +2,9 @@
 import { COMPLETION_SHELLS } from "./completionHandler-O2G_WZLf.js";
 import { buildApplication, buildCommand, buildRouteMap, proposeCompletions, run, text_en } from "@stricli/core";
 import chalk from "chalk";
+import { existsSync, readFileSync } from "node:fs";
+import { isAbsolute, resolve } from "node:path";
+import { parse } from "dotenv";
 
 //#region src/utils/logger.ts
 var Logger = class {
@@ -210,7 +213,7 @@ Examples:
 		aliases: { d: "device-code" }
 	},
 	loader: async () => {
-		const { loginHandler } = await import("./loginHandler-swasPGVn.js");
+		const { loginHandler } = await import("./loginHandler-BE9JX12M.js");
 		return loginHandler;
 	}
 });
@@ -249,7 +252,7 @@ Examples:
 	},
 	parameters: {},
 	loader: async () => {
-		const { statusHandler } = await import("./statusHandler-DBC29FE4.js");
+		const { statusHandler } = await import("./statusHandler-CyhoTEEI.js");
 		return statusHandler;
 	}
 });
@@ -361,7 +364,7 @@ Examples:
 		}
 	},
 	loader: async () => {
-		const { pushHandler } = await import("./pushHandler-DD18UAh3.js");
+		const { pushHandler } = await import("./pushHandler-BukCooiv.js");
 		return pushHandler;
 	}
 });
@@ -382,9 +385,9 @@ Required-field rule: a column must map to test_name or the commit is
 rejected — interactive mode will ask; non-interactive runs exit 1.
 
 Examples:
-  levr import ./testrail-export.csv --team-id <uuid>
-  levr import ./cases.xlsx --team-id <uuid> --map "Title=test_name"
-  levr import --sheets-url "https://docs.google.com/spreadsheets/d/..." --team-id <uuid> --yes
+  levr import ./testrail-export.csv --team-key ENG
+  levr import ./cases.xlsx --team-key ENG --map "Title=test_name"
+  levr import --sheets-url "https://docs.google.com/spreadsheets/d/..." --team-key ENG --yes
   levr import ./cases.csv --team-id <uuid> --save-mapping mapping.json
   levr import ./cases.csv --team-id <uuid> --mapping-file mapping.json --yes   # CI replay`
 	},
@@ -409,9 +412,16 @@ Examples:
 			"team-id": {
 				kind: "parsed",
 				parse: String,
-				brief: "Team the imported test cases will belong to",
+				brief: "Team doing the import, by UUID. Optional: use --team-key instead, or omit both when the workspace has one team",
 				placeholder: "uuid",
-				optional: false
+				optional: true
+			},
+			"team-key": {
+				kind: "parsed",
+				parse: String,
+				brief: "Team doing the import, by key (e.g. ENG). Case-insensitive. Alternative to --team-id",
+				placeholder: "key",
+				optional: true
 			},
 			"sheets-url": {
 				kind: "parsed",
@@ -466,6 +476,7 @@ Examples:
 		aliases: {
 			w: "workspace-id",
 			t: "team-id",
+			k: "team-key",
 			f: "format",
 			m: "map",
 			y: "yes",
@@ -473,7 +484,7 @@ Examples:
 		}
 	},
 	loader: async () => {
-		const { importHandler } = await import("./importHandler-B2zyKkUX.js");
+		const { importHandler } = await import("./importHandler-zfgrGCyh.js");
 		return importHandler;
 	}
 });
@@ -494,7 +505,7 @@ Examples:
 	},
 	parameters: {},
 	loader: async () => {
-		const { listHandler } = await import("./listHandler-BaXnaQkv.js");
+		const { listHandler } = await import("./listHandler-B5KqRcYG.js");
 		return listHandler;
 	}
 });
@@ -527,7 +538,7 @@ Examples:
 		flags: {}
 	},
 	loader: async () => {
-		const { selectHandler } = await import("./selectHandler-CH0fcc2P.js");
+		const { selectHandler } = await import("./selectHandler-CXwxIKL9.js");
 		return selectHandler;
 	}
 });
@@ -551,7 +562,7 @@ Examples:
 
 //#endregion
 //#region package.json
-var version = "0.7.7";
+var version = "0.8.0";
 
 //#endregion
 //#region src/app.ts
@@ -622,9 +633,88 @@ async function proposeCompletionLines(rawArgs, compLine, context) {
 	}
 }
 
+//#endregion
+//#region src/utils/load-env-file.ts
+/**
+* Only `LEVR_`-prefixed keys are taken from a `.env` file.
+*
+* `dotenv.config()` would merge EVERY key into `process.env`, which is more
+* than this CLI needs and more than a user consents to by running it in a
+* directory. A checkout's `.env` routinely carries things that change how Node
+* itself behaves — `NODE_TLS_REJECT_UNAUTHORIZED`, `HTTP_PROXY`,
+* `NODE_OPTIONS` — and none of them are ours to apply. Parsing and filtering
+* means an unrelated project's `.env` cannot reach this process at all; only
+* keys deliberately namespaced for this CLI have any effect.
+*/
+const LEVR_KEY = /^LEVR_[A-Z0-9_]*$/;
+/**
+* Where to look. `LEVR_ENV_FILE` names an explicit file (absolute, or relative
+* to cwd); otherwise `.env` in the working directory.
+*/
+function envFilePath(cwd, env) {
+	const explicit = env["LEVR_ENV_FILE"];
+	if (explicit) return isAbsolute(explicit) ? explicit : resolve(cwd, explicit);
+	return resolve(cwd, ".env");
+}
+/**
+* Merge `LEVR_*` variables from a `.env` file into `process.env`.
+*
+* A real environment variable ALWAYS wins: an already-set key is left alone,
+* so `LEVR_URL=… levr push` still overrides the file, and CI variables are
+* never shadowed by a checked-in `.env`. This matches dotenv's own precedence
+* and is the behaviour anyone who has used a `.env` expects — the file is a
+* default, not an override.
+*
+* Missing or unreadable files are not an error: `.env` is optional by nature,
+* and a CLI that refused to start because a directory has no `.env` would be
+* broken for every user who does not use one. An explicitly requested
+* `LEVR_ENV_FILE` that does not exist IS reported, because naming a file and
+* getting silence is the one case where quiet failure hides a real mistake.
+*/
+function loadEnvFile(cwd = process.cwd(), env = process.env) {
+	const path = envFilePath(cwd, env);
+	const explicitlyRequested = Boolean(env["LEVR_ENV_FILE"]);
+	if (!existsSync(path)) {
+		if (explicitlyRequested) throw new Error(`LEVR_ENV_FILE points at "${path}", which does not exist.`);
+		return {
+			path: null,
+			applied: []
+		};
+	}
+	let parsed;
+	try {
+		parsed = parse(readFileSync(path, "utf8"));
+	} catch (error) {
+		if (explicitlyRequested) throw error;
+		return {
+			path: null,
+			applied: []
+		};
+	}
+	const applied = [];
+	for (const [key, value] of Object.entries(parsed)) {
+		if (!LEVR_KEY.test(key)) continue;
+		if (env[key] !== void 0 && env[key] !== "") continue;
+		env[key] = value;
+		applied.push(key);
+	}
+	return {
+		path,
+		applied
+	};
+}
+
 //#endregion
 //#region src/bin/cli.ts
 const argv = process.argv.slice(2);
+try {
+	loadEnvFile();
+} catch (err) {
+	if (argv[0] !== "__complete") {
+		process.stderr.write(`${err instanceof Error ? err.message : String(err)}\n`);
+		process.exit(1);
+	}
+}
 if (argv[0] === "__complete") for (const line of await proposeCompletionLines(argv, process.env["COMP_LINE"], buildContext(process))) process.stdout.write(`${line}\n`);
 else {
 	let savedExitCode;

package/dist/{importHandler-B2zyKkUX.js → importHandler-zfgrGCyh.js}

@@ -1,7 +1,7 @@
-import { getApiUrl } from "./env-CHeKHu5S.js";
-import { configureClient, testCaseImportCommitV1, testCaseImportPreviewV1 } from "./sdk-client-BLUjDoCb.js";
+import { getApiUrl, getTeamId } from "./env-CHeKHu5S.js";
+import { authGetProfileV1, configureClient, teamFindAllV1, testCaseImportCommitV1, testCaseImportPreviewV1 } from "./sdk-client-CATjlmyX.js";
 import "./workspace-store-DDOxnut1.js";
-import { resolveWorkspace } from "./resolve-workspace-anULXxRH.js";
+import { resolveWorkspace } from "./resolve-workspace-NupdkCvd.js";
 import "./token-refresh-Cu5RpkLJ.js";
 import { resolveToken } from "./resolve-token-DbQsmn03.js";
 import chalk from "chalk";
@@ -11,8 +11,167 @@ import { readFile, writeFile } from "node:fs/promises";
 import { createInterface } from "node:readline/promises";
 import ora from "ora";
 
+//#region src/commands/resolve-team.ts
+/**
+* The importing team — NOT the same thing as the `teams` column in the file.
+*
+* `teams` is OWNERSHIP: which teams the imported tests belong to. The
+* importing team is who is doing the import, and the backend uses it as
+* `primaryTeamId` for decisions the file cannot answer:
+*
+*   - where an UNQUALIFIED `estimate` column lands (ENG-4396) — a bare
+*     `estimate` is not team-neutral, it sizes one specific team;
+*   - the guaranteed owner when the file names no teams at all
+*     (`ensureMinimumOfOneOwner`);
+*   - which team is excluded from the "extra teams" set.
+*
+* So it is not redundant with the file. It IS unreasonable to demand a UUID
+* for it when the file speaks in keys, which is what `--team-key` fixes.
+*
+* Deliberately NOT inferred from the file's first `teams` value. That value
+* decides where an unqualified estimate lands, and row order is a proxy for
+* that question, not an answer to it — a reordered export would silently size
+* a different team. When it cannot be determined, ask.
+*/
+async function fetchTeams() {
+	const PAGE_SIZE = 200;
+	const MAX_PAGES = 25;
+	const out = [];
+	for (let page = 1; page <= MAX_PAGES; page++) {
+		const result = await teamFindAllV1({ query: {
+			limit: PAGE_SIZE,
+			page
+		} });
+		if (result.error || !result.data) throw new Error(`Failed to list teams (${describeError(result.error)}).`);
+		const body = result.data;
+		const rows = body.data ?? [];
+		for (const r of rows) if (r.id && r.key) out.push({
+			id: r.id,
+			key: r.key,
+			name: r.name ?? r.key
+		});
+		const totalPages = body.meta?.totalPages;
+		if (rows.length === 0) break;
+		if (totalPages !== void 0 ? page >= totalPages : rows.length < PAGE_SIZE) break;
+		if (page === MAX_PAGES) process.stderr.write(`[levr] Warning: stopped after ${MAX_PAGES} pages of teams; a --team-key may not resolve.\n`);
+	}
+	return out;
+}
+/**
+* The caller's ACTIVE team, when it belongs to this workspace.
+*
+* `active_team_id` is the user's HOME team, not "the team they are looking
+* at" — a distinction `apps/client/CLAUDE.md` is emphatic about. That makes it
+* a fair default for "who is importing" and a poor one for anything scoped to
+* a view, so it is used here and reported, never assumed silently.
+*
+* MUST be checked against this workspace's teams. A user can belong to several
+* workspaces while their home team lives in one of them; adopting it unchecked
+* would import into a team that is not in the workspace being imported to.
+*
+* A failure here is not fatal — it just means no default. The explicit flags
+* and the single-team case still work without a profile call succeeding.
+*/
+async function activeTeamIn(teams) {
+	let activeId;
+	try {
+		const result = await authGetProfileV1();
+		if (result.error || !result.data) return null;
+		activeId = result.data.active_team_id ?? void 0;
+	} catch {
+		return null;
+	}
+	if (!activeId) return null;
+	return teams.some((t) => t.id === activeId) ? activeId : null;
+}
+/**
+* F-012: `JSON.stringify(new Error("boom"))` is `"{}"` — Error's own fields are
+* non-enumerable. The detail that was meant to end the misdirection produced
+* "Failed to list teams ({})", which is worse than saying nothing.
+*/
+function describeError(error) {
+	if (error === void 0 || error === null) return "no data returned";
+	if (error instanceof Error) return error.message;
+	if (typeof error === "string") return error;
+	try {
+		const json = JSON.stringify(error);
+		if (json !== void 0 && json !== "{}") return json;
+	} catch {}
+	return "unserialisable error";
+}
+function describe(teams) {
+	return teams.map((t) => `  ${t.key}  ${t.name}`).join("\n");
+}
+/**
+* Resolve the importing team from `--team-id`, `--team-key`, or the workspace.
+*
+* Key matching is case-insensitive: the file writes `ENG`, and a user typing
+* `eng` means the same team. Errors list the valid keys, because "no team
+* matched" without saying what would have is the least useful thing this can
+* print.
+*/
+async function resolveTeamId(teamId, teamKey, envTeamId) {
+	if (teamId && teamKey) throw new Error("Pass either --team-id or --team-key, not both — they can disagree, and there is no sensible winner.");
+	if (teamId) return {
+		teamId,
+		source: "team-id"
+	};
+	const teams = await fetchTeams();
+	if (teams.length === 0) throw new Error("No teams found in this workspace.");
+	if (teamKey) {
+		const wanted = teamKey.trim().toLowerCase();
+		const matches = teams.filter((t) => t.key.toLowerCase() === wanted);
+		if (matches.length === 1) return {
+			teamId: matches[0].id,
+			source: "team-key"
+		};
+		if (matches.length === 0) throw new Error(`No team with key "${teamKey}" in this workspace. Available:\n${describe(teams)}`);
+		throw new Error(`More than one team has key "${teamKey}" in this workspace — refusing to guess.`);
+	}
+	if (envTeamId) {
+		if (teams.some((t) => t.id === envTeamId)) return {
+			teamId: envTeamId,
+			source: "env"
+		};
+		throw new Error(`LEVR_TEAM_ID (${envTeamId}) is not a team in this workspace. Available:\n${describe(teams)}`);
+	}
+	const active = await activeTeamIn(teams);
+	if (active) return {
+		teamId: active,
+		source: "active-team"
+	};
+	if (teams.length === 1) return {
+		teamId: teams[0].id,
+		source: "only-team"
+	};
+	throw new Error(`--team-key or --team-id is required: this workspace has ${teams.length} teams and you have no active team here.\n${describe(teams)}`);
+}
+
+//#endregion
 //#region src/utils/import-mapping.ts
-const IMPORT_TARGETS = [
+/**
+* Every import target the CLI will accept for `--map`.
+*
+* ENG-3857 shipped a `teams` target on the server and in the web client, and
+* this list was left behind — so `levr import --map "Team=teams"` threw
+* "Invalid --map target" and team ownership was simply unreachable from the
+* CLI. Nothing caught it: this is the THIRD hand-maintained copy of the
+* target list (backend `IMPORT_TARGET_PROPERTIES`, client
+* `TEST_CASE_KNOWN_PROPERTIES`, here), and while the ELEMENT type is derived
+* from the SDK, TypeScript cannot see that an array literal omits a member of
+* a union — `ImportTarget[]` is satisfied by any subset, including `[]`.
+*
+* So the literal is `as const` and the exhaustiveness assertions below turn a
+* missing target into a COMPILE error naming it, rather than a runtime
+* rejection the user discovers. Adding a target server-side now breaks this
+* build until it is listed here.
+*
+* Caveat worth knowing: `@levr/sdk` resolves to `dist/`, so the union this
+* is checked against is only as fresh as the last SDK build. A regenerated
+* but unbuilt SDK delays detection to the next build — it does not restore
+* the old failure, where the omission was invisible indefinitely.
+*/
+const IMPORT_TARGET_LIST = [
 	"row_type",
 	"test_id",
 	"test_key",
@@ -29,6 +188,7 @@ const IMPORT_TARGETS = [
 	"is_automated",
 	"assignee_email",
 	"labels",
+	"teams",
 	"attachment_filenames",
 	"data_set_names",
 	"sequence",
@@ -42,6 +202,7 @@ const IMPORT_TARGETS = [
 	"steps",
 	"preconditions"
 ];
+const IMPORT_TARGETS = IMPORT_TARGET_LIST;
 const REQUIRED_TARGETS = ["test_name"];
 /**
 * Parse a --map value: "Source Column=target", "Source Column=" (drop),
@@ -166,16 +327,33 @@ async function importHandler(flags, file) {
 		this.logger.debug(`API:  ${getApiUrl()}`);
 	}
 	if (auth.type === "jwt") {
-		if (!await resolveWorkspace.call(this, flags["workspace-id"])) {
+		let workspace;
+		try {
+			workspace = await resolveWorkspace.call(this, flags["workspace-id"]);
+		} catch (err) {
+			this.logger.error(err instanceof Error ? err.message : "Could not resolve the workspace.");
 			this.process.exitCode = 1;
 			return;
 		}
+		configureClient(auth, workspace.workspaceId);
+	}
+	let teamId;
+	try {
+		const team = await resolveTeamId(flags["team-id"], flags["team-key"], getTeamId());
+		teamId = team.teamId;
+		if (team.source === "active-team") this.logger.info(`Importing as your active team (${teamId}).`);
+		else if (team.source === "env") this.logger.info(`Importing as LEVR_TEAM_ID (${teamId}).`);
+		else if (team.source === "only-team") this.logger.info(`Importing as the only team in this workspace.`);
+	} catch (err) {
+		this.logger.error(err instanceof Error ? err.message : "Could not resolve the team.");
+		this.process.exitCode = 1;
+		return;
 	}
 	const spinner = ora("Uploading and analyzing the file…").start();
 	let preview;
 	try {
 		const body = {
-			team_id: flags["team-id"],
+			team_id: teamId,
 			format: flags.format,
 			sheets_url: flags["sheets-url"]
 		};

package/dist/{listHandler-BaXnaQkv.js → listHandler-B5KqRcYG.js}

@@ -1,5 +1,5 @@
 import "./env-CHeKHu5S.js";
-import { authGetSitesV1, configureClient } from "./sdk-client-BLUjDoCb.js";
+import { authGetSitesV1, configureClient } from "./sdk-client-CATjlmyX.js";
 import { loadWorkspace } from "./workspace-store-DDOxnut1.js";
 import "./token-refresh-Cu5RpkLJ.js";
 import { resolveToken } from "./resolve-token-DbQsmn03.js";