zames_pro 2.96.5 → 2.96.7

package/CHANGELOG.md

@@ -7,6 +7,26 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [2.96.7] - 2026-11-06
+
+### Fixed
+
+- **tui:** a raw DEL byte is not inserted into the input line
+- **tui:** Down after editing a recalled entry moves to the next entry
+- **tui:** Up/Down keep the visual column on wide-glyph wrapped lines
+- **goal:** a quoted verdict example no longer wins over the real verdict
+- **input:** hold only a partial paste marker, not a lone ESC
+
+## [2.96.6] - 2026-11-06
+
+### Fixed
+
+- **input:** drop a dangling ZWJ when a truncation cuts an emoji sequence
+- **input:** count a ZWJ emoji sequence as one glyph in the layout
+- **tui:** ZAMES_NO_BG=0 no longer disables the themed background
+- **markdown:** an unknown code-fence language falls back to plain text
+- **web:** decode an escaped HTML entity only once
+
 ## [2.96.5] - 2026-11-06
 
 ### Fixed

package/dist/goal.js

@@ -151,7 +151,31 @@ export function parseGoalVerdict(text) {
     const label = /VERDICT\s*[*_`]*\s*:\s*[*_`]*\s*(IMPOSSIBLE|NOT[_ -]?MET|MET)(?![\w-])/gi;
     let m;
     let status = null;
+    // True when the backtick run ending right BEFORE `at` is the CLOSING fence
+    // of a block that started at the beginning of a line.
+    const isFenceClose = (src, at) => {
+        let i = at - 1;
+        while (i >= 0 && src.charAt(i) === '`')
+            i--;
+        const fenceEnd = at;
+        const fenceStart = i + 1;
+        if (fenceEnd - fenceStart < 1)
+            return false;
+        // The opening run must be at the START of a line (or of the text).
+        const lineStart = src.lastIndexOf(String.fromCharCode(10), fenceStart - 1) + 1;
+        return src.slice(lineStart, fenceStart).trim() === '';
+    };
     while ((m = label.exec(t)) !== null) {
+        // A QUOTED or BACKTICKED label is an EXAMPLE the evaluator is explaining,
+        // not its verdict: `A verdict such as "VERDICT: MET" would be wrong here`
+        // followed by the real `VERDICT: NOT_MET` used to clear the goal as DONE.
+        // A quote/backtick directly BEFORE the label marks an example — unless it
+        // is the CLOSING fence of a fenced verdict (```` ```Verdict: MET``` ````),
+        // which is the accepted format and must parse.
+        const before = t.charAt(m.index - 1);
+        if (before === '"' || (before === '`' && !isFenceClose(t, m.index))) {
+            continue;
+        }
         // Skip whitespace/backticks between the value and whatever follows.
         let k = m.index + m[0].length;
         while (k < t.length && /[\s*_`]/.test(t.charAt(k)))
@@ -159,6 +183,10 @@ export function parseGoalVerdict(text) {
         const after = t.charAt(k);
         if (after === '|' || after === '/')
             continue;
+        // A CLOSING quote/backtick right after the value is the other end of an
+        // example (`VERDICT: MET` in prose).
+        if (after === '"' || after === '`')
+            continue;
         const raw = m[1].toUpperCase();
         status =
             raw === 'MET' ? 'met' : raw === 'IMPOSSIBLE' ? 'impossible' : 'not_met';

package/dist/input/layout.js

@@ -167,6 +167,15 @@ export function visRows(s, cols) {
 // zero-width joiner/space/variation selectors count as 0.
 export function visLen(s) {
     s = String(s);
+    // An emoji SEQUENCE renders as ONE glyph but occupies several code
+    // points: ZWJ-joined, a skin-tone modifier, a keycap. Counting each
+    // point separately measured a family as 6 columns instead of 2, so the
+    // layout reserved extra rows and the cursor column drifted. The sequence
+    // is replaced by ONE placeholder cell before measuring.
+    s = s.replace(/(?:\p{Extended_Pictographic}|\p{Emoji_Modifier}|\u200d|\ufe0f)+/gu, 
+    // The placeholder is a WIDE glyph (2 columns), the width an emoji renders
+    // at: a single `X` would under-count it.
+    '😀');
     let n = 0;
     let i = 0;
     while (i < s.length) {
@@ -399,6 +408,11 @@ export function truncateToWidth(s, maxCols) {
         used += w;
         i += ch.length;
     }
+    // A cut inside a ZWJ emoji sequence leaves a DANGLING joiner: the terminal
+    // glues the following `…` to the emoji, so the marker becomes invisible and
+    // the visible line is wider than requested. Drop the dangling tail (the
+    // whole incomplete grapheme) before closing the style.
+    out = out.replace(/\u200d+$/, '');
     // The visible text was cut BEFORE any trailing RESET, so the ellipsis and
     // everything printed after it on the row would inherit the last colour.
     // Close any style the kept prefix opened (the text may still be plain, in
@@ -645,7 +659,10 @@ export function splitPasteTail(s) {
     // The longest marker prefix to consider is one char short of the marker.
     const markers = [PASTE_START, PASTE_END];
     const maxHold = Math.max(...markers.map((m) => m.length)) - 1;
-    const upto = Math.min(maxHold, str.length - 1);
+    // Hold a PARTIAL marker up to the full marker length: for a 1-char chunk
+    // (`str.length - 1` = 0) nothing was held, so a read ending on a lone ESC
+    // leaked it into the buffer instead of waiting for the rest of `ESC[200~`.
+    const upto = Math.min(maxHold, str.length);
     for (let len = upto; len >= 1; len--) {
         const tail = str.slice(str.length - len);
         if (!tail.startsWith(ESC))

package/dist/input.js

@@ -1640,7 +1640,11 @@ export class LineEditor {
         // to _pasteBuf (it is not parsed by this loop), so a held tail would be
         // re-attached to `s` and inserted as text BEFORE the body — the tail
         // (`1~`) leaked into the input line and was not part of the pasted text.
-        const held = this._inPaste
+        // The bare-ESC disambiguation below must still see a chunk that IS just an
+        // ESC: holding it as a partial paste marker would swallow the Escape key
+        // (and a following arrow tail would then be read as literal text). Only a
+        // PARTIAL marker of 2+ chars is held here; the lone ESC keeps its own path.
+        const held = this._inPaste || s === ESC
             ? { head: s, tail: null }
             : splitPasteTail(s);
         if (held.tail) {

package/dist/markdown.js

@@ -98,11 +98,31 @@ export function renderMarkdown(text) {
             // opt-out is NO_COLOR, handled above.
             color: true,
             theme: mdTheme(p),
-            highlighter: (code, lang) => highlight(code, {
-                language: lang || 'plaintext',
-                ignoreIllegals: true,
-                theme: hlTheme(p),
-            }),
+            // An UNKNOWN language tag throws inside cli-highlight, and the throw
+            // escaped the whole render: ONE ```` ```kotlin-ish ```` fence turned the
+            // entire answer into raw markdown (no wrap, no theme, no code box). Fall
+            // back to plaintext for this block only.
+            highlighter: (code, lang) => {
+                try {
+                    return highlight(code, {
+                        language: lang || 'plaintext',
+                        ignoreIllegals: true,
+                        theme: hlTheme(p),
+                    });
+                }
+                catch {
+                    try {
+                        return highlight(code, {
+                            language: 'plaintext',
+                            ignoreIllegals: true,
+                            theme: hlTheme(p),
+                        });
+                    }
+                    catch {
+                        return String(code);
+                    }
+                }
+            },
             codeBox: true,
             tableBorder: 'unicode',
             listIndent: 2,

package/dist/tui/background.js

@@ -23,7 +23,13 @@ function hexToOsc(hex) {
  * testable without a terminal.
  */
 export function backgroundOsc(name) {
-    if (process.env.NO_COLOR || process.env.ZAMES_NO_BG)
+    // NO_COLOR is "present means off" (no-color.org). ZAMES_NO_BG is our OWN
+    // switch: a shell profile writing `ZAMES_NO_BG=0` clearly means KEEP it, so
+    // only a truthy value opts out.
+    const noBg = String(process.env.ZAMES_NO_BG ?? '').toLowerCase();
+    if (process.env.NO_COLOR)
+        return '';
+    if (noBg && noBg !== '0' && noBg !== 'false' && noBg !== 'no')
         return '';
     const id = isThemeName(name) ? name : DEFAULT_THEME;
     return '\x1b]11;rgb:' + hexToOsc(ALL_PALETTES[id].bg) + '\x07';

package/dist/tui/edit.js

@@ -199,9 +199,14 @@ export function moveVisualRows(s, delta, promptStr, cols) {
     // caret past the target row's text and onto the NEXT row entirely.
     const curRow = lay.rows[lay.cursorRow];
     const curChars = Array.from(curRow.text);
+    // Walk the characters of the current row until their VISUAL width reaches
+    // the caret's column. The old loop ran `cursorCol` TIMES over characters:
+    // with a wide (CJK) glyph two columns come from ONE character, so it read
+    // past the row (undefined counted as 1) and Up/Down landed too far right.
     let want = 0;
-    for (let k = 0; k < lay.cursorCol - visLen(curRow.prefix); k++) {
-        want += charWidth(curChars[k]?.codePointAt(0) ?? 32);
+    const targetCol = lay.cursorCol - visLen(curRow.prefix);
+    for (let k = 0; k < curChars.length && want < targetCol; k++) {
+        want += charWidth(curChars[k].codePointAt(0));
     }
     let idx = row.start;
     let w = 0;
@@ -221,8 +226,16 @@ export function moveVisualRows(s, delta, promptStr, cols) {
     // rows" bug.
     if (idx > row.start + rowChars.length)
         idx = row.start + rowChars.length;
-    if (idx === row.start + rowChars.length && rowChars.length > 0)
-        idx--;
+    // Keep the caret INSIDE the target row — but a caret at the row's END is
+    // the natural position for a WIDE last glyph: stepping back one CODE UNIT
+    // put it in the MIDDLE of a surrogate pair or before a CJK glyph, so Up
+    // from the end of a wrapped CJK line landed a glyph too far left. Only step
+    // back when the end is followed by MORE text on the same visual line.
+    if (idx === row.start + rowChars.length && rowChars.length > 0) {
+        const atWide = idx > 0 && charWidth(s.buf.codePointAt(idx - 1) ?? 32) === 2;
+        if (!atWide)
+            idx--;
+    }
     return { ...s, cursor: idx };
 }
 /** Insert a newline; a trailing «\» before the cursor is removed first. */
@@ -274,7 +287,18 @@ export function historyDown(s) {
     // The CURRENT line is the operator's, not a history entry: leaving the
     // history must bring it back. Without this, editing a recalled line and then
     // pressing Down restored the OLD draft and dropped the edit.
-    const draft = browsingHistory(s) ? s.histDraft : s.buf;
+    //
+    // An EDIT of a recalled entry is a scratch copy of THAT slot, not the
+    // operator's draft: `typingRun` marks it. Treating it as "not browsing"
+    // jumped OUT of the history and dropped the entry the operator was fixing —
+    // Down must still move to the NEXT entry.
+    // A REAL edit means the buffer no longer matches the recalled entry: the
+    // `typingRun` flag is set by the KEY layer, not by `insertText` itself, so
+    // the comparison is what is reliable here.
+    const recalled = s.history[s.histIndex];
+    const editingRecalled = s.histIndex < s.history.length &&
+        (s.typingRun || (recalled !== undefined && s.buf !== recalled));
+    const draft = browsingHistory(s) && !editingRecalled ? s.histDraft : s.buf;
     let buf;
     if (histIndex < s.history.length - 1) {
         histIndex++;

package/dist/tui/keys.js

@@ -197,7 +197,11 @@ export function dispatchKey(input, key, ctx, isVisualEdge) {
     if (key.escape)
         return { type: 'escape' };
     // ---- printable text ----
-    if (input && input >= ' ') {
+    // A raw DEL byte (0x7f) is NOT printable: some terminals hand it over as
+    // plain `input` without `key.backspace`, and `input >= ' '` (DEL is 127)
+    // inserted it as an invisible control character into the buffer.
+    const printable = input && input !== String.fromCharCode(127) && input >= ' ';
+    if (printable) {
         return { type: 'insert', text: input };
     }
     return null;

package/dist/web.js

@@ -77,10 +77,13 @@ export function htmlToText(html) {
     // A parked `<pre>` block comes back AFTER the tag strip, with only its own
     // tags removed and its text kept verbatim.
     s = s.replace(/\u0000PRE(\d+)PRE\u0000/g, (_m, i) => (parkedPre[Number(i)] ?? '').replace(/<[^>]+>/g, ''));
-    // HTML entities — the basic ones
+    // HTML entities — the basic ones. `&amp;` is decoded LAST, otherwise an
+    // escaped entity (`&amp;#38;` — the literal text `&#38;`) is decoded TWICE
+    // and the page's text is changed.
+    const AMP = String.fromCharCode(0) + 'amp' + String.fromCharCode(0);
+    s = s.replace(/&amp;/g, AMP);
     s = s
         .replace(/&nbsp;/g, ' ')
-        .replace(/&amp;/g, '&')
         .replace(/&lt;/g, '<')
         .replace(/&gt;/g, '>')
         .replace(/&quot;/g, '"')
@@ -89,7 +92,9 @@ export function htmlToText(html) {
         .replace(/&ndash;/g, '–')
         .replace(/&hellip;/g, '…')
         .replace(/&#(\d+);/g, (_, n) => decodeEntityCode(Number(n)))
-        .replace(/&#x([0-9a-f]+);/gi, (_, n) => decodeEntityCode(parseInt(n, 16)));
+        .replace(/&#x([0-9a-f]+);/gi, (_, n) => decodeEntityCode(parseInt(n, 16)))
+        .split(AMP)
+        .join('&');
     // Collapse blank lines
     s = s
         .split('\n')

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "zames_pro",
-  "version": "2.96.5",
+  "version": "2.96.7",
   "description": "Terminal coding agent that drives a real browser to chat with DeepSeek, Qwen or GLM — no API key required. Reads and edits files, runs commands, commits to git, like Claude Code / Codex CLI.",
   "type": "module",
   "bin": {