Skip to content

Conversation

@GODrums
Copy link
Collaborator

@GODrums GODrums commented Oct 25, 2025

Steam moved stored asset data into two separate child inventories (separation of trade protected and normal items). This causes the extension to completely fail any item detection in the item grid:
Screenshot 2025-10-25 at 17 41 16

This PR addresses this by adapting the way asset data is retrieved from the stored g_ActiveInventory.
image


Note

Update inventory asset lookup to use app-wide child inventories with trade-protected detection, and align types/constants.

  • Inventory item resolution:
    • Determine contextId via new isTradeProtected flag on ItemHolderMetadata (checks provisional_item).
    • Fetch assets from g_ActiveInventory.m_rgChildInventories[contextId].m_rgAssets[assetId].description instead of top-level.
  • Types:
    • Introduce CAppwideInventory with m_rgChildInventories and m_rgContextIds; add m_parentInventory to CInventory.
    • Change global g_ActiveInventory type to CAppwideInventory.
  • Constants:
    • Add ContextId.PROTECTED = 16.

Written by Cursor Bugbot for commit 6d6c7a2. Configure here.

@GODrums GODrums requested a review from Step7750 October 25, 2025 15:57
@GODrums GODrums self-assigned this Oct 25, 2025
@GODrums GODrums added the bug label Oct 25, 2025
Copy link
Member

@Step7750 Step7750 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @GODrums !

@GODrums GODrums merged commit 017fc85 into master Oct 25, 2025
2 checks passed
@GODrums GODrums deleted the fix/steam-inventory-assets branch October 25, 2025 18:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants