-
-
Notifications
You must be signed in to change notification settings - Fork 765
Open
Description
Current State
cuabot duplicates telemetry code that exists in @trycua/core:
- PostHog configuration (API key, host)
- Installation ID management (
~/.cua/installation_id) - Telemetry event structure
The duplication is intentional to keep cuabot standalone, but means changes need to be synced manually.
Consideration
Evaluate whether cuabot should:
- Keep standalone - Current approach, duplicate code but no dependency
- Import @trycua/core - Share telemetry code, adds dependency
- Extract shared telemetry package - New
@trycua/telemetrypackage used by both
Files involved
libs/cuabot/src/telemetry.ts- cuabot's telemetry implementationlibs/core/- @trycua/core telemetry
Trade-offs
| Approach | Pros | Cons |
|---|---|---|
| Standalone | No deps, simpler install | Code duplication |
| Import core | Single source of truth | Heavier dependency |
| Shared package | Clean separation | More packages to maintain |
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels