revert: keep pushing non-DML statements (reverts #179)

[veksen]

Reverts #179 (80af5fb).

Goal

Stop dropping non-DML events at the capture layer. #179 quieted the migration-day alert spam (Query-Doctor/Site#3578) by filtering statementType === "other" at both push sites, so CREATE/DROP/ALTER/COPY/VACUUM never left the analyzer. That fixes the noise but discards the events before QD ever ingests them — which permanently removes downstream optionality: an index dropped in prod, a schema-changing migration, correlating a later CI regression against a schema change, an MCP "recent events" feed. None of that is possible once the event is dropped at capture.

Suppressing the alert does not require dropping the data. That's an alerting-layer decision (scope + cadence), so ingestion should stay complete and the noise handled where alerts are decided — tracked in Query-Doctor/Site#3581.

What

• Before (on main): only DML is pushed; DDL/COPY/VACUUM are filtered out and never reach the catalog or the dashboard.
• After (this PR): every statement sampled from pg_stat_statements is pushed again, restoring the pre-fix: skip non-DML statements when pushing queries #179 behavior.

Accepted trade-off: the migration-day alert spam returns until the alerting-side handling lands. That's deliberate — the fix belongs there, not here.

How

Plain git revert of the squash commit. Restores the two push sites in src/remote/api-client.ts to push unconditionally and drops the statementType instance field re-added to RecentQuery.

Tests

npm run typecheck clean; full suite passes (306/306 — the 4 removed cases are the assertions #179 added for the dropped filter; one Docker-teardown flake under the parallel run passes 3/3 in isolation).

[github-actions]

Query Doctor Analysis

3 queries analyzed

0 regressed · 0 improved · 0 new · 0 removed

2 pre-existing issues

• SELECT "guests"."id", "guests"."session_id", "guests"."username", "guests"."avatar_path", "guests"."color", "guests"."side", "guests"."audio_recording_path", "guests"."audio_recording_public", "gue...
  index assets(event_id, inserted_at desc)
  cost 31,003,449 → 1,498 (100% reduction)
• SELECT * FROM guest_ip_addresses WHERE ip_address = '127.0.0.1';
  index guest_ip_addresses(ip_address)
  cost 154,402 → 8 (100% reduction)

_{Using assumed statistics (10000000 rows/table). For better results, sync production stats.}

_{More detail → get_ci_run({ runId: "019f663e-626c-7ea4-80c5-b2807f166dda" }) · view run · docs}