ptarrant/code_of_conquest_dnd
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge fix/npc-harness-luck-descriptor into dev

Browse Source 
NPC harness built a canon log with an empty player.luck_descriptor, which the
schema rejects (minLength 1) → /npc/speak 422. Build the player through
LogPlayer with a non-empty §7 fortune line. Human-confirmed working live.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This commit is contained in:
Phillip Tarrant
2026-07-10 13:38:17 -05:00
parent d788225754 81db962a9d
commit 1a6e8e8616
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
client/scripts/harness/npc_harness.gd
View File

@@ -33,8 +33,10 @@ func _ready() -> void:
_game_state = GameState.new() _game_state = GameState.new()
_canon_log = CanonLog.new() _canon_log = CanonLog.new()
_canon_log.set_location("greywater_docks", "Greywater Docks") _canon_log.set_location("greywater_docks", "Greywater Docks")
_canon_log.player.name = "Aldric" # Build the player through LogPlayer so luck_descriptor is non-empty — the
_canon_log.player.class_id = "sellsword" # canon-log schema requires it (minLength 1), and it's the §7 fortune line
# the digest surfaces to the NPC. An empty string here 422s the request.
_canon_log.player = LogPlayer.new("Aldric", "sellsword", "the dice are kind today")
# Deliberately no find_the_ledger quest here — leave it un-started so Fenn # Deliberately no find_the_ledger quest here — leave it un-started so Fenn
# can offer_quest it during the conversation. # can offer_quest it during the conversation.