Merged
Conversation
|
Caution Review failedThe pull request is closed. ℹ️ Recent review infoConfiguration used: Organization UI Review profile: CHILL Plan: Pro ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (2)
📝 Walkthrough概览向 README.md 和 README_CN.md 中添加了新的预览（Preview/预览）部分，包含居中的预览图片。变更为纯文档内容添加，无运行时逻辑或配置改动。两个文件中均发现轻微的编码工件。 变更
预计代码审查工作量🎯 2 (简单) | ⏱️ ~8 分钟 诗
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🧹 Nitpick comments (1)
web/src/pages/stats/index.tsx (1)
901-903: 建议将图表高度提取为单一常量。当前
400同时出现在 Tailwind class 和ResponsiveContainer参数中，后续改动容易不一致。建议统一来源。可选重构示例
+const CHART_HEIGHT = 400; ... -<div className="w-full h-[400px] min-h-[400px]"> - <ResponsiveContainer width="100%" height={400}> +<div className="w-full" style={{ height: CHART_HEIGHT, minHeight: CHART_HEIGHT }}> + <ResponsiveContainer width="100%" height={CHART_HEIGHT}>🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@web/src/pages/stats/index.tsx` around lines 901 - 903, Extract the repeated numeric height into a single constant (e.g. CHART_HEIGHT = 400) and use it for both the container styles and the chart prop: replace the hardcoded Tailwind arbitrary classes h-[400px] and min-h-[400px] with a style prop on the wrapper div (e.g. style={{ height: CHART_HEIGHT, minHeight: CHART_HEIGHT }}) and set ResponsiveContainer height={CHART_HEIGHT}; update the wrapper’s className to keep layout classes like "w-full" and remove the duplicated numeric literals so ResponsiveContainer and the surrounding div both reference CHART_HEIGHT (symbols to change: ResponsiveContainer, ComposedChart, wrapper div className).
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@README_CN.md`:
- Line 1: The file README_CN.md contains a hidden BOM (U+FEFF) at the start of
line 1 before the "<p align=\"center\">" tag; remove the leading BOM so the
first character of the file is the '<' of "<p align=\"center\">". Ensure the
file is saved without a UTF-8 BOM (e.g., re-save as UTF-8 without BOM or strip
U+FEFF) so editors and CI see no invisible character.
In `@README.md`:
- Line 1: The README.md file contains a leading invisible BOM character U+FEFF
at the start of the file which can break Markdown tooling; remove the BOM by
deleting the first invisible character (U+FEFF) so the file begins with the
visible '<p align="center">' text, and save the file without BOM (ensure your
editor or the commit uses UTF-8 without BOM).
---
Nitpick comments:
In `@web/src/pages/stats/index.tsx`:
- Around line 901-903: Extract the repeated numeric height into a single
constant (e.g. CHART_HEIGHT = 400) and use it for both the container styles and
the chart prop: replace the hardcoded Tailwind arbitrary classes h-[400px] and
min-h-[400px] with a style prop on the wrapper div (e.g. style={{ height:
CHART_HEIGHT, minHeight: CHART_HEIGHT }}) and set ResponsiveContainer
height={CHART_HEIGHT}; update the wrapper’s className to keep layout classes
like "w-full" and remove the duplicated numeric literals so ResponsiveContainer
and the surrounding div both reference CHART_HEIGHT (symbols to change:
ResponsiveContainer, ComposedChart, wrapper div className).
ℹ️ Review info
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
web/public/preview.pngis excluded by!**/*.png
📒 Files selected for processing (7)
README.mdREADME_CN.mdweb/src/components/layout/app-sidebar/nav-user.tsxweb/src/components/ui/chart.tsxweb/src/hooks/queries/use-usage-stats.tsweb/src/pages/projects/index.tsxweb/src/pages/stats/index.tsx
📜 Review details
🔇 Additional comments (7)
web/src/components/layout/app-sidebar/nav-user.tsx (1)
204-204: 仅末尾换行，无功能影响。该变更无需额外审查意见。
web/src/components/ui/chart.tsx (1)
66-68: 当前改动无阻塞问题。此片段无需补充审查意见。
README.md (1)
11-15: 预览区块插入位置与结构清晰。放在 Features 前，且图片居中展示，和本次文档目标一致。
web/src/hooks/queries/use-usage-stats.ts (1)
6-6: 该片段当前无阻塞性问题。无需补充审查意见。
Also applies to: 120-120
README_CN.md (1)
11-15: “预览”章节补充到位。位置、结构和 README 英文版保持一致，文档体验更完整。
web/src/pages/projects/index.tsx (2)
44-60: 排序派生逻辑清晰且可预测。在渲染前统一按
createdAt与id排序，有助于列表顺序稳定。
110-113: 渲染侧切换到sortedProjects一致性良好。数据来源和排序策略保持一致，避免 UI 顺序波动。
| @@ -1,4 +1,4 @@ | |||
| <p align="center"> | |||
| <p align="center"> | |||
There was a problem hiding this comment.
建议移除文件头隐藏 BOM 字符。
Line 1 存在不可见 U+FEFF，可能导致部分编辑器或 CI 文档检查出现异常差异。
建议修复
-<p align="center">
+<p align="center">🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@README_CN.md` at line 1, The file README_CN.md contains a hidden BOM (U+FEFF)
at the start of line 1 before the "<p align=\"center\">" tag; remove the leading
BOM so the first character of the file is the '<' of "<p align=\"center\">".
Ensure the file is saved without a UTF-8 BOM (e.g., re-save as UTF-8 without BOM
or strip U+FEFF) so editors and CI see no invisible character.
| @@ -1,4 +1,4 @@ | |||
| <p align="center"> | |||
| <p align="center"> | |||
There was a problem hiding this comment.
移除文件头隐藏 BOM 字符。
Line 1 前存在不可见 U+FEFF，可能引发部分 Markdown 工具链的解析/差异显示异常，建议删除。
建议修复
-<p align="center">
+<p align="center">🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@README.md` at line 1, The README.md file contains a leading invisible BOM
character U+FEFF at the start of the file which can break Markdown tooling;
remove the BOM by deleting the first invisible character (U+FEFF) so the file
begins with the visible '<p align="center">' text, and save the file without BOM
(ensure your editor or the commit uses UTF-8 without BOM).
ymkiux
force-pushed
the
docs/readme-preview
branch
from
3f361a5 to
70cd140
Compare
SurviveM
approved these changes
Feb 27, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
变更说明
影响范围
验证
Summary by CodeRabbit