Skip to content

chore(ui): 更新 packages/ui 至内联链接验证约定 - #1378

Merged
qqqqqf-q merged 1 commit into
mainfrom
chore/bump-ui-inline-link-rules
Sep 24, 2026
Merged

qqqqqf-q merged 1 commit into
mainfrom
chore/bump-ui-inline-link-rules

Conversation

@qqqqqf-q

@qqqqqf-q qqqqqf-q commented Sep 24, 2026 •

Copy link
Copy Markdown
Member

Author

  • Human
  • Agent

Type

  • bug
  • feat
  • test

Summary

把 packages/ui 从 7a918e3 更新到 3865894,引入 felinics/ui#29 的 Tooltip 与内联图标验证约定。该约定是 #1367 的配套规则,本应随 #1367 一起更新 gitlink,合并时遗漏,在此补上。两个提交之间只有 felinics/ui#29 这一个提交。

Related Issues

跟进 #1367;配套 felinics/ui#29。

Validation

Screenshots / Recordings

无界面变化(只更新文档规则的子模块指针)。

Human QA

  • Human QA passed

@qqqqqf-q
qqqqqf-q requested a review from a team September 24, 2026 18:31
@github-actions github-actions Bot added change:web Changes web frontend or shared frontend packages feat Adds or improves functionality size:XS PR size uses the larger of added or deleted lines, excluding generated files labels Sep 24, 2026

@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: dc51250ecd

ℹ️ 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 thread packages/ui
@@ -1 +1 @@
Subproject commit 7a918e31eee74f93415e3a563d92a9cfa6cd4cfb
Subproject commit 386589453ba542a4b29d5d0e18b9e482914dc772

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Complete the mandatory runtime and UI verification

For this submodule-pointer update, the recorded validation only checks commit ancestry and explicitly provides no screenshot. Repository policy applies to every file modification, including changes without a direct UI surface, and requires running the development environment, exercising the real application, and attaching current screenshots; complete and document those checks before accepting the new pointer.

AGENTS.md reference: AGENTS.md:L109-L114

Useful? React with 👍 / 👎.

Comment thread packages/ui
@@ -1 +1 @@
Subproject commit 7a918e31eee74f93415e3a563d92a9cfa6cd4cfb
Subproject commit 386589453ba542a4b29d5d0e18b9e482914dc772

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Remove the extra Human QA warning

The submission text adds a warning paragraph beneath the unchecked Human QA passed box, but the repository requires that section to contain only the single checkbox when human verification has not occurred. Remove the warning line while leaving the checkbox unchecked.

AGENTS.md reference: AGENTS.md:L132-L137

Useful? React with 👍 / 👎.

@github-actions github-actions Bot added the needs:format Description needs template corrections; removed automatically once fixed label Sep 24, 2026
@github-actions

Copy link
Copy Markdown
Contributor

@qqqqqf-q Please complete the following information using the template:

  • Select exactly one option for "Author": Human / Agent.

Editing the description triggers another check; format feedback does not block or cancel code CI.

@qqqqqf-q
qqqqqf-q merged commit 11fcc8c into main Sep 24, 2026
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

change:web Changes web frontend or shared frontend packages feat Adds or improves functionality needs:format Description needs template corrections; removed automatically once fixed size:XS PR size uses the larger of added or deleted lines, excluding generated files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant