Skip to content

fix: restore docker web runtime routing#54

Closed
lWolvesl wants to merge 2 commits intoqxcnm:mainfrom
lWolvesl:main
Closed

fix: restore docker web runtime routing#54
lWolvesl wants to merge 2 commits intoqxcnm:mainfrom
lWolvesl:main

Conversation

@lWolvesl
Copy link
Contributor

@lWolvesl lWolvesl commented Mar 16, 2026

变更摘要

变更摘要

    1. 修复项目重构后 Docker 相关构建路径失效的问题,恢复 crates/docker 下 compose 与镜像构建链路,并同步更新运行与部署指南中的 Docker 说明
    1. 修复 Docker / Web 模式下前端初始化链路,避免页面持续显示“服务未连接”
    • 原因是旧逻辑将“浏览器运行时”和“桌面运行时”按同一方式处理,在非 Tauri 环境下直接跳过初始化,导致 initializeService(...) 未实际执行,前端无法正确探活后端 service
    1. 修复 Web 前端在嵌入静态资源模式下的 SPA 路由响应,避免侧边栏切页触发下载而不是页面跳转
    • 原因是请求 /accounts 等前端路由时不存在对应真实文件,服务端虽会回退返回 index.html,但响应类型未按实际命中的 HTML 资源正确设置,导致浏览器将响应识别为下载(修复后会把响应内容设置为text/html从而防止下载并正常跳转),此条修复与未合并的Pull Request #52 应存在类似修复位置
  • !!!请注意:本次提交与d3cb5ec版本可以无冲突合并,但本次未测试app端的效果,代码逻辑上对app端无影响,因此请对https://github.com/lWolvesl/Codex-Manager.git测试后再选择是否合并,建议只合并 docker 构建相关问题,后端刺探和路由响应可以在彻底完成重构后再选用方案
  • 本次提交代码由gpt-5.4生成

改动范围

  • Frontend
  • Desktop / Tauri
  • Service
  • Gateway / Protocol Adapter
  • Docs / Governance
  • Workflow / Release

主要文件

验证

  • pnpm -C apps run test
  • pnpm -C apps run build
  • pnpm -C apps run test:ui
  • cargo test --workspace
  • docker compose build
  • docker compose up
  • 前端功能测试
  • api功能测试

@qxcnm
Copy link
Owner

qxcnm commented Mar 18, 2026

感谢这次整理。PR 里提到的 Docker / Web 路径、Web 初始化探活、以及前端路由相关问题,当前 main 已经通过后续提交按现有代码结构收口处理了。

由于这个 PR 现在和主线冲突较多,而且对应的问题已被后续改动覆盖,这边先按 superseded 关闭,不再继续合并。如果你确认在最新 main 上还有可稳定复现的具体问题,欢迎单独提 issue 或按最小修复范围重新开 PR。感谢。

@qxcnm qxcnm closed this Mar 18, 2026
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.

2 participants