markform 0.1.12 → 0.1.14

package/dist/{session-DX-DvjRP.mjs → session-XDrocA3j.mjs}

@@ -1,5 +1,5 @@
 
-import { U as SessionTranscriptSchema } from "./coreTypes-Z8SvQyoL.mjs";
+import { U as SessionTranscriptSchema } from "./coreTypes-DiCddBKu.mjs";
 import YAML from "yaml";
 
 //#region src/engine/session.ts

package/dist/{src-DMQCFp2l.mjs → src-BKRKMdgR.mjs}

@@ -1,6 +1,6 @@
 
-import { L as PatchSchema, V as RunModeSchema } from "./coreTypes-Z8SvQyoL.mjs";
-import { C as DEFAULT_RESEARCH_MAX_PATCHES_PER_TURN, L as getWebSearchConfig, S as DEFAULT_RESEARCH_MAX_ISSUES_PER_TURN, T as DEFAULT_ROLE_INSTRUCTIONS, V as MarkformConfigError, W as MarkformParseError, _ as DEFAULT_MAX_ISSUES_PER_TURN, d as serializeForm, i as inspect, m as AGENT_ROLE, r as getFieldsForRoles, t as applyPatches, v as DEFAULT_MAX_PATCHES_PER_TURN, w as DEFAULT_ROLES, x as DEFAULT_PRIORITY, y as DEFAULT_MAX_TURNS } from "./apply-WeeBXwXg.mjs";
+import { L as PatchSchema, V as RunModeSchema } from "./coreTypes-DiCddBKu.mjs";
+import { C as DEFAULT_RESEARCH_MAX_ISSUES_PER_TURN, E as DEFAULT_ROLE_INSTRUCTIONS, G as MarkformParseError, H as MarkformConfigError, R as getWebSearchConfig, S as DEFAULT_PRIORITY, T as DEFAULT_ROLES, _ as DEFAULT_MAX_ISSUES_PER_TURN, b as DEFAULT_MAX_TURNS, d as serializeForm, i as inspect, m as AGENT_ROLE, r as getFieldsForRoles, t as applyPatches, v as DEFAULT_MAX_PATCHES_PER_TURN, w as DEFAULT_RESEARCH_MAX_PATCHES_PER_TURN, y as DEFAULT_MAX_STEPS_PER_TURN } from "./apply-CUH_16RD.mjs";
 import { z } from "zod";
 import Markdoc from "@markdoc/markdoc";
 import YAML from "yaml";
@@ -2139,12 +2139,41 @@ function coerceToCheckboxes(field, rawValue) {
 			value: {}
 		}
 	};
+	const validOptions = new Set(field.options.map((o) => o.id));
+	const checkboxMode = field.checkboxMode;
+	if (Array.isArray(rawValue)) {
+		const defaultState = checkboxMode === "explicit" ? "yes" : "done";
+		const values$1 = {};
+		for (const item of rawValue) {
+			if (typeof item !== "string") return {
+				ok: false,
+				error: `Array items for checkboxes field '${field.id}' must be strings (option IDs), got ${typeof item}`
+			};
+			if (!validOptions.has(item)) return {
+				ok: false,
+				error: `Invalid option '${item}' for checkboxes field '${field.id}'. Valid options: ${Array.from(validOptions).join(", ")}`
+			};
+			values$1[item] = defaultState;
+		}
+		const patch$1 = {
+			op: "set_checkboxes",
+			fieldId: field.id,
+			value: values$1
+		};
+		if (rawValue.length === 0) return {
+			ok: true,
+			patch: patch$1
+		};
+		return {
+			ok: true,
+			patch: patch$1,
+			warning: `Coerced array to checkboxes object with '${defaultState}' state for field '${field.id}'`
+		};
+	}
 	if (!isPlainObject(rawValue)) return {
 		ok: false,
-		error: `checkboxes field '${field.id}' requires a Record<string, CheckboxValue>, got ${typeof rawValue}`
+		error: `checkboxes field '${field.id}' requires a Record<string, CheckboxValue> or array of option IDs, got ${typeof rawValue}`
 	};
-	const validOptions = new Set(field.options.map((o) => o.id));
-	const checkboxMode = field.checkboxMode;
 	const values = {};
 	let hadBooleanCoercion = false;
 	const validValues = new Set(checkboxMode === "explicit" ? [
@@ -2538,12 +2567,12 @@ var FormHarness = class {
 		if (this.state !== "wait") throw new Error(`Cannot apply in state: ${this.state}`);
 		if (patches.length > this.config.maxPatchesPerTurn) throw new Error(`Too many patches: ${patches.length} > ${this.config.maxPatchesPerTurn}`);
 		const applyResult = applyPatches(this.form, patches);
-		const patchesActuallyApplied = applyResult.applyStatus === "applied" ? patches.length : 0;
+		const patchesActuallyApplied = applyResult.appliedPatches.length;
 		const result = inspect(this.form, { targetRoles: this.config.targetRoles });
 		const stepResult = this.computeStepResult(result);
 		stepResult.patchesApplied = patchesActuallyApplied;
 		stepResult.rejectedPatches = applyResult.rejectedPatches;
-		this.recordTurn(issues, patches, result, llmStats, context, applyResult.rejectedPatches, wire);
+		this.recordTurn(issues, patches, result, llmStats, context, applyResult.rejectedPatches, applyResult.warnings, wire);
 		if (stepResult.issues.length === 0 || this.turnNumber >= this.config.maxTurns) this.state = "complete";
 		else this.state = "wait";
 		return stepResult;
@@ -2567,7 +2596,7 @@ var FormHarness = class {
 	/**
 	* Record a turn in the session transcript.
 	*/
-	recordTurn(issues, patches, result, llmStats, context, rejectedPatches, wire) {
+	recordTurn(issues, patches, result, llmStats, context, rejectedPatches, warnings, wire) {
 		const hash = sha256(serializeForm(this.form));
 		const requiredIssueCount = result.issues.filter((i) => i.severity === "required").length;
 		const turn = {
@@ -2575,7 +2604,8 @@ var FormHarness = class {
 			inspect: { issues },
 			apply: {
 				patches,
-				...rejectedPatches && rejectedPatches.length > 0 && { rejectedPatches }
+				...rejectedPatches && rejectedPatches.length > 0 && { rejectedPatches },
+				...warnings && warnings.length > 0 && { warnings }
 			},
 			after: {
 				requiredIssueCount,
@@ -7860,25 +7890,51 @@ var anthropic = createAnthropic();
 * fabrication of data.
 */
 const DEFAULT_SYSTEM_PROMPT = `# Form Instructions
-Carefully research answers to all questions in the form, using all available tools you have.
 
-Guidelines:
-1. Focus on required fields first (severity: "required"), then address optional fields (severity: "recommended")
-2. You MUST address ALL issues shown to you - both required AND recommended (optional)
-3. NEVER fabricate or guess information - only use data you can verify
-4. If you cannot find verifiable information for a field, use skip_field to mark it as skipped with a reason
-5. For string fields: use appropriate text from verified sources
-6. For number fields: use appropriate numeric values from verified sources
-7. For single_select: choose one valid option ID
-8. For multi_select: choose one or more valid option IDs
-9. For checkboxes: use the appropriate state for the checkbox mode:
-   - Mode "simple": done (checked) or todo (unchecked)
-   - Mode "multi": done, todo, or na (not applicable)
-   - Mode "explicit": yes or no (must explicitly answer)
+Research and fill the form fields using all available tools. Focus on accuracy over completeness.
+
+## Guidelines
+1. Address required fields first (severity: "required"), then optional fields (severity: "recommended")
+2. NEVER fabricate or guess information - only use data you can verify
+3. If you cannot find verifiable information, use skip_field with a reason
+
+## Patch Format Examples
+
+Use the fill_form tool with patches in these formats:
+
+| Type | Example |
+|------|---------|
+| string | \`{ op: "set_string", fieldId: "name", value: "Acme Corp" }\` |
+| number | \`{ op: "set_number", fieldId: "age", value: 32 }\` |
+| string_list | \`{ op: "set_string_list", fieldId: "tags", value: ["ai", "ml"] }\` |
+| url | \`{ op: "set_url", fieldId: "website", value: "https://example.com" }\` |
+| url_list | \`{ op: "set_url_list", fieldId: "sources", value: ["https://a.com", "https://b.com"] }\` |
+| date | \`{ op: "set_date", fieldId: "event_date", value: "2024-06-15" }\` |
+| year | \`{ op: "set_year", fieldId: "founded", value: 2024 }\` |
+| single_select | \`{ op: "set_single_select", fieldId: "priority", value: "high" }\` |
+| multi_select | \`{ op: "set_multi_select", fieldId: "categories", value: ["frontend", "backend"] }\` |
+| checkboxes | \`{ op: "set_checkboxes", fieldId: "tasks", value: { "task1": "done", "task2": "todo" } }\` |
+| table | \`{ op: "set_table", fieldId: "team", value: [{ "name": "Alice", "role": "Engineer" }] }\` |
 
-CRITICAL: Accuracy is more important than completeness. Use skip_field when information cannot be verified.
+## Important: checkboxes vs multi_select
 
-Always use the fill_form tool to submit your field values.
+These two types look similar but have DIFFERENT value formats:
+
+- **multi_select** → array of option IDs: \`["opt1", "opt2"]\`
+- **checkboxes** → object mapping IDs to states: \`{ "opt1": "done", "opt2": "todo" }\`
+
+**Checkbox states by mode:**
+- Mode "simple": \`"done"\` or \`"todo"\`
+- Mode "multi": \`"done"\`, \`"todo"\`, \`"incomplete"\`, \`"active"\`, or \`"na"\`
+- Mode "explicit": \`"yes"\` or \`"no"\` (if unknown, use abort_field)
+
+**WRONG:** \`{ op: "set_checkboxes", value: ["task1", "task2"] }\`
+**RIGHT:** \`{ op: "set_checkboxes", value: { "task1": "done", "task2": "done" } }\`
+
+## Skipping Fields
+
+If you cannot find verifiable information:
+\`{ op: "skip_field", fieldId: "...", reason: "Could not find verified data" }\`
 `;
 /**
 * Web search instructions appended when web search tools are available.
@@ -7914,33 +7970,49 @@ function getIssuesIntro(issueCount) {
 * Used in PATCH_FORMAT_INSTRUCTIONS and rejection feedback hints.
 */
 const PATCH_FORMATS = {
-	string: "{ op: \"set_string\", fieldId: \"...\", value: \"...\" }",
-	number: "{ op: \"set_number\", fieldId: \"...\", value: 123 }",
-	string_list: "{ op: \"set_string_list\", fieldId: \"...\", value: [\"...\", \"...\"] }",
+	string: "{ op: \"set_string\", fieldId: \"...\", value: \"text here\" }",
+	number: "{ op: \"set_number\", fieldId: \"...\", value: 42 }",
+	string_list: "{ op: \"set_string_list\", fieldId: \"...\", value: [\"item1\", \"item2\"] }",
 	single_select: "{ op: \"set_single_select\", fieldId: \"...\", value: \"option_id\" }",
 	multi_select: "{ op: \"set_multi_select\", fieldId: \"...\", value: [\"opt1\", \"opt2\"] }",
 	checkboxes: "{ op: \"set_checkboxes\", fieldId: \"...\", value: { \"opt1\": \"done\", \"opt2\": \"todo\" } }",
-	url: "{ op: \"set_url\", fieldId: \"...\", value: \"https://...\" }",
-	url_list: "{ op: \"set_url_list\", fieldId: \"...\", value: [\"https://...\", \"https://...\"] }",
-	date: "{ op: \"set_date\", fieldId: \"...\", value: \"2024-01-15\" }",
+	url: "{ op: \"set_url\", fieldId: \"...\", value: \"https://example.com\" }",
+	url_list: "{ op: \"set_url_list\", fieldId: \"...\", value: [\"https://a.com\", \"https://b.com\"] }",
+	date: "{ op: \"set_date\", fieldId: \"...\", value: \"2024-06-15\" }",
 	year: "{ op: \"set_year\", fieldId: \"...\", value: 2024 }",
-	table: "{ op: \"set_table\", fieldId: \"...\", value: [{ col1: \"value1\", col2: \"value2\" }, ...] }"
+	table: "{ op: \"set_table\", fieldId: \"...\", value: [{ col1: \"val1\", col2: \"val2\" }] }"
 };
 /**
 * Get the correct patch format for a field kind.
 *
 * @param fieldKind - The field kind (e.g., "table", "string")
-* @param fieldId - Optional field ID to substitute in the example
-* @param columnIds - Optional column IDs for table fields
+* @param options - Optional configuration for the hint
 * @returns The patch format example string
 */
-function getPatchFormatHint(fieldKind, fieldId, columnIds) {
+function getPatchFormatHint(fieldKind, fieldIdOrOptions, columnIds) {
+	let options = {};
+	if (typeof fieldIdOrOptions === "string") options = {
+		fieldId: fieldIdOrOptions,
+		columnIds
+	};
+	else if (fieldIdOrOptions) options = fieldIdOrOptions;
 	let format = PATCH_FORMATS[fieldKind];
 	if (!format) return `Use the correct set_${fieldKind} operation for this field type.`;
-	if (fieldId) format = format.replace("fieldId: \"...\"", `fieldId: "${fieldId}"`);
-	if (fieldKind === "table" && columnIds && columnIds.length > 0) {
-		const colExample = columnIds.map((id) => `"${id}": "..."`).join(", ");
-		format = format.replace("{ col1: \"value1\", col2: \"value2\" }", `{ ${colExample} }`);
+	if (options.fieldId) format = format.replace("fieldId: \"...\"", `fieldId: "${options.fieldId}"`);
+	if (fieldKind === "checkboxes") {
+		const mode = options.checkboxMode ?? "multi";
+		const optIds = options.optionIds ?? ["opt1", "opt2"];
+		const [state1, state2] = {
+			simple: ["done", "todo"],
+			multi: ["done", "todo"],
+			explicit: ["yes", "no"]
+		}[mode] ?? ["done", "todo"];
+		const valueExample = optIds.length >= 2 ? `{ "${optIds[0]}": "${state1}", "${optIds[1]}": "${state2}" }` : optIds.length === 1 ? `{ "${optIds[0]}": "${state1}" }` : `{ "opt1": "${state1}", "opt2": "${state2}" }`;
+		format = format.replace("{ \"opt1\": \"done\", \"opt2\": \"todo\" }", valueExample);
+	}
+	if (fieldKind === "table" && options.columnIds && options.columnIds.length > 0) {
+		const colExample = options.columnIds.map((id) => `"${id}": "..."`).join(", ");
+		format = format.replace("{ col1: \"val1\", col2: \"val2\" }", `{ ${colExample} }`);
 	}
 	return format;
 }
@@ -8011,7 +8083,7 @@ var LiveAgent = class {
 	callbacks;
 	constructor(config) {
 		this.model = config.model;
-		this.maxStepsPerTurn = config.maxStepsPerTurn ?? 3;
+		this.maxStepsPerTurn = config.maxStepsPerTurn ?? DEFAULT_MAX_STEPS_PER_TURN;
 		this.systemPromptAddition = config.systemPromptAddition;
 		this.targetRole = config.targetRole ?? AGENT_ROLE;
 		this.provider = config.provider;
@@ -8274,11 +8346,16 @@ function buildContextPrompt(issues, form, maxPatches, previousRejections) {
 			const field = findField(form, issue.ref);
 			if (field) {
 				lines.push(`  Type: ${field.kind}`);
+				let optionIds;
 				if ("options" in field && field.options) {
-					const optionIds = field.options.map((o) => o.id).join(", ");
-					lines.push(`  Options: ${optionIds}`);
+					optionIds = field.options.map((o) => o.id);
+					lines.push(`  Options: ${optionIds.join(", ")}`);
+				}
+				let checkboxMode;
+				if (field.kind === "checkboxes" && "checkboxMode" in field) {
+					checkboxMode = field.checkboxMode ?? "multi";
+					lines.push(`  Mode: ${checkboxMode}`);
 				}
-				if (field.kind === "checkboxes" && "checkboxMode" in field) lines.push(`  Mode: ${field.checkboxMode ?? "multi"}`);
 				let columnIds;
 				if (field.kind === "table" && "columns" in field && field.columns) {
 					columnIds = field.columns.map((c) => c.id);
@@ -8291,7 +8368,12 @@ function buildContextPrompt(issues, form, maxPatches, previousRejections) {
 						lines.push(`  Rows: ${constraints.join(", ")}`);
 					}
 				}
-				const patchHint = getPatchFormatHint(field.kind, field.id, columnIds);
+				const patchHint = getPatchFormatHint(field.kind, {
+					fieldId: field.id,
+					columnIds,
+					checkboxMode,
+					optionIds
+				});
 				lines.push(`  Set: ${patchHint}`);
 				if (issue.severity === "required") lines.push(`  This field is required.`);
 				else lines.push(`  Skip: { op: "skip_field", fieldId: "${field.id}", reason: "..." }`);
@@ -8728,7 +8810,8 @@ async function fillForm(options) {
 		provider,
 		enableWebSearch: options.enableWebSearch,
 		additionalTools: options.additionalTools,
-		callbacks: options.callbacks
+		callbacks: options.callbacks,
+		maxStepsPerTurn: options.maxStepsPerTurn
 	});
 	let turnCount = startingTurnNumber;
 	let turnsThisCall = 0;
@@ -8957,7 +9040,7 @@ function validateResearchForm(form) {
 //#endregion
 //#region src/index.ts
 /** Markform version (injected at build time). */
-const VERSION = "0.1.12";
+const VERSION = "0.1.14";
 
 //#endregion
 export { parseRawTable as A, parseScopeRef as C, parseForm as D, formToJsonSchema as E, parseCellValue as O, isQualifiedRef as S, fieldToJsonSchema as T, coerceToFieldPatch as _, resolveHarnessConfig as a, isCellRef as b, getProviderNames as c, createLiveAgent as d, MockAgent as f, coerceInputContext as g, createHarness as h, runResearch as i, parseMarkdownTable as k, resolveModel as l, FormHarness as m, isResearchForm as n, fillForm as o, createMockAgent as p, validateResearchForm as r, getProviderInfo as s, VERSION as t, buildMockWireFormat as u, findFieldById as v, serializeScopeRef as w, isFieldRef as x, getFieldId as y };

package/docs/markform-apis.md

@@ -199,6 +199,7 @@ const result = await fillForm({
 | `startingTurnNumber` | `number` | `0` | Starting turn for progress tracking |
 | `maxPatchesPerTurn` | `number` | `20` | Maximum patches per turn |
 | `maxIssuesPerTurn` | `number` | `10` | Maximum issues shown per turn |
+| `maxStepsPerTurn` | `number` | `20` | Maximum AI SDK steps (tool call rounds) per turn |
 | `targetRoles` | `string[]` | `['agent']` | Roles to fill |
 | `fillMode` | `FillMode` | `'continue'` | `'continue'` or `'overwrite'` |
 | `callbacks` | `FillCallbacks` | `undefined` | Progress callbacks |