@xynogen/pix-ask 0.2.0 → 0.2.2

package/package.json

@@ -1,7 +1,7 @@
 {
 	"name": "@xynogen/pix-ask",
-	"version": "0.2.0",
-	"description": "Pi tool — structured questionnaire UI (ask_user)",
+	"version": "0.2.2",
+	"description": "Pi tool \u2014 structured questionnaire UI (ask_user)",
 	"type": "module",
 	"main": "src/index.ts",
 	"scripts": {
@@ -34,7 +34,8 @@
 		"access": "public"
 	},
 	"dependencies": {
-		"typebox": "^1.1.38"
+		"typebox": "^1.1.38",
+		"@xynogen/pix-pretty": "*"
 	},
 	"peerDependencies": {
 		"@earendil-works/pi-coding-agent": "*",

package/src/frame.ts

@@ -1,30 +1,54 @@
+/**
+ * Modal frame primitives — inlined from pix-pretty/modal-frame to avoid
+ * cross-package subpath imports that break under Node CJS require.
+ */
+
 import { truncateToWidth, visibleWidth } from "@earendil-works/pi-tui";
 
+// ── Constants ─────────────────────────────────────────────────────────────────
+
 const MIN_WIDTH = 40;
 const MAX_WIDTH = 96;
 const MARGIN = 4;
-// 2 border cols + 2 padding spaces
+/** 2 border cols + 2 padding spaces */
 const CHROME = 4;
 
+// ── Width ─────────────────────────────────────────────────────────────────────
+
+/** Clamp terminal width to a sane modal width (40–96 cols). */
 export function modalWidth(termWidth: number): number {
 	return Math.min(MAX_WIDTH, Math.max(MIN_WIDTH, termWidth - MARGIN));
 }
 
-export function frameLines(opts: {
+// ── Frame ─────────────────────────────────────────────────────────────────────
+
+export interface FrameOptions {
 	width: number;
 	lines: string[];
+	/** Color function for border glyphs — e.g. `(s) => theme.fg("accent", s)` */
 	color: (s: string) => string;
+	/** Background fill function — e.g. `(s) => theme.bg("customMessageBg", s)` */
 	bg?: (s: string) => string;
+	/** Optional pre-styled string rendered as the first content row (tab bar etc.) */
 	top?: string;
-}): string[] {
+}
+
+/**
+ * Render a rounded modal box.
+ *
+ * Returns an array of full-width ANSI strings:
+ *   ╭──────────────────╮
+ *   │ [top row]        │   ← only if top is set
+ *   │ content line 1   │
+ *   │ content line 2   │
+ *   ╰──────────────────╯
+ */
+export function frameLines(opts: FrameOptions): string[] {
 	const { width, lines, color, top } = opts;
 	const bg = opts.bg ?? ((s: string) => s);
 	const inner = Math.max(1, width - CHROME);
 	const dashes = "─".repeat(width - 2);
 
-	// Derive the bg OPEN sequence so we can re-assert it after any full reset
-	// (\x1b[0m) or bg reset (\x1b[49m) embedded in content — theme fg/bold spans
-	// emit \x1b[0m, which would otherwise punch transparent holes in the fill.
 	const SENTINEL = "\x00";
 	const bgOpen = bg(SENTINEL).split(SENTINEL)[0] ?? "";
 	const reassert = (s: string): string =>