Revert PR #32 — VIN decode PostHog enrichment was duplicating query_logs #33

Merged
root merged 1 commits from dev into main 2026-05-18 11:04:29 +03:00

1 Commits

Author SHA1 Message Date
0defbe7eb6 Revert "feat(telemetry): enrich VIN decode events for Süper Panel observability"
Some checks failed
QA Gate (P0/P1) / Test affected app (pull_request) Has been cancelled
Reverts PR #32 (commit 0b1f033 / 4645c27).

Reason: the backend telemetry was duplicating data that already lives
in query_logs. Every field the new PostHog events carried — provider
chain, response_time_ms, success/error, cache_source, pl24_circuit_open,
brandId, aborted, partial_result — is captured per-row in:

  query_logs { userId, vin, brandId, source, success, errorMessage,
               responseTimeMs, timings (jsonb), createdAt }

Süper Panel already has read-only access to the Sase.tr Postgres via
DATABASE_URL_SASE_RO; the VIN Decode Observability Module (SP-VIN-001)
will read query_logs directly. PostHog hop adds latency (5min poll),
event-volume cost, KVKK weight (even sanitized VINs leaving the
service), and architectural duplication for zero observability gain.

Frontend PostHog events (vin_decode_candidates, candidate_selected,
search_input_validation_failed, vin_decode_retry_clicked) stay — those
are user-interaction signals query_logs doesn't capture.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
2026-05-18 11:04:15 +03:00