Repository navigation
port(go): Grok video bridge from #582 #661
Description
Context
Transition workflow for #582 (Grok video bridge).
- Merged to
dev:257c626c(feat(videos): add Grok video bridge for non-OpenAI models #582) - Carried TS/docs/tests to
dev2-go: feat(videos): carry Grok video bridge onto dev2-go (#582) #660
Why a port is needed
go/internal/images/ already has the image bridge. #582 adds video on the TS side. No matching Go video surface was found on dev2-go.
Source commits
257c626c—feat(videos): add Grok video bridge for non-OpenAI models (#582)
Scope (suggested)
Port against the current TS oracle on dev2-go:
src/images/xai-video-client.tssrc/images/fulfill-video.tssrc/images/loop.ts/plan.ts/synthetic-tool.ts/artifacts.ts/types.tsdeltas- wiring in
src/server/responses/core.ts+ config flags insrc/types.ts - tests under
tests/videos/
Done when
Go video bridge behaves like the TS oracle for the covered cases, with Go tests green on dev2-go.
Activity
github-actions commentedon Jul 29, 2026
Automated translation bookkeeping — detected language: English.
lidge-jun commentedon Jul 29, 2026
Closing: the
dev2-goGo native-port line is retired, so this port obligation no longer exists.The dual-track policy required every merge into
devto be rebased ontodev2-goand ported undergo/. That cost outran its return — the carry backlog never cleared and dogfooding the Go runtime kept producing new defects. Bun-native TypeScript ondevis the single runtime line again.The branch history is preserved read-only at https://github.com/lidge-jun/opencodex-go-archive, and its final tip stays reachable in this repository as the
archive/dev2-gotag. Theneeds-go-portlabel is retired.The TypeScript change this issue tracked is already on
dev; only the Go counterpart is dropped. Future native work is expected to be an incremental module ondev(Rust via N-API is the current candidate) rather than a second integration branch. SeeMAINTAINERS.mdfor the decision record.lidge-jun commentedon Jul 29, 2026
Closing this as not planned: the Go native-port line it belonged to has been retired.
What changed. We ran
dev(Bun-native TypeScript) anddev2-go(Go native port) as two integration lines, and every merge intodevcarried an obligation to port the same change ontodev2-go. In practice that dual track cost more than the port returned — dogfooding the Go runtime kept surfacing new defects faster than we closed them, and the carry queue never emptied. Sodevis once again the single integration line, and there is no port obligation attached to adevmerge any more.Nothing is lost. The full
dev2-gohistory is archived at lidge-jun/opencodex-go-archive, and the final tip is taggedarchive/dev2-goin this repository. If native code returns, the plan is an incremental module (for example Rust via N-API) landing ondev, not a second full-runtime branch.The underlying fix is already on
dev— this issue only tracked the missing Go counterpart, so there is no behavior regression from closing it. Theneeds-go-portlabel is retired as well.Sorry for the churn, and thank you for the carry and port work you put into this line. It was genuinely useful for learning where the runtime boundaries are; the direction change is on me, not on the work.