[j0sh]

Depends on #40. Merge after that PR.

The existing bounded observer refresh reads activity once for the configured ticket recipient at the observation's canonical block. Challenge, receipt, and readiness use that local result; receipt keeps the activity result attached to its payer-funds observation.

Unavailable activity refuses admission before credit or replay state is published. The recipient remains distinct from the node signing key. This uses the shared observer and fixture without another activity map, worker, or event reducer.

Validation: affected package suites and race checks, vet, and diff checks passed, including warm zero-RPC reads, round boundaries, reorg/replacement recovery, and separate-recipient startup. Reused code/tests retain provenance; no documentation changes.

Closes #35.

[Copilot]

🟢 Approval recommended

No active review issues were identified for the recipient-activity caching changes.

No open issues 🎉

What changed in this PR

This PR extends the shared payment observer to cache activity for the configured ticket recipient at the observation’s canonical block. It builds on retained contract bindings so warm payment admission and readiness checks can use local activity state while keeping the recipient distinct from the node signing key.

Changes:

• Store recipient activity and lookup errors alongside each payment observation.
• Use observed activity for payment checks and paid readiness, refusing unavailable or stale state.
• Extend fixtures and tests for warm reads, round boundaries, reorgs, contract replacement, and separate-recipient startup.

File	Description
signer/​payment_test.go	Update observer construction without a configured recipient.
signer/​app.go	Initialize the signer observer without recipient tracking.
orchestrator/​payment_recipient_test.go	Test local activity checks and refusal without balance or replay-state changes.
orchestrator/​metrics.go	Base paid chain readiness on current observed recipient activity.
orchestrator/​metrics_test.go	Configure a recipient for observer-backed readiness tests.
orchestrator/​app.go	Initialize activity tracking after selecting the ticket recipient.
orchestrator/​app_test.go	Verify activity queries target the configured recipient.
internal/​test/​chainrpc.go	Model recipient activation and deactivation rounds.
eth/​payment.go	Retain activity in snapshots and serve observer-backed checks locally.
eth/​payment_observer.go	Refresh recipient activity and expose it through observer status.
eth/​payment_observer_test.go	Cover activity boundaries, snapshot consistency, failures, reorgs, and staleness.
eth/​payment_contracts_test.go	Extend binding-replacement tests to cover local activity reads.
eth/​contracts_test.go	Update observer construction in redemption tests.

_{Review effort: Balanced}

[Copilot]

🟡 Changes recommended

The new core package cannot compile due to undefined types and missing state infrastructure, and its unused helper leaves activity checks outside the shared canonical payment observation.

2 open issues

• Adapt the helper to existing types and state infrastructure to fix compilation · New
• Use the shared activity observation in challenge, receipt, and readiness checks · New

_{Review effort: Balanced}

[Copilot]

🔵 Needs a closer look

The new helper cannot compile because required types and imports are absent, and it is unused by payment paths, leaving the required local activity checks unimplemented.

2 open issues

• Adapt the helper to existing types and state infrastructure to fix compilation
• Use the shared activity observation in challenge, receipt, and readiness checks

_{Review effort: Balanced}

[Copilot]

🟢 Approval recommended

No active review issues are reported for the recipient-activity caching change.

2 open issues

• Adapt the helper to existing types and state infrastructure to fix compilation
• Use the shared activity observation in challenge, receipt, and readiness checks

_{Review effort: Balanced}