Skip to content

fix(transfer): remap mental model evidence ids - #3833

Merged
nicoloboschi merged 2 commits into
mainfrom
fix/whole-bank-transfer-based-on
Aug 27, 2026
Merged

fix(transfer): remap mental model evidence ids#3833
nicoloboschi merged 2 commits into
mainfrom
fix/whole-bank-transfer-based-on

Conversation

@nicoloboschi

Copy link
Copy Markdown
Collaborator

Summary

  • preserve source memory-unit IDs in whole-bank transfer archives for remapping
  • rewrite reflect_response.based_on references for current mental models and history after import
  • cover serialized and decoded JSONB payloads, facts, and observations
  • add unit and whole-bank round-trip regression tests

Verification

  • ./scripts/hooks/lint.sh
  • focused remapping regression test ✅
  • whole-bank integration regression test added; local execution is currently blocked by embedded PostgreSQL port 5567 already being occupied

@strix-security

strix-security Bot commented Aug 27, 2026

Copy link
Copy Markdown

Strix Security Review

Warning

This pull request has 1 commit after the last Strix review (24d56d3). Strix has not reviewed these changes.
Automatic review on push is off for this repository. To review the latest changes, tag @strix-security in a comment, or turn on re-review on push.

No security issues found.

Updated for 24d56d3.


Reviewed by Strix
Re-run review · Configure security review settings

@nicoloboschi
nicoloboschi merged commit c143de2 into main Aug 27, 2026
105 of 107 checks passed
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