Conversation
apps/extension を extension/, apps/safari-shell を safari-shell/ に移動。 pnpm-workspace.yaml の packages を更新し、関連スクリプトと README のパス参照を新構成に追従させた。
@header-injector/core パッケージを廃止し、ソースを extension/src/lib/ 配下に移動。 全 import を相対パスへ書き換え、extension の dependencies と pnpm-workspace.yaml の packages 指定からも除外した。
shared/ports と shared/adapters/{chrome,safari} を lib/ 配下にフラットに移動し、
adapters/ 階層を排して lib/ports lib/chrome lib/safari の構造にした。
shared を参照していた相対 import も新しい lib パスへ追従させた。
apps/ と packages/ を廃止して extension/ と safari-shell/ をルート直下に並べた構成を反映し、 core ロジックが extension に統合された旨を補足した。
ルート直下に並んでいた extension/ と safari-shell/ を apps/extension, apps/safari-shell へ戻し、 pnpm-workspace.yaml の packages を apps/* に。関連スクリプト/README のパス参照も追従させた。
extension パッケージの root から src/ 直下へ manifest を移動して、 ソース関連ファイルを src/ 配下に集約した。build-extension.mjs の copyFile 参照元も追従させた。
ドメイン層であることを名前で表現するため lib/ を core/ にリネームし、 全 consumer の import path を新しいパスに追従させた。 options/ui/lib/ は別 commit で helpers/ に整理予定。
ports/ という命名と実体のずれを解消するため、KeyValueStorage と HeaderEngine の interface を core/interfaces.ts に集約し、ExtensionSettingsRepository と DNR 変換ロジックは core/settings-repository.ts と core/dnr-rules.ts に独立した。 chrome/safari adapter と background の import 経路も新構成に追従させた。
core/chrome/{storage,header-engine}.ts と core/safari/* をそれぞれ
platforms/chrome.ts と platforms/safari.ts に統合し、1 ブラウザ 1 ファイルにした。
ChromeExtensionLike と BrowserExtensionLike の名前衝突を避けるため、
storage 用と header-engine 用で interface を分けて命名し直した。
options/ui/ サブディレクトリを廃止し、view.tsx (旧 options-view.tsx) と components/ を options/ 直下にフラットに配置した。 options/ui/lib/ は options/helpers/ にリネームして src/core/ との lib 命名重複を解消し、関連する import の相対パスも追従させた。
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.