「アドレス」や「Addressableのグループ」を基にラベルやバージョンを振れるように #78

Haruma-K · 2025-07-01T05:28:02Z

「アドレス」や「Addressableのグループ」を基にラベルやバージョンを振れるようにしました
具体的な変更内容は以下のリリースノートを参照してください

リリースノート

新機能

新しいAsset Filter
- Address Filter: アドレスパターンに基づいてアセットをフィルタリング（AddressRuleでは使用不可）
- Addressable Group Filter: アセットが所属するグループに基づいてフィルタリング（AddressRuleでは使用不可）
新しいProvider
- Address Based Label/Version Provider: アセットのアドレスからラベル/バージョンを生成
- Addressable Asset Group Name Based Label/Version Provider: グループ名からラベル/バージョンを生成

破壊的変更とマイグレーション

Asset Filterに以下のように引数が増えたので、カスタムアセットフィルタを作っている場合は引数を追加してください

// 旧
bool IsMatch(string assetPath, Type assetType, bool isFolder)

// 新 
bool IsMatch(string assetPath, Type assetType, bool isFolder, string address, AddressableAssetGroup addressableAssetGroup)

Label Provider / Version Providerに以下のように引数が増えたので、カスタムプロバイダーを作っている場合は引数を追加してください

// 旧
string Provide(string assetPath, Type assetType, bool isFolder)

// 新
string Provide(string assetPath, Type assetType, bool isFolder, string address, AddressableAssetGroup addressableAssetGroup)

IProviderは現在IAddressProviderでしか使用されていないため、 IProviderを削除し、そのメソッドをIAddressProviderに直接定義するように変更。 - IAddressProviderにSetup、Provide、GetDescriptionメソッドを直接定義 - AssetPathBasedProviderからIProvider<string>の実装を削除 - AssetPathBasedVersionProvider/LabelProviderをbaseクラスのメソッドを呼び出すように修正 - IProvider.csとそのメタファイルを削除 - テストコードのIProvider<string>へのキャストを削除 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>

- IAssetFilter.IsMatchにaddressとaddressableAssetGroupパラメータを追加 - AssetGroup/AssetGroupObservableListのContainsメソッドを更新 - AddressRuleではアドレスが未定のためnullを渡す - LabelRule/VersionRuleでは実際のアドレスとグループ情報を渡す - AssetFilterBase/AssetFilterAssetに新シグネチャを実装（後方互換性を維持） - CustomAssetFilterを新しいシグネチャに更新これにより、フィルター実装でアドレスやグループ情報に基づいたより柔軟なフィルタリングが可能になりました。 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>

AssetFilterBaseの抽象メソッドIsMatchにaddressとaddressableAssetGroup パラメータを追加し、全ての継承クラスで対応する修正を実施 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>

全てのAssetFilterテストファイルでIsMatchメソッドの新しいシグネチャに対応 - addressとaddressableAssetGroupパラメータを追加 - 両パラメータにはnullを指定 - 必要なusing文を追加 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>

アドレスに対して正規表現でフィルタリングするAddressBasedAssetFilterを実装 - RegexBasedAssetFilterを参考に実装 - アドレスパラメータを使用してマッチングを実行 - 対応するエディタUI用のDrawerも追加 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>

特定のAddressableAssetGroupに属するアセットをフィルタリングする機能を実装 - AddressableAssetGroupの参照をListablePropertyで保持 - 専用のListablePropertyGUIクラスも追加 - エディタUI用のDrawerを実装 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>

- RestrictedToRulesAttribute を追加し、特定のRuleタイプでのみ使用可能なAssetFilterを指定できるように - AddressBasedAssetFilter と AddressableAssetGroupBasedAssetFilter に制限を追加（Label、Versionルールでのみ使用可能） - AssetGroupPanelPresenter と AssetGroupCollectionPanelPresenter のコンストラクタでRuleTypeを受け取るように変更 - 各RulePresenterから適切なRuleTypeを渡すように修正これにより、AddressRuleのGUIではAddressBasedAssetFilterとAddressableAssetGroupBasedAssetFilterが選択できなくなりました。 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>

アドレスベースでラベルとバージョンを生成するプロバイダーを実装: - AddressBasedProvider: 共通ロジックを持つベースクラス - AddressBasedLabelProvider: アドレスからラベルを生成 - AddressBasedVersionProvider: アドレスからバージョンを生成 AssetPathBasedProviderと同様の設計パターンを採用し、フルアドレスまたはファイル名のみの使用、正規表現による置換機能をサポート 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>

