@zalom/plastic 2.0.0-alpha.6 → 2.0.0-alpha.7

package/hooks/message-display

@@ -24,6 +24,8 @@
 # ALWAYS handed off, whatever it looks like, since it is the one that must
 # not race. Ruby is the one place that actually waits (bounded, injectable
 # for tests); this script only ever decides once, fast, and never sleeps.
+#
+# Claude adapter: Claude Code only; the core is harness-agnostic.
 IFS= read -r -d '' INPUT          # returns 1 at EOF: do NOT set -e
 case $INPUT in *'"message_id"'*) ;; *) exit 0 ;; esac
 rest=${INPUT#*\"message_id\"}; rest=${rest#*\"}; mid=${rest%%\"*}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zalom/plastic",
-  "version": "2.0.0-alpha.6",
+  "version": "2.0.0-alpha.7",
   "description": "Intent-driven idea development system for AI coding agents",
   "type": "module",
   "bin": {

package/scripts/hook-message-display

@@ -12,6 +12,8 @@
 # This script is the one place in the O4 stack allowed to read ENV, the
 # clock, or Dir.tmpdir: MessageDisplay itself takes tmp_root/plastic_home/
 # color/now as constructor arguments and touches none of them directly.
+#
+# Claude adapter: Claude Code only; the core is harness-agnostic.
 
 require "json"
 require "time"

package/scripts/intent-screen

@@ -26,6 +26,10 @@
 # Exit codes:
 #   0 - the screen is on stdout
 #   2 - usage error, or the path is not an intent directory (one line on stderr)
+#
+# Harness-agnostic core: no harness assumption lives here. This script holds
+# the plain/ANSI selection (intent 316a1, D3) and belongs to the same
+# harness-agnostic core as lib/intent_screen.rb and lib/intent_screen_ansi.rb.
 
 require_relative "lib/intent_screen"
 require_relative "lib/intent_screen_ansi"

package/scripts/lib/hook_registry.rb

@@ -54,9 +54,11 @@ module HookRegistry
           { "name" => "capture", "status" => "Capturing prompt into the session ledger..." },
         ] },
       ],
-      # Intent 316a (D6): Claude Code only, no Codex projection — MessageDisplay
-      # is not one of CODEX_LIVE_STATE_EVENTS, so codex_hooks_json (below) never
-      # picks it up; codex_hook_names stays exactly what it was (pinned by
+      # This entry belongs to the Claude adapter half of Plastic's
+      # harness-agnostic-core / Claude-adapter split (intent 316a1, D3
+      # supersedes 316a's D6): MessageDisplay is not one of
+      # CODEX_LIVE_STATE_EVENTS, so codex_hooks_json (below) never picks it
+      # up; codex_hook_names stays exactly what it was (pinned by
       # test/hook_registry_test.rb:82 and :110-111). Fires on every streamed
       # chunk of every assistant message (D11); the launcher (hooks/message-
       # display) decides with shell builtins and forks nothing on the common

package/scripts/lib/intent_screen.rb

@@ -12,6 +12,8 @@
 # text cut mid-sentence. `step_text`, `insight_fields` and `next_fields` are
 # public so the ANSI renderer reuses the exact same trims (D3) rather than
 # re-deriving them and drifting.
+#
+# Harness-agnostic core: no harness assumption lives here.
 module IntentScreen
   BAR_WIDTH = 20
   ON = "█"

package/scripts/lib/intent_screen_ansi.rb

@@ -15,6 +15,13 @@ require_relative "intent_screen"
 # `color:` is a constructor/call argument, never an environment read (D18):
 # the plain path (`color: false`) is one call away and testable without
 # touching NO_COLOR or a TTY. No ENV, no Dir.pwd, no Dir.home.
+#
+# Harness-agnostic core: no harness assumption lives here. `markdown_safe:`
+# (intent 316a1, D3/D5) is the one choice a caller supplies rather than a
+# choice this module makes for itself: a display surface that passes raw
+# ANSI through untouched should not inherit a concession it never needed.
+# See docs/reference/harness-adapters.md for which caller asks for it and
+# why.
 module IntentScreenAnsi
   ESC = "\e"
   RESET = "#{ESC}[0m".freeze
@@ -32,7 +39,7 @@ module IntentScreenAnsi
 
   ELLIPSIS = "…"
 
-  def self.render(intent_dir:, store_root:, color: true, width: DEFAULT_WIDTH)
+  def self.render(intent_dir:, store_root:, color: true, width: DEFAULT_WIDTH, markdown_safe: false)
     base = File.basename(intent_dir)
     id = base.split("--", 2).first
     intent_text = File.read(File.join(intent_dir, "#{base}.md"))
