alibaba/open-code-review

Description ocr review connects to configured MCP servers, registers their tools alongside the built-in tools, and exposes their definitions to the review agent. ocr scan loads the same user configuration but currently only passes its built-in tool definitions to the full-file review agent. Consequently, configured MCP tools cannot be used while scanning files. Make configured MCP tools available to the main task of ocr scan, with the same server configuration, allowlist, name-conflict handling, connection-failure behavior, and bounded shutdown as ocr review. Reuse the existing MCP client registration and definition-collection path rather than introducing a scan-specific MCP implementation. Keep file_read_diff hidden in scan mode; enabling MCP must not add a diff where none exists. The scan pre-plan remains tool-free. Expected behavior: with a configured MCP server, ocr scan --path <file> can invoke an allowed external tool for context while reviewing that file. Without a configured se

Summary #1426 closed #1329, but three items from its scope did not actually land, and one more needs a decision. Line references are against main at a003b934. Scope 1. The in-table focus ring still draws outside the scroll container .table a:focus-visible got outline-offset: -2px so the ring is not clipped by .table-scroll (whose overflow-y computes to auto), but three existing rules are more specific and keep +2px: .sessions-page .session-id:focus-visible — style.css:1536, (0,2,0) .repos-page .col-repository a:focus-visible — style.css:1803, (0,2,1) .repos-page .repo-check:focus-visible — style.css:1819, (0,2,0) That is every link in the repos table plus the session id, so the new rule only takes effect on the sessions Compare link. Fix: set -2px on those three, or fold them into the inset group placed after style.css:1819. Give the three rules the inset offset Assert the effective offset, not just that the selector exists 2. A long CWD or branch is still unreadable at nar

Part of the viewer UI redesign (see #1319). Depends on #1320, #1321. Summary Restyle the lower portion of the session detail page: the Files Reviewed list, the Session Tasks cards, and the Conversations transcript (per-request tool calls). Design reference Files Reviewed (flat list of paths), Session Tasks (grouped task cards — e.g. GROUPING_TASK with request/model/token badges), and Conversations (collapsible per-file groups; each PLAN_TASK / MAIN_TASK request shows model + token badges and expandable TOOL CALLS with arguments/results). Current state Template: internal/viewer/templates/session.html — Files Reviewed list, task cards, and conversation blocks. Helpers in server.go: orderedTasks, taskTypeClass, sessionTaskLabel, groupingView, cardCount (task ordering: Plan → Main → ReLocation → MemoryCompression → Grouping). Script: internal/viewer/static/session.js for collapsibles. Scope Restyle the Files Reviewed list to the mockup. Restyle task cards (type label, r

Part of the viewer UI redesign (see #1319). Depends on #1320, #1321. Summary Restyle the top of the session detail page (GET /r/{repo}/{sessionID}): the title and the metadata bar. Design reference Header shows Session: <id> with a back chevron, then a compact meta row: CWD · BRANCH · MODE · FROM · TO · MODEL · DURATION · FILES · STATUS. Current state Template: internal/viewer/templates/session.html (title at ~line 16; meta fields follow) Handler: handleSession in internal/viewer/handler.go Scope Restyle the title row with the back chevron. Lay out the meta fields as a compact, labeled row/grid matching the mockup, including truncation for long CWD/branch values. Apply status styling (complete/partial/failed) consistent with tokens from #1320. Out of scope Coverage/Token cards (#1325), comments (#1326), tasks/conversations (#1327). Acceptance criteria Header + meta bar match the mockup in light and dark. make check and make test pass. Parallelization & conflicts D

Part of the viewer UI redesign (see #1319). Depends on #1320, #1321. Summary Restyle the per-repo Sessions list (GET /r/{repo}) to match the mockup. Design reference A back chevron + title "Sessions: ", and a table with columns Session ID | Branch | Mode | Model | Files | Status | Comments | Duration | Started At | Action (green Check), with bottom-right pagination. Current state Template: internal/viewer/templates/sessions.html Handler: handleSessions in internal/viewer/handler.go Back navigation exists (added in PR #1217). Scope Restyle the title row with the back-navigation chevron (use the icon from #1321). Match the table columns, alignment, truncation (Session ID) and status styling. Style Check as the accent action; align pagination with the mockup. Keep column semantics identical to current data. Out of scope Session detail screen (#1324–#1327). Acceptance criteria Matches the mockup in light and dark; back nav and pagination work. make check and make test pass.