Skip to content

Conversation

@uzulla
Copy link
Owner

@uzulla uzulla commented Nov 20, 2025

This PR removes support for two services that have been shut down and are no longer accessible:

  • chirb.it (Chirbit) - Audio sharing service
  • sassmeister.com (SassMeister) - Sass playground

Services Status Evidence

chirb.it (Chirbit)

  • Status: Service down and inaccessible
  • Evidence:
  • Multiple user reports since January 2023 (over 2 years ago)
  • Connection failures: "Failed to connect to chirb.it port 80"
  • Monitoring site: https://issitedownrightnow.com/status/chirbit.com
  • User reports indicate: "The url chirbit.com does not resolve and there is no access to the site"

sassmeister.com (SassMeister)

Changes Made

chirb.it removal:

  • Removed oEmbed provider entry from src/resources/oembed.php
  • Removed test case from tests/PagesTest.php
  • Deleted test fixtures and cached responses

sassmeister.com removal:

  • Removed adapter registration from src/ExtractorFactory.php
  • Deleted src/Adapters/Sassmeister/ directory (Extractor and Code detector)
  • Removed testSassmeister() test method from tests/PagesTest.php
  • Deleted test fixtures and cached responses

ref

Summary by CodeRabbit

リリースノート

  • Chores
    • SassMeister および Chirb.it の埋め込みサポートを廃止しました。
    • これらのプロバイダに対応するアダプタおよびエンドポイントマッピングを削除しました。
    • 関連するテストケースとテスト用フィクスチャを削除しました。

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

uzulla and others added 2 commits November 20, 2025 14:57
The chirb.it audio sharing service is no longer accessible (connection failure). This commit removes all related code and test files.

Changes:
- Remove chirb.it from oEmbed providers list
- Remove test case from PagesTest
- Delete cached responses and test fixtures

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
The sassmeister.com Sass playground service is no longer accessible (DNS resolution failure). This commit removes the adapter, related code, and test files.

Changes:
- Remove Sassmeister adapter from ExtractorFactory
- Delete Sassmeister adapter directory (Extractor and Code detector)
- Remove testSassmeister() from PagesTest
- Delete cached responses and test fixtures

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@coderabbitai
Copy link

coderabbitai bot commented Nov 20, 2025

Walkthrough

SassMeister アダプター と chirb.it oEmbed エンドポイントのサポートを削除します。削除される変更には、SassMeister アダプタークラス、そのディテクター、アダプタファクトリマッピング、oEmbed エンドポイント設定、関連テスト、および関連するテストフィクスチャが含まれます。

Changes

Cohort / File(s) Summary
SassMeister アダプター削除
src/Adapters/Sassmeister/Detectors/Code.php, src/Adapters/Sassmeister/Extractor.php
SassMeister ディテクターおよびエクストラクタークラス全体を削除しました。gist パターン検出と HTML/JavaScript ベースの埋め込み生成ロジックが削除されます。
フレームワーク設定更新
src/ExtractorFactory.php, src/resources/oembed.php
ExtractorFactory からの SassMeister アダプタマッピングと oembed.php からの chirb.it oEmbed エンドポイント(http://chirb.it/oembed.json) を削除しました。
テスト削除
tests/PagesTest.php
testSassmeister() メソッドと urlDataProvider() から http://chirb.it/7A9L9B テストケースを削除しました。
テストフィクスチャ削除
tests/cache/chirb.it.7a9l9b.php, tests/cache/chirb.it.oembed.json.0b6fc4888b3b29e8dea759841f8f9940.php, tests/cache/sassmeister.com.gist-1122e441f0434ec6eb2a.php, tests/fixtures/chirb.it.7a9l9b.php, tests/fixtures/sassmeister.com.gist-1122e441f0434ec6eb2a.php
5 つのテストフィクスチャファイルと キャッシュファイルを削除しました。これらは SassMeister gist および Chirbit oEmbed レスポンスのモック応答を含んでいました。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 分

  • 削除パターンの確認: すべての変更は一貫性のある削除パターンに従っていることを確認してください
  • 残存する参照の確認: 削除されたアダプタへの他の参照が残存していないことを確認してください
  • テストカバレッジ: テスト削除の妥当性と、他の依存するテストが影響を受けていないことを確認してください

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 タイトルは、廃止されたサービス (chirb.it と sassmeister.com) のサポート削除という主要な変更を明確に要約しており、変更セット全体と完全に関連しています。
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-closed-services

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: ASSERTIVE

Plan: Pro

Knowledge base: Disabled due to Reviews -> Disable Knowledge Base setting

📥 Commits

Reviewing files that changed from the base of the PR and between a0bdfef and 4542b4b.

📒 Files selected for processing (10)
  • src/Adapters/Sassmeister/Detectors/Code.php (0 hunks)
  • src/Adapters/Sassmeister/Extractor.php (0 hunks)
  • src/ExtractorFactory.php (0 hunks)
  • src/resources/oembed.php (0 hunks)
  • tests/PagesTest.php (0 hunks)
  • tests/cache/chirb.it.7a9l9b.php (0 hunks)
  • tests/cache/chirb.it.oembed.json.0b6fc4888b3b29e8dea759841f8f9940.php (0 hunks)
  • tests/cache/sassmeister.com.gist-1122e441f0434ec6eb2a.php (0 hunks)
  • tests/fixtures/chirb.it.7a9l9b.php (0 hunks)
  • tests/fixtures/sassmeister.com.gist-1122e441f0434ec6eb2a.php (0 hunks)
💤 Files with no reviewable changes (10)
  • tests/PagesTest.php
  • src/resources/oembed.php
  • tests/cache/sassmeister.com.gist-1122e441f0434ec6eb2a.php
  • src/Adapters/Sassmeister/Extractor.php
  • tests/fixtures/chirb.it.7a9l9b.php
  • src/ExtractorFactory.php
  • tests/cache/chirb.it.oembed.json.0b6fc4888b3b29e8dea759841f8f9940.php
  • src/Adapters/Sassmeister/Detectors/Code.php
  • tests/cache/chirb.it.7a9l9b.php
  • tests/fixtures/sassmeister.com.gist-1122e441f0434ec6eb2a.php

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

@uzulla uzulla merged commit 7aa590c into master Nov 22, 2025
8 checks passed
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