アドレッサブルアセットグループ名でラベルとバージョンを生成するプロバイダーを実装: - AddressableAssetGroupNameBasedProvider: 共通ロジックを持つベースクラス - AddressableAssetGroupNameBasedLabelProvider: グループ名からラベルを生成 - AddressableAssetGroupNameBasedVersionProvider: グループ名からバージョンを生成グループ名をそのまま使用、または正規表現による置換機能をサポート 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>

コミット67a0f76以降の変更内容をドキュメントに反映: Asset Filter: - Address Filter: アドレスベースのフィルタリング - Addressable Group Filter: グループベースのフィルタリング - 両フィルタはAddressRuleでは使用不可 Provider: - Address Based Label/Version Provider: アドレスからラベル/バージョンを生成 - Addressable Asset Group Name Based Label/Version Provider: グループ名からラベル/バージョンを生成 API変更: - IAssetFilter.IsMatchメソッドにaddressとaddressableAssetGroupパラメータを追加 - Provider.ProvideメソッドにaddressとaddressableAssetGroupパラメータを追加 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>

IsMatchメソッドの古いシグネチャ（address, addressableAssetGroupパラメータなし）を削除し、新しいシグネチャのみを使用するように変更。これにより全てのAssetFilterで新しいパラメータを使用できるようになった。 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>

- AddressBasedLabelProviderDrawer - AddressBasedVersionProviderDrawer - AddressableAssetGroupNameBasedLabelProviderDrawer - AddressableAssetGroupNameBasedVersionProviderDrawer これらのDrawerクラスにより、エディタ上で各Providerの設定をGUIで行えるようになります。 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>

- AddressBasedLabelProviderDrawer - AddressBasedVersionProviderDrawer - AddressableAssetGroupNameBasedLabelProviderDrawer - AddressableAssetGroupNameBasedVersionProviderDrawer これらのDrawerクラスにより、エディタ上で各Providerの設定をGUIで行えるようになります。

…tGameEntertainment/SmartAddresser into feature/group-based-filter

- AddressBasedProviderクラスからUseFullAddressプロパティとフィールドを削除 - 常にフルアドレスを使用するように変更 - 関連するDrawerクラスからUseFullAddressのUI要素を削除

hkmt-mmy · 2025-07-01T08:35:36Z

...tor/Core/Tools/Addresser/LayoutRuleEditor/LabelRuleEditor/AddressBasedLabelProviderDrawer.cs

+            var replaceWithRegexLabel = ObjectNames.NicifyVariableName(nameof(target.ReplaceWithRegex));
+            target.ReplaceWithRegex = EditorGUILayout.Toggle(replaceWithRegexLabel, target.ReplaceWithRegex);
+
+            GUI.enabled = target.ReplaceWithRegex;


enabledに関しても DisabledGroupScope を使ったほうが良いかも

hkmt-mmy · 2025-07-01T08:41:30Z

...dresser/Editor/Core/Tools/Addresser/Shared/AssetGroups/AssetGroupCollectionPanelPresenter.cs

            new Dictionary<string, AssetGroupPanelPresenter>();

        private readonly AutoIncrementHistory _history;
+        private readonly RuleType? _ruleContext;


変数名がContextなのは何かの名残でしょうか。普通に _ruleType で良さそうに思います

hkmt-mmy

AddressやGroupベースで動作するケースに関してもテストを書いたほうが良さそうです

hkmt-mmy · 2025-07-01T08:51:44Z

README_JA.md

 |---------------------------------|-----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|
 | Constant Label Provider         | 固定値でラベルを付与します。<br><br>Label<br>付与するラベルの名前。                                                                                                                                                                                                                                  |
 | Asset Path Based Label Provider | 対象アセットのアセットパスを元にラベルを付与します。<br><br>**Source**<br>以下のパターンからラベルを指定します。<br>・File Name: 拡張子付きファイル名<br>・File Name Without Extensions: 拡張子なしアセット名<br>・Asset Path: アセットパス<br><br>**Replace With Regex**<br>チェックをつけると、Pattern を元に正規表現を作成し、Source を Replacement で Regex.Replace したものをラベルとします。 |
+| Address Based Label Provider | 対象アセットのアドレスを元にラベルを付与します。<br><br>**Use Full Address**<br>チェックをつけると、フルアドレスを使用します。チェックを外すと、最後の'/'以降の部分（ファイル名）のみを使用します。<br><br>**Replace With Regex**<br>チェックをつけると、Pattern を元に正規表現を作成し、アドレスを Replacement で Regex.Replace したものをラベルとします。 |


AddressBasedLabelProviderDrawerとAddressBasedVersionProviderDrawerを確認しましたが "Use Full Address" という設定項目はなさそうですが、見るところ間違えてますかね

不要だったのでコードから消したやつでした、対応します。

- AddressBasedProvider関連のテスト - AddressableAssetGroupNameBasedProvider関連のテスト - LabelProvider及びVersionProviderのテスト - AssetFilter実装のテスト - READMEの微修正

