Skip to content

[codex] Trace skill prompt injection paths - #29800

Closed
alexsong-oai wants to merge 2 commits into
codex/emit-implicit-skill-usage-eventsfrom
codex/emit-implicit-skill-usage-events-clean
Closed

[codex] Trace skill prompt injection paths#29800
alexsong-oai wants to merge 2 commits into
codex/emit-implicit-skill-usage-eventsfrom
codex/emit-implicit-skill-usage-events-clean

Conversation

@alexsong-oai

Copy link
Copy Markdown
Contributor

Summary

  • Add tracing around core and extension skill selection, prompt reads, and instruction fragment rendering.
  • Log host, executor, and orchestrator skill resource reads, including cache hits and read sizes.
  • Keep this as a stacked PR on top of [codex] Emit implicit skill usage for support reads #29731 so the diff isolates the additional diagnostics cleanup.

Impact

This makes skill prompt injection easier to debug across host, executor, and orchestrator-backed skill sources without changing the intended user-visible behavior.

Validation

  • git diff --check origin/codex/emit-implicit-skill-usage-events...HEAD

Full crate/workspace tests were not run in this PR creation pass.

@alexsong-oai
alexsong-oai requested a review from a team as a code owner June 24, 2026 04:59
This reverts commit ee1fa16.
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