Add improved inventory system for First Born items

[RobGruhl]

Implements a comprehensive item token browser with:

• Category-based navigation (Weapons, Armor, Equipment, First Born Tech, etc.)
• Full-text search across item names, types, and abilities
• Size filtering (Nano, Small, Medium, Large)
• Multiple sort options (name, buy/sell price, size)
• Expandable item cards with detailed stats and ability descriptions
• Visual indicators for token colors and crystal requirements
• Only includes First Born base game and Yamato expansion items

Data sourced from xinix/core-space open source project (MIT License).
Integrates seamlessly with existing site design and navigation.

[claude]

Pull Request Review

Overview

This PR adds a comprehensive item token browser. The implementation is solid with good code organization.

Strengths

• Clean Architecture: Well-organized separation of data and logic
• IIFE Pattern: Prevents global namespace pollution
• Security: Proper XSS prevention via escapeHtml()
• UX: 200ms debounce, expandable cards, responsive design

Bugs Found

CRITICAL - inventory.js:367
Extra quote in move stat display. Line has extra quote before closing span tag.

IMPORTANT - inventory-data.js:197
Undefined icon reference: physical_armour is not defined in icons object

Performance

• Full re-render on every state change (300+ items)
• Event listeners re-attached on every render
• Consider: event delegation, virtual scrolling, memoization

Security

No issues identified. Good XSS protection throughout.

Missing

• Automated tests
• Accessibility features (ARIA labels, keyboard nav)
• Error handling for missing data

Recommendation

Approve with minor fixes

Fix the two bugs above and this is ready to merge. Accessibility and performance can be follow-up PRs.

Great work! UI is polished and implementation is solid.