kanbo-cli 0.1.3 → 0.2.1

package/dist/lib/index.mjs

@@ -1,257 +1,3 @@
-import { $ as commentAuthorReference, A as maskDatabaseUrls, B as readEntryRules, C as cardDisplayTitle, D as readProjectBinding, E as readBinding, F as ENTRY_RULES, G as normalizeStatusName, H as readChangeSeq, I as ENTRY_RULE_DESCRIPTIONS, J as BOARD_ACTOR, K as TRACKED_FIELDS, L as UNMET_ENTRY_RULE_REASONS, M as formatExternalSessionRef, N as parseExternalSessionRef, O as writeBinding, P as pullRequestFactKeys, Q as activityToken, R as evaluateEntryRules, S as renderCapabilitiesMarkdown, T as ignoreBoardFile, U as runBoardWrite, V as serializeEntryRules, W as DEFAULT_STATUSES, X as activityKindOrder, Y as currentUnixSeconds, Z as activityText, a as BOARD_ERROR_RESPONSES, at as isOutsideWriter, ct as formatIssueId, dt as nextIssueOrder, et as formatActivityIssueReference, ft as readIssuePrefix, i as KANBO_TOOL_NAMES, it as readSystemCommentKind, j as createBoardOps, k as maskDatabaseUrl, lt as nextIssueIdentity, n as buildCapabilitiesManifest, nt as isEmptyActivityValue, ot as normalizeCommentAuthorKind, q as diffFieldChanges, r as KANBO_TOOL_CLI_EQUIVALENTS, rt as parseActivityStringArray, st as toIssueView, t as CAPABILITIES_MANIFEST_VERSION, tt as formatPlainActivityValue, ut as nextIssueNumber, w as BINDING_FILE_PATH, x as DEFAULT_BOARD_FILE_PATH, z as readChecklist } from "./manifest-mkDJmxt2.mjs";
+import { $ as commentAuthorReference, A as maskDatabaseUrls, B as readEntryRules, C as cardDisplayTitle, D as readProjectBinding, E as readBinding, F as ENTRY_RULES, G as normalizeStatusName, H as readChangeSeq, I as ENTRY_RULE_DESCRIPTIONS, J as BOARD_ACTOR, K as TRACKED_FIELDS, L as UNMET_ENTRY_RULE_REASONS, M as formatExternalSessionRef, N as parseExternalSessionRef, O as writeBinding, P as pullRequestFactKeys, Q as activityToken, R as evaluateEntryRules, S as renderCapabilitiesMarkdown, T as ignoreBoardFile, U as runBoardWrite, V as serializeEntryRules, W as DEFAULT_STATUSES, X as activityKindOrder, Y as currentUnixSeconds, Z as activityText, a as BOARD_ERROR_RESPONSES, at as isOutsideWriter, ct as formatIssueId, dt as nextIssueOrder, et as formatActivityIssueReference, ft as readIssuePrefix, i as KANBO_TOOL_NAMES, it as readSystemCommentKind, j as createBoardOps, k as maskDatabaseUrl, lt as nextIssueIdentity, n as buildCapabilitiesManifest, nt as isEmptyActivityValue, ot as normalizeCommentAuthorKind, q as diffFieldChanges, r as KANBO_TOOL_CLI_EQUIVALENTS, rt as parseActivityStringArray, st as toIssueView, t as CAPABILITIES_MANIFEST_VERSION, tt as formatPlainActivityValue, ut as nextIssueNumber, w as BINDING_FILE_PATH, x as DEFAULT_BOARD_FILE_PATH, z as readChecklist } from "./manifest-DHV2fRdO.mjs";
 import { a as BoardError } from "./migrations-path-Ciwan537.mjs";
