Repository navigation
Add verified private CloudKit sessions for 0.3.0 - #16
Conversation
|
Reasoning Relay reviewed base Remediated in
Validation: 87 tests in 15 suites, strict Swift Format, build, warning-free static DocC, consumer setup, Markdown wrapping, and diff checks pass. Race tests use deterministic suspension and callback-drain observations. Opaque checkpoint-write ordering is modeled at the exact callback lease boundary used by The new engine event policy is exercised through its shared manifest/error policy functions and configured-zone fetch coordinator; signed engine startup remains a consumer gate. No manual Codex review was requested. A delta-scoped Relay review of this new head is required before merge. |
|
Reasoning Relay delta review: APPROVED with no unresolved P0/P1 findings and explicit OK to merge this exact package head.
The reviewer verified all three prior P1 remediations and green CI run 40752574056. Local validation remains 87 tests across 15 suites plus strict formatting, build, DocC, consumer and wrapping checks. DEFER-P2: pruning obsolete fetch-generation errors could bound retained diagnostic memory during repeated failures. This optional improvement does not block the package release and is outside this approved source head. Current GitHub reviews, review threads and reactions are empty. Automatic Codex configuration is unknown with no current-head execution evidence; no manual Codex review was requested. The owner authorized the package release and merge after approval and green CI. This approval covers CloudSaveKit only; WortJagd's integration and owner-coordinated Apple TV acceptance remain separate gates. |
Personal CloudKit collections currently require the host app to resolve accounts, provision manifests, and fetch assets directly. This adds
CloudSavePrivateCoordinatorand immutableCloudSavePrivateSessionhandles so the package owns those operations while the host retains its schema, local database, media, and durable ledger.Conditional first provisioning rereads the server's canonical winner. Known missing or contradictory manifests fail closed during bootstrap and active sync. Retiring sessions drain admitted host callbacks before a replacement reads the same store. Configured-zone network failures retain offline eligibility instead of becoming host-recovery failures. Account transitions fence the old session before cancellation or host notification; temporary unavailability preserves local writes while retiring cloud work. Personal sessions opt out of mutable host rebinding and automatic owned-zone recreation, while existing engine/sharing integrations retain their current policies.
This prepares version
0.3.0and documents a neutral notebook client. It does not perform the dependent WortJagd integration, migration, cloud deletion, or real Apple TV profile acceptance.Validation:
swift test: 87 tests in 15 suites pass, including concurrent creation, lost acknowledgements, repeated notifications during suspended identity resolution, late asset/ledger/materialization results, A → B → A ledger isolation, invalidation before cancellation, temporary-unavailability durability, host-write retirement barriers, active manifest drift/malformed replacement, configured-zone error classification, and privacy-safe logging.swift build, strict Swift Format, consumer setup validation, Markdown wrapping check, andgit diff --checkpass..gitignoreexactly matches the shared active patterns. Current package registry has 16 unique canonical emojis; CloudSaveKit remains ☁️ with its own subsystem/category.0.0.42provenance matches central tagged tree8e755afa7cf7737ff23a5fdeeada1af34ee7cfb2; subtree markedlinguist-generated. Review the integration, not the verified subtree contents.The supported boundary is trusted host schema callbacks and immutable account-scoped stores. Tests use controlled internal transport/engine seams and require no account credentials. Real signed consumer startup, CloudKit delivery, and tvOS profile switching remain consumer acceptance work; unit tests do not establish those outcomes. Pure token/value helpers deliberately emit no lifecycle noise.
Reasoning Relay exact-base/head review is pending; keep this PR Draft until it reports no P0/P1 findings and explicitly approves the current pair. Package publication follows the approved merge under the owner's existing release authorization.