Skip to content

fix(design-code): repoint :18745 legs to antigravity-direct :18751 (#1413, unblocks opencode#1409) - #4626

Merged
dezren39 merged 1 commit into
mainfrom
fix/1413-design-code-repoint-18751
Nov 2, 2026
Merged

dezren39 merged 1 commit into
mainfrom
fix/1413-design-code-repoint-18751

Conversation

@dezren39

@dezren39 dezren39 commented Nov 2, 2026 •

Copy link
Copy Markdown
Member

Part of developing-today-forks/opencode#1413 (design-code door wave, Wave C); clears the blocker recorded on developing-today-forks/opencode#1409 (comment 6590071718).

Change

  • ANTIGRAVITY_LS_PORT / ANTIGRAVITY_LS_PROXY_PORT (default 18745, the retired antigravity-ls door) → ANTIGRAVITY_DIRECT_PORT (default 18751, antigravity-direct). Every MODELS recipe primary/fallback leg and the gpt/codex keyword-fallback leg follows — the backend model ids are unchanged (gemini-3.8-flash, claude-sonnet-5-5), and antigravity-direct serves both (verified live against http://127.0.0.1:18751/v1/models before the change).
  • FLEET_BACKEND_PORTS: [18744, 19744, 18745, 19745] → [18744, 19744, 18751, 19751], so live backend-catalog validation probes the direct door instead of the dead port.
  • Test harness env var renamed to match.

Verification (laptop amd-6, local — CI-on-main policy)

  • node --test bin/design-code-proxy.test.mjs — 16/16 pass.
  • Live, repointed proxy on a scratch port against the real backends: design-code-flash (non-stream) and design-code-pro (stream, [DONE]-terminated, usage present) dispatched to 127.0.0.1:18751 (gemini-3.8-flash) per the proxy log and returned well-formed completions; gpt-6-luna dispatched to 127.0.0.1:18744 as before; unknown model id still answers 400. Startup log now reads Backends: Antigravity Direct (18751), Codex (18744), AMR Upstream (...).
  • Zero remaining 18745 / ANTIGRAVITY_LS references in bin/design-code-proxy.mjs or its test.

The deployed unit executes $HOME/code/bin/design-code-proxy.mjs directly, so merging + pulling ~/code + restarting design-code-proxy.service makes this live with no Nix rebuild.


Autofix with [code]smith
Open in [code]smith · Need help on this PR? Tag @codesmith-bot with what you need.

…5 to antigravity-direct :18751 (#1413, unblocks #1409)

ANTIGRAVITY_LS_PORT/ANTIGRAVITY_LS_PROXY_PORT (default 18745) becomes ANTIGRAVITY_DIRECT_PORT (default 18751); every recipe primary/fallback leg and the keyword-fallback leg follows. FLEET_BACKEND_PORTS swaps 18745/19745 for 18751/19751 so live catalog validation probes the direct door. antigravity-direct serves the same backend model ids (gemini-3.8-flash, claude-sonnet-5-5 — verified live against :18751/v1/models). Tests: node --test bin/design-code-proxy.test.mjs 16/16. Live: repointed proxy on a scratch port completed design-code-flash + design-code-pro (stream) via 127.0.0.1:18751 and gpt-6-luna via :18744; unknown id still 400.
@dezren39
dezren39 requested a review from a team as a code owner November 2, 2026 22:32
@dezren39
dezren39 merged commit 2f62ce6 into main Nov 2, 2026
1 check passed
@coderabbitai

coderabbitai Bot commented Nov 2, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 57 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 3e727117-8ef3-44e4-a4d0-6483b8080d90
Commits

Reviewing files that changed from the base of the PR and between 8ef2721 and f05ab40.

Files selected for processing: 2
  • bin/design-code-proxy.mjs
  • bin/design-code-proxy.test.mjs
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 2, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-11-02T22:33:50.099858Z f05ab40 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@dezren39
dezren39 deleted the fix/1413-design-code-repoint-18751 branch November 2, 2026 22:32
This was referenced Nov 2, 2026
dezren39 pushed a commit that referenced this pull request Nov 3, 2026
…3923)

The native arm64 step ran cargo test -p id, which builds only the id
package's lib test target. After the #3447 crate split, id-core owns test
targets of its own, so -p id skipped compiling them — the silent skip
that let the missing id-core dev-dependencies (#3923, fixed in #4626)
go unnoticed.

Selecting the workspace default members (default-members = [".",
"crates/id-core"]) compiles id-core's lib test target in the same
invocation while the exact name filter still runs only the determinism
test.

Refs #3923
dezren39 added a commit that referenced this pull request Nov 3, 2026
Resolve the module repoint (#4626): the branch's new cli.Run files move
from github.com/dezren39/mcpx to github.com/developing-today/code/pkgs/mcpx,
and cmd/mcpx/main.go keeps the slim signal+Run entrypoint on the new
import path. No behavioural change from the stage-1 head.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant