pi-roundtable-drawing 0.7.4 → 0.7.5

package/CHANGELOG.md

@@ -5,6 +5,12 @@ The format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
 
 ## [Unreleased]
 
+## [0.7.5] - 2026-10-03
+
+### Added
+
+- `permissive` accepts what a looser host sends; `mapLimits` values must be whole numbers of at least 1.
+
 ## [0.7.4] - 2026-10-03
 
 - Add `cardPresentation.result` for the card draw result text and `mapLimits` for longer relationship-map text.

package/README.md

@@ -83,7 +83,8 @@ export default {
 | `random` | `() => number` | `Math.random` | The random source of the map layouts and the card draws, returning a number from 0 up to, not including, 1. A seeded source makes a run reproducible: a new `seededRandom(seed)` given the same calls draws the same pictures. `seededRandom(seed)` is exported for that |
 | `minTier` | `"owner" \| "admin" \| "member"` | `member` | The lowest tier that may use the tools. The operator's `toolTiers` setting still wins |
 | `cardPresentation` | `CardPresentation` | package captions | Trusted operator-local headings, reversal suffixes and result text |
-| `mapLimits` | `{ title?, text? }` | 80 and 60 | Longest relationship-map title, and node id, node label or edge label, in characters, for a host whose callers already draw longer text; the renderer's size limit still applies |
+| `mapLimits` | `{ title?, text? }` | 80 and 60 | Longest relationship-map title, and node id, node label or edge label, in characters, for a host whose callers already draw longer text; each is a whole number of at least 1 (anything else stops the start), and the renderer's size limit still applies |
+| `permissive` | `boolean` | `false` | Accept what a looser host's callers already send: a card exclusion that is not a card of the deck is ignored, spread positions may share a cell, and a relationship map may repeat or leave empty a node id and may draw an edge from a node to itself. By default each is refused with a message the model can correct |
 
 The plugin is named `drawing`.
 `cardPresentation.heading(draw, turn)` returns `{ title, subtitle }`; `draw` holds the selected `deck`, `count`, and optional `question`, and `turn` carries the host-bound speaker/channel.

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "pi-roundtable-drawing",
-	"version": "0.7.4",
+	"version": "0.7.5",
 	"description": "Relationship maps, magic circles, sigils, sacred geometry, and card spreads drawn locally for pi-roundtable",
 	"type": "module",
 	"license": "MIT",
@@ -48,7 +48,7 @@
 	"devDependencies": {
 		"@biomejs/biome": "2.5.15",
 		"@types/bun": "1.4.2",
-		"pi-roundtable": "0.7.4",
+		"pi-roundtable": "0.7.5",
 		"typescript": "7.0.2"
 	}
 }

package/src/cards/draw.ts

@@ -15,11 +15,21 @@ export interface DrawParams {
 	/** Whether cards may come up reversed; the deck's own default when absent. */
 	allowReversed?: boolean | undefined;
 	random: Random;
+	/** Ignore exclusions that are not cards of the deck, instead of refusing them. */
+	ignoreUnknownExclusions?: boolean | undefined;
 }
 
 /** Draws from the whole deck minus the exclusions; there is no persistent deck between draws. */
 export function drawCards(params: DrawParams): DrawnCard[] {
-	const { deck, count, group, exclude, allowReversed, random } = params;
+	const {
+		deck,
+		count,
+		group,
+		exclude,
+		allowReversed,
+		random,
+		ignoreUnknownExclusions,
+	} = params;
 	if (!Number.isInteger(count) || count < 1)
 		throw new DrawingError(
 			`count must be a whole number of at least 1; got ${count}.`,
@@ -35,7 +45,7 @@ export function drawCards(params: DrawParams): DrawnCard[] {
 	const known = new Set(deck.cards.map((card) => card.id));
 	const excluded = new Set(exclude ?? []);
 	for (const id of excluded)
-		if (!known.has(id))
+		if (!known.has(id) && !ignoreUnknownExclusions)
 			throw new DrawingError(
 				`exclude names ${JSON.stringify(id)}, which is not a card of deck ${deck.id}. Use the card ids the deck lists.`,
 			);

package/src/cards/spread.ts

@@ -43,6 +43,7 @@ export function autoLayout(count: number): SpreadPosition[] {
 export function checkSpread(
 	positions: readonly SpreadPosition[],
 	count: number,
+	allowSharedCells = false,
 ): void {
 	if (positions.length !== count)
 		throw new DrawingError(
@@ -55,7 +56,7 @@ export function checkSpread(
 				`Spread positions need a row and a column that are whole numbers from 0; got row ${row}, column ${col}.`,
 			);
 		const cell = `${row}:${col}`;
-		if (cells.has(cell))
+		if (cells.has(cell) && !allowSharedCells)
 			throw new DrawingError(
 				`Two cards share row ${row}, column ${col}. Give each card its own position.`,
 			);
@@ -113,8 +114,9 @@ export async function renderSpread(
 	positions: readonly SpreadPosition[],
 	heading: { title: string; subtitle: string },
 	reversedSuffix = " (reversed)",
+	allowSharedCells = false,
 ): Promise<Uint8Array> {
-	checkSpread(positions, cards.length);
+	checkSpread(positions, cards.length, allowSharedCells);
 	ensureFont();
 	const faces = await Promise.all(
 		cards.map(async (card) => {

package/src/plugin.ts

@@ -56,6 +56,13 @@ export interface DrawingOptions {
 	cardPresentation?: CardPresentation;
 	/** Longer map text than the defaults, for a host whose callers already draw longer labels. */
 	mapLimits?: RelationshipMapLimits;
+	/**
+	 * Accept what a looser host's callers already send: a card exclusion that is not a card of the
+	 * deck is ignored, spread positions may share a cell, and a relationship map may repeat or leave
+	 * empty a node id and may draw an edge from a node to itself. Off by default, where each is
+	 * refused with a message the model can correct.
+	 */
+	permissive?: boolean;
 }
 
 function readDecks(deckDir: string): Deck[] {
@@ -77,6 +84,11 @@ export function drawing(options: DrawingOptions = {}) {
 	return definePlugin({
 		name: "drawing",
 		setup: () => {
+			for (const [name, limit] of Object.entries(options.mapLimits ?? {}))
+				if (!Number.isSafeInteger(limit) || limit < 1)
+					throw new PluginError(
+						`plugin drawing: mapLimits.${name} must be a whole number of at least 1; got ${limit}.`,
+					);
 			// Read at setup, so a wrong deck directory stops the start with a message that names this plugin.
 			const decks = options.deckDir ? readDecks(options.deckDir) : [];
 			const env: ImageToolEnv = {
@@ -84,12 +96,25 @@ export function drawing(options: DrawingOptions = {}) {
 			};
 			return {
 				tools: [
-					relationshipMapTool(env, random, options.mapLimits),
+					relationshipMapTool(
+						env,
+						random,
+						options.mapLimits,
+						options.permissive,
+					),
 					magicCircleTool(env),
 					sigilTool(env),
 					sacredGeometryTool(env),
 					...(decks.length > 0
-						? [drawCardsTool(decks, env, random, options.cardPresentation)]
+						? [
+								drawCardsTool(
+									decks,
+									env,
+									random,
+									options.cardPresentation,
+									options.permissive,
+								),
+							]
 						: []),
 				],
 			};

package/src/relationship-map.ts

@@ -126,7 +126,7 @@ interface SimLink extends SimulationLinkDatum<SimNode> {
 
 type RoughCanvas = ReturnType<typeof rough.canvas>;
 
-function validate(input: MapInput): void {
+function validate(input: MapInput, permissive: boolean): void {
 	if (input.nodes.length === 0)
 		throw new DrawingError("nodes cannot be empty. Give at least one node.");
 	if (input.nodes.length > MAX_NODES)
@@ -139,9 +139,9 @@ function validate(input: MapInput): void {
 		);
 	const ids = new Set<string>();
 	for (const node of input.nodes) {
-		if (node.id.trim() === "")
+		if (node.id.trim() === "" && !permissive)
 			throw new DrawingError("A node has an empty id. Give every node a name.");
-		if (ids.has(node.id))
+		if (ids.has(node.id) && !permissive)
 			throw new DrawingError(
 				`Two nodes share the id ${JSON.stringify(node.id)}. Node ids must be unique.`,
 			);
@@ -153,7 +153,7 @@ function validate(input: MapInput): void {
 				throw new DrawingError(
 					`An edge references the unknown node ${JSON.stringify(end)}. Add it to nodes or fix the edge.`,
 				);
-		if (edge.from === edge.to)
+		if (edge.from === edge.to && !permissive)
 			throw new DrawingError(
 				`An edge from ${JSON.stringify(edge.from)} to itself cannot be drawn. Connect two different nodes.`,
 			);
@@ -390,8 +390,10 @@ function drawNode(
 export function renderRelationshipMap(
 	input: MapInput,
 	random: Random = Math.random,
+	/** Draw maps with repeated or empty node ids and edges from a node to itself instead of refusing them. */
+	permissive = false,
 ): Uint8Array {
-	validate(input);
+	validate(input, permissive);
 	ensureFont();
 	// The shapes grow to hold their text, so the text is measured before the layout.
 	const measure = createCanvas(1, 1).getContext("2d");

package/src/tools/draw-cards-tool.ts

@@ -34,6 +34,7 @@ export function drawCardsTool(
 	env: ImageToolEnv,
 	random: Random,
 	presentation: CardPresentation = {},
+	permissive = false,
 ) {
 	const [first, ...rest] = decks;
 	if (!first) throw new DrawingError("draw_cards needs at least one deck.");
@@ -92,7 +93,7 @@ export function drawCardsTool(
 				const positions: SpreadPosition[] =
 					args.spread ?? autoLayout(args.count);
 				// Checked before the draw, so a refused spread leaves the random source where it was.
-				checkSpread(positions, args.count);
+				checkSpread(positions, args.count, permissive);
 				const cards = drawCards({
 					deck,
 					count: args.count,
@@ -100,6 +101,7 @@ export function drawCardsTool(
 					exclude: args.exclude,
 					allowReversed: args.allow_reversed,
 					random,
+					ignoreUnknownExclusions: permissive,
 				});
 				const heading = presentation.heading?.(
 					{ deck, count: cards.length, question: args.question },
@@ -116,6 +118,7 @@ export function drawCardsTool(
 					positions,
 					heading,
 					presentation.reversedSuffix,
+					permissive,
 				);
 				const lines = cards.map((card, i) => {
 					const label = positions[i]?.label ? `${positions[i]?.label}: ` : "";

package/src/tools/relationship-map-tool.ts

@@ -21,6 +21,7 @@ export function relationshipMapTool(
 	env: ImageToolEnv,
 	random: Random,
 	limits: RelationshipMapLimits = {},
+	permissive = false,
 ) {
 	const maxTitle = limits.title ?? MAX_TITLE;
 	const maxText = limits.text ?? MAX_TEXT;
@@ -63,7 +64,7 @@ export function relationshipMapTool(
 			}),
 			draw: (args) => ({
 				stem: "relationship-map",
-				image: renderRelationshipMap(args, random),
+				image: renderRelationshipMap(args, random, permissive),
 				text: "The relationship map is attached to your reply as {file}.",
 			}),
 		},