fix: preserve social action targets from pinned OASIS traces by rudycelekli · Pull Request #915 · 666ghj/MiroFish · GitHub
Skip to content

fix: preserve social action targets from pinned OASIS traces - #915

Open
rudycelekli wants to merge 1 commit into
666ghj:mainfrom
rudycelekli:fix/preserve-mute-target-in-action-evidence-20261021
Open

rudycelekli wants to merge 1 commit into
666ghj:mainfrom
rudycelekli:fix/preserve-mute-target-in-action-evidence-20261021

Conversation

@rudycelekli

Copy link
Copy Markdown

Social-action enrichment expects fields that differ from the pinned camel-oasis 0.2.5 trace contract. Muting records mutee_id, which the action exporter discarded; comment creation records comment_id but not post_id, so enrichment never found its parent discussion. Logged actions and graph-memory descriptions therefore lost the affected user or discussion.

Retain mute targets, including user 0, and resolve a created comment's parent post through its recorded comment ID. Existing explicit post IDs and follow-target enrichment remain supported.

Pinned contract source: OASIS v0.2.5 platform implementation, specifically mute and create_comment.

Validation:

  • Real SQLite trace rows matching the pinned SDK flow through production extraction, action logger, monitor and memory text generation.
  • Baseline: 3 failures and 2 passing controls (follow-to-user 0 and explicit comment post ID). Fixed full backend suite: 134 passed.
  • Complete production runner module is imported with explicitly disclosed import-only Camel/OASIS name doubles. No simulation, model, or hosted graph call is executed.
  • git diff --check passes.

AI assistance was used to investigate and prepare this locally verified patch.

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