docs: stop pointing at the deleted Flutter tree — #2710
Every comment naming a path in flutter_client/ now resolves to nothing, which is the failure mode this project has already been bitten by twice -- drift #572 came from delete.go describing behaviour it no longer had, and that same docstring was still wrong when it was fixed last week. A pointer to a deleted directory is the same thing in slower motion: the reader follows it, finds nothing, and cannot tell whether the comment is stale or they are looking in the wrong place. Three treatments, per what each comment was actually doing: - Naming a concept ("mirrors db.dart's CachedTracks Drift table"): keep the concept, drop the path. The Drift table is why the entity looks as it does; the file it lived in is not. - Pure port bookkeeping ("Mirrors <path>." and nothing else): deleted. Git history records the port; the comment only restated it. - Substance introduced by a pointer (a lifecycle list, a 200 px/s threshold, an inverted control-row placement): keep the substance, drop the lead-in. The two Go comments were the valuable ones and got more than a trim. They stated a live contract -- "field names match the client's FromJson helpers exactly, or fields are silently dropped" -- against a client that no longer exists. They now name the real consumer, SyncResponseWire.kt, and say why the failure is silent there too: kotlinx.serialization skips unknown keys, so a renamed field arrives as a default value rather than an error. The ticket counted 64 files by grepping flutter_client/. A second tier turned up during the sweep: 15 more references naming bare Dart files (player_bar.dart, now_playing_screen.dart:464, auth_provider.dart) with no directory prefix. Same dead tree, same treatment, folded in here. Comments only -- verified no non-comment line is touched in the diff.
This commit is contained in:
@@ -11,9 +11,11 @@ import (
|
||||
|
||||
// These tests pin the wire-format keys for /api/library/sync upserts.
|
||||
// Without them, sqlc model field-name drift or accidental
|
||||
// `json.Marshal(rawStruct)` regressions would silently break the
|
||||
// Flutter client (which reads snake_case keys via _*FromJson helpers
|
||||
// in flutter_client/lib/cache/sync_controller.dart).
|
||||
// `json.Marshal(rawStruct)` regressions would silently break the Android
|
||||
// client, which reads these snake_case keys via @SerialName in
|
||||
// models/wire/SyncResponseWire.kt. "Silently" is the operative word:
|
||||
// kotlinx.serialization skips unknown keys, so a renamed field arrives as
|
||||
// a default value rather than an error.
|
||||
|
||||
// validUUID is a deterministic test UUID — not a real value, just
|
||||
// something that pgtype.UUID.Valid will accept.
|
||||
|
||||
Reference in New Issue
Block a user