📝 Address PR code review comments for documentation and formatting

- Add cross-references between command line flags and environment variables
- Clarify default behavior when PYTHON_TRACEBACK_TIMESTAMPS is unset
- Simplify traceback.rst wording to use "canonical output"
- Fix alphabetical ordering in test_config.py
- Improve comment formatting in test_sys.py for better readability

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: gpshead

Doc/library/exceptions.rst

@@ -165,7 +165,8 @@ The following exceptions are used mostly as base classes for other exceptions.
       (usually: when it was raised); the same accuracy as :func:`time.time_ns`.
       Display of these timestamps after the exception message in tracebacks is
       off by default but can be configured using the
-      :envvar:`PYTHON_TRACEBACK_TIMESTAMPS` environment variable.  In
+      :envvar:`PYTHON_TRACEBACK_TIMESTAMPS` environment variable or the
+      :option:`-X traceback_timestamps <-X>` command line option.  In
       applications with complicated exception chains and exception groups it
       may be useful to help understand what happened when.  The value will be
       ``0`` if a timestamp was not recorded.  :exc:`StopIteration` and

Doc/library/sys.rst

@@ -607,7 +607,8 @@ always available. Unless explicitly noted otherwise, all variables are read-only
           :envvar:`PYTHON_CONTEXT_AWARE_WARNINGS`
 
       * - .. attribute:: flags.traceback_timestamps
-        - :option:`-X traceback_timestamps <-X>`.  This is a string containing
+        - :option:`-X traceback_timestamps <-X>` and
+          :envvar:`PYTHON_TRACEBACK_TIMESTAMPS`.  This is a string containing
           the selected format (``us``, ``ns``, ``iso``),  or an empty string
           when disabled.
 

Doc/library/traceback.rst

@@ -213,8 +213,7 @@ Module-Level Functions
    via the :envvar:`PYTHON_TRACEBACK_TIMESTAMPS` environment variable or the
    :option:`-X traceback_timestamps <-X>` command line option, any timestamp
    after the exception message will be omitted. This is useful for tests or
-   other situations where you need consistent output regardless of when
-   exceptions occur.
+   other situations where you need canonical output.
 
    .. versionchanged:: 3.10
       The *etype* parameter has been renamed to *exc* and is now

Doc/using/cmdline.rst

@@ -1322,10 +1322,10 @@ conflict.
    * ``iso``: Prints the timestamp formatted by :meth:`~datetime.datetime.isoformat` (also microsecond precision).
    * ``0``: Explicitly disables timestamps.
 
-   The time is not recorded on the :exc:`StopIteration` family of exceptions
-   for performance reasons as those are used for control flow rather than
-   errors. If unset, empty, or set to invalid values, this feature remains disabled
-   when using the environment variable.
+   When unset, timestamps are disabled by default. The time is not recorded on
+   the :exc:`StopIteration` family of exceptions for performance reasons as those
+   are used for control flow rather than errors. If set to empty or invalid values,
+   this feature remains disabled when using the environment variable.
 
    Note that the command line option :option:`-X` ``traceback_timestamps`` takes
    precedence over this environment variable when both are specified.

Lib/test/test_capi/test_config.py

@@ -86,8 +86,8 @@ def test_config_get(self):
             ("stdio_encoding", str, None),
             ("stdio_errors", str, None),
             ("stdlib_dir", str | None, "_stdlib_dir"),
-            ("tracemalloc", int, None),
             ("traceback_timestamps", str, None),
+            ("tracemalloc", int, None),
             ("use_environment", bool, None),
             ("use_frozen_modules", bool, None),
             ("use_hash_seed", bool, None),

Lib/test/test_sys.py

@@ -1890,8 +1890,11 @@ def test_pythontypes(self):
         # symtable entry
         # XXX
         # sys.flags
-        # FIXME: The +3 is for the 'gil', 'thread_inherit_context',
-        # 'context_aware_warnings', and 'traceback_timestamps' flags.
+        # FIXME: The non_sequence_fields adjustment is for these flags:
+        # - 'gil'
+        # - 'thread_inherit_context'
+        # - 'context_aware_warnings'
+        # - 'traceback_timestamps'
         # It will not be necessary once GH-122575 is fixed.
         non_sequence_fields = 4
         check(sys.flags, vsize('') + self.P + self.P * (non_sequence_fields + len(sys.flags)))