crackerjack 0.89.2__tar.gz → 0.91.0__tar.gz

{crackerjack-0.89.2 → crackerjack-0.91.0}/CHANGELOG.md

@@ -10,6 +10,33 @@ blocks_on: []
 topic: changelog
 ---
 
+## [0.91.0] - 2026-10-20
+
+### Added
+
+- crackerjack: Dispatch plan_index_sync to oneiric.plan_index via feature flag
+
+### Fixed
+
+- crackerjack: Drop redundant bool() cast on already-Pydantic-bool setting
+
+### Internal
+
+- deps: Bump oneiric>= to 0.33 (plan_index library floor)
+
+## [0.90.0] - 2026-10-19
+
+### Added
+
+- crackerjack: Adopt OneiricSettings.substrate for cache + pool fields (Phase E2 substrate rollout)
+- crackerjack: FallbackChainCodeFixer uses oneiric BifrostLLMRouter (plan 6, phase 5)
+- crackerjack: Wire oneiric_report_runtime component-boot hook per oneiric-substrate umbrella Phase 3 (2026-10-18)
+
+### Internal
+
+- Crackerjack slice — finish residual ollama strings (plan 5)
+- deps: Bump fastmcp pin to >=4.0.0,<5 (operator correction 2026-10-18)
+
 ## [0.89.2] - 2026-10-16
 
 ### Added

{crackerjack-0.89.2 → crackerjack-0.91.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: crackerjack
-Version: 0.89.2
+Version: 0.91.0
 Summary: Crackerjack Python project management tool
 Project-URL: documentation, https://github.com/lesleslie/crackerjack
 Project-URL: homepage, https://github.com/lesleslie/crackerjack
@@ -29,7 +29,7 @@ Requires-Dist: creosote>=5.2.0
 Requires-Dist: cryptography>=50.0.2
 Requires-Dist: docstring-to-markdown>=0.17
 Requires-Dist: fastapi>=0.142.2
-Requires-Dist: fastmcp>=3.4.7
+Requires-Dist: fastmcp<5,>=4.0.0
 Requires-Dist: hatchling>=1.32.4
 Requires-Dist: httpx2>=2.13.1
 Requires-Dist: hypothesis>=6.168.3
@@ -42,7 +42,7 @@ Requires-Dist: mcp-common>=0.30.2
 Requires-Dist: mdformat-ruff>=0.1.3
 Requires-Dist: mdformat>=1.0.0
 Requires-Dist: numpy>=2.5.3
-Requires-Dist: oneiric>=0.29.2
+Requires-Dist: oneiric>=0.33
 Requires-Dist: pydantic>=2.13.5
 Requires-Dist: pyleak>=0.2.0
 Requires-Dist: pymetrica>=1.6.0

{crackerjack-0.89.2 → crackerjack-0.91.0}/crackerjack/adapters/ai/unified.py

@@ -7,6 +7,8 @@ from uuid import UUID
 
 from mcp_common.llm import FallbackChain
 from mcp_common.llm.config import LLMSettings, ProviderConfig
+from oneiric.adapters.llm.bifrost import get_router
+from oneiric.adapters.llm.llm_interface import LLMProvider
 
 from crackerjack.adapters.ai.base import BaseCodeFixer, BaseCodeFixerSettings
 from crackerjack.models.adapter_metadata import AdapterStatus
@@ -47,12 +49,36 @@ _DEFAULT_PROVIDERS: dict[str, dict[str, t.Any]] = {
 class FallbackChainSettings(BaseCodeFixerSettings):
     model: str = "MiniMax-M2.7"
     task_type: str = "code_generation"
-    llama_server_url: str = "http://localhost: 8081"
+    llama_server_url: str = "http://localhost:8081"
 
 
 def _build_llm_settings() -> LLMSettings:
+    # Phase 5 (REQ-BIFROST-010, bifrost plan §5): the OPENAI-compatible
+    # ``minimax`` provider's ``base_url`` comes from the oneiric
+    # ``BifrostLLMRouter`` singleton. When the router is enabled and
+    # routes OPENAI, the provider transparently hits the local
+    # Bifrost gateway (which forwards to the upstream cloud);
+    # when the router returns ``None`` (gateway disabled or OPENAI
+    # route unconfigured), falls through to the ``MINIMAX_BASE_URL``
+    # env-var (operator override) and then to the public
+    # ``api.minimax.io`` default. The router → env-var → hardcoded
+    # precedence mirrors ``mahavishnu/workers/cloud_worker.py:86-96``.
+    # The ``llama_server`` provider keeps its env-var-only resolution
+    # (the router's ``_PROVIDER_ROUTE`` only maps ANTHROPIC + OPENAI).
+    bifrost_openai_url = get_router().resolve(LLMProvider.OPENAI)
     providers = _DEFAULT_PROVIDERS.copy()
-    llama_url = os.environ.get("LLAMA_SERVER_URL", "http://localhost: 8081")
+    providers["minimax"] = providers["minimax"].copy()
+    if bifrost_openai_url:
+        providers["minimax"]["base_url"] = str(bifrost_openai_url)
+    else:
+        # Router disabled — honor a ``MINIMAX_BASE_URL`` operator
+        # override (added in Phase 5 follow-up so the env var matches
+        # the comment that has been there since Phase 0). The default
+        # stays at the public ``api.minimax.io`` direct-cloud URL.
+        env_minimax_url = os.environ.get("MINIMAX_BASE_URL")
+        if env_minimax_url:
+            providers["minimax"]["base_url"] = env_minimax_url
+    llama_url = os.environ.get("LLAMA_SERVER_URL", "http://localhost:8081")
     providers["llama_server"] = providers["llama_server"].copy()
     providers["llama_server"]["base_url"] = llama_url
     return LLMSettings(

{crackerjack-0.89.2 → crackerjack-0.91.0}/crackerjack/cli/facade.py

@@ -105,11 +105,26 @@ class CrackerjackCLIFacade:
 
     def _start_mcp_server(self) -> None:
         try:
+            from crackerjack.mcp.oneiric_runtime import report_runtime_to_oneiric
+            from crackerjack.mcp.server_core import _get_default_mcp_app
             from crackerjack.mcp.server_core import main as start_mcp_main
 
             self.console.print(
                 "[bold cyan]🤖 Starting Crackerjack MCP Server...[/bold cyan]",
             )
+            # Phase 3 component-boot hook per the oneiric-substrate umbrella
+            # (2026-10-15): fire-and-forget push of runtime facts to Oneiric
+            # before the server starts. The helper spawns a daemon thread
+            # so this call returns immediately; the thread swallows all errors.
+            # The FastMCP app is built lazily via _get_default_mcp_app() so
+            # the asyncio.run() inside create_mcp_server() does not fire at
+            # module-import time (which would collide with test runners
+            # that start an event loop during collection). When FastMCP is
+            # unavailable the getter returns None and we skip the early push;
+            # main() also reports runtime once its own server is built.
+            mcp_app = _get_default_mcp_app()
+            if mcp_app is not None:
+                report_runtime_to_oneiric(mcp_app)
             start_mcp_main(str(self.pkg_path))
         except ImportError:
             self.console.print(

{crackerjack-0.89.2 → crackerjack-0.91.0}/crackerjack/cli/handlers/provider_selection.py

@@ -46,7 +46,7 @@ class ProviderSelectionCLI:
         header = Panel(
             "[bold cyan]AI Provider Selection[/bold cyan]\n\n"
             "Configure the AI provider for crackerjack's code fixing features.\n"
-            "Current options: MiniMax, llama-server, Ollama",
+            "Current options: MiniMax, llama-server",
             title="Crackerjack Configuration",
             border_style="cyan",
         )

{crackerjack-0.89.2 → crackerjack-0.91.0}/crackerjack/cli/options.py

@@ -671,7 +671,7 @@ CLI_OPTIONS = {
         "--select-provider",
         help=(
             "Launch interactive provider selection menu. "
-            "Configure AI provider for code fixing (Claude, Qwen, Ollama). "
+            "Configure AI provider for code fixing (minimax, llama_server). "
             "Saves selection to settings/local.yaml."
         ),
     ),

{crackerjack-0.89.2 → crackerjack-0.91.0}/crackerjack/config/settings.py

@@ -4,6 +4,7 @@ import typing as t
 from pathlib import Path
 
 from oneiric.core.config import OneiricMCPConfig
+from oneiric.core.substrate_settings import SubstrateSettings
 from pydantic import Field, model_validator
 
 from .validators import reshape_adapter_timeouts, warn_unknown_pyproject_subtables
@@ -143,6 +144,7 @@ class DocumentationSettings(OneiricMCPConfig):
     auto_coverage_ratchet_on_publish: bool = True
     dry_run_by_default: bool = False
     backup_before_cleanup: bool = True
+    use_shared_regenerator: bool = False  # NEW: per Round 3 Decision #10
     essential_files: list[str] = [
         "AGENTS.md",
         "CHANGELOG.md",
@@ -485,6 +487,15 @@ class AsyncTaskSettings(OneiricMCPConfig):
 class CrackerjackSettings(OneiricMCPConfig):
     pkg_path: Path | None = None
 
+    substrate: SubstrateSettings = Field(
+        default_factory=SubstrateSettings,
+        description=(
+            "Shared Bodai substrate (Phase E1/E2): cache/warm/cold/embedding/"
+            "pool/retry/circuit-breaker/telemetry/auth. Owns the cache_* and "
+            "pool__* settings that were previously local to CrackerjackSettings."
+        ),
+    )
+
     @model_validator(mode="before")
     @classmethod
     def _reshape_adapter_timeouts(cls, data: t.Any) -> t.Any:  # REQ-004
@@ -526,10 +537,6 @@ class CrackerjackSettings(OneiricMCPConfig):
     enable_orchestration: bool = True
     orchestration_mode: str = "oneiric"
     enable_caching: bool = True
-    cache_backend: str = "memory"
-    cache_ttl: int = 3600
-    cache_max_entries: int = 100
-    max_parallel_hooks: int = 4
     default_timeout: int = 1800
     stop_on_critical_failure: bool = True
     enable_dependency_resolution: bool = True

{crackerjack-0.89.2 → crackerjack-0.91.0}/crackerjack/services/async_file_io.py

@@ -34,11 +34,11 @@ def get_io_executor() -> ThreadPoolExecutor:
                     from crackerjack.config import CrackerjackSettings
 
                     settings = CrackerjackSettings()
-                    max_workers = settings.max_parallel_hooks
+                    max_workers = settings.substrate.pool__max_workers
                 except Exception:
                     max_workers = 4
                     logger.warning(
-                        "Could not load max_parallel_hooks from settings, using default: 4"
+                        "Could not load pool__max_workers from substrate, using default: 4"
                     )
 
                 _io_executor = ThreadPoolExecutor(

{crackerjack-0.89.2 → crackerjack-0.91.0}/crackerjack/tools/plan_index_sync.py

@@ -45,6 +45,8 @@ except ImportError as exc:  # pragma: no cover - import guard
     )
     raise SystemExit(2) from exc
 
+from crackerjack.config.loader import load_settings
+from crackerjack.config.settings import CrackerjackSettings
 
 # ---------------------------------------------------------------------------
 # Output formatting — local-only helpers for console lines below.
@@ -107,7 +109,6 @@ DEFAULT_STORES: tuple[str, ...] = (
     "docs/adr/",
     "docs/plans/",
     "docs/specs/",
-    "docs/plans/",
     ".claude/decisions/",
     "docs/followups/",
 )
@@ -366,7 +367,7 @@ def _render_index(entries_by_store: dict[str, list[Entry]], generated_at: str) -
         (
             (
                 f"_Regenerated {generated_at} by "
-                f"`crackerjack.tools.plan_index_sync` (fast-hook)._"
+                f"`crackerjack.tools.plan_index_sync` (phase)._"
             ),
             "",
         )
@@ -410,7 +411,7 @@ def main(argv: list[str] | None = None) -> int:
         prog="plan_index_sync",
         description=(
             "Regenerate docs/plans/PLAN_INDEX.md from per-store frontmatter. "
-            "Used as a crackerjack fast-hook (plan-index-sync)."
+            "Used as a crackerjack phase (plan-index-sync)."
         ),
     )
     parser.add_argument(
@@ -432,6 +433,27 @@ def main(argv: list[str] | None = None) -> int:
     )
     args = parser.parse_args(argv)
 
+    settings = load_settings(CrackerjackSettings)
+    use_shared = getattr(settings.documentation, "use_shared_regenerator", False)
+    if use_shared:
+        from oneiric.plan_index import regenerate as shared_regenerate
+
+        # --dry-run ≡ check=True (per spec acceptance criterion I-1)
+        return shared_regenerate(
+            repo_root=args.repo_root.resolve(),
+            out_path=(args.out.resolve() if args.out else None),
+            check=bool(args.dry_run),
+        )
+
+    # LEGACY BRANCH: inline the pre-dispatch body of main() (originally
+    # at crackerjack/tools/plan_index_sync.py:407-454 before this rewrite).
+    #
+    # Per crackerjack-specialist review C3: do NOT extract to
+    # ``_legacy_render(args)``. Inlining is preferable — fewer new
+    # symbols, tighter review diff, and the legacy branch is small
+    # once the shared import is removed. The CLI surface
+    # (``--repo-root``, ``--out``, ``--dry-run``) stays identical so the
+    # existing ``test_help_smoke`` continues to pass.
     new_content = render_plan_index(args.repo_root)
 
     if args.dry_run:

{crackerjack-0.89.2 → crackerjack-0.91.0}/docs/.archive/2026-08-06-ai-fix-removal-cleanup/README.md

@@ -172,4 +172,4 @@ forward-references to them in the active docs were either rewritten
 
 See `docs/audits/2026-09-09-crackerjack-docs-audit.md` Category F
 ("`crackerjack.agents.*` deleted surface still narratively alive")
-for the trigger rationale and a list of broken forward-links.
+for the trigger rationale and a list of broken forward-links.

{crackerjack-0.89.2 → crackerjack-0.91.0}/docs/.archive/uncategorized/CLAUDE_ARCHITECTURE.md

@@ -1,4 +1,3 @@
 # CLAUDE Architecture
 
 This document is a lightweight placeholder for architecture guidance used by tests.
-

{crackerjack-0.89.2 → crackerjack-0.91.0}/docs/.archive/uncategorized/CLAUDE_PATTERNS.md

@@ -1,4 +1,3 @@
 # CLAUDE Patterns
 
 This document is a lightweight placeholder for patterns guidance used by tests.
-

{crackerjack-0.89.2 → crackerjack-0.91.0}/docs/.archive/uncategorized/CLAUDE_PROTOCOLS.md

@@ -1,4 +1,3 @@
 # CLAUDE Protocols
 
 This document is a lightweight placeholder for protocol guidance used by tests.
-

{crackerjack-0.89.2 → crackerjack-0.91.0}/docs/.archive/uncategorized/CLAUDE_QUICKSTART.md

@@ -1,4 +1,3 @@
 # CLAUDE Quickstart
 
 This document is a lightweight placeholder quickstart guide used by tests.
-

{crackerjack-0.89.2 → crackerjack-0.91.0}/docs/decisions/.archive/implementation-plans/2026-10-16-plan-index-git-aware-and-tool-flag.md

@@ -40,4 +40,4 @@ Empty strategies (unknown hook name) run zero hooks in that phase — silent sig
 
 - **`plan_index_sync` test fixtures in non-git sandboxes** must use the rglob fallback path. The `_tracked_md_names` helper returns `None` on git failure and the discovery falls back to rglob; new tests should construct fixtures via `monkeypatch.setattr(pis, "_tracked_md_names", lambda *_: None)` (see `tests/unit/test_plan_index_sync.py::test_discover_files_falls_back_to_rglob_when_git_missing`).
 - **`--tool <name>` with a mis-typed hook name** completes the phase with zero results — the user sees a successful phase that ran nothing. This is intentional (the gate still passes, the user has a clear signal to check spelling) but worth flagging in onboarding if anyone interprets a fast phase that finishes in <1s as "the hook passed."
-- **Hook naming convention**: the generator normalises `-` ↔ `_` (CLI ergonomics: `--tool ruff-check` matches a hook stored as `ruff_check`). New hooks should pick one and stay consistent within a single repo.
+- **Hook naming convention**: the generator normalises `-` ↔ `_` (CLI ergonomics: `--tool ruff-check` matches a hook stored as `ruff_check`). New hooks should pick one and stay consistent within a single repo.

{crackerjack-0.89.2 → crackerjack-0.91.0}/docs/plans/PLAN_INDEX.md

@@ -1,6 +1,6 @@
 # Plan Index
 
-_Regenerated 2026-10-17 by `crackerjack.tools.plan_index_sync` (fast-hook)._
+_Regenerated 2026-10-20 by `crackerjack.tools.plan_index_sync` (phase)._
 
 ### Docs: adr
 
@@ -53,30 +53,6 @@ _Regenerated 2026-10-17 by `crackerjack.tools.plan_index_sync` (fast-hook)._
 | [`docs/specs/2026-09-07-crackerjack-multi-language-design.md`](../specs/2026-09-07-crackerjack-multi-language-design.md) | 2026-09-07 | `complete` | `canonical` | `architecture` | Crackerjack Multi-Language Extension |
 | [`docs/specs/2026-10-10-crackerjack-oneiric-loader-migration-design.md`](../specs/2026-10-10-crackerjack-oneiric-loader-migration-design.md) | 2026-10-10 | `draft` | `implementation` | `settings-resolution` | Spec: Crackerjack Settings Loader — Migration to Oneiric |
 
-### Docs: plans
-
-| Path | Date | Status | Role | Topic | Title |
-|---|---|---|---|---|---|
-| [`docs/plans/2026-05-20-phase-0-event-bus-plan.md`](2026-05-20-phase-0-event-bus-plan.md) | 2026-07-17 | `active` | `implementation` | `lifecycle` | Phase 0 Implementation Plan — AI-Fix Event Bus |
-| [`docs/plans/2026-06-02-ai-fix-display-loop-bugs.md`](2026-06-02-ai-fix-display-loop-bugs.md) | 2026-07-17 | `active` | `implementation` | `lifecycle` | AI-Fix Display & Loop Bugs Implementation Plan |
-| [`docs/plans/2026-06-03-dhara-mcp-migration.md`](2026-06-03-dhara-mcp-migration.md) | 2026-07-17 | `active` | `implementation` | `lifecycle` | Dhara MCP Migration Implementation Plan |
-| [`docs/plans/2026-06-29-ty-ratchet-cleanup.md`](2026-06-29-ty-ratchet-cleanup.md) | 2026-07-17 | `active` | `implementation` | `lifecycle` | ty-ratchet pipeline cleanup (E.3 + missing-dir) Implementation Plan |
-| [`docs/plans/2026-07-08-fix-sandbox-integration.md`](2026-07-08-fix-sandbox-integration.md) | 2026-07-17 | `active` | `implementation` | `lifecycle` | FixSandbox Production Integration Plan |
-| [`docs/plans/2026-07-10-libcst-surgeon-extract-method-fallback.md`](2026-07-10-libcst-surgeon-extract-method-fallback.md) | 2026-07-17 | `active` | `implementation` | `lifecycle` | W3 LibcstSurgeon Extract-Method Fallback Fix — Implementation Plan |
-| [`docs/plans/2026-07-10-output-validator-traceback-details.md`](2026-07-10-output-validator-traceback-details.md) | 2026-07-17 | `active` | `implementation` | `lifecycle` | OutputValidator Traceback Details Implementation Plan |
-| [`docs/plans/2026-07-10-validation-coordinator-serialization.md`](2026-07-10-validation-coordinator-serialization.md) | 2026-07-17 | `active` | `implementation` | `lifecycle` | ValidationCoordinator Concurrency Serialization Implementation Plan |
-| [`docs/plans/2026-07-11-ai-fix-e501-post-processor.md`](2026-07-11-ai-fix-e501-post-processor.md) | 2026-07-17 | `active` | `implementation` | `lifecycle` | AI-fix ruff E501 line-length post-processor — Implementation Plan |
-| [`docs/plans/2026-07-11-ai-fix-no-op-circuit-breaker.md`](2026-07-11-ai-fix-no-op-circuit-breaker.md) | 2026-07-17 | `active` | `implementation` | `lifecycle` | AI-fix no-op circuit breaker — Implementation Plan |
-| [`docs/plans/2026-07-11-ai-fix-regen-timeout.md`](2026-07-11-ai-fix-regen-timeout.md) | 2026-07-17 | `active` | `implementation` | `lifecycle` | AI-fix plan-regeneration timeout — Implementation Plan |
-| [`docs/plans/2026-07-12-eventbridge-publisher.md`](2026-07-12-eventbridge-publisher.md) | 2026-07-17 | `active` | `implementation` | `lifecycle` | Crackerjack EventBridge Publisher Implementation Plan |
-| [`docs/plans/2026-08-06-ai-fix-external-loop.md`](2026-08-06-ai-fix-external-loop.md) | 2026-08-06 | `complete` | `historical` | `lifecycle` | AI-Fix External Loop + Akosha Hook Implementation Plan |
-| [`docs/plans/2026-08-06-crackerjack-ruff-fix-policy.md`](2026-08-06-crackerjack-ruff-fix-policy.md) | 2026-08-06 | `draft` | `implementation` | `lifecycle` | Crackerjack Ruff Fix Policy Implementation Plan |
-| [`docs/plans/2026-08-10-ai-fix-loop-task-1-kickoff.md`](2026-08-10-ai-fix-loop-task-1-kickoff.md) | 2026-08-10 | `draft` | `implementation` | `lifecycle` | AI-Fix Loop Plan — Task 1 Execution Kickoff |
-| [`docs/plans/2026-08-11-darnlink-replaces-check-local-links.md`](2026-08-11-darnlink-replaces-check-local-links.md) | 2026-08-11 | `draft` | `implementation` | `lifecycle` | Replace `check-local-links` with `darnlink` across the Bodai ecosystem |
-| [`docs/plans/2026-09-07-bodai-shared-gitignore-snippet.md`](2026-09-07-bodai-shared-gitignore-snippet.md) | 2026-09-07 | `draft` | `implementation` | `gitignore-conformance` | Bodai Shared `.gitignore` Snippet — crackerjack-enforced |
-| [`docs/plans/2026-10-10-crackerjack-oneiric-loader-migration.md`](2026-10-10-crackerjack-oneiric-loader-migration.md) | 2026-10-10 | `draft` | `implementation` | `settings-resolution` | Crackerjack Settings Loader — Migration to Oneiric — Implementation Plan |
-| [`docs/plans/2026-10-11-remove-zuban-skylos-lsp-from-crackerjack.md`](2026-10-11-remove-zuban-skylos-lsp-from-crackerjack.md) | 2026-10-11 | `active` | `implementation` | `architecture` | Remove Zuban + Skylos LSP Adapters from Crackerjack |
-
 ### .claude/decisions
 
 | Path | Date | Status | Role | Topic | Title |
@@ -107,8 +83,8 @@ _Regenerated 2026-10-17 by `crackerjack.tools.plan_index_sync` (fast-hook)._
 | Role \\ Lifecycle | draft | active | partial | shipped | complete | Total |
 |---|---|---|---|---|---|---|
 | `canonical` | · | 10 | · | · | 2 | **12** |
-| `implementation` | 21 | 34 | · | · | 1 | **56** |
+| `implementation` | 16 | 21 | · | · | 1 | **38** |
 | `umbrella` | · | · | · | · | · | **0** |
-| `historical` | · | · | · | · | 2 | **2** |
+| `historical` | · | · | · | · | 1 | **1** |
 | `superseded` | · | · | · | · | · | **0** |
-| **Total** | **21** | **44** | **·** | **·** | **5** | **70** |
+| **Total** | **16** | **31** | **·** | **·** | **4** | **51** |

{crackerjack-0.89.2 → crackerjack-0.91.0}/pyproject.toml

@@ -6,7 +6,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "crackerjack"
-version = "0.89.2"
+version = "0.91.0"
 description = "Crackerjack Python project management tool"
 readme = "README.md"
 authors = [
@@ -36,7 +36,7 @@ dependencies = [
     "cryptography>=50.0.2",  # Phase 1.5 skills_signer (ed25519)
     "docstring-to-markdown>=0.17",
     "fastapi>=0.142.2",
-    "fastmcp>=3.4.7",
+    "fastmcp>=4.0.0,<5",
     "hatchling>=1.32.4",
     "hypothesis>=6.168.3",
     "ipython>=9.17.1",
@@ -48,7 +48,12 @@ dependencies = [
     "mdformat>=1.0.0",
     "mdformat-ruff>=0.1.3",
     "numpy>=2.5.3",
-    "oneiric>=0.29.2",
+    # REQ-BIFROST-010 (bifrost plan §5 Phase 5, 2026-10-17): the
+    # FallbackChainCodeFixer consults the oneiric BifrostLLMRouter
+    # singleton (``oneiric.adapters.llm.bifrost.get_router()``) for
+    # the OPENAI-compatible provider's ``base_url``. The 0.32+ floor
+    # is the router module's home version.
+    "oneiric>=0.33",
     "pyleak>=0.2.0",
     "pyscn==1.5.0",
     "pydantic>=2.13.5",

{crackerjack-0.89.2 → crackerjack-0.91.0}/settings/crackerjack.yaml

@@ -92,6 +92,7 @@ documentation:
   auto_cleanup_on_publish: true
   dry_run_by_default: false
   backup_before_cleanup: true
+  use_shared_regenerator: false  # NEW: opt-in per sibling; flip when oneiric.plan_index is available
 
   essential_files:
     - AGENTS.md

crackerjack-0.91.0/tests/adapters/test_ai_registry.py

@@ -0,0 +1,191 @@
+"""Tests for AI adapter registry and provider info.
+
+Covers: crackerjack/adapters/ai/registry.py
+"""
+
+from __future__ import annotations
+
+import pytest
+
+from crackerjack.adapters.ai.registry import (
+    PROVIDER_INFO,
+    ProviderID,
+    ProviderInfo,
+    get_code_fixer,
+    get_provider_info,
+    list_providers,
+)
+
+
+class TestProviderID:
+    """Tests for ProviderID enum."""
+
+    def test_provider_id_values(self):
+        assert ProviderID.MINIMAX == "minimax"
+        assert ProviderID.LLAMA_SERVER == "llama_server"
+
+
+class TestProviderInfo:
+    """Tests for ProviderInfo dataclass."""
+
+    def test_provider_info_str(self):
+        info = PROVIDER_INFO[ProviderID.MINIMAX]
+        assert "MiniMax" in str(info)
+        assert "minimax" in str(info)
+
+    def test_provider_info_fields(self):
+        info = PROVIDER_INFO[ProviderID.MINIMAX]
+        assert info.id == ProviderID.MINIMAX
+        assert info.name == "MiniMax"
+        assert info.requires_api_key is True
+        assert info.default_model == "MiniMax-M2.7"
+        assert info.cost_tier == "medium"
+
+    def test_llama_server_info(self):
+        info = PROVIDER_INFO[ProviderID.LLAMA_SERVER]
+        assert info.requires_api_key is False
+        assert info.default_model == "qwen3.5"
+        assert info.cost_tier == "free"
+
+
+class TestListProviders:
+    """Tests for list_providers function."""
+
+    def test_returns_all_providers(self):
+        providers = list_providers()
+        assert len(providers) == 2
+
+    def test_returns_provider_info_objects(self):
+        providers = list_providers()
+        assert all(isinstance(p, ProviderInfo) for p in providers)
+
+    def test_contains_minimax(self):
+        providers = list_providers()
+        ids = [p.id for p in providers]
+        assert ProviderID.MINIMAX in ids
+
+
+class TestGetProviderInfo:
+    """Tests for get_provider_info function."""
+
+    def test_get_by_provider_id(self):
+        info = get_provider_info(ProviderID.MINIMAX)
+        assert info.id == ProviderID.MINIMAX
+
+    def test_get_by_string_lowercase(self):
+        info = get_provider_info("llama_server")
+        assert info.id == ProviderID.LLAMA_SERVER
+
+    def test_get_by_string_uppercase(self):
+        info = get_provider_info("LLAMA_SERVER")
+        assert info.id == ProviderID.LLAMA_SERVER
+
+    def test_get_unknown_provider_raises(self):
+        with pytest.raises(ValueError, match="Unknown provider"):
+            get_provider_info("unknown_provider")
+
+    def test_unknown_provider_error_message(self):
+        with pytest.raises(ValueError) as exc_info:
+            get_provider_info("claude")
+        assert "claude" in str(exc_info.value)
+        assert "Available" in str(exc_info.value)
+
+
+class TestGetCodeFixer:
+    """Tests for get_code_fixer function."""
+
+    def test_returns_code_fixer_instance(self):
+        fixer = get_code_fixer()
+        from crackerjack.adapters.ai.unified import FallbackChainCodeFixer
+
+        assert isinstance(fixer, FallbackChainCodeFixer)
+
+    def test_returns_same_instance_type(self):
+        fixer1 = get_code_fixer()
+        fixer2 = get_code_fixer()
+        # Both should be the same type
+        assert type(fixer1) == type(fixer2)
+
+
+class TestProviderInfoShape:
+    """Structural tests for the ``ProviderInfo`` shape.
+
+    Phase 5 (bifrost plan §5, generalist H-3) pinned the absence
+    of a ``uses_bifrost: bool`` decision field from
+    ``PROVIDER_INFO``. The field was already absent on main at
+    the time of the migration; the structural tests below are
+    regression guards so a future refactor that re-adds the
+    field (or any structurally-similar per-provider decision
+    key that duplicates the router's "is this provider routed
+    through the gateway?" query) trips a test failure.
+
+    Note: legitimate per-provider capability flags
+    (``uses_streaming``, ``uses_tool_calling``, ``uses_vision``,
+    etc.) are NOT pinned by these tests — only the specific
+    set of fields that would re-introduce the
+    structural-coupling bug is locked.
+    """
+
+    # The set of field names that would re-introduce the
+    # ``route_qwen_code: True while enabled: False`` bug shape:
+    # any per-provider boolean that duplicates the router's
+    # "is this provider routed through the gateway?" decision.
+    # Naming is intentional — a maintainer adding a field
+    # here must rename to ``supports_*`` / ``has_*`` / etc.
+    # to avoid the test trip, which surfaces the decision in
+    # code review.
+    _FORBIDDEN_BIFROST_DECISION_KEYS: frozenset[str] = frozenset(
+        {
+            "uses_bifrost",
+            "uses_claude_route",
+            "uses_qwen_route",
+            "uses_anthropic_route",
+            "uses_gateway",
+        }
+    )
+
+    def test_uses_bifrost_field_removed(self) -> None:
+        """REQ-BIFROST-010 evidence (bifrost plan line 573): the
+        ``uses_bifrost`` field is gone from every ``ProviderInfo``.
+        Operators no longer need to set it; the URL source lives
+        in the oneiric ``BifrostLLMRouter`` singleton.
+        """
+        import dataclasses
+
+        for provider_id, info in PROVIDER_INFO.items():
+            field_names = {f.name for f in dataclasses.fields(info)}
+            assert "uses_bifrost" not in field_names, (
+                f"ProviderInfo for {provider_id.value!r} unexpectedly has "
+                "a 'uses_bifrost' field; the bifrost plan (generalist "
+                "H-3) pinned its absence so the URL source stays in "
+                "the oneiric router, not per-provider flags."
+            )
+
+    def test_no_bifrost_decision_keys(self) -> None:
+        """Pin the absence of any per-provider decision field
+        that would re-introduce the structural coupling to the
+        bifrost gateway configuration. The URL source is the
+        oneiric router, not per-provider boolean flags.
+
+        Note: the deny-list is a *name set*, not a ``uses_*``
+        prefix ban — legitimate per-provider capability flags
+        (``uses_streaming``, ``uses_tool_calling``, etc.) are
+        not pinned by this test. The test guards specifically
+        against fields that would duplicate the router's
+        "is this provider routed through the gateway?" decision.
+        """
+        import dataclasses
+
+        for provider_id, info in PROVIDER_INFO.items():
+            for f in dataclasses.fields(info):
+                assert (
+                    f.name not in self._FORBIDDEN_BIFROST_DECISION_KEYS
+                ), (
+                    f"ProviderInfo for {provider_id.value!r} has a "
+                    f"forbidden decision key ({f.name!r}); the bifrost "
+                    "plan removed 'uses_bifrost' and the URL source "
+                    "moved to the oneiric router. Add a per-provider "
+                    "decision key only with a clear consumer (and a "
+                    "name outside the _FORBIDDEN_BIFROST_DECISION_KEYS "
+                    "set, so future maintainers can grep for it)."
+                )