fix(payment): price quotes from current_chunks() to match freshness gate

[jacderida]

Problem

On the STG-01 testnet (ant-node 0.11.6-rc.1, ~907 nodes, 30% NAT-simulated, 10 concurrent uploaders) ~100% of uploads failed on the first cycle, every chunk PUT rejected by the storing node:

payment error: Quote from peer ... stale by 29 records (quoted 8 vs current 37, tolerance 5)
→ Error: Upload failed: 0/N stored

Root cause

The storage-delta freshness gate (validate_quote_freshness) compares two different record counts:

	Source	Counts
quoted (price-derived)	QuotingMetricsTracker side counter	client-paid PUTs only
current (freshness gate)	LmdbStorage::current_chunks()	all records: client PUTs + replication + repair

record_store() (which bumps the side counter) fires only on the client-paid PUT path in storage/handler.rs. The replication store paths (replication/mod.rs fresh fan-out + repair fetch) write to LMDB without touching it. So on any replicating network current >> quoted, the delta blows past tolerance = max(5, 5% of quoted), and every payment is rejected.

33db86a made the verifier's read authoritative (current_chunks()) but left pricing on the side counter — that is what introduced the divergence. Smaller test runs only passed because per-node record counts stayed under the 5-record absolute floor; the bug was latent, not absent.

Fix

Price quotes from the same authoritative source the gate reads. QuoteGenerator gains an attached Arc<LmdbStorage> (mirroring PaymentVerifier) and prices from current_chunks(), falling back to the in-memory counter only when no store is attached (unit tests / misconfigured startup). AntProtocol::new attaches the store to the generator right beside the verifier, so the invariant holds for every construction path.

• calculate_price ↔ derive_records_stored_from_price round-trip exactly (the coefficient divides evenly), so feeding larger current_chunks() counts through pricing is lossless and the verifier recovers the exact count — leaving the freshness delta at ~genuine in-flight growth.
• current_chunks() is also a more accurate measure of node fullness for pricing than direct-paid-PUTs-only.

Tests

• New regression test test_pricing_tracks_attached_storage_not_side_counter: writes records straight to the store (as replication would, without bumping the side counter) and asserts the quote prices off the store count — the divergence the prior unit tests structurally could not express (they drove both sides with a single number via set_records_stored_for_tests).
• cargo test -p ant-node --lib payment:: → 133 passed, 0 failed.
• cargo clippy -p ant-node --tests clean.

Follow-up (not in this PR)

The side counter / record_store() is now fallback-only and effectively dead in production. A later change could drop it entirely, symmetric with what 33db86a did on the verifier side. Kept out here to keep the patch minimal and low-risk.

🤖 Generated with Claude Code