Skip to content

Simplify trace plugin setup by removing extra CLI dependencies#7

Merged
adagradschool merged 1 commit into
masterfrom
remove-cli-deps
Mar 23, 2026
Merged

Simplify trace plugin setup by removing extra CLI dependencies#7
adagradschool merged 1 commit into
masterfrom
remove-cli-deps

Conversation

@adagradschool
Copy link
Copy Markdown
Contributor

Summary

This simplifies the trace plugin runtime and setup flow by removing the need for jq, curl, and uuidgen.

Users now only need python3 for the trace hooks to run.

User impact

  • fewer system-level dependencies to install
  • more consistent behavior across environments
  • simpler setup and CI configuration
  • no expected change to plugin behavior

What changed

  • moved hook implementation details into a Python runtime
  • kept the existing shell hook entrypoints, but made them thin wrappers
  • replaced shell-based JSON handling, ID generation, and HTTP export logic
  • updated setup and validation scripts to avoid extra CLI tools
  • added broader automated test coverage
  • switched development and CI workflows to uv and pytest

Verification

  • ran make test
  • unit tests passed
  • fixture replay passed
  • manifest validation passed

@adagradschool adagradschool merged commit f5acd22 into master Mar 23, 2026
2 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