From 8864e5cec34be1da8befbae6d1a6db1b3afbaa72 Mon Sep 17 00:00:00 2001
From: Julian Noble <julian@precisium.com.au>
Date: Mon, 3 Aug 2026 21:59:34 +1000
Subject: [PATCH] vfscommonupdate: promote shellfilter 0.2.5 into _vfscommon

Promotes the minted shellfilter 0.2.5 (transchan clear/flush fixes - see prior
commit) into src/vfs/_vfscommon.vfs for the next bake, replacing 0.2.4.
Also carries the punk::mix::templates modpod refresh from the mint's
thin-layout sync: the punk.project layout's bootsupport include_modules.config
payload catches up to the commandstack manifest repoint (src/vendormodules ->
modules, G-160 follow-through) plus the modpod container restamp.

Claude-Session: https://claude.ai/code/session_01QgaxV27VZkmEec7oNbEVFc
Assisted-by: harness=claude; primary-model=claude-fable-5; api-location=anthropic.com
---
 .../modules/punk/mix/templates-0.2.0.tm       | Bin 375428 -> 375432 bytes
 ...llfilter-0.2.4.tm => shellfilter-0.2.5.tm} |  72 ++++++++++++------
 2 files changed, 47 insertions(+), 25 deletions(-)
 rename src/vfs/_vfscommon.vfs/modules/{shellfilter-0.2.4.tm => shellfilter-0.2.5.tm} (98%)

