T081, 16.4: "no model configured" is a state, shown before any turn (plan 034)

[brettheap]

Lane: openxfactory-4 (openXfactory-4-openDox_extraction)

Arc: neutral-product-standalone-operability

Phase 3. T063 has landed (openxFactory#1218 → a883bbf6), so phase 2 is closed. This PR was drafted ahead of that under Brett Heap's word of 2026-10-02 (openxFactory#656 comment 5960162524, "Draft all of them now (Recommended)"). The holder posts READY, and the landers merge. Claimed in #656 comment 5960438561 (P3-D: T085, then T081).

T085 has landed: openDox-code#71 → 2680eb5e. This PR was stacked on it because the served catalog route answers standalone only with T085's validators. The holder retargeted it to main. 9551f20d merges main at 7ff434d9, which carries #71, T088 (#65) and T071 (#60). The merge was clean, and against main this PR's diff is T081's seven files alone. Plan 034's T081 (specs/034-opendox-standalone-operation/tasks.md, read at openxFactory main 2140f5a7) realizes #1144's 16.4: "No model configured" is a STATE, shown before any turn.

Rulings

• R1Q22 (a), 5817152735.
• R1Q10 (a) and R1Q12 (a), 5850003126.
• The holder's answers to this writer, 2026-10-02:
  • Q2 (the tick rule). T085's falsifier holds whole only here: the served catalog route answers standalone with no available entry. Plan 034's T085 box is ticked at this PR's landing, with T081's.
  • Q3. doxbench_install answers openDox's no-model port when there is no approved binding and no harness. _workbench_model_port() answers that port as absent.
  • Model approval. It is not on the standalone path. T084 makes the approval route refuse by name standalone. This PR does NOT rely on the console's approval route. See "Binding states" below.
  • The hoist, RULED option (a). Commit 0a12dc58 stays. The holder's three conditions are met below: the console verdict stays first (file:line under "The turn route"), there is one ordering test, and the remaining window is named.
  • tests/test_model_provider_broker.py. The harness_present= probe is accepted as written. Whichever of this PR and T078, 16.1: the OpenAI-compatible dialect joins DIALECTS (plan 034) #61–Broker-path credential hardening (follows T080): a minted token keeps a built-in credential's rules (plan 034) #64 lands second merges main.
  • Landing plan, once T063 is in. T085, the standalone doxBench defaults for T027's seams, and T058's two workbench rules move into opendox.validator (plan 034) #71 and this PR land back to back. Before T085, the standalone doxBench defaults for T027's seams, and T058's two workbench rules move into opendox.validator (plan 034) #71 lands, the holder retargets this PR to main, because the lander's --delete-branch would close a PR stacked on T085, the standalone doxBench defaults for T027's seams, and T058's two workbench rules move into opendox.validator (plan 034) #71's branch.

What changes

• src/opendox/doxbench_model.py:

  • NoModelConfigured, with one instance, NO_MODEL_CONFIGURED. It sits beside EMPTY_CATALOG and the WorkbenchModelPort protocol it implements. Its catalog() is EMPTY_CATALOG, and dispatch() raises NoModelConfiguredError without spawning or contacting anything.
  • NO_MODEL_CONFIGURED_REMEDY says how to configure a model.

• src/opendox/doxbench_install.py:

  • harness_installed(): is doxbench_bridge.HARNESS_COMMAND (omp) on the PATH, the precondition F16.1 itself checks with command -v omp. It spawns nothing.
  • no_model_port_factory().
  • declared_model_port_factory(..., harness_present=None). With no approved binding, it answers the harness bridge where the harness is installed (the harness route stays), and the no-model port where it is not. The probe defaults to harness_installed.
  • The module docstring's "the unconfigured posture is unchanged byte for byte" is now false, so it states the new rule. The unreadable-document notice no longer claims a harness fallback.

• src/opendox/serve_workbench.py:

  • _workbench_model_port() answers NO_MODEL_CONFIGURED, by identity, as no port. So GET /workbench/model-catalog serves the editor-only posture, and a turn or an abstract that reaches its model step is refused model_capability_unavailable. This is the holder's Q3 edit.
  • The hoist in 0a12dc58, described below.

• src/opendox/web/views/doxbench-chat.js: a new VISIBLE line, doxchat-no-model, whose text is NO_MODEL_CONFIGURED_REMEDY:

  No model configured. To configure one, declare a model binding with "opendox model-binding add" ("--help" lists its fields), or put the local harness "omp" on PATH, then restart this console.

  It shows only once the catalog has ANSWERED empty, with no catalog failure recorded and no intake option rendered.

  • Empty, not "nothing available" (Copilot, round 1). A configured model can be unavailable: after a broker refusal, BrokeredProviderPort.catalog() keeps the binding with available: false. That operator has a model configured, so the line stays hidden, and the send button's configured-none sentence still states the posture. The server's no-model port serves exactly the empty catalog.
  • Announced once (Copilot, round 1). The catalog settles asynchronously with no focus change, so render() also writes the remedy to the rail's polite announce region. It does so only when the text changes, behind the same guard the context and retry notes keep, so a keystroke does not re-announce it.
  • Disclosed limit. Where the shell's intake-surface read settles AFTER the catalog, the line shows and is announced once, and is then hidden when the offer arrives. The two reads are independent fetches, and the rail cannot know whether a shell will ever push an offer. Where the offer arrives first, nothing is shown or announced.
  • The send button's configured-none sentence is unchanged, byte for byte. The ratified ideation-dashboard requirement keeps it: add-doxchat-model-intake §1, "the rail's existing sentence continues to state that no approved model is configured", whose scenario reads "the existing no-approved-model sentence unchanged".
  • What that sentence never named is HOW to configure a model (plan 034's research R15). So this is a separate line, not a rewrite.
  • Where the intake flow is offered, the selector's first option is the remedy's home, and the line stays hidden, since the intake requirement refuses a "second, weaker statement".
  • Its stated limit (RULED by the holder, 2026-10-03, option (a), on Copilot's r4170956940):
    • The remedy names openDox's own ways to configure a model. Every openDox entry point declares doxbench_install.declared_model_port_factory, whose empty catalog means exactly no binding and no harness.
    • An embedder that injects its own port supplies its own intake offer, and where intake is offered the line is hidden.
    • The server cannot carry a model posture within the released contract. xfactory-workbench-model-catalog is closed (additionalProperties: false), and #1144's 16.5 (T082) holds /capabilities and the intake surface equal with and without a model.
    • The comment at the check in noModelConfiguredRemedy says so.
  • The web census moves views/doxbench-chat.js loc from 1828 to 1890, and the class-A total from 18073 to 18135.

• tests/test_chat_model_configuration.py (new, 49 cases):

  • F16.1's catalog block, word for word;
  • each binding state;
  • the harness route, and the default probe;
  • the port and the accessor;
  • the served catalog standalone, with no available entry;
  • the standalone turn, refused console_required without the console token, then model_capability_unavailable with it;
  • the turn route's order, 30 cases (10 defects by 3 postures), described under "The turn route";
  • the rail, run over node, in each state, plus the pure verdict, and the announce region's writes;
  • the remedy's two spellings, held together.

• tests/test_model_provider_broker.py: three cases pinned the harness fallback. Each now passes harness_present=lambda: True and keeps its assertion:

  • the yaml-missing case (:322-386);
  • test_a_checkout_with_no_bindings_resolves_exactly_the_harness_declaration (:1159);
  • test_an_unreadable_bindings_document_falls_back_and_says_so (:1187).
  • This file is also edited by T078, 16.1: the OpenAI-compatible dialect joins DIALECTS (plan 034) #61–Broker-path credential hardening (follows T080): a minted token keeps a built-in credential's rules (plan 034) #64. Their hunks sit around the fixed-diagnostics test (:1111) and the helpers, not at these lines. The holder accepted the probe as written: whichever of T081 and T078, 16.1: the OpenAI-compatible dialect joins DIALECTS (plan 034) #61–Broker-path credential hardening (follows T080): a minted token keeps a built-in credential's rules (plan 034) #64 lands second merges main. doxbench_binding.py and doxbench_provider.py are untouched here.

Binding states the port reads (as the holder asked, stated and not changed)

declared_model_port_factory, at doxbench_install.py:324-327 at 047bb4fa, reads the bindings document and the intake declarations document:

• No declaration, from the CLI or a hand edit: present. opendox model-binding add writes the bindings document alone, and never the declarations document, so such a binding is never pending. That is how a standalone user configures a model with no console approval route.
• A declaration approved: present. It is unreachable standalone, because approval goes through the console route, which T084 makes refuse by name standalone.
• A declaration pending: suppressed. It contributes no available entry, and stderr says so.
• An unreadable bindings document: read as declaring none.
• With no binding present: the harness bridge if omp is on the PATH, else the no-model port. This is what changes here.
• Pre-existing, and not changed here:
  • the port uses only the FIRST approved binding (approved[0], :327);
  • a declaration's expires_at is not enforced;
  • every binding record still needs broker_argv until T080's auth kind none and the built-in resolver.

The turn route (0a12dc58, RULED option (a))

Measured with a schema-valid v2 turn (openDox-spec's workbench-chat-turn-v2-loaded-set example), from a standalone generate-and-open child, siblings refused, console token presented:

• main 047bb4fa: 403 model_capability_unavailable. That is an accident: no validators were registered, so the validators refusal, hoisted above the kind check, answered.
• T085's head (T085, the standalone doxBench defaults for T027's seams, and T058's two workbench rules move into opendox.validator (plan 034) #71 at e0298cf4): the connection drops. T085's validators answer, so the turn reaches step 5's from openxdox import doxbench_scope (serve_workbench.py:1669 at T085, the standalone doxBench defaults for T027's seams, and T058's two workbench rules move into opendox.validator (plan 034) #71's head, T084's :1665 reach at main), and the child logs ModuleNotFoundError: No module named 'openxdox'.
• This PR's first commit alone does not change that. Step 7's model check comes after step 5.
• 0a12dc58 runs step 7's no-port refusal (the same _refuse_turn call, code and envelope) just above step 5. A plane with no model configured, or no model factory, refuses a well-formed turn before anything is spawned or contacted. Idempotency (step 8) stays after the model verdict, as it was.

What still comes first, at 9061b22a (serve_workbench.py, _handle_workbench_chat_turn at :1531). Everything that authenticates the request precedes the hoisted refusal, so an unauthenticated caller learns nothing about model availability:

1. :1536, the plane: loopback, the session capability, and a resolved actor. This is pre-existing. It answers model_capability_unavailable in the fixed pre-identity shape whatever the model posture, so it reveals nothing about the posture.
2. :1542, the console verdict, _not_the_human_console(): the console token, the Host, the JSON or token-bearing submission, and the origin. It answers console_required.
3. :1550, the body bound, then the not-an-object refusal.
4. :1607, the validators; :1623, the kind; :1630, the schema; :1643, the parse.
5. :1690, the hoisted model verdict.
6. :1698, step 5's scope import, then step 6 (:1791), step 7 (:1855) and step 8 (:1940).

The ordering test is test_the_turn_routes_order_with_and_without_a_port. It drives _handle_workbench_chat_turn itself, over openDox's real validators (T085) and real identity checks. Only the HTTP plumbing (the console verdict, the bounded body read, the reply) and openxdox's scope module, which openDox's suite does not install, are stood in. It has 10 defects across 3 postures (a port; no model configured; no factory):

• A console, body-bound, not-an-object, kind, schema or parse defect answers first in every posture.
• With no port, the no-model refusal answers before a scope, identity or limits defect. The limits defect is a document of 400 001 bytes, within the schema's 1 MiB maxLength. The scope registry is never read.
• With a port, every defect answers what it answered before the hoist: turn_scope_refused, content_identity_mismatch, request_limit_exceeded. A well-formed turn reaches step 7 and is refused model_unavailable by a port whose catalog offers nothing.

The remaining window. With a binding configured (or omp on the PATH), a standalone turn still drops at step 5's import, serve_workbench.py:1698 at 9061b22a (:1669 at #71's head 83213eb2), until T084 routes it (:1665 at main). The holder has told T084's writer to cover the turn route in its test. The hoist sits just above that import, so whichever of this PR and T084 lands second merges main.

Falsifier, failing before and passing after

T081's falsifier is F16.1's catalog block and tests/test_chat_model_configuration.py.

F16.1's 16.4 block, in a fresh venv (python3 -m venv --clear "$W/v16", then pip install ".[test]"). It first asserts that neither sibling imports and that no omp is on the PATH, and then runs the block verbatim:

• at T085's head 83213eb2: AssertionError: no model is configured, yet the catalog offers ['omp-local'];
• at 9061b22a: no model configured: the catalog offers nothing.

The named test was then run in the same venv: 47 passed at 9061b22a (16 passed at round 1's 38a4bba9).

tests/test_chat_model_configuration.py, run over T085's head 83213eb2 (worktree, PYTHONPATH=<worktree>/src, LANG=C.UTF-8): 13 failed, 4 errors in 1.38s. Among them:

E   AssertionError: no model is configured, yet the catalog offers ['omp-local']
E   AssertionError: assert [{'available'... model', ...}] == []      (the served catalog)
E   http.client.RemoteDisconnected: Remote end closed connection without response   (the turn)
E   AssertionError: ['openxdox']                                     (the turn's refused import)

The ordering test over ef0c5546 (this PR's first commit, before the hoist; same worktree method): 8 of its 30 cases fail, exactly the no-port postures for the scope, identity, limits and well-formed turns. The standalone turn case fails too:

FAILED ...test_the_turn_routes_order_with_and_without_a_port[no model configured-identity]
FAILED ...test_the_turn_routes_order_with_and_without_a_port[no model configured-limits]
FAILED ...test_the_turn_routes_order_with_and_without_a_port[no model configured-none]
FAILED ...test_the_turn_routes_order_with_and_without_a_port[no model configured-scope]
FAILED ...test_the_turn_routes_order_with_and_without_a_port[no factory-identity]
FAILED ...test_the_turn_routes_order_with_and_without_a_port[no factory-limits]
FAILED ...test_the_turn_routes_order_with_and_without_a_port[no factory-none]
FAILED ...test_the_turn_routes_order_with_and_without_a_port[no factory-scope]
FAILED ...test_a_turn_is_refused_model_capability_unavailable
9 failed, 22 passed, 1 error in 1.04s

The round-2 rail cases over 38a4bba9's rail (the file reverted, then restored with its digest checked): 3 failed, 14 passed. The unavailable-only rail showed the remedy, the region was never written (assert 0 == 1), and the pure verdict answered the remedy for [OFF].

At 9061b22a: 47 passed.

Mutants

Applied one at a time by a harness that restores each file and checks its digest. 24 of 24 are killed (runs/mutants-t081-r4.txt), plus P01 to P04 of the test helper (runs/mutants-t081-r3.txt):

• N01: the declaration ignores the harness probe.
• N02, N03: harness_installed() always answers True, or always False.
• N04: the accessor hands the no-model port back.
• N05: the accessor treats any NoModelConfigured as absence (imitation).
• N06: the no-model catalog offers the harness entry.
• N07: dispatch answers instead of refusing.
• N08, N09, N10, N16: the rail shows the remedy where intake is offered, on a recorded catalog failure, while loading, or with a non-empty catalog.
• N11: the rail never shows it.
• N12: the JS spelling drifts from the Python twin.
• N13: the rail replaces the configured-none sentence.
• N14, N20: the turn's model verdict is not hoisted. N20 is killed by the ordering test alone.
• N15: a pending declaration is not suppressed.
• N17: the remedy shows for a catalog of unavailable models (round 1's check).
• N18: the remedy is not announced.
• N19: the remedy is announced on every render, with no change guard.
• N21: the hoist moves above the kind, schema and parse checks.
• N22: the hoist moves above the console verdict.
• N23: the hoist refuses with a port configured too.
• N24: step 7 resolves the model port a second time.
• P01 to P04: the test PATH helper drops a directory holding omp whole, keeps omp in the mirror, moves the mirror out of the directory's place, or links to relative targets.

N09 first SURVIVED the mounted cases alone, because a mount that fails its catalog never adopts one. 38a4bba9 adds a probe of the exported verdict over a failure recorded beside an adopted empty catalog, and that kills it.

The whole suite

All runs were local, in the foreground, with LANG=C.UTF-8 and no PostgreSQL service, so the runtime cases skip here.

• T085's head: 2924 passed, 177 skipped.
• Round 1, 38a4bba9: 2940 passed, 177 skipped. Its inventory differs from T085's only by the 16 new cases.
• 9061b22a: 2971 passed, 177 skipped. Against round 1, 32 cases are added (the 30 ordering cases, the unavailable-is-not-no-model case and the announce case) and 1 is removed (the round-1 unavailable case, renamed and flipped). No other case changes status.

The skip sets are equal node for node.

Review rounds

• Copilot at 38a4bba9, two findings, both fixed in 9061b22a, answered, and resolved:

  • r4169715519: limit the remedy to an empty catalog;
  • r4169715599: announce it through the polite region, behind a change guard.

  Copilot was re-requested.

• CI at 38a4bba9: validate green, triple: selected=3117 passed=3106 skipped=11. SonarCloud passed.

• Copilot at 9061b22a, one finding (r4169897199). The description described round 1: 16 cases and the round-1 census. Its review began before this description was refreshed. Every count is now labelled with the revision it was measured at.

• CI at 9061b22a (run 37065083051): validate green, triple: selected=3148 passed=3137 skipped=11 failures=0 errors=0. That is 31 more selected than 38a4bba9, the net of this round's cases. SonarCloud passed.

• 9551f20d, the merge of main 7ff434d9 after T085, the standalone doxBench defaults for T027's seams, and T058's two workbench rules move into opendox.validator (plan 034) #71 landed: no conflict, and the whole suite, run locally, gives 2998 passed, 177 skipped. CI (run 37079669644): validate green, triple: selected=3175 passed=3164 skipped=11 failures=0 errors=0.

• Copilot at 9551f20d, one finding (r4170794383), fixed in 4ef7575a, answered and resolved. The tests' _no_omp_path() dropped a whole PATH directory that held omp, and with it any git beside the harness. It now mirrors that directory without omp, in place. test_a_path_without_omp_keeps_every_other_command holds that, and three mutants of it are killed. CI at 4ef7575a (run 37080287448): selected=3197 passed=3186 skipped=11.

• Merges of main as the stack landed, each clean:

  • f6e777ef merges T078 (T078, 16.1: the OpenAI-compatible dialect joins DIALECTS (plan 034) #61). Its CI run, 37080863326, ran against a main that already carried T070 (T070, 13.4-13.6: OPENDOX_INSTALL_MODE and generate-and-open --local (plan 034) #67), so its two served-route cases errored on the hosted-mode refusal. This was fixed in the next commit.
  • 8104fa6e merges T070 (T070, 13.4-13.6: OPENDOX_INSTALL_MODE and generate-and-open --local (plan 034) #67). The standalone fixture now runs generate-and-open --local, as T085, the standalone doxBench defaults for T027's seams, and T058's two workbench rules move into opendox.validator (plan 034) #71's case on main does. CI run 37081223225: selected=3254 passed=3243 skipped=11.
  • fd7cccd7 merges T079 (T079, 16.2: a model field the provider receives (plan 034) #62). CI run 37082713077: selected=3273 passed=3262 skipped=11.
  • Locally, the whole suite on fd7cccd7 gives 3095 passed, 178 skipped.

• Copilot at 8104fa6e, one finding (r4170839174), fixed in b930af49: the PATH mirror now links to absolute targets, so a relative PATH entry keeps its commands. test_a_relative_path_entry_without_omp_keeps_its_commands holds this, and mutant P04 is killed. CI run 37081588647: green.

• Copilot at 8104fa6e, a second finding (r4170882125), fixed in 7b1ecbe2: the model port is resolved once per turn. The hoisted verdict binds port, and step 7 reads it. The ordering test counts the factory's calls in all 30 cases: once where the turn reaches the model verdict, and never otherwise. Mutant N24, step 7 resolving the port again, is killed, so 24 of 24 are killed. CI run 37082406902: green.

• Copilot at fd7cccd7, one finding (r4170956940): an empty catalog is not unique to openDox's no-model port. RULED (a) by the holder on 2026-10-03, so the scope is stated, not changed. 407f6ecd adds the comment and the census move, the thread is answered with the closed-envelope facts and resolved, and Copilot was re-requested. Locally, the whole suite gives 3095 passed, 178 skipped. CI at 407f6ecd (run 37084541090): validate green, triple: selected=3273 passed=3262 skipped=11 failures=0 errors=0. Copilot at 407f6ecd: "Approval recommended", with 0 unresolved threads.

Not in this PR

• 16.5, every other surface working with no model: T082.
• The configured turn over a stand-in OpenAI-compatible server: T078–T080, with F16.1 run whole at T083.
• The standalone scope and approval reaches: T084.

🤖 Generated with Claude Code

Summary by Sourcery

Make “no model configured” an explicit standalone state that is visible before any turn and returns a safe capability-unavailable response.

New Features:

• Represent an unconfigured model as an explicit state with an empty catalog and actionable configuration guidance in the chat rail.

Bug Fixes:

• Prevent standalone turns from reaching scope or provider processing when no model is configured, returning the expected capability-unavailable response without spawning or contacting external processes.
• Select the harness provider only when the harness executable is actually available, while preserving the harness route for installed environments.

Enhancements:

• Expose the no-model state consistently across model resolution, served catalog access, turn handling, and the chat interface, including one-time accessible announcements.
• Preserve existing model-binding, pending/approved declaration, and configured-but-unavailable behavior while distinguishing it from having no model configured.

Tests:

• Add comprehensive coverage for no-model binding states, standalone catalog and turn behavior, request-validation ordering, and chat-rail visibility and announcements.
• Update broker tests and web boundary census expectations for conditional harness fallback and the new chat guidance.

[sourcery-ai]

Reviewer's Guide

Implements plan 034 T081 by representing “no model configured” as an explicit singleton state, selecting it when neither an approved binding nor the local harness is available, serving an empty catalog and early capability refusal, and displaying an actionable chat-rail remedy before any turn while preserving existing configured-model and harness behavior.

Sequence diagram for no-model catalog and turn handling

sequenceDiagram
    participant Browser
    participant Workbench as serve_workbench
    participant Factory as declared_model_port_factory
    participant NoModel as NO_MODEL_CONFIGURED

    Browser->>Workbench: GET /workbench/model-catalog
    Workbench->>Factory: _workbench_model_port()
    Factory-->>Workbench: NO_MODEL_CONFIGURED
    Workbench-->>Browser: Empty catalog
    Browser->>Workbench: Chat turn
    Workbench->>Workbench: _workbench_model_port()
    Workbench-->>Browser: model_capability_unavailable

Loading

State diagram for chat-rail no-model remedy

stateDiagram-v2
    [*] --> Loading
    Loading --> CatalogFailure: catalog failure
    Loading --> CatalogAnswered: catalog answered
    CatalogAnswered --> ModelAvailable: available entry exists
    CatalogAnswered --> IntakeOffered: intake offered
    CatalogAnswered --> NoModelConfigured: no available entry
    NoModelConfigured --> RemedyVisible: noModelConfiguredRemedy
    CatalogFailure --> [*]
    ModelAvailable --> [*]
    IntakeOffered --> [*]
    RemedyVisible --> [*]

Loading

File-Level Changes

Change	Details	Files
Introduces an explicit no-model capability state and selects it when no approved binding exists and the local harness is unavailable.	Adds the singleton no-model port, empty catalog, refusal error, and configuration remedy. Probes PATH for the harness and preserves the harness fallback when it is installed. Treats pending bindings as unavailable and unreadable binding documents as declaring no binding, while retaining existing broker behavior for present bindings.	src/opendox/doxbench_model.py src/opendox/doxbench_install.py tests/test_model_provider_broker.py
Makes standalone serving and turn handling reflect model absence before any model interaction.	Maps the no-model singleton to capability absence for catalog and other model-dependent routes. Hoists the model-capability refusal ahead of scope loading and process or endpoint interaction for valid turns. Adds coverage for standalone catalog and turn responses.	src/opendox/serve_workbench.py tests/test_chat_model_configuration.py
Exposes a visible, actionable no-model message in the chat rail without changing the existing configured-none sentence.	Adds a Python/JavaScript-matched remedy explaining CLI binding and harness configuration. Shows the remedy only after an empty successful catalog with no intake flow, and hides it during loading, failures, or when a model is available. Updates the web boundary census for the added UI code.	src/opendox/web/views/doxbench-chat.js tests/test_chat_model_configuration.py tests/fixtures/web_boundary_census.yaml
Adds comprehensive regression and mutation-oriented coverage for model configuration states and UI behavior.	Covers binding, harness, port, accessor, served-route, turn, rail, and exact-text behavior. Pins existing harness fallback tests to an explicit harness-present condition. Reports the full-suite increase and complete mutant-kill result as validation points for review.	tests/test_chat_model_configuration.py tests/test_model_provider_broker.py

---

Tips and commands

Interacting with Sourcery

• Trigger a new review: Comment @sourcery-ai review on the pull request.
• Continue discussions: Reply directly to Sourcery's review comments.
• Generate a GitHub issue from a review comment: Ask Sourcery to create an
  issue from a review comment by replying to it. You can also reply to a
  review comment with @sourcery-ai issue to create an issue from it.
• Generate a pull request title: Write @sourcery-ai anywhere in the pull
  request title to generate a title at any time. You can also comment
  @sourcery-ai title on the pull request to (re-)generate the title at any time.
• Generate a pull request summary: Write @sourcery-ai summary anywhere in
  the pull request body to generate a PR summary at any time exactly where you
  want it. You can also comment @sourcery-ai summary on the pull request to
  (re-)generate the summary at any time.
• Generate reviewer's guide: Comment @sourcery-ai guide on the pull
  request to (re-)generate the reviewer's guide at any time.
• Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
  pull request to resolve all Sourcery comments. Useful if you've already
  addressed all the comments and don't want to see them anymore.
• Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
  request to dismiss all existing Sourcery reviews. Especially useful if you
  want to start fresh with a new review - don't forget to comment
  @sourcery-ai review to trigger a new review!

Customizing Your Experience

Access your dashboard to:

• Enable or disable review features such as the Sourcery-generated pull request
  summary, the reviewer's guide, and others.
• Change the review language.
• Add, remove or edit custom review instructions.
• Adjust other review settings.

Getting Help

• Contact our support team for questions or feedback.
• Visit our documentation for detailed guides and information.
• Keep in touch with the Sourcery team by following us on X/Twitter, LinkedIn or GitHub.

[Copilot]

Copilot review overview

🔵 Needs a closer look

Unresolved UI issues and the pending ruling on turn-refusal precedence require human review before approval.

Review effort: Balanced
Findings: 2

Open (2)

• Unavailable model is treated as missing configuration · New
• No-model remedy is not announced to screen readers · New

What changed in this PR

Implements T081’s explicit “no model configured” state for standalone openDox, building on T085.

Changes:

• Adds an empty model port and checks harness availability.
• Shows configuration guidance and refuses model-less turns before scope loading.
• Adds regression coverage and updates the web census.

File	Description
tests/​test_model_provider_broker.py	Makes harness assumptions explicit.
tests/​test_chat_model_configuration.py	Covers configuration states, routes, and chat guidance.
tests/​fixtures/​web_boundary_census.yaml	Updates chat-view line counts.
src/​opendox/​web/​views/​doxbench-chat.js	Displays no-model configuration guidance.
src/​opendox/​serve_workbench.py	Recognizes model absence and refuses turns earlier.
src/​opendox/​doxbench_model.py	Defines the no-model port and remedy.
src/​opendox/​doxbench_install.py	Selects the harness only when installed.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Copilot]

Copilot review overview

🔵 Needs a closer look

This stacked draft’s prerequisite landings and turn-refusal ordering still require human confirmation.

Review effort: Balanced
Findings: 1

Open (1)

• Refresh PR description with current test counts and metrics · New

Resolved since last review (2)

• No-model remedy is not announced to screen readers
• Unavailable model is treated as missing configuration

[Copilot]

Copilot review overview

🔵 Needs a closer look

The unresolved test setup issue and coordinated T085/T084 route integration require correction and human review.

Review effort: Balanced
Findings: 1

Open (1)

• Removing PATH directories containing omp also removes git · New

Resolved since last review (1)

• Refresh PR description with current test counts and metrics

[Copilot]

Copilot review overview

🟢 Approval recommended

The implementation and regression coverage are consistent, with no unresolved blocking issues.

Review effort: Balanced
Findings: None

Resolved since last review (1)

• Avoid showing default-install remedy for configured empty catalogs

[brettheap]

READY at 407f6ec — phase 3 is open (T063 landed, #1218 → a883bbf6); T085 (#71) landed as 2680eb5; Brett: draft phase 3 ahead, land in plan order when green. The holder checked: validate and SonarCloud green, Copilot recommends approval with no findings, 0 unresolved threads; dependent #76 retargeted to main and not READY. T085's box ticks at this landing (ruled). Lane: openxfactory-4 (openXfactory-4-openDox_extraction)

[sourcery-ai]

Sorry @brettheap, you've used your own review budget of 250,000 diff characters for the last 7 days.

You can request another review in 20 hours and 33 minutes by commenting @sourcery-ai review. Upgrade to get a review now.

[brettheap]

Lane: openxfactory-4 (openXfactory-4-openDox_extraction)

LANDED — lane openxfactory-4, 2026-10-03T01:08:30Z, PR #74 → 9a49040 (opensoft/openDox-code main; plain gate)

Brett: land phase 1 / phase 2 PRs when green