fix: update stale interactive-loop snippets in implementation.md (#1167)

[microsasa]

Closes #1167

Changes

Documentation (src/copilot_usage/docs/implementation.md)

1. _read_line_nonblocking snippet — Updated to match the actual implementation that checks for empty readline() and raises EOFError("stdin closed") to prevent infinite polling.

2. Fallback reader snippet — Added the except EOFError: break clause before except (ValueError, OSError): to reflect the actual control flow in _interactive_loop.

3. Prose updates — Updated the "Non-blocking input with select()" subsection to mention that _read_line_nonblocking raises EOFError when stdin is closed, and updated the fallback section to explain that EOFError causes an immediate loop break rather than starting the fallback reader.

Tests (tests/test_docs.py)

Added test_read_line_nonblocking_doc_snippet_matches_implementation which:

• Asserts EOFError appears in _read_line_nonblocking's source via inspect.getsource
• Asserts except EOFError appears in _interactive_loop's source
• Asserts EOFError appears in implementation.md

This prevents the docs from silently diverging from the implementation again.

Warning

⚠️ Firewall blocked 2 domains

The following domains were blocked by the firewall during workflow execution:

• astral.sh
• pypi.org

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "astral.sh"
    - "pypi.org"

See Network Configuration for more information.

Generated by Issue Implementer · ● 7.4M · ◷

[Copilot]

Pull request overview

Updates the interactive-loop documentation to match the current copilot_usage.cli EOF handling behavior, and adds regression tests to prevent future doc/implementation drift.

Changes:

• Refreshes implementation.md snippets/prose to include _read_line_nonblocking raising EOFError("stdin closed") and _interactive_loop breaking on EOFError.
• Adds a new docs regression test that checks the implementation and documentation mention EOFError handling.
• Imports inspect to enable source-based assertions in the new test.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
tests/test_docs.py	Adds a regression test using inspect.getsource to ensure EOFError handling remains reflected in both code and docs.
src/copilot_usage/docs/implementation.md	Updates interactive-loop snippets and prose to accurately document EOFError behavior and fallback reader control flow.