diff --git a/src/vfs/_vfscommon.vfs/modules/punk/mix/templates-0.2.0.tm b/src/vfs/_vfscommon.vfs/modules/punk/mix/templates-0.2.0.tm
index dfcc2357347898e22b7313cfbf0007106b66c831..8b65b3dcba6493cc36b4570da67301ce4bad78f2 100644
GIT binary patch
delta 1861
zcmY+_dpuNG902g{y>oW7NYR+Bl~7b@^bn0wrgxO4KBz`WO^+=}EA%AWLoYNY^Bb8A
z(I-2`J4We6t5oYT<()-(DQ%3JwoTbRv--?Vf1G<izwh~--#O==^ZDE(YSZIt)4cI!
z<lmCRHf9PNGoqVkI9)>dDL=+WwxJ<I!W8BaL=P1eY~GwhiS%;)4JGL*Th#aO=2X6+
zG4`LB{LG4VYMK1x@ky^eNW=IQdH(O^hG`Be7nQHTn%X%f>f`~teRl5amgO*B%&t0L
zu;f;g(}bo&Z;u@DIb~wJhMTG19qFnTz0RL{YTlaR;^!6PH{OW2&s6nIjtY2&nA?{H
z)$y0o)KLSY#ZQAA3jSi2Z%^EO-!x^_sFcpt3}$<$UEJGewJ)~HrnS{1xE2lkG&RO9
zTB^$HDRNnCU%N1j$Lc-0&1p+zpdv-YYmCfZF!5Ob)*UAp>BbIpX2gR5>x;?*CtODq
z=k;_c>poeZu<4(AbkSz-KMgYu+N?U-uNGJL4~WIZQH!G2c^CPx`%4a#+>I2lZT|3{
zxn*>6y`z7byxz%0vfNrwAbXi#J?Fu7h-+9adL_ECAXgS`6BOH*88(T<lLR~$%0qv5
z8LMCEU4Nm>Q55mXC6YbAsnJtSgw{(f&G?IjD_?BdRm^|Y;J9sV$DXXhDaXa>?sC2Q
zO^hy!eNk<t^gBVZ(p~K8V5ylvm7Cu!Uv(+vd;VGLzX~et>-KiK);u<N=iN$w^t9RP
zcGKuNGK(7H_R=MuOItqKv#XqEGHXnV%;=$(hKnCk{q{-~vuSO?8k-OHWoc_0SnbEo
zGuQ=B;wP>M;3v(t^QhQqMc-ZcNW7DIkN8qCDKs-m$Z~F77Q1_Ej{Rk~#n44FVto<~
zG-g-`7lvO8>GidIz-XqwQUt`A$L?#9Ga8<{=(iXpE@wKri=!_ZF|9abonn5hQLO52
zO;~<9{iE>=I&%!t|D40#Q`g&}I{f^W$^JTr4~fwyd-wOfy)@>@U88NK{r7J3`{r^z
zDB8#&aO@yq3Vrbe7g7bt1pYZln1eKeKrt4P1a2g56mB$yMI<sL@pV=>974p1Z|^d=
z3Xp=_8VG3#Xbboypm{_Kq$VI4>COkG5P3tI5U~gz)C$oVf)0UaQ37#A^WuYN450_>
z6Ok=x6$FBK#Dx$MGJw3}gaN$QaO(~&bI9IcZD0UKfSgI&5FN<?G=t>t(h+Adnnn8j
ztQ8ux#^DKT#klgVP#xwbAy;y2*tenw*#crHyidZ};o)EV&Y{`lvj`y0BQJQDff&F{
zM&{(ZJsMd9@yUoos6m;GfUMlFF@LC(AZNlzbRH!V?fB^r!RLsPpqGNHGGa9;evt7i
zF&ySd@XoVSaJBB(!O6x*(KKQlaHZ%pxibz{O3`ZgD5H$v%tf5aO~t7d@q_Fs$J`Nz
zxf3%n28c8~C{3v8<wAQJ-kz5jRO!eU_Fh2K$>3y7z=y*!yoCwrxKhaljf4R2CN&J!
zmf%WC%tSu)rlU)w$yL~ugM6ST1Aq14(!n4v3(?_ABc-odkS$q#O^5xOsUP6940F|W
z9bVT$jRIB{=C&I;yyG#>uF-JSO&$Jn3C$;N04EbkiHGnc6Qz>n<=XzC&=`kTFS&?U
zZ}poFhs%%~X<4Bo%q+wqr`^%YQquCSHW&&#Iqo*D)RE6}G?%owrz4VVT-5ZwM*LxR
z4w_DcKu8W=SU3#N#mC)H1tW6NMli_3d0o|-AP{o%@RgibI~eE*(RkRNk5}uZ1WqBo
zK<)YXC}Ns4fghNv@ev*>z~3OfQzPrapb*cp6x<4tnDY0cfgpqy2HeZYmg*afHbDDj
zG>aN`iw2xlS`d8Bz{i$Ygv_b1vi~Rm=xww`;9HExxRrmUm&JHY@@<+><>dh(3N(Z2
zxT6Wi!Q?2U2Xmg%hQSjB9ueB0!|W2|PI}n^Cy5w8n25y;s!EUp`3p~nJHn9>u&>~K
n%nj7?Fn}w_k$m&N7z1?`vB<^^+8ibXUq!>e)92$q<dA;>9#{=_

delta 1922
zcmZvd4OEO-7{`0>%vDNXqL@Z)6n0dsu&io_qpYo^EKzOSv`S3!k=l@&A%yfXWAYSz
zr!;+NDm4}I5fNoepH|tlk<vD-uPm~6_T4j^bM`&wp7Z{H&vXCxKF_`H%;8st2VNP<
z%-Kl8EOI(qKAlZH_0X2&Ag?ckPcl2Pnwmgv5=to;0$2ZLi9*Ohh3g!W65_l>_r`MG
zjrY9i@PlE}dY1l+7GSbW(s&o2xEn9^?g{CUPjwpefH!{PTep{Pl8Qv1S<ZV;a{WX?
zYU@fF)iri(q2Ua3S)Pko+lM^^R;?RC4G+pVb6TTU)SgyIX7@bu9I3vkW!;w8m3v35
z!TC6Uj^M`U-oA>WL+zXfiIr{kT-)nK9P8@q>)2g&84sJ*_+3b|2xFl$*V#ijqPC{G
zeSlremA7uLylj+rFWD^0D$IlL^{#Au#-yRFG7Gn3{J{jPgXjG|itlc!zE-Q@*%A73
z6aRyFnaxPz(wfq|*JJM58%VEOggN(x3!I(bWpM>(^E~pC!#la18=7yjk6T-^7sNF1
zSw*}yzVQ^DT=6&VtuNO5r#*V$v^?7KcCeFo=DJp{LAm4ZmZ}Lxg^fjju4r4dF(nT7
z=JA_&tN`2aDVLJX1_RFgC9E9yMNg+Bt8V6tl7+mI-~d*c<@O7$?vs|)yIQy%7%0&U
zX4xsa1dI*kwYpVofBCA|o5SrLA#T#|SK+z0#9{8fh6A?GQ~LrBz2yu=_CMbu^4X$o
zx;Ebbh{DVKjCo-AiR8z|?AX2|9<j}p@rS1M?+ZWiWY}}i(%sS^>_m5?WdLlexpS0#
z@7Z$6nb{@IsR`LV?IK3iz0LLV=w9JSk7QBH3J@4?&2ju{-a12%^em^~{p#{?r;mcw
z@pV%d|G03tHU81j!N`}h<}8nMv<qz0e7YkA{}FX7M~h`qfB9~dmMC5KR835ur!-`1
z-Sk@5y+=+)M#t{Q5f9n6{9Vt6`-8fV*bNxHyTx@Fw%h%tIriD_>SbLYdVS`;HFa@f
zFej*~X+WeHPXIju<qtT5Vu46Nbw!OtP-Gx77-TTzBN5n!A~#uXBgB;cr`H&kh_D=O
z^ns>S>;gOiWu!Ek1b0N3gW7&l;n_+u4(1%kCdkp3#_7kg4RZ6NMfW~xA`BLj97hSs
zVen}zJ4TIzq(hVzG)RbJ*=Xz(Md^ZX0>LI*{xwO!OHoJQsHwsecs?53I*J$o{t`{!
zt|CTI3dDl9Lq!bWLky(}&WQxC4pLzzbjMN}(4xc_cGB1-2`@p$Au2Hr6iKA0pr2{-
zUQCS#Pbn!%d=i^b-B5fIA4ZB@baZ7Rv9nIb8gS|qUWA_S`4rouYhg60PR0&MD_j-U
zfek5m0lNPm(T3p^Y)|P!V=8t9Vyq3hQc4f9Q!$6?Q_@Ie8UsE#<dCFNhCbct<x+;8
zuCLnYCjcI%VG+6#N9Q^fFbqMVEZ}iEIhDyG8oOoS#VF-CEoR6V2JkwAEaU=anK+K(
zLvbcfMCUT;xcw!DHF#AJkJc>WQCOnFpMl?LQipDplDNa`(|8`@pH~WRh|VT`ymf&V
zra72P)d4RDXP{rJX}0q|+1fw7{Vku^)?K18BbS^=)n!^_=i;x?oh!8Pe?U**pz5oT
zQ9#ysEG280T%*OFJj_K?uG7LepSW1p(&A#iGIB#Hc;H<?wxR>+1vrhG3fw}RPLB9t
zAr_-8x0UO-0k;^dgIt8C!K<@mV?p&QtOeTr3@v)oF-0VHw_#LFM~0VDz2GXtfb1Jp
zf$xw@lTx@scQI*JWCtx8F>@TSUNLlGaw%zTd<n5j>rusYL9dh)=g_YrY$z!uH`ikg
z=vQEK5I$m90Wo1h_ztEHyxGn)fHmhx!qEREn8HhCO!M-n$?v<Ex*#kgU(=2&pW?OP
zT~4xXua3f<<@j3!H8hdMQZr#nH&Y7)a+26ttHhSjF2@V0ZJ<|yLy^lYXsRU4t^%{6
vsRGZYB7myI8>z+6Scz@mR3)BAJyE{<p=R<sQ;8QqU=^M?M%|bEy{i2UdCx5V

diff --git a/src/vfs/_vfscommon.vfs/modules/shellfilter-0.2.4.tm b/src/vfs/_vfscommon.vfs/modules/shellfilter-0.2.5.tm
similarity index 98%
rename from src/vfs/_vfscommon.vfs/modules/shellfilter-0.2.4.tm
rename to src/vfs/_vfscommon.vfs/modules/shellfilter-0.2.5.tm
index 0bf17fd8..bd5db3d8 100644
--- a/src/vfs/_vfscommon.vfs/modules/shellfilter-0.2.4.tm
+++ b/src/vfs/_vfscommon.vfs/modules/shellfilter-0.2.5.tm
@@ -419,7 +419,10 @@ namespace eval shellfilter::chan {
         }
         method initialize {transform_handle mode} {
             #return [list initialize read drain write flush clear finalize]
-            return [list initialize write flush clear finalize]
+            #'clear' deliberately NOT declared: when a transform declares it, the core
+            #delivers a 'clear' op before EVERY write (tclIORTrans.c ReflectOutput), not
+            #just on seek - see goals/archive/G-145-piped-usage-ansi-remnants.md.
+            return [list initialize write flush finalize]
         }
         method finalize {transform_handle} {
             #Note that an error in the finalize can stop 'chan pop' from running properly.
@@ -430,8 +433,15 @@ namespace eval shellfilter::chan {
             # must be present but we ignore it because we do not
             # post any events
         }
+        #G-145 defect class: 'clear' must NOT discard o_encbuf (held partial multi-byte
+        #character). With clear declared, the core calls it before every write; dropping
+        #the carry leaves the next chunk starting with orphan continuation bytes, making
+        #whole chunks unconvertible - they accumulate in o_encbuf and successive clears
+        #discard them, eating contiguous ranges of multibyte-dense output (the 2026-08-03
+        #unix repl result-echo corruption). Kept as a state-preserving no-op in case
+        #'clear' is ever re-declared; this write-only transform has no read-side state
+        #(the documented scope of 'clear').
         method clear {transform_handle} {
-            set o_encbuf ""
             return
         }
         #method drain {transform_handle} {
@@ -471,16 +481,22 @@ namespace eval shellfilter::chan {
         #    return $clear
         #}
         method flush {transform_handle} {
-            set clear $o_buffered$o_encbuf
-            if {[catch {tcl::encoding::convertfrom $o_enc $clear} stringdata]} {
-                #if we can't convert the buffer contents to a string - does it make sense to emit the raw bytes?
-                # - probably not.
-                #REVIEW?
+            #this class holds only o_encbuf (raw bytes of a trailing incomplete
+            #multi-byte character) - there is no o_buffered ansi-carry here (that
+            #belongs to the ansiwrap/ansistrip style transforms).
+            #An incomplete char is not decodable on its own: hold it for the next write
+            #rather than emitting garbage or discarding (G-145: dropping held stream
+            #state corrupts content split across write chunks). finalize dropping it at
+            #true end of stream is acceptable.
+            if {$o_encbuf eq ""} {
+                return ""
+            }
+            if {[catch {tcl::encoding::convertfrom $o_enc $o_encbuf} stringdata]} {
                 return ""
             }
-            set o_buffered ""
             set o_encbuf ""
-            return $stringdata
+            puts -nonewline $o_localchan $stringdata
+            return [tcl::encoding::convertto $o_enc $stringdata]
         }
         method write {transform_handle bytes} {
             #set logdata [tcl::encoding::convertfrom $o_enc $bytes]
@@ -570,16 +586,19 @@ namespace eval shellfilter::chan {
         #    return $clear
         #}
         method flush {transform_handle} {
-            set clear $o_buffered$o_encbuf
-            if {[catch {tcl::encoding::convertfrom $o_enc $clear} stringdata]} {
-                #if we can't convert the buffer contents to a string - does it make sense to emit the raw bytes?
-                # - probably not.
-                #REVIEW?
+            #only o_encbuf exists in this class (no o_buffered ansi-carry as in ansiwrap).
+            #An incomplete multi-byte char is not decodable on its own: hold it for the
+            #next write rather than discarding (G-145: dropping held stream state
+            #corrupts content split across write chunks).
+            if {$o_encbuf eq ""} {
+                return ""
+            }
+            if {[catch {tcl::encoding::convertfrom $o_enc $o_encbuf} stringdata]} {
                 return ""
             }
-            set o_buffered ""
             set o_encbuf ""
-            return $stringdata
+            ::shellfilter::log::write $o_logsource $stringdata
+            return [tcl::encoding::convertto $o_enc $stringdata]
         }
         method write {ch bytes} {
             #set logdata [tcl::encoding::convertfrom $o_enc $bytes]
@@ -668,16 +687,19 @@ namespace eval shellfilter::chan {
         #    return
         #}
         method flush {transform_handle} {
-            set clear $o_buffered$o_encbuf
-            if {[catch {tcl::encoding::convertfrom $o_enc $clear} stringdata]} {
-                #if we have data in the buffer that we haven't been able to convert to a string
-                #- then we probably have some kind of encoding mismatch. Is it safer to discard it than to emit garbage chars to the log?
-                #REVIEW. - we are writing the raw bytes to the log here because we can't convert them to a string.
-                #This may be useful for debugging issues, but it may also result in garbage data in the log.
-                ::shellfilter::log::write $o_logsource $o_encbuf
+            #only o_encbuf exists in this class (no o_buffered ansi-carry as in ansiwrap).
+            #logonly emits nothing downstream. An incomplete multi-byte char is not
+            #decodable on its own: hold it for the next write rather than discarding
+            #(G-145: dropping held stream state corrupts content split across write
+            #chunks); if it decodes anyway (e.g after an encoding change), log it now.
+            if {$o_encbuf eq ""} {
+                return ""
+            }
+            if {![catch {tcl::encoding::convertfrom $o_enc $o_encbuf} stringdata]} {
+                ::shellfilter::log::write $o_logsource $stringdata
                 set o_encbuf ""
             }
-            return
+            return ""
         }
         method write {transform_handle bytes} {
             #set logdata [encoding convertfrom $o_enc $bytes]
@@ -3838,5 +3860,5 @@ namespace eval shellfilter {
 
 package provide shellfilter [namespace eval shellfilter {
     variable version
-    set version 0.2.4
+    set version 0.2.5
 }]