fix(console): turn emoji substitution off globally

print_list and print_files print directly instead of going through the log
methods, so they still replaced shortcodes with glyphs. Passing emoji=False
to print does not reach file names inside the tree, so the console itself is
now built with emoji off.

Author: bananaplain

plain2code_console.py

@@ -27,7 +27,9 @@ class Plain2CodeConsole(Console):
     DEBUG_STYLE = Style(color="purple")
 
     def __init__(self):
-        super().__init__()
+        # Emoji shortcodes must never be substituted. Set on the console because the
+        # emoji flag of Console.print does not reach text inside renderables like Tree.
+        super().__init__(emoji=False)
         try:
             import tiktoken
 
@@ -66,13 +68,12 @@ def _log_and_print(self, level, base_style, args, color, kwargs):
         """
         logger.log(level, " ".join(map(str, args)), extra={"log_color": color})
         style = base_style + Style(color=color) if color else base_style
-        # Log messages must render exactly as logged: don't interpret square brackets
-        # in interpolated content (error texts, file names) as Rich markup, and don't
-        # let the repr highlighter restyle brackets and numbers inside them.
-        # Don't let Rich substitute emoji shortcodes with glyphs. Literal emojis unaffected.
+        # Messages must render exactly as logged and not pick its own styling:
+        # -- no Rich markup (via square brackets) in interpolated content (error texts, file names)
+        # -- no repr highlighter restyling brackets and numbers inside them.
+        # -- emoji are off console-wide via __init__
         kwargs.setdefault("markup", False)
         kwargs.setdefault("highlight", False)
-        kwargs.setdefault("emoji", False)
         super().print(*args, **kwargs, style=style)
 
     def print_list(self, items, style=None):

tests/test_console_log_styles.py

@@ -116,6 +116,25 @@ def test_concept_name_matching_emoji_prints_verbatim(self):
         assert "📦" not in capture.get()
 
 
+class TestPrintHelpersDoNotSubstituteEmoji:
+    """
+    Same rule as TestTerminalTextRendersVerbatim, for the two helpers that bypass _log_and_print.
+    Only emoji is covered here, markup and highlighting still apply to them.
+    """
+
+    def test_print_list_does_not_substitute_emoji(self):
+        console = Plain2CodeConsole()
+        with console.capture() as capture:
+            console.print_list(["The :Package: is a concept"])
+        assert ":Package:" in capture.get()
+
+    def test_print_files_does_not_substitute_emoji_in_tree_labels(self):
+        console = Plain2CodeConsole()
+        with console.capture() as capture:
+            console.print_files("Files added:", "build", {":Package:.py": "x"})
+        assert ":Package:" in capture.get()
+
+
 class TestLoggingHandlerForwardsColor:
     """LoggingHandler must forward the log_color record attribute into LogMessageEmitted."""