kanbo-cli 0.1.3 → 0.2.0

package/dist/cli.cjs

@@ -3084,7 +3084,7 @@ var import_picocolors = /* @__PURE__ */ __toESM((/* @__PURE__ */ __commonJSMin((
 * `package.version`. A build carries it as a literal; running from source
 * (tests, `tsx`) reads the package's own `package.json` instead.
 */
-const KANBO_PACKAGE_VERSION = "0.1.3";
+const KANBO_PACKAGE_VERSION = "0.2.0";
 /**
 * Every exit code this tool promises, and what it means — the capabilities
 * manifest's `exitCodes` reads this rather than naming the codes again. `0` and
@@ -10789,15 +10789,12 @@ async function approve(store, issueId, input, actor, scope) {
 * leaves even says "returned by you".
 */
 async function returnCard(store, issueId, input, actor, scope) {
-	if (actor.kind !== "user") throw new BoardError("board_return_requires_user", {
-		issueId,
-		actorKind: actor.kind
-	});
 	return await runBoardWrite(store, async ({ tx }) => {
+		await ensureDefaultColumns(tx, (await requireCard$1(tx, issueId)).workspaceId, { tx });
+		const target = await assertReturnable(tx, issueId, input, actor);
 		const card = await requireCard$1(tx, issueId);
-		const columns = await ensureDefaultColumns(tx, card.workspaceId, { tx });
 		const updates = {
-			statusId: (input.toStatusName ? await requireColumn(tx, card.workspaceId, input.toStatusName) : previousColumn(columns, card)).id,
+			statusId: target.id,
 			waitingFor: null,
 			updatedAt: currentUnixSeconds()
 		};
@@ -10818,6 +10815,27 @@ async function returnCard(store, issueId, input, actor, scope) {
 		};
 	}, scope);
 }
+/**
+* Every reason `returnCard` would refuse, asked on reads alone, and the column
+* the card would go back to when none applies.
+*
+* `returnCard` asks exactly this inside its write. A caller that has to undo
+* something of its own before the return — stop listening for the approval the
+* card is waiting on, say — asks it first, so a return the board refuses leaves
+* that untouched. The refusals are the same `BoardError` codes: an actor who is
+* not a person (`board_return_requires_user`), a card that is not there
+* (`issue_not_found`), a named column the workspace does not hold
+* (`issue_status_not_found`), and a card already in the first column with no
+* column named (`board_return_no_previous_column`).
+*/
+async function assertReturnable(store, issueId, input, actor) {
+	if (actor.kind !== "user") throw new BoardError("board_return_requires_user", {
+		issueId,
+		actorKind: actor.kind
+	});
+	const card = await requireCard$1(store, issueId);
+	return input.toStatusName ? await requireColumn(store, card.workspaceId, input.toStatusName) : previousColumn(await listColumns(store, card.workspaceId), card);
+}
 /** Write a comment on the card, as the actor unless the caller names another author. */
 async function addComment(store, input, actor, scope) {
 	return await runBoardWrite(store, async ({ tx }) => {
@@ -11695,6 +11713,7 @@ function createBoardOps(store) {
 		waitApproval: async (issueId, input, actor, scope) => await waitApproval(store, issueId, input, actor, scope),
 		approve: async (issueId, input, actor, scope) => await approve(store, issueId, input, actor, scope),
 		returnCard: async (issueId, input, actor, scope) => await returnCard(store, issueId, input, actor, scope),
+		assertReturnable: async (issueId, input, actor) => await assertReturnable(store, issueId, input, actor),
 		readApproval: async (issueId) => await readApproval(store, issueId),
 		linkPullRequest: async (issueId, urlOrRef, actor, scope) => await linkPullRequest(store, issueId, urlOrRef, actor, scope),
 		unlinkPullRequest: async (issueId, linkId, actor, scope) => await unlinkPullRequest(store, issueId, linkId, actor, scope),
@@ -43395,21 +43414,6 @@ function sendBoardPage(response, pathname, assets) {
 * reason turned around: the answer will be yes once the card meets them, and
 * `details.unmet` says what is missing.
 *
-* The five failures about where a board is kept read the same rule. A database
-* that holds no board this build can speak for is 409 — the setting is there,
-* the board behind it is not yet, and `board-storage prepare` is what settles
-* that — and one nobody could reach at all is 503, because nothing about the
-* request was wrong. A workspace naming no board at all is 404: the caller
-* asked about a setting that is not there. A board still holding cards when
-* somebody asks to point the workspace elsewhere is 409, because the answer
-* would have been yes on an empty board and `board-storage migrate` is what
-* makes it one. An operation naming cards from two different boards is 400: no
-* transaction reaches both, so the caller asked for something that cannot be
-* done. A copy that would land on a card number or a column name the target
-* already holds under a different id is 409 for the same reason the others
-* are: the answer would have been yes on a target that did not hold that
-* board, and `details.conflicts` names every value that clashed.
-*
 * A failure may also be renamed here: the package names its codes after the
 * board, while every error the `/issues` routes answer with is named after the
 * issue, and a client matching on `issue_*` should not have to learn a second
@@ -43431,10 +43435,6 @@ const BOARD_ERROR_RESPONSES = {
 		message: "The card does not meet what that column asks for yet",
 		code: "issue_column_rules_unmet"
 	},
-	board_copy_conflict: {
-		status: 409,
-		message: "That board already holds cards or columns of this workspace under the same numbers or names; nothing was copied"
-	},
 	board_entry_rule_invalid: {
 		status: 400,
 		message: "Unknown column entry rule"
@@ -43443,10 +43443,6 @@ const BOARD_ERROR_RESPONSES = {
 		status: 400,
 		message: "Execution mode cannot change once the card has been launched"
 	},
-	board_mixed_workspaces: {
-		status: 400,
-		message: "Those cards are not all on the same board"
-	},
 	board_pull_request_invalid: {
 		status: 400,
 		message: "Not a GitHub pull request: give its URL or owner/repo#number"
@@ -43506,22 +43502,6 @@ const BOARD_ERROR_RESPONSES = {
 		status: 400,
 		message: "A sprint cannot start after it is due"
 	},
-	board_storage_not_configured: {
-		status: 404,
-		message: "This workspace names no board storage"
-	},
-	board_storage_not_empty: {
-		status: 409,
-		message: "That board still holds cards of this workspace; move them with migrate before changing where the board is kept"
-	},
-	board_storage_not_prepared: {
-		status: 409,
-		message: "That board has not been prepared yet"
-	},
-	board_storage_unavailable: {
-		status: 503,
-		message: "That board could not be reached"
-	},
 	issue_comment_delete_requires_user: {
 		status: 403,
 		message: "Only a person can delete a comment the board wrote"