Skip to content

feat: 支持一键部署到腾讯云 EdgeOne Pages - #105

Open
ShiYuanS wants to merge 1 commit into
Yu9191:mainfrom
ShiYuanS:main
Open

feat: 支持一键部署到腾讯云 EdgeOne Pages#105
ShiYuanS wants to merge 1 commit into
Yu9191:mainfrom
ShiYuanS:main

Conversation

@ShiYuanS

Copy link
Copy Markdown

本次修改主要为项目增加对腾讯 EO Makers 平台的部署支持,调整了构建和资源配置。经测试:

苹果地图(系统定位)功能正常,可修改位置;

高德地图(Web JS API)存在定位异常,无论实际位置在哪,地图始终显示为深圳(。

,恳请维护者协助,非常感谢!

将选点页面后端从 Hono 改为零依赖(纯 Web 标准 API),使 Cloudflare Workers / Cloudflare Pages / EdgeOne Pages 三端共用同一份代码。

  • 新增 worker/edge-functions/ 边缘函数入口(适配 EdgeOne V8 运行时,不支持 npm 包)

  • 新增 worker/edgeone.json 构建配置

  • 新增 .github/workflows/deploy-edgeone.yml 一键部署工作流

  • worker/src/index.js 重写为平台无关的 handleRequest

  • 移除 hono 依赖并同步 package-lock.json

  • README 新增「部署到腾讯云 EdgeOne Pages」章节

将选点页面后端从 Hono 改为零依赖(纯 Web 标准 API),使 Cloudflare Workers / Cloudflare Pages / EdgeOne Pages 三端共用同一份代码。

- 新增 worker/edge-functions/ 边缘函数入口(适配 EdgeOne V8 运行时,不支持 npm 包)

- 新增 worker/edgeone.json 构建配置

- 新增 .github/workflows/deploy-edgeone.yml 一键部署工作流

- worker/src/index.js 重写为平台无关的 handleRequest

- 移除 hono 依赖并同步 package-lock.json

- README 新增「部署到腾讯云 EdgeOne Pages」章节
@Yu9191

Yu9191 commented Aug 21, 2026

Copy link
Copy Markdown
Owner

感谢贡献!但暂时无法合入这个 PR,原因如下:

关于移除 Hono:

项目选用 Hono 是经过考虑的:

  • 路由清晰易维护,后续扩展新端点(如 API、WebSocket)成本很低
  • 内置的中间件(CORS、缓存控制、错误处理)省去手写样板代码
  • Hono 专为边缘运行时优化,体积极小(~14KB),对 Worker 冷启动无负担
  • Cloudflare / Vercel / Deno 等主流边缘平台都兼容,不存在移植性问题

重写为零依赖纯手工路由,虽然减少了一个依赖,但增加了维护成本且容易出 bug。

关于高德地图异常:

PR 描述中提到「高德地图始终显示深圳」,但没有给出:

  1. 具体的错误日志 / 控制台输出
  2. 复现步骤和截图
  3. 是部署到 EdgeOne 才出现,还是本地也能复现

请补充以上信息,方便定位问题。

建议:

如果想支持 EdgeOne 部署,建议以「新增 EdgeOne 适配层」的方式提 PR,而不是重写现有 Worker 核心代码。可以新增一个独立的 edge-functions 入口调用现有的 handleRequest 逻辑,保持 Cloudflare 侧代码不变。

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