@mattstack/rt-client 0.29.0 → 0.30.0

package/dist/commands.d.ts

@@ -153,6 +153,7 @@ export interface GateAnswer {
     answers: Record<string, string | string[] | {
         value: string | string[];
         note?: string;
+        text?: string;
     }>;
     by: string;
     answeredAt: number;

package/dist/gate-answers.d.ts

@@ -1,7 +1,8 @@
 import type { GateQuestion, GateAnswer } from "./commands.ts";
 export type GateAnswerWire = GateAnswer["answers"][string];
-/** Both wire shapes carry the same value underneath: bare, or {value, note?}
-    when a panel attaches free text. Validation reads only the value. */
+/** Both wire shapes carry the same value underneath: bare, or {value,
+    note?, text?} when a panel attaches free text or a replacement for text
+    the gate offered. Unwrapping keeps only the value. */
 export declare function unwrapGateAnswerValue(raw: unknown): unknown;
 /** Option membership is required whenever a question declares options,
     checked against the unwrapped value (every element, for multi); an

package/dist/gate.js

@@ -54,12 +54,18 @@ function unwrapGateAnswerValue(raw) {
   }
   return raw;
 }
-function wrapperNoteIsValid(raw) {
+function wrapperFieldError(qid, raw) {
   if (!raw || typeof raw !== "object" || Array.isArray(raw) || !("value" in raw)) {
-    return true;
+    return null;
   }
-  const note = raw.note;
-  return note === undefined || typeof note === "string";
+  const { note, text } = raw;
+  if (note !== undefined && typeof note !== "string")
+    return `question ${qid} note must be a string`;
+  if (text !== undefined && typeof text !== "string")
+    return `question ${qid} text must be a string`;
+  if (typeof text === "string" && text.trim() === "")
+    return `question ${qid} text must not be empty`;
+  return null;
 }
 function validateGateAnswers(questions, answers) {
   const byId = new Map(questions.map((q) => [q.id, q]));
@@ -67,8 +73,9 @@ function validateGateAnswers(questions, answers) {
     const question = byId.get(qid);
     if (!question)
       return `unknown question id: ${qid}`;
-    if (!wrapperNoteIsValid(raw))
-      return `question ${qid} note must be a string`;
+    const wrapperError = wrapperFieldError(qid, raw);
+    if (wrapperError)
+      return wrapperError;
     const value = unwrapGateAnswerValue(raw);
     const isArray = Array.isArray(value);
     if (question.multi && !isArray)

package/dist/index.js

@@ -643,12 +643,18 @@ function unwrapGateAnswerValue(raw) {
   }
   return raw;
 }
-function wrapperNoteIsValid(raw) {
+function wrapperFieldError(qid, raw) {
   if (!raw || typeof raw !== "object" || Array.isArray(raw) || !("value" in raw)) {
-    return true;
+    return null;
   }
-  const note = raw.note;
-  return note === undefined || typeof note === "string";
+  const { note, text } = raw;
+  if (note !== undefined && typeof note !== "string")
+    return `question ${qid} note must be a string`;
+  if (text !== undefined && typeof text !== "string")
+    return `question ${qid} text must be a string`;
+  if (typeof text === "string" && text.trim() === "")
+    return `question ${qid} text must not be empty`;
+  return null;
 }
 function validateGateAnswers(questions, answers) {
   const byId = new Map(questions.map((q) => [q.id, q]));
@@ -656,8 +662,9 @@ function validateGateAnswers(questions, answers) {
     const question = byId.get(qid);
     if (!question)
       return `unknown question id: ${qid}`;
-    if (!wrapperNoteIsValid(raw))
-      return `question ${qid} note must be a string`;
+    const wrapperError = wrapperFieldError(qid, raw);
+    if (wrapperError)
+      return wrapperError;
     const value = unwrapGateAnswerValue(raw);
     const isArray = Array.isArray(value);
     if (question.multi && !isArray)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mattstack/rt-client",
-  "version": "0.29.0",
+  "version": "0.30.0",
   "type": "module",
   "exports": {
     ".": {

package/src/commands.ts

@@ -133,7 +133,7 @@ export { gateOptionValue, gateOptionLabel } from "./gate-options.ts";
     doorbell it would otherwise send back to the writer. Optional: a caller
     that supplies none (the board status-bin answers `by: "pane"` with no
     session) still matches self by `by === GATE_BY_PANE`. */
-export interface GateAnswer { answers: Record<string, string | string[] | { value: string | string[]; note?: string }>; by: string; answeredAt: number; overridden?: boolean; session?: string }
+export interface GateAnswer { answers: Record<string, string | string[] | { value: string | string[]; note?: string; text?: string }>; by: string; answeredAt: number; overridden?: boolean; session?: string }
 export interface GateRow {
   id: string; subject: string; kind: string;
   questions: GateQuestion[]; meta: Record<string, unknown> | null;

package/src/gate-answers.ts

@@ -3,8 +3,9 @@ import { gateOptionValue } from "./gate-options.ts";
 
 export type GateAnswerWire = GateAnswer["answers"][string];
 
-/** Both wire shapes carry the same value underneath: bare, or {value, note?}
-    when a panel attaches free text. Validation reads only the value. */
+/** Both wire shapes carry the same value underneath: bare, or {value,
+    note?, text?} when a panel attaches free text or a replacement for text
+    the gate offered. Unwrapping keeps only the value. */
 export function unwrapGateAnswerValue(raw: unknown): unknown {
   if (raw && typeof raw === "object" && !Array.isArray(raw) && "value" in (raw as Record<string, unknown>)) {
     return (raw as { value: unknown }).value;
@@ -12,12 +13,15 @@ export function unwrapGateAnswerValue(raw: unknown): unknown {
   return raw;
 }
 
-function wrapperNoteIsValid(raw: unknown): boolean {
+function wrapperFieldError(qid: string, raw: unknown): string | null {
   if (!raw || typeof raw !== "object" || Array.isArray(raw) || !("value" in (raw as Record<string, unknown>))) {
-    return true;
+    return null;
   }
-  const note = (raw as Record<string, unknown>).note;
-  return note === undefined || typeof note === "string";
+  const { note, text } = raw as Record<string, unknown>;
+  if (note !== undefined && typeof note !== "string") return `question ${qid} note must be a string`;
+  if (text !== undefined && typeof text !== "string") return `question ${qid} text must be a string`;
+  if (typeof text === "string" && text.trim() === "") return `question ${qid} text must not be empty`;
+  return null;
 }
 
 /** Option membership is required whenever a question declares options,
@@ -32,7 +36,8 @@ export function validateGateAnswers(
   for (const [qid, raw] of Object.entries(answers)) {
     const question = byId.get(qid);
     if (!question) return `unknown question id: ${qid}`;
-    if (!wrapperNoteIsValid(raw)) return `question ${qid} note must be a string`;
+    const wrapperError = wrapperFieldError(qid, raw);
+    if (wrapperError) return wrapperError;
     const value = unwrapGateAnswerValue(raw);
     const isArray = Array.isArray(value);
     if (question.multi && !isArray) return `question ${qid} expects an array (multi)`;