-//#region src/ops/board-copy.ts
-/** Nothing copied — what a move that had nothing to move reports. */
-function emptyCopyCounts() {
-	return {
-		statuses: 0,
-		milestones: 0,
-		issues: 0,
-		comments: 0,
-		runs: 0,
-		relations: 0,
-		fieldChanges: 0,
-		contextRefs: 0,
-		pullRequests: 0
-	};
-}
-/**
-* Everything the workspace has on this board, read in one pass before a single
-* row is written anywhere.
-*
-* Links are collected by id because an edge is listed by both of the cards it
-* touches, and one reaching a card outside this workspace is left behind rather
-* than copied: the target holds this workspace's board and nothing else, so the
-* far end of that edge is not there to point at.
-*/
-async function readBoardSlice(from, workspaceId) {
-	const warnings = [];
-	const statuses = await from.statuses.listByWorkspace(workspaceId);
-	const milestones = await from.milestones.listByWorkspace(workspaceId);
-	const issues = orderParentsFirst(await from.issues.listByWorkspace(workspaceId));
-	const issueIds = new Set(issues.map((issue) => issue.id));
-	const comments = [];
-	const runs = [];
-	const relations = /* @__PURE__ */ new Map();
-	const fieldChanges = [];
-	const pullRequests = [];
-	const skippedRelations = /* @__PURE__ */ new Set();
-	for (const issue of issues) {
-		comments.push(...await from.comments.listByIssue(issue.id));
-		runs.push(...await from.runs.listByIssue(issue.id));
-		for (const relation of await from.relations.listByIssue(issue.id)) {
-			if (issueIds.has(relation.sourceIssueId) && issueIds.has(relation.targetIssueId)) {
-				relations.set(relation.id, relation);
-				continue;
-			}
-			skippedRelations.add(relation.id);
-		}
-		fieldChanges.push(...await from.fieldChanges.listByIssue(issue.id));
-		pullRequests.push(...await from.pullRequests.listByIssue(issue.id));
-	}
-	if (skippedRelations.size > 0) warnings.push(`${skippedRelations.size} links to cards in other workspaces were not moved`);
-	return {
-		statuses,
-		milestones,
-		issues,
-		comments,
-		runs,
-		relations: [...relations.values()],
-		fieldChanges,
-		pullRequests,
-		warnings
-	};
-}
-/**
-* Write a slice into the target board, in one transaction, and say what was
-* written and what a person should know about it — the slice's own warnings
-* first.
-*
-* It is idempotent by id. A row the target already has is left exactly as it
-* is and counted as already present, so a copy interrupted half-way is finished
-* by running it again, and a second run of a finished one copies nothing. A
-* clash that is not the same row — a column name or a card number the target
-* holds under another id — refuses the whole copy before its first insert
-* (`board_copy_conflict`, ruling 3-55).
-*/
-async function writeBoardSlice(to, slice, workspaceId) {
-	const warnings = [...slice.warnings];
-	const copied = emptyCopyCounts();
-	let alreadyPresent = 0;
-	await to.transaction(async (tx) => {
-		await assertTheCopyFits(tx, slice, workspaceId);
-		for (const status of slice.statuses) {
-			if (await tx.statuses.findById(status.id)) {
-				alreadyPresent += 1;
-				continue;
-			}
-			await tx.statuses.create(status);
-			copied.statuses += 1;
-		}
-		for (const milestone of slice.milestones) {
-			if (await tx.milestones.findById(milestone.id)) {
-				alreadyPresent += 1;
-				continue;
-			}
-			await tx.milestones.create(milestone);
-			copied.milestones += 1;
-		}
-		const movingIssueIds = new Set(slice.issues.map((issue) => issue.id));
-		for (const issue of slice.issues) {
-			if (await tx.issues.findById(issue.id)) {
-				alreadyPresent += 1;
-				continue;
-			}
-			const parentIsReachable = issue.parentIssueId === null || movingIssueIds.has(issue.parentIssueId) || await tx.issues.findById(issue.parentIssueId) !== null;
-			if (!parentIsReachable) warnings.push(`${issue.id} lost its parent card, which is not on this board`);
-			const { contextRefs, ...row } = issue;
-			await tx.issues.create({
-				...row,
-				parentIssueId: parentIsReachable ? issue.parentIssueId : null
-			});
-			copied.issues += 1;
-			if (contextRefs !== "[]") {
-				await tx.contextRefs.replace(issue.id, contextRefs, issue.updatedAt);
-				copied.contextRefs += 1;
-			}
-		}
-		for (const comment of slice.comments) {
-			if (await tx.comments.findById(comment.id)) {
-				alreadyPresent += 1;
-				continue;
-			}
-			await tx.comments.create(comment);
-			copied.comments += 1;
-		}
-		for (const run of slice.runs) {
-			if (await tx.runs.findById(run.id)) {
-				alreadyPresent += 1;
-				continue;
-			}
-			await tx.runs.create(run);
-			copied.runs += 1;
-		}
-		for (const relation of slice.relations) {
-			if (await tx.relations.findById(relation.id)) {
-				alreadyPresent += 1;
-				continue;
-			}
-			await tx.relations.create(relation);
-			copied.relations += 1;
-		}
-		const changesByIssue = /* @__PURE__ */ new Map();
-		for (const change of slice.fieldChanges) changesByIssue.set(change.issueId, [...changesByIssue.get(change.issueId) ?? [], change]);
-		for (const [issueId, changes] of changesByIssue) {
-			const present = new Set((await tx.fieldChanges.listByIssue(issueId)).map((change) => change.id));
-			for (const change of changes) {
-				if (present.has(change.id)) {
-					alreadyPresent += 1;
-					continue;
-				}
-				await tx.fieldChanges.create(change);
-				copied.fieldChanges += 1;
-			}
-		}
-		const linksByIssue = /* @__PURE__ */ new Map();
-		for (const link of slice.pullRequests) linksByIssue.set(link.issueId, [...linksByIssue.get(link.issueId) ?? [], link]);
-		for (const [issueId, links] of linksByIssue) {
-			const present = new Set((await tx.pullRequests.listByIssue(issueId)).map((link) => link.id));
-			for (const link of links) {
-				if (present.has(link.id) || (await tx.pullRequests.link(link)).id !== link.id) {
-					alreadyPresent += 1;
-					continue;
-				}
-				copied.pullRequests += 1;
-			}
-		}
-		await tx.meta.bumpRevision("board", currentUnixSeconds());
-	});
-	if (alreadyPresent > 0) warnings.push(`${alreadyPresent} rows were already present on the target board and were left as they are`);
-	return {
-		copied,
-		warnings
-	};
-}
-/**
-* Empty the slice off the board it was read from, in a transaction of its own.
-*
-* Only the cards, the columns and the milestones are named: comments, runs,
-* links, pull request links and the field-change history hang off a card by a
-* foreign key that cascades on both engines, so deleting the card takes them
-* with it.
-*/
-async function deleteBoardSlice(from, slice) {
-	await from.transaction(async (tx) => {
-		for (const issue of slice.issues) await tx.issues.delete(issue.id);
-		for (const milestone of slice.milestones) await tx.milestones.delete(milestone.id);
-		for (const status of slice.statuses) await tx.statuses.delete(status.id);
-		await tx.meta.bumpRevision("board", currentUnixSeconds());
-	});
-}
-/**
-* What a copy would collide with on the target, said before a row is written.
-*
-* The board carries two uniqueness rules that a *second* board of the same
-* workspace can break: a column's name within its workspace, and a card's
-* number within it. In practice it is the columns that collide: a standard set
-* created a second time carries the same names under new ids, while a card's id
-* is its key (`BST-001`), so two generations of card *number* 1 are two rows
-* with one id — which the copy treats as already present rather than as a
-* clash. The number check covers the case where they are not: a workspace whose
-* identifier changed between the two generations numbers from 1 again under
-* different ids.
-*
-* A row the target already has *by id* is not a collision — it is the
-* idempotent half of the copy, which is why the holder's id is compared rather
-* than merely counted. Milestones need no check: neither engine makes their
-* titles unique.
-*/
-async function assertTheCopyFits(tx, slice, workspaceId) {
-	const conflicts = [];
-	const numbers = new Map((await tx.issues.listByWorkspace(workspaceId)).map((issue) => [issue.number, issue.id]));
-	for (const issue of slice.issues) {
-		const holder = numbers.get(issue.number);
-		if (holder !== void 0 && holder !== issue.id) conflicts.push({
-			kind: "card",
-			value: issue.number,
-			heldBy: holder
-		});
-	}
-	const names = new Map((await tx.statuses.listByWorkspace(workspaceId)).map((status) => [status.name, status.id]));
-	for (const status of slice.statuses) {
-		const holder = names.get(status.name);
-		if (holder !== void 0 && holder !== status.id) conflicts.push({
-			kind: "column",
-			value: status.name,
-			heldBy: holder
-		});
-	}
-	if (conflicts.length > 0) throw new BoardError("board_copy_conflict", {
-		workspaceId,
-		conflicts
-	});
-}
-/**
-* The same cards, ordered so that a parent is always written before its
-* children. The column is a foreign key onto the same table, so any other order
-* writes a card pointing at one that is not there yet.
-*/
-function orderParentsFirst(issues) {
-	const byId = new Map(issues.map((issue) => [issue.id, issue]));
-	const ordered = [];
-	const placed = /* @__PURE__ */ new Set();
-	function place(issue, ancestors) {
-		if (placed.has(issue.id) || ancestors.has(issue.id)) return;
-		ancestors.add(issue.id);
-		const parent = issue.parentIssueId === null ? void 0 : byId.get(issue.parentIssueId);
-		if (parent) place(parent, ancestors);
-		placed.add(issue.id);
-		ordered.push(issue);
-	}
-	for (const issue of issues) place(issue, /* @__PURE__ */ new Set());
-	return ordered;
-}
-//#endregion
-export { BINDING_FILE_PATH, BOARD_ACTOR, BOARD_ERROR_RESPONSES, BoardError, CAPABILITIES_MANIFEST_VERSION, DEFAULT_BOARD_FILE_PATH, DEFAULT_STATUSES, ENTRY_RULES, ENTRY_RULE_DESCRIPTIONS, KANBO_TOOL_CLI_EQUIVALENTS, KANBO_TOOL_NAMES, TRACKED_FIELDS, UNMET_ENTRY_RULE_REASONS, activityKindOrder, activityText, activityToken, buildCapabilitiesManifest, cardDisplayTitle, commentAuthorReference, createBoardOps, currentUnixSeconds, deleteBoardSlice, diffFieldChanges, emptyCopyCounts, evaluateEntryRules, formatActivityIssueReference, formatExternalSessionRef, formatIssueId, formatPlainActivityValue, ignoreBoardFile, isEmptyActivityValue, isOutsideWriter, maskDatabaseUrl, maskDatabaseUrls, nextIssueIdentity, nextIssueNumber, nextIssueOrder, normalizeCommentAuthorKind, normalizeStatusName, parseActivityStringArray, parseExternalSessionRef, pullRequestFactKeys, readBinding, readBoardSlice, readChangeSeq, readChecklist, readEntryRules, readIssuePrefix, readProjectBinding, readSystemCommentKind, renderCapabilitiesMarkdown, runBoardWrite, serializeEntryRules, toIssueView, writeBinding, writeBoardSlice };
-
-//# sourceMappingURL=index.mjs.map
+export { BINDING_FILE_PATH, BOARD_ACTOR, BOARD_ERROR_RESPONSES, BoardError, CAPABILITIES_MANIFEST_VERSION, DEFAULT_BOARD_FILE_PATH, DEFAULT_STATUSES, ENTRY_RULES, ENTRY_RULE_DESCRIPTIONS, KANBO_TOOL_CLI_EQUIVALENTS, KANBO_TOOL_NAMES, TRACKED_FIELDS, UNMET_ENTRY_RULE_REASONS, activityKindOrder, activityText, activityToken, buildCapabilitiesManifest, cardDisplayTitle, commentAuthorReference, createBoardOps, currentUnixSeconds, diffFieldChanges, evaluateEntryRules, formatActivityIssueReference, formatExternalSessionRef, formatIssueId, formatPlainActivityValue, ignoreBoardFile, isEmptyActivityValue, isOutsideWriter, maskDatabaseUrl, maskDatabaseUrls, nextIssueIdentity, nextIssueNumber, nextIssueOrder, normalizeCommentAuthorKind, normalizeStatusName, parseActivityStringArray, parseExternalSessionRef, pullRequestFactKeys, readBinding, readChangeSeq, readChecklist, readEntryRules, readIssuePrefix, readProjectBinding, readSystemCommentKind, renderCapabilitiesMarkdown, runBoardWrite, serializeEntryRules, toIssueView, writeBinding };

package/dist/lib/{manifest-BBJIrMw7.cjs → manifest-D9nCRtXM.cjs}

@@ -1926,15 +1926,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 require_migrations_path.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()
 		};
@@ -1955,6 +1952,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 require_migrations_path.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 }) => {
@@ -2832,6 +2850,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),
@@ -4334,7 +4353,7 @@ function readRequestedRules(options) {
 * `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
 /**
@@ -7020,21 +7039,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
@@ -7056,10 +7060,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"
@@ -7068,10 +7068,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"
@@ -7131,22 +7127,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"
@@ -9368,4 +9348,4 @@ Object.defineProperty(exports, "writeBinding", {
 	}
 });
 
-//# sourceMappingURL=manifest-BBJIrMw7.cjs.map
+//# sourceMappingURL=manifest-D9nCRtXM.cjs.map