refactor: 保守性改善(コード重複・logger.bind・docstring 乖離・~ 展開ほか)
内容
コードレビューで見つかった軽微な改善点のまとめ。
コード整理
-
config.load_config()とConfigReader._load_file()がほぼ同一実装(load_config は本体コードから未使用)。片方に統合する -
generator.run()の ConfigKind 二重変換(cli.py:143-144で変換済みなのにgenerator.py:281で再変換)。sys.exit(1)とtyper.Exitの混在も統一 -
reader.pyの_print_fetch_tasks/_print_backup_tasksがほぼコピー。kind を引数に取る1関数へ統合
挙動の改善
-
reader._expand_env_varsにos.path.expanduserを追加(現状~/backupは./~/backup扱い、$HOMEは動くという非対称) -
verify_directoryのリモート判定":" in pathは Windows ドライブレターを誤判定(classifiers は OS Independent) -
env.pyのホスト名正規化衝突(my-server.comとmy.server.comが同一環境変数名)をドキュメントに明記
その他
-
logger.get_logger()のlogger.bind(name=...)は extra に入るだけでフォーマットの{name}には効いていない。戻り値型注釈"logger"も不正確 - docstring の出力例が実装と乖離(例: fetcher は "Fetching logs from ... to ..." と記載だが実出力は "Fetching logs" のみ)
-
reader.pyのテストカバレッジ 59%。_get_default_config_pathの探索順・_expand_env_varsのテスト追加