chore: clean up and modernize repo after example migration - #51
Merged
Conversation
Remove references to the deleted CLI, providers, and test modes. Update README, CLAUDE.md, run.sh scripts, package.json, and tsconfig.json to reflect the current state of the repo: internal tooling only (demo data, trace generator, test scripts, example runner).
- Move scripts/, trace-generator/, screenshot-demo/ to root - Replace requirements.txt with pyproject.toml - Replace venv/pip setup with uv (run.sh scripts, Makefile) - Remove old python/run.sh (100+ line venv setup) - Clean up .gitignore (remove node-specific entries)
richardsolomou
changed the title
- Add root package.json with deps for Node test scripts (Vercel AI, OTel) - Restore test_vercel_ai_otel.ts and test_vercel_anthropic.ts into scripts/ - Fix dotenv paths in all scripts (../../.env → ../.env) after flattening - Update Makefile setup target to run both uv sync and pnpm install - Update README and CLAUDE.md to document Node scripts - Remove stale version-pinning vars from .env.example
andrewm4894
approved these changes
Mar 20, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
After migrating provider examples to posthog-python and posthog-js (#50), the repo had stale references to the deleted CLI/providers, an unnecessarily nested structure (
python/,node/subdirectories), and used pip/venv for dependency management.Changes
Remove dead code
python/main.py,node/src/index.ts(already deleted in chore: remove provider examples ported to SDK repos #50), and theirrun.shlaunchersnode/directory — removed entirely (config files for code that no longer exists)prepare.sh— localhost API key sync script, nothing sourced it anymoretest-audio.mp3— test fixture for deleted transcription provider.env.exampleFlatten and modernize
python/to root —scripts/,trace-generator/,screenshot-demo/now live at the top levelpyproject.toml+uvreplacesrequirements.txt+ pip/venvpackage.json+pnpmat root for Node test scripts (Vercel AI OTel, Anthropic)make setupruns bothuv syncandpnpm install../../.env→../.env) after flatteningUpdate docs
.env.example— cleaned upscripts/OTEL_TESTING_REFERENCE.md— updated commands to useuv run