v0.3.3-pre.006

crates/ksp-store-postgres-lib/tests/dependency_boundary.rs

@@ -1,5 +1,5 @@
 // file: crates/ksp-store-postgres-lib/tests/dependency_boundary.rs
-// version: 10
+// version: 11
 
 #![warn(missing_docs)]
 #![deny(unreachable_pub)]
@@ -168,9 +168,6 @@ fn pre_005_raw_write_sql_is_atomic_idempotent_and_keeps_later_scope_closed() {
     }
     for forbidden in [
         "DELETE FROM",
-        "list_raw_transactions",
-        "RawPage",
-        "RawCursor",
         "transition_raw_transaction_retention",
         "impl ksp_store_api::RawTransactionWrite",
         "impl ksp_store_api::RawTransactionObservationWrite",
@@ -179,3 +176,48 @@ fn pre_005_raw_write_sql_is_atomic_idempotent_and_keeps_later_scope_closed() {
     }
     return;
 }
+
+#[test]
+fn pre_006_raw_pagination_is_keyset_cursor_bound_and_policy_free() {
+    let raw = include_str!("../src/raw_transaction.rs");
+    let cursor = include_str!("../src/raw_transaction/cursor.rs");
+    let index = include_str!("../migrations/v001_raw_transaction/indexes/001_ix_ksp_raw_transactions_slot_signature.sql");
+    for required in [
+        "LIST_TRANSACTIONS_ASC_SQL",
+        "LIST_TRANSACTIONS_DESC_SQL",
+        "retention_state <> 'purged'",
+        "(slot, signature) >",
+        "(slot, signature) <",
+        "ORDER BY slot ASC, signature ASC",
+        "ORDER BY slot DESC, signature DESC",
+        "LIMIT $5",
+        "list_raw_transactions",
+    ] {
+        assert!(raw.contains(required), "missing pre.006 keyset pagination contract: {required}");
+    }
+    for required in [
+        "CURSOR_BYTES: usize = 109",
+        "CURSOR_MAGIC",
+        "b\"KSPT\"",
+        "CURSOR_VERSION: u8 = 1",
+        "KSP/raw-transaction-cursor/v1",
+        "sha2::Sha256",
+        "query.network().as_str()",
+        "query.direction()",
+        "query.slots().start_inclusive()",
+        "query.slots().end_inclusive()",
+        "last_slot",
+        "last_signature",
+        "PageLimitUnsupported",
+        "9_223_372_036_854_775_806",
+    ] {
+        assert!(cursor.contains(required), "missing pre.006 cursor/binding contract: {required}");
+    }
+    assert!(index.contains("ON ksp_raw_transactions (slot, signature)"));
+    assert!(index.contains("WHERE retention_state <> 'purged'"));
+    for forbidden in [" OFFSET ", "limit.min(", "clamp(", "500", "1000", "DELETE FROM", "transition_raw_transaction_retention"] {
+        assert!(!raw.contains(forbidden), "pre.006 contains forbidden pagination/policy/later-scope material: {forbidden}");
+        assert!(!cursor.contains(forbidden), "pre.006 cursor contains forbidden pagination/policy/later-scope material: {forbidden}");
+    }
+    return;
+}

crates/ksp-store-postgres-lib/tests/hardening_completeness.rs

@@ -1,5 +1,5 @@
 // file: crates/ksp-store-postgres-lib/tests/hardening_completeness.rs
-// version: 5
+// version: 6
 
 #![warn(missing_docs)]
 #![deny(unreachable_pub)]
@@ -168,7 +168,13 @@ fn pre_009_backend_error_bridge_cannot_retain_external_error_or_secret_text() {
     let runtime = include_str!("../src/runtime.rs");
     assert!(runtime.contains("deadpool_postgres::PoolError::Backend(_)"));
     assert!(!runtime.contains("deadpool_postgres::PoolError::Backend(error)"));
-    for source in [runtime, include_str!("../src/migration.rs"), include_str!("../src/health.rs"), include_str!("../src/raw_transaction.rs")] {
+    for source in [
+        runtime,
+        include_str!("../src/migration.rs"),
+        include_str!("../src/health.rs"),
+        include_str!("../src/raw_transaction.rs"),
+        include_str!("../src/raw_transaction/cursor.rs"),
+    ] {
         for forbidden in ["format!(\"{error", "format!(\"{error:?", "error = ?", "error = %"] {
             assert!(!source.contains(forbidden), "backend source renders external error material: {forbidden}");
         }
@@ -191,6 +197,7 @@ fn pre_009_backend_has_no_env_bypass_or_direct_store_trait_implementation() {
         include_str!("../src/lib.rs"),
         include_str!("../src/migration.rs"),
         include_str!("../src/raw_transaction.rs"),
+        include_str!("../src/raw_transaction/cursor.rs"),
         include_str!("../src/runtime.rs"),
         include_str!("../src/schema.rs")
     );

crates/ksp-store-postgres-lib/tests/public_api.rs

@@ -1,5 +1,5 @@
 // file: crates/ksp-store-postgres-lib/tests/public_api.rs
-// version: 6
+// version: 7
 
 #![warn(missing_docs)]
 #![deny(unreachable_pub)]
@@ -44,6 +44,8 @@ fn pre_005_backend_error_projection_is_safe_and_static() {
         ksp_store_postgres_lib::PostgresBackendErrorKind::HealthFailed,
         ksp_store_postgres_lib::PostgresBackendErrorKind::MigrationFailed,
         ksp_store_postgres_lib::PostgresBackendErrorKind::MigrationMismatch,
+        ksp_store_postgres_lib::PostgresBackendErrorKind::PageLimitUnsupported,
+        ksp_store_postgres_lib::PostgresBackendErrorKind::QueryInvalid,
         ksp_store_postgres_lib::PostgresBackendErrorKind::ReadFailed,
         ksp_store_postgres_lib::PostgresBackendErrorKind::ReferenceNotFound,
         ksp_store_postgres_lib::PostgresBackendErrorKind::SchemaNewer,
@@ -52,7 +54,7 @@ fn pre_005_backend_error_projection_is_safe_and_static() {
         ksp_store_postgres_lib::PostgresBackendErrorKind::WriteFailed,
         ksp_store_postgres_lib::PostgresBackendErrorKind::WrongNetwork,
     ];
-    assert_eq!(kinds.len(), 15);
+    assert_eq!(kinds.len(), 17);
     return;
 }
 
@@ -87,3 +89,9 @@ fn pre_005_raw_write_bridge_uses_only_backend_independent_models_and_outcomes()
     let _observation = ksp_store_postgres_lib::PostgresBackend::record_raw_transaction_observation;
     return;
 }
+
+#[test]
+fn pre_006_raw_list_bridge_uses_backend_independent_query_page_and_reference_models() {
+    let _list = ksp_store_postgres_lib::PostgresBackend::list_raw_transactions;
+    return;
+}