AddressBasedとAddressableAssetGroupNameBasedの各Provider系テストから、実用性の低い複雑なRegexパターンのテストケースを削除しました。基本的な動作確認に必要なテストケースは残しています。 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>

- 空のアドレスのテストケース（IsMatch_WithEmptyAddress_ReturnsFalse）を削除 - 複雑な正規表現パターンのテストケース（IsMatch_WithComplexPattern_WorksCorrectly）を削除 - メソッド名を改善（SetupForMatching_WithInvalidRegex_DoesNotThrow） 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>

全てのProviderDrawerクラスで、GUI.enabledの直接的な操作から EditorGUI.DisabledScopeを使用したusing文の方式に変更。これによりコードがより現代的なUnityのベストプラクティスに準拠。 🤖 Generated with Claude Code Co-Authored-By: Claude <[email protected]>

AssetGroupCollectionPanelPresenterとAssetGroupPanelPresenterで使用されていた _ruleContextフィールド名を_ruleTypeに統一。パラメータ名も同様に変更。

Haruma-K · 2025-07-03T02:19:50Z

レビューいただいた点、修正しました。
再レビューお願いします 🙏

Haruma-K and others added 15 commits June 27, 2025 18:31

LabelRuleにアドレスとAddressableAssetGroupNameを渡すように変更

67a0f76

VersionRuleにアドレスとAddressableAssetGroupNameを渡すように変更

1c7fd67

remp

d918926

不要なファイルを削除

e6f86e9

Haruma-K requested review from hkmt-mmy and qua-iy July 1, 2025 05:28

Haruma-K and others added 3 commits July 1, 2025 14:42

Merge branch 'feature/wip_group-based-filter' of github.com:CyberAgen…

d00926e

…tGameEntertainment/SmartAddresser into feature/group-based-filter

Haruma-K marked this pull request as draft July 1, 2025 05:54

Haruma-K marked this pull request as ready for review July 1, 2025 07:44

hkmt-mmy reviewed Jul 1, 2025

View reviewed changes

Haruma-K and others added 3 commits July 1, 2025 22:08

test: 各種Provider及びAssetFilterのユニットテストを追加

cb5226c

- AddressBasedProvider関連のテスト - AddressableAssetGroupNameBasedProvider関連のテスト - LabelProvider及びVersionProviderのテスト - AssetFilter実装のテスト - READMEの微修正

不要なテストを削除

a380d75

Haruma-K and others added 3 commits July 3, 2025 10:51

refactor: AssetGroupPresenterクラスのruleContextフィールドをruleTypeに変更

2b1cfe7

AssetGroupCollectionPanelPresenterとAssetGroupPanelPresenterで使用されていた _ruleContextフィールド名を_ruleTypeに統一。パラメータ名も同様に変更。

Haruma-K requested a review from hkmt-mmy July 3, 2025 02:19

hkmt-mmy approved these changes Jul 3, 2025

View reviewed changes

Base automatically changed from feature/fix_readme to main July 3, 2025 03:31

Haruma-K merged commit af7fbfc into main Jul 3, 2025
1 check passed

Haruma-K deleted the feature/group-based-filter branch July 3, 2025 03:32

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

「アドレス」や「Addressableのグループ」を基にラベルやバージョンを振れるように #78

「アドレス」や「Addressableのグループ」を基にラベルやバージョンを振れるように #78

Uh oh!

Haruma-K commented Jul 1, 2025

Uh oh!

hkmt-mmy Jul 1, 2025 •

edited

Loading

Uh oh!

hkmt-mmy Jul 1, 2025

Uh oh!

hkmt-mmy left a comment

Uh oh!

hkmt-mmy Jul 1, 2025

Uh oh!

Haruma-K Jul 1, 2025

Uh oh!

Haruma-K commented Jul 3, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

「アドレス」や「Addressableのグループ」を基にラベルやバージョンを振れるように #78

「アドレス」や「Addressableのグループ」を基にラベルやバージョンを振れるように #78

Uh oh!

Conversation

Haruma-K commented Jul 1, 2025

リリースノート

新機能

破壊的変更とマイグレーション

Uh oh!

hkmt-mmy Jul 1, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

hkmt-mmy Jul 1, 2025

Choose a reason for hiding this comment

Uh oh!

hkmt-mmy left a comment

Choose a reason for hiding this comment

Uh oh!

hkmt-mmy Jul 1, 2025

Choose a reason for hiding this comment

Uh oh!

Haruma-K Jul 1, 2025

Choose a reason for hiding this comment

Uh oh!

Haruma-K commented Jul 3, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

hkmt-mmy Jul 1, 2025 •

edited

Loading