cppa-cursor-browser: trim over-broad models/__init__.py exports #131

Calendar Day

Thursday, July 16, 2026 (PR 2 of 3)

Planned Effort

2 story points (Low) — sprint item #7 (Test 20)

Problem

models/__init__.py exports 15 names in __all__ (L10–26), including low-level types that should not be public — notably BubbleRole and BubbleMetadata re-exported from models.bubble_display (L1). PR #118 trimmed one test-only re-export (16→15); this finishes trimming so the public API is the domain surface (Bubble, Composer, Conversation, Workspace, SearchResult, SchemaError, …).

Goal

One merged PR trimming __all__ and repointing in-repo imports. No code moves.

Scope

• Confirm no external consumer does from models import BubbleRole / BubbleMetadata.
• Remove internal-only names from __all__.
• Repoint any in-repo imports to defining module (models.bubble_display).
• Keep change to export surface only.

Acceptance Criteria

• __all__ trimmed to intended public surface.
• All in-repo imports of removed names repointed — no breakage.
• mypy --strict + tests green; CI green; PR approved.

Verification

cd C:\Users\Jasen\CppAliance\cppa-cursor-browser
.\.venv\Scripts\Activate.ps1
pytest -q
mypy --strict .
# Confirm: from models import BubbleRole raises ImportError
python -c "from models import BubbleRole"  # expect ImportError
python -c "from models.bubble_display import BubbleRole"  # expect success