@routerhub/agent-rules 1.5.73 → 1.5.75

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
package/AGENTS.base.md CHANGED
@@ -116,16 +116,12 @@ Closes #456
116
116
  - 用户说”Copilot review 出了一些问题”时,同样触发以下循环流程。
117
117
  - Copilot review 循环流程:
118
118
  1. 打开 PR 页面,逐一查看 Copilot review 提出的每一条问题。
119
- 2. 逐条判断问题是否有道理:确实存在的代码缺陷、逻辑错误、安全风险、性能问题等 → 必须修复;误报、与需求不符、风格偏好无实质影响等 → 在评论中回复解释为何不改,然后 Resolve。
119
+ 2. 逐条判断问题是否有道理:确实存在的代码缺陷、逻辑错误、安全风险、性能问题等 → 必须修复;误报、与需求不符、风格偏好无实质影响等 → 在评论中回复解释为何不改,直接 Resolve。
120
120
  3. 对于有道理的问题,直接修改代码、提交并推送到该 PR 分支。
121
- 4. ⚠️ **强制步骤:所有问题处理完毕后,重新请求 Copilot review。**
122
- 5. ⚠️ **Resolve 规则:必须等下一轮 Copilot review 确认改对了再点 Resolve,禁止 push 后立刻 Resolve。** 具体做法:
123
- - 本轮修复了问题 A → push → 请求下一轮 review → 查看新一轮 review 结果:
124
- - 若 Copilot **未再提及问题 A**(说明已改对)→ 回到上轮 review 的对应评论,点击 **Resolve conversation**。
125
- - 若 Copilot **再次提出问题 A**(说明没改对或改法有问题)→ 不点 Resolve,继续修改并 push,再请求新一轮 review,直到该问题不再出现为止。
126
- - 这样做的好处:每条 conversation 的 Resolve 状态真正代表「该问题已修复且通过 reviewer 验证」,而不是「我改了一下但不知道对不对」。
121
+ 4. ⚠️ **强制步骤:修改代码并 push 后,必须立刻在 GitHub PR 页面上点击 Resolve conversation。每 push 一条修复,立即 Resolve 对应的那条评论。不可批量延后、不可等到下一轮 review 后再 Resolve。** Copilot 每次 review 只看当前代码 diff,不依赖 Resolve 状态——代码改了就不会再提同一问题。因此 push = 问题已处理,应立即 Resolve 让 reviewer 知道该条已处理完毕。
122
+ 5. 所有问题处理完毕后,重新请求 Copilot review
127
123
  6. 重复以上步骤,直到满足以下任一终止条件为止。
128
- - ⚠️ **Resolve 时机:必须等下一轮 Copilot review 确认该问题不再出现后,才能点击 Resolve conversation。** 禁止 push 修复代码后立刻 Resolve——此时尚不知道修复是否正确。只有新一轮 review 未再提及该问题时,才能确认改对了并 Resolve。
124
+ - ⚠️ **强制步骤:每 push 一条修复代码后,必须立刻点击该条评论的 Resolve conversation。不可漏点、不可批量延后。** Copilot 每次 review 只看当前代码 diff——代码改了就不会再提同一问题,因此 push 即意味着问题已处理,应立即 Resolve。
129
125
  - 每次修改后提交的 commit 信息必须中文,格式:`修复 Copilot review 问题:[简要描述]`。
130
126
  - 如果 Copilot review 连续两轮对同一问题提出类似建议,但代码实际上已正确修复(或该建议本身无道理/属于误报),需在 PR 评论中明确说明原因并直接 Resolve,不再反复修改。
131
127
  - ⚠️ **循环终止条件(满足任意一条即停止)**:
@@ -243,6 +239,12 @@ Closes #456
243
239
  - 复杂/重复样式提取为 Tailwind 组件类或 React 组件,避免工具类堆砌。
244
240
  - 类名小驼峰,禁止 `span` 标签选择器。
245
241
 
242
+ ## 前端尺寸单位规范
243
+
244
+ - ⚠️ **前端项目所有尺寸单位必须使用 `rem`,禁止使用 `px`**。包括但不限于:`width`、`height`、`margin`、`padding`、`font-size`、`border-radius`、`gap`、`line-height`、`top`、`left`、`right`、`bottom` 等所有 CSS 属性的数值。
245
+ - 根元素 `html` 的 `font-size` 默认设置为 `16px`(即 `1rem = 16px`),设计稿中的 `px` 值需转换为 `rem`(公式:`rem = px / 16`)。
246
+ - Tailwind CSS 的 spacing 配置需同步改为 `rem` 单位,禁止在 `tailwind.config` 中使用 `px` 值。
247
+
246
248
  ## 组件复用
247
249
 
248
250
  - 优先复用已有组件,能通过 `props` 定制就不建新组件;新功能尽量封装为可复用组件。
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "@routerhub/agent-rules",
3
- "version": "1.5.73",
3
+ "version": "1.5.75",
4
4
  "description": "Shared Copilot agent rules and guidelines for RouterHub projects",
5
5
  "main": "AGENTS.base.md",
6
6
  "bin": {
package/rules/frontend.md CHANGED
@@ -38,6 +38,12 @@ outputName: "frontend"
38
38
  - 复杂/重复样式提取为 Tailwind 组件类或 React 组件,避免工具类堆砌。
39
39
  - 类名小驼峰,禁止 `span` 标签选择器。
40
40
 
41
+ ## 前端尺寸单位规范
42
+
43
+ - ⚠️ **前端项目所有尺寸单位必须使用 `rem`,禁止使用 `px`**。包括但不限于:`width`、`height`、`margin`、`padding`、`font-size`、`border-radius`、`gap`、`line-height`、`top`、`left`、`right`、`bottom` 等所有 CSS 属性的数值。
44
+ - 根元素 `html` 的 `font-size` 默认设置为 `16px`(即 `1rem = 16px`),设计稿中的 `px` 值需转换为 `rem`(公式:`rem = px / 16`)。
45
+ - Tailwind CSS 的 spacing 配置需同步改为 `rem` 单位,禁止在 `tailwind.config` 中使用 `px` 值。
46
+
41
47
  ## 组件复用
42
48
 
43
49
  - 优先复用已有组件,能通过 `props` 定制就不建新组件;新功能尽量封装为可复用组件。
package/rules/global.md CHANGED
@@ -118,16 +118,12 @@ Closes #456
118
118
  - 用户说”Copilot review 出了一些问题”时,同样触发以下循环流程。
119
119
  - Copilot review 循环流程:
120
120
  1. 打开 PR 页面,逐一查看 Copilot review 提出的每一条问题。
121
- 2. 逐条判断问题是否有道理:确实存在的代码缺陷、逻辑错误、安全风险、性能问题等 → 必须修复;误报、与需求不符、风格偏好无实质影响等 → 在评论中回复解释为何不改,然后 Resolve。
121
+ 2. 逐条判断问题是否有道理:确实存在的代码缺陷、逻辑错误、安全风险、性能问题等 → 必须修复;误报、与需求不符、风格偏好无实质影响等 → 在评论中回复解释为何不改,直接 Resolve。
122
122
  3. 对于有道理的问题,直接修改代码、提交并推送到该 PR 分支。
123
- 4. ⚠️ **强制步骤:所有问题处理完毕后,重新请求 Copilot review。**
124
- 5. ⚠️ **Resolve 规则:必须等下一轮 Copilot review 确认改对了再点 Resolve,禁止 push 后立刻 Resolve。** 具体做法:
125
- - 本轮修复了问题 A → push → 请求下一轮 review → 查看新一轮 review 结果:
126
- - 若 Copilot **未再提及问题 A**(说明已改对)→ 回到上轮 review 的对应评论,点击 **Resolve conversation**。
127
- - 若 Copilot **再次提出问题 A**(说明没改对或改法有问题)→ 不点 Resolve,继续修改并 push,再请求新一轮 review,直到该问题不再出现为止。
128
- - 这样做的好处:每条 conversation 的 Resolve 状态真正代表「该问题已修复且通过 reviewer 验证」,而不是「我改了一下但不知道对不对」。
123
+ 4. ⚠️ **强制步骤:修改代码并 push 后,必须立刻在 GitHub PR 页面上点击 Resolve conversation。每 push 一条修复,立即 Resolve 对应的那条评论。不可批量延后、不可等到下一轮 review 后再 Resolve。** Copilot 每次 review 只看当前代码 diff,不依赖 Resolve 状态——代码改了就不会再提同一问题。因此 push = 问题已处理,应立即 Resolve 让 reviewer 知道该条已处理完毕。
124
+ 5. 所有问题处理完毕后,重新请求 Copilot review
129
125
  6. 重复以上步骤,直到满足以下任一终止条件为止。
130
- - ⚠️ **Resolve 时机:必须等下一轮 Copilot review 确认该问题不再出现后,才能点击 Resolve conversation。** 禁止 push 修复代码后立刻 Resolve——此时尚不知道修复是否正确。只有新一轮 review 未再提及该问题时,才能确认改对了并 Resolve。
126
+ - ⚠️ **强制步骤:每 push 一条修复代码后,必须立刻点击该条评论的 Resolve conversation。不可漏点、不可批量延后。** Copilot 每次 review 只看当前代码 diff——代码改了就不会再提同一问题,因此 push 即意味着问题已处理,应立即 Resolve。
131
127
  - 每次修改后提交的 commit 信息必须中文,格式:`修复 Copilot review 问题:[简要描述]`。
132
128
  - 如果 Copilot review 连续两轮对同一问题提出类似建议,但代码实际上已正确修复(或该建议本身无道理/属于误报),需在 PR 评论中明确说明原因并直接 Resolve,不再反复修改。
133
129
  - ⚠️ **循环终止条件(满足任意一条即停止)**: