kanbo-cli 0.1.3 → 0.2.1

package/dist/lib/{manifest-mkDJmxt2.mjs → manifest-DHV2fRdO.mjs}

@@ -1903,15 +1903,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()
 		};
@@ -1932,6 +1929,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 }) => {
@@ -2809,6 +2827,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),
@@ -4311,7 +4330,7 @@ dirname(fileURLToPath(import.meta.url));
 * `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.1";
 //#endregion
 //#region src/cli/failure.ts
 /**
@@ -6997,21 +7016,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
@@ -7033,10 +7037,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"
@@ -7045,10 +7045,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"
@@ -7108,22 +7104,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"
@@ -8950,4 +8930,4 @@ function buildCapabilitiesManifest() {
 //#endregion
 export { commentAuthorReference as $, maskDatabaseUrls as A, readEntryRules as B, cardDisplayTitle as C, readProjectBinding as D, readBinding as E, ENTRY_RULES as F, normalizeStatusName as G, readChangeSeq as H, ENTRY_RULE_DESCRIPTIONS as I, BOARD_ACTOR as J, TRACKED_FIELDS as K, UNMET_ENTRY_RULE_REASONS as L, formatExternalSessionRef as M, parseExternalSessionRef as N, writeBinding as O, pullRequestFactKeys as P, activityToken as Q, evaluateEntryRules as R, renderCapabilitiesMarkdown as S, ignoreBoardFile as T, runBoardWrite as U, serializeEntryRules as V, DEFAULT_STATUSES as W, activityKindOrder as X, currentUnixSeconds as Y, activityText as Z, projectKanboPullRequest as _, BOARD_ERROR_RESPONSES as a, isOutsideWriter as at, projectKanboSubCard as b, registerKanboTools as c, formatIssueId as ct, KANBO_MCP_INSTRUCTIONS as d, nextIssueOrder as dt, formatActivityIssueReference as et, KANBO_CAPABILITIES_MARKDOWN_RESOURCE_URI as f, readIssuePrefix as ft, projectKanboColumn as g, projectKanboCard as h, KANBO_TOOL_NAMES as i, readSystemCommentKind as it, createBoardOps as j, maskDatabaseUrl as k, KANBO_TOOLS as l, nextIssueIdentity as lt, registerKanboCapabilitiesResources as m, buildCapabilitiesManifest as n, isEmptyActivityValue as nt, KANBO_MCP_SERVER_NAME as o, normalizeCommentAuthorKind as ot, KANBO_CAPABILITIES_RESOURCE_URI as p, diffFieldChanges as q, KANBO_TOOL_CLI_EQUIVALENTS as r, parseActivityStringArray as rt, createKanboMcpServer as s, toIssueView as st, CAPABILITIES_MANIFEST_VERSION as t, formatPlainActivityValue as tt, buildKanboTools as u, nextIssueNumber as ut, projectKanboRun as v, BINDING_FILE_PATH as w, DEFAULT_BOARD_FILE_PATH as x, projectKanboSprint as y, readChecklist as z };
 
-//# sourceMappingURL=manifest-mkDJmxt2.mjs.map
+//# sourceMappingURL=manifest-DHV2fRdO.mjs.map