chore: upgrade to latest sdk packages#14
Open
JaredZanette-unity3d wants to merge 7 commits into
Open
Conversation
There was a problem hiding this comment.
Pull request overview
Upgrades the Unity commerce SDK packages to version 1.2.1 and refreshes Unity Addressables/URP/scene and related project settings to match the updated packages and editor state.
Changes:
- Bumps
com.unity.walmart-commerce-sdkto1.2.1(withcom.unity.commerce-sdkbecoming transitive) and updates the lockfile accordingly. - Regenerates Addressables settings/assets (groups, schemas, profile vars, link.xml) and updates build/editor settings references.
- Adds editor utilities for Addressables validation/debugging and enables detailed logging in-editor.
Reviewed changes
Copilot reviewed 43 out of 62 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| Projects/UntitledCatGame/ProjectSettings/ScriptableBuildPipeline.json | Adds Scriptable Build Pipeline settings file. |
| Projects/UntitledCatGame/ProjectSettings/ProjectSettings.asset | Updates Android application identifier formatting/value. |
| Projects/UntitledCatGame/ProjectSettings/Packages/com.unity.services.core/Settings.json | Unity Services Core settings file present (content unchanged/empty in this diff view). |
| Projects/UntitledCatGame/ProjectSettings/EditorBuildSettings.asset | Updates Addressables config object GUID reference. |
| Projects/UntitledCatGame/Packages/packages-lock.json | Updates commerce SDK package versions/relationships to 1.2.1. |
| Projects/UntitledCatGame/Packages/manifest.json | Upgrades Walmart commerce SDK to 1.2.1 and removes direct commerce-sdk dependency. |
| Projects/UntitledCatGame/Assets/Settings/UniversalRP.asset | URP asset updated with new serialized fields from Unity/URP versions. |
| Projects/UntitledCatGame/Assets/Scripts/CatGameCommerceOpportunityController.cs | Minor import ordering / debug comment / whitespace update. |
| Projects/UntitledCatGame/Assets/Scenes/GameScene.unity | Scene reserialized with updated component versions/fields (colliders, camera, roots, etc.). |
| Projects/UntitledCatGame/Assets/Resources/Unity Immersive Commerce/ARSConfiguration.asset | Updates relay headers configuration (now empty list). |
| Projects/UntitledCatGame/Assets/Resources/Commerce Opportunity Editor/USS Variables/42ca99e9-e6ea-4974-9436-910fdd698449.uss.meta | Removes generated USS variables meta. |
| Projects/UntitledCatGame/Assets/Resources/Commerce Opportunity Editor/USS Variables/42ca99e9-e6ea-4974-9436-910fdd698449.uss | Removes generated USS variables stylesheet. |
| Projects/UntitledCatGame/Assets/Resources/Commerce Opportunity Editor/USS Variables.meta | Removes USS Variables folder meta. |
| Projects/UntitledCatGame/Assets/Resources/Commerce Opportunity Editor/ProductPackagesDatastore.asset.meta | Updates GUID for the datastore asset meta. |
| Projects/UntitledCatGame/Assets/Resources/Commerce Opportunity Editor/ProductPackagesDatastore.asset | Clears stored last sync timestamp. |
| Projects/UntitledCatGame/Assets/Resources/Commerce Opportunity Editor/ConfigDatastore.asset.meta | Updates GUID for the config datastore meta. |
| Projects/UntitledCatGame/Assets/Resources/Commerce Opportunity Editor/ConfigDatastore.asset | Updates config structure (organization configuration nesting). |
| Projects/UntitledCatGame/Assets/Resources/Commerce Opportunity Editor/CommerceOpportunityDatastore.asset | Updates commerce opportunity/style template data and SDK version reference. |
| Projects/UntitledCatGame/Assets/Editor/VerifyAddressablesSetup.cs.meta | Adds meta for new editor utility. |
| Projects/UntitledCatGame/Assets/Editor/VerifyAddressablesSetup.cs | Adds Addressables Android setup verification menu tool. |
| Projects/UntitledCatGame/Assets/Editor/ValidateAddressablesPaths.cs.meta | Adds meta for new editor utility. |
| Projects/UntitledCatGame/Assets/Editor/ValidateAddressablesPaths.cs | Adds Addressables profile/group path validation menu tool. |
| Projects/UntitledCatGame/Assets/Editor/EnableDetailedLogging.cs.meta | Adds meta for new editor utility. |
| Projects/UntitledCatGame/Assets/Editor/EnableDetailedLogging.cs | Adds an editor-time hook to force stack trace verbosity. |
| Projects/UntitledCatGame/Assets/Editor/AddressablesDebugBuild.cs.meta | Adds meta for new editor utility. |
| Projects/UntitledCatGame/Assets/Editor/AddressablesDebugBuild.cs | Adds Addressables build command with extensive debug logging. |
| Projects/UntitledCatGame/Assets/Editor.meta | Updates GUID for Assets/Editor folder meta. |
| Projects/UntitledCatGame/Assets/AddressableAssetsData/Windows/addressables_content_state.bin | Removes Windows content-state artifact from repo. |
| Projects/UntitledCatGame/Assets/AddressableAssetsData/ProfileDataSourceSettings.asset.meta | Updates GUID for profile data source settings meta. |
| Projects/UntitledCatGame/Assets/AddressableAssetsData/ProfileDataSourceSettings.asset | Adds new serialized environment fields for Addressables settings. |
| Projects/UntitledCatGame/Assets/AddressableAssetsData/link.xml.meta | Updates importer/meta for new link.xml. |
| Projects/UntitledCatGame/Assets/AddressableAssetsData/link.xml | Adds link.xml to preserve Addressables/Commerce/UIElements types for stripping (IL2CPP). |
| Projects/UntitledCatGame/Assets/AddressableAssetsData/DefaultObject.asset.meta | Updates GUID for Addressables DefaultObject meta. |
| Projects/UntitledCatGame/Assets/AddressableAssetsData/DefaultObject.asset | Updates Addressables settings GUID reference. |
| Projects/UntitledCatGame/Assets/AddressableAssetsData/DataBuilders/BuildScriptVirtualMode.asset.meta | Updates GUID for data builder meta. |
| Projects/UntitledCatGame/Assets/AddressableAssetsData/DataBuilders/BuildScriptPackedPlayMode.asset.meta | Updates GUID for data builder meta. |
| Projects/UntitledCatGame/Assets/AddressableAssetsData/DataBuilders/BuildScriptPackedMode.asset.meta | Updates GUID for data builder meta. |
| Projects/UntitledCatGame/Assets/AddressableAssetsData/DataBuilders/BuildScriptFastMode.asset.meta | Updates GUID for data builder meta. |
| Projects/UntitledCatGame/Assets/AddressableAssetsData/DataBuilders.meta | Updates GUID for DataBuilders folder meta. |
| Projects/UntitledCatGame/Assets/AddressableAssetsData/AssetGroupTemplates/Packed Assets.asset.meta | Updates GUID for group template meta. |
| Projects/UntitledCatGame/Assets/AddressableAssetsData/AssetGroupTemplates/Packed Assets.asset | Reserializes group template asset (schema object IDs + settings reference). |
| Projects/UntitledCatGame/Assets/AddressableAssetsData/AssetGroupTemplates.meta | Updates GUID for AssetGroupTemplates folder meta. |
| Projects/UntitledCatGame/Assets/AddressableAssetsData/AssetGroups/Schemas/Default Local Group_PlayerDataGroupSchema.asset.meta | Updates GUID for schema meta. |
| Projects/UntitledCatGame/Assets/AddressableAssetsData/AssetGroups/Schemas/Default Local Group_PlayerDataGroupSchema.asset | Updates schema name/group reference for new Addressables GUIDs. |
| Projects/UntitledCatGame/Assets/AddressableAssetsData/AssetGroups/Schemas/Default Local Group_ContentUpdateGroupSchema.asset.meta | Updates GUID for schema meta. |
| Projects/UntitledCatGame/Assets/AddressableAssetsData/AssetGroups/Schemas/Default Local Group_ContentUpdateGroupSchema.asset | Updates schema group reference for new Addressables GUIDs. |
| Projects/UntitledCatGame/Assets/AddressableAssetsData/AssetGroups/Schemas/Default Local Group_BundledAssetGroupSchema.asset.meta | Updates GUID for schema meta. |
| Projects/UntitledCatGame/Assets/AddressableAssetsData/AssetGroups/Schemas/Default Local Group_BundledAssetGroupSchema.asset | Updates bundled schema group/build&load path variable IDs. |
| Projects/UntitledCatGame/Assets/AddressableAssetsData/AssetGroups/Schemas.meta | Updates GUID for Schemas folder meta. |
| Projects/UntitledCatGame/Assets/AddressableAssetsData/AssetGroups/Default Local Group.asset.meta | Updates GUID for group meta. |
| Projects/UntitledCatGame/Assets/AddressableAssetsData/AssetGroups/Default Local Group.asset | Reserializes default local group and settings reference (now empty entries list). |
| Projects/UntitledCatGame/Assets/AddressableAssetsData/AssetGroups/Built In Data.asset.meta | Updates GUID for built-in group meta. |
| Projects/UntitledCatGame/Assets/AddressableAssetsData/AssetGroups/Built In Data.asset | Reserializes built-in group GUID/settings + adds restriction flags. |
| Projects/UntitledCatGame/Assets/AddressableAssetsData/AssetGroups.meta | Updates GUID for AssetGroups folder meta. |
| Projects/UntitledCatGame/Assets/AddressableAssetsData/Android/addressables_content_state.bin.meta | Updates GUID for Android content-state meta. |
| Projects/UntitledCatGame/Assets/AddressableAssetsData/Android/addressables_content_state.bin | Updates Android content-state artifact pointer (LFS) for new build. |
| Projects/UntitledCatGame/Assets/AddressableAssetsData/Android.meta | Updates GUID for Android folder meta. |
| Projects/UntitledCatGame/Assets/AddressableAssetsData/AddressableAssetSettings.asset.meta | Updates GUID for main Addressables settings meta. |
| Projects/UntitledCatGame/Assets/AddressableAssetsData/AddressableAssetSettings.asset | Reserializes Addressables settings (profile vars, group GUIDs, hashes, request concurrency, etc.). |
| Projects/UntitledCatGame/Assets/AddressableAssetsData.meta | Updates GUID for AddressableAssetsData folder meta. |
| Projects/UntitledCatGame/3b97e878caae54470becd6724de1c2db/walmartgroup_assets_all_4b8102058b828a1e6de02def933d84c4.bundle | Removes built Addressables bundle artifact from repo. |
| Projects/UntitledCatGame/3b97e878caae54470becd6724de1c2db/commerceopportunitygroup_assets_all_8e8ee885561a7346fcbf146fd83ff5b6.bundle | Removes built Addressables bundle artifact from repo. |
Files not reviewed (18)
- Projects/UntitledCatGame/Assets/AddressableAssetsData.meta: Language not supported
- Projects/UntitledCatGame/Assets/AddressableAssetsData/AddressableAssetSettings.asset.meta: Language not supported
- Projects/UntitledCatGame/Assets/AddressableAssetsData/Android.meta: Language not supported
- Projects/UntitledCatGame/Assets/AddressableAssetsData/Android/addressables_content_state.bin.meta: Language not supported
- Projects/UntitledCatGame/Assets/AddressableAssetsData/AssetGroupTemplates.meta: Language not supported
- Projects/UntitledCatGame/Assets/AddressableAssetsData/AssetGroupTemplates/Packed Assets.asset.meta: Language not supported
- Projects/UntitledCatGame/Assets/AddressableAssetsData/AssetGroups.meta: Language not supported
- Projects/UntitledCatGame/Assets/AddressableAssetsData/AssetGroups/Built In Data.asset.meta: Language not supported
- Projects/UntitledCatGame/Assets/AddressableAssetsData/AssetGroups/Default Local Group.asset.meta: Language not supported
- Projects/UntitledCatGame/Assets/AddressableAssetsData/AssetGroups/Schemas.meta: Language not supported
- Projects/UntitledCatGame/Assets/AddressableAssetsData/AssetGroups/Schemas/Default Local Group_BundledAssetGroupSchema.asset.meta: Language not supported
- Projects/UntitledCatGame/Assets/AddressableAssetsData/AssetGroups/Schemas/Default Local Group_ContentUpdateGroupSchema.asset.meta: Language not supported
- Projects/UntitledCatGame/Assets/AddressableAssetsData/DataBuilders.meta: Language not supported
- Projects/UntitledCatGame/Assets/AddressableAssetsData/DataBuilders/BuildScriptFastMode.asset.meta: Language not supported
- Projects/UntitledCatGame/Assets/AddressableAssetsData/DataBuilders/BuildScriptPackedMode.asset.meta: Language not supported
- Projects/UntitledCatGame/Assets/AddressableAssetsData/DataBuilders/BuildScriptPackedPlayMode.asset.meta: Language not supported
- Projects/UntitledCatGame/Assets/AddressableAssetsData/DataBuilders/BuildScriptVirtualMode.asset.meta: Language not supported
- Projects/UntitledCatGame/Assets/AddressableAssetsData/DefaultObject.asset.meta: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 39 out of 58 changed files in this pull request and generated 3 comments.
Files not reviewed (18)
- Projects/UntitledCatGame/Assets/AddressableAssetsData.meta: Language not supported
- Projects/UntitledCatGame/Assets/AddressableAssetsData/AddressableAssetSettings.asset.meta: Language not supported
- Projects/UntitledCatGame/Assets/AddressableAssetsData/Android.meta: Language not supported
- Projects/UntitledCatGame/Assets/AddressableAssetsData/Android/addressables_content_state.bin.meta: Language not supported
- Projects/UntitledCatGame/Assets/AddressableAssetsData/AssetGroupTemplates.meta: Language not supported
- Projects/UntitledCatGame/Assets/AddressableAssetsData/AssetGroupTemplates/Packed Assets.asset.meta: Language not supported
- Projects/UntitledCatGame/Assets/AddressableAssetsData/AssetGroups.meta: Language not supported
- Projects/UntitledCatGame/Assets/AddressableAssetsData/AssetGroups/Built In Data.asset.meta: Language not supported
- Projects/UntitledCatGame/Assets/AddressableAssetsData/AssetGroups/Default Local Group.asset.meta: Language not supported
- Projects/UntitledCatGame/Assets/AddressableAssetsData/AssetGroups/Schemas.meta: Language not supported
- Projects/UntitledCatGame/Assets/AddressableAssetsData/AssetGroups/Schemas/Default Local Group_BundledAssetGroupSchema.asset.meta: Language not supported
- Projects/UntitledCatGame/Assets/AddressableAssetsData/AssetGroups/Schemas/Default Local Group_ContentUpdateGroupSchema.asset.meta: Language not supported
- Projects/UntitledCatGame/Assets/AddressableAssetsData/DataBuilders.meta: Language not supported
- Projects/UntitledCatGame/Assets/AddressableAssetsData/DataBuilders/BuildScriptFastMode.asset.meta: Language not supported
- Projects/UntitledCatGame/Assets/AddressableAssetsData/DataBuilders/BuildScriptPackedMode.asset.meta: Language not supported
- Projects/UntitledCatGame/Assets/AddressableAssetsData/DataBuilders/BuildScriptPackedPlayMode.asset.meta: Language not supported
- Projects/UntitledCatGame/Assets/AddressableAssetsData/DataBuilders/BuildScriptVirtualMode.asset.meta: Language not supported
- Projects/UntitledCatGame/Assets/AddressableAssetsData/DefaultObject.asset.meta: Language not supported
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
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.
Proposed change(s)
Upgrading the sdk packages to the latest version (1.2.1).
Upgrading the addressables included.
Adding in tools to help with addressable build issues.
Useful links (GitHub issues, JIRA tickets, forum threads, etc.)
Types of change(s)
Testing and Verification
Tested in editor and on android device
Test Configuration
Checklist
mainbranchmainbranchOther comments