Skip to content

feat: S1 新增「你熟悉的 Agent」卡(Claude Code / Codex)+ 审计与下载修复 - #11

Closed
qqqqqf-q wants to merge 5 commits into
felinics:mainfrom
qqqqqf-q:fix/landing-audit
Closed

feat: S1 新增「你熟悉的 Agent」卡(Claude Code / Codex)+ 审计与下载修复#11
qqqqqf-q wants to merge 5 commits into
felinics:mainfrom
qqqqqf-q:fix/landing-audit

Conversation

@qqqqqf-q

@qqqqqf-q qqqqqf-q commented Aug 5, 2026

Copy link
Copy Markdown
Member

这个 PR 包含

1. S1 新增「你熟悉的 Agent」卡(主要改动)

让用户知道 Memoh 的 agent runtime 支持 Claude Code / Codex。

  • S1(一台属于 Ta 的电脑)grid 三卡变四卡,Agent 卡位于第一张;桌面端四列、中屏 2x2、移动端单列
  • 卡内视觉为倾斜三卡扇形构图(Claude / Memoh / Codex),cqw 相对单位随卡位整体缩放,中卡严格居中
  • S1 内容容器 1080 → 1200,保证四卡宽度;描述块固定三行最小高度,四张卡 proof 面板顶对齐
  • 图标:claude-code-color.svg 字节级复制自主仓 packages/icons/icons/ @f256bff;codex-blob.svg 同源适配(去掉白底圆角方块,避免在浅灰展示底上显成白砖)

2. 分支上已有的审计/下载修复(4 个 commit)

  • cadf0e4 fix: 修复全站链接与文案审计发现的问题
  • 16bb069 fix: 下载代理在 GitHub token 失效时降级为匿名请求
  • 083626b fix: 静态 meta 与 seo.description 对齐,footer 链接列改为两列
  • ef79440 fix: resolve desktop assets from release metadata

Test plan

  • npm run build(含 vue-tsc)通过
  • 无头浏览器截图自验:桌面 zh/en、900px 中屏、390px 移动端,S1 四卡与扇形构图渲染正常
  • 人工 QA:真实浏览器过首页 happy path(进入动画下四卡间距、扇形居中、双语切换)
  • 人工 QA:下载页(desktop assets 元数据与代理降级)

qqqqqf-q and others added 5 commits July 20, 2026 08:19
- TopBar 撤下 Pricing 入口:/pricing 路由不存在,点击后正文空白
- footer X 链接改回真实账号 x.com/memoh_ai(原 x.com/memohai 为 404)
- footer 撤下 Privacy/Terms 占位链接(均指向首页,无实际页面)
- zh.json 补齐英文残留:博客/加入等候名单/抢先体验等 6 处
- 新增 404 页面与 catch-all 路由,替代原来的空白正文
- create-spa-fallback.mjs 补 /download 静态入口(此前线上返回 404 状态码)
- index.html 与 fallback 脚本的 meta 更新为当前产品定位
线上 GITHUB_TOKEN 过期导致 /downloads/desktop/* 全部 502(manifest 与
安装包均不可下载)。公开仓库的 release 元数据匿名 API 即可读,token 被
拒(401)或触限(403)时去掉 Authorization 重试一次,下载功能不再单点
依赖 token 有效性;匿名限流由 release 缓存(5 分钟)稀释。
- home 描述补回 cloud computer,与 src/locales/en.json seo.description
  逐字一致(Codex review P2):爬虫/链接预览拿到的静态 HTML 与 SPA 水合后
  的 meta 不再发布不一致的产品描述
- Legal 列撤下后 footer 链接区只剩两列,grid 从 sm:grid-cols-3 改为固定
  两列,消除 sm 以上断点的空列
- grid 三卡变四卡,Agent 卡第一位;桌面端四列、中屏 2x2、移动端单列
- 卡内为倾斜三卡扇形构图(Claude / Memoh / Codex),cqw 随卡位整体缩放,中卡严格居中
- S1 内容容器 1080 → 1200 保证四卡宽度;描述块固定三行最小高度,四卡 proof 面板顶对齐
- 图标字节级复制自主仓 packages/icons @f256bff;codex 变体去白底以适配浅灰展示底

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 6de36b1033

ℹ️ 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".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +131 to +132
ctx.waitUntil(cache.put(cacheKey, jsonResponse(release, {
headers: cacheHeaders(ASSET_TTL_SECONDS),

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Don't cache incomplete release metadata for 30 days

When the first request for a versioned tag happens before all platform assets have been attached to the GitHub release, this stores that incomplete release object for ASSET_TTL_SECONDS (30 days). Because proxyAsset() then reads the cached metadata and returns Release asset not found without refetching GitHub, the affected installer URL can keep 404ing long after the asset is uploaded; use a short metadata TTL or avoid caching releases that don't yet contain the expected assets.

Useful? React with 👍 / 👎.

@qqqqqf-q

qqqqqf-q commented Aug 5, 2026

Copy link
Copy Markdown
Member Author

关闭:本 PR 混合了 feature 与无关的 fix commit。其中审计/下载修复内容已通过 #8#10 进入 main;S1 Agent 卡已移至干净分支 PR(仅含 feature 单 commit)。

@qqqqqf-q qqqqqf-q closed this Aug 5, 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