Skip to content

fix(root): ローカル updatedAt キャッシュ機能を削除#74

Merged
HappyOnigiri merged 1 commit intomainfrom
fix/remove-local-updatedat-cache
Mar 12, 2026
Merged

fix(root): ローカル updatedAt キャッシュ機能を削除#74
HappyOnigiri merged 1 commit intomainfrom
fix/remove-local-updatedat-cache

Conversation

@HappyOnigiri
Copy link
Owner

@HappyOnigiri HappyOnigiri commented Mar 12, 2026

概要 / Summary

  • PR updatedAt のローカルキャッシュ早期スキップ機能を撤去し、常に通常フローで PR を処理するように戻しました。

変更内容 / Changes

  • src/cache_manager.py を削除し、src/auto_fixer.py から load_cache / save_cache / get_cached_updated_at / set_cached_updated_at の依存を削除
  • process_repo から updated_at_cache 引数と updatedAt 一致時の early skip 分岐を削除
  • main からキャッシュのロード・保存処理を削除し、process_repo 呼び出しシグネチャを整理
  • tests/test_auto_fixer.py からキャッシュ前提のモック・引数期待値・早期スキップ関連テストを削除
  • tests/test_cache_manager.pytests/test_imports.pycache_manager import 検証を削除

テスト / Testing

  • make ci

関連 issues / Related issues

  • なし

Summary by CodeRabbit

  • Refactor

    • キャッシング機構を削除し、リポジトリ処理フローを簡素化しました。すべてのリポジトリが以前のキャッシュを使わずに処理されるようになります。
  • Chores

    • ローカルキャッシュマネージャーモジュールと関連するテストを削除しました。

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 12, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 9541f9ba-de5b-47e9-b0d1-118bf5822451

📥 Commits

Reviewing files that changed from the base of the PR and between c3056e7 and 3f8b46f.

📒 Files selected for processing (5)
  • src/auto_fixer.py
  • src/cache_manager.py
  • tests/test_auto_fixer.py
  • tests/test_cache_manager.py
  • tests/test_imports.py
💤 Files with no reviewable changes (3)
  • tests/test_cache_manager.py
  • tests/test_imports.py
  • src/cache_manager.py

📝 Walkthrough

Walkthrough

キャッシュ管理機能の統合を削除。src/cache_manager.py モジュール全体と、src/auto_fixer.py 内のキャッシュ処理ロジックを削除し、PR更新時刻のローカルキャッシング機構を廃止。関連するテストも削除・更新。

Changes

Cohort / File(s) Summary
キャッシュ管理モジュール削除
src/cache_manager.py, tests/test_cache_manager.py
キャッシュマネージャー全体とそのユニットテストを削除。CacheData型、キャッシュロード/セーブ関数、更新日時の取得/設定関数を廃止。
自動修正ロジック簡略化
src/auto_fixer.py
キャッシュインポートと使用箇所を削除。process_repo() 署名から updated_at_cache パラメータを削除。キャッシュ駆動スキップロジックを廃止し、全リポジトリを直線的に処理。CI確認処理でRESTAPIデータをリスト統一処理。
テスト更新
tests/test_auto_fixer.py, tests/test_imports.py
キャッシュ関連パッチとパラメータを削除。process_repo() 呼び出し元を調整。キャッシュマネージャーのインポートテストを削除。

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Suggested labels

refix:done

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 30.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed プルリクエストのタイトルはローカルの updatedAt キャッシュ機能削除という変更内容を明確に反映しており、実装内容と完全に一致しています。
Description check ✅ Passed プルリクエストの説明は、テンプレートの全セクション(概要、変更内容、テスト、関連issues)を記載しており、変更内容の詳細が適切に記述されています。

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/remove-local-updatedat-cache
📝 Coding Plan
  • Generate coding plan for human review comments

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

@HappyOnigiri HappyOnigiri added the refix:done Refix finished review checks/fixes for now. label Mar 12, 2026
@HappyOnigiri HappyOnigiri merged commit 070eed0 into main Mar 12, 2026
2 checks passed
@HappyOnigiri HappyOnigiri deleted the fix/remove-local-updatedat-cache branch March 12, 2026 21:30
@HappyOnigiri HappyOnigiri added refix:auto-merge-requested Refix has requested auto-merge for this PR. refix:merged PR has been merged after Refix auto-merge. and removed refix:auto-merge-requested Refix has requested auto-merge for this PR. labels Mar 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refix:done Refix finished review checks/fixes for now. refix:merged PR has been merged after Refix auto-merge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant