chore: more refs

[zombieJ]

需要有更多的数据来判断是否启动了动画

Summary by CodeRabbit

• 重构
  • 优化了动画组件状态检测逻辑，采用直接的布尔属性判断动画状态，并新增了指示动画支持情况的属性。
• 测试
  • 调整了相关测试用例，以确保与优化后的动画状态逻辑保持一致。

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
motion	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Feb 11, 2025 7:41am

[coderabbitai]

"""

Walkthrough

该 PR 对 CSSMotion 组件做出了更新。主要修改了 CSSMotionRef 接口：将原来作为方法的 inMotion 改为布尔属性，并新增了 enableMotion 属性以直接反映是否支持运动。在组件内部，refObj 对 inMotion 的赋值使用了更直接的布尔判断。同时，相关测试文件中的对 inMotion 的调用方式调整为直接属性访问，简化了调用过程。

Changes

文件	变更摘要
src/CSSMotion.tsx	更新 CSSMotionRef 接口：将 inMotion 由函数改为布尔属性，新增 enableMotion 属性；在 refObj 中改进属性获取逻辑。
tests/StrictMode.spec.tsx	修改测试代码：将对 inMotion 的函数调用，改为直接使用布尔属性访问，并新增对 enableMotion 的断言。

Possibly related PRs

• feat: motion support ref #61: 修改 CSSMotionRef 接口中 inMotion 属性，与本次改动直接相关。
• refactor: use rc-util getNodeRef #57: 调整 CSSMotion.tsx 中引用处理方式，与接口更新相关。
• refactor: make ref stable #62: 涉及 inMotion 属性的处理，直接关联到本次改动。

Poem

我是一只跳跃的小兔子，
在代码森林中欢快奔走。
inMotion从方法变成了真，
enableMotion点亮了新梦。
代码世界如诗如画，
变化中舞出快乐的节奏！
🐇✨🌟
"""

---

📜 Recent review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 462d1ad and 3d38c0d.

📒 Files selected for processing (1)

• tests/StrictMode.spec.tsx (1 hunks)

🚧 Files skipped from review as they are similar to previous changes (1)

• tests/StrictMode.spec.tsx

---

Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

• Review comments: Directly reply to a review comment made by CodeRabbit. Example:
  • I pushed a fix in commit <commit_id>, please review it.
  • Generate unit testing code for this file.
  • Open a follow-up GitHub issue for this discussion.
• Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
  • @coderabbitai generate unit testing code for this file.
  • @coderabbitai modularize this function.
• PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
  • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
  • @coderabbitai read src/utils.ts and generate unit testing code.
  • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
  • @coderabbitai help me debug CodeRabbit configuration file.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (Invoked using PR comments)

• @coderabbitai pause to pause the reviews on a PR.
• @coderabbitai resume to resume the paused reviews.
• @coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
• @coderabbitai full review to do a full review from scratch and review all the files again.
• @coderabbitai summary to regenerate the summary of the PR.
• @coderabbitai generate docstrings to generate docstrings for this PR. (Beta)
• @coderabbitai resolve resolve all the CodeRabbit review comments.
• @coderabbitai configuration to show the current CodeRabbit configuration for the repository.
• @coderabbitai help to get help.

Other keywords and placeholders

• Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
• Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
• Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

• You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
• Please see the configuration documentation for more information.
• If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

• Visit our Documentation for detailed information on how to use CodeRabbit.
• Join our Discord Community to get help, request features, and share feedback.
• Follow us on X/Twitter for updates and announcements.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.47%. Comparing base (b32f8f8) to head (3d38c0d).
Report is 1 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master      #63   +/-   ##
=======================================
  Coverage   98.47%   98.47%           
=======================================
  Files          11       11           
  Lines         394      394           
  Branches      106      108    +2     
=======================================
  Hits          388      388           
  Misses          6        6           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (1)

src/CSSMotion.tsx (1)

174-181: 实现逻辑更加简洁！

refObj 中的属性实现更加直接：

• inMotion 直接通过状态判断，逻辑清晰
• enableMotion 反映了底层动画支持情况

建议考虑添加相关的 JSDoc 文档，说明这些属性的具体用途。

  inMotion: {
    enumerable: true,
+   /** 当前是否正在执行动画 */
    get: () => getStatus() !== STATUS_NONE,
  },
  enableMotion: {
    enumerable: true,
+   /** 当前环境是否支持动画 */
    get: () => supportMotion,
  },

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between b32f8f8 and 462d1ad.

📒 Files selected for processing (2)

• src/CSSMotion.tsx (2 hunks)
• tests/StrictMode.spec.tsx (1 hunks)

🔇 Additional comments (3)

tests/StrictMode.spec.tsx (1)

53-53: 测试代码更新正确！

测试代码已正确更新，将 inMotion() 方法调用改为直接访问 inMotion 属性，与 CSSMotionRef 接口的最新变更保持一致。

Also applies to: 61-61

src/CSSMotion.tsx (2)

19-23: 接口改进提升了可用性！

CSSMotionRef 接口的改进：

• 将 inMotion 从方法改为布尔属性，使用更直观
• 新增 enableMotion 属性，可直接判断动画是否支持

这些改动使接口更加清晰和易用。

---

167-184: 验证 React.useMemo 依赖项

请确认 useMemo 的依赖数组是否需要包含 getStatus 和 supportMotion。

✅ Verification successful

🏁 Script executed:

#!/bin/bash
# 搜索 getStatus 和 supportMotion 的变化情况
rg "getStatus|supportMotion" --type ts

Length of output: 1482

---

React.useMemo 依赖项验证

调查表明：

• getStatus 来自 useStatus（内部可能通过 useSyncState 实现），其返回的函数引用设计为稳定，并且内部能获取最新的状态；
• supportMotion 则是在渲染时计算出来的初始值（由 isSupportTransition 得到），预期在整个生命周期中不会发生变化；

因此，当前在 React.useMemo 的依赖数组中省略 getStatus 和 supportMotion 是有意为之，用以保证返回的 ref 对象恒定，而其 getter 方式可以获取到最新的状态/初始设定值。不需要将它们添加到依赖数组中。