GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — chore: 清理过时与冗余的代码注释 by longsizhuo · Pull Request #347 · InvolutionHell/involutionhell

Conversation

@longsizhuo

@longsizhuo longsizhuo commented May 12, 2026

Copy link
Copy Markdown
Member

Summary

按项目注释规范全量审计源码，删除"重复说明 WHAT"的冗余注释、误导性残留标记和与 issue/PR 绑定的任务引用；保留所有解释 WHY 的踩坑记录、不变式与降级策略。

主要改动

  • app/components/assistant-ui/thread.tsx：删除 ThreadProps 内重复属性名的注释、各组件前"X 组件"式 header 注释、关于 Composer Viewport 重构的残留备注。
  • app/components/DocsAssistant.tsx：移除占位注释（// 其他需要的属性...）和复述变量名的 state 注释；保留并精炼 showSuggestionsLoader / pendingSuggestions 的真正 WHY（与流式回复抢渲染、loader 解耦）。
  • app/components/rank/HotDocsTab.tsx：删除纯 JSX 区段标签（{/* 加载状态 */} 等），条件本身已自解释。
  • app/[locale]/feed/components/LinkCard.tsx：删除整卡可点击区域、卡片内容区、标题、分类 badge 等 JSX 标签注释；保留 referrerPolicy 等踩坑说明。
  • app/components/ZoteroFeed.tsx：删除误导性 // Debug helpers (removed in production) 残留标记。
  • app/components/assistant-ui/assistant-modal.tsx：剥离 issue #285 任务引用，保留"受控 open 是为了让 X 按钮能主动关闭"的 WHY；精炼埋点条件注释。
  • scripts/generate-leaderboard.mjs：删除已不适用的"早期直连 Postgres 5432"历史段落。

Test plan

  • npx tsc --noEmit 仅剩仓库既有的 tsconfig baseUrl 弃用 warning，无新增报错
  • CI lint / typecheck 通过
  • 视觉检查：feed 卡片、热门文档、AI 助手模态、关闭按钮交互未受影响

Generated by Claude Code

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@claude
chore: 清理项目中冗余/过时的代码注释
a5dc2c0 
- 删除 thread.tsx 中重复说明属性名/函数名的"是什么"注释（标识符已自解释）
- 移除 DocsAssistant.tsx 的占位注释（"其他需要的属性..."）和重复 state 说明，
  保留并精炼真正解释"为什么"需要 loader/pending 缓存的注释
- 删除 HotDocsTab.tsx、LinkCard.tsx 中纯标签 JSX 区段（"加载状态/列表/标题"等）
- 删除 ZoteroFeed.tsx 中误导性的 "Debug helpers (removed in production)" 残留标记
- 删除 assistant-modal.tsx 中 issue #285 任务引用，保留受控 state 的 WHY 说明
- 删除 generate-leaderboard.mjs 中已不适用的"历史：早期直连 Postgres"段落

保留所有解释 WHY 的踩坑记录、不变式、降级策略等有价值注释。
@vercel

vercel Bot commented May 12, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
involutionhell-github-io Canceled Canceled May 12, 2026 8:23pm
website-preview Canceled Canceled May 12, 2026 8:23pm
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@claude
Revert "chore: 清理项目中冗余/过时的代码注释"
b416011 
This reverts commit a5dc2c0.
@vercel
vercel Bot deployed to Preview – website-preview May 12, 2026 20:17
View deployment
@vercel
vercel Bot deployed to Preview – involutionhell-github-io May 12, 2026 20:22
View deployment

This branch was successfully deployed

2 active deployments
Preview – involutionhell-github-io — b4160114 Deployed May 12, 2026 by vercel[bot]
View deployment
Preview – website-preview — b4160114 Deployed May 12, 2026 by vercel[bot]
View deployment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

2 participants

@longsizhuo @claude
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.