Skip to content

refactor(adapters): centralize runtime adapter authority - Part 1 - #1721

Merged
lidge-jun merged 120 commits into
devfrom
refactor/adapter-registry-authority
Aug 15, 2026
Merged

lidge-jun merged 120 commits into
devfrom
refactor/adapter-registry-authority

Conversation

@Wibias

@Wibias Wibias commented Aug 15, 2026

Copy link
Copy Markdown
Contributor

Stack

  1. This PR: adapter registry + runtime construction authority with behavior parity
  2. test(adapters): derive routed tool conformance from registry - Part 2 #1722 registry-derived routed-tool conformance
  3. test(adapters): cover buffered freeform restoration - Part 3 #1723 buffered parser conformance

Scope

Fresh reimplementation of the architecture slice from #1623 on current dev.

  • one authoritative runtime adapter registry
  • resolveAdapter() constructs through that registry
  • Azure / Azure OpenAI declare semantic inheritance from OpenAI Responses without constructor rewrites
  • MiMo Free declares semantic inheritance from OpenAI Chat without constructor rewrites
  • Cursor remains a direct contract because its runTurn transport and gated native-file fallback are distinct
  • persisted adapter ids remain runtime-validated strings
  • wrapper-cycle, runtime-validation, and extension policy documented in structure/10_adapter-registry.md

Deliberately not included

Verification

The registry contract was added test-first. The red run failed only because src/adapters/registry.ts did not yet exist. On the implementation tree, TypeScript/gates and all four Linux test shards passed before the branch was squashed to one commit.

The PR remains draft while exact-head post-squash CI finishes.

Summary by CodeRabbit

  • New Features

    • Added centralized adapter registration, inheritance, discovery, and creation.
    • Added privacy-safe public evidence preview, export, verification, import, listing, and revocation workflows.
    • Added Codex app-server status and restart controls, including stale-model notifications.
    • Added custom-model reasoning-effort configuration across the CLI, API, and GUI.
    • Added community evidence summaries to the compatibility view.
    • Added static model catalogs and improved provider discovery controls.
  • Bug Fixes

    • Improved schema normalization, model metadata, Windows process handling, and transport timeout reporting.
  • Documentation

    • Documented adapter registration and public evidence workflows.

Wibias added 30 commits August 14, 2026 21:18
Rebased onto current dev with the reviewed public evidence trust core, consumer durability recovery, sparse-array JCS hardening, and required Windows publisher-key ACL hardening.
Rebase the reviewed CL-10 operator/community layer onto the current public-evidence core. This squashes the child history onto cl10-public-core while preserving the exact conflict-free GitHub merge tree, including the final review fixes.
The CLI parity sweep reads GUI sources for /api paths, so naming the catalog
route inside a comment made it look like an endpoint the GUI calls with no CLI
mirror. The comment only needed to explain why the new value is appServerState
rather than catalogState; it did not need to spell the route.
Resolve purge.ts: re-express #1706 export-purge deferral inside #1709 withLedgerMutation
Resolve i18n-allowlist: take #1714 case-insensitive HTTP match (superset of #1715)
@github-actions

github-actions Bot commented Aug 15, 2026

Copy link
Copy Markdown
Contributor

✅ READY

  • all PR quality gates passed.

UI screenshot waived by the gui-screenshot-waived label.

@github-actions
github-actions Bot marked this pull request as draft August 15, 2026 05:29
@Wibias Wibias added the gui-screenshot-waived Maintainer waiver for false-positive GUI screenshot requirements label Aug 15, 2026
@lidge-jun
lidge-jun merged commit 88463de into dev Aug 15, 2026
51 of 54 checks passed
@github-actions
github-actions Bot marked this pull request as ready for review August 15, 2026 05:32
@lidge-jun
lidge-jun deleted the refactor/adapter-registry-authority branch August 15, 2026 05:35
agentHits pushed a commit to agentHits/opencodex that referenced this pull request Sep 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Maintenance, CI, tests, refactors, or build changes (not a user-facing bug or feature). gui-screenshot-waived Maintainer waiver for false-positive GUI screenshot requirements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants