Skip to content

feat: persist closed NPC conversation memory#105

Merged
JOY (JOY) merged 1 commit into
devfrom
codex/conversation-summary-memory
May 20, 2026
Merged

feat: persist closed NPC conversation memory#105
JOY (JOY) merged 1 commit into
devfrom
codex/conversation-summary-memory

Conversation

@JOY
Copy link
Copy Markdown
Contributor

Summary

  • persist a durable relationship memory summary when an NPC conversation session closes
  • mark closed conversation objectives as complete even when the previous stored objective was active
  • preserve existing runtime memories while writing the closed-conversation summary

Local review

  • Local code-review fallback: APPROVED
  • ADR compliance: compliant with Nakama as game backend and server-side LLM intent validation boundary
  • Security: no Unity client key path added, no LLM authority expansion, no gameplay state mutation from model output

Tests

  • npm run build
  • npm test
  • git diff --check
  • em-dash scan on changed diff

@JOY
Copy link
Copy Markdown
Contributor Author

Codex (@codex) review

@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@JOY JOY (JOY) merged commit 9c393c5 into dev May 20, 2026
1 check passed
@JOY JOY (JOY) deleted the codex/conversation-summary-memory branch May 20, 2026 19:34
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