Skip to content

Conversation

@huangdijia
Copy link
Contributor

@huangdijia huangdijia commented Nov 15, 2025

Summary

  • Removed the http-logger component from the monorepo
  • Deleted all source files and documentation for http-logger
  • Updated composer.json to remove http-logger dependencies and autoload configuration
  • Removed http-logger from documentation sidebars across all language variants (en, zh-cn, zh-hk, zh-tw)
  • Updated component lists in README and documentation files

Changes

This PR completely removes the http-logger component from the repository, following the pattern of previous component removals (pest-plugin-hyperf, async-queue-closure-job).

Test plan

  • Verify composer.json is valid
  • Confirm documentation builds without broken links
  • Ensure sidebar navigation works correctly in all language variants

Summary by CodeRabbit

发布说明

  • 已移除
    • Http Logger 组件及其依赖已从项目中完全删除
    • 所有语言版本的 Http Logger 文档已移除
    • 组件将不再可用

This commit removes the http-logger component from the monorepo, including:
- Deleted src/http-logger directory and all its source files
- Removed http-logger from composer.json dependencies and autoload
- Removed http-logger documentation from all language variants (en, zh-cn, zh-hk, zh-tw)
- Updated sidebar configurations across all documentation languages
- Updated component list in README and documentation
@coderabbitai
Copy link

coderabbitai bot commented Nov 15, 2025

Walkthrough

此PR完全移除了Http Logger组件,包括从composer.json删除依赖声明、从所有语言版本的文档侧边栏移除入口、删除组件文档文件、从支持组件表中移除条目,以及删除src/http-logger/目录下的所有源代码、配置和工作流文件。

Changes

Cohort / File(s) Summary
Package & Dependency Configuration
composer.json
移除Http Logger的require/replace依赖项、PSR-4自动加载映射和ConfigProvider引用
Documentation Sidebars
docs/.vitepress/src/en/sidebars.ts, docs/.vitepress/src/zh-cn/sidebars.ts, docs/.vitepress/src/zh-hk/sidebars.ts, docs/.vitepress/src/zh-tw/sidebars.ts
从所有语言版本的组件导航中移除Http Logger入口
Component Documentation Files
docs/en/components/http-logger.md, docs/zh-cn/components/http-logger.md, docs/zh-hk/components/http-logger.md, docs/zh-tw/components/http-logger.md
删除所有语言版本的Http Logger组件文档
Component Lists
docs/en/guide/start/components.md, docs/zh-cn/guide/start/components.md, docs/zh-hk/guide/start/components.md, docs/zh-tw/guide/start/components.md, src/.github/profile/README.md
从支持的组件列表表格中移除http-logger条目
Http Logger Source Code
src/http-logger/.gitattributes, src/http-logger/.github/FUNDING.yml, src/http-logger/.github/workflows/close-pull-request.yml, src/http-logger/.github/workflows/release.yaml, src/http-logger/LICENSE, src/http-logger/README.md, src/http-logger/composer.json, src/http-logger/publish/http_logger.php
删除Http Logger组件的配置、工作流和文档文件
Http Logger Implementation
src/http-logger/src/ConfigProvider.php, src/http-logger/src/Middleware/HttpLogger.php, src/http-logger/src/Profile/LogProfile.php, src/http-logger/src/Profile/DefaultLogProfile.php, src/http-logger/src/Profile/DisableLogProfile.php, src/http-logger/src/Profile/ProfileFactory.php, src/http-logger/src/Writer/LogWriter.php, src/http-logger/src/Writer/DefaultLogWriter.php, src/http-logger/src/Writer/NullLogWriter.php, src/http-logger/src/Writer/WriterFactory.php
删除Http Logger的核心实现代码,包括中间件、日志策略接口/实现、日志写入器接口/实现及工厂类

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

  • 主要由同质的文件删除组成,需确认删除的完整性和一致性
  • 需检查所有四个语言版本的侧边栏和组件列表是否同步更新
  • 验证composer.json中依赖移除是否彻底(require、replace、autoload、configProvider)

Possibly related PRs

  • Adds translate support #815: 该PR在同一文档和侧边栏区域增加Http Logger组件的文档和入口,与本PR在概念上相反且修改相同文件。

Poem

🐰✨ 我们说再见,Http Logger君,
日志不再尾随每个请求的脚步,
代码变得轻盈,文档归档尘埃,
一个篇章落幕,新的篇章待续~🌙

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 拉取请求标题清晰准确地总结了变更集的主要内容——完全移除http-logger组件。标题与所有文件变更(源代码、文档、配置)完全相关。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch remove-http-logger-component

📜 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 428c8d4 and 1093f51.

📒 Files selected for processing (32)
  • composer.json (0 hunks)
  • docs/.vitepress/src/en/sidebars.ts (0 hunks)
  • docs/.vitepress/src/zh-cn/sidebars.ts (0 hunks)
  • docs/.vitepress/src/zh-hk/sidebars.ts (0 hunks)
  • docs/.vitepress/src/zh-tw/sidebars.ts (0 hunks)
  • docs/en/components/http-logger.md (0 hunks)
  • docs/en/guide/start/components.md (0 hunks)
  • docs/zh-cn/components/http-logger.md (0 hunks)
  • docs/zh-cn/guide/start/components.md (0 hunks)
  • docs/zh-hk/components/http-logger.md (0 hunks)
  • docs/zh-hk/guide/start/components.md (0 hunks)
  • docs/zh-tw/components/http-logger.md (0 hunks)
  • docs/zh-tw/guide/start/components.md (0 hunks)
  • src/.github/profile/README.md (0 hunks)
  • src/http-logger/.gitattributes (0 hunks)
  • src/http-logger/.github/FUNDING.yml (0 hunks)
  • src/http-logger/.github/workflows/close-pull-request.yml (0 hunks)
  • src/http-logger/.github/workflows/release.yaml (0 hunks)
  • src/http-logger/LICENSE (0 hunks)
  • src/http-logger/README.md (0 hunks)
  • src/http-logger/composer.json (0 hunks)
  • src/http-logger/publish/http_logger.php (0 hunks)
  • src/http-logger/src/ConfigProvider.php (0 hunks)
  • src/http-logger/src/Middleware/HttpLogger.php (0 hunks)
  • src/http-logger/src/Profile/DefaultLogProfile.php (0 hunks)
  • src/http-logger/src/Profile/DisableLogProfile.php (0 hunks)
  • src/http-logger/src/Profile/LogProfile.php (0 hunks)
  • src/http-logger/src/Profile/ProfileFactory.php (0 hunks)
  • src/http-logger/src/Writer/DefaultLogWriter.php (0 hunks)
  • src/http-logger/src/Writer/LogWriter.php (0 hunks)
  • src/http-logger/src/Writer/NullLogWriter.php (0 hunks)
  • src/http-logger/src/Writer/WriterFactory.php (0 hunks)
💤 Files with no reviewable changes (32)
  • src/http-logger/README.md
  • docs/.vitepress/src/zh-tw/sidebars.ts
  • src/http-logger/src/Profile/ProfileFactory.php
  • src/http-logger/src/Writer/NullLogWriter.php
  • docs/zh-tw/components/http-logger.md
  • src/http-logger/src/Writer/LogWriter.php
  • src/http-logger/src/ConfigProvider.php
  • docs/en/components/http-logger.md
  • src/.github/profile/README.md
  • composer.json
  • src/http-logger/composer.json
  • src/http-logger/LICENSE
  • src/http-logger/src/Profile/DisableLogProfile.php
  • src/http-logger/.gitattributes
  • docs/.vitepress/src/zh-cn/sidebars.ts
  • docs/zh-tw/guide/start/components.md
  • docs/.vitepress/src/en/sidebars.ts
  • docs/en/guide/start/components.md
  • docs/zh-cn/components/http-logger.md
  • src/http-logger/.github/FUNDING.yml
  • docs/zh-hk/guide/start/components.md
  • src/http-logger/src/Profile/DefaultLogProfile.php
  • docs/.vitepress/src/zh-hk/sidebars.ts
  • src/http-logger/src/Profile/LogProfile.php
  • src/http-logger/src/Writer/WriterFactory.php
  • docs/zh-hk/components/http-logger.md
  • src/http-logger/.github/workflows/release.yaml
  • docs/zh-cn/guide/start/components.md
  • src/http-logger/publish/http_logger.php
  • src/http-logger/src/Middleware/HttpLogger.php
  • src/http-logger/.github/workflows/close-pull-request.yml
  • src/http-logger/src/Writer/DefaultLogWriter.php
⏰ 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). (18)
  • GitHub Check: Test on PHP 8.1 with Swoole 6.0.2
  • GitHub Check: Test on PHP 8.2 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 5.1.8
  • GitHub Check: Test on PHP 8.1 with Swoole 6.1.2
  • GitHub Check: Test on PHP 8.2 with Swoole 5.1.8
  • GitHub Check: Test on PHP 8.2 with Swoole 6.1.2
  • GitHub Check: Test on PHP 8.3 with Swoole 6.1.2
  • GitHub Check: Test on PHP 8.3 with Swoole 6.0.2
  • GitHub Check: Test on PHP 8.1 with Swoole 5.1.8
  • GitHub Check: Test on PHP 8.1 with Swoole 6.1.2
  • GitHub Check: Test on PHP 8.2 with Swoole 6.0.2
  • GitHub Check: Test on PHP 8.2 with Swoole 5.1.8
  • GitHub Check: Test on PHP 8.3 with Swoole 5.1.8
  • GitHub Check: Test on PHP 8.3 with Swoole 6.0.2
  • GitHub Check: Test on PHP 8.2 with Swoole 6.1.2
  • GitHub Check: Test on PHP 8.3 with Swoole 6.1.2
  • GitHub Check: translate

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

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

@huangdijia huangdijia merged commit e1fefda into main Nov 15, 2025
24 checks passed
@huangdijia huangdijia deleted the remove-http-logger-component branch November 15, 2025 12:06
huangdijia added a commit that referenced this pull request Nov 15, 2025
This commit removes the http-logger component from the monorepo, including:
- Deleted src/http-logger directory and all its source files
- Removed http-logger from composer.json dependencies and autoload
- Removed http-logger documentation from all language variants (en, zh-cn, zh-hk, zh-tw)
- Updated sidebar configurations across all documentation languages
- Updated component list in README and documentation

Co-authored-by: Deeka Wong <8337659+huangdijia@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants