feat: implement InventoryComponent for Epic 5.2

[mcj-coder]

Implements Epic 5.2 by adding InventoryComponent for character inventory management.

Changes

• InventoryComponent sealed record with immutable operations
  • AddItem(materialId, quantity) - adds or increments material quantity
  • RemoveItem(materialId, quantity) - removes material with validation
  • GetQuantity(materialId) - retrieves current quantity (0 if not present)
  • HasSufficientQuantity(materialId, quantity) - checks availability

Design

• Follows existing component patterns (ActiveCraft, ClassProgressCollection)
• Immutable operations return new instances
• Internal dictionary tracks MaterialId → quantity mappings
• Throws InvalidOperationException when removing insufficient quantity

Testing

• 11 comprehensive tests covering:
  • Empty inventory initialization
  • Adding new/existing materials
  • Removing materials (partial/complete)
  • Insufficient quantity validation
  • Multiple material tracking
  • Quantity checking helper
• All 526 tests pass (438 Core + 88 Simulation)
• Zero compiler warnings

Verification

dotnet build --warnaserror  # 0 warnings
dotnet test                 # 526/526 passed

Closes #41

Co-Authored-By: Claude Sonnet 4.5 noreply@anthropic.com

[github-actions]

	Fails
🚫	📝 PR body must include a Summary section (used for squash merge commit body). Required format: ## Summary [Brief description of what this PR does - at least 20 characters]
🚫	📝 PR description missing required sections: Summary Required PR structure: ## Summary [What this PR does] ## Issue Closes #[number] ## Changes - Change 1 - Change 2 ## Testing - [ ] Tests added/updated - [ ] Manual testing done

	Messages
📖	✅ Issue properly linked: Closes #41
📖	✅ PR title follows Conventional Commits format

Recommended Persona Reviews

💻 .NET Specialist - C# code changes detected

See docs/roles/ for persona details.

---

Danger analysis complete for feat: implement InventoryComponent for Epic 5.2. 2 files changed, 1 commits.

Generated by 🚫 dangerJS against a070320

[mcj-coder]

Closing due to multiple process violations. See issue #141 for details:

1. PR opened by Maintainer account instead of Contributor account
2. New work started while PR feat: add silver and gold conversions to Currency #139 has unresolved CHANGES_REQUESTED feedback

Required workflow:

• First: Address PR feat: add silver and gold conversions to Currency #139 rebase (blocking)
• Then: Recreate this PR from martincjarvis (Contributor) account

Contributors must prioritize addressing review feedback over starting new tasks.