Skip to content

feat(diagnostics): add inventoryDump command, moved from JoshariasSurvival - #56

Open
soloturn wants to merge 1 commit into
developfrom
soloturn-inventory-debug-diagnostics
Open

soloturn wants to merge 1 commit into
developfrom
soloturn-inventory-debug-diagnostics

Conversation

@soloturn

Copy link
Copy Markdown
Contributor

AI-assisted change proposal. Filed by agent driven by @soloturn via GDD.

Summary

  • Split from feat(demo): add debug-gated inventory diagnostics JoshariasSurvival#77, per @BenjaminAmos's review: JoshariasSurvival is a meta module and shouldn't carry logic - this belongs wherever the data it inspects lives.
  • inventoryDump console command + a DEBUG-level slot-change trace, both scoped to plain InventoryManager/ItemComponent - no Workstation dependency, unlike the sibling PR going to that module.
  • describeItem/dumpSlots are public static so Workstation's own diagnostics can format things the same way without duplicating the logic.

Test plan

  • :modules:Inventory:compileJava - builds clean.

Related

…vival

Split from Terasology/JoshariasSurvival#77 per @BenjaminAmos - a meta module
shouldn't carry logic that belongs to the module it inspects. This part
only touches InventoryManager, no Workstation dependency.

Co-Authored-By: soloturn <soloturn@gmail.com>
@coderabbitai

coderabbitai Bot commented Aug 28, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 13fd4dc7-e4b3-49ca-8ec4-94d458a0733a


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant