Skip to content

Add entity context for Codex memory saves#17

Open
ishaanxgupta wants to merge 1 commit into
mainfrom
codex/entity-context
Open

Add entity context for Codex memory saves#17
ishaanxgupta wants to merge 1 commit into
mainfrom
codex/entity-context

Conversation

@ishaanxgupta
Copy link
Copy Markdown
Contributor

@ishaanxgupta ishaanxgupta commented Jun 5, 2026

What changed

  • Added Codex-specific user and project entityContext prompts.
  • Forwarded optional entityContext through SupermemoryClient.addMemory().
  • Sent user context during automatic transcript capture.
  • Sent project context from the explicit supermemory-save skill, with custom-container context derived from the configured container description when --container is used.
  • Documented the behavior and added regression coverage for the wiring.

Why

Claude Supermemory already sends entity-specific context on write calls, which lets Supermemory persist extraction guidance on the container tag. Codex was creating/saving to tags without that field, so container tags could show no entity context and extraction had less guidance.

@ishaanxgupta ishaanxgupta changed the title [codex] Add entity context for Codex memory saves Add entity context for Codex memory saves Jun 5, 2026
@ishaanxgupta ishaanxgupta force-pushed the codex/entity-context branch from cc44ba7 to 6ea9102 Compare June 5, 2026 11:00
@ishaanxgupta ishaanxgupta marked this pull request as ready for review June 5, 2026 11:22
Copy link
Copy Markdown
Member

@MaheshtheDev MaheshtheDev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what about results? any improvements?

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.

2 participants