feat: Addressables REST module for hot-update asset management - #55
feat: Addressables REST module for hot-update asset management#55ShutovKS wants to merge 3 commits into
Conversation
- Add Addressables to SkillCategory enum - Implement AddressablesSkills.cs with reflection-based optional dependency - Skills: check_installed, group_list, group_create, group_delete, group_add_entry, profile_get, profile_set, build - Zero hard references to Unity.Addressables.Editor assembly - Follows HybridCLRSkills reflection pattern for optional packages
|
Okay, I'll review your PR when I have time. |
|
Tested in Unity 6.3 LTS against Addressables 3.1.0. The direction is useful, but this is not merge-ready. Current reflection assumes APIs that do not match 3.1.0: IsDefaultGroup is a method, entries is not IList, CreateGroup needs the params Type[] argument, and BuildPlayerContent has multiple overloads. In real execution, group_create and build fail, entryCount/default flags are wrong, version reports 0.0.0.0, and group_delete was able to delete Default Local Group. Please correct the reflection bindings, make default-group protection reliable, and add endpoint tests against a supported Addressables version. |
|
Review fixes are now on this PR branch only (head
Verification on Unity 6000.3.11f1 + Addressables 3.1.0: full EditMode |
Summary
Adds an optional reflection-based Addressables REST module, validated against
com.unity.addressables3.1.0 without a hard assembly dependency.Endpoints
addressables_check_installedaddressables_group_listaddressables_group_createaddressables_group_deleteaddressables_group_add_entryaddressables_profile_getaddressables_profile_setaddressables_buildAddressables 3.1 compatibility
IsDefaultGroup()correctly and also verifiessettings.DefaultGroup.ICollection.CreateGroupoverload.BundledAssetGroupSchemaandContentUpdateGroupSchema.BuildPlayerContent(out result)and surfacesresult.Error.PackageInfo.FindForAssembly.Verification
220/220passed.3.1.0.git diff --check: passed.