From eedbc0a86a1d2c3145b25f1a4bf3b936cc318a68 Mon Sep 17 00:00:00 2001 From: seonghobae Date: Tue, 1 Sep 2026 13:25:57 +0000 Subject: [PATCH 1/7] feat(analysis): bind relation-absence refusals to an analysis-run profile GAP-004 leftover / ADR 0071. Bind existing ObservationStatus and refuse_absence_as_negative to cutoff-safe relation_absence_v1 (tepp.relation_absence.v1). Mixed observed/inferred/unobserved census, AvailableTime cutoff exclusion, and MAX_EVIDENCE_UNITS fail closed. status_recovery_rate stays library-side. Distinct from outcome-order, membership-target, location-membership, copied-text, and copy-identity. --- CHANGELOG.md | 2 + Cargo.lock | 1 + DOCUMENTATION.md | 1 + crates/analysis_engine/Cargo.toml | 1 + crates/analysis_engine/src/lib.rs | 15 + .../src/relation_absence_artifact.rs | 420 ++++++++++++++++++ .../relation_absence_execution_contract.rs | 250 +++++++++++ docs/TRACEABILITY.md | 1 + .../adr/0071-relation-absence-analysis-run.md | 98 ++++ docs/adr/README.md | 2 + .../relation-absence-analysis-run.md | 17 + 11 files changed, 808 insertions(+) create mode 100644 crates/analysis_engine/src/relation_absence_artifact.rs create mode 100644 crates/analysis_engine/tests/relation_absence_execution_contract.rs create mode 100644 docs/adr/0071-relation-absence-analysis-run.md create mode 100644 docs/doctoring/relation-absence-analysis-run.md diff --git a/CHANGELOG.md b/CHANGELOG.md index 062a69412..9b5991aed 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -38,6 +38,8 @@ All notable changes to TEPP are documented here. The format follows Keep a Chang ## [Unreleased] +- **Relation-absence analysis-run profile**: `analysis_engine` binds existing `relation_absence::ObservationStatus` and `refuse_absence_as_negative` to cutoff-safe `relation_absence_v1` (`tepp.relation_absence.v1`) with inference status `unobserved_is_not_negative_observed_inferred_are_presence`. `status_recovery_rate` stays library-side. Not outcome-order, not membership-target, not location-membership, not membership-posterior ICC, not copied-text, not copy-identity, not citation-edge, not GPU, not MCMC, and not topic birth/split/merge. + - `event_core` adds bounded Allen interval-consistency classification, atomic path-consistency closure, contradiction/resource refusals, and an explicit dependency-error fallback without claiming unrestricted global satisfiability. - `psychometric_core` recovers the Driver, Oud, and Voelkle (2017, Table 2, p. 12 `MANIFESTTRAITVAR`; §7.1, p. 19; p. 16 `MANIFESTTRAITVARstd`; footnote 4; 2017-era ctsem `summary.ctsemFit.R`; JSS PDF re-opened 2026-08-27T14:20Z from https://www.jstatsoft.org/index.php/jss/article/download/v077i05/1104) scalar standardised manifest-trait variance on current main after `0ce16e8` dropped the pre-consolidation code while research notes already named the map (register items 83–84). Table 2 names `MANIFESTTRAITVAR` `Ψ_τ` the additional time-invariant variance-covariance on the measurement level and sets it `NULL` when there is no manifest trait. Equation 5 writes `Γ ~ N(τ, Ψ)` and names that covariance the manifest traits. Section 7.1 names manifest traits stable individual differences in indicator levels, distinct from process-level `TRAITVAR` `φ_ξ`. Page 16 prints standardised matrices with the suffix `std` when appropriate. The printed example on p. 16 is `discreteDRIFTstd`, not `MANIFESTTRAITVARstd`. Footnote 4 standardises using only the relevant variance, not the total. The relevant variance for that named indicator-level correlation is `MANIFESTTRAITVAR`, not process-level `TRAITVAR` and not residual `MANIFESTVAR` `θ`. The 2017-era source forms `MANIFESTTRAITVARstd` only when `MANIFESTTRAITVAR != 0`, as `solve(sqrt(diag(MANIFESTTRAITVAR) + ridging)) %&% MANIFESTTRAITVAR` when `verbose = TRUE`. OpenMx `%&%` is `t(A) %*% B %*% A`. Unlike `TRAITVARstd`, that formation adds `diag(c(ridging), n.manifest)`. The default `ridging = FALSE` adds 0, not `0.0001`; that ridge is a numerical hack and is not this exact map. The scalar correlation is `ψ / ψ = 1` after strictly positive `MANIFESTTRAITVAR`. Form strictly positive `ψ` first, then `1 / √ψ`, then `(1 / √ψ) ψ (1 / √ψ)`. Unstandardised `MANIFESTTRAITVAR` is defined for a zero trait; standardised `MANIFESTTRAITVAR` is not. Zero `MANIFESTTRAITVAR` skips forming `MANIFESTTRAITVARstd` in the 2017-era source and fails closed here. Indicator-level trait variance is an event-time structural quantity, so a non-event clock fails closed. `MANIFESTTRAITVAR` does not require stable `a < 0`. Distinct positive `ψ` recover the same 1. `trait / trait = 1` is `TRAITVARstd` and recovers the same number and remains a distinct named quantity. `θ` is `MANIFESTVAR` and is measurement error, not this correlation. Meredith (1993) remains unread (web search 2026-08-27T14:20Z: Springer/Cambridge Core paywalled; Unpaywall historically `is_oa: false`; Springer `content/pdf` is an HTML stub). Mislevy (1991, *Psychometrika, 56*, 177–196) remains unread on the same terms (DOI `10.1007/bf02294457`). Still not a Kalman filter, not a matrix `expm`, not ESEM estimation, not DSEM, and not ctsem estimation. diff --git a/Cargo.lock b/Cargo.lock index 454a7d612..7ca904290 100644 --- a/Cargo.lock +++ b/Cargo.lock @@ -74,6 +74,7 @@ dependencies = [ "corpus_split", "event_core", "membership_core", + "relation_absence", "relation_graph", "serde", "serde_json", diff --git a/DOCUMENTATION.md b/DOCUMENTATION.md index 6fa4b9683..ae6ad8a92 100644 --- a/DOCUMENTATION.md +++ b/DOCUMENTATION.md @@ -71,6 +71,7 @@ TEPP's approved PRD v0.4 and implementation plan are the primary product baselin | Hourly NIM OpenCode doctoring | [`docs/doctoring/hourly-nim-opencode-development.md`](docs/doctoring/hourly-nim-opencode-development.md) | | Analysis engine v1 doctoring | [`docs/doctoring/analysis-engine-v1.md`](docs/doctoring/analysis-engine-v1.md) | | Analysis engine gap-closure doctoring | [`docs/doctoring/analysis-engine-gap-closure.md`](docs/doctoring/analysis-engine-gap-closure.md) | +| Relation-absence analysis-run doctoring | [`docs/doctoring/relation-absence-analysis-run.md`](docs/doctoring/relation-absence-analysis-run.md) | | Corpus-split leakage-audit wire doctoring | [`docs/research/corpus-split-manifest-wire.md`](docs/research/corpus-split-manifest-wire.md) | | Unicode canonical-identity doctoring | [`docs/research/unicode-canonical-identity.md`](docs/research/unicode-canonical-identity.md) | | Change history | [`CHANGELOG.md`](CHANGELOG.md) | diff --git a/crates/analysis_engine/Cargo.toml b/crates/analysis_engine/Cargo.toml index 7322212b2..517b7016c 100644 --- a/crates/analysis_engine/Cargo.toml +++ b/crates/analysis_engine/Cargo.toml @@ -15,6 +15,7 @@ publish = false [dependencies] event_core = { path = "../event_core", version = "0.2.0" } +relation_absence = { path = "../relation_absence", version = "0.2.0" } serde = { workspace = true } serde_json = { workspace = true } sha2 = { workspace = true } diff --git a/crates/analysis_engine/src/lib.rs b/crates/analysis_engine/src/lib.rs index 72bd5854c..5033eac9b 100644 --- a/crates/analysis_engine/src/lib.rs +++ b/crates/analysis_engine/src/lib.rs @@ -12,6 +12,7 @@ mod case_deletion_refit; mod lineage_criterion; +mod relation_absence_artifact; mod topic_context_posterior; mod topic_lineage_artifact; @@ -46,6 +47,13 @@ pub use lineage_criterion::{ LineageCriterionFit, LineageCriterionFitError, LineageCriterionObservation, fit_lineage_criterion_posteriors, }; +/// Relation-absence artifact and execution contracts from this engine. +pub use relation_absence_artifact::{ + RELATION_ABSENCE_ARTIFACT_BYTE_LIMIT, RELATION_ABSENCE_ARTIFACT_SCHEMA_VERSION, + RELATION_ABSENCE_MODEL_CONTRACT_VERSION, RELATION_ABSENCE_OUTPUT_PROFILE, + RelationAbsenceArtifact, RelationAbsenceExecution, RelationAbsencePair, + execute_relation_absence_run, +}; /// Bounded posterior topic-context producer contract and record types. pub use topic_context_posterior::{ TOPIC_CONTEXT_POSTERIOR_BYTE_LIMIT, TOPIC_CONTEXT_POSTERIOR_SCHEMA_VERSION, @@ -248,6 +256,8 @@ pub enum AnalysisEngineError { TopicMeasurement(TopicMeasurementError), /// A topic-lineage artifact violated its bounded schema or count invariants. InvalidTopicLineageArtifact, + /// A relation-absence artifact violated its bounded schema or count invariants. + InvalidRelationAbsenceArtifact, } impl fmt::Display for AnalysisEngineError { @@ -262,6 +272,7 @@ impl fmt::Display for AnalysisEngineError { Self::LimitExceeded => "analysis corpus exceeded its execution bound", Self::TopicMeasurement(error) => return error.fmt(formatter), Self::InvalidTopicLineageArtifact => "invalid topic lineage artifact", + Self::InvalidRelationAbsenceArtifact => "invalid relation-absence artifact", }; formatter.write_str(message) } @@ -681,6 +692,10 @@ mod tests { AnalysisEngineError::InvalidTopicLineageArtifact, "invalid topic lineage artifact", ), + ( + AnalysisEngineError::InvalidRelationAbsenceArtifact, + "invalid relation-absence artifact", + ), ]; for (error, message) in messages { assert_eq!(error.to_string(), message); diff --git a/crates/analysis_engine/src/relation_absence_artifact.rs b/crates/analysis_engine/src/relation_absence_artifact.rs new file mode 100644 index 000000000..b434ab939 --- /dev/null +++ b/crates/analysis_engine/src/relation_absence_artifact.rs @@ -0,0 +1,420 @@ +//! Digest-bound relation-absence refusals as an analysis-run profile. + +use relation_absence::{ObservationStatus, RelationAbsenceError, refuse_absence_as_negative}; +use serde::{Deserialize, Serialize}; +use sha2::{Digest, Sha256}; +use temporal_core::{AvailableTime, KnowledgeCutoff}; +use tepp_api::{ + AnalysisResultSummary, AnalysisRunAccepted, AnalysisRunRequest, AnalysisRunTerminalResult, +}; + +use crate::{ + AnalysisEngineError, MAX_EVIDENCE_UNITS, format_digest, require_receipt_identity, + valid_identifier, +}; + +/// Versioned schema for a completed relation-absence artifact. +pub const RELATION_ABSENCE_ARTIFACT_SCHEMA_VERSION: &str = "tepp.relation_absence.v1"; +/// Model contract required by the relation-absence execution path. +pub const RELATION_ABSENCE_MODEL_CONTRACT_VERSION: &str = "relation_absence_v1"; +/// Analysis-run output profile required for a relation-absence artifact. +pub const RELATION_ABSENCE_OUTPUT_PROFILE: &str = "relation_absence_v1"; +/// Maximum canonical artifact JSON size. +pub const RELATION_ABSENCE_ARTIFACT_BYTE_LIMIT: usize = 256 * 1024; +const RELATION_ABSENCE_INFERENCE_STATUS: &str = + "unobserved_is_not_negative_observed_inferred_are_presence"; + +/// One cutoff-admitted relation pair with closed observation status. +#[derive(Clone, Debug, Eq, PartialEq)] +pub struct RelationAbsencePair { + pair_id: String, + status: ObservationStatus, + available_time: AvailableTime, +} + +impl RelationAbsencePair { + /// Construct a bounded relation-absence pair. + /// + /// # Errors + /// + /// Returns [`AnalysisEngineError::InvalidEvidence`] when the pair identity + /// is empty or oversized. + pub fn new( + pair_id: impl Into, + status: ObservationStatus, + available_time: AvailableTime, + ) -> Result { + let pair_id = pair_id.into(); + if !valid_identifier(&pair_id) { + return Err(AnalysisEngineError::InvalidEvidence); + } + Ok(Self { + pair_id, + status, + available_time, + }) + } + + /// Return the opaque pair identity. + #[must_use] + pub fn pair_id(&self) -> &str { + &self.pair_id + } + + /// Return the closed observation status. + #[must_use] + pub const fn status(&self) -> ObservationStatus { + self.status + } + + /// Return the availability time used for cutoff eligibility. + #[must_use] + pub const fn available_time(&self) -> AvailableTime { + self.available_time + } +} + +/// Completed, bounded relation-absence census for analysis-run clients. +#[derive(Clone, Debug, Deserialize, PartialEq, Serialize)] +#[serde(deny_unknown_fields)] +pub struct RelationAbsenceArtifact { + /// Exact versioned schema identity. + pub schema_version: String, + /// Opaque accepted-run identity. + pub run_id: String, + /// Immutable source snapshot identity. + pub snapshot_id: String, + /// Historical evidence cutoff used to admit pairs. + pub knowledge_cutoff: String, + /// Number of pairs admitted at the cutoff. + pub pair_count: u64, + /// Directly observed pairs admitted at the cutoff. + pub observed_count: u64, + /// Inferred pairs admitted at the cutoff. + pub inferred_count: u64, + /// Unobserved pairs admitted at the cutoff. + pub unobserved_count: u64, + /// Unobserved pairs refused as negative evidence. + pub refused_as_negative_count: u64, + /// Fixed claim boundary for consumer copy. + pub inference_status: String, +} + +impl RelationAbsenceArtifact { + /// Parse and fully validate a bounded artifact JSON payload. + /// + /// # Errors + /// + /// Returns [`AnalysisEngineError::InvalidRelationAbsenceArtifact`] when the + /// schema, identifiers, counts, or claim boundary fail. + pub fn from_json(payload: &str) -> Result { + if payload.len() > RELATION_ABSENCE_ARTIFACT_BYTE_LIMIT { + return Err(AnalysisEngineError::LimitExceeded); + } + let artifact: Self = serde_json::from_str(payload) + .map_err(|_| AnalysisEngineError::InvalidRelationAbsenceArtifact)?; + artifact.validate()?; + Ok(artifact) + } + + /// Serialize canonical validated artifact JSON. + /// + /// # Errors + /// + /// Returns a typed validation, serialization, or size failure. + pub fn to_json(&self) -> Result { + self.validate()?; + let payload = + serde_json::to_string(self).map_err(|_| AnalysisEngineError::SerializationFailure)?; + if payload.len() > RELATION_ABSENCE_ARTIFACT_BYTE_LIMIT { + return Err(AnalysisEngineError::LimitExceeded); + } + Ok(payload) + } + + /// Return the lowercase SHA-256 digest of canonical artifact JSON. + /// + /// # Errors + /// + /// Returns a typed validation or serialization failure. + pub fn sha256(&self) -> Result { + self.to_json() + .map(|json| format_digest(Sha256::digest(json.into_bytes()))) + } + + fn validate(&self) -> Result<(), AnalysisEngineError> { + let presence_sum = self.observed_count.checked_add(self.inferred_count); + let status_sum = presence_sum.and_then(|value| value.checked_add(self.unobserved_count)); + if self.schema_version != RELATION_ABSENCE_ARTIFACT_SCHEMA_VERSION + || !valid_identifier(&self.run_id) + || !valid_identifier(&self.snapshot_id) + || KnowledgeCutoff::parse_rfc3339(&self.knowledge_cutoff).is_err() + || self.pair_count < 3 + || self.observed_count == 0 + || self.inferred_count == 0 + || self.unobserved_count == 0 + || status_sum != Some(self.pair_count) + || self.refused_as_negative_count != self.unobserved_count + || self.inference_status != RELATION_ABSENCE_INFERENCE_STATUS + { + return Err(AnalysisEngineError::InvalidRelationAbsenceArtifact); + } + Ok(()) + } +} + +/// One completed relation-absence artifact and its terminal result. +#[derive(Clone, Debug, PartialEq)] +pub struct RelationAbsenceExecution { + /// Digest-bound completed relation-absence census. + pub artifact: RelationAbsenceArtifact, + /// Terminal result carrying the artifact identity, digest, and schema. + pub terminal_result: AnalysisRunTerminalResult, +} + +/// Execute cutoff-safe relation-absence refusals as one analysis-run profile. +/// +/// The executor invokes [`refuse_absence_as_negative`] already on protected +/// main. Observed and inferred statuses stay presence. Unobserved stays a +/// missing-status, never a negative edge. It does not emit +/// `status_recovery_rate`, a `scientific_acceptance` inspect metric, GPU +/// kernels, MCMC, or topic birth/split/merge events. +/// +/// # Errors +/// +/// Returns a request/receipt/snapshot/cutoff/profile error, empty or +/// single-class corpus, unobserved treated as negative, duplicate pair +/// identity, oversized corpus, or invalid artifact error. +pub fn execute_relation_absence_run( + request: &AnalysisRunRequest, + accepted: &AnalysisRunAccepted, + snapshot_id: &str, + knowledge_cutoff: KnowledgeCutoff, + pairs: &[RelationAbsencePair], + completed_at: impl Into, +) -> Result { + request.to_json()?; + accepted.to_json()?; + require_receipt_identity(request, accepted)?; + if request.snapshot_id != snapshot_id { + return Err(AnalysisEngineError::SnapshotMismatch); + } + if request.knowledge_cutoff != knowledge_cutoff.to_rfc3339() + || request.model_contract_version != RELATION_ABSENCE_MODEL_CONTRACT_VERSION + || request.output_profile != RELATION_ABSENCE_OUTPUT_PROFILE + { + return Err(AnalysisEngineError::InvalidEvidence); + } + if pairs.len() > MAX_EVIDENCE_UNITS { + return Err(AnalysisEngineError::LimitExceeded); + } + + let mut seen = std::collections::BTreeSet::new(); + let mut observed_count = 0_u64; + let mut inferred_count = 0_u64; + let mut unobserved_count = 0_u64; + let mut refused_as_negative_count = 0_u64; + for pair in pairs { + if !seen.insert(pair.pair_id()) { + return Err(AnalysisEngineError::DuplicateEvidence); + } + if pair.available_time().instant() > knowledge_cutoff.instant() { + continue; + } + match pair.status() { + ObservationStatus::Observed => { + refuse_absence_as_negative(pair.status()).map_err(map_relation_absence_error)?; + observed_count = increment(observed_count)?; + } + ObservationStatus::Inferred => { + refuse_absence_as_negative(pair.status()).map_err(map_relation_absence_error)?; + inferred_count = increment(inferred_count)?; + } + ObservationStatus::Unobserved => { + match refuse_absence_as_negative(pair.status()) { + Err(RelationAbsenceError::AbsenceIsNotNegative) => { + refused_as_negative_count = increment(refused_as_negative_count)?; + } + Ok(()) | Err(_) => return Err(AnalysisEngineError::InvalidEvidence), + } + unobserved_count = increment(unobserved_count)?; + } + } + } + + let pair_count = observed_count + .checked_add(inferred_count) + .and_then(|value| value.checked_add(unobserved_count)) + .ok_or(AnalysisEngineError::ArithmeticOverflow)?; + if pair_count < 3 + || observed_count == 0 + || inferred_count == 0 + || unobserved_count == 0 + || refused_as_negative_count != unobserved_count + { + return Err(AnalysisEngineError::InvalidEvidence); + } + + let artifact = RelationAbsenceArtifact { + schema_version: RELATION_ABSENCE_ARTIFACT_SCHEMA_VERSION.into(), + run_id: accepted.run_id.clone(), + snapshot_id: snapshot_id.to_owned(), + knowledge_cutoff: knowledge_cutoff.to_rfc3339(), + pair_count, + observed_count, + inferred_count, + unobserved_count, + refused_as_negative_count, + inference_status: RELATION_ABSENCE_INFERENCE_STATUS.into(), + }; + let digest = artifact.sha256()?; + let summary = AnalysisResultSummary::new( + "relation_absence", + pair_count, + 4, + RELATION_ABSENCE_INFERENCE_STATUS, + )?; + let terminal_result = AnalysisRunTerminalResult::succeeded( + request, + accepted, + format!("relation_absence_artifact_{}", &digest[..16]), + digest, + RELATION_ABSENCE_ARTIFACT_SCHEMA_VERSION, + completed_at, + summary, + )?; + Ok(RelationAbsenceExecution { + artifact, + terminal_result, + }) +} + +fn increment(count: u64) -> Result { + count + .checked_add(1) + .ok_or(AnalysisEngineError::ArithmeticOverflow) +} + +fn map_relation_absence_error(error: RelationAbsenceError) -> AnalysisEngineError { + match error { + RelationAbsenceError::AbsenceIsNotNegative + | RelationAbsenceError::InvalidObservationPayload + | _ => AnalysisEngineError::InvalidEvidence, + } +} + +#[cfg(test)] +mod tests { + use super::{ + RELATION_ABSENCE_ARTIFACT_BYTE_LIMIT, RELATION_ABSENCE_ARTIFACT_SCHEMA_VERSION, + RELATION_ABSENCE_INFERENCE_STATUS, RelationAbsenceArtifact, + }; + use crate::AnalysisEngineError; + + fn artifact() -> RelationAbsenceArtifact { + RelationAbsenceArtifact { + schema_version: RELATION_ABSENCE_ARTIFACT_SCHEMA_VERSION.into(), + run_id: "run-1".into(), + snapshot_id: "snapshot-1".into(), + knowledge_cutoff: "2026-08-01T00:00:00Z".into(), + pair_count: 3, + observed_count: 1, + inferred_count: 1, + unobserved_count: 1, + refused_as_negative_count: 1, + inference_status: RELATION_ABSENCE_INFERENCE_STATUS.into(), + } + } + + fn assert_invalid(artifact: &RelationAbsenceArtifact) { + assert_eq!( + artifact.to_json(), + Err(AnalysisEngineError::InvalidRelationAbsenceArtifact) + ); + } + + #[test] + fn artifact_round_trip_and_size_bounds_fail_closed() { + let artifact = artifact(); + let payload = artifact.to_json().expect("json"); + assert_eq!( + RelationAbsenceArtifact::from_json(&payload), + Ok(artifact.clone()) + ); + assert_eq!(artifact.sha256().expect("digest").len(), 64); + assert_eq!( + RelationAbsenceArtifact::from_json("{}"), + Err(AnalysisEngineError::InvalidRelationAbsenceArtifact) + ); + assert_eq!( + RelationAbsenceArtifact::from_json( + &"x".repeat(RELATION_ABSENCE_ARTIFACT_BYTE_LIMIT + 1) + ), + Err(AnalysisEngineError::LimitExceeded) + ); + } + + #[test] + fn artifact_metadata_tampering_fails_closed() { + let artifact = artifact(); + let invalid_artifacts = [ + { + let mut value = artifact.clone(); + value.schema_version.clear(); + value + }, + { + let mut value = artifact.clone(); + value.run_id.clear(); + value + }, + { + let mut value = artifact.clone(); + value.snapshot_id.clear(); + value + }, + { + let mut value = artifact.clone(); + value.knowledge_cutoff = "invalid".into(); + value + }, + { + let mut value = artifact.clone(); + value.pair_count = 2; + value + }, + { + let mut value = artifact.clone(); + value.observed_count = 0; + value.pair_count = 2; + value + }, + { + let mut value = artifact.clone(); + value.inferred_count = 0; + value.pair_count = 2; + value + }, + { + let mut value = artifact.clone(); + value.unobserved_count = 0; + value.refused_as_negative_count = 0; + value.pair_count = 2; + value + }, + { + let mut value = artifact.clone(); + value.refused_as_negative_count = 0; + value + }, + { + let mut value = artifact.clone(); + value.inference_status.clear(); + value + }, + ]; + for invalid in invalid_artifacts { + assert_invalid(&invalid); + } + } +} diff --git a/crates/analysis_engine/tests/relation_absence_execution_contract.rs b/crates/analysis_engine/tests/relation_absence_execution_contract.rs new file mode 100644 index 000000000..ce7ca89eb --- /dev/null +++ b/crates/analysis_engine/tests/relation_absence_execution_contract.rs @@ -0,0 +1,250 @@ +//! End-to-end contract for cutoff-safe relation-absence refusals. + +use analysis_engine::{ + AnalysisEngineError, MAX_EVIDENCE_UNITS, RELATION_ABSENCE_ARTIFACT_SCHEMA_VERSION, + RELATION_ABSENCE_MODEL_CONTRACT_VERSION, RELATION_ABSENCE_OUTPUT_PROFILE, RelationAbsencePair, + execute_relation_absence_run, +}; +use relation_absence::ObservationStatus; +use temporal_core::{AvailableTime, KnowledgeCutoff}; +use tepp_api::{AnalysisRunAccepted, AnalysisRunRequest, AnalysisRunTerminalState}; + +fn cutoff() -> KnowledgeCutoff { + KnowledgeCutoff::parse_rfc3339("2026-08-01T00:00:00Z").expect("cutoff") +} + +fn available(stamp: &str) -> AvailableTime { + AvailableTime::parse_rfc3339(stamp).expect("available") +} + +fn request() -> AnalysisRunRequest { + AnalysisRunRequest { + contract_version: 1, + idempotency_key: "relation-absence-idem".into(), + tenant_workspace_id: "tenant-workspace".into(), + snapshot_id: "snapshot-relation-absence".into(), + knowledge_cutoff: "2026-08-01T00:00:00Z".into(), + model_contract_version: RELATION_ABSENCE_MODEL_CONTRACT_VERSION.into(), + output_profile: RELATION_ABSENCE_OUTPUT_PROFILE.into(), + } +} + +fn accepted(request: &AnalysisRunRequest) -> AnalysisRunAccepted { + AnalysisRunAccepted::new("run-relation-absence", "accepted", &request.idempotency_key) + .expect("accepted") +} + +fn pair(pair_id: &str, status: ObservationStatus, stamp: &str) -> RelationAbsencePair { + RelationAbsencePair::new(pair_id, status, available(stamp)).expect("pair") +} + +fn mixed_pairs() -> Vec { + vec![ + pair( + "observed-a", + ObservationStatus::Observed, + "2026-07-01T00:00:00Z", + ), + pair( + "inferred-b", + ObservationStatus::Inferred, + "2026-07-02T00:00:00Z", + ), + pair( + "unobserved-c", + ObservationStatus::Unobserved, + "2026-07-03T00:00:00Z", + ), + ] +} + +fn execute( + request: &AnalysisRunRequest, + pairs: &[RelationAbsencePair], +) -> Result { + execute_relation_absence_run( + request, + &accepted(request), + "snapshot-relation-absence", + cutoff(), + pairs, + "2026-08-02T00:00:00Z", + ) +} + +#[test] +fn mixed_statuses_emit_digest_bound_refusals_without_recovery_metric() { + let request = request(); + let execution = execute(&request, &mixed_pairs()).expect("execution"); + assert_eq!( + execution.artifact.schema_version, + RELATION_ABSENCE_ARTIFACT_SCHEMA_VERSION + ); + assert_eq!(execution.artifact.pair_count, 3); + assert_eq!(execution.artifact.observed_count, 1); + assert_eq!(execution.artifact.inferred_count, 1); + assert_eq!(execution.artifact.unobserved_count, 1); + assert_eq!(execution.artifact.refused_as_negative_count, 1); + assert_eq!( + execution.artifact.inference_status, + "unobserved_is_not_negative_observed_inferred_are_presence" + ); + let payload = execution.artifact.to_json().expect("json"); + assert!(!payload.contains("status_recovery_rate")); + assert!(!payload.contains("scientific_acceptance")); + assert!(!payload.contains("no_relationship")); + assert_eq!( + execution.terminal_result.run_state, + AnalysisRunTerminalState::Succeeded + ); + assert_eq!( + execution.terminal_result.result_sha256.as_deref(), + Some(execution.artifact.sha256().expect("digest").as_str()) + ); + assert_eq!( + execution.terminal_result.result_schema_version.as_deref(), + Some(RELATION_ABSENCE_ARTIFACT_SCHEMA_VERSION) + ); +} + +#[test] +fn future_available_pairs_are_excluded() { + let request = request(); + let mut with_future = mixed_pairs(); + with_future.push(pair( + "future-d", + ObservationStatus::Observed, + "2026-08-02T00:00:00Z", + )); + let execution = execute(&request, &with_future).expect("cutoff"); + assert_eq!(execution.artifact.pair_count, 3); + assert_eq!(execution.artifact.observed_count, 1); +} + +#[test] +fn empty_or_single_class_and_duplicate_fail_closed() { + let request = request(); + let stamp = "2026-07-01T00:00:00Z"; + assert_eq!( + execute(&request, &[]), + Err(AnalysisEngineError::InvalidEvidence) + ); + let observed_only = vec![ + pair("observed-a", ObservationStatus::Observed, stamp), + pair("observed-b", ObservationStatus::Observed, stamp), + pair("observed-c", ObservationStatus::Observed, stamp), + ]; + assert_eq!( + execute(&request, &observed_only), + Err(AnalysisEngineError::InvalidEvidence) + ); + let presence_only = vec![ + pair("observed-a", ObservationStatus::Observed, stamp), + pair("inferred-b", ObservationStatus::Inferred, stamp), + ]; + assert_eq!( + execute(&request, &presence_only), + Err(AnalysisEngineError::InvalidEvidence) + ); + let unobserved_only = vec![ + pair("unobserved-a", ObservationStatus::Unobserved, stamp), + pair("unobserved-b", ObservationStatus::Unobserved, stamp), + pair("unobserved-c", ObservationStatus::Unobserved, stamp), + ]; + assert_eq!( + execute(&request, &unobserved_only), + Err(AnalysisEngineError::InvalidEvidence) + ); + let duplicates = vec![ + pair("same", ObservationStatus::Observed, stamp), + pair("same", ObservationStatus::Unobserved, stamp), + ]; + assert_eq!( + execute(&request, &duplicates), + Err(AnalysisEngineError::DuplicateEvidence) + ); + assert_eq!( + RelationAbsencePair::new("", ObservationStatus::Observed, available(stamp)), + Err(AnalysisEngineError::InvalidEvidence) + ); +} + +#[test] +fn execution_refuses_snapshot_profile_cutoff_mismatch_and_oversize() { + let request = request(); + let pairs = mixed_pairs(); + assert_eq!( + execute_relation_absence_run( + &request, + &accepted(&request), + "other-snapshot", + cutoff(), + &pairs, + "2026-08-02T00:00:00Z", + ), + Err(AnalysisEngineError::SnapshotMismatch) + ); + let mut mismatched = request.clone(); + mismatched.knowledge_cutoff = "2026-07-01T00:00:00Z".into(); + assert_eq!( + execute_relation_absence_run( + &mismatched, + &accepted(&mismatched), + "snapshot-relation-absence", + cutoff(), + &pairs, + "2026-08-02T00:00:00Z", + ), + Err(AnalysisEngineError::InvalidEvidence) + ); + for profile in [ + "trsl_topic_lineage_v1", + "fitted_candidate_k_v1", + "pareto_candidate_k_v1", + "joint_posterior_draws_v1", + "method_effects_v1", + "copy_identity_v1", + "style_source_v1", + "prompt_source_v1", + "modality_source_v1", + "corpus_background_v1", + "citation_edge_v1", + "copied_text_v1", + "lineage_criterion_v1", + "composed_fitted_lineage_v1", + "case_deletion_refit_v1", + "topic_activity_v1", + "location_membership_v1", + "topic_context_posterior_v1", + "membership_posterior_icc_v1", + "membership_target_v1", + "outcome_order_v1", + ] { + let mut reused = request.clone(); + reused.output_profile = profile.into(); + assert_eq!( + execute_relation_absence_run( + &reused, + &accepted(&reused), + "snapshot-relation-absence", + cutoff(), + &pairs, + "2026-08-02T00:00:00Z", + ), + Err(AnalysisEngineError::InvalidEvidence) + ); + } + let oversized: Vec = (0..=MAX_EVIDENCE_UNITS) + .map(|index| { + pair( + &format!("pair-{index}"), + ObservationStatus::Observed, + "2026-07-01T00:00:00Z", + ) + }) + .collect(); + assert_eq!( + execute(&request, &oversized), + Err(AnalysisEngineError::LimitExceeded) + ); +} diff --git a/docs/TRACEABILITY.md b/docs/TRACEABILITY.md index 2b783c2ab..b3b785acc 100644 --- a/docs/TRACEABILITY.md +++ b/docs/TRACEABILITY.md @@ -58,6 +58,7 @@ The full APA 7th standards/literature register remains `docs/research/standards- | versioned service/API contracts and exports | PRD; API contract; ADR 0011/0013 | `tepp_api` analysis-run/export/JSON-LD/GraphML contracts on protected main (PR #21); HTTP service remaining accepted-target | partial | | versioned service/API contracts and exports | PRD; API contract; ADR 0011/0013 | `tepp_api` analysis-run/export/JSON-LD/GraphML contracts on protected main (PR #21); LineageWeave loopback contracts and request-bound terminal result are composed on the active product branch; production TLS remaining | partial | | executable cutoff-safe analysis runs | ADR 0012/0022; temporal research; API terminal-result contract | `analysis_engine` availability cutoff, snapshot binding, multiple-membership aggregation, digest-bound readiness artifact, and `tepp.trsl_topic_lineage.v1` execution through `topic_measurement`; synthetic recovery plus tamper/non-convergence tests and exact coverage on the active product branch | active-PR | +| relation-absence analysis-run profile | ADR 0003/0022/0071; `unobserved` is not a negative edge; `observed`/`inferred` stay presence | `analysis_engine` `relation_absence_v1` binds `refuse_absence_as_negative`; digest-bound refusals, not `status_recovery_rate` inspect metric, not outcome-order, not membership-target, not location-membership, not GPU, not MCMC, not topic birth/split/merge; not implemented-main | active-PR | | immutable split/run/reproducibility manifests | ADR 0013; ERD | `tepp_api` reproducibility manifest contract on protected main; `persistence_postgres` append-only SQL insert/lookup for `reproducibility_manifest`, `corpus_split_manifest`, `model_run`, and `model_artifact` (migration `0003`); full physical ERD constraints remaining | partial | | multilingual shared latent semantic space | PRD; ADR 0004; ADR 0020 | `semantic_core` span-grounded units (active-PR); concept dictionary and shared latent estimator remaining | active-PR | | TRSL-TM temporal/relational topic posterior and backend compatibility | ADR 0012; ADR 0004 | `topic_measurement` stable ALR/ILR coordinates and bounded CPU `f64` reference estimator on protected main; `model_selection` fitted candidate-`K` scoring on this PR; calibrated posterior promotion, method effects, persistence, and accelerated backends remaining | partial | diff --git a/docs/adr/0071-relation-absence-analysis-run.md b/docs/adr/0071-relation-absence-analysis-run.md new file mode 100644 index 000000000..1424e6ee0 --- /dev/null +++ b/docs/adr/0071-relation-absence-analysis-run.md @@ -0,0 +1,98 @@ +# ADR 0071 — Relation-absence refusals as an analysis-run output profile + +**Decision status:** Accepted +**Implementation maturity:** active-PR — composed on this branch; not implemented-main +**Date:** 2026-09-01 +**Supersedes:** None; complements ADR 0003 (unobserved pairs are not negative edges) and ADR 0022 (cutoff-safe analysis-run execution). Does not reuse ADR 0070 (outcome-order), ADR 0069 (membership-target), ADR 0068 (topic-context posterior), ADR 0066 (location-membership), ADR 0065 (copied-text residue), ADR 0064 (provenance-is-not-transition / citation-edge), or ADR 0058 (copy-identity / template-copy). +**Figma File ID:** N/A — this increment changes a Rust service crate and has no user-interface surface. +**Storybook inventory:** N/A — no reusable web object or interaction changed. + +## Context + +Protected main already refuses to treat an unobserved relation pair as +evidence of no relationship, via `relation_absence::ObservationStatus` +and `refuse_absence_as_negative`. Operators still cannot request that +census as a digest-bound analysis-run output. + +Outcome-order (#458 / ADR 0070) binds IPO event-time order. +Membership-target (#434 / ADR 0069) binds `MembershipTargetKind`. +Location-membership (#430 / ADR 0066) binds geographic/market assignment. +Copied-text (#427 / ADR 0065) binds unique-content/stopword vocabulary. +Copy-identity (#416 / ADR 0058) binds template-copy identity. +Provenance-is-not-transition (#426 / ADR 0064) binds citation-edge. + +`status_recovery_rate` stays library-side. This slice does not put a +`scientific_acceptance` metric on inspect payloads. `no_relationship` +is not a wire status. + +GPU kernels, MCMC, and topic birth/split/merge remain later GAP-004 work +and are not this slice. + +## Decision + +Add the `relation_absence_v1` analysis-run output profile to +`analysis_engine`. The executor: + +- consumes already-validated `RelationAbsencePair` rows with closed + `ObservationStatus` values and availability time; +- requires the request snapshot and knowledge cutoff to match the offered + input construction; +- excludes pairs whose availability is later than the knowledge cutoff; +- invokes `refuse_absence_as_negative` without reimplementing the + `observed` / `inferred` / `unobserved` vocabulary; +- requires a mixed census of at least one `observed`, one `inferred`, + and one `unobserved` after cutoff exclusion; +- emits a canonical SHA-256-digested `tepp.relation_absence.v1` artifact + with per-status counts, matching unobserved refusal counts, and + inference status `unobserved_is_not_negative_observed_inferred_are_presence`; +- does not emit `status_recovery_rate`, invent MCMC, select GPU backends, + or emit topic birth/split/merge events. + +## Alternatives considered + +1. Duplicate outcome-order (#458 / ADR 0070) — rejected because that + profile binds IPO event-time order, not observation status. +2. Duplicate membership-target (#434 / ADR 0069) — rejected because that + profile binds `MembershipTargetKind`, not absence-is-not-negative. +3. Duplicate location-membership (#430 / ADR 0066) — rejected because + that profile binds `location_membership` LocationKind refusals. +4. Duplicate copied-text (#427) or copy-identity (#416) — rejected + because those profiles bind residue/template identity, not observation + status. +5. Put `status_recovery_rate` on the operator artifact — rejected because + inspect payloads stay metric-free and `tepp.scientific_acceptance.v1` + never appears. +6. Bind the existing relation-absence refusals to ADR 0022's analysis-run + profile — accepted. + +## Consequences + +Operators can request cutoff-safe relation-absence refusals as a +digest-bound terminal result. The artifact does not claim MCMC, GPU +parity, outcome-order, membership-target, location-membership, +membership-posterior ICC, copied-text, copy-identity, citation-edge, +corpus-background, method-effect estimation, or topic birth/split/merge. +Snapshot / profile / cutoff mismatch, empty or single-class corpora, +duplicate pair identities, and oversized corpora fail closed. + +## Verification + +The PR includes Rust unit and integration tests for mixed +`observed` / `inferred` / `unobserved` corpora, cutoff exclusion, +empty/single-class/duplicate refusal, snapshot / profile / cutoff +mismatch, oversize, and artifact tampering. Run: + +```text +cargo fmt --all -- --check +cargo test -p analysis_engine +cargo clippy -p analysis_engine --all-targets -- -D warnings +python3 scripts/validate_documentation.py +``` + +## Rollback and supersession + +Rollback removes the `relation_absence_v1` profile. No persisted schema +migration is introduced. Supersede only with an ADR that keeps +`unobserved` out of the negative-edge vocabulary, keeps `observed` and +`inferred` as presence, and keeps `status_recovery_rate` off inspect +payloads. diff --git a/docs/adr/README.md b/docs/adr/README.md index 1254c8079..3a98f91ef 100644 --- a/docs/adr/README.md +++ b/docs/adr/README.md @@ -30,6 +30,7 @@ Read [`ADR_POLICY.md`](ADR_POLICY.md) first. **Decision status and implementatio | [0022](0022-deterministic-analysis-run-execution.md) | Deterministic cutoff-safe analysis-run execution | Accepted | active-PR | Closes the first executable product path from accepted run to digest-bound terminal result without claiming estimator authority. | | [0024](0024-lineage-pair-criterion-and-project-journey-posterior.md) | Independent Event Lineage pair criterion and posterior Project Journey | Proposed | active-PR | Strict artifacts preserve criterion/event-time draws, branches, ties, and CPU/GPU receipts without claiming the scientific estimator is complete. | | [0025](0025-macos-native-rust-mlx-metal-boundary.md) | macOS-native Rust-owned MLX Metal execution | Accepted | accepted-target | Compose authenticates to a native host service; Linux never claims Metal, and actual backend/parity receipts fail closed. | +| [0071](0071-relation-absence-analysis-run.md) | Relation-absence refusals as an analysis-run profile | Accepted | active-PR | Complements ADR 0003/0022; `ObservationStatus` + `refuse_absence_as_negative`, not outcome-order, not membership-target. | | [0023](0023-lineage-criterion-anchor-contract.md) | TEPP-owned Event Lineage criterion anchor | Accepted | active-PR | PR #237 publishes the strict accepted/rejected artifact and identities; estimator execution remains fail-closed future work. | | [0024](0024-independent-topic-importance-anchor.md) | Posterior topic-context producer contract | Accepted | contract-only active-PR | Strict DTO/schema only; the current estimator does not emit it. fast-mlsirm owns case-deletion influence. | | [0001](0001-rust-first-modular-msa.md) | Rust-first numerical core and CPU `f64` reference | Accepted | partial | ADR 0011 owns cross-service/MSA authority; 0001 retains numerical/backend authority. | @@ -138,6 +139,7 @@ Use the narrowest owning ADR when decisions overlap: - **project-history wire-size symmetry:** ADR 0019. - **LineageWeave project-history service boundary:** ADR 0021. - **accepted-run execution and terminal artifact production:** ADR 0022. +- **relation-absence analysis-run profile:** ADR 0071. - **independent lineage criterion and posterior Project Journey:** ADR 0023. - **macOS-native Rust-owned MLX Metal execution:** ADR 0024. diff --git a/docs/doctoring/relation-absence-analysis-run.md b/docs/doctoring/relation-absence-analysis-run.md new file mode 100644 index 000000000..5be073e04 --- /dev/null +++ b/docs/doctoring/relation-absence-analysis-run.md @@ -0,0 +1,17 @@ +# Relation-absence analysis-run composition + +**Active slice:** ADR 0071 / `relation_absence_v1` +**Protected-main status:** not implemented-main + +`relation_absence` already refuses to treat an unobserved pair as +evidence of no relationship. This slice binds `ObservationStatus` and +`refuse_absence_as_negative` to a cutoff-safe analysis-run profile so +operators can request a digest-bound identity artifact. + +The artifact inference status is +`unobserved_is_not_negative_observed_inferred_are_presence`. +`status_recovery_rate` stays library-side. `no_relationship` is not a +wire status. This is not outcome-order, not membership-target, not +location-membership, not membership-posterior ICC, not copied-text, not +copy-identity, not citation-edge, not GPU, not MCMC, and not topic +birth/split/merge. From 8c6c6865e5e8ec3625e792b4c7e17c556fc4d042 Mon Sep 17 00:00:00 2001 From: Seongho Bae Date: Wed, 2 Sep 2026 07:04:33 +0900 Subject: [PATCH 2/7] test(analysis): reject impossible compact relation census --- .../relation_absence_execution_contract.rs | 31 +++++++++++++++++-- 1 file changed, 29 insertions(+), 2 deletions(-) diff --git a/crates/analysis_engine/tests/relation_absence_execution_contract.rs b/crates/analysis_engine/tests/relation_absence_execution_contract.rs index ce7ca89eb..c728636ab 100644 --- a/crates/analysis_engine/tests/relation_absence_execution_contract.rs +++ b/crates/analysis_engine/tests/relation_absence_execution_contract.rs @@ -2,8 +2,8 @@ use analysis_engine::{ AnalysisEngineError, MAX_EVIDENCE_UNITS, RELATION_ABSENCE_ARTIFACT_SCHEMA_VERSION, - RELATION_ABSENCE_MODEL_CONTRACT_VERSION, RELATION_ABSENCE_OUTPUT_PROFILE, RelationAbsencePair, - execute_relation_absence_run, + RELATION_ABSENCE_MODEL_CONTRACT_VERSION, RELATION_ABSENCE_OUTPUT_PROFILE, + RelationAbsenceArtifact, RelationAbsencePair, execute_relation_absence_run, }; use relation_absence::ObservationStatus; use temporal_core::{AvailableTime, KnowledgeCutoff}; @@ -107,6 +107,33 @@ fn mixed_statuses_emit_digest_bound_refusals_without_recovery_metric() { ); } +#[test] +fn compact_oversized_artifact_counts_fail_closed() { + let pair_count = MAX_EVIDENCE_UNITS as u64 + 1; + let unobserved_count = pair_count - 2; + let artifact = RelationAbsenceArtifact { + schema_version: RELATION_ABSENCE_ARTIFACT_SCHEMA_VERSION.into(), + run_id: "run-compact-oversize".into(), + snapshot_id: "snapshot-compact-oversize".into(), + knowledge_cutoff: "2026-08-01T00:00:00Z".into(), + pair_count, + observed_count: 1, + inferred_count: 1, + unobserved_count, + refused_as_negative_count: unobserved_count, + inference_status: "unobserved_is_not_negative_observed_inferred_are_presence".into(), + }; + let raw_payload = serde_json::to_string(&artifact).expect("raw json"); + assert_eq!( + artifact.to_json(), + Err(AnalysisEngineError::InvalidRelationAbsenceArtifact) + ); + assert_eq!( + RelationAbsenceArtifact::from_json(&raw_payload), + Err(AnalysisEngineError::InvalidRelationAbsenceArtifact) + ); +} + #[test] fn future_available_pairs_are_excluded() { let request = request(); From 6b280c31687d83f35f7d5b2910266a706514ba39 Mon Sep 17 00:00:00 2001 From: Seongho Bae Date: Wed, 2 Sep 2026 07:05:15 +0900 Subject: [PATCH 3/7] fix(analysis): bound serialized relation census --- crates/analysis_engine/src/relation_absence_artifact.rs | 1 + 1 file changed, 1 insertion(+) diff --git a/crates/analysis_engine/src/relation_absence_artifact.rs b/crates/analysis_engine/src/relation_absence_artifact.rs index b434ab939..9dcfc99d5 100644 --- a/crates/analysis_engine/src/relation_absence_artifact.rs +++ b/crates/analysis_engine/src/relation_absence_artifact.rs @@ -150,6 +150,7 @@ impl RelationAbsenceArtifact { || !valid_identifier(&self.snapshot_id) || KnowledgeCutoff::parse_rfc3339(&self.knowledge_cutoff).is_err() || self.pair_count < 3 + || self.pair_count > MAX_EVIDENCE_UNITS as u64 || self.observed_count == 0 || self.inferred_count == 0 || self.unobserved_count == 0 From aaeab0a8825ee78bb76f09329fe24cb8bf7a8341 Mon Sep 17 00:00:00 2001 From: Seongho Bae Date: Wed, 2 Sep 2026 16:01:40 +0900 Subject: [PATCH 4/7] test(analysis): reproduce relation-absence cutoff representation bug --- ...ation_absence_cutoff_semantics_contract.rs | 58 +++++++++++++++++++ 1 file changed, 58 insertions(+) create mode 100644 crates/analysis_engine/tests/relation_absence_cutoff_semantics_contract.rs diff --git a/crates/analysis_engine/tests/relation_absence_cutoff_semantics_contract.rs b/crates/analysis_engine/tests/relation_absence_cutoff_semantics_contract.rs new file mode 100644 index 000000000..09d472280 --- /dev/null +++ b/crates/analysis_engine/tests/relation_absence_cutoff_semantics_contract.rs @@ -0,0 +1,58 @@ +//! Regression contract for semantic knowledge-cutoff equality in relation-absence runs. + +use analysis_engine::{ + RELATION_ABSENCE_MODEL_CONTRACT_VERSION, RELATION_ABSENCE_OUTPUT_PROFILE, + RelationAbsencePair, execute_relation_absence_run, +}; +use relation_absence::ObservationStatus; +use temporal_core::{AvailableTime, KnowledgeCutoff}; +use tepp_api::{AnalysisRunAccepted, AnalysisRunRequest}; + +fn available() -> AvailableTime { + AvailableTime::parse_rfc3339("2026-07-01T00:00:00Z").expect("available time") +} + +fn pair(pair_id: &str, status: ObservationStatus) -> RelationAbsencePair { + RelationAbsencePair::new(pair_id, status, available()).expect("pair") +} + +#[test] +fn equivalent_rfc3339_cutoff_offsets_are_admitted() { + let request = AnalysisRunRequest { + contract_version: 1, + idempotency_key: "relation-absence-equivalent-cutoff".into(), + tenant_workspace_id: "tenant-workspace".into(), + snapshot_id: "snapshot-relation-absence".into(), + knowledge_cutoff: "2026-08-01T09:00:00+09:00".into(), + model_contract_version: RELATION_ABSENCE_MODEL_CONTRACT_VERSION.into(), + output_profile: RELATION_ABSENCE_OUTPUT_PROFILE.into(), + }; + let accepted = AnalysisRunAccepted::new( + "run-relation-absence", + "accepted", + &request.idempotency_key, + ) + .expect("accepted"); + let execution_cutoff = + KnowledgeCutoff::parse_rfc3339("2026-08-01T00:00:00Z").expect("execution cutoff"); + let pairs = vec![ + pair("observed-a", ObservationStatus::Observed), + pair("inferred-b", ObservationStatus::Inferred), + pair("unobserved-c", ObservationStatus::Unobserved), + ]; + + let execution = execute_relation_absence_run( + &request, + &accepted, + "snapshot-relation-absence", + execution_cutoff, + &pairs, + "2026-08-02T00:00:00Z", + ) + .expect("equivalent RFC 3339 spellings denote the same cutoff instant"); + + assert_eq!( + execution.artifact.knowledge_cutoff, + execution_cutoff.to_rfc3339() + ); +} From 38873c326bb86ca14c3e791cabe3993c1ccbc434 Mon Sep 17 00:00:00 2001 From: Seongho Bae Date: Wed, 2 Sep 2026 16:02:18 +0900 Subject: [PATCH 5/7] fix(analysis): compare relation-absence cutoffs by instant --- crates/analysis_engine/src/relation_absence_artifact.rs | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/crates/analysis_engine/src/relation_absence_artifact.rs b/crates/analysis_engine/src/relation_absence_artifact.rs index 9dcfc99d5..b7e06f6d9 100644 --- a/crates/analysis_engine/src/relation_absence_artifact.rs +++ b/crates/analysis_engine/src/relation_absence_artifact.rs @@ -200,7 +200,9 @@ pub fn execute_relation_absence_run( if request.snapshot_id != snapshot_id { return Err(AnalysisEngineError::SnapshotMismatch); } - if request.knowledge_cutoff != knowledge_cutoff.to_rfc3339() + let request_cutoff = KnowledgeCutoff::parse_rfc3339(&request.knowledge_cutoff) + .map_err(|_| AnalysisEngineError::InvalidEvidence)?; + if request_cutoff.instant() != knowledge_cutoff.instant() || request.model_contract_version != RELATION_ABSENCE_MODEL_CONTRACT_VERSION || request.output_profile != RELATION_ABSENCE_OUTPUT_PROFILE { From ce161e089938bcc4caceaf261478d1ea65178be6 Mon Sep 17 00:00:00 2001 From: Seongho Bae Date: Wed, 2 Sep 2026 17:04:11 +0900 Subject: [PATCH 6/7] test(analysis): require validated relation-absence terminal summary --- .../tests/relation_absence_execution_contract.rs | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/crates/analysis_engine/tests/relation_absence_execution_contract.rs b/crates/analysis_engine/tests/relation_absence_execution_contract.rs index c728636ab..66df71e2b 100644 --- a/crates/analysis_engine/tests/relation_absence_execution_contract.rs +++ b/crates/analysis_engine/tests/relation_absence_execution_contract.rs @@ -97,6 +97,15 @@ fn mixed_statuses_emit_digest_bound_refusals_without_recovery_metric() { execution.terminal_result.run_state, AnalysisRunTerminalState::Succeeded ); + assert_eq!( + execution + .terminal_result + .summary + .as_ref() + .expect("summary") + .validation_status, + "validated" + ); assert_eq!( execution.terminal_result.result_sha256.as_deref(), Some(execution.artifact.sha256().expect("digest").as_str()) From dfab4eab5ff733731e565a9348072b8dab2e4912 Mon Sep 17 00:00:00 2001 From: Seongho Bae Date: Wed, 2 Sep 2026 17:04:54 +0900 Subject: [PATCH 7/7] fix(analysis): separate relation inference from validation status --- crates/analysis_engine/src/relation_absence_artifact.rs | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/crates/analysis_engine/src/relation_absence_artifact.rs b/crates/analysis_engine/src/relation_absence_artifact.rs index b7e06f6d9..6761ebac8 100644 --- a/crates/analysis_engine/src/relation_absence_artifact.rs +++ b/crates/analysis_engine/src/relation_absence_artifact.rs @@ -271,12 +271,7 @@ pub fn execute_relation_absence_run( inference_status: RELATION_ABSENCE_INFERENCE_STATUS.into(), }; let digest = artifact.sha256()?; - let summary = AnalysisResultSummary::new( - "relation_absence", - pair_count, - 4, - RELATION_ABSENCE_INFERENCE_STATUS, - )?; + let summary = AnalysisResultSummary::new("relation_absence", pair_count, 4, "validated")?; let terminal_result = AnalysisRunTerminalResult::succeeded( request, accepted,