Skip to content

Track implicit skill loads#32

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

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

Conversation

@Qard

@Qard Stephen Belanger (Qard) commented Jul 2, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Summary

  • normalize Claude Skill tool calls as observed skill-load tool spans
  • attach singular skill metadata including skill name and original tool name
  • omit the optional skill_load_trigger field for phase 1, so these loads default to implicit under the spec
  • add post-tool hook coverage for the normalized span payload

Spec

Validation

  • ./run_tests.sh test_post_tool_use

@Qard
Stephen Belanger (Qard) force-pushed the implicit-skill-load-tracking branch from 13e1f38 to cafb67b Compare July 2, 2026 18:14
@Qard
Stephen Belanger (Qard) marked this pull request as ready for review July 2, 2026 18:14
@Qard
Stephen Belanger (Qard) force-pushed the implicit-skill-load-tracking branch from cafb67b to 6c74556 Compare July 2, 2026 18:31
@Qard
Stephen Belanger (Qard) force-pushed the implicit-skill-load-tracking branch from 6c74556 to b15c993 Compare July 2, 2026 18:40
@ekon15 eric kontargyris (ekon15) mentioned this pull request Jul 3, 2026
Closed
@Qard
Stephen Belanger (Qard) merged commit 84cf5d9 into main Jul 3, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@realark Andrew Kent (realark) realark approved these changes

Assignees

@Qard Stephen Belanger (Qard)

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Qard @realark