Telemetry should send tool and token events

[derekmisler]

Resolves #98

[Copilot]

Pull Request Overview

This PR refactors the telemetry system to use synchronous event processing instead of asynchronous background processing. The change simplifies the telemetry client by removing event queues, background goroutines, and async processing infrastructure.

• Switches from asynchronous event queuing to synchronous event processing
• Simplifies client initialization and removes shutdown/cleanup complexity
• Updates telemetry configuration to default to enabled instead of requiring explicit opt-in

Reviewed Changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
internal/telemetry/utils.go	Changes telemetry default behavior to enabled and simplifies the enable/disable logic
internal/telemetry/types.go	Removes async processing types and replaces slog.Logger with custom telemetryLogger
internal/telemetry/telemetry_test.go	Updates tests to work with synchronous processing and simplified HTTP client mocking
internal/telemetry/http.go	Updates logging messages to use consistent event_type field names
internal/telemetry/global.go	Removes deprecated global client setter and updates initialization logging
internal/telemetry/events.go	Replaces async event queuing with direct synchronous event processing
internal/telemetry/client.go	Major refactor: removes async processing, adds telemetryLogger wrapper, simplifies client creation
docs/TELEMETRY.md	Updates documentation to reflect synchronous processing
Taskfile.yml	Removes TELEMETRY_ENABLED build argument since telemetry now defaults to enabled
Dockerfile	Removes TELEMETRY_ENABLED build argument from Docker builds
.github/workflows/ci.yml	Removes TELEMETRY_ENABLED environment variable from CI workflows

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}