@crouton-kit/humanloop 0.3.17 → 0.3.19

package/dist/api.d.ts

@@ -12,14 +12,6 @@ export interface AskOpts {
  * on completion, `<dir>/response.json`.
  */
 export declare function ask(deck: Deck, opts?: AskOpts): Promise<ResolutionEnvelope>;
-export interface ApproveOpts {
-    subtitle?: string;
-    body?: string;
-    dir?: string;
-    sessionId?: string;
-}
-/** Sugar: a single `kind:'validation'` Yes/No interaction. */
-export declare function approve(title: string, opts?: ApproveOpts): Promise<boolean>;
 /** Sugar: a single `kind:'notify'` acknowledgement. */
 export declare function notify(title: string, body?: string): Promise<void>;
 export interface InboxOpts {

package/dist/api.js

@@ -6,7 +6,7 @@ import { scanInbox } from './inbox/scan.js';
 import { pickFromInbox } from './inbox/tui.js';
 import { deckPath, atomicWriteJson, readJson, stampCanvasNode } from './inbox/convention.js';
 import { getTerminalSize } from './tui/terminal.js';
-import { approveDeck, notifyDeck } from './inbox/deck-factories.js';
+import { notifyDeck } from './inbox/deck-factories.js';
 const RESPONSE_SCHEMA_ID = 'humanloop.response/v2';
 function managedDir() {
     return mkdtempSync(join(tmpdir(), 'hl-ix-'));
@@ -78,12 +78,6 @@ export async function ask(deck, opts = {}) {
         completedAt,
     };
 }
-/** Sugar: a single `kind:'validation'` Yes/No interaction. */
-export async function approve(title, opts = {}) {
-    const deck = approveDeck(title, { subtitle: opts.subtitle, body: opts.body });
-    const env = await ask(deck, { dir: opts.dir, sessionId: opts.sessionId });
-    return env.responses[0]?.selectedOptionId === 'yes';
-}
 /** Sugar: a single `kind:'notify'` acknowledgement. */
 export async function notify(title, body) {
     const deck = notifyDeck(title, body !== undefined ? { body } : {});

package/dist/cli.js

@@ -217,7 +217,7 @@ const REQUEST_SCHEMA = {
                     },
                     allowFreetext: { type: 'boolean' },
                     freetextLabel: { type: 'string' },
-                    kind: { type: 'string', enum: ['notify', 'validation', 'decision', 'context', 'error'] },
+                    kind: { type: 'string', enum: ['notify', 'decision', 'context', 'error'] },
                 },
             },
         },

package/dist/editor/review.js