@@ -49,7 +56,7 @@ module IntentScreenAnsi
     fields.merge!(IntentScreen.progress_fields(items))
     fields.merge!(IntentScreen.next_fields(items, status, checklist_present: IntentScreen.items_present?(intent_dir), escape_pipes: false))
     fields.merge!(IntentScreen.insight_fields(intent_text, escape_pipes: false))
-    fields.transform_values! { |v| clean(v) }
+    fields.transform_values! { |v| markdown_safe ? clean(v) : v }
 
     done_n = fields["progress.done"].to_i
     total_n = fields["progress.total"].to_i
@@ -103,7 +110,8 @@ module IntentScreenAnsi
         badge = status_cell(item[:done], color)
         prefix_plain = "  #{num}  [ #{item[:done] ? 'done' : 'open'} ]  "
         text_budget = [width - prefix_plain.length, 0].max
-        text = fit_plain(clean(IntentScreen.step_text(item[:text])), text_budget)
+        step = IntentScreen.step_text(item[:text])
+        text = fit_plain(markdown_safe ? clean(step) : step, text_budget)
         out << "  #{num}  [#{badge}]  #{text}\n"
       end
     end
@@ -111,12 +119,13 @@ module IntentScreenAnsi
     out
   end
 
-  # --- markdown-noise stripping (intent 316a, S1 answer 5 / matrix 19b) ------
+  # --- markdown-noise stripping, adapter-optional (intent 316a1, D3/D5) ------
   #
-  # `displayContent` is still Markdown-processed by Claude Code even inside a
-  # raw ANSI block (a live capture showed backticks silently stripped from
-  # step text). Strip backticks and neutralise `*`/`_` runs from every value
-  # before it reaches the block, so nothing is left for that pass to act on.
+  # Not every display surface passes text through a Markdown renderer, so
+  # stripping is not this module's call to make (see `markdown_safe:` on
+  # `render` above; the justification for WHY a caller would ever ask for
+  # this lives with that caller, in scripts/lib/message_display.rb). When
+  # asked, strips backticks and neutralises `*`/`_` runs from a value.
   # Single underscores are left alone: they are common inside ordinary words
   # (`intent_screen.rb`) and GFM does not treat an intraword underscore as
   # emphasis; only a run of 2+ (the bold marker `__`) is markdown-active.
@@ -126,9 +135,11 @@ module IntentScreenAnsi
 
   # --- width cap (D15, matrix 18) --------------------------------------------
 
-  # Truncates `text` (already markdown-clean) to `max` visible columns with a
-  # trailing ellipsis when cut, then yields the truncated plain text to the
-  # block for coloring. Coloring never adds visible width.
+  # Truncates `text` to `max` visible columns with a trailing ellipsis when
+  # cut, then yields the truncated plain text to the block for coloring.
+  # Coloring never adds visible width. The cap itself is harness-neutral: a
+  # fixed width, not a re-flow, is what lets a column layout survive whatever
+  # display eventually shows it — no display's own wrapping is assumed here.
   def self.fit(text, max)
     plain = fit_plain(text, max)
     block_given? ? yield(plain) : plain

package/scripts/lib/message_display.rb

@@ -15,6 +15,11 @@ require_relative "store_provisioning"
 # the real Time.now/Kernel#sleep — the thin CLI (scripts/hook-message-display)
 # is the one place allowed to read any of those.
 #
+# Claude adapter: Claude Code only; the core is harness-agnostic. (intent
+# 316a1, D3 supersedes 316a's D6.) This is the sole caller that asks
+# IntentScreenAnsi.render for `markdown_safe: true` (scripts/lib/
+# intent_screen_ansi.rb) — see `finalize` below for why.
+#
 # A live run under a real pty (round 3) found that Claude Code fires the
 # per-chunk hook processes CONCURRENTLY, not strictly in order. Chunk 0 is
 # the one that recognizes the screen and creates the buffer (D13), and it can
@@ -230,7 +235,13 @@ class MessageDisplay
   def finalize(buffered, decision)
     intent_dir = decision[:intent_dir]
     store_root = decision[:store_root]
-    ansi = IntentScreenAnsi.render(intent_dir: intent_dir, store_root: store_root, color: true)
+    # markdown_safe: true (intent 316a1, D5) - Claude Code still Markdown-
+    # processes displayContent even inside a raw ANSI block (316a's live
+    # capture showed backticks silently stripped from step text), so the
+    # Claude adapter asks the harness-agnostic core to strip markdown noise
+    # before it ever reaches the block. A harness whose display surface
+    # passes raw ANSI through untouched would ask for false instead.
+    ansi = IntentScreenAnsi.render(intent_dir: intent_dir, store_root: store_root, color: true, markdown_safe: true)
     plain = IntentScreen.render(intent_dir: intent_dir, store_root: store_root, template: File.read(template_path))
     splice(buffered, plain, ansi)
   end