Skip to content

refactor(webview,core): refactor webview and fetch - #61

Merged
Transparent-fish merged 5 commits into
mainfrom
refactor/refactor-webview
Aug 11, 2026
Merged

refactor(webview,core): refactor webview and fetch#61
Transparent-fish merged 5 commits into
mainfrom
refactor/refactor-webview

Conversation

@Transparent-fish

@Transparent-fish Transparent-fish commented Aug 11, 2026

Copy link
Copy Markdown
Owner

关联 Issue

#39 #60

变更概述

WebView 前端重构:将 WebviewApp / ContestApp / SidebarApp / SubmissionDetailApp 中的内联 UI 与逻辑抽取为可复用组件、hooks 与工具函数。

  • 新增组件:ProblemView、TaskList、SubmitPanel、SubmissionHistory、SampleBox、StatusBadge、CookieSettingsPanel
  • 新增 hooks:useWebviewMessage(统一消息处理)、useTranslation(翻译状态与按题目缓存)
  • 新增 utils:format(formatStart)、status(statusColor)
  • 精简 4 个主应用文件,功能行为不变,仅做结构拆分
  • 另更新根目录 README.md(项目结构与脚本修正)
  • 统计:16 个文件

变更类型

  • fix Bug
  • Feature
  • UI / Style
  • Refactor
  • Documentation

涉及文件

  • 前端
packages/webview/src/WebviewApp.tsx             (M)
packages/webview/src/ContestApp.tsx             (M)
packages/webview/src/SidebarApp.tsx             (M)
packages/webview/src/SubmissionDetailApp.tsx    (M)
packages/webview/src/components/ProblemView.tsx       (A)
packages/webview/src/components/TaskList.tsx          (A)
packages/webview/src/components/SubmitPanel.tsx       (A)
packages/webview/src/components/SubmissionHistory.tsx (A)
packages/webview/src/components/SampleBox.tsx         (A)
packages/webview/src/components/StatusBadge.tsx       (A)
packages/webview/src/components/CookieSettingsPanel.tsx (A)
packages/webview/src/hooks/useWebviewMessage.ts       (A)
packages/webview/src/hooks/useTranslation.ts          (A)
packages/webview/src/utils/format.ts                  (A)
packages/webview/src/utils/status.ts                  (A)
  • 后端
fetch.ts

@github-actions

Copy link
Copy Markdown

PR 审查报告

检查结果

检查项 状态
pnpm build
pnpm lint
pnpm test
函数 <= 130 行

未通过的检查

全部通过,无需修改。


自动生成 by PR Review workflow

@github-actions

Copy link
Copy Markdown

PR 审查报告

检查结果

检查项 状态
pnpm build
pnpm lint
pnpm test
函数 <= 80 行

未通过的检查

  • 函数长度不超过 80 行

自动生成 by PR Review workflow

@Transparent-fish

Copy link
Copy Markdown
Owner Author

ci 怎么有点似了

@github-actions

github-actions Bot commented Aug 11, 2026

Copy link
Copy Markdown

PR 审查报告

检查结果

检查项 状态
pnpm lint
pnpm build
pnpm test:unit
函数 <= 80 行

未通过的检查

  • 函数长度不超过 80 行

失败详情

函数长度 <= 80 行(最后 40 行)

handleResponse in apps/vscode-extension/src/tools/fetch.ts (105 lines, line 212)

自动生成 by PR Review workflow

@Transparent-fish

Copy link
Copy Markdown
Owner Author

好歹是把 ci 修好了 :(

@github-actions

Copy link
Copy Markdown

PR 审查报告

检查结果

检查项 状态
pnpm lint
pnpm build
pnpm test:unit
函数 ≤ 80 行 / tsx ≤ 600 行

未通过的检查

全部通过,无需修改。


自动生成 by PR Review workflow

@Transparent-fish

Copy link
Copy Markdown
Owner Author

终于是修好了/

@Transparent-fish

Copy link
Copy Markdown
Owner Author

不拆 fetch 过不掉 ci /

@Transparent-fish Transparent-fish changed the title refactor(webview): refactor webview refactor(webview,core): refactor webview and fetch Aug 11, 2026
@Transparent-fish

Copy link
Copy Markdown
Owner Author

testing

@Transparent-fish

Transparent-fish commented Aug 11, 2026

Copy link
Copy Markdown
Owner Author

??? 测出奇奇怪怪的问题了
image
image
观察到 fetchSubmitPage 被渲染了两边,还被加了 ui. 的前缀

@Transparent-fish

Copy link
Copy Markdown
Owner Author

??? 测出奇奇怪怪的问题了 image image 观察到 fetchSubmitPage 被渲染了两边,还被加了 ui. 的前缀

#62 本问题不在本分支修复

@Transparent-fish

Copy link
Copy Markdown
Owner Author

test pass

@Transparent-fish
Transparent-fish merged commit 221b44e into main Aug 11, 2026
2 checks passed
@Transparent-fish
Transparent-fish deleted the refactor/refactor-webview branch August 11, 2026 11:53
@Transparent-fish Transparent-fish linked an issue Aug 11, 2026 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Refactor]: 拆分 fetch.ts

1 participant