failed CONTEXT UNSUBMITTED agent_error · 19 tool calls · 153 s · Textualize/rich
perf: reduce Console and RichHandler import time by deferring unused imports ## E2E Results | Import | master | This PR | Speedup | |---|---|---|---| | `from rich.console import Console` | 78.1ms | 52.2ms | **1.50x faster** | | `from rich.logging import RichHandler` | 99.4ms | 56.9ms | **1.75x faster** | | `import rich` | 18.2ms | 18.3ms | (already lean) | ## Summary Defer module-level imports that are only needed in specific code paths, move annotation-only imports to `TYPE_CHECKING`, and remove dead code: | File | Change | Savings | |---|---|---| | `logging.py` | Defer `Traceback` to `emit()` (only when `rich_tracebacks=True`) | ~20ms | | `logging.py` | `from __future__ import annotations` + `TYPE_CHECKING` for `Console`, `ConsoleRenderable`, `Highlighter`, `FormatTimeCallable` | ~6ms | | `logging.py` | Replace `pathlib.Path` → `os.path.basename` (also a minor runtime win) | ~4-5ms | | `console.py` | Eliminate `import inspect`; replace `isclass` → `isinstance(x, type)`, `currentframe` → `sys._getframe` | ~10ms | | `console.py` | Defer `pretty` to `Console.print()` | ~3-5ms | | `console.py` | Defer `scope` to `Console.log()` (only when `log_locals=True`) | ~3-5ms | | `console.py` | Defer `getpass` to `Console.input()` (only when `password=True`) | ~2ms | | `console.py` | Defer `html.escape` and `zlib` to export methods | ~2.3ms | | `console.py` | Remove dead `_svg_hash` function (unused since 113997ac, fixes latent NameError) | cleanup | | `segment.py` | Remove dead `logging` import (`getLogger` assigned but never used) | ~2-3ms | | `theme.py` | Defer `configparser` to `Theme.from_file()` | ~1.5ms | | `syntax.py` | Move `Console`, `ConsoleOptions`, `JustifyMethod`, `RenderResult` to `TYPE_CHECKING` | eliminates console.py from syntax import chain | | `protocol.py` | Replace `from inspect import isclass` → `isinstance(x, type)` | prepares for dataclasses removal | | `repr.py` | Defer `import inspect` to `auto_rich_repr()` | prepares for dataclasses removal | All deferred imports are still available at runtime — they're loaded when the code path that needs them is actually hit. The `TYPE_CHECKING` imports are annotation-only and never needed at runtime (files use `from __future__ import annotations`). ## Benchmarks Environment: Standard_D2s_v5 (non-burstable), CPython 3.13.13, hyperfine --min-runs 30 ### Per-module breakdown (this PR) | Module | Time | |---|---| | `import rich` | 18.2ms | | `Console` | 52.5ms | | `RichHandler` | 56.4ms | | `Syntax` | 64.7ms | | `Traceback` | 92.5ms | | `Markdown` | 103.0ms | ## Motivation pip vendors Rich, and `Console`/`RichHandler` are imported on every `pip` invocation. These savings directly improve startup time for pip and every other tool that imports Rich. ## Testing - `pytest tests/` — 952 passed, 25 skipped - `mypy -p rich --no-incremental` — no issues - `black --check` — passes ## Disclosure This PR was developed with AI assistance (codeflash-agent). Benchmarked in a controlled environment (dedicated non-burstable Azure VM). All tests verified across multiple Python implementations.
| # | Tool | Arguments | Result |
|---|---|---|---|
| 1 | run_command | {
"command": "ls -R rich | grep -E \"logging.py|console.py|segment.py|theme.py|syntax.py|protocol.py|repr.py\""
} | {"status": "ok", "stdout": "_win32_console.py\nconsole.py\nlogging.py\nprotocol.py\nrepr.py\nsegment.py\nsyntax.py\nterminal_theme.py\ntheme.py\n", "stderr": ""full{"status": "ok", "stdout": "_win32_console.py\nconsole.py\nlogging.py\nprotocol.py\nrepr.py\nsegment.py\nsyntax.py\nterminal_theme.py\ntheme.py\n", "stderr": "", "exit_code": 0} |
| 2 | read_file | {
"filepath": "rich/logging.py"
} | {"status": "ok", "filepath": "rich/logging.py", "content": "import logging\nfrom datetime import datetime\nfrom logging import Handler, LogRecord\nfrom pathlib full{"status": "ok", "filepath": "rich/logging.py", "content": "import logging\nfrom datetime import datetime\nfrom logging import Handler, LogRecord\nfrom pathlib import Path\nfrom types import ModuleType\nfrom typing import ClassVar, Iterable, List, Optional, Type, Union\n\nfrom rich._null_file import NullFile\n\nfrom . import get_console\nfrom ._log_render import FormatTimeCallable, LogRender\nfrom .console import Console, ConsoleRenderable\nfrom .highlighter import Highlighter, ReprHighlighter\nfrom .text import Text\nfrom .traceback import Traceback\n\n\nclass RichHandler(Handler):\n \"\"\"A logging handler that renders output with Rich. The time / level / message and file are displayed in columns.\n The level is color coded, and the message is syntax highlighted.\n\n Note:\n Be careful when enabling console markup in log messages if you have configured logging for libraries not\n under your control. If a dependency writes messages containing square brackets, it may not produce the intended output.\n\n Args:\n level (Union[int, str], optional): Log level. Defaults to logging.NOTSET.\n console (:class:`~rich.console.Console`, optional): Optional console instance to write logs.\n Default will use a global console instance writing to stdout.\n show_time (bool, optional): Show a column for the time. Defaults to True.\n omit_repeated_times (bool, optional): Omit repetition of the same time. Defaults to True.\n show_level (bool, optional): Show a column for the level. Defaults to True.\n show_path (bool, optional): Show the path to the original log call. Defaults to True.\n enable_link_path (bool, optional): Enable terminal link of path column to file. Defaults to True.\n highlighter (Highlighter, optional): Highlighter to style log messages, or None to use ReprHighlighter. Defaults to None.\n markup (bool, optional): Enable console markup in log messages. Defaults to Fals |
| 3 | read_file | {
"filepath": "rich/logging.py",
"start_line\\\"": 151,
"end_line\\\"": 297
} | {"status": "ok", "filepath": "rich/logging.py", "content": "import logging\nfrom datetime import datetime\nfrom logging import Handler, LogRecord\nfrom pathlib full{"status": "ok", "filepath": "rich/logging.py", "content": "import logging\nfrom datetime import datetime\nfrom logging import Handler, LogRecord\nfrom pathlib import Path\nfrom types import ModuleType\nfrom typing import ClassVar, Iterable, List, Optional, Type, Union\n\nfrom rich._null_file import NullFile\n\nfrom . import get_console\nfrom ._log_render import FormatTimeCallable, LogRender\nfrom .console import Console, ConsoleRenderable\nfrom .highlighter import Highlighter, ReprHighlighter\nfrom .text import Text\nfrom .traceback import Traceback\n\n\nclass RichHandler(Handler):\n \"\"\"A logging handler that renders output with Rich. The time / level / message and file are displayed in columns.\n The level is color coded, and the message is syntax highlighted.\n\n Note:\n Be careful when enabling console markup in log messages if you have configured logging for libraries not\n under your control. If a dependency writes messages containing square brackets, it may not produce the intended output.\n\n Args:\n level (Union[int, str], optional): Log level. Defaults to logging.NOTSET.\n console (:class:`~rich.console.Console`, optional): Optional console instance to write logs.\n Default will use a global console instance writing to stdout.\n show_time (bool, optional): Show a column for the time. Defaults to True.\n omit_repeated_times (bool, optional): Omit repetition of the same time. Defaults to True.\n show_level (bool, optional): Show a column for the level. Defaults to True.\n show_path (bool, optional): Show the path to the original log call. Defaults to True.\n enable_link_path (bool, optional): Enable terminal link of path column to file. Defaults to True.\n highlighter (Highlighter, optional): Highlighter to style log messages, or None to use ReprHighlighter. Defaults to None.\n markup (bool, optional): Enable console markup in log messages. Defaults to Fals |
| 4 | read_file | {
"filepath": "rich/logging.py",
"start_line": 151
} | {"status": "ok", "filepath": "rich/logging.py", "content": " theme=self.tracebacks_theme,\n word_wrap=self.tracebacks_word_wrap,\n show_locals=self.tracebacks_sfull{"status": "ok", "filepath": "rich/logging.py", "content": " theme=self.tracebacks_theme,\n word_wrap=self.tracebacks_word_wrap,\n show_locals=self.tracebacks_show_locals,\n locals_max_length=self.locals_max_length,\n locals_max_string=self.locals_max_string,\n suppress=self.tracebacks_suppress,\n max_frames=self.tracebacks_max_frames,\n )\n message = record.getMessage()\n if self.formatter:\n record.message = record.getMessage()\n formatter = self.formatter\n if hasattr(formatter, \"usesTime\") and formatter.usesTime():\n record.asctime = formatter.formatTime(record, formatter.datefmt)\n message = formatter.formatMessage(record)\n\n message_renderable = self.render_message(record, message)\n log_renderable = self.render(\n record=record, traceback=traceback, message_renderable=message_renderable\n )\n if isinstance(self.console.file, NullFile):\n # Handles pythonw, where stdout/stderr are null, and we return NullFile\n # instance from Console.file. In this case, we still want to make a log record\n # even though we won't be writing anything to a file.\n self.handleError(record)\n else:\n try:\n self.console.print(log_renderable)\n except Exception:\n self.handleError(record)\n\n def render_message(self, record: LogRecord, message: str) -> \"ConsoleRenderable\":\n \"\"\"Render message text in to Text.\n\n Args:\n record (LogRecord): logging Record.\n message (str): String containing log message.\n\n Returns:\n ConsoleRenderable: Renderable to display log message.\n \"\"\"\n use_markup = getattr(record, \"markup\", self.markup)\n message_ |
| 5 | read_file | {
"filepath": "rich/console.py"
} | {"status": "ok", "filepath": "rich/console.py", "content": "import inspect\nimport os\nimport sys\nimport threading\nimport zlib\nfrom abc import ABC, abstractmfull{"status": "ok", "filepath": "rich/console.py", "content": "import inspect\nimport os\nimport sys\nimport threading\nimport zlib\nfrom abc import ABC, abstractmethod\nfrom dataclasses import dataclass, field\nfrom datetime import datetime\nfrom functools import wraps\nfrom getpass import getpass\nfrom html import escape\nfrom inspect import isclass\nfrom itertools import islice\nfrom math import ceil\nfrom time import monotonic\nfrom types import FrameType, ModuleType, TracebackType\nfrom typing import (\n IO,\n TYPE_CHECKING,\n Any,\n Callable,\n Dict,\n Iterable,\n List,\n Literal,\n Mapping,\n NamedTuple,\n Optional,\n Protocol,\n TextIO,\n Tuple,\n Type,\n Union,\n cast,\n runtime_checkable,\n)\n\nfrom rich._null_file import NULL_FILE\n\nfrom . import errors, themes\nfrom ._emoji_replace import _emoji_replace\nfrom ._export_format import CONSOLE_HTML_FORMAT, CONSOLE_SVG_FORMAT\nfrom ._fileno import get_fileno\nfrom ._log_render import FormatTimeCallable, LogRender\nfrom .align import Align, AlignMethod\nfrom .color import ColorSystem, blend_rgb\nfrom .control import Control\nfrom .emoji import EmojiVariant\nfrom .highlighter import NullHighlighter, ReprHighlighter\nfrom .markup import render as render_markup\nfrom .measure import Measurement, measure_renderables\nfrom .pager import Pager, SystemPager\nfrom .pretty import Pretty, is_expandable\nfrom .protocol import rich_cast\nfrom .region import Region\nfrom .scope import render_scope\nfrom .screen import Screen\nfrom .segment import Segment\nfrom .style import Style, StyleType\nfrom .styled import Styled\nfrom .terminal_theme import DEFAULT_TERMINAL_THEME, SVG_EXPORT_THEME, TerminalTheme\nfrom .text import Text, TextType\nfrom .theme import Theme, ThemeStack\n\nif TYPE_CHECKING:\n from ._windows import WindowsConsoleFeatures\n from .live import Live\n from .status import Status\n\nJUPYTER_DEFAULT_COLUMNS = 115\nJUPYTER_DEFAULT_LINES = 100\nWINDOWS = |
| 6 | read_file | {
"filepath": "rich/console.py",
"start_line": 151
} | {"status": "ok", "filepath": "rich/console.py", "content": " return not self.encoding.startswith(\"utf\")\n\n def copy(self) -> \"ConsoleOptions\":\n \"\"\"Retufull{"status": "ok", "filepath": "rich/console.py", "content": " return not self.encoding.startswith(\"utf\")\n\n def copy(self) -> \"ConsoleOptions\":\n \"\"\"Return a copy of the options.\n\n Returns:\n ConsoleOptions: a copy of self.\n \"\"\"\n options: ConsoleOptions = ConsoleOptions.__new__(ConsoleOptions)\n options.__dict__ = self.__dict__.copy()\n return options\n\n def update(\n self,\n *,\n width: Union[int, NoChange] = NO_CHANGE,\n min_width: Union[int, NoChange] = NO_CHANGE,\n max_width: Union[int, NoChange] = NO_CHANGE,\n justify: Union[Optional[JustifyMethod], NoChange] = NO_CHANGE,\n overflow: Union[Optional[OverflowMethod], NoChange] = NO_CHANGE,\n no_wrap: Union[Optional[bool], NoChange] = NO_CHANGE,\n highlight: Union[Optional[bool], NoChange] = NO_CHANGE,\n markup: Union[Optional[bool], NoChange] = NO_CHANGE,\n height: Union[Optional[int], NoChange] = NO_CHANGE,\n ) -> \"ConsoleOptions\":\n \"\"\"Update values, return a copy.\"\"\"\n options = self.copy()\n if not isinstance(width, NoChange):\n options.min_width = options.max_width = max(0, width)\n if not isinstance(min_width, NoChange):\n options.min_width = min_width\n if not isinstance(max_width, NoChange):\n options.max_width = max_width\n if not isinstance(justify, NoChange):\n options.justify = justify\n if not isinstance(overflow, NoChange):\n options.overflow = overflow\n if not isinstance(no_wrap, NoChange):\n options.no_wrap = no_wrap\n if not isinstance(highlight, NoChange):\n options.highlight = highlight\n if not isinstance(markup, NoChange):\n options.markup = markup\n if not isinstance(height, NoChange):\n if height is not None:\n options.max_height = heig |
| 7 | read_file | {
"filepath": "rich/segment.py"
} | {"status": "ok", "filepath": "rich/segment.py", "content": "from enum import IntEnum\nfrom functools import lru_cache\nfrom itertools import filterfalse\nfrom lfull{"status": "ok", "filepath": "rich/segment.py", "content": "from enum import IntEnum\nfrom functools import lru_cache\nfrom itertools import filterfalse\nfrom logging import getLogger\nfrom operator import attrgetter\nfrom typing import (\n TYPE_CHECKING,\n Dict,\n Iterable,\n List,\n NamedTuple,\n Optional,\n Sequence,\n Tuple,\n Type,\n Union,\n)\n\nfrom .cells import (\n _is_single_cell_widths,\n cached_cell_len,\n cell_len,\n get_character_cell_size,\n set_cell_size,\n)\nfrom .repr import Result, rich_repr\nfrom .style import Style\n\nif TYPE_CHECKING:\n from .console import Console, ConsoleOptions, RenderResult\n\nlog = getLogger(\"rich\")\n\n\nclass ControlType(IntEnum):\n \"\"\"Non-printable control codes which typically translate to ANSI codes.\"\"\"\n\n BELL = 1\n CARRIAGE_RETURN = 2\n HOME = 3\n CLEAR = 4\n SHOW_CURSOR = 5\n HIDE_CURSOR = 6\n ENABLE_ALT_SCREEN = 7\n DISABLE_ALT_SCREEN = 8\n CURSOR_UP = 9\n CURSOR_DOWN = 10\n CURSOR_FORWARD = 11\n CURSOR_BACKWARD = 12\n CURSOR_MOVE_TO_COLUMN = 13\n CURSOR_MOVE_TO = 14\n ERASE_IN_LINE = 15\n SET_WINDOW_TITLE = 16\n\n\nControlCode = Union[\n Tuple[ControlType],\n Tuple[ControlType, Union[int, str]],\n Tuple[ControlType, int, int],\n]\n\n\n@rich_repr()\nclass Segment(NamedTuple):\n \"\"\"A piece of text with associated style. Segments are produced by the Console render process and\n are ultimately converted in to strings to be written to the terminal.\n\n Args:\n text (str): A piece of text.\n style (:class:`~rich.style.Style`, optional): An optional style to apply to the text.\n control (Tuple[ControlCode], optional): Optional sequence of control codes.\n\n Attributes:\n cell_length (int): The cell length of this Segment.\n \"\"\"\n\n text: str\n style: Optional[Style] = None\n control: Optional[Sequence[ControlCode]] = None\n\n @property\n def c |
| 8 | read_file | {
"filepath": "rich/theme.py"
} | {"status": "ok", "filepath": "rich/theme.py", "content": "import configparser\nfrom typing import IO, Dict, List, Mapping, Optional\n\nfrom .default_styles impofull{"status": "ok", "filepath": "rich/theme.py", "content": "import configparser\nfrom typing import IO, Dict, List, Mapping, Optional\n\nfrom .default_styles import DEFAULT_STYLES\nfrom .style import Style, StyleType\n\n\nclass Theme:\n \"\"\"A container for style information, used by :class:`~rich.console.Console`.\n\n Args:\n styles (Dict[str, Style], optional): A mapping of style names on to styles. Defaults to None for a theme with no styles.\n inherit (bool, optional): Inherit default styles. Defaults to True.\n \"\"\"\n\n styles: Dict[str, Style]\n\n def __init__(\n self, styles: Optional[Mapping[str, StyleType]] = None, inherit: bool = True\n ):\n self.styles = DEFAULT_STYLES.copy() if inherit else {}\n if styles is not None:\n self.styles.update(\n {\n name: style if isinstance(style, Style) else Style.parse(style)\n for name, style in styles.items()\n }\n )\n\n @property\n def config(self) -> str:\n \"\"\"Get contents of a config file for this theme.\"\"\"\n config = \"[styles]\\n\" + \"\\n\".join(\n f\"{name} = {style}\" for name, style in sorted(self.styles.items())\n )\n return config\n\n @classmethod\n def from_file(\n cls, config_file: IO[str], source: Optional[str] = None, inherit: bool = True\n ) -> \"Theme\":\n \"\"\"Load a theme from a text mode file.\n\n Args:\n config_file (IO[str]): An open conf file.\n source (str, optional): The filename of the open file. Defaults to None.\n inherit (bool, optional): Inherit default styles. Defaults to True.\n\n Returns:\n Theme: A New theme instance.\n \"\"\"\n config = configparser.ConfigParser()\n config.read_file(config_file, source=source)\n styles = {name: Style.parse(value) for name, value in config.items(\"styles |
| 9 | read_file | {
"filepath": "rich/syntax.py"
} | {"status": "ok", "filepath": "rich/syntax.py", "content": "from __future__ import annotations\n\nimport os.path\nimport re\nimport sys\nimport textwrap\nfrom abfull{"status": "ok", "filepath": "rich/syntax.py", "content": "from __future__ import annotations\n\nimport os.path\nimport re\nimport sys\nimport textwrap\nfrom abc import ABC, abstractmethod\nfrom pathlib import Path\nfrom typing import (\n Any,\n Dict,\n Iterable,\n List,\n NamedTuple,\n Optional,\n Sequence,\n Set,\n Tuple,\n Type,\n Union,\n)\n\nfrom pygments.lexer import Lexer\nfrom pygments.lexers import get_lexer_by_name, guess_lexer_for_filename\nfrom pygments.style import Style as PygmentsStyle\nfrom pygments.styles import get_style_by_name\nfrom pygments.token import (\n Comment,\n Error,\n Generic,\n Keyword,\n Name,\n Number,\n Operator,\n String,\n Token,\n Whitespace,\n)\nfrom pygments.util import ClassNotFound\n\nfrom rich.containers import Lines\nfrom rich.padding import Padding, PaddingDimensions\n\nfrom ._loop import loop_first\nfrom .cells import cell_len\nfrom .color import Color, blend_rgb\nfrom .console import Console, ConsoleOptions, JustifyMethod, RenderResult\nfrom .jupyter import JupyterMixin\nfrom .measure import Measurement\nfrom .segment import Segment, Segments\nfrom .style import Style, StyleType\nfrom .text import Text\n\nTokenType = Tuple[str, ...]\n\nWINDOWS = sys.platform == \"win32\"\nDEFAULT_THEME = \"monokai\"\n\n# The following styles are based on https://github.com/pygments/pygments/blob/master/pygments/formatters/terminal.py\n# A few modifications were made\n\nANSI_LIGHT: Dict[TokenType, Style] = {\n Token: Style(),\n Whitespace: Style(color=\"white\"),\n Comment: Style(dim=True),\n Comment.Preproc: Style(color=\"cyan\"),\n Keyword: Style(color=\"blue\"),\n Keyword.Type: Style(color=\"cyan\"),\n Operator.Word: Style(color=\"magenta\"),\n Name.Builtin: Style(color=\"cyan\"),\n Name.Function: Style(color=\"green\"),\n Name.Namespace: Style(color=\"cyan\", underline=True),\n Name.Class: Style(color=\"green\", underline=True),\n Name.E |
| 10 | read_file | {
"filepath": "rich/protocol.py"
} | {"status": "ok", "filepath": "rich/protocol.py", "content": "from typing import Any, cast, Set, TYPE_CHECKING\nfrom inspect import isclass\n\nif TYPE_CHECKING:\full{"status": "ok", "filepath": "rich/protocol.py", "content": "from typing import Any, cast, Set, TYPE_CHECKING\nfrom inspect import isclass\n\nif TYPE_CHECKING:\n from rich.console import RenderableType\n\n_GIBBERISH = \"\"\"aihwerij235234ljsdnp34ksodfipwoe234234jlskjdf\"\"\"\n\n\ndef is_renderable(check_object: Any) -> bool:\n \"\"\"Check if an object may be rendered by Rich.\"\"\"\n return (\n isinstance(check_object, str)\n or hasattr(check_object, \"__rich__\")\n or hasattr(check_object, \"__rich_console__\")\n )\n\n\ndef rich_cast(renderable: object) -> \"RenderableType\":\n \"\"\"Cast an object to a renderable by calling __rich__ if present.\n\n Args:\n renderable (object): A potentially renderable object\n\n Returns:\n object: The result of recursively calling __rich__.\n \"\"\"\n from rich.console import RenderableType\n\n rich_visited_set: Set[type] = set() # Prevent potential infinite loop\n while hasattr(renderable, \"__rich__\") and not isclass(renderable):\n # Detect object which claim to have all the attributes\n if hasattr(renderable, _GIBBERISH):\n return repr(renderable)\n cast_method = getattr(renderable, \"__rich__\")\n renderable = cast_method()\n renderable_type = type(renderable)\n if renderable_type in rich_visited_set:\n break\n rich_visited_set.add(renderable_type)\n\n return cast(RenderableType, renderable)", "start_line": 1, "end_line": 42, "total_lines": 42, "is_truncated": false} |
| 11 | read_file | {
"filepath": "rich/repr.py"
} | {"status": "ok", "filepath": "rich/repr.py", "content": "import inspect\nfrom functools import partial\nfrom typing import (\n Any,\n Callable,\n Iterable,\n Lifull{"status": "ok", "filepath": "rich/repr.py", "content": "import inspect\nfrom functools import partial\nfrom typing import (\n Any,\n Callable,\n Iterable,\n List,\n Optional,\n Tuple,\n Type,\n TypeVar,\n Union,\n overload,\n)\n\nT = TypeVar(\"T\")\n\n\nResult = Iterable[Union[Any, Tuple[Any], Tuple[str, Any], Tuple[str, Any, Any]]]\nRichReprResult = Result\n\n\nclass ReprError(Exception):\n \"\"\"An error occurred when attempting to build a repr.\"\"\"\n\n\n@overload\ndef auto(cls: Optional[Type[T]]) -> Type[T]:\n ...\n\n\n@overload\ndef auto(*, angular: bool = False) -> Callable[[Type[T]], Type[T]]:\n ...\n\n\ndef auto(\n cls: Optional[Type[T]] = None, *, angular: Optional[bool] = None\n) -> Union[Type[T], Callable[[Type[T]], Type[T]]]:\n \"\"\"Class decorator to create __repr__ from __rich_repr__\"\"\"\n\n def do_replace(cls: Type[T], angular: Optional[bool] = None) -> Type[T]:\n def auto_repr(self: T) -> str:\n \"\"\"Create repr string from __rich_repr__\"\"\"\n repr_str: List[str] = []\n append = repr_str.append\n\n angular: bool = getattr(self.__rich_repr__, \"angular\", False) # type: ignore[attr-defined]\n for arg in self.__rich_repr__(): # type: ignore[attr-defined]\n if isinstance(arg, tuple):\n if len(arg) == 1:\n append(repr(arg[0]))\n else:\n key, value, *default = arg\n if key is None:\n append(repr(value))\n else:\n if default and default[0] == value:\n continue\n append(f\"{key}={value!r}\")\n else:\n append(repr(arg))\n if angular:\n return f\"<{self.__class__.__name__} {' '.join(repr_str)}>\"\n else:\n |
| 12 | edit_file | {
"filepath": "rich/logging.py",
"new_string": "from __future__ import annotations\n\nimport logging\nfrom datetime import datetime\nfrom logging import Handler, LogRecord\nimport os\nfrom types import ModuleType\nfrom typing import TYPE_CHECKING, ClassVar, Iterable, List, Optional, Type, Union\n\nif TYPE_CHECKING:\n from .console import Console, ConsoleRenderable\n from .highlighter import Highlighter\n\nfrom rich._null_file import NullFile\n\nfrom . import get_console\nfrom ._log_render import FormatTimeCallable, LogRender\nfrom .highlighter import ReprHighlighter\nfrom .text import Text\n",
"old_string": "import logging\nfrom datetime import datetime\nfrom logging import Handler, LogRecord\nfrom pathlib import Path\nfrom types import ModuleType\nfrom typing import ClassVar, Iterable, List, Optional, Type, Type, Union\n\nfrom rich._null_file import NullFile\n\nfrom . import get_console\nfrom ._log_render import FormatTimeCallable, LogRender\nfrom .console import Console, ConsoleRenderable\nfrom .highlighter import Highlighter, ReprHighlighter\nfrom .text import Text\nfrom .traceback import Traceback\n"
} | {"status": "error", "error_type": "FileEditError", "error_message": "Failed to replace: old_string not found. Ensure you're not escaping content incorrectly andfull{"status": "error", "error_type": "FileEditError", "error_message": "Failed to replace: old_string not found. Ensure you're not escaping content incorrectly and check whitespace, indentation, and context."} |
| 13 | read_file | {
"filepath": "rich/logging.py",
"start_line": 1,
"end_line": 20
} | {"status": "ok", "filepath": "rich/logging.py", "content": "import logging\nfrom datetime import datetime\nfrom logging import Handler, LogRecord\nfrom pathlib full{"status": "ok", "filepath": "rich/logging.py", "content": "import logging\nfrom datetime import datetime\nfrom logging import Handler, LogRecord\nfrom pathlib import Path\nfrom types import ModuleType\nfrom typing import ClassVar, Iterable, List, Optional, Type, Union\n\nfrom rich._null_file import NullFile\n\nfrom . import get_console\nfrom ._log_render import FormatTimeCallable, LogRender\nfrom .console import Console, ConsoleRenderable\nfrom .highlighter import Highlighter, ReprHighlighter\nfrom .text import Text\nfrom .traceback import Traceback\n\n\nclass RichHandler(Handler):\n \"\"\"A logging handler that renders output with Rich. The time / level / message and file are displayed in columns.\n The level is color coded, and the message is syntax highlighted.", "start_line": 1, "end_line": 20, "total_lines": 297, "is_truncated": false} |
| 14 | edit_file | {
"filepath": "rich/logging.py",
"new_string": "from __future__ import annotations\n\nimport logging\nfrom datetime import datetime\nfrom logging import Handler, LogRecord\nimport os\nfrom types import ModuleType\nfrom typing import TYPE_CHECKING, ClassVar, Iterable, List, Optional, Type, Union\n\nif TYPE_CHECKING:\n from .console import Console, ConsoleRenderable\n from .highlighter import Highlighter\n\nfrom rich._null_file import NullFile\n\nfrom . import get_console\nfrom ._log_render import FormatTimeCallable, LogRender\nfrom .highlighter import ReprHighlighter\nfrom .text import Text\n",
"old_string": "import logging\nfrom datetime import datetime\nfrom logging import Handler, LogRecord\nfrom pathlib import Path\nfrom types import ModuleType\nfrom typing import ClassVar, Iterable, List, Optional, Type, Union\n\nfrom rich._null_file import NullFile\n\nfrom . import get_console\nfrom ._log_render import FormatTimeCallable, LogRender\nfrom .console import Console, ConsoleRenderable\nfrom .highlighter import Highlighter, ReprHighlighter\nfrom .text import Text\nfrom .traceback import Traceback\n"
} | {"status": "ok", "filepath": "rich/logging.py", "occurrences": 1, "strategy": "exact", "diff": "--- a/rich/logging.py\n+++ b/rich/logging.py\n@@ -1,18 +1,22 @@\full{"status": "ok", "filepath": "rich/logging.py", "occurrences": 1, "strategy": "exact", "diff": "--- a/rich/logging.py\n+++ b/rich/logging.py\n@@ -1,18 +1,22 @@\n+from __future__ import annotations\n+\n import logging\n from datetime import datetime\n from logging import Handler, LogRecord\n-from pathlib import Path\n+import os\n from types import ModuleType\n-from typing import ClassVar, Iterable, List, Optional, Type, Union\n+from typing import TYPE_CHECKING, ClassVar, Iterable, List, Optional, Type, Union\n+\n+if TYPE_CHECKING:\n+ from .console import Console, ConsoleRenderable\n+ from .highlighter import Highlighter\n \n from rich._null_file import NullFile\n \n from . import get_console\n from ._log_render import FormatTimeCallable, LogRender\n-from .console import Console, ConsoleRenderable\n-from .highlighter import Highlighter, ReprHighlighter\n+from .highlighter import ReprHighlighter\n from .text import Text\n-from .traceback import Traceback\n \n \n class RichHandler(Handler):\n", "is_truncated": false} |
| 15 | edit_file | {
"filepath": "rich/logging.py",
"new_string": " def emit(self, record: LogRecord) -> None:\\n \\\"\\\"\\\"Invoked by logging.\\\"\\\"\\\"\\n message = self.format(record)\\n traceback = None\\n if (\\n self.rich_tracebacks\\n and record.exc_info\\n and record.exc_info != (None, None, None)\\n ):\\n from .traceback import Traceback\\n exc_type, exc_value, exc_traceback = record.exc_info\\n assert exc_type is not None\\n assert exc_value is not None\\n traceback = Traceback.from_exception(\\n exc_type,\\n exc_value,\\n exc_traceback,\\n width=self.tracebacks_width,\\n code_width=self.tracebacks_code_width,\\n extra_lines=self.tracebacks_extra_lines,\\n theme=self.tracebacks_theme,\\n word_wrap=self.tracebacks_word_wrap,\\n show_locals=self.tracebacks_show_locals,\\n locals_max_length=self.locals_max_length,\\n locals_max_string=self.locals_max_string,\\n suppress=self.tracebacks_suppress,\\n max_frames=self.tracebacks_max_frames,\\n )\\n message = record.getMessage()\\n if self.formatter:\\n record.message = record.getMessage()\\n formatter = self.formatter\\n if hasattr(formatter, \\\"usesTime\\\") and formatter.usesTime():\\n record.asctime = formatter.formatTime(record, formatter.datefmt)\\n message = formatter.formatMessage(record)\\n",
"old_string": " def emit(self, record: LogRecord) -> None:\\n \\\"\\\"\\\"Invoked by logging.\\\"\\\"\\\"\\n message = self.format(record)\\n traceback = None\\n if (\\n self.rich_tracebacks\\n and record.exc_info\\n and record.exc_info != (None, None, None)\\n ):\\n exc_type, exc_value, exc_traceback = record.exc_info\\n assert exc_type is not None\\n assert exc_value is not None\\n traceback = Traceback.from_exception(\\n exc_type,\\n exc_value,\\n exc_traceback,\\n width=self.tracebacks_width,\\n code_width=self.tracebacks_code_width,\\n extra_lines=self.tracebacks_extra_lines,\\n theme=self.tracebacks_theme,\\n word_wrap=self.tracebacks_word_wrap,\\n show_locals=self.tracebacks_show_locals,\\n locals_max_length=self.locals_max_length,\\n locals_max_string=self.locals_max_string,\\n suppress=self.tracebacks_suppress,\\n max_frames=self.tracebacks_max_frames,\\n )\\n message = record.getMessage()\\n if self.formatter:\\n record.message = record.getMessage()\\n formatter = self.formatter\\n if hasattr(formatter, \\\"usesTime\\\") and formatter.usesTime():\\n record.asctime = formatter.formatTime(record, formatter.datefmt)\\n message = formatter.formatMessage(record)\\n"
} | {"status": "error", "error_type": "FileEditError", "error_message": "Failed to replace: old_string not found. Ensure you're not escaping content incorrectly andfull{"status": "error", "error_type": "FileEditError", "error_message": "Failed to replace: old_string not found. Ensure you're not escaping content incorrectly and check whitespace, indentation, and context."} |
| 16 | read_file | {
"filepath": "rich/logging.py",
"start_line": 150,
"end_line": 180
} | {"status": "ok", "filepath": "rich/logging.py", "content": " exc_value,\n exc_traceback,\n width=self.tracebacks_width,\n code_width=self.tracebacks_code_width,full{"status": "ok", "filepath": "rich/logging.py", "content": " exc_value,\n exc_traceback,\n width=self.tracebacks_width,\n code_width=self.tracebacks_code_width,\n extra_lines=self.tracebacks_extra_lines,\n theme=self.tracebacks_theme,\n word_wrap=self.tracebacks_word_wrap,\n show_locals=self.tracebacks_show_locals,\n locals_max_length=self.locals_max_length,\n locals_max_string=self.locals_max_string,\n suppress=self.tracebacks_suppress,\n max_frames=self.tracebacks_max_frames,\n )\n message = record.getMessage()\n if self.formatter:\n record.message = record.getMessage()\n formatter = self.formatter\n if hasattr(formatter, \"usesTime\") and formatter.usesTime():\n record.asctime = formatter.formatTime(record, formatter.datefmt)\n message = formatter.formatMessage(record)\n\n message_renderable = self.render_message(record, message)\n log_renderable = self.render(\n record=record, traceback=traceback, message_renderable=message_renderable\n )\n if isinstance(self.console.file, NullFile):\n # Handles pythonw, where stdout/stderr are null, and we return NullFile\n # instance from Console.file. In this case, we still want to make a log record\n # even though we won't be writing anything to a file.\n self.handleError(record)\n else:", "start_line": 150, "end_line": 180, "total_lines": 301, "is_truncated": false} |
| 17 | edit_file | {
"filepath": "rich/logging.py",
"new_string": " def emit(self, record: LogRecord) -> None:\\n \\\"\\\"\\\"Invoked by logging.\\\"\\\"\\\"\\n message = self.format(record)\\n traceback = None\\n if (\\n self.rich_tracebacks\\n and record.exc_info\\n and record.exc_info != (None, None, None)\\n ):\\n from .traceback import Traceback\\n exc_type, exc_value, exc_traceback = record.exc_info\\n assert exc_type is not None\\n assert exc_value is not None\\n traceback = Traceback.from_exception(\\n exc_type,\\n exc_value,\\n exc_traceback,\\n width=self.tracebacks_width,\\n code_width=self.tracebacks_code_width,\\n extra_lines=self.tracebacks_extra_lines,\\n theme=self.tracebacks_theme,\\n word_wrap=self.tracebacks_word_wrap,\\n show_locals=self.tracebacks_show_locals,\\n locals_max_length=self.locals_max_length,\\n locals_max_string=self.locals_max_string,\\n suppress=self.tracebacks_suppress,\\n max_frames=self.tracebacks_max_frames,\\n )\\n message = record.getMessage()\\n if self.formatter:\\n record.message = record.getMessage()\\n formatter = self.formatter\\n if hasattr(formatter, \\\"usesTime\\\") and formatter.usesTime():\\n record.asctime = formatter.formatTime(record, formatter.datefmt)\\n message = formatter.formatMessage(record)\\n",
"old_string": " def emit(self, record: LogRecord) -> None:\\n \\\"\\\"\\\"Invoked by logging.\\\"\\\"\\\"\\n message = self.format(record)\\n traceback = None\\n if (\\n self.rich_tracebacks\\n and record.exc_info\\n and record.exc_info != (None, None, None)\\n ):\\n exc_type, exc_value, exc_traceback = record.exc_info\\n assert exc_type is not None\\n assert exc_value is not None\\n traceback = Traceback.from_exception(\\n exc_type,\\n exc_value,\\n exc_traceback,\\n width=self.tracebacks_width,\\n code_width=self.tracebacks_code_width,\\n extra_lines=self.tracebacks_extra_lines,\\n theme=self.tracebacks_theme,\\n word_wrap=self.tracebacks_word_wrap,\\n show_locals=self.tracebacks_show_locals,\\n locals_max_length=self.locals_max_length,\\n locals_max_string=self.locals_max_string,\\n suppress=self.tracebacks_suppress,\\n max_frames=self.tracebacks_max_frames,\\n )\\n message = record.getMessage()\\n if self.formatter:\\n record.message = record.getMessage()\\n formatter = self.formatter\\n if hasattr(formatter, \\\"usesTime\\\") and formatter.usesTime():\\n record.asctime = formatter.formatTime(record, formatter.datefmt)\\n message = formatter.formatMessage(record)\\n"
} | {"status": "error", "error_type": "FileEditError", "error_message": "Failed to replace: old_string not found. Ensure you're not escaping content incorrectly andfull{"status": "error", "error_type": "FileEditError", "error_message": "Failed to replace: old_string not found. Ensure you're not escaping content incorrectly and check whitespace, indentation, and context."} |
| 18 | read_file | {
"end_line": 170,
"filepath": "rich/logging.py",
"start_line": 130
} | {"status": "ok", "filepath": "rich/logging.py", "content": " level_name = record.levelname\n level_text = Text.styled(\n level_name.ljust(8), f\"logging.level.{full{"status": "ok", "filepath": "rich/logging.py", "content": " level_name = record.levelname\n level_text = Text.styled(\n level_name.ljust(8), f\"logging.level.{level_name.lower()}\"\n )\n return level_text\n\n def emit(self, record: LogRecord) -> None:\n \"\"\"Invoked by logging.\"\"\"\n message = self.format(record)\n traceback = None\n if (\n self.rich_tracebacks\n and record.exc_info\n and record.exc_info != (None, None, None)\n ):\n exc_type, exc_value, exc_traceback = record.exc_info\n assert exc_type is not None\n assert exc_value is not None\n traceback = Traceback.from_exception(\n exc_type,\n exc_value,\n exc_traceback,\n width=self.tracebacks_width,\n code_width=self.tracebacks_code_width,\n extra_lines=self.tracebacks_extra_lines,\n theme=self.tracebacks_theme,\n word_wrap=self.tracebacks_word_wrap,\n show_locals=self.tracebacks_show_locals,\n locals_max_length=self.locals_max_length,\n locals_max_string=self.locals_max_string,\n suppress=self.tracebacks_suppress,\n max_frames=self.tracebacks_max_frames,\n )\n message = record.getMessage()\n if self.formatter:\n record.message = record.getMessage()\n formatter = self.formatter\n if hasattr(formatter, \"usesTime\") and formatter.usesTime():\n record.asctime = formatter.formatTime(record, formatter.datefmt)\n message = formatter.formatMessage(record)\n", "start_line": 130, "end_line": 170, "total_lines": 301, "is_truncated": false} |
| 19 | edit_file | {
"filepath": "rich/logging.py",
"new_string": " def emit(self, record: LogRecord) -> None:\\n \\\"\\\"\\\"Invoked by logging.\\\"\\\"\\\"\\n message = self.format(record)\\n traceback = None\\n if (\\n self.rich_tracebacks\\n and record.exc_info\\n and record.exc_info != (None, None, None)\\n ):\\n from .traceback import Traceback\\n exc_type, exc_value, exc_traceback = record.exc_info\\n assert exc_type is not None\\n assert exc_value is not None\\n traceback = Traceback.from_exception(\\n exc_type,\\n exc_value,\\n exc_traceback,\\n width=self.tracebacks_width,\\n code_width=self.tracebacks_code_width,\\n extra_lines=self.tracebacks_extra_lines,\\n theme=self.tracebacks_theme,\\n word_wrap=self.tracebacks_word_wrap,\\n show_locals=self.tracebacks_show_locals,\\n locals_max_length=self.locals_max_length,\\n locals_max_string=self.locals_max_string,\\n suppress=self.tracebacks_suppress,\\n max_frames=self.tracebacks_max_frames,\\n )\\n message = record.getMessage()\\n if self.formatter:\\n record.message = record.getMessage()\\n formatter = self.formatter\\n if hasattr(formatter, \\\"usesTime\\\") and formatter.usesTime():\\n record.asctime = formatter.formatTime(record, formatter.datefmt)\\n message = formatter.formatMessage(record)\\n",
"old_string": " def emit(self, record: LogRecord) -> None:\\n \\\"\\\"\\\"Invoked by logging.\\\"\\\"\\\"\\n message = self.format(record)\\n traceback = None\\n if (\\n self.rich_tracebacks\\n and record.exc_info\\n and record.exc_info != (None, None, None)\\n ):\\n exc_type, exc_value, exc_traceback = record.exc_info\\n assert exc_type is not None\\n assert exc_value is not None\\n traceback = Traceback.from_exception(\\n exc_type,\\n exc_value,\\n exc_traceback,\\n width=self.tracebacks_width,\\n code_width=self.tracebacks_code_width,\\n extra_lines=self.tracebacks_extra_lines,\\n theme=self.tracebacks_theme,\\n word_wrap=self.tracebacks_word_wrap,\\n show_locals=self.tracebacks_show_locals,\\n locals_max_length=self.locals_max_length,\\n locals_max_string=self.locals_max_string,\\n suppress=self.tracebacks_suppress,\\n max_frames=self.tracebacks_max_frames,\\n )\\n message = record.getMessage()\\n if self.formatter:\\n record.message = record.getMessage()\\n formatter = self.formatter\\n if hasattr(formatter, \\\"usesTime\\\") and formatter.usesTime():\\n record.asctime = formatter.formatTime(record, formatter.datefmt)\\n message = formatter.formatMessage(record)\\n"
} | {"status": "error", "error_type": "FileEditError", "error_message": "Failed to replace: old_string not found. Ensure you're not escaping content incorrectly andfull{"status": "error", "error_type": "FileEditError", "error_message": "Failed to replace: old_string not found. Ensure you're not escaping content incorrectly and check whitespace, indentation, and context."} |
diff --git a/rich/logging.py b/rich/logging.py index c3e7a5f6..eb7b2aa1 100644 --- a/rich/logging.py +++ b/rich/logging.py @@ -1,18 +1,22 @@ +from __future__ import annotations + import logging from datetime import datetime from logging import Handler, LogRecord -from pathlib import Path +import os from types import ModuleType -from typing import ClassVar, Iterable, List, Optional, Type, Union +from typing import TYPE_CHECKING, ClassVar, Iterable, List, Optional, Type, Union + +if TYPE_CHECKING: + from .console import Console, ConsoleRenderable + from .highlighter import Highlighter from rich._null_file import NullFile from . import get_console from ._log_render import FormatTimeCallable, LogRender -from .console import Console, ConsoleRenderable -from .highlighter import Highlighter, ReprHighlighter +from .highlighter import ReprHighlighter from .text import Text -from .traceback import Traceback class RichHandler(Handler):
..........................................F
=================================== FAILURES ===================================
_____________________________ test_input_password ______________________________
prompt = '', stream = None
def unix_getpass(prompt='Password: ', stream=None):
"""Prompt for a password, with echo turned off.
Args:
prompt: Written on stream to ask for the input. Default: 'Password: '
stream: A writable file object to display the prompt. Defaults to
the tty. If no tty is available defaults to sys.stderr.
Returns:
The seKr3t input.
Raises:
EOFError: If our input tty or stdin was closed.
GetPassWarning: When we were unable to turn echo off on the input.
Always restores terminal settings before returning.
"""
passwd = None
with contextlib.ExitStack() as stack:
try:
# Always try reading and writing directly on the tty first.
> fd = os.open('/dev/tty', os.O_RDWR|os.O_NOCTTY)
E OSError: [Errno 6] Device not configured: '/dev/tty'
/Users/jp/.local/share/uv/python/cpython-3.13.15-macos-aarch64-none/lib/python3.13/getpass.py:47: OSError
During handling of the above exception, another exception occurred:
prompt = '', stream = None
def unix_getpass(prompt='Password: ', stream=None):
"""Prompt for a password, with echo turned off.
Args:
prompt: Written on stream to ask for the input. Default: 'Password: '
stream: A writable file object to display the prompt. Defaults to
the tty. If no tty is available defaults to sys.stderr.
Returns:
The seKr3t input.
Raises:
EOFError: If our input tty or stdin was closed.
GetPassWarning: When we were unable to turn echo off on the input.
Always restores terminal settings before returning.
"""
passwd = None
with contextlib.ExitStack() as stack:
try:
# Always try reading and writing directly on the tty first.
fd = os.open('/dev/tty', os.O_RDWR|os.O_NOCTTY)
tty = io.FileIO(fd, 'w+')
stack.enter_context(tty)
input = io.TextIOWrapper(tty)
stack.enter_context(input)
if not stream:
stream = input
except OSError:
# If that fails, see if stdin can be controlled.
stack.close()
try:
> fd = sys.stdin.fileno()
E io.UnsupportedOperation: redirected stdin is pseudofile, has no fileno()
/Users/jp/.local/share/uv/python/cpython-3.13.15-macos-aarch64-none/lib/python3.13/getpass.py:58: UnsupportedOperation
During handling of the above exception, another exception occurred:
monkeypatch = <_pytest.monkeypatch.MonkeyPatch object at 0x1029b06e0>
capsys = <_pytest.capture.CaptureFixture object at 0x10282cb90>
def test_input_password(monkeypatch, capsys) -> None:
def fake_input(prompt, stream=None):
console.file.write(prompt)
return "bar"
import getpass
monkeypatch.setattr(getpass, "getpass", fake_input)
console = Console()
> user_input = console.input(prompt="foo:", password=True)
tests/test_console.py:407:
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _
rich/console.py:2169: in input
result = getpass("", stream=stream)
/Users/jp/.local/share/uv/python/cpython-3.13.15-macos-aarch64-none/lib/python3.13/getpass.py:61: in unix_getpass
passwd = fallback_getpass(prompt, stream)
/Users/jp/.local/share/uv/python/cpython-3.13.15-macos-aarch64-none/lib/python3.13/getpass.py:126: in fallback_getpass
return _raw_input(prompt, stream)
/Users/jp/.local/share/uv/python/cpython-3.13.15-macos-aarch64-none/lib/python3.13/getpass.py:146: in _raw_input
line = input.readline()
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _
self = <_pytest.capture.DontReadFromInput object at 0x1027d1550>, size = -1
def read(self, size: int = -1) -> str:
> raise OSError(
"pytest: reading from stdin while output is captured! Consider using `-s`."
)
E OSError: pytest: reading from stdin while output is captured! Consider using `-s`.
/Users/jp/repos/kaggle-gemini-coding-agent-post-training/.envs/rich/lib/python3.13/site-packages/_pytest/capture.py:209: OSError
----------------------------- Captured stdout call -----------------------------
foo:
----------------------------- Captured stderr call -----------------------------
Warning: Password input may be echoed.
=============================== warnings summary ===============================
test_console.py::test_input_password
/Users/jp/.local/share/uv/python/cpython-3.13.15-macos-aarch64-none/lib/python3.13/getpass.py:61: GetPassWarning: Can not control echo on the terminal.
passwd = fallback_getpass(prompt, stream)
-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
!!!!!!!!!!!!!!!!!!!!!!!!!! stopping after 1 failures !!!!!!!!!!!!!!!!!!!!!!!!!!!
1 failed, 42 passed, 1 warning in 0.27s