AbstractLock use Macroable

[tw2066]

Summary by CodeRabbit

• 文档

  • 更新繁体中文（台湾）与香港中文组件文档措辞，将“請求追蹤功能”调整为“請求跟蹤功能”，提升表述准确性。

• 重构

  • 增强锁驱动的可扩展性，支持更灵活的动态扩展能力，向后兼容，不影响现有功能与接口。

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

在繁体中文文档中统一了“請求追蹤功能”为“請求跟蹤功能”，并在 AbstractLock 类中引入并使用 Macroable 特征，同时在 src/lock/composer.json 中新增对 hyperf/macroable 的依赖声明。

Changes

Cohort / File(s)	变更说明
文档术语更新 docs/zh-hk/components/telescope.md, docs/zh-tw/components/telescope.md	将文本中“請求追蹤功能”替换为“請求跟蹤功能”，无功能性改动。
锁驱动与依赖 src/lock/src/Driver/AbstractLock.php, src/lock/composer.json	在 AbstractLock 中引入 use Hyperf\Macroable\Macroable; 并使用 Macroable 特征；在 composer.json 中添加依赖 "hyperf/macroable": "~3.1.0"。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

• 重点检查 AbstractLock.php 中引入特征是否会与现有方法或命名冲突（魔术方法/同名方法）。
• 验证 composer.json 的依赖版本与项目整体依赖兼容，不会引入冲突或不一致性。
• 文档改动无需功能性测试，仅需校对用词。

Poem

🐰 术语换新更妥帖，
繁体两本语义接，
AbstractLock 得魔法，
Macroable 轻轻来，
小兔欢跳庆一回。

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PR标题清晰准确地总结了主要变更：为AbstractLock类添加Macroable特性支持，与所有代码改动直接相关。
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 590bd46 and 8b2c7c4.

📒 Files selected for processing (1)

• src/lock/composer.json (1 hunks)

🧰 Additional context used

📓 Path-based instructions (2)

src/*/composer.json

📄 CodeRabbit inference engine (.github/copilot-instructions.md)

src/*/composer.json: Component composer.json must include PSR-4 autoload configuration pointing to src/ directory
Component composer.json must include extra.hyperf.config pointing to the ConfigProvider class
All component composer.json files must require PHP >=8.1 and hyperf/framework ~3.1.0

src/*/composer.json: Individual component composer.json files define component-specific dependencies and can be used for split repository publishing
Use Carbon for date/time handling and Guzzle for HTTP operations (via hyperf/guzzle) as common dependencies

Files:

• src/lock/composer.json

**/*.{json,toml}

📄 CodeRabbit inference engine (AGENTS.md)

JSON/TOML: Use 2-space indent and avoid comments in JSON files

Files:

• src/lock/composer.json

🧠 Learnings (7)

📚 Learning: 2025-12-12T05:33:43.947Z

Learnt from: CR
Repo: friendsofhyperf/components PR: 0
File: CLAUDE.md:0-0
Timestamp: 2025-12-12T05:33:43.947Z
Learning: Applies to composer.json : PHP version requirement is >=8.1 and Hyperf framework requirement is >=3.1.0

Applied to files:

• src/lock/composer.json

📚 Learning: 2025-12-12T05:33:20.686Z

Learnt from: CR
Repo: friendsofhyperf/components PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-12-12T05:33:20.686Z
Learning: Applies to src/*/composer.json : All component `composer.json` files must require PHP >=8.1 and hyperf/framework ~3.1.0

Applied to files:

• src/lock/composer.json

📚 Learning: 2025-12-12T05:33:20.686Z

Learnt from: CR
Repo: friendsofhyperf/components PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-12-12T05:33:20.686Z
Learning: Applies to src/*/composer.json : Component `composer.json` must include extra.hyperf.config pointing to the ConfigProvider class

Applied to files:

• src/lock/composer.json

📚 Learning: 2025-12-12T05:33:43.947Z

Learnt from: CR
Repo: friendsofhyperf/components PR: 0
File: CLAUDE.md:0-0
Timestamp: 2025-12-12T05:33:43.947Z
Learning: Applies to src/*/composer.json : Use Carbon for date/time handling and Guzzle for HTTP operations (via hyperf/guzzle) as common dependencies

Applied to files:

• src/lock/composer.json

📚 Learning: 2025-12-12T05:33:43.947Z

Learnt from: CR
Repo: friendsofhyperf/components PR: 0
File: CLAUDE.md:0-0
Timestamp: 2025-12-12T05:33:43.947Z
Learning: Applies to src/*/src/**/*.php : Use Hyperf's Context API for request-scoped data instead of global state

Applied to files:

• src/lock/composer.json

📚 Learning: 2025-12-12T05:33:43.947Z

Learnt from: CR
Repo: friendsofhyperf/components PR: 0
File: CLAUDE.md:0-0
Timestamp: 2025-12-12T05:33:43.947Z
Learning: Applies to src/*/src/**/*.php : Ensure component namespace doesn't conflict with existing Hyperf components or other packages in the ecosystem

Applied to files:

• src/lock/composer.json

📚 Learning: 2025-12-12T05:33:43.947Z

Learnt from: CR
Repo: friendsofhyperf/components PR: 0
File: CLAUDE.md:0-0
Timestamp: 2025-12-12T05:33:43.947Z
Learning: Applies to tests/**/*.php : Mock Hyperf services appropriately such as ValidatorFactory, ConfigInterface, etc. in tests

