Use default_factory for OrderResponse order_id and created_at

[Wired4ncer]

While reading through publsp closely (considering helping revive it), I noticed OrderResponse.order_id and OrderResponse.created_at use plain default values rather than default factories:

order_id: str = str(uuid.uuid4())
created_at: int = int(time.time())

Because these are evaluated once, at class-definition (import) time, every OrderResponse constructed without explicit values shares the same order_id and the same created_at for the lifetime of the process. Two orders created in one run would collide on order_id and report an import-time timestamp instead of their real creation time.

Fix

Switch both to default_factory so each instance gets a fresh value:

order_id: str = Field(default_factory=lambda: str(uuid.uuid4()))
created_at: int = Field(default_factory=lambda: int(time.time()))

Tests

Adds tests/test_order_response_defaults.py — asserts two OrderResponse instances get distinct order_ids and monotonic/independent created_ats. Node-free (no LND connection needed), so it runs in CI as-is.

All up for discussion — happy to adjust naming or style to match your conventions.

🤖 Generated with Claude Code