Skip to content

test: cover inventory validation rejections#231

Merged
JOY (JOY) merged 1 commit into
devfrom
codex/inventory-validation-coverage
May 24, 2026
Merged

test: cover inventory validation rejections#231
JOY (JOY) merged 1 commit into
devfrom
codex/inventory-validation-coverage

Conversation

@JOY
Copy link
Copy Markdown
Contributor

Summary

  • add inventory rejection coverage for same-slot replacement, invalid slot equip, and full body carry capacity
  • assert rejected inventory paths leave the storage object unchanged
  • assert a DOS.AI/model inventory mutation intent is denied by server validation and does not mutate inventory
  • update the inventory design checklist and changelog for the completed Add inventory lifecycle validation tests and guards #219 validation coverage

Verification

  • npm.cmd run build in backend/nakama
  • npm.cmd test in backend/nakama
  • npx.cmd markdownlint-cli2 "CHANGELOG.md" "docs/design/45-inventory-and-equipment-system.md"
  • git diff --check

Local review fallback

  • APPROVED: test/docs-only backend slice, no Unity package/scene changes, no client-side keys, and LLM/model output remains non-authoritative.

Completes the explicit validation-test checklist item in #219. Broader custody lifecycle state expansion remains tracked separately in the same issue.

@JOY JOY (JOY) merged commit 107a756 into dev May 24, 2026
2 checks passed
@JOY JOY (JOY) deleted the codex/inventory-validation-coverage branch May 24, 2026 20:25
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