Applied to files:

• src/lock/composer.json

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (10)

• GitHub Check: Test on PHP 8.1 with Swoole 6.0.2
• GitHub Check: Test on PHP 8.3 with Swoole 6.1.3
• GitHub Check: Test on PHP 8.1 with Swoole 5.1.8
• GitHub Check: Test on PHP 8.2 with Swoole 5.1.8
• GitHub Check: Test on PHP 8.3 with Swoole 6.0.2
• GitHub Check: Test on PHP 8.3 with Swoole 5.1.8
• GitHub Check: Test on PHP 8.1 with Swoole 6.1.3
• GitHub Check: Test on PHP 8.2 with Swoole 6.0.2
• GitHub Check: Test on PHP 8.2 with Swoole 6.1.3
• GitHub Check: Seer Code Review

🔇 Additional comments (1)

src/lock/composer.json (1)

25-25: LGTM！新增的 macroable 依赖符合预期。

新增的 hyperf/macroable 依赖版本约束 ~3.1.0 与其他 Hyperf 组件保持一致，依赖项按字母顺序正确排列，与 PR 目标（在 AbstractLock 中使用 Macroable）相符。

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

🧹 Nitpick comments (1)

src/lock/src/Driver/AbstractLock.php (1)

24-25: Macroable trait 集成良好

Macroable trait 的使用是正确的,它为 AbstractLock 启用了动态方法注册功能,不会破坏现有 API。这使得用户可以在运行时扩展锁的功能。

建议在组件文档中添加关于 macroable 功能的使用说明,例如如何使用 AbstractLock::macro() 方法注册自定义方法。

📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 364db04 and 590bd46.

📒 Files selected for processing (3)

• docs/zh-hk/components/telescope.md (1 hunks)
• docs/zh-tw/components/telescope.md (1 hunks)
• src/lock/src/Driver/AbstractLock.php (2 hunks)

🧰 Additional context used

📓 Path-based instructions (6)

{docs/**/*,README*.md}

📄 CodeRabbit inference engine (AGENTS.md)

Keep code snippets in docs/ and README*.md files current when components change

Files:

• docs/zh-hk/components/telescope.md
• docs/zh-tw/components/telescope.md

**/*.md

📄 CodeRabbit inference engine (AGENTS.md)

Markdown: Use ATX headings, fenced code blocks with language hints, and wrap lines near 100 chars

Files:

• docs/zh-hk/components/telescope.md
• docs/zh-tw/components/telescope.md

**/*.php

📄 CodeRabbit inference engine (.github/copilot-instructions.md)

**/*.php: All PHP files must include declare(strict_types=1) at the top
Use PSR-12 coding standards for PHP code formatting
Use 4-space indentation with short array syntax in PHP code
Use .php-cs-fixer.php configuration for PHP code formatting

All PHP files must include a strict_types declaration at the top

**/*.php: Follow project PHP coding standard enforced by php-cs-fixer with PSR-12 style, 4-space indentation, and short array syntax
Maintain type coverage by updating or adding tests when public APIs change; ensure composer test:types stays green before pushing

Files:

• src/lock/src/Driver/AbstractLock.php

src/**/src/**/*.php

📄 CodeRabbit inference engine (.github/copilot-instructions.md)

Use namespace pattern FriendsOfHyperf\{ComponentName} for all component classes

Files:

• src/lock/src/Driver/AbstractLock.php

src/*/src/**/*.php

📄 CodeRabbit inference engine (CLAUDE.md)

src/*/src/**/*.php: Namespace convention for components must follow FriendsOfHyperf{ComponentName}
All code must be coroutine-safe and avoid global state without proper context management, blocking I/O operations, and non-coroutine-safe third-party libraries without wrappers
Use Hyperf's Context API for request-scoped data instead of global state
Follows PSR-12 coding standards and use PHP-CS-Fixer for automatic formatting
Use PHPStan at maximum level for static analysis
Ensure component namespace doesn't conflict with existing Hyperf components or other packages in the ecosystem
Integrate deeply with Hyperf's Dependency Injection container for service registration
Leverage Hyperf's AOP (Aspect-Oriented Programming) for cross-cutting concerns via aspects defined in ConfigProvider
Use Hyperf's Event System to register listeners for framework events in components
All code must support coroutine-based concurrency using Swoole/Swow compatibility

Files:

• src/lock/src/Driver/AbstractLock.php

src/**/*.php

📄 CodeRabbit inference engine (AGENTS.md)

Each component lives in its own subdirectory with PSR-4 namespaces matching FriendsOfHyperf\*

Files:

• src/lock/src/Driver/AbstractLock.php

🔇 Additional comments (2)

docs/zh-tw/components/telescope.md (1)

53-53: 此文档更改与 PR 目标不一致

该行仅是措辞更新(追蹤 → 跟蹤),但与 PR 标题"AbstractLock use Macroable"无关。建议将不相关的文档更新分离到单独的 PR 中,以保持 PR 的单一职责原则。

docs/zh-hk/components/telescope.md (1)

53-53: 此文档更改与 PR 目标不一致

与 docs/zh-tw/components/telescope.md 的更改相同,这是一个与 PR 主要目标无关的措辞更新。建议将这些文档更新移至单独的 PR。