@@ -115,20 +115,25 @@ export function reviewVimscript() {
         `  endif`,
         `  call setreg('z', l:sz, l:szt)`,
         `  let l:label = l:l1 == l:l2 ? ('line ' . l:l1) : ('lines ' . l:l1 . '-' . l:l2)`,
-        `  let l:txt = input('Comment on ' . l:label . ': ')`,
-        `  redraw`,
-        `  if empty(trim(l:txt))`,
+        `  call s:OpenInput('Comment on ' . l:label, '', function('s:CommentSave', [l:l1, l:l2, l:quote, l:cs, l:ce, a:mode]))`,
+        `endfunction`,
+        ``,
+        `" Persist a comment once the input buffer is submitted. Bound args carry the`,
+        `" anchor (line range, quote, col span, mode) captured before the prompt`,
+        `" opened; a:txt is the (possibly multi-line) comment body.`,
+        `function! s:CommentSave(l1, l2, quote, cs, ce, mode, txt) abort`,
+        `  if empty(trim(a:txt))`,
         `    echohl WarningMsg | echo 'Comment cancelled' | echohl NONE`,
         `    return`,
         `  endif`,
         `  let s:idseq += 1`,
-        `  let l:item = {'id': 'c' . localtime() . s:idseq, 'line': l:l1, 'endLine': l:l2, 'lineText': join(getline(l:l1, l:l2), "\\n"), 'comment': l:txt, 'createdAt': strftime('%Y-%m-%dT%H:%M:%S')}`,
-        `  if a:mode ==# 'v' && !empty(l:quote)`,
-        `    let l:item['quote'] = l:quote`,
+        `  let l:item = {'id': 'c' . localtime() . s:idseq, 'line': a:l1, 'endLine': a:l2, 'lineText': join(getline(a:l1, a:l2), "\\n"), 'comment': a:txt, 'createdAt': strftime('%Y-%m-%dT%H:%M:%S')}`,
+        `  if a:mode ==# 'v' && !empty(a:quote)`,
+        `    let l:item['quote'] = a:quote`,
         `  endif`,
-        `  if l:cs >= 0 && l:ce > l:cs`,
-        `    let l:item['colStart'] = l:cs`,
-        `    let l:item['colEnd'] = l:ce`,
+        `  if a:cs >= 0 && a:ce > a:cs`,
+        `    let l:item['colStart'] = a:cs`,
+        `    let l:item['colEnd'] = a:ce`,
         `  endif`,
         `  call add(s:comments, l:item)`,
         `  call s:Save()`,
@@ -136,6 +141,58 @@ export function reviewVimscript() {
         `  echo 'Saved — ' . len(s:comments) . ' comment' . (len(s:comments)==1?'':'s')`,
         `endfunction`,
         ``,
+        `" Multi-line comment entry. Vim's input() is single-line: a long comment`,
+        `" overflows the cmdline and Vim re-echoes the prompt on every wrapped row,`,
+        `" cascading it down the screen. So comments are typed in a real scratch`,
+        `" buffer that wraps naturally and accepts newlines. a:Cb is invoked with the`,
+        `" buffer text on submit; cancel (or empty submit) just closes.`,
+        `function! s:OpenInput(title, default, Cb) abort`,
+        `  let l:rw = win_getid()`,
+        `  botright 8new`,
+        `  let b:hl_cb = a:Cb`,
+        `  let b:hl_rw = l:rw`,
+        `  let b:hl_done = 0`,
+        `  setlocal buftype=nofile bufhidden=wipe noswapfile nobuflisted`,
+        `  setlocal nonumber winfixheight wrap linebreak signcolumn=no`,
+        `  if !empty(a:default)`,
+        `    call setline(1, split(a:default, "\\n", 1))`,
+        `  endif`,
+        `  let &l:statusline = ' ' . a:title . '    Ctrl-S / <Space>s submit · Ctrl-C / q cancel '`,
+        `  nnoremap <buffer> <silent> <C-s> :call <SID>InputSubmit()<CR>`,
+        `  inoremap <buffer> <silent> <C-s> <Esc>:call <SID>InputSubmit()<CR>`,
+        `  nnoremap <buffer> <silent> <Space>s :call <SID>InputSubmit()<CR>`,
+        `  nnoremap <buffer> <silent> <C-c> :call <SID>InputCancel()<CR>`,
+        `  inoremap <buffer> <silent> <C-c> <Esc>:call <SID>InputCancel()<CR>`,
+        `  nnoremap <buffer> <silent> q :call <SID>InputCancel()<CR>`,
+        `  if !empty(a:default)`,
+        `    call cursor(line('$'), 1)`,
+        `    startinsert!`,
+        `  else`,
+        `    startinsert`,
+        `  endif`,
+        `endfunction`,
+        ``,
+        `function! s:InputSubmit() abort`,
+        `  if get(b:, 'hl_done', 0) | return | endif`,
+        `  let b:hl_done = 1`,
+        `  let l:txt = join(getline(1, '$'), "\\n")`,
+        `  let l:Cb = b:hl_cb`,
+        `  let l:rw = b:hl_rw`,
+        `  close!`,
+        `  call win_gotoid(l:rw)`,
+        `  call l:Cb(l:txt)`,
+        `endfunction`,
+        ``,
+        `function! s:InputCancel() abort`,
+        `  if get(b:, 'hl_done', 0) | return | endif`,
+        `  let b:hl_done = 1`,
+        `  let l:rw = b:hl_rw`,
+        `  close!`,
+        `  call win_gotoid(l:rw)`,
+        `  redraw`,
+        `  echohl WarningMsg | echo 'Cancelled' | echohl NONE`,
+        `endfunction`,
+        ``,
         `function! s:Undo() abort`,
         `  if empty(s:comments)`,
         `    echohl WarningMsg | echo 'No comments to undo' | echohl NONE`,
@@ -163,7 +220,7 @@ export function reviewVimscript() {
         `      let l:ln = get(l:c,'line',0)`,
         `      let l:end = get(l:c,'endLine',l:ln)`,
         `      let l:loc = l:ln == l:end ? ('L' . l:ln) : ('L' . l:ln . '-' . l:end)`,
-        `      call add(l:lines, (l:idx+1) . '. [' . l:loc . ']  ' . get(l:c,'comment',''))`,
+        `      call add(l:lines, (l:idx+1) . '. [' . l:loc . ']  ' . substitute(get(l:c,'comment',''), "\\n", ' / ', 'g'))`,
         `      call add(l:map, l:idx)`,
         `      if !empty(get(l:c,'quote',''))`,
         `        call add(l:lines, '      > ' . substitute(get(l:c,'quote',''), "\\n", ' / ', 'g'))`,
@@ -211,12 +268,14 @@ export function reviewVimscript() {
         `    echohl WarningMsg | echo 'No comment on this line' | echohl NONE | return`,
         `  endif`,
         `  let l:cur = get(s:comments[l:idx], 'comment', '')`,
-        `  let l:txt = input('Edit comment: ', l:cur)`,
-        `  redraw`,
-        `  if empty(trim(l:txt))`,
+        `  call s:OpenInput('Edit comment', l:cur, function('s:ListEditSave', [l:idx]))`,
+        `endfunction`,
+        ``,
+        `function! s:ListEditSave(idx, txt) abort`,
+        `  if empty(trim(a:txt))`,
         `    echohl WarningMsg | echo 'Edit cancelled — comment unchanged' | echohl NONE | return`,
         `  endif`,
-        `  let s:comments[l:idx]['comment'] = l:txt`,
+        `  let s:comments[a:idx]['comment'] = a:txt`,
         `  call s:Save()`,
         `  call s:RenderList()`,
         `  echo 'Updated comment'`,
@@ -419,7 +478,7 @@ export function formatFeedbackSummary(result, feedbackJsonPath) {
                 const lines = original.split('\n');
                 out.push(` ${i + 1}. ${rangeLabel(c)}`);
                 lines.forEach((ln, k) => out.push(k === 0 ? `    text:    ${ln}` : `             ${ln}`));
-                out.push(`    comment: ${c.comment}`);
+                c.comment.split('\n').forEach((ln, k) => out.push(k === 0 ? `    comment: ${ln}` : `             ${ln}`));
                 out.push('');
             });
         }
@@ -438,7 +497,7 @@ export function formatFeedbackSummary(result, feedbackJsonPath) {
             const lines = original.split('\n');
             out.push(` ${i + 1}. ${rangeLabel(c)}`);
             lines.forEach((ln, k) => out.push(k === 0 ? `    text:    ${ln}` : `             ${ln}`));
-            out.push(`    comment: ${c.comment}`);
+            c.comment.split('\n').forEach((ln, k) => out.push(k === 0 ? `    comment: ${ln}` : `             ${ln}`));
             out.push('');
         });
     }

package/dist/inbox/deck-factories.d.ts

@@ -1,10 +1,4 @@
 import type { Deck } from '../types.js';
-export interface ApproveDeckOpts {
-    subtitle?: string;
-    body?: string;
-}
-/** Build a validated Yes/No validation deck. id: 'approve', kind: 'validation'. */
-export declare function approveDeck(title: string, opts?: ApproveDeckOpts): Deck;
 export interface NotifyDeckOpts {
     body?: string;
 }

package/dist/inbox/deck-factories.js

@@ -1,20 +1,4 @@
 import { validateDeck } from './deck-schema.js';
-/** Build a validated Yes/No validation deck. id: 'approve', kind: 'validation'. */
-export function approveDeck(title, opts = {}) {
-    return validateDeck({
-        interactions: [{
-                id: 'approve',
-                title,
-                ...(opts.subtitle !== undefined ? { subtitle: opts.subtitle } : {}),
-                ...(opts.body !== undefined ? { body: opts.body } : {}),
-                kind: 'validation',
-                options: [
-                    { id: 'yes', label: 'Yes' },
-                    { id: 'no', label: 'No' },
-                ],
-            }],
-    });
-}
 /** Build a validated single-option notify deck. id: 'notify', kind: 'notify'. */
 export function notifyDeck(title, opts = {}) {
     return validateDeck({

package/dist/inbox/deck-schema.d.ts

@@ -35,7 +35,6 @@ export declare const deckSchema: z.ZodObject<{
         freetextLabel: z.ZodOptional<z.ZodString>;
         kind: z.ZodOptional<z.ZodEnum<{
             notify: "notify";
-            validation: "validation";
             decision: "decision";
             context: "context";
             error: "error";

package/dist/inbox/deck-schema.js

@@ -29,7 +29,7 @@ const interactionSchema = z.object({
     multiSelect: z.boolean().optional(),
     allowFreetext: z.boolean().optional(),
     freetextLabel: z.string().optional(),
-    kind: z.enum(['notify', 'validation', 'decision', 'context', 'error']).optional(),
+    kind: z.enum(['notify', 'decision', 'context', 'error']).optional(),
     preAnswered: preAnswerSchema.optional(),
 });
 const deckSourceSchema = z.object({

package/dist/inbox/tui.js

@@ -23,14 +23,12 @@ function ansiColor(text, color) {
 // ── Row model (ported verbatim from sisyphus cross-session-inbox.ts:8-21) ────
 export const KIND_ICON = {
     notify: '✉',
-    validation: '✓',
     decision: '◆',
     context: '✎',
     error: '⚠',
 };
 export const KIND_COLOR = {
     notify: 'gray',
-    validation: 'cyan',
     decision: 'cyan',
     context: 'cyan',
     error: 'red',

package/dist/index.d.ts

@@ -3,13 +3,13 @@ export { defaultGenerateVisual } from './visuals/generate.js';
 export { launchReview } from './editor/review.js';
 export { launchReview as review } from './editor/review.js';
 export type { ReviewOptions } from './editor/review.js';
-export { ask, approve, notify, inbox } from './api.js';
+export { ask, notify, inbox } from './api.js';
 export { display } from './surfaces/display.js';
 export { scanInbox } from './inbox/scan.js';
 export { renderMarkdown, checkMarkdown, ensureRenderer, isRendererReady, } from './render/termrender.js';
 export { parseDeck, validateDeck, deckSchema } from './inbox/deck-schema.js';
-export { approveDeck, notifyDeck } from './inbox/deck-factories.js';
-export type { ApproveDeckOpts, NotifyDeckOpts } from './inbox/deck-factories.js';
+export { notifyDeck } from './inbox/deck-factories.js';
+export type { NotifyDeckOpts } from './inbox/deck-factories.js';
 export { deckPath, responsePath, progressPath, visualsDir, interactionState, isResolved, isClaimed, atomicWriteJson, readJson, writeResponse, writeProgress, clearProgress, } from './inbox/convention.js';
 export type { InteractionState } from './inbox/convention.js';
 export type { Interaction, InteractionOption, InteractionResponse, InteractionKind, Deck, DeckSource, MountedPanel, MountedPanelOpts, GenerateVisual, VisualBlock, FeedbackComment, FeedbackResult, ResolutionEnvelope, InboxItem, DisplayOpts, } from './types.js';

package/dist/index.js

@@ -3,7 +3,7 @@ export { defaultGenerateVisual } from './visuals/generate.js';
 export { launchReview } from './editor/review.js';
 export { launchReview as review } from './editor/review.js';
 // Interaction-layer surface (SDK).
-export { ask, approve, notify, inbox } from './api.js';
+export { ask, notify, inbox } from './api.js';
 export { display } from './surfaces/display.js';
 export { scanInbox } from './inbox/scan.js';
 // Renderer binding — the sole org-wide termrender caller. Consumers
@@ -13,6 +13,6 @@ export { renderMarkdown, checkMarkdown, ensureRenderer, isRendererReady, } from
 export { parseDeck, validateDeck, deckSchema } from './inbox/deck-schema.js';
 // Deck factories — pure builders for common deck shapes (sugar for SDK consumers
 // who want validated Yes/No or notify decks without inline construction).
-export { approveDeck, notifyDeck } from './inbox/deck-factories.js';
+export { notifyDeck } from './inbox/deck-factories.js';
 // Interaction-directory convention helpers (§B) — names humanloop owns.
 export { deckPath, responsePath, progressPath, visualsDir, interactionState, isResolved, isClaimed, atomicWriteJson, readJson, writeResponse, writeProgress, clearProgress, } from './inbox/convention.js';

package/dist/types.d.ts

@@ -1,5 +1,5 @@
 import type { Key } from './tui/terminal.js';
-export type InteractionKind = 'notify' | 'validation' | 'decision' | 'context' | 'error' | 'review';
+export type InteractionKind = 'notify' | 'decision' | 'context' | 'error' | 'review';
 export interface InteractionOption {
     id: string;
     label: string;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@crouton-kit/humanloop",
-  "version": "0.3.17",
+  "version": "0.3.19",
   "description": "Human-in-the-loop decision TUI — agents write questions, humans answer them",
   "type": "module",
   "main": "dist/index.js",