Skip to content

fix(sim): leave non-playable characters out of the character list - #66

Merged
nichinichisou0609 merged 1 commit into
mainfrom
fix/non-playable-characters
Oct 18, 2026
Merged

nichinichisou0609 merged 1 commit into
mainfrom
fix/non-playable-characters

Conversation

@nichinichisou0609

Copy link
Copy Markdown
Contributor

Summary

  • MasterCharacter rows read the optional _isNonPlayable column (absent means false).
  • A player save lists only playable characters: under a partial character list an unlisted non-playable character is no longer required, and complete coverage leaves it out of the total character rank. A listed non-playable character is read like any other.
  • The owned-snapshot input applies the same rule to its total-rank and coverage checks.
  • docs/account-input.md describes the rule.

Tests

  • a_partial_character_list_requires_only_unlisted_playable_characters
  • complete_character_coverage_leaves_non_playable_characters_out_of_the_total

🤖 Generated with Claude Code

MasterCharacter rows read the optional `_isNonPlayable` column (absent
means false). A player save lists only playable characters, so account
resolution no longer requires an unlisted non-playable character under a
partial character list, and complete coverage leaves it out of the total
character rank. A listed non-playable character is read like any other.
The owned-snapshot input applies the same rule to its rank checks.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@nichinichisou0609
nichinichisou0609 merged commit 3ff10cf into main Oct 18, 2026
6 checks passed
@nichinichisou0609
nichinichisou0609 deleted the fix/non-playable-characters branch October 19, 2026 07:07
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