refactor(adapter): fix config at construction, drop setters/getters & query logging

[loks0n]

What

Adapter configuration is now passed via readonly constructor params instead of post-construction setters. setTenant is the one mutable exception — a single adapter serves many tenants across requests, so tenant legitimately changes over an instance's lifetime.

ClickHouse adapter

• Setters → readonly constructor params: setNamespace, setDatabase, setSecure, setSharedTables, setAsyncInserts (→ asyncInserts + asyncInsertWait), setDualReadSampleRate. Host/port/namespace/database validation and dual-read-rate clamping moved into the constructor. New params sit after $client, so existing positional callers are unaffected.
• Removed config-mirror getters (all unused): getNamespace, getTenant, isSharedTables.
• Kept: setTenant, setNextQueryId (per-query runtime state), and the operational getRouteLog/clearRouteLog/getConnectionStats.

Database adapter

• Dropped setNamespace/setSharedTables — callers configure the injected Utopia\Database instance directly (as the tests already do). Kept setTenant.

Usage facade

• Dropped setNamespace/setSharedTables/setDualReadSampleRate (config is now construction-time). Kept setTenant (flushes buffer + delegates).

Query logging removed

The enableQueryLogging flag, queryLog, logQuery(), and getQueryLog()/clearQueryLog() are gone. A code search across appwrite + appwrite-labs found zero consumers. getConnectionStats() stays (used by tests for request_count + async flags) minus its query_logging_enabled key.

Why

Less surface area, immutable-by-default configuration, and a net −225 LoC.

⚠️ Breaking change

Consumers that call the removed setters must move that config into the constructor. Notably cloud (app/cli.php, app/worker.php) calls ->setDatabase(...) on the usage adapter:

$clickhouseAdapter = new ClickHouse(
    host: $dsn->getHost(),
    port: $dsn->getPort(),
    username: $dsn->getUser(),
    password: $dsn->getPassword(),
    secure: $secure === 'true',
    database: ltrim($dsn->getPath(), '/'), // was ->setDatabase(...)
);

A coordinated cloud update is needed before/with the release that bumps this dependency.

Test plan

• pint --test ✅
• phpstan (level config) ✅
• Non-DB unit suites (MetricTest, UsageQueryTest) ✅
• ClickHouse/MariaDB integration tests lint + type-check clean but need live services (not run here)

🤖 Generated with Claude Code

[greptile-apps]

Greptile encountered an error while reviewing this PR. Please reach out to support@greptile.com for assistance.