Skip to content

[codex] Unify Persona Runtime prompt context pipeline#1

Draft
murphys7017 wants to merge 38 commits into
masterfrom
codex/unify-prompt-context-pipeline
Draft

[codex] Unify Persona Runtime prompt context pipeline#1
murphys7017 wants to merge 38 commits into
masterfrom
codex/unify-prompt-context-pipeline

Conversation

@murphys7017

Copy link
Copy Markdown
Owner

Summary

  • replace the Prompt Selector/Profile path with one canonical ContextPack and deterministic Router, Persona, and Core projections
  • add versioned context building, explicit plugin-context collection, conflict detection, and a provider-neutral PromptTreeBuilder
  • preserve official conversation history and group context while isolating persona/effect semantics from Core execution
  • keep structured user content parts intact and persist scaffold-free conversation messages
  • update Yakumo architecture documentation and the execution flow

Why

Prompt construction had accumulated separate local paths for Router, Persona Expression, Core execution, plugin contexts, and provider rendering. Those paths could silently overwrite context, leak target-specific extensions, or lose structured content. This change makes data collection canonical and moves target differences into explicit projection rules.

Validation

  • 463 prompt, interaction, group-context, and main-agent unit tests passed
  • 138 postprocess, memory-runtime, and tool-loop tests passed
  • Ruff checks passed for all affected implementation and test modules
  • Python compile checks, YAML parsing, document encoding, and git diff --check passed

Notes

This branch also contains the ten previously committed Persona Runtime changes that were ahead of origin/master, so the PR represents the complete current local architecture state.

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Sorry @murphys7017, your pull request is larger than the review limit of 150000 diff characters

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