Skip to content

Track implicit skill loads#7

Merged
Stephen Belanger (Qard) merged 1 commit into
mainfrom
implicit-skill-load-tracking-main
Jul 3, 2026
Merged

Track implicit skill loads#7
Stephen Belanger (Qard) merged 1 commit into
mainfrom
implicit-skill-load-tracking-main

Conversation

@Qard

@Qard Stephen Belanger (Qard) commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Summary

  • detect observable Codex skill loads from skills.read, SKILL.md reads, and skill script executions
  • normalize detected loads as observed skill-load tool spans while preserving the original tool name
  • attach singular skill metadata including name/path when available
  • omit the optional skill_load_trigger field for phase 1, so these loads default to implicit under the spec

Spec

Validation

  • bun test src/agents/codex/event-processor.test.ts
  • bun run check

@Qard Stephen Belanger (Qard) force-pushed the implicit-skill-load-tracking-main branch from cfc3f8a to 487f690 Compare July 2, 2026 18:14
@Qard Stephen Belanger (Qard) marked this pull request as ready for review July 2, 2026 18:21
@Qard Stephen Belanger (Qard) force-pushed the implicit-skill-load-tracking-main branch from 487f690 to 7ad5322 Compare July 2, 2026 18:31
@Qard Stephen Belanger (Qard) force-pushed the implicit-skill-load-tracking-main branch from 7ad5322 to 9cc840d Compare July 2, 2026 18:40
@Qard Stephen Belanger (Qard) merged commit c6691c9 into main Jul 3, 2026
4 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.

2 participants