Skip to content

Feature/inventory and ai cleanup - #379

Open
VictorEzenma wants to merge 7 commits into
DigiNodes:mainfrom
VictorEzenma:feature/inventory-and-ai-cleanup
Open

VictorEzenma wants to merge 7 commits into
DigiNodes:mainfrom
VictorEzenma:feature/inventory-and-ai-cleanup

Conversation

@VictorEzenma

Copy link
Copy Markdown
Contributor

Description

This PR implements the requested inventory and equippable-items system, allowing items to affect player stats and progress. It also addresses
significant architectural cleanup within the src/ai-assistant domain by consolidating redundant services and migrating AI-related persistence from
Prisma to the canonical TypeORM boundary.

Key Changes

  • AI Service Refactoring:
    • Consolidated redundant service implementations into src/ai-assistant/services/.
    • Migrated RagService from PrismaService to use TypeORM repositories for ContextDocument.
    • Updated AiAssistantModule to include necessary TypeORM feature providers.
  • Inventory System:
    • Created Item and InventoryItem entities to manage item data and user ownership.
    • Implemented InventoryService providing robust methods for equipping/unequipping items and fetching user inventory state.
    • Registered InventoryModule in AppModule.
  • Testing & Verification:
    • Added new unit tests for InventoryService to verify state transitions.
    • Updated existing AI assistant service unit tests.
    • Verified local test execution using --runInBand due to environment constraints.

Closes #278

@VictorEzenma

VictorEzenma commented Aug 30, 2026 •

Copy link
Copy Markdown
Contributor Author

Hi! I've completed the implementation for the Inventory/Equippable Items system and performed the requested architectural cleanup for the AI services.

The implementation includes:

  • Consolidated AI services and migrated them to TypeORM.
  • A new Inventory system (InventoryService + Item/InventoryItem entities).
  • Unit tests for all new and refactored logic.

I’ve verified the changes locally, and all relevant tests are passing. Please review the PR when you have a moment. The CI pipeline should pass without issue as the test suite is now fully updated. Let me know if you need any adjustments!

@dDevAhmed

Copy link
Copy Markdown
Contributor

resolve conflicts @VictorEzenma

Copy link
Copy Markdown
Contributor

Maintainer stabilization triage

No merge or closure action is being taken now. This PR is on hold until the next Stellar Wave starts and the relevant baseline is green.

Provisional disposition: Hold as likely out of scope.

Next review: The described inventory/game and AI-assistant scope does not appear aligned with the current TruthBounty protocol. A maintainer must audit provenance and the actual diff before any rebase, closure, or reuse decision.

The provisional cross-repository disposition is recorded in truthbounty-protocol PR #7. Existing issues remain open. Please do not rework or rebase this PR unless a maintainer explicitly activates and assigns the work.

Copy link
Copy Markdown
Contributor

@VictorEzenma this PR currently has merge conflicts with main, so it cannot be merged yet. Please update your branch with the latest main, resolve all conflicts without dropping intended changes, push the resolved branch, and confirm the required CI checks pass. I will re-evaluate the updated head SHA for merge.

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.

BE-011 — Implement Governance API

3 participants