refactor(pebble): convert orchestrator.py into a package (slice 0 of PR #191 carve)

[jpb33333]

Summary

Pure refactor with zero behavior change. Renames pebble/orchestrator.py (862 lines, untouched in content) to pebble/orchestrator/_pipeline.py and adds pebble/orchestrator/__init__.py that re-exports the seven symbols imported by callers outside the package today.

This is slice 0 of carving PR #191 (feat/pebble-l2-swarm) into focused slices off current dev. The orchestrator package is the foundation: future slices add sibling modules (schemas.py, planner.py, renderer.py, …) under this same package.

Why

PR #191 currently can't be merged in any form — it's stale enough to phantom-delete CODEOWNERS, all home pages, and the entire projects-overhaul. Splitting it in place would carry those phantom deletions into every slice. So the strategy is carve: start fresh branches off current dev, copy over only intentional L2 swarm additions. See ~/.claude/plans/pr191-carve-strategy.md for the full 12-slice plan.

This slice unblocks every subsequent orchestrator slice by making pebble/orchestrator/ an importable package instead of a single module.

How tested

• pytest pebble/tests/ -q — 290 passed (identical to dev baseline; no regressions).
• pytest financial_forecasting/tests/ -q — 827 passed, 22 skipped (unchanged).
• All seven re-exported symbols import cleanly:

  from pebble.orchestrator import (
      ProspectBudgetTracker, research_single_prospect, score_source_richness,
      activate_foragers, quorum_verify_claims, synthesize_profile, verify_urls,
  )

What changes mechanically

• git mv pebble/orchestrator.py pebble/orchestrator/_pipeline.py — Git tracks as 99% similar rename.
• 6 lines in _pipeline.py: single-dot relative imports (from .model_client) bump to double-dot (from ..model_client) since the file now sits one level deeper.
• New pebble/orchestrator/__init__.py (28 lines) re-exports the public API.

Callers verified unchanged on dev

Caller	Import
pebble/main.py:591	from .orchestrator import research_single_prospect
pebble/clusters/__init__.py:15	from ..orchestrator import ProspectBudgetTracker
pebble/handlers/tier2.py:41	from ..orchestrator import (score_source_richness, activate_foragers, ProspectBudgetTracker)
pebble/handlers/tier3.py:33	from ..orchestrator import (research_single_prospect, quorum_verify_claims, synthesize_profile, verify_urls, ProspectBudgetTracker)
pebble/tests/test_clusters.py	4 imports of ProspectBudgetTracker

All keep working because __init__.py re-exports those exact names.

Risk

Extremely low.

• Zero behavior change. Same source code, same line numbers within the renamed file (only the import prefix flipped from . to ..).
• Same module-level constants, same logger name (pebble.orchestrator), same public functions.
• The only externally observable change is that pebble.orchestrator is now a package whose __init__.py re-exports the API, instead of a flat module. Python resolves the imports identically from a caller's perspective.

Rollback

Revert this single commit. pebble/orchestrator.py returns to its previous location with identical content; the directory + __init__.py disappear; all imports keep working because they were never broken to begin with.

Next slices

• Slice 1: pebble/orchestrator/schemas.py + tests (Pydantic data models — Plan, FinalResponse, Citation, etc.). Foundation for the rest.
• See ~/.claude/plans/pr191-carve-strategy.md for the full sequence.

🤖 Generated with Claude Code

[claude]

⚠️ Code review skipped — your organization's overage spend limit has been reached.

Code review is billed via overage credits. To resume reviews, an organization admin can raise the monthly limit at claude.ai/admin-settings/claude-code.

Once credits are available, push a new commit or reopen this pull request to trigger a review.