tatara_process/crd.rs
1//! The `Process` CRD — `tatara.pleme.io/v1alpha1`.
2
3use chrono::{DateTime, Utc};
4use kube::CustomResource;
5use schemars::JsonSchema;
6use serde::{Deserialize, Serialize};
7use tatara_lisp::DeriveTataraDomain;
8
9use crate::attestation::ProcessAttestation;
10use crate::boundary::Boundary;
11use crate::classification::Classification;
12use crate::compliance::ComplianceSpec;
13use crate::encapsulates::EncapsulatesSpec;
14use crate::identity::Identity;
15use crate::intent::Intent;
16use crate::lifetime::{EphemeralLifetime, Lifetime};
17use crate::phase::ProcessPhase;
18use crate::routing::RoutingSpec;
19use crate::signal::ProcessSignal;
20use crate::spec::{DependsOn, IdentitySpec, SignalPolicy};
21use crate::status::{BoundaryStatus, ComplianceStatus, FluxResourceRef, ProcessCondition};
22
23/// Process — one element of the tatara convergence lattice, reconciled as a Unix process.
24///
25/// ```yaml
26/// apiVersion: tatara.pleme.io/v1alpha1
27/// kind: Process
28/// metadata:
29/// name: observability-stack
30/// namespace: seph
31/// spec:
32/// identity:
33/// parent: seph.1
34/// classification:
35/// pointType: Gate
36/// substrate: Observability
37/// intent:
38/// nix:
39/// flakeRef: github:pleme-io/k8s?dir=shared/infrastructure
40/// attribute: observability
41/// compliance:
42/// baseline: fedramp-moderate
43/// bindings:
44/// - framework: nist-800-53
45/// controlId: SC-7
46/// phase: AtBoundary
47/// dependsOn:
48/// - name: secret-injection
49/// ```
50#[derive(CustomResource, DeriveTataraDomain, Clone, Debug, Deserialize, Serialize, JsonSchema)]
51#[kube(
52 group = "tatara.pleme.io",
53 version = "v1alpha1",
54 kind = "Process",
55 plural = "processes",
56 shortname = "proc",
57 namespaced,
58 status = "ProcessStatus",
59 printcolumn = r#"{"name":"PID","type":"string","jsonPath":".status.pid"}"#,
60 printcolumn = r#"{"name":"Phase","type":"string","jsonPath":".status.phase"}"#,
61 printcolumn = r#"{"name":"Type","type":"string","jsonPath":".spec.classification.pointType"}"#,
62 printcolumn = r#"{"name":"Substrate","type":"string","jsonPath":".spec.classification.substrate"}"#,
63 printcolumn = r#"{"name":"Gen","type":"integer","jsonPath":".status.attestation.generation"}"#,
64 printcolumn = r#"{"name":"Age","type":"date","jsonPath":".metadata.creationTimestamp"}"#
65)]
66#[serde(rename_all = "camelCase")]
67#[tatara(keyword = "defpoint")]
68pub struct ProcessSpec {
69 /// Identity (parent, name override).
70 #[serde(default)]
71 pub identity: IdentitySpec,
72
73 /// Lattice position (6 dimensions).
74 pub classification: Classification,
75
76 /// Where rendered artifacts come from. Exactly one variant must be set.
77 pub intent: Intent,
78
79 /// Boundary predicates (preconditions / postconditions).
80 #[serde(default)]
81 pub boundary: Boundary,
82
83 /// Compliance bindings + baseline.
84 #[serde(default)]
85 pub compliance: ComplianceSpec,
86
87 /// Lattice dependencies — must reach phase before we proceed.
88 #[serde(default)]
89 pub depends_on: Vec<DependsOn>,
90
91 /// Signal policy (grace, SIGHUP strategy, start-suspended).
92 #[serde(default)]
93 pub signals: SignalPolicy,
94
95 /// Lifetime — `Permanent` (default, re-converging) or `Ephemeral`
96 /// (auto-SIGTERM per `teardown_policy` + TTL clock).
97 #[serde(default, skip_serializing_if = "Lifetime::is_default")]
98 pub lifetime: Lifetime,
99
100 /// External edges — DNS + Ingress. When `None`, the Process is
101 /// internal-only (matches today's default). See
102 /// [`crate::routing`] for the full shape.
103 #[serde(default, skip_serializing_if = "Option::is_none")]
104 pub routing: Option<RoutingSpec>,
105
106 /// Pre-existing in-cluster state this Process wraps. When `None`,
107 /// the Process is greenfield (Manage mode implicitly applied to
108 /// nothing pre-existing). See [`crate::encapsulates`] for the
109 /// three modes (Manage / Adopt / Observe).
110 #[serde(default, skip_serializing_if = "Option::is_none")]
111 pub encapsulates: Option<EncapsulatesSpec>,
112
113 /// Soft-suspend marker — reconciler treats as SIGSTOP.
114 /// Same effect as delivering SIGSTOP, but persistent across restarts.
115 #[serde(default)]
116 pub suspended: bool,
117}
118
119// Coordinate primitives — the `(namespace, name)` pair every downstream
120// composer (annotation writers, claim arbiter, boundary evaluator,
121// render owner-metadata seed) pulled by hand from `Process.metadata`
122// pre-lift, each restating the same two `Option<String>`-to-`&str`
123// unwrap incantations with the same two workspace-wide fallback
124// strings sprayed inline. Post-lift the pair lives at ONE substrate
125// primitive on `Process` — a future normalization (case-fold,
126// unicode-safe collation, cross-cluster prefix, a rename of either
127// fallback) lands here and every downstream composer inherits the
128// upgrade mechanically. Peer to `qualified_process_ref` in
129// `tatara-reconciler::ssapply`, whose two `&str` arguments are
130// exactly the pair `Process::coordinates_or_defaults` returns.
131impl Process {
132 /// The K8s canonical default namespace — the fallback every
133 /// consumer of a `Process` whose `metadata.namespace` is `None`
134 /// substitutes. Matches the string K8s itself substitutes on
135 /// namespaced resource writes with no explicit namespace.
136 pub const DEFAULT_NAMESPACE: &'static str = "default";
137
138 /// Workspace-wide fallback for a `Process`'s `metadata.name` when
139 /// it is `None` — the sentinel every annotation writer, claim
140 /// arbiter, and owner-metadata seed substitutes so downstream
141 /// grepping / label-selecting sees a stable spelling rather than
142 /// a per-callsite ad-hoc placeholder (`""`, `"<unnamed>"`, or the
143 /// empty `unwrap_or_default()` fallback). A Process authored
144 /// through the reconciler's fork path always has a name; this
145 /// constant covers the surface where an untyped `Process` value
146 /// (test fixture, dynamic API response, adopted resource pre-
147 /// name-resolution) surfaces without one.
148 pub const UNNAMED_PLACEHOLDER: &'static str = "unnamed";
149
150 /// Namespace slice with the [`Self::DEFAULT_NAMESPACE`] fallback
151 /// applied — the ONE-line collapse of the `metadata.namespace
152 /// .as_deref().unwrap_or("default")` incantation every consumer
153 /// spelled by hand pre-lift.
154 ///
155 /// Peer to [`Self::name_or_placeholder`] on the (metadata slot ×
156 /// fallback shape) axis; both compose through
157 /// [`Self::coordinates_or_defaults`] when a consumer needs the
158 /// pair together (annotation writers, claim-arbiter row builders,
159 /// render owner-metadata seed).
160 pub fn namespace_or_default(&self) -> &str {
161 self.metadata
162 .namespace
163 .as_deref()
164 .unwrap_or(Self::DEFAULT_NAMESPACE)
165 }
166
167 /// Name slice with the [`Self::UNNAMED_PLACEHOLDER`] fallback
168 /// applied — the ONE-line collapse of the `metadata.name.as_deref
169 /// ().unwrap_or("unnamed")` incantation every consumer spelled by
170 /// hand pre-lift.
171 ///
172 /// Peer to [`Self::namespace_or_default`] on the (metadata slot ×
173 /// fallback shape) axis; both compose through
174 /// [`Self::coordinates_or_defaults`] when a consumer needs the
175 /// pair together.
176 pub fn name_or_placeholder(&self) -> &str {
177 self.metadata
178 .name
179 .as_deref()
180 .unwrap_or(Self::UNNAMED_PLACEHOLDER)
181 }
182
183 /// `(namespace, name)` coordinates with the workspace-wide default
184 /// fallbacks applied — the ONE-line collapse of the paired
185 /// `metadata.namespace.as_deref().unwrap_or("default")` +
186 /// `metadata.name.as_deref().unwrap_or("unnamed")` extraction
187 /// every downstream composer restated by hand pre-lift.
188 ///
189 /// Return-tuple order matches the axis order of the substrate's
190 /// paired-composer primitive
191 /// `tatara_reconciler::ssapply::qualified_process_ref(ns, name)`:
192 /// the (namespace, name) pair this method returns feeds that
193 /// primitive positionally without an axis-swap step.
194 pub fn coordinates_or_defaults(&self) -> (&str, &str) {
195 (self.namespace_or_default(), self.name_or_placeholder())
196 }
197
198 /// `(namespace, name)` coordinates as owned `String`s, with the
199 /// namespace half fallback-defaulted to [`Self::DEFAULT_NAMESPACE`]
200 /// but the name half REQUIRED — an [`anyhow::Error`] is returned
201 /// when `metadata.name` is absent, because "unnamed" is a display
202 /// placeholder, not a valid K8s API path segment. Fed straight into
203 /// kube-rs API calls (`Api::patch`, `Api::delete`, `Api::get`) that
204 /// take owned `String` arguments; the [`Self::DEFAULT_NAMESPACE`]
205 /// fallback matches what K8s itself substitutes on namespaced
206 /// resource writes with no explicit namespace, so the surface is
207 /// safe against a `Process` whose `metadata.namespace` slot is
208 /// absent (test fixture, dynamic API response pre-defaulting) but
209 /// refuses to guess a name.
210 ///
211 /// Peer to [`Self::coordinates_or_defaults`] on the (return-form ×
212 /// name gate) axis pair:
213 /// * borrow + name-defaulted → `coordinates_or_defaults` (display,
214 /// annotation writers, ownership-tag composers — every consumer
215 /// whose downstream drops `"unnamed"` in place of a missing name
216 /// without an operator-visible failure);
217 /// * owned + name-required → this method (kube-rs API calls —
218 /// every consumer whose downstream must NOT silently substitute
219 /// a placeholder for the API call target, because the caller is
220 /// about to `patch`/`delete`/`get` at `metadata.name`).
221 ///
222 /// The error wording is pinned by
223 /// [`tests::owned_coordinates_or_err_error_message_matches_pre_lift_reconciler_wording`]
224 /// to match the exact spelling every pre-lift `tatara-reconciler`
225 /// helper produced (`"Process has no metadata.name"`) so log-line
226 /// / test greps that anchored on that wording keep matching post-
227 /// lift, and no operator-visible message drift lands as a side
228 /// effect of the substrate move.
229 pub fn owned_coordinates_or_err(&self) -> anyhow::Result<(String, String)> {
230 let ns = self
231 .metadata
232 .namespace
233 .clone()
234 .unwrap_or_else(|| Self::DEFAULT_NAMESPACE.into());
235 let name = self
236 .metadata
237 .name
238 .clone()
239 .ok_or_else(|| anyhow::anyhow!("Process has no metadata.name"))?;
240 Ok((ns, name))
241 }
242
243 /// `(namespace, name)` coordinates in the BORROW + NAME-REQUIRED
244 /// corner of the primitive family — namespace half falls back to
245 /// [`Self::DEFAULT_NAMESPACE`], but the name half is REQUIRED
246 /// (`None` on a `Process` whose `metadata.name` is absent, so the
247 /// caller stops with an `else { continue; }` / `else { return
248 /// …; }` guard rather than proceeding with the empty-string
249 /// sentinel every pre-lift consumer had to spell inline).
250 ///
251 /// Peer to [`Self::coordinates_or_defaults`] +
252 /// [`Self::owned_coordinates_or_err`] on the (return-form ×
253 /// name-gate) axis pair — closes the corner the family previously
254 /// left open:
255 ///
256 /// * borrow + name-defaulted → [`Self::coordinates_or_defaults`]
257 /// (annotation writers, render owner-metadata seed — consumers
258 /// whose downstream tolerates the `"unnamed"` display placeholder
259 /// without operator-visible failure);
260 /// * borrow + name-required → **this method** (claim-arbiter
261 /// probes, child-Process delete-fan-out — consumers that need a
262 /// real API-path leaf and cleanly SKIP the row when the name is
263 /// absent rather than issuing a K8s call with an empty-string
264 /// name argument);
265 /// * owned + name-required → [`Self::owned_coordinates_or_err`]
266 /// (kube-rs API-path calls — consumers whose downstream requires
267 /// owned `String` arguments and rejects the missing-name corner
268 /// with a load-bearing error message).
269 ///
270 /// The primitive family's `None`-on-missing-name semantics
271 /// intentionally differs from [`Self::owned_coordinates_or_err`]'s
272 /// error-on-missing-name semantics: the caller sites for this form
273 /// (child-Process fan-out, claim-arbiter row probes) are non-fatal
274 /// SKIPS rather than reportable failures — an `Option::None` at
275 /// the primitive lets the caller thread that "skip" through a
276 /// let-else without stringifying / logging an anyhow chain per
277 /// missing-name occurrence.
278 ///
279 /// The namespace fallback matches [`Self::coordinates_or_defaults`]
280 /// (via [`Self::namespace_or_default`]), so a consumer that
281 /// switches between the two borrow-form primitives based on its
282 /// name-gate need never sees a different namespace-fallback string
283 /// as a side effect.
284 pub fn coordinates_or_none(&self) -> Option<(&str, &str)> {
285 let name = self.metadata.name.as_deref()?;
286 Some((self.namespace_or_default(), name))
287 }
288
289 /// Canonical `<ns>/<name>` **namespace-qualified process reference**
290 /// composed straight off the live [`Process`] — the ONE-liner
291 /// collapse of the paired
292 /// `let (ns, name) = process.coordinates_or_defaults(); let r =
293 /// qualified_process_ref(ns, name);` incantation every consumer
294 /// whose downstream keys a Process by "which cluster location owns
295 /// it" hand-authored at scattered sites across `tatara-reconciler`.
296 ///
297 /// Pre-lift the 2-step `coordinates_or_defaults() →
298 /// qualified_process_ref(ns, name)` composition was hand-authored
299 /// at THREE sites past the ★★ PRIME-DIRECTIVE ≥ 2 duplication
300 /// threshold in `tatara-reconciler`, each restating the SAME
301 /// paired projection + `<ns>/<name>` shape:
302 /// * `render::render_routing` — routing-graph `PROCESS=<ref>`
303 /// annotation seed on every emitted Ingress / DNSEndpoint,
304 /// feeding [`crate::status::FluxResourceRef`] downstream.
305 /// * `render::render_export_jobs` — export-Job `PROCESS=<ref>`
306 /// annotation seed on every emitted export `batch/v1` Job.
307 /// * `table_controller::reconcile` — claim-arbiter row-key +
308 /// `Candidate.process_ref` seed on the stable-name claim
309 /// registry (the reference lands verbatim in
310 /// [`crate::table::ClaimRecord.holder`], where every downstream
311 /// claim query greps it).
312 ///
313 /// All THREE sites walked the SAME 2-step chain — pull the
314 /// `(ns, name)` pair through [`Self::coordinates_or_defaults`],
315 /// then feed the pair positionally into
316 /// [`crate::qualified_process_ref`]. Post-lift each caller reads
317 /// `process.qualified_ref()` — the paired projection + shape
318 /// composer now sit at ONE substrate owner, so a rename of either
319 /// workspace-wide fallback (`"default"` / `"unnamed"`), a swap of
320 /// the `<ns>/<name>` separator, a normalization pass inserted
321 /// between the paired projection and the shape composer, or a
322 /// future `<ns>/<name>@<gen>` / `<cluster>/<ns>/<name>` cross-
323 /// cluster extension lands here exactly once and every consumer
324 /// (annotation seed, claim-row key, holder-slot writer, export-
325 /// Job seed, `Candidate` composer) inherits the upgrade
326 /// mechanically.
327 ///
328 /// Peer to [`Self::coordinates_or_defaults`] on the (return-form ×
329 /// composition-depth) axis pair:
330 /// * pair + defaulted → [`Self::coordinates_or_defaults`]
331 /// (consumers that thread each half into a separate positional
332 /// slot — `Api::namespaced(client, &ns) + Api::patch(&name, …)`,
333 /// `one_export_job(ns, name, …)`, `EdgeContext { process_name,
334 /// process_namespace, … }`);
335 /// * shape + defaulted → **this method** (consumers that key on
336 /// the composed `<ns>/<name>` reference directly — the
337 /// `PROCESS=<ref>` annotation seed, the `ClaimRecord.holder`
338 /// slot, the label-selector composer).
339 ///
340 /// The namespace-fallback discipline matches
341 /// [`Self::coordinates_or_defaults`] (via
342 /// [`Self::namespace_or_default`]) and the name-fallback discipline
343 /// matches [`Self::name_or_placeholder`], so a consumer that
344 /// switches between the pair-returning primitive and this shape-
345 /// composing primitive never sees a different fallback string as
346 /// a side effect. The composed reference is byte-identical to the
347 /// pre-lift hand-authored `format!("{ns}/{name}")` with `ns` /
348 /// `name` supplied by the pair-returning primitive, so downstream
349 /// greps keyed on the reference shape (`PROCESS=<ref>` on emitted
350 /// resources, `holder = <ref>` on claim-registry queries) match
351 /// bytewise post-lift.
352 ///
353 /// Theory anchor: THEORY.md §VI.1 (generation over composition —
354 /// the 2-step paired-projection + shape-composer chain recurred at
355 /// three hand-authored sites past the ★★ PRIME-DIRECTIVE ≥ 2
356 /// duplication trigger, and is lifted onto ONE workspace-wide
357 /// owner here). THEORY.md §II.1 invariant 5 (composition preserves
358 /// proofs — a regression that inserted a normalization step at
359 /// only two of three sites, or that drifted the fallback strings
360 /// between the paired projection and the shape composer, surfaces
361 /// at [`tests::qualified_ref_*`] rather than as silent operator-
362 /// visible skew across the three annotation / claim-key /
363 /// export-Job seed writers).
364 #[must_use]
365 pub fn qualified_ref(&self) -> String {
366 let (ns, name) = self.coordinates_or_defaults();
367 crate::qualified_process_ref(ns, name)
368 }
369
370 /// Borrowed lookup of ONE key in `metadata.annotations`, with
371 /// BOTH the missing-`annotations` corner AND the missing-key
372 /// corner collapsed to `None` — the ONE-liner collapse of the
373 /// paired `self.metadata.annotations.as_ref().and_then(|m|
374 /// m.get(key)).map(String::as_str)` incantation every consumer
375 /// restated by hand pre-lift.
376 ///
377 /// Pre-lift the 3-line `.metadata.annotations.as_ref().and_then
378 /// (|m| m.get(KEY))` chain (in three tail variants — `.cloned()`,
379 /// `.cloned().unwrap_or_default()`, `.map(String::as_str)`) was
380 /// hand-authored at THREE sites past the ★★ PRIME-DIRECTIVE ≥ 2
381 /// duplication threshold across the workspace:
382 /// * `tatara-reconciler::signals::ingest` — SIGNAL annotation
383 /// lookup (pre-lift `.cloned()` for owned parsing).
384 /// * `tatara-reconciler::phase_machine::released_from_annotation`
385 /// — RELEASED_FROM annotation lookup (pre-lift `.cloned()
386 /// .unwrap_or_default()` for `match v.as_str()`).
387 /// * `tatara-pool-reconciler::controller_pool::process_belongs_to_pool`
388 /// — POOL annotation lookup (pre-lift `.map(String::as_str)`
389 /// for `== Some(pool_name)`).
390 ///
391 /// All THREE sites walked the SAME 3-line chain — read the
392 /// annotations map, gate on presence, index by key — differing
393 /// only in the tail that shaped the result. Post-lift each
394 /// caller routes through the ONE substrate primitive here and
395 /// applies its own tail at its own site (`.map(str::to_string)`
396 /// / bare match / `==`).
397 ///
398 /// Return-form axis: `Option<&str>` mirrors the existing borrow-
399 /// first discipline of the peer metadata primitives
400 /// [`Self::namespace_or_default`], [`Self::name_or_placeholder`],
401 /// [`Self::coordinates_or_none`]. The two corners the chain
402 /// swallowed pre-lift (missing `metadata.annotations` map,
403 /// missing key inside the map) BOTH collapse to `None` so
404 /// `.is_some()` / `if let Some(_)` / `Option::map` behave
405 /// identically on a `Process` whose annotations block is `None`
406 /// and on one whose annotations block is populated but omits the
407 /// key — matching what the pre-lift `.and_then(...)` chain
408 /// produced.
409 ///
410 /// A future normalization step (a key-canonicalization pass,
411 /// a case-fold lookup, a per-key alias table for renamed
412 /// annotations across API versions, a per-namespace override
413 /// substrate) lands at ONE substrate method here and all three
414 /// downstream consumers pick up the upgrade mechanically — no
415 /// per-callsite hand-edit at `ingest` / `released_from_annotation`
416 /// / `process_belongs_to_pool`.
417 ///
418 /// Sibling to the peer metadata primitives
419 /// ([`Self::namespace_or_default`], [`Self::name_or_placeholder`],
420 /// [`Self::coordinates_or_defaults`], [`Self::coordinates_or_none`],
421 /// [`Self::owned_coordinates_or_err`]) on the metadata axis;
422 /// this method opens the borrow-form peer on the ANNOTATION
423 /// axis. Future annotation projections (a paired
424 /// `label(&str) -> Option<&str>` on `metadata.labels`, a
425 /// `has_annotation(&str) -> bool` boolean gate for presence-
426 /// only consumers) land as peer methods on this same axis.
427 ///
428 /// Theory anchor: THEORY.md §VI.1 (generation over composition
429 /// — the 3-line annotation-lookup chain recurred at three
430 /// hand-authored sites past the ★★ PRIME-DIRECTIVE ≥ 2
431 /// duplication trigger, and is lifted to ONE owner here).
432 /// THEORY.md §II.1 invariant 5 (composition preserves proofs —
433 /// the pins bind the missing-`annotations` corner + the
434 /// missing-key corner + the borrow-form `&str` lifetime + the
435 /// byte-identical parity with the pre-lift 3-line chain, so a
436 /// regression that drifted any surface at
437 /// `tests::annotation_*` rather than as silent operator-facing
438 /// skew between the SIGNAL / RELEASED_FROM / POOL annotation
439 /// readers).
440 pub fn annotation(&self, key: &str) -> Option<&str> {
441 self.metadata
442 .annotations
443 .as_ref()
444 .and_then(|m| m.get(key))
445 .map(String::as_str)
446 }
447
448 /// Borrow-form metadata-projection primitive on the `metadata.uid`
449 /// axis: returns the K8s-API-server-assigned uid as a `&str`, with
450 /// the missing-uid corner collapsed to the load-bearing empty-string
451 /// sentinel — the ONE-liner collapse of the paired
452 /// `self.metadata.uid.as_deref().unwrap_or("")` incantation every
453 /// owner-reference-emitting consumer restated by hand pre-lift.
454 ///
455 /// The empty-string fallback is NOT arbitrary — it is the exact
456 /// sentinel value the sibling substrate composer
457 /// [`crate::owner_references_json`] gates on (`if uid.is_empty()
458 /// { vec![] } else { vec![owner_reference_json(name, uid)] }`) to
459 /// stamp `metadata.ownerReferences: []` on a resource whose owning
460 /// Process pre-dates the API server's `metadata.uid` assignment
461 /// (test fixture, mid-Forking snapshot before the first `patch`
462 /// round-trip, dynamic API response pre-uid-resolution). Pre-lift
463 /// each consumer spelled the fallback as `.unwrap_or("")` at its
464 /// callsite; the two literals in two files could drift silently to
465 /// `.unwrap_or_default()`, `.unwrap_or("<unknown>")`, or an
466 /// `if let Some(u) = &process.metadata.uid` gate that returned a
467 /// different owner-refs shape for the missing-uid corner. Post-lift
468 /// the sentinel value is composed at ONE substrate site so the
469 /// empty-uid gate at `owner_references_json` and its per-callsite
470 /// producers share the SAME `""` byte-string, and a rename of the
471 /// sentinel would land at ONE substrate site rather than at every
472 /// downstream `owner_references_json(name, uid)` call.
473 ///
474 /// Peer to [`Self::namespace_or_default`] +
475 /// [`Self::name_or_placeholder`] on the metadata-slot × fallback-
476 /// shape axis: `namespace_or_default` returns the K8s-canonical
477 /// `"default"` fallback (matching what the API server substitutes
478 /// on namespaced writes with no explicit namespace);
479 /// `name_or_placeholder` returns the workspace-wide `"unnamed"`
480 /// sentinel (a display placeholder for downstream grepping /
481 /// label-selecting); this method returns the empty-string sentinel
482 /// (a load-bearing gate value that composes with
483 /// [`crate::owner_references_json`]'s `is_empty` check). The three
484 /// primitives partition the metadata-slot family by whether the
485 /// consumer wants a K8s-canonical fallback (namespace), a display
486 /// placeholder (name), or a gate sentinel (uid).
487 ///
488 /// Pre-lift the `.metadata.uid.as_deref().unwrap_or("")` chain was
489 /// hand-authored at TWO sites past the ★★ PRIME-DIRECTIVE ≥ 2
490 /// duplication threshold in `tatara-reconciler::render`, both
491 /// feeding a downstream owner-reference emitter:
492 /// * `render_routing` — the routing-edge seed that binds
493 /// `process_uid` into every routing-form `EdgeContext` (Ingress +
494 /// DNSEndpoint) built inside the fanout loop over
495 /// `RoutingSpec::hostnames`; each `Edge::render` impl then walks
496 /// its `EdgeContext` through `build_owner_refs` →
497 /// [`crate::owner_references_json`] to stamp
498 /// `metadata.ownerReferences` on the emitted resource.
499 /// * `render_export_jobs` — the ephemeral-export Job builder that
500 /// passes the same uid slice to `tatara_process::
501 /// owner_references_json(name, uid)` per rendered Job, stamping
502 /// the export-Job's `metadata.ownerReferences` back at the
503 /// owning Process.
504 ///
505 /// Both sites walked the SAME `.as_deref().unwrap_or("")` chain and
506 /// both wanted the `&str` form the primitive returns — as the
507 /// second positional argument to `owner_references_json(name, uid)`
508 /// on the ownership-tag axis. Post-lift each callsite reads
509 /// `let uid = process.uid_or_empty();` and the produced slice feeds
510 /// the same downstream composer unchanged.
511 ///
512 /// Return-form axis: `&str` mirrors the existing borrow-first
513 /// discipline of the peer metadata-fallback primitives
514 /// ([`Self::namespace_or_default`], [`Self::name_or_placeholder`]);
515 /// all three return owned-metadata borrows with a slot-specific
516 /// fallback baked in so downstream consumers compose the slice
517 /// directly into their next call without re-spelling the fallback.
518 ///
519 /// A future normalization step (a canonicalization pass that
520 /// rejects a malformed uid before the owner-ref stamp, a cross-
521 /// cluster uid rewrite for multi-tenant control planes, a stale-
522 /// uid warning annotation for a Process whose uid changed under
523 /// the reconciler mid-generation) lands at ONE substrate method
524 /// here and both downstream `owner_references_json` consumers
525 /// pick up the upgrade mechanically — no per-callsite hand-edit
526 /// at `render_routing` / `render_export_jobs`.
527 ///
528 /// Theory anchor: THEORY.md §VI.1 (generation over composition —
529 /// the `.metadata.uid.as_deref().unwrap_or("")` chain recurred at
530 /// two hand-authored sites past the ★★ PRIME-DIRECTIVE ≥ 2
531 /// duplication trigger, and is lifted to ONE owner here).
532 /// THEORY.md §II.1 invariant 5 (composition preserves proofs —
533 /// the pins bind the missing-uid corner + the empty-string
534 /// sentinel byte-shape + the borrow-form `&str` lifetime + the
535 /// byte-identical parity with the pre-lift chain + the composition
536 /// coherence with [`crate::owner_references_json`]'s `is_empty`
537 /// gate, so a regression that drifted any surface at
538 /// `tests::uid_or_empty_*` rather than as silent operator-facing
539 /// skew between the two owner-reference emitters on the SAME
540 /// Process).
541 pub fn uid_or_empty(&self) -> &str {
542 self.metadata.uid.as_deref().unwrap_or("")
543 }
544
545 /// Owned-form metadata-projection primitive on the `metadata.name`
546 /// axis: returns an owned `String` copy of the K8s object name, with
547 /// the missing-name corner collapsed to the load-bearing empty-string
548 /// sentinel — the ONE-liner collapse of the paired
549 /// `self.metadata.name.clone().unwrap_or_default()` incantation every
550 /// keying / row-builder consumer restated by hand pre-lift.
551 ///
552 /// Pre-lift the `.metadata.name.clone().unwrap_or_default()` chain
553 /// was hand-authored at TWO sites past the ★★ PRIME-DIRECTIVE ≥ 2
554 /// duplication threshold in `tatara-pool-reconciler::controller_pool`,
555 /// both stamping the `PoolMember` / `PoolMemberSnapshot`
556 /// `process_name: String` slot inside a struct-literal fanout over
557 /// pool-owned `Process`es:
558 /// * `reconcile_pool`'s pool-member seed (annotation-matched Process
559 /// list → `PoolMember { process_name, state, entered_state_at, .. }`)
560 /// — the row every operator sees on the pool's status page.
561 /// * `reconcile_pool`'s desired-count snapshot seed
562 /// (`PoolMemberSnapshot { process_name, phase, created_at }`)
563 /// — the row fed into `decide_pool_convergence`.
564 ///
565 /// Both sites walked the SAME `.clone().unwrap_or_default()` chain
566 /// and both wanted the `String` form the primitive returns — as the
567 /// owned-form `process_name: String` slot on a struct literal
568 /// composed inside a `.iter().map(...)` fanout over the same
569 /// pool-owned `Process` list. Post-lift each callsite reads
570 /// `process_name: p.owned_name_or_empty()` and the produced value
571 /// feeds the same struct-literal slot unchanged.
572 ///
573 /// The empty-string fallback is the SAME sentinel the sibling
574 /// borrow-form primitive [`Self::uid_or_empty`] returns — the two
575 /// primitives partition the owned-form × borrow-form corner of the
576 /// metadata-slot family on identical fallback semantics (empty
577 /// string means "the slot is unset"), so a consumer that switches
578 /// between them based on downstream ownership requirements never
579 /// sees a different missing-slot spelling as a side effect.
580 ///
581 /// Peer to [`Self::name_or_placeholder`] on the (return-form ×
582 /// fallback-value) axis pair — closes the corner the family
583 /// previously left open:
584 ///
585 /// * borrow + display placeholder → [`Self::name_or_placeholder`]
586 /// (log lines, annotation writers, ownership-tag composers —
587 /// consumers whose downstream drops `"unnamed"` in place of a
588 /// missing name without operator-visible failure);
589 /// * owned + empty sentinel → **this method** (row-builder /
590 /// HashMap-key / struct-literal fanout consumers whose downstream
591 /// fills a `String` field with the load-bearing `""` sentinel to
592 /// flag "no name to key by" rather than substituting a display
593 /// placeholder that would misalign a downstream lookup);
594 /// * owned + name-required → [`Self::owned_coordinates_or_err`] (kube-rs
595 /// API-path calls — consumers whose downstream must NOT silently
596 /// substitute a placeholder for the API call target).
597 ///
598 /// The primitive family's `""`-on-missing-name semantics
599 /// intentionally differs from [`Self::name_or_placeholder`]'s
600 /// `"unnamed"` semantics: the caller sites for this form (pool
601 /// membership row seeds, HashMap keys) are load-bearing keys — a
602 /// display placeholder like `"unnamed"` would silently alias every
603 /// missing-name Process to the same key, collapsing distinct rows
604 /// in the pool's member list. The empty-string sentinel keeps the
605 /// pre-lift byte-shape and lets downstream consumers gate on
606 /// `String::is_empty` if they need to filter the missing-name
607 /// corner explicitly.
608 ///
609 /// A future normalization step (a name-canonicalization pass, a
610 /// case-fold key builder, a per-pool alias table for renamed
611 /// Processes across generations) lands at ONE substrate method
612 /// here and both downstream `PoolMember` / `PoolMemberSnapshot`
613 /// seeds pick up the upgrade mechanically — no per-callsite hand-
614 /// edit at `reconcile_pool`.
615 ///
616 /// Theory anchor: THEORY.md §VI.1 (generation over composition —
617 /// the `.metadata.name.clone().unwrap_or_default()` chain recurred
618 /// at two hand-authored sites past the ★★ PRIME-DIRECTIVE ≥ 2
619 /// duplication trigger, and is lifted to ONE owner here).
620 /// THEORY.md §II.1 invariant 5 (composition preserves proofs —
621 /// the pins bind the missing-name corner + the empty-string
622 /// sentinel byte-shape + the owned-form `String` return type +
623 /// the byte-identical parity with the pre-lift chain + the
624 /// fallback-value coherence with the sibling [`Self::uid_or_empty`]
625 /// on the metadata-slot × empty-sentinel axis, so a regression
626 /// that drifted any surface at `tests::owned_name_or_empty_*`
627 /// rather than as silent operator-facing skew between the pool-
628 /// member seed and the desired-count snapshot seed on the SAME
629 /// pool).
630 pub fn owned_name_or_empty(&self) -> String {
631 self.metadata.name.clone().unwrap_or_default()
632 }
633
634 /// Borrow-form spec-projection primitive on the declared parent-PID
635 /// axis: returns the hierarchical PID path (e.g. `"seph.1"`) the
636 /// author declared at `spec.identity.parent`, with the empty-slot
637 /// corner collapsed to `None` — the ONE-liner collapse of the
638 /// paired `self.spec.identity.parent.as_deref()` incantation every
639 /// consumer restated by hand pre-lift.
640 ///
641 /// Pre-lift the `.spec.identity.parent.as_deref()` chain was hand-
642 /// authored at TWO sites past the ★★ PRIME-DIRECTIVE ≥ 2
643 /// duplication threshold in `tatara-reconciler::phase_machine`:
644 /// * `handle_forking` — the ALLOCATE-PID composer that threads the
645 /// declared parent PID into [`pid::allocate_pid`] and also into
646 /// the status patch payload (`{ "pid": new_pid, "parent":
647 /// parent_pid }`), so the reconciler-observed
648 /// [`ProcessStatus::parent`] slot mirrors the author-declared
649 /// [`IdentitySpec::parent`] at fork time. The `info!` tracing
650 /// span also reads the same slice as the `parent` field on the
651 /// PID-assigned log line.
652 /// * `handle_exiting` — the SIGTERM cascade's child-fan-out filter
653 /// that enumerates every Process cluster-wide and picks children
654 /// whose `spec.identity.parent` equals this Process's currently-
655 /// observed PID (`.filter(|c| c.spec.identity.parent.as_deref()
656 /// == Some(pid))`). The filter runs per candidate child, so the
657 /// borrow-form projection avoids allocating one `String` clone
658 /// per non-matching row in the cluster-wide list.
659 ///
660 /// Both sites walked the SAME `.as_deref()` chain and both wanted
661 /// the `Option<&str>` form the primitive returns — the
662 /// `handle_forking` site to feed positionally into
663 /// `pid::allocate_pid(&identity, parent_pid, next_seq)` and the
664 /// tracing span's `parent = ?parent_pid` debug print + the JSON
665 /// payload's `"parent": parent_pid` slot; the `handle_exiting`
666 /// filter to compare directly against `Some(pid)` where `pid:
667 /// &str` came off the borrow-form peer [`Self::observed_pid`].
668 ///
669 /// Return-form axis: `Option<&str>` mirrors the borrow-first
670 /// discipline of every peer primitive on the metadata / status
671 /// slot family ([`Self::namespace_or_default`],
672 /// [`Self::name_or_placeholder`], [`Self::observed_pid`],
673 /// [`Self::annotation`]). The empty-slot corner
674 /// (`spec.identity.parent = None`, matching `init` / PID 1 with
675 /// no parent) collapses to `None` so `.is_some()` / `if let
676 /// Some(_)` / `.map(...)` behave identically on a `Process`
677 /// authored at cluster init (PID 1, parent absent) and on any
678 /// PID-N child (parent present) — matching the pre-lift
679 /// `.as_deref()` chain's `None` byte-identically.
680 ///
681 /// Peer to [`Self::observed_pid`] on the (spec-declared ×
682 /// status-observed) axis pair: `observed_pid` returns the PID
683 /// path this Process currently OWNS (the reconciler-persisted
684 /// child position in the hierarchy), while `declared_parent_pid`
685 /// returns the PID path this Process's parent OWNS (the author-
686 /// declared upstream position). The SIGTERM cascade at
687 /// `handle_exiting` composes both: it reads its own
688 /// [`Self::observed_pid`] and matches each candidate child's
689 /// [`Self::declared_parent_pid`] against that value — the child-
690 /// fan-out relation IS the spec-declared × status-observed axis
691 /// pair collapsed to a single comparator, both sides routed
692 /// through the same borrow-form skeleton.
693 ///
694 /// A future normalization step (a per-slot canonicalization pass
695 /// that rejects malformed hierarchical PIDs, a case-fold lookup
696 /// against a table of renamed identities, a cross-cluster prefix
697 /// stripper, an alias-table lookup that maps a legacy PID to its
698 /// current spelling) lands at ONE substrate method here and both
699 /// downstream consumers pick up the upgrade mechanically — no
700 /// per-callsite hand-edit at `handle_forking` / `handle_exiting`.
701 ///
702 /// Sibling to the peer metadata-projection primitives
703 /// ([`Self::namespace_or_default`], [`Self::name_or_placeholder`],
704 /// [`Self::coordinates_or_defaults`], [`Self::coordinates_or_none`],
705 /// [`Self::owned_coordinates_or_err`], [`Self::annotation`]) on the
706 /// metadata axis; this method opens the borrow-form peer on the
707 /// declared-identity axis. Future identity projections
708 /// (`declared_name_override` on the `spec.identity.name_override`
709 /// axis, a paired `declared_identity` composite that returns both
710 /// halves) land as peer methods on this same axis.
711 ///
712 /// Theory anchor: THEORY.md §VI.1 (generation over composition —
713 /// the `.spec.identity.parent.as_deref()` chain recurred at two
714 /// hand-authored sites past the ★★ PRIME-DIRECTIVE ≥ 2 duplication
715 /// trigger, and is lifted to ONE owner here). THEORY.md §II.1
716 /// invariant 5 (composition preserves proofs — the pins bind the
717 /// empty-slot corner + the borrow-form `&str` lifetime + the
718 /// byte-identical parity with the pre-lift `.as_deref()` chain,
719 /// so a regression that drifted any surface at
720 /// `tests::declared_parent_pid_*` rather than as silent operator-
721 /// facing skew between the ALLOCATE-PID composer and the SIGTERM
722 /// cascade's child-fan-out filter on the SAME parent-child pair).
723 pub fn declared_parent_pid(&self) -> Option<&str> {
724 self.spec.identity.parent.as_deref()
725 }
726
727 /// Borrow-form spec-projection primitive on the declared
728 /// name-override axis: returns the human name the author declared
729 /// at `spec.identity.name_override` (used verbatim instead of the
730 /// content-hash-derived name in [`derive_identity`]), with the
731 /// empty-slot corner collapsed to `None` — the ONE-liner collapse
732 /// of the paired `self.spec.identity.name_override.as_deref()`
733 /// incantation every consumer restated by hand pre-lift.
734 ///
735 /// Pre-lift the `.spec.identity.name_override.as_deref()` chain
736 /// was hand-authored at TWO sites past the ★★ PRIME-DIRECTIVE ≥ 2
737 /// duplication threshold in `tatara-reconciler::phase_machine`,
738 /// both feeding the second positional argument of
739 /// [`derive_identity`]:
740 /// * `handle_pending` — the DECLARE composer that computes the
741 /// Process's [`Identity`] on entry to the state machine (before
742 /// `patch::phase_status` writes it into `status.identity`).
743 /// * `handle_forking` — the ALLOCATE-PID composer that recomputes
744 /// the same [`Identity`] on a rehydration path (status may
745 /// already carry an identity from a prior reconcile, in which
746 /// case the `.and_then(|s| s.identity.clone())` short-circuit
747 /// takes it; otherwise this `.unwrap_or_else` branch fires and
748 /// recomputes the identity fresh from the spec) so `pid::
749 /// allocate_pid` sees the SAME [`Identity`] the DECLARE phase
750 /// produced.
751 ///
752 /// Both sites walked the SAME `.as_deref()` chain and both wanted
753 /// the `Option<&str>` form the primitive returns — as the second
754 /// positional argument to `derive_identity(&self.spec, …)`, which
755 /// internally trims + filters empty strings + dispatches on
756 /// `Some(non_empty)` (verbatim name, `name_override: true`) vs
757 /// `None | Some(empty | whitespace)` (content-hash-derived name,
758 /// `name_override: false`). The primitive itself preserves the
759 /// raw slot byte-identically (the trim happens IN
760 /// `derive_identity`, not at the borrow site), so the two live
761 /// paths compose through the SAME borrow-form skeleton.
762 ///
763 /// Return-form axis: `Option<&str>` mirrors the borrow-first
764 /// discipline of every peer primitive on the metadata / status /
765 /// spec-identity slot family ([`Self::namespace_or_default`],
766 /// [`Self::name_or_placeholder`], [`Self::observed_pid`],
767 /// [`Self::annotation`], [`Self::declared_parent_pid`]). The
768 /// empty-slot corner (`spec.identity.name_override = None`,
769 /// matching a Process authored WITHOUT the human-name-override
770 /// escape hatch — the default; `derive_identity` then computes
771 /// the name from the content hash) collapses to `None` so
772 /// `.is_some()` / `if let Some(_)` / `.map(...)` behave
773 /// identically on the two Process shapes an operator can author.
774 ///
775 /// Peer to [`Self::declared_parent_pid`] on the (parent × name-
776 /// override) sub-axis of the declared-identity axis: both
777 /// primitives project a `Option<String>` slot on `IdentitySpec`
778 /// through the SAME borrow-form skeleton, so a future
779 /// `declared_identity` composite that returns both halves
780 /// together (e.g. as a `(Option<&str>, Option<&str>)` tuple or a
781 /// borrow-form `DeclaredIdentityView<'_>` newtype) lands as ONE
782 /// method that COMPOSES the two peer primitives, not as three
783 /// hand-authored `.as_deref()` chains restated at each callsite.
784 ///
785 /// A future normalization step (a per-slot canonicalization pass
786 /// that rejects malformed names, a case-fold lookup against a
787 /// table of renamed identities, an alias-table lookup that maps
788 /// a legacy name-override to its current spelling, a whitespace-
789 /// trim lift OUT of `derive_identity` INTO the primitive so both
790 /// consumers see the trimmed form) lands at ONE substrate method
791 /// here and both downstream consumers pick up the upgrade
792 /// mechanically — no per-callsite hand-edit at `handle_pending` /
793 /// `handle_forking`.
794 ///
795 /// Sibling to the peer spec-identity projection
796 /// [`Self::declared_parent_pid`] on the declared-identity axis;
797 /// this method opens the borrow-form peer on the name-override
798 /// sub-axis of the same closed set (`IdentitySpec { parent,
799 /// name_override }`). Future identity projections (a paired
800 /// `declared_identity` composite that returns both halves
801 /// together) land as peer methods on this same axis.
802 ///
803 /// Theory anchor: THEORY.md §VI.1 (generation over composition —
804 /// the `.spec.identity.name_override.as_deref()` chain recurred
805 /// at two hand-authored sites past the ★★ PRIME-DIRECTIVE ≥ 2
806 /// duplication trigger, and is lifted to ONE owner here).
807 /// THEORY.md §II.1 invariant 5 (composition preserves proofs —
808 /// the pins bind the empty-slot corner + the borrow-form `&str`
809 /// lifetime + the byte-identical parity with the pre-lift
810 /// `.as_deref()` chain + the invariance under
811 /// [`derive_identity`]'s internal trim/filter step, so a
812 /// regression that drifted any surface at
813 /// `tests::declared_name_override_*` rather than as silent
814 /// operator-facing skew between the DECLARE composer and the
815 /// ALLOCATE-PID rehydration branch on the SAME Process spec).
816 pub fn declared_name_override(&self) -> Option<&str> {
817 self.spec.identity.name_override.as_deref()
818 }
819
820 /// Borrowed slice of the FluxCD resources this Process's status
821 /// currently persists at `status.flux_resources`, with the
822 /// missing-`status` corner collapsed to an empty slice — the ONE-
823 /// line collapse of the paired `self.status.as_ref().map(|s|
824 /// s.flux_resources.clone()).unwrap_or_default()` incantation
825 /// every VERIFY-phase / ATTEST-heartbeat consumer restated by hand
826 /// pre-lift.
827 ///
828 /// Pre-lift the 5-line `.status.as_ref().map(|s| s.flux_resources
829 /// .clone()).unwrap_or_default()` chain was hand-authored at TWO
830 /// sites past the ★★ PRIME-DIRECTIVE ≥ 2 duplication threshold in
831 /// `tatara-reconciler::phase_machine`:
832 /// * `handle_running` — the VERIFY-phase per-ref readiness probe
833 /// seed that walks every ref through
834 /// [`crate::status::FluxResourceRef::fetch_coords`] via
835 /// `ssapply::fetch_flux_ref` and rebuilds an updated
836 /// `Vec<FluxResourceRef>` with `ready` + `message` + `last_check`
837 /// observed at reconcile time.
838 /// * `handle_attested` — the ATTEST-heartbeat drift detector that
839 /// short-circuits on the first non-Ready ref via
840 /// `ssapply::fetch_flux_ref` + `ssapply::ready_condition`.
841 ///
842 /// Both sites walked the SAME 5-line chain — clone the vector
843 /// eagerly for the length of the reconcile pass, then iterate it
844 /// by reference — even though neither site ever mutates the vector
845 /// nor keeps it alive past the enclosing async fn. Post-lift both
846 /// callers borrow the slice directly from `self.status`; the two
847 /// pre-lift `.clone()` calls disappear because the slice lives for
848 /// the borrow of `&self`, and both call sites' subsequent
849 /// downstream calls (`ssapply::fetch_flux_ref` / the
850 /// `patch::patch_process_status` write) do not touch the borrowed
851 /// `p: &Process`, so the borrow lifetime holds.
852 ///
853 /// Return-form axis: `&[FluxResourceRef]` mirrors the existing
854 /// borrow-first discipline every pre-lift consumer already
855 /// iterated by reference (`for r in &refs`), and the shape of
856 /// [`crate::status::FluxResourceRef::fetch_coords`]'s per-ref
857 /// borrow projection extends mechanically to the slice-level
858 /// projection here. The missing-`status` corner collapses to the
859 /// empty slice `&[]` so `.is_empty()` / `.len()` / iteration all
860 /// behave identically on a `Process` whose status is `None` and
861 /// on one whose status carries an empty `flux_resources` slot —
862 /// matching what the pre-lift `.unwrap_or_default()` produced
863 /// (an empty `Vec`).
864 ///
865 /// A future normalization step (a per-ref canonicalization pass
866 /// that skips duplicated refs, an owner-filter that returns only
867 /// refs stamped with the CURRENT `metadata.generation`, a
868 /// staleness gate that drops refs whose `last_check` predates a
869 /// reconcile deadline) lands at ONE substrate method here and
870 /// both downstream consumers pick up the upgrade mechanically —
871 /// no per-callsite hand-edit at `handle_running` /
872 /// `handle_attested`.
873 ///
874 /// Sibling to the [`Self::coordinates_or_none`] borrow-first
875 /// primitive on the metadata axis; this method opens the
876 /// analogous borrow-first primitive on the status-projection
877 /// axis. Future status projections (`observed_attestation` on
878 /// the attestation-chain axis, `observed_pid` on the PID axis,
879 /// `observed_children` on the child-fan-out axis) land as peer
880 /// methods on this same axis.
881 ///
882 /// Theory anchor: THEORY.md §VI.1 (generation over composition —
883 /// the 5-line status-projection chain recurred at two hand-
884 /// authored sites past the ★★ PRIME-DIRECTIVE ≥ 2 duplication
885 /// trigger, and is lifted to ONE owner here). THEORY.md §II.1
886 /// invariant 5 (composition preserves proofs — the pins bind the
887 /// missing-`status` corner + the slice-lifetime borrow discipline
888 /// + the byte-identical parity with the pre-lift 5-line chain, so
889 /// a regression that drifted any of the three surfaces at
890 /// `tests::observed_flux_resources_*` rather than as silent
891 /// operator-facing skew between the VERIFY-phase and ATTEST-
892 /// heartbeat consumers).
893 pub fn observed_flux_resources(&self) -> &[FluxResourceRef] {
894 self.status
895 .as_ref()
896 .map(|s| s.flux_resources.as_slice())
897 .unwrap_or(&[])
898 }
899
900 /// The borrow-form status-projection primitive on the PID axis:
901 /// returns the hierarchical PID path (e.g. `"seph.1.7"`) the
902 /// reconciler currently persists at `status.pid`, with BOTH the
903 /// missing-`status` corner AND the empty-slot corner collapsed
904 /// to `None` — the ONE-liner collapse of the paired
905 /// `self.status.as_ref().and_then(|s| s.pid.clone())` incantation
906 /// every consumer restated by hand pre-lift.
907 ///
908 /// Pre-lift the 3-line `.status.as_ref().and_then(|s| s.pid
909 /// .clone())` chain was hand-authored at TWO sites past the ★★
910 /// PRIME-DIRECTIVE ≥ 2 duplication threshold in
911 /// `tatara-reconciler::phase_machine`:
912 /// * `handle_forking` — the ALLOCATE-PID gate that short-
913 /// circuits the PID allocator when the reconciler already
914 /// assigned a PID on a prior reconcile pass (pre-lift the
915 /// chain composed with `.is_some()` and threw the clone away
916 /// without ever reading the string).
917 /// * `handle_exiting` — the SIGTERM cascade that enumerates
918 /// child Processes and terminates them by matching each
919 /// child's `spec.identity.parent` against the PID this Process
920 /// currently owns (pre-lift the chain bound an owned
921 /// `Option<String>` and threaded `pid.as_str()` into the
922 /// downstream `.as_deref() == Some(...)` comparator).
923 ///
924 /// Both sites walked the SAME 3-line chain — clone the `String`
925 /// eagerly, then either drop it (the `handle_forking` gate) or
926 /// re-borrow it through `.as_str()` (the `handle_exiting`
927 /// comparator) — even though neither site ever mutates the PID
928 /// nor keeps it alive past the enclosing async fn. Post-lift
929 /// both callers borrow the PID directly from `self.status`; the
930 /// pre-lift `.clone()` at both sites disappears because the
931 /// `&str` lives for the borrow of `&self`, and both call sites'
932 /// subsequent downstream calls (the K8s API list/patch, the
933 /// child-Process comparator) do not touch the borrowed
934 /// `p: &Process`, so the borrow lifetime holds.
935 ///
936 /// Return-form axis: `Option<&str>` mirrors the existing
937 /// borrow-first discipline every pre-lift consumer already
938 /// re-borrowed through `.as_str()` before use, and the shape of
939 /// [`Self::coordinates_or_none`]'s `Option<(&str, &str)>`
940 /// projection extends mechanically to the single-slot
941 /// projection here. The missing-`status` corner AND the
942 /// populated-status-with-`pid=None` corner BOTH collapse to
943 /// `None` so `.is_some()` / `if let Some(_)` / `.map(...)`
944 /// behave identically on a `Process` whose status is `None`
945 /// and on one whose status carries an unpopulated `pid` slot —
946 /// matching what the pre-lift `.and_then(...)` chain produced.
947 ///
948 /// A future normalization step (a per-slot canonicalization
949 /// pass that rejects malformed hierarchical PIDs, a
950 /// generation-filter that returns `None` for a PID stamped
951 /// with a stale `metadata.generation`, a staleness gate that
952 /// drops a PID whose observing `phase_since` predates a
953 /// reconcile deadline) lands at ONE substrate method here and
954 /// both downstream consumers pick up the upgrade mechanically
955 /// — no per-callsite hand-edit at `handle_forking` /
956 /// `handle_exiting`.
957 ///
958 /// Sibling to the peer [`Self::observed_flux_resources`]
959 /// borrow-first primitive on the flux-resources axis; both
960 /// methods compose the same missing-`status` fallback +
961 /// borrow-form return-shape skeleton on distinct
962 /// `ProcessStatus` slots. Future status projections
963 /// (`observed_parent` on the parent-pointer axis,
964 /// `observed_message` on the human-readable-status axis,
965 /// `observed_attestation` on the attestation-chain axis) land
966 /// as peer methods on this same axis.
967 ///
968 /// Theory anchor: THEORY.md §VI.1 (generation over
969 /// composition — the 3-line status-projection chain recurred
970 /// at two hand-authored sites past the ★★ PRIME-DIRECTIVE ≥ 2
971 /// duplication trigger, and is lifted to ONE owner here).
972 /// THEORY.md §II.1 invariant 5 (composition preserves proofs —
973 /// the pins bind the missing-`status` corner + the empty-slot
974 /// corner + the borrow-form `&str` lifetime + the
975 /// byte-identical parity with the pre-lift 3-line chain, so a
976 /// regression that drifted any surface at
977 /// `tests::observed_pid_*` rather than as silent operator-
978 /// facing skew between the ALLOCATE-PID gate and the SIGTERM
979 /// cascade on the SAME `Process`).
980 pub fn observed_pid(&self) -> Option<&str> {
981 self.status.as_ref().and_then(|s| s.pid.as_deref())
982 }
983
984 /// The borrow-form status-projection primitive on the
985 /// attestation-chain axis: returns the last
986 /// [`ProcessAttestation`] the reconciler persisted at
987 /// `status.attestation`, with the missing-`status` corner AND the
988 /// empty-slot corner BOTH collapsed to `None` — the ONE-liner
989 /// collapse of the paired `self.status.as_ref().and_then(|s|
990 /// s.attestation.as_ref())` incantation every consumer restated
991 /// by hand pre-lift.
992 ///
993 /// Pre-lift the 3-line `.status.as_ref().and_then(|s| s
994 /// .attestation.as_ref())` chain was hand-authored at TWO sites
995 /// past the ★★ PRIME-DIRECTIVE ≥ 2 duplication threshold in
996 /// `tatara-reconciler`:
997 /// * `phase_machine::advance_to_attested` — the ATTEST composer
998 /// that chains `prior.next(pillars)` when a prior attestation
999 /// is persisted and seeds with `ProcessAttestation::initial`
1000 /// otherwise.
1001 /// * `render::render_export_jobs` — the ephemeral-export Job
1002 /// builder that pulls the prior `composed_root` off the last
1003 /// persisted attestation and threads it into every rendered
1004 /// Job's `previousRoot` env var, so the export receipt chains
1005 /// into the Process's BLAKE3 attestation tree at the correct
1006 /// generation boundary.
1007 ///
1008 /// Both sites walked the SAME 3-line chain — the borrow-form
1009 /// `Option<&ProcessAttestation>` shape both consumers wanted
1010 /// already — even though neither site ever mutated the
1011 /// attestation nor kept it alive past the enclosing async fn.
1012 /// Post-lift both callers borrow the attestation directly from
1013 /// `self.status`; the pre-lift 3-line chain shrinks to a single
1014 /// method call at both sites, and both consumers' subsequent
1015 /// downstream calls (`ProcessAttestation::next` for the ATTEST
1016 /// composer, `.composed_root.clone()` for the export Job builder)
1017 /// do not touch the borrowed `p: &Process`, so the borrow
1018 /// lifetime holds.
1019 ///
1020 /// Return-form axis: `Option<&ProcessAttestation>` mirrors the
1021 /// existing borrow-first discipline every pre-lift consumer
1022 /// already re-borrowed through `.as_ref()`, and the shape of the
1023 /// peer [`Self::observed_pid`] projection extends mechanically
1024 /// to the whole-attestation-record projection here. The missing-
1025 /// `status` corner AND the populated-status-with-`attestation
1026 /// =None` corner BOTH collapse to `None` so `.is_some()` / `if
1027 /// let Some(_)` / `.map(...)` behave identically on a `Process`
1028 /// whose status is `None` and on one whose status carries an
1029 /// unpopulated `attestation` slot — matching what the pre-lift
1030 /// `.and_then(...)` chain produced.
1031 ///
1032 /// A future normalization step (a per-slot canonicalization pass
1033 /// that rejects a persisted attestation whose `composed_root`
1034 /// fails `verify`, a generation-filter that returns `None` for
1035 /// an attestation stamped with a stale `metadata.generation`, a
1036 /// staleness gate that drops an attestation whose `attested_at`
1037 /// predates a reconcile deadline) lands at ONE substrate method
1038 /// here and both downstream consumers pick up the upgrade
1039 /// mechanically — no per-callsite hand-edit at
1040 /// `advance_to_attested` / `render_export_jobs`.
1041 ///
1042 /// Sibling to the peer [`Self::observed_pid`] +
1043 /// [`Self::observed_flux_resources`] borrow-first primitives on
1044 /// the PID + flux-resources axes; all three methods compose the
1045 /// same missing-`status` fallback + borrow-form return-shape
1046 /// skeleton on distinct `ProcessStatus` slots. Future status
1047 /// projections (`observed_parent` on the parent-pointer axis,
1048 /// `observed_message` on the human-readable-status axis) land
1049 /// as peer methods on this same axis.
1050 ///
1051 /// Theory anchor: THEORY.md §VI.1 (generation over composition
1052 /// — the 3-line status-projection chain recurred at two hand-
1053 /// authored sites past the ★★ PRIME-DIRECTIVE ≥ 2 duplication
1054 /// trigger, and is lifted to ONE owner here). THEORY.md §II.1
1055 /// invariant 5 (composition preserves proofs — the pins bind
1056 /// the missing-`status` corner + the empty-slot corner + the
1057 /// borrow-form `&ProcessAttestation` lifetime + the byte-
1058 /// identical parity with the pre-lift 3-line chain, so a
1059 /// regression that drifted any surface at
1060 /// `tests::observed_attestation_*` rather than as silent
1061 /// operator-facing skew between the ATTEST composer and the
1062 /// ephemeral-export receipt chain on the SAME `Process`).
1063 pub fn observed_attestation(&self) -> Option<&ProcessAttestation> {
1064 self.status.as_ref().and_then(|s| s.attestation.as_ref())
1065 }
1066
1067 /// The borrow-form status-projection primitive on the resolved-
1068 /// identity axis: returns the [`Identity`] the reconciler
1069 /// currently persists at `status.identity` (name + content hash +
1070 /// override flag), with the missing-`status` corner AND the
1071 /// empty-slot corner BOTH collapsed to `None` — the ONE-liner
1072 /// collapse of the paired `self.status.as_ref().and_then(|s|
1073 /// s.identity.as_ref())` incantation every consumer restated by
1074 /// hand pre-lift.
1075 ///
1076 /// Pre-lift the paired `.status.as_ref().and_then(|s|
1077 /// s.identity.<clone|as_ref>())` chain was hand-authored at TWO
1078 /// sites past the ★★ PRIME-DIRECTIVE ≥ 2 duplication threshold
1079 /// in `tatara-reconciler`:
1080 /// * `phase_machine::handle_forking` — the FORK-time identity
1081 /// seed that reuses the reconciler-persisted `Identity` if
1082 /// present and falls back to a fresh `derive_identity(&spec,
1083 /// name_override)` otherwise. Pre-lift the site cloned the
1084 /// whole `Identity` off the borrow before threading it through
1085 /// `.unwrap_or_else(...)` even though the fallback path
1086 /// allocates its own owned `Identity` — the pre-lift clone
1087 /// allocated a fresh `Identity` on the happy path just so the
1088 /// `Option`'s shape matched the fallback's `Identity` return
1089 /// type.
1090 /// * `ssapply::inject_annotations` — the SSA-time annotation
1091 /// composer that stamps the content-hash annotation onto every
1092 /// owned resource. Pre-lift the site nested the identity
1093 /// borrow-form check inside a manual `if let Some(status) =
1094 /// &process.status { … }` guard alongside sibling `status.pid`
1095 /// and `status.attestation` accesses — three siblings the peer
1096 /// primitives [`Self::observed_pid`] and
1097 /// [`Self::observed_attestation`] already own, so the outer
1098 /// status guard was the last hand-authored `.status.as_ref()`
1099 /// destructure at this composer.
1100 ///
1101 /// Both sites walked the SAME 3-line chain (one via `.clone()`,
1102 /// one via `.as_ref()`) — the borrow-form
1103 /// `Option<&Identity>` shape both consumers wanted already, even
1104 /// though the FORK-time seed then had to `.clone()` off the
1105 /// borrow to compose with the owned-`Identity` fallback. Post-
1106 /// lift the seed calls `.observed_identity().cloned()` at the
1107 /// exact composition point where the owned value is required
1108 /// (the empty-borrow corner clones nothing, since
1109 /// `Option::cloned` on `None` is `None`), and the SSA-time
1110 /// consumer drops the outer status guard entirely — the
1111 /// three-sibling primitive family (pid + identity + attestation)
1112 /// now peers through `observed_pid` +
1113 /// `observed_identity` + `observed_attestation` at ONE call each
1114 /// with no shared status destructure between them.
1115 ///
1116 /// Return-form axis: `Option<&Identity>` mirrors the
1117 /// existing borrow-first discipline every pre-lift consumer
1118 /// already re-borrowed through `.as_ref()` / re-cloned through
1119 /// `.clone()`, and the shape of the peer
1120 /// [`Self::observed_attestation`] projection extends
1121 /// mechanically to the whole-`Identity`-record projection here.
1122 /// The missing-`status` corner AND the populated-status-with-
1123 /// `identity=None` corner BOTH collapse to `None` so
1124 /// `.is_some()` / `if let Some(_)` / `.map(...)` behave
1125 /// identically on a `Process` whose status is `None` and on one
1126 /// whose status carries an unpopulated `identity` slot —
1127 /// matching what the pre-lift `.and_then(...)` chain produced.
1128 ///
1129 /// A future normalization step (a per-slot canonicalization
1130 /// pass that rejects an `Identity` whose `content_hash` fails
1131 /// re-derivation against the current spec, a generation-filter
1132 /// that returns `None` for an identity stamped with a stale
1133 /// `metadata.generation`, a staleness gate that drops an
1134 /// identity whose observing `phase_since` predates a reconcile
1135 /// deadline) lands at ONE substrate method here and both
1136 /// downstream consumers pick up the upgrade mechanically — no
1137 /// per-callsite hand-edit at `handle_forking` /
1138 /// `inject_annotations`.
1139 ///
1140 /// Sibling to the peer [`Self::observed_pid`] +
1141 /// [`Self::observed_attestation`] +
1142 /// [`Self::observed_flux_resources`] borrow-first primitives on
1143 /// the PID + attestation-chain + flux-resources axes; all four
1144 /// methods compose the same missing-`status` fallback +
1145 /// borrow-form return-shape skeleton on distinct `ProcessStatus`
1146 /// slots. Future status projections (`observed_parent` on the
1147 /// parent-pointer axis, `observed_message` on the human-
1148 /// readable-status axis) land as peer methods on this same
1149 /// axis.
1150 ///
1151 /// Theory anchor: THEORY.md §VI.1 (generation over composition
1152 /// — the 3-line status-projection chain recurred at two hand-
1153 /// authored sites past the ★★ PRIME-DIRECTIVE ≥ 2 duplication
1154 /// trigger, and is lifted to ONE owner here). THEORY.md §II.1
1155 /// invariant 5 (composition preserves proofs — the pins bind
1156 /// the missing-`status` corner + the empty-slot corner + the
1157 /// borrow-form `&Identity` lifetime + the byte-identical parity
1158 /// with the pre-lift 3-line chain, so a regression that drifted
1159 /// any surface at `tests::observed_identity_*` rather than as
1160 /// silent operator-facing skew between the FORK-time identity
1161 /// seed and the SSA-time content-hash annotation stamp on the
1162 /// SAME `Process`).
1163 pub fn observed_identity(&self) -> Option<&Identity> {
1164 self.status.as_ref().and_then(|s| s.identity.as_ref())
1165 }
1166
1167 /// The copy-form status-projection primitive on the phase axis:
1168 /// returns the [`ProcessPhase`] the reconciler currently persists
1169 /// at `status.phase`, wrapped in an `Option` so the missing-
1170 /// `status` corner collapses to `None` — the ONE-liner collapse
1171 /// of the paired `self.status.as_ref().map(|s| s.phase)`
1172 /// incantation every consumer restated by hand pre-lift.
1173 ///
1174 /// Peer to the borrow-form projections
1175 /// [`Self::observed_pid`] (PID axis, `Option<&str>`),
1176 /// [`Self::observed_flux_resources`] (flux-resources axis,
1177 /// `&[FluxResourceRef]`), and [`Self::observed_attestation`]
1178 /// (attestation-chain axis, `Option<&ProcessAttestation>`); this
1179 /// method opens the copy-form peer for `ProcessPhase` — a
1180 /// `Copy` scalar with a `Default` impl (`Pending`), so the
1181 /// return is `Option<ProcessPhase>` rather than
1182 /// `Option<&ProcessPhase>` (borrow would give the caller
1183 /// nothing over the copy for a 1-byte enum) and neither the
1184 /// missing-`status` corner nor a "empty slot" corner is
1185 /// meaningful — the underlying slot is a bare `ProcessPhase`,
1186 /// not `Option<ProcessPhase>`, so the primitive returns `None`
1187 /// iff `status: None`.
1188 ///
1189 /// Pre-lift the 3-line `.status.as_ref().map(|s| s.phase)`
1190 /// chain was hand-authored at FIVE sites past the ★★
1191 /// PRIME-DIRECTIVE ≥ 2 duplication threshold in
1192 /// `tatara-reconciler`:
1193 /// * `controller::reconcile` — the top-level dispatcher's
1194 /// `current_phase` seed that feeds the deletion-preempt +
1195 /// signal-ingestion gates + the per-phase handler dispatch.
1196 /// Pre-lift `.unwrap_or(ProcessPhase::Pending)`.
1197 /// * `boundary::evaluate_process_phase` — the boundary
1198 /// evaluator's `ProcessPhase` condition (a peer-Process
1199 /// `phase`-reached postcondition). Pre-lift
1200 /// `.unwrap_or(ProcessPhase::Pending)`.
1201 /// * `boundary::check_depends_on` — the `depends_on`
1202 /// pre-condition audit that stashes the observed phase into
1203 /// the `UnmetDependency::actual: Option<ProcessPhase>` slot
1204 /// (keeps the `Option` form). Pre-lift the raw
1205 /// `.map(|s| s.phase)` shape.
1206 /// * `phase_machine::p_current_phase_str` — the released-from
1207 /// annotation composer that emits `"Attested"` for every
1208 /// non-`Failed` phase (SIGSTOP/SIGCONT release gate).
1209 /// Pre-lift `.unwrap_or(ProcessPhase::Attested)` — the ONE
1210 /// site whose default is not `Pending`; the primitive
1211 /// returns the raw `Option` so the caller's `.unwrap_or`
1212 /// default choice stays local rather than baked in.
1213 /// * `table_controller::stable_name_group_key` — the routing-
1214 /// groupby seed that pairs the phase with the PID + creation
1215 /// timestamp when partitioning Processes claiming the same
1216 /// stable name. Pre-lift `.unwrap_or(ProcessPhase::Pending)`.
1217 ///
1218 /// All FIVE sites walked the SAME 3-line `.status.as_ref()
1219 /// .map(|s| s.phase)` chain — three closed with `unwrap_or
1220 /// (ProcessPhase::Pending)` (the `Default`), one closed with
1221 /// `unwrap_or(ProcessPhase::Attested)`, one kept the raw
1222 /// `Option<ProcessPhase>` — so the ONE substrate accessor
1223 /// returns the raw `Option<ProcessPhase>` and each consumer
1224 /// keeps its `.unwrap_or(...)` default choice at its own site.
1225 ///
1226 /// A future normalization step (a generation-filter that
1227 /// returns `None` for a phase stamped with a stale
1228 /// `metadata.generation`, a staleness gate that drops a phase
1229 /// whose observing `phase_since` predates a reconcile
1230 /// deadline, a canonicalization pass that maps a phase that
1231 /// no longer belongs to the CRD's closed set to `None`) lands
1232 /// at ONE substrate method here and all five consumers pick
1233 /// up the upgrade mechanically — no per-callsite hand-edit at
1234 /// `reconcile` / `evaluate_process_phase` / `check_depends_on`
1235 /// / `p_current_phase_str` / `stable_name_group_key`.
1236 ///
1237 /// Future status projections (`observed_parent` on the
1238 /// parent-pointer axis, `observed_message` on the human-
1239 /// readable-status axis, `observed_children` on the child
1240 /// fan-out axis, `observed_exit_code` on the terminal-exit
1241 /// axis) land as peer methods on this same axis.
1242 ///
1243 /// Theory anchor: THEORY.md §VI.1 (generation over
1244 /// composition — the 3-line status-projection chain recurred
1245 /// at FIVE hand-authored sites past the ★★ PRIME-DIRECTIVE
1246 /// ≥ 2 duplication trigger, and is lifted to ONE owner here).
1247 /// THEORY.md §II.1 invariant 5 (composition preserves proofs
1248 /// — the pins bind the missing-`status` corner + the
1249 /// per-variant enum round-trip + the byte-identical parity
1250 /// with the pre-lift 3-line chain, so a regression that
1251 /// drifted any surface at `tests::observed_phase_*` rather
1252 /// than as silent operator-facing skew between the
1253 /// controller's dispatch seed and the boundary evaluator's
1254 /// depends-on audit on the SAME `Process` within one
1255 /// reconcile pass).
1256 pub fn observed_phase(&self) -> Option<ProcessPhase> {
1257 self.status.as_ref().map(|s| s.phase)
1258 }
1259
1260 /// The copy-form status-projection primitive on the phase axis
1261 /// with the `Pending` sink applied — the ONE-liner collapse of
1262 /// the paired `self.observed_phase().unwrap_or(ProcessPhase::
1263 /// Pending)` incantation every reconciler consumer restated by
1264 /// hand at the `Option`-flattening tail of the `observed_phase`
1265 /// call. Sibling to [`Self::observed_phase`] on the (return-form
1266 /// × fallback shape) axis pair — the raw-`Option` corner stays
1267 /// as `observed_phase`, this method opens the `Pending`-defaulted
1268 /// corner that four of the five hand-authored `observed_phase`
1269 /// consumers chose (the fifth chose `Attested`; it keeps the raw
1270 /// `Option` accessor because a `Pending` sink would silently drop
1271 /// its released-from-annotation branch into the wrong label).
1272 ///
1273 /// The primitive returns [`ProcessPhase::Pending`] on any missing
1274 /// `status` slot — the same sentinel [`ProcessPhase::default`]
1275 /// returns, and the same fallback all four pre-lift consumers
1276 /// wrote by hand. `ProcessPhase::Pending` is load-bearing as the
1277 /// "not yet observed" default because the top-level dispatcher's
1278 /// `Pending → Forking` transition, the boundary evaluator's
1279 /// per-Process phase-reached postcondition, the routing groupby's
1280 /// stable-name claim-arbiter row seed, and the pool controller's
1281 /// desired-count snapshot all read a freshly-forked Process (no
1282 /// `status` yet stamped by the reconciler) as being at the
1283 /// entrypoint phase of the closed lifecycle. A caller with a
1284 /// different default choice (currently only the SIGSTOP/SIGCONT
1285 /// release gate's `Attested` fallback in
1286 /// `phase_machine::p_current_phase_str`) keeps the raw
1287 /// [`Self::observed_phase`] accessor at its own site.
1288 ///
1289 /// Pre-lift the two-link `.observed_phase().unwrap_or
1290 /// (ProcessPhase::Pending)` chain was hand-authored at FOUR
1291 /// sites past the ★★ PRIME-DIRECTIVE ≥ 2 duplication threshold
1292 /// across the workspace:
1293 /// * `tatara-reconciler::controller::reconcile` — the top-level
1294 /// dispatcher's `current_phase` seed that feeds the
1295 /// deletion-preempt + signal-ingestion gates + the per-phase
1296 /// handler dispatch.
1297 /// * `tatara-reconciler::boundary::evaluate_process_phase` — the
1298 /// boundary evaluator's [`ConditionKind::ProcessPhase`]
1299 /// evaluator that compares a peer-Process's observed phase
1300 /// against the operator-declared `phase`-reached postcondition.
1301 /// * `tatara-reconciler::table_controller::stable_name_group_key`
1302 /// — the routing-groupby seed that pairs the phase with the
1303 /// PID + creation timestamp when partitioning Processes
1304 /// claiming the same stable name.
1305 /// * `tatara-pool-reconciler::controller_pool::reconcile_pool` —
1306 /// the desired-count loop's per-member snapshot seed that feeds
1307 /// `decide_pool_convergence` with each owned Process's
1308 /// `(phase, created_at)` pair.
1309 ///
1310 /// All FOUR sites walked the SAME two-link chain and all four
1311 /// closed with `ProcessPhase::Pending` as the sink; post-lift
1312 /// each callsite reads `process.observed_phase_or_pending()` and
1313 /// the produced `ProcessPhase` feeds the same downstream branch
1314 /// (dispatch on the `current_phase` value, comparison against a
1315 /// declared threshold, groupby-key composition, member-state
1316 /// snapshot construction) unchanged.
1317 ///
1318 /// Return-form axis: `ProcessPhase` matches the copy discipline
1319 /// of [`Self::observed_phase`] (a `Copy` scalar one byte wide),
1320 /// with the [`Option`] wrapper collapsed at the primitive rather
1321 /// than at every consumer. A caller that needs the missing-`status`
1322 /// corner as a distinguishable value keeps the raw
1323 /// [`Self::observed_phase`] accessor.
1324 ///
1325 /// A future normalization step (a generation-filter that
1326 /// treats a phase stamped with a stale `metadata.generation` as
1327 /// unobserved and therefore `Pending`, a staleness gate that
1328 /// drops a phase whose observing `phase_since` predates a
1329 /// reconcile deadline, a canonicalization pass that maps a phase
1330 /// that no longer belongs to the CRD's closed set to `Pending`)
1331 /// lands at ONE substrate method here — because this primitive
1332 /// composes on top of [`Self::observed_phase`], the normalization
1333 /// applies to both the raw-`Option` and the `Pending`-sinked
1334 /// return through the SAME upstream body — and all four
1335 /// downstream consumers pick up the upgrade mechanically.
1336 ///
1337 /// Peer to the sibling defaulted-fallback primitive family
1338 /// [`Self::namespace_or_default`] +
1339 /// [`Self::name_or_placeholder`] + [`Self::uid_or_empty`] on the
1340 /// (return-shape × fallback-value) axis — those three open the
1341 /// borrow-form defaulted corner for the metadata slots; this
1342 /// method opens the copy-form defaulted corner for the phase
1343 /// slot on `status`. Future defaulted-fallback status
1344 /// projections (an `observed_pid_or_empty` on the PID axis, an
1345 /// `observed_exit_code_or_zero` on the terminal-exit axis) land
1346 /// as peer methods on this same axis.
1347 ///
1348 /// Theory anchor: THEORY.md §VI.1 (generation over composition —
1349 /// the two-link `.observed_phase().unwrap_or(Pending)` chain
1350 /// recurred at four hand-authored sites past the ★★
1351 /// PRIME-DIRECTIVE ≥ 2 duplication trigger, and is lifted to ONE
1352 /// owner here). THEORY.md §II.1 invariant 5 (composition
1353 /// preserves proofs — the pins bind the missing-`status` sink to
1354 /// `Pending` + populated-status pass-through + every
1355 /// `ProcessPhase` variant round-trip + byte-identical parity
1356 /// with the pre-lift two-link chain, so a regression that
1357 /// drifted any surface at `tests::observed_phase_or_pending_*`
1358 /// rather than as silent operator-facing skew between the
1359 /// top-level dispatcher's `Pending → Forking` seed and the
1360 /// boundary evaluator's per-Process phase-reached postcondition
1361 /// on the SAME `Process` within one reconcile pass).
1362 pub fn observed_phase_or_pending(&self) -> ProcessPhase {
1363 self.observed_phase().unwrap_or(ProcessPhase::Pending)
1364 }
1365
1366 /// Copy-form metadata-projection primitive on the deletion-tombstone
1367 /// axis: returns `true` iff the K8s API server has stamped a
1368 /// `metadata.deletionTimestamp` on this Process (the moment the
1369 /// object entered the "being deleted" corner of its lifecycle,
1370 /// after which further mutating writes are refused and finalizers
1371 /// are drained before the object is actually removed) — the ONE-
1372 /// liner collapse of the paired `self.metadata.deletion_timestamp
1373 /// .is_some()` incantation every consumer restated by hand
1374 /// pre-lift.
1375 ///
1376 /// Pre-lift the `.metadata.deletion_timestamp.is_some()` chain
1377 /// was hand-authored at TWO sites past the ★★ PRIME-DIRECTIVE
1378 /// ≥ 2 duplication threshold in `tatara-reconciler`, both
1379 /// projecting the SAME tombstone-presence predicate on a
1380 /// `Process` value:
1381 /// * `controller::reconcile` — the top-level dispatcher's
1382 /// deletion-preempt gate that forces the SIGTERM cascade
1383 /// (`→ Exiting`) as soon as the API server stamps the
1384 /// tombstone, before the phase handler for the current
1385 /// [`ProcessPhase`] gets a chance to run. Composed with
1386 /// [`ProcessPhase::is_alive`] so the preempt only fires on a
1387 /// Process still in an alive phase — a Process already in
1388 /// `Zombie` / `Reaped` / `Failed` runs its normal handler.
1389 /// * `phase_machine::handle_exiting` — the SIGTERM cascade's
1390 /// child-fan-out loop that enumerates every child Process and
1391 /// skips ones the API server has already tombstoned (so the
1392 /// reconciler does not re-issue a `DELETE` against a child
1393 /// whose deletion the API server is already draining through
1394 /// its own finalizer). The skip composes with
1395 /// [`Self::coordinates_or_none`]'s name-required probe so a
1396 /// child missing either its tombstone-absent gate or its
1397 /// `metadata.name` slot is a clean `continue` rather than an
1398 /// attempted `child_api.delete("")` no-op.
1399 ///
1400 /// Both sites walked the SAME `.metadata.deletion_timestamp
1401 /// .is_some()` chain and both wanted the `bool` form the
1402 /// primitive returns — the `controller::reconcile` site to gate
1403 /// the SIGTERM preempt with `&& current_phase.is_alive()` and
1404 /// the `handle_exiting` site to gate the DELETE-skip with a
1405 /// bare `if child.is_being_deleted() { continue; }`. Post-lift
1406 /// each callsite reads `process.is_being_deleted()` and the
1407 /// produced `bool` feeds the same downstream gate unchanged.
1408 ///
1409 /// Return-form axis: `bool` matches the copy-form discipline of
1410 /// [`Self::observed_phase`] (an `Option<Copy>` scalar) — the
1411 /// underlying slot is a wire-format `Option<Time>` that carries
1412 /// only presence information at this axis (the RFC-3339 timestamp
1413 /// payload itself is not what the two consumers read; both only
1414 /// probe presence to detect the tombstone-stamped state).
1415 /// Returning the raw `Option<&Time>` would push the `.is_some()`
1416 /// probe back to every callsite, restating the pre-lift chain
1417 /// one link shorter without collapsing the primitive.
1418 ///
1419 /// Peer to the metadata-fallback primitives
1420 /// [`Self::namespace_or_default`], [`Self::name_or_placeholder`],
1421 /// [`Self::uid_or_empty`], [`Self::coordinates_or_defaults`],
1422 /// [`Self::coordinates_or_none`], [`Self::owned_coordinates_or_err`],
1423 /// [`Self::annotation`] on the metadata axis; this method opens
1424 /// the copy-form peer for the presence-probe corner. Future
1425 /// metadata-presence projections (an `is_being_finalized`
1426 /// projection on `metadata.finalizers.is_empty()`'s negation,
1427 /// a `has_owner` projection on `metadata.owner_references.is_empty()`'s
1428 /// negation) land as peer methods on this same axis.
1429 ///
1430 /// A future normalization step (a per-tombstone staleness gate
1431 /// that returns `false` for a tombstone older than the reconciler's
1432 /// grace-period budget, a canonicalization pass that treats a
1433 /// tombstone from a paused controller as absent, a cross-cluster
1434 /// tombstone-observation clock skew guard) lands at ONE substrate
1435 /// method here and both downstream consumers pick up the upgrade
1436 /// mechanically — no per-callsite hand-edit at
1437 /// `controller::reconcile` / `phase_machine::handle_exiting`.
1438 ///
1439 /// Theory anchor: THEORY.md §VI.1 (generation over composition —
1440 /// the `.metadata.deletion_timestamp.is_some()` chain recurred at
1441 /// two hand-authored sites past the ★★ PRIME-DIRECTIVE ≥ 2
1442 /// duplication trigger, and is lifted to ONE owner here).
1443 /// THEORY.md §II.1 invariant 5 (composition preserves proofs —
1444 /// the pins bind the missing-tombstone corner + the present-
1445 /// tombstone corner + the copy-form `bool` return + the byte-
1446 /// identical parity with the pre-lift `.is_some()` chain, so a
1447 /// regression that drifted any surface at
1448 /// `tests::is_being_deleted_*` rather than as silent operator-
1449 /// facing skew between the top-level dispatcher's SIGTERM
1450 /// preempt and the SIGTERM cascade's child-fan-out DELETE-skip
1451 /// on the SAME `Process` within one reconcile pass).
1452 pub fn is_being_deleted(&self) -> bool {
1453 self.metadata.deletion_timestamp.is_some()
1454 }
1455
1456 /// Copy-form metadata-projection primitive on the
1457 /// `metadata.creationTimestamp` axis: returns the K8s-API-server-
1458 /// assigned creation moment as a `DateTime<Utc>`, hiding the wire-
1459 /// format `k8s_openapi::apimachinery::pkg::apis::meta::v1::Time`
1460 /// newtype behind an inherent projection — the ONE-liner collapse
1461 /// of the paired `self.metadata.creation_timestamp.as_ref().map(|t|
1462 /// t.0)` incantation every timestamp-driven consumer restated by
1463 /// hand pre-lift.
1464 ///
1465 /// Pre-lift the paired `.metadata.creation_timestamp.as_ref()` +
1466 /// `t.0` unwrap chain was hand-authored at THREE sites past the
1467 /// ★★ PRIME-DIRECTIVE ≥ 2 duplication threshold across the
1468 /// workspace, all projecting the SAME creation-moment `DateTime<Utc>`
1469 /// on a `Process`:
1470 /// * `tatara-process::lifetime_clock::evaluate` — TTL-expiry gate
1471 /// in the ephemeral-lifetime decision (`elapsed = now
1472 /// .signed_duration_since(creation.0)`), inside the non-terminal-
1473 /// phase guard that fires the `AutoTerminate::Now { TtlExpired }`
1474 /// branch. Pre-lift the site read `if let Some(creation) = process
1475 /// .metadata.creation_timestamp.as_ref() { ... creation.0 ... }`.
1476 /// * `tatara-process::lifetime_clock::requeue_with_ttl` — sleep-
1477 /// budget picker for the reconciler's next requeue, choosing the
1478 /// smaller of HEARTBEAT and TTL-remaining so the reconciler
1479 /// doesn't oversleep past a TTL boundary. Pre-lift the site read
1480 /// `let Some(creation) = process.metadata.creation_timestamp
1481 /// .as_ref() else { return default; };` + `creation.0`.
1482 /// * `tatara-reconciler::table_controller::reconcile_process_table`
1483 /// — stable-name claim-arbiter row builder, seeding each
1484 /// candidate row's `created_at` for the tie-break ordering
1485 /// (oldest wins). Pre-lift the site read `p.metadata
1486 /// .creation_timestamp.as_ref().map(|t| t.0).unwrap_or_else(Utc
1487 /// ::now)`.
1488 ///
1489 /// All THREE sites walked the SAME two-link chain — read the
1490 /// `Option<Time>` slot as a borrow, then unwrap the `Time` newtype
1491 /// to its inner `DateTime<Utc>` — differing only in the tail
1492 /// (`if-let-Some` guard, `let-else` short-circuit, `Utc::now`
1493 /// fallback). Post-lift each callsite reads
1494 /// `process.created_at()` and applies its own tail at its own site
1495 /// (`if let Some(creation) = ...`, `let Some(creation) = ... else`,
1496 /// `.unwrap_or_else(Utc::now)`).
1497 ///
1498 /// Return-form axis: `Option<DateTime<Utc>>` matches the copy-form
1499 /// discipline of the sibling status-projection primitive
1500 /// [`Self::observed_phase`] — both return `Option<T>` where `T:
1501 /// Copy` and hide the wire-format wrapper (`ProcessStatus` on the
1502 /// status side; `Time` on the metadata side). Returning the raw
1503 /// `Option<&Time>` would push the `.0` unwrap back to every
1504 /// callsite, restating the pre-lift chain one link shorter without
1505 /// collapsing the primitive; returning owned `Option<Time>` would
1506 /// force a `Time` import at every consumer for a projection every
1507 /// consumer immediately discards past `.0`.
1508 ///
1509 /// Peer to the metadata-fallback + presence-probe primitives
1510 /// [`Self::namespace_or_default`], [`Self::name_or_placeholder`],
1511 /// [`Self::uid_or_empty`], [`Self::coordinates_or_defaults`],
1512 /// [`Self::coordinates_or_none`], [`Self::owned_coordinates_or_err`],
1513 /// [`Self::annotation`], [`Self::is_being_deleted`] on the metadata
1514 /// axis; this method opens the copy-form timestamp corner. Future
1515 /// metadata-timestamp projections (a
1516 /// `deletion_at() -> Option<DateTime<Utc>>` peer on the
1517 /// tombstone-payload axis for staleness gates that need the
1518 /// timestamp value alongside the presence bit) land as peer
1519 /// methods on this same axis.
1520 ///
1521 /// A future normalization step (a per-cluster clock-skew guard
1522 /// that offsets the returned timestamp by the observing controller's
1523 /// measured skew, a canonicalization pass that maps a suspiciously-
1524 /// zero creation moment to `None`, a per-namespace override that
1525 /// substitutes a `spec.identity`-declared creation anchor for the
1526 /// metadata slot on adopted resources) lands at ONE substrate
1527 /// method here and all three downstream consumers pick up the
1528 /// upgrade mechanically — no per-callsite hand-edit at `evaluate`
1529 /// / `requeue_with_ttl` / `reconcile_process_table`.
1530 ///
1531 /// Theory anchor: THEORY.md §VI.1 (generation over composition —
1532 /// the `.metadata.creation_timestamp.as_ref().map(|t| t.0)` chain
1533 /// recurred at three hand-authored sites past the ★★
1534 /// PRIME-DIRECTIVE ≥ 2 duplication trigger, and is lifted to ONE
1535 /// owner here). THEORY.md §II.1 invariant 5 (composition preserves
1536 /// proofs — the pins bind the missing-timestamp corner + the
1537 /// present-timestamp corner + the copy-form `DateTime<Utc>` return
1538 /// + the byte-identical parity with the pre-lift `.as_ref().map(|t|
1539 /// t.0)` chain, so a regression that drifted any surface at
1540 /// `tests::created_at_*` rather than as silent operator-facing
1541 /// skew between the TTL-expiry gate, the requeue-budget picker,
1542 /// and the stable-name claim-arbiter tie-break on the SAME
1543 /// `Process` within one reconcile pass).
1544 pub fn created_at(&self) -> Option<DateTime<Utc>> {
1545 self.metadata.creation_timestamp.as_ref().map(|t| t.0)
1546 }
1547
1548 /// Compound spec-projection primitive on the `spec.lifetime` axis:
1549 /// returns `Some(&e)` iff the resolver unambiguously picks the
1550 /// `Ephemeral` slot, `None` otherwise — the ONE-liner collapse of
1551 /// the 4-step `self.spec.lifetime.resolved_ephemeral()` chain the
1552 /// two `lifetime_clock` consumers previously reached through and
1553 /// the coherence-tightening lift of the naked
1554 /// `self.spec.lifetime.ephemeral.as_ref()` raw-field access
1555 /// `tatara_reconciler::render::render_export_jobs` previously
1556 /// walked past.
1557 ///
1558 /// Pre-lift THREE consumer sites past the ★★ PRIME-DIRECTIVE ≥ 2
1559 /// duplication threshold reached the ephemeral inner through TWO
1560 /// different chains that disagreed on the ambiguous corner:
1561 /// * `tatara_process::lifetime_clock::evaluate` — ambiguity-aware:
1562 /// `process.spec.lifetime.resolved_ephemeral()` collapses BOTH-
1563 /// slots-set to `None`, matching the "no ephemeral action"
1564 /// outcome (`AutoTerminate::Skip`) the ambiguous case must yield.
1565 /// * `tatara_process::lifetime_clock::requeue_with_ttl` —
1566 /// ambiguity-aware peer of `evaluate`; both share the SAME
1567 /// `resolved_ephemeral()` gate and MUST agree on the ambiguous
1568 /// corner or the reconciler's teardown decision and requeue-
1569 /// budget picker drift apart on the SAME `Process` within one
1570 /// reconcile pass.
1571 /// * `tatara_reconciler::render::render_export_jobs` — RAW field
1572 /// access: `process.spec.lifetime.ephemeral.as_ref()` returned
1573 /// `Some(&e)` on the ambiguous corner, so an operator-authored
1574 /// `Process` with BOTH `permanent:` AND `ephemeral:` slots
1575 /// populated would emit export Jobs whose teardown-triggered
1576 /// fire semantics `lifetime_clock` refused to honor. The two
1577 /// consumers drifted at the mis-configuration corner.
1578 ///
1579 /// Post-lift ALL THREE consumers reach through ONE `Process` method
1580 /// that composes `self.spec.lifetime.resolved_ephemeral()` — the
1581 /// ambiguity-aware `variant().ok() + as_ephemeral` chain
1582 /// [`crate::lifetime::Lifetime::resolved_ephemeral`] owns — and
1583 /// the drift between the reconciler's export-render arm and the
1584 /// lifetime clock's teardown/TTL arm CLOSES at ONE substrate site.
1585 ///
1586 /// Return-form axis: `Option<&EphemeralLifetime>` matches the
1587 /// borrow-form discipline of the underlying
1588 /// [`crate::lifetime::Lifetime::resolved_ephemeral`] projection so
1589 /// the borrow carries the `'_self` lifetime through directly
1590 /// without a temporary `LifetimeVariant` binding. Peer to the
1591 /// borrow-form status-projection primitives
1592 /// [`Self::observed_attestation`], [`Self::observed_identity`] and
1593 /// the borrow-form metadata-projection primitive
1594 /// [`Self::uid_or_empty`] — all four hide a wrapping `Option`-
1595 /// carrying wire slot behind an inherent projection.
1596 ///
1597 /// A future normalization step (a canonicalization pass that maps
1598 /// a suspiciously-zero `ttl` to a per-cluster default, a per-
1599 /// namespace override that substitutes an operator-declared
1600 /// teardown policy on adopted resources, a wire-schema migration
1601 /// that renames `spec.lifetime.ephemeral` to `spec.lifetime.timed`
1602 /// with a bridging `From` shim) lands at ONE substrate method here
1603 /// and all three downstream consumers pick up the upgrade
1604 /// mechanically — no per-callsite hand-edit at `evaluate` /
1605 /// `requeue_with_ttl` / `render_export_jobs`.
1606 ///
1607 /// Theory anchor: THEORY.md §II.1 invariant 5 (composition
1608 /// preserves proofs — the pins bind the Permanent-only corner, the
1609 /// Ephemeral-only corner, the Both-set-ambiguous corner, the
1610 /// empty-default corner, and the byte-identity parity with the
1611 /// underlying `self.spec.lifetime.resolved_ephemeral()` delegate,
1612 /// so a regression that silently swapped the projection back to
1613 /// the raw `.ephemeral.as_ref()` field would surface here rather
1614 /// than as operator-facing drift between the export-render arm
1615 /// and the teardown/TTL arm on the SAME `Process`). THEORY.md
1616 /// §VI.1 (generation over composition — the ambiguity-aware
1617 /// projection recurred at three hand-authored sites past the ★★
1618 /// PRIME-DIRECTIVE ≥ 2 duplication trigger, and is lifted to ONE
1619 /// owner here).
1620 pub fn resolved_ephemeral(&self) -> Option<&EphemeralLifetime> {
1621 self.spec.lifetime.resolved_ephemeral()
1622 }
1623}
1624
1625/// Process status — every field optional until the reconciler writes it.
1626#[derive(Clone, Debug, Default, Deserialize, Serialize, JsonSchema)]
1627#[serde(rename_all = "camelCase")]
1628pub struct ProcessStatus {
1629 /// Hierarchical PID path — e.g., `"seph.1.7"`.
1630 #[serde(default, skip_serializing_if = "Option::is_none")]
1631 pub pid: Option<String>,
1632
1633 /// Parent PID path (mirror of `spec.identity.parent`, resolved at fork).
1634 #[serde(default, skip_serializing_if = "Option::is_none")]
1635 pub parent: Option<String>,
1636
1637 /// Direct children's PID paths.
1638 #[serde(default)]
1639 pub children: Vec<String>,
1640
1641 /// Resolved identity (name + content hash).
1642 #[serde(default, skip_serializing_if = "Option::is_none")]
1643 pub identity: Option<Identity>,
1644
1645 /// Current phase.
1646 #[serde(default)]
1647 pub phase: ProcessPhase,
1648
1649 /// When the process entered the current phase.
1650 #[serde(default, skip_serializing_if = "Option::is_none")]
1651 pub phase_since: Option<DateTime<Utc>>,
1652
1653 /// Three-pillar attestation (written at end of every successful cycle).
1654 #[serde(default, skip_serializing_if = "Option::is_none")]
1655 pub attestation: Option<ProcessAttestation>,
1656
1657 /// FluxCD resources currently owned by this Process.
1658 #[serde(default)]
1659 pub flux_resources: Vec<FluxResourceRef>,
1660
1661 /// Boundary verification state.
1662 #[serde(default)]
1663 pub boundary: BoundaryStatus,
1664
1665 /// Compliance summary at the latest attestation.
1666 #[serde(default)]
1667 pub compliance: ComplianceStatus,
1668
1669 /// Pending signals (delivered, not yet handled).
1670 #[serde(default)]
1671 pub signal_queue: Vec<ProcessSignal>,
1672
1673 /// Standard K8s Conditions.
1674 #[serde(default)]
1675 pub conditions: Vec<ProcessCondition>,
1676
1677 /// Human-readable last status message.
1678 #[serde(default, skip_serializing_if = "Option::is_none")]
1679 pub message: Option<String>,
1680
1681 /// Exit code (only set on Failed / Reaped).
1682 #[serde(default, skip_serializing_if = "Option::is_none")]
1683 pub exit_code: Option<i32>,
1684}
1685
1686#[cfg(test)]
1687mod tests {
1688 use super::*;
1689 use crate::classification::{ConvergencePointType, SubstrateType};
1690 use crate::intent::NixIntent;
1691
1692 #[test]
1693 fn minimal_spec_serializes() {
1694 let spec = ProcessSpec {
1695 identity: IdentitySpec::default(),
1696 classification: Classification {
1697 point_type: ConvergencePointType::Gate,
1698 substrate: SubstrateType::Observability,
1699 horizon: Default::default(),
1700 calm: Default::default(),
1701 data_classification: Default::default(),
1702 },
1703 intent: Intent {
1704 nix: Some(NixIntent {
1705 flake_ref: "github:pleme-io/k8s".into(),
1706 attribute: "obs".into(),
1707 system: None,
1708 attic_cache: None,
1709 extra_args: vec![],
1710 delegate_to_nix_build: false,
1711 }),
1712 ..Intent::default()
1713 },
1714 boundary: Default::default(),
1715 compliance: Default::default(),
1716 depends_on: vec![],
1717 signals: Default::default(),
1718 lifetime: Default::default(),
1719 routing: None,
1720 encapsulates: None,
1721 suspended: false,
1722 };
1723 let yaml = serde_yaml::to_string(&spec).unwrap();
1724 assert!(yaml.contains("pointType: Gate"));
1725 assert!(yaml.contains("substrate: Observability"));
1726 assert!(yaml.contains("flakeRef: github:pleme-io/k8s"));
1727 }
1728
1729 // ─── Process::coordinates_or_defaults substrate pins ────────────────
1730 //
1731 // Pins the (namespace, name) coordinate-primitive family on the
1732 // (metadata slot × fallback shape) axis. Fail-before-pass-after
1733 // granularity: a regression that flipped either fallback string,
1734 // swapped the return-tuple axis order, or dropped the
1735 // `Option::as_deref` unwrap surfaces here rather than as silent
1736 // drift at every downstream annotation writer / claim-arbiter row
1737 // builder / render owner-metadata seed.
1738
1739 fn empty_spec() -> ProcessSpec {
1740 ProcessSpec {
1741 identity: IdentitySpec::default(),
1742 classification: Classification {
1743 point_type: ConvergencePointType::Gate,
1744 substrate: SubstrateType::Compute,
1745 horizon: Default::default(),
1746 calm: Default::default(),
1747 data_classification: Default::default(),
1748 },
1749 intent: Intent::default(),
1750 boundary: Default::default(),
1751 compliance: Default::default(),
1752 depends_on: vec![],
1753 signals: Default::default(),
1754 lifetime: Default::default(),
1755 routing: None,
1756 encapsulates: None,
1757 suspended: false,
1758 }
1759 }
1760
1761 #[test]
1762 fn default_namespace_constant_is_k8s_canonical_default() {
1763 // Pins the load-bearing convention that this primitive's
1764 // namespace fallback matches K8s's own implicit-namespace
1765 // spelling. A regression that renamed this to "kube-system"
1766 // or any other K8s-reserved name would silently misroute
1767 // every downstream namespaced-Api call on a Process without
1768 // a metadata.namespace.
1769 assert_eq!(Process::DEFAULT_NAMESPACE, "default");
1770 }
1771
1772 #[test]
1773 fn unnamed_placeholder_constant_matches_prior_annotation_writer_fallback() {
1774 // Pins the load-bearing convention that this primitive's name
1775 // fallback matches the exact spelling every annotation writer
1776 // (tatara-reconciler::ssapply::inject_annotations,
1777 // tatara-reconciler::render::render, and
1778 // tatara-reconciler::table_controller's claim-row builder)
1779 // was hand-authoring pre-lift ("unnamed", NOT "<unnamed>" or
1780 // ""). A regression that renamed this would break the
1781 // annotation-writer / claim-arbiter grep contract silently.
1782 assert_eq!(Process::UNNAMED_PLACEHOLDER, "unnamed");
1783 }
1784
1785 #[test]
1786 fn namespace_or_default_falls_back_when_metadata_namespace_is_none() {
1787 let mut p = Process::new("some-proc", empty_spec());
1788 p.metadata.namespace = None;
1789 assert_eq!(p.namespace_or_default(), Process::DEFAULT_NAMESPACE);
1790 }
1791
1792 #[test]
1793 fn namespace_or_default_returns_metadata_slice_when_some() {
1794 let mut p = Process::new("some-proc", empty_spec());
1795 p.metadata.namespace = Some("prod-app".into());
1796 assert_eq!(p.namespace_or_default(), "prod-app");
1797 }
1798
1799 #[test]
1800 fn name_or_placeholder_falls_back_when_metadata_name_is_none() {
1801 let mut p = Process::new("real-name", empty_spec());
1802 p.metadata.name = None;
1803 assert_eq!(p.name_or_placeholder(), Process::UNNAMED_PLACEHOLDER);
1804 }
1805
1806 #[test]
1807 fn name_or_placeholder_returns_metadata_slice_when_some() {
1808 let p = Process::new("api-gateway", empty_spec());
1809 assert_eq!(p.name_or_placeholder(), "api-gateway");
1810 }
1811
1812 #[test]
1813 fn coordinates_or_defaults_composes_both_halves() {
1814 // Both slots present — returns metadata slices in
1815 // (namespace, name) axis order.
1816 let mut p = Process::new("api", empty_spec());
1817 p.metadata.namespace = Some("staging".into());
1818 assert_eq!(p.coordinates_or_defaults(), ("staging", "api"));
1819 }
1820
1821 #[test]
1822 fn coordinates_or_defaults_falls_back_on_both_slots() {
1823 // Both slots None — returns (DEFAULT_NAMESPACE,
1824 // UNNAMED_PLACEHOLDER) in axis order.
1825 let mut p = Process::new("scratch", empty_spec());
1826 p.metadata.name = None;
1827 p.metadata.namespace = None;
1828 assert_eq!(
1829 p.coordinates_or_defaults(),
1830 (Process::DEFAULT_NAMESPACE, Process::UNNAMED_PLACEHOLDER)
1831 );
1832 }
1833
1834 #[test]
1835 fn coordinates_or_defaults_mixes_slotted_and_fallback_halves() {
1836 // Namespace set, name missing — the (namespace, name) tuple
1837 // pins each half independently. A regression that returned
1838 // BOTH fallbacks when EITHER metadata slot was None would
1839 // surface here rather than at every downstream reader.
1840 let mut p = Process::new("kept-name", empty_spec());
1841 p.metadata.namespace = Some("prod".into());
1842 assert_eq!(p.coordinates_or_defaults(), ("prod", "kept-name"));
1843
1844 // Name set, namespace missing — the peer corner.
1845 let mut q = Process::new("api", empty_spec());
1846 q.metadata.namespace = None;
1847 assert_eq!(
1848 q.coordinates_or_defaults(),
1849 (Process::DEFAULT_NAMESPACE, "api")
1850 );
1851 }
1852
1853 // ─── Process::qualified_ref substrate pins ─────────────────────────
1854 //
1855 // Pins the paired-projection + shape-composer chain
1856 // `coordinates_or_defaults() → qualified_process_ref(ns, name)` on
1857 // the (return-form × composition-depth) axis pair. Fail-before-
1858 // pass-after granularity: a regression that swapped the `<ns>/<name>`
1859 // axis order, dropped either half, drifted the fallback strings
1860 // between the paired-projection primitive and the shape composer, or
1861 // inserted a normalization step at only the composed site and not
1862 // the pair-returning primitive (or vice versa) surfaces here rather
1863 // than as silent operator-visible skew across the three pre-lift
1864 // `tatara-reconciler` sites (`render::render_routing`,
1865 // `render::render_export_jobs`, `table_controller::reconcile`)
1866 // whose downstream greps the reference shape verbatim (the
1867 // `PROCESS=<ref>` annotation seed on every emitted Ingress /
1868 // DNSEndpoint / export Job, the `ClaimRecord.holder` slot on the
1869 // stable-name claim registry).
1870
1871 #[test]
1872 fn qualified_ref_composes_ns_and_name_with_slash_when_both_slots_present() {
1873 // Happy path — both metadata slots populated. The composed
1874 // reference is EXACTLY `<ns>/<name>`, in that order, joined by
1875 // a single `/`. A regression that swapped the two axes at
1876 // this primitive would silently break every downstream
1877 // `PROCESS=<ref>` annotation grep + claim-registry lookup.
1878 let mut p = Process::new("api-gateway", empty_spec());
1879 p.metadata.namespace = Some("prod-app".into());
1880 assert_eq!(p.qualified_ref(), "prod-app/api-gateway");
1881 }
1882
1883 #[test]
1884 fn qualified_ref_falls_back_to_default_namespace_when_metadata_namespace_is_none() {
1885 // Namespace-fallback pin: an absent `metadata.namespace` rides
1886 // through `namespace_or_default()` → `DEFAULT_NAMESPACE`, so
1887 // the composed reference lands as `default/<name>`. Matches
1888 // what a pre-lift `qualified_process_ref(process.
1889 // coordinates_or_defaults())` composition produced.
1890 let mut p = Process::new("api-gateway", empty_spec());
1891 p.metadata.namespace = None;
1892 assert_eq!(p.qualified_ref(), "default/api-gateway");
1893 }
1894
1895 #[test]
1896 fn qualified_ref_falls_back_to_unnamed_placeholder_when_metadata_name_is_none() {
1897 // Name-fallback pin: an absent `metadata.name` rides through
1898 // `name_or_placeholder()` → `UNNAMED_PLACEHOLDER`, so the
1899 // composed reference lands as `<ns>/unnamed`. A pre-lift
1900 // consumer whose paired projection returned the placeholder
1901 // (annotation writer, render owner-metadata seed) sees the
1902 // exact same `<ns>/unnamed` shape post-lift, so downstream
1903 // greps keyed on the pre-metadata Process's reference match
1904 // bytewise.
1905 let mut p = Process::new("ignored", empty_spec());
1906 p.metadata.namespace = Some("staging".into());
1907 p.metadata.name = None;
1908 assert_eq!(p.qualified_ref(), "staging/unnamed");
1909 }
1910
1911 #[test]
1912 fn qualified_ref_falls_back_on_both_slots_when_both_metadata_are_none() {
1913 // Both slots absent → both fallbacks land in the composed
1914 // reference. The `default/unnamed` shape is what every pre-
1915 // lift caller produced when a Process fixture (test or
1916 // dynamic API response) surfaced without populated metadata;
1917 // pinning it here holds the primitive's contract against a
1918 // regression that dropped either fallback at only the
1919 // composed site.
1920 let mut p = Process::new("ignored", empty_spec());
1921 p.metadata.namespace = None;
1922 p.metadata.name = None;
1923 assert_eq!(
1924 p.qualified_ref(),
1925 format!(
1926 "{}/{}",
1927 Process::DEFAULT_NAMESPACE,
1928 Process::UNNAMED_PLACEHOLDER
1929 )
1930 );
1931 }
1932
1933 #[test]
1934 fn qualified_ref_matches_pre_lift_paired_composition_bytewise() {
1935 // Byte-identical parity with the exact pre-lift 2-step
1936 // composition every `tatara-reconciler` site hand-authored:
1937 // `let (ns, name) = process.coordinates_or_defaults(); let r
1938 // = qualified_process_ref(ns, name);`. Sweeps every metadata-
1939 // slot combination the three pre-lift consumers plausibly
1940 // encountered — both slots populated (steady state), one
1941 // slot absent (Process mid-fork before API-server metadata
1942 // stamp), both slots absent (dynamic API response / test
1943 // fixture) — so a regression that reshaped the composition at
1944 // the substrate primitive would surface here rather than as
1945 // silent drift at the three consumer sites.
1946 let fixtures: [(Option<&str>, Option<&str>); 4] = [
1947 (Some("prod-app"), Some("api-gateway")),
1948 (None, Some("api-gateway")),
1949 (Some("staging"), None),
1950 (None, None),
1951 ];
1952 for (ns_slot, name_slot) in fixtures {
1953 let mut p = Process::new(name_slot.unwrap_or("seed"), empty_spec());
1954 p.metadata.namespace = ns_slot.map(str::to_string);
1955 p.metadata.name = name_slot.map(str::to_string);
1956 let via_primitive = p.qualified_ref();
1957 let (ns, name) = p.coordinates_or_defaults();
1958 let via_paired = crate::qualified_process_ref(ns, name);
1959 assert_eq!(
1960 via_primitive, via_paired,
1961 "qualified_ref must be byte-identical to the pre-lift \
1962 paired composition on (ns={ns_slot:?}, name={name_slot:?})"
1963 );
1964 }
1965 }
1966
1967 #[test]
1968 fn qualified_ref_composes_from_the_shared_coordinates_or_defaults_owner() {
1969 // Composition invariant: the composed reference decomposes at
1970 // the single `/` separator into EXACTLY the (ns, name) pair
1971 // `coordinates_or_defaults` returns. A regression that
1972 // introduced a per-callsite normalization at the shape
1973 // composer (URL-escape, case-fold, path-normalize) or that
1974 // pulled the pair from a different metadata source than the
1975 // paired-projection primitive would surface here rather than
1976 // at every downstream reference-shape grep.
1977 let mut p = Process::new("api-gateway", empty_spec());
1978 p.metadata.namespace = Some("prod-app".into());
1979 let composed = p.qualified_ref();
1980 let (ns, name) = p.coordinates_or_defaults();
1981 let (composed_ns, composed_name) = composed.split_once('/').unwrap();
1982 assert_eq!(composed_ns, ns);
1983 assert_eq!(composed_name, name);
1984 }
1985
1986 // ─── Process::owned_coordinates_or_err substrate pins ──────────────
1987 //
1988 // Pins the owned + name-required peer of the coordinate-primitive
1989 // family on the (return-form × name gate) axis pair. Fail-before-
1990 // pass-after granularity: a regression that flipped the namespace
1991 // fallback string, dropped the `Option::clone` unwrap, changed the
1992 // return-tuple axis order, or altered the "Process has no
1993 // metadata.name" error wording surfaces here rather than as silent
1994 // drift at every pre-lift caller (10 sites in
1995 // `tatara-reconciler::phase_machine` + 2 sites in
1996 // `tatara-reconciler::signals` pre-lift).
1997
1998 #[test]
1999 fn owned_coordinates_or_err_returns_owned_strings_when_both_slots_present() {
2000 // Happy path — both slots populated, method returns owned
2001 // Strings in (namespace, name) axis order.
2002 let mut p = Process::new("api-gateway", empty_spec());
2003 p.metadata.namespace = Some("prod-app".into());
2004 let (ns, name) = p.owned_coordinates_or_err().unwrap();
2005 assert_eq!(ns, "prod-app");
2006 assert_eq!(name, "api-gateway");
2007 // Ownership pin: type inference above binds ns/name as
2008 // owned Strings — a regression that returned &str would
2009 // fail to compile at the following .push() call. This
2010 // holds the "owned" half of the primitive's contract.
2011 let mut owned_ns = ns;
2012 owned_ns.push_str("-mutated");
2013 assert_eq!(owned_ns, "prod-app-mutated");
2014 }
2015
2016 #[test]
2017 fn owned_coordinates_or_err_falls_back_on_namespace_but_returns_owned_name() {
2018 // Namespace absent → DEFAULT_NAMESPACE. Name present → owned.
2019 let p = Process::new("api", empty_spec());
2020 // Process::new leaves metadata.namespace = None by default.
2021 let (ns, name) = p.owned_coordinates_or_err().unwrap();
2022 assert_eq!(ns, Process::DEFAULT_NAMESPACE);
2023 assert_eq!(name, "api");
2024 }
2025
2026 #[test]
2027 fn owned_coordinates_or_err_errors_when_metadata_name_absent_regardless_of_namespace() {
2028 // Name absent → Err, REGARDLESS of whether the namespace is
2029 // populated. The name gate is strictly on `metadata.name` and
2030 // does NOT fall back to `Self::UNNAMED_PLACEHOLDER` (that
2031 // fallback is on the peer `coordinates_or_defaults`, which
2032 // exists precisely for consumers that can tolerate a
2033 // display placeholder).
2034 for ns_slot in [None, Some("prod".to_string())] {
2035 let mut p = Process::new("scratch", empty_spec());
2036 p.metadata.name = None;
2037 p.metadata.namespace = ns_slot.clone();
2038 let err = p.owned_coordinates_or_err().unwrap_err();
2039 assert!(
2040 err.to_string().contains("metadata.name"),
2041 "err on missing name (ns={ns_slot:?}) should mention metadata.name; got {err}"
2042 );
2043 }
2044 }
2045
2046 #[test]
2047 fn owned_coordinates_or_err_error_message_matches_pre_lift_reconciler_wording() {
2048 // Load-bearing wording pin — every pre-lift `tatara-reconciler`
2049 // helper (`phase_machine::namespace_and_name`,
2050 // `signals::ingest`, `signals::consume_effect`) errored with
2051 // EXACTLY this wording. Post-lift the substrate owner produces
2052 // the same wording so log-line / test greps that anchored on
2053 // it keep matching, and no operator-visible message drift
2054 // lands as a side effect of the substrate move.
2055 let mut p = Process::new("scratch", empty_spec());
2056 p.metadata.name = None;
2057 let err = p.owned_coordinates_or_err().unwrap_err();
2058 assert_eq!(err.to_string(), "Process has no metadata.name");
2059 }
2060
2061 #[test]
2062 fn owned_coordinates_or_err_namespace_fallback_matches_default_namespace_const() {
2063 // Byte-identity pin between the owned form's namespace
2064 // fallback and the workspace-wide `DEFAULT_NAMESPACE` const.
2065 // A regression that spelled this fallback as any other
2066 // string ("kube-system", "", "default-ns") would silently
2067 // misroute every downstream namespaced-Api call on a
2068 // Process without a metadata.namespace — surfaces here
2069 // rather than at every kube-rs API caller.
2070 let mut p = Process::new("api", empty_spec());
2071 p.metadata.namespace = None;
2072 let (ns, _) = p.owned_coordinates_or_err().unwrap();
2073 assert_eq!(ns, Process::DEFAULT_NAMESPACE);
2074 }
2075
2076 #[test]
2077 fn owned_coordinates_or_err_matches_pre_lift_reconciler_helper_shape() {
2078 // Byte-identical parity pin between the owned + name-required
2079 // primitive here and the pre-lift `tatara-reconciler` helper
2080 // shape — the exact 2-slot unwrap chain each pre-lift caller
2081 // spelled by hand:
2082 //
2083 // let ns = p.metadata.namespace.clone().unwrap_or_else(|| "default".into());
2084 // let name = p.metadata.name.clone().ok_or_else(|| anyhow!(...))?;
2085 // Ok((ns, name))
2086 //
2087 // Sweeps every corner every callsite plausibly encounters
2088 // (both slots present, namespace absent, name absent, both
2089 // absent). A regression that inserted a normalization step
2090 // at the primitive that the pre-lift chain does NOT apply —
2091 // or vice versa — surfaces here rather than as silent drift
2092 // between the 12 pre-lift consumer callsites and the ONE
2093 // substrate owner they now route through.
2094 fn pre_lift(p: &Process) -> anyhow::Result<(String, String)> {
2095 let ns = p
2096 .metadata
2097 .namespace
2098 .clone()
2099 .unwrap_or_else(|| "default".into());
2100 let name = p
2101 .metadata
2102 .name
2103 .clone()
2104 .ok_or_else(|| anyhow::anyhow!("Process has no metadata.name"))?;
2105 Ok((ns, name))
2106 }
2107 // Both present.
2108 let mut p = Process::new("api", empty_spec());
2109 p.metadata.namespace = Some("prod".into());
2110 assert_eq!(p.owned_coordinates_or_err().unwrap(), pre_lift(&p).unwrap());
2111 // Namespace absent.
2112 let p = Process::new("api", empty_spec());
2113 assert_eq!(p.owned_coordinates_or_err().unwrap(), pre_lift(&p).unwrap());
2114 // Name absent → both variants error with the same wording.
2115 let mut p = Process::new("api", empty_spec());
2116 p.metadata.name = None;
2117 p.metadata.namespace = Some("prod".into());
2118 assert_eq!(
2119 p.owned_coordinates_or_err().unwrap_err().to_string(),
2120 pre_lift(&p).unwrap_err().to_string(),
2121 );
2122 // Both absent → still errors on the name gate.
2123 let mut p = Process::new("api", empty_spec());
2124 p.metadata.name = None;
2125 p.metadata.namespace = None;
2126 assert_eq!(
2127 p.owned_coordinates_or_err().unwrap_err().to_string(),
2128 pre_lift(&p).unwrap_err().to_string(),
2129 );
2130 }
2131
2132 #[test]
2133 fn owned_coordinates_or_err_axis_order_matches_coordinates_or_defaults() {
2134 // Cross-primitive coherence pin between the owned + name-
2135 // required form and the borrow + name-defaulted peer:
2136 // (namespace, name) axis order is IDENTICAL across both
2137 // return-forms. A regression that swapped the tuple slots on
2138 // only ONE of the two primitives would silently misroute
2139 // every consumer that picked between the two forms based on
2140 // its callsite's ownership needs. The pin re-reads both
2141 // primitives at test time so the equality holds iff both
2142 // live paths are the current implementation.
2143 let mut p = Process::new("app", empty_spec());
2144 p.metadata.namespace = Some("infra".into());
2145 let (borrow_ns, borrow_name) = p.coordinates_or_defaults();
2146 let (owned_ns, owned_name) = p.owned_coordinates_or_err().unwrap();
2147 assert_eq!(owned_ns, borrow_ns);
2148 assert_eq!(owned_name, borrow_name);
2149 // Explicit slot labels — pins the (namespace, name) axis
2150 // order as opposed to (name, namespace).
2151 assert_eq!(owned_ns, "infra"); // NOT "app"
2152 assert_eq!(owned_name, "app"); // NOT "infra"
2153 }
2154
2155 // ─── Process::coordinates_or_none substrate pins ──────────────────
2156 //
2157 // Pins the borrow + name-required peer of the coordinate-primitive
2158 // family on the (return-form × name-gate) axis pair. Closes the
2159 // corner previously left open (borrow + name-required) so the
2160 // three consumer shapes (child-Process delete-fan-out at
2161 // `phase_machine::handle_exiting`, claim-arbiter probe at
2162 // `phase_machine::process_holds_any_claim`, any future non-fatal
2163 // skip site) route through ONE primitive rather than three hand-
2164 // authored empty-string / `unwrap_or_default()` sentinel chains.
2165 // Fail-before-pass-after granularity: a regression that flipped
2166 // the namespace fallback, swapped the return-tuple axis order,
2167 // returned an owned form, or promoted a missing name to an error
2168 // rather than `None` surfaces here rather than as silent drift at
2169 // every borrow + name-required consumer.
2170
2171 #[test]
2172 fn coordinates_or_none_returns_slices_when_both_slots_present() {
2173 // Happy path — both slots populated, method returns borrowed
2174 // (&str, &str) in (namespace, name) axis order wrapped in
2175 // `Some`.
2176 let mut p = Process::new("api-gateway", empty_spec());
2177 p.metadata.namespace = Some("prod-app".into());
2178 let (ns, name) = p.coordinates_or_none().expect("Some when name set");
2179 assert_eq!(ns, "prod-app");
2180 assert_eq!(name, "api-gateway");
2181 }
2182
2183 #[test]
2184 fn coordinates_or_none_falls_back_on_namespace_but_returns_name_slice() {
2185 // Namespace absent → DEFAULT_NAMESPACE (shared with the peer
2186 // `coordinates_or_defaults` + `namespace_or_default`). Name
2187 // present → the metadata slice, wrapped in `Some`.
2188 let mut p = Process::new("api", empty_spec());
2189 p.metadata.namespace = None;
2190 let (ns, name) = p.coordinates_or_none().expect("Some when name set");
2191 assert_eq!(ns, Process::DEFAULT_NAMESPACE);
2192 assert_eq!(name, "api");
2193 }
2194
2195 #[test]
2196 fn coordinates_or_none_returns_none_when_metadata_name_absent_regardless_of_namespace() {
2197 // Name absent → `None`, REGARDLESS of whether the namespace
2198 // slot is populated. The name gate is strictly on
2199 // `metadata.name` and does NOT fall back to
2200 // `Self::UNNAMED_PLACEHOLDER` (that fallback is on the peer
2201 // `coordinates_or_defaults`, which exists precisely for
2202 // consumers that tolerate a display placeholder). Peer to
2203 // `owned_coordinates_or_err_errors_when_metadata_name_absent_regardless_of_namespace`
2204 // on the sibling primitive; a regression that widened THIS
2205 // form to substitute the placeholder while leaving the owned
2206 // form strict would silently drift the two borrow-form
2207 // primitives out of the coherence the family carries.
2208 for ns_slot in [None, Some("prod".to_string())] {
2209 let mut p = Process::new("scratch", empty_spec());
2210 p.metadata.name = None;
2211 p.metadata.namespace = ns_slot.clone();
2212 assert!(
2213 p.coordinates_or_none().is_none(),
2214 "coordinates_or_none must be None on missing name (ns={ns_slot:?})",
2215 );
2216 }
2217 }
2218
2219 #[test]
2220 fn coordinates_or_none_namespace_fallback_matches_default_namespace_const() {
2221 // Byte-identity pin between the borrow + name-required form's
2222 // namespace fallback and the workspace-wide `DEFAULT_NAMESPACE`
2223 // const. Sibling to
2224 // `owned_coordinates_or_err_namespace_fallback_matches_default_namespace_const`
2225 // on the peer primitive — the two forms MUST substitute the
2226 // same fallback string, else a consumer that switches between
2227 // them based on its ownership need silently observes a
2228 // different namespace-fallback shape as a side effect.
2229 let mut p = Process::new("api", empty_spec());
2230 p.metadata.namespace = None;
2231 let (ns, _) = p.coordinates_or_none().unwrap();
2232 assert_eq!(ns, Process::DEFAULT_NAMESPACE);
2233 }
2234
2235 #[test]
2236 fn coordinates_or_none_axis_order_matches_coordinates_or_defaults_when_name_present() {
2237 // Cross-primitive coherence pin between the two borrow-form
2238 // primitives: when the name is present, the (namespace, name)
2239 // return-tuple axis order is IDENTICAL across the two forms,
2240 // and the returned slices are the SAME `&str` view onto the
2241 // same metadata slots. A regression that swapped the tuple
2242 // slots on ONE form would silently misroute every consumer
2243 // that picked between the two forms based on its name-gate
2244 // need. The pin re-reads both primitives at test time so the
2245 // equality holds iff both live paths are the current
2246 // implementation.
2247 let mut p = Process::new("app", empty_spec());
2248 p.metadata.namespace = Some("infra".into());
2249 let (defaulted_ns, defaulted_name) = p.coordinates_or_defaults();
2250 let (required_ns, required_name) = p.coordinates_or_none().unwrap();
2251 assert_eq!(defaulted_ns, required_ns);
2252 assert_eq!(defaulted_name, required_name);
2253 // Explicit slot labels — pins the (namespace, name) axis order
2254 // as opposed to (name, namespace).
2255 assert_eq!(required_ns, "infra"); // NOT "app"
2256 assert_eq!(required_name, "app"); // NOT "infra"
2257 }
2258
2259 #[test]
2260 fn coordinates_or_none_axis_pair_diverges_from_coordinates_or_defaults_on_missing_name() {
2261 // Divergence pin between the two borrow-form primitives when
2262 // the name gate fires: `coordinates_or_defaults` substitutes
2263 // the display placeholder AND still returns a tuple;
2264 // `coordinates_or_none` returns `None`. A regression that
2265 // collapsed the two behaviors (either by dropping the gate
2266 // from the required form or by adding a `None` corner to the
2267 // defaulted form) would blur the axis pair's whole reason to
2268 // exist as two peer primitives.
2269 let mut p = Process::new("scratch", empty_spec());
2270 p.metadata.name = None;
2271 p.metadata.namespace = Some("prod".into());
2272 // Defaulted form: substitutes placeholder, no gate.
2273 assert_eq!(
2274 p.coordinates_or_defaults(),
2275 ("prod", Process::UNNAMED_PLACEHOLDER)
2276 );
2277 // Required form: gate fires, `None`.
2278 assert!(p.coordinates_or_none().is_none());
2279 }
2280
2281 #[test]
2282 fn coordinates_or_none_matches_pre_lift_reconciler_helper_shape() {
2283 // Byte-identical parity pin between the borrow + name-required
2284 // primitive here and the pre-lift `tatara-reconciler` helper
2285 // shapes — the exact 2-slot unwrap + gate chains each pre-lift
2286 // caller spelled by hand (`phase_machine::process_holds_any_claim`
2287 // spelled it as `unwrap_or("")` + `is_empty` early-return;
2288 // `phase_machine::handle_exiting`'s child-fan-out spelled it
2289 // as `unwrap_or_default()` + implicit no-op delete on the
2290 // empty API-path). Sweeps every corner every callsite plausibly
2291 // encounters (both slots present, namespace absent, name
2292 // absent + ns present, both absent). A regression that
2293 // inserted a normalization step at the primitive the pre-lift
2294 // chain does NOT apply — or vice versa — surfaces here rather
2295 // than as silent drift between the pre-lift consumer sites
2296 // and the ONE substrate owner they now route through.
2297 fn pre_lift_holds_any_claim(p: &Process) -> Option<(&str, &str)> {
2298 let ns = p.metadata.namespace.as_deref().unwrap_or("default");
2299 let name = p.metadata.name.as_deref().unwrap_or("");
2300 if name.is_empty() {
2301 return None;
2302 }
2303 Some((ns, name))
2304 }
2305 // Both present.
2306 let mut p = Process::new("api", empty_spec());
2307 p.metadata.namespace = Some("prod".into());
2308 assert_eq!(p.coordinates_or_none(), pre_lift_holds_any_claim(&p));
2309 // Namespace absent.
2310 let p = Process::new("api", empty_spec());
2311 assert_eq!(p.coordinates_or_none(), pre_lift_holds_any_claim(&p));
2312 // Name absent → both variants return `None` regardless of ns.
2313 let mut p = Process::new("api", empty_spec());
2314 p.metadata.name = None;
2315 p.metadata.namespace = Some("prod".into());
2316 assert_eq!(p.coordinates_or_none(), pre_lift_holds_any_claim(&p));
2317 // Both absent → still `None` on the name gate.
2318 let mut p = Process::new("api", empty_spec());
2319 p.metadata.name = None;
2320 p.metadata.namespace = None;
2321 assert_eq!(p.coordinates_or_none(), pre_lift_holds_any_claim(&p));
2322 }
2323
2324 #[test]
2325 fn coordinates_or_none_axis_order_matches_owned_coordinates_or_err_on_happy_path() {
2326 // Cross-primitive coherence pin at the sibling corner: when
2327 // BOTH slots are present, the borrow + name-required form
2328 // (this method) and the owned + name-required peer
2329 // (`owned_coordinates_or_err`) return the SAME `(ns, name)`
2330 // pair — the axis order is IDENTICAL and neither primitive
2331 // silently applies a normalization the other omits. A
2332 // regression that skewed one form's normalization would
2333 // surface here rather than as silent drift between the two
2334 // name-required corners of the primitive family.
2335 let mut p = Process::new("app", empty_spec());
2336 p.metadata.namespace = Some("infra".into());
2337 let (borrow_ns, borrow_name) = p.coordinates_or_none().unwrap();
2338 let (owned_ns, owned_name) = p.owned_coordinates_or_err().unwrap();
2339 assert_eq!(borrow_ns, owned_ns.as_str());
2340 assert_eq!(borrow_name, owned_name.as_str());
2341 }
2342
2343 #[test]
2344 fn coordinates_or_defaults_axis_order_matches_qualified_process_ref() {
2345 // Pins the load-bearing convention that the return-tuple
2346 // axis order is (namespace, name) — the exact positional
2347 // argument order the substrate's paired-composer primitive
2348 // `tatara_reconciler::ssapply::qualified_process_ref(ns,
2349 // name)` consumes. A regression that swapped the tuple
2350 // slots would silently misroute every annotation writer /
2351 // claim-arbiter row / owner-metadata seed built by feeding
2352 // this pair into the composer — every downstream `<ns>/
2353 // <name>` grep would suddenly see `<name>/<ns>`. The test
2354 // verifies the tuple's first slot is what a hand-authored
2355 // `.metadata.namespace.as_deref()...` produced pre-lift, and
2356 // the second slot is what `.metadata.name.as_deref()...`
2357 // produced.
2358 let mut p = Process::new("app", empty_spec());
2359 p.metadata.namespace = Some("infra".into());
2360 let (ns, name) = p.coordinates_or_defaults();
2361 assert_eq!(ns, "infra"); // NOT "app"
2362 assert_eq!(name, "app"); // NOT "infra"
2363 }
2364
2365 // ─── Process::annotation substrate pins ────────────────────────────
2366 //
2367 // Pins the borrow-form annotation-lookup primitive that owns the
2368 // 3-line `.metadata.annotations.as_ref().and_then(|m| m.get(KEY))`
2369 // chain three hand-authored sites restated by hand pre-lift:
2370 // `tatara-reconciler::signals::ingest` (SIGNAL),
2371 // `tatara-reconciler::phase_machine::released_from_annotation`
2372 // (RELEASED_FROM), and
2373 // `tatara-pool-reconciler::controller_pool::process_belongs_to_pool`
2374 // (POOL). Fail-before-pass-after granularity: a regression that
2375 // widened the missing-`annotations` corner (returning `Some("")`
2376 // instead of `None`), promoted a missing key to an error, dropped
2377 // the borrow-form return, or changed the two swallowed corners'
2378 // shared collapse to `None` surfaces here rather than as silent
2379 // drift at the three consumer sites.
2380 fn process_with_annotation(key: &str, value: &str) -> Process {
2381 let mut p = Process::new("some-proc", empty_spec());
2382 let mut anns = std::collections::BTreeMap::new();
2383 anns.insert(key.to_string(), value.to_string());
2384 p.metadata.annotations = Some(anns);
2385 p
2386 }
2387
2388 #[test]
2389 fn annotation_returns_none_when_metadata_annotations_is_none() {
2390 // Missing-`annotations` corner: a Process with no annotations
2391 // block at all returns `None` for every key. Peer to
2392 // `observed_flux_resources_returns_empty_slice_when_status_is_none`
2393 // on the status-projection axis; both primitives collapse the
2394 // outer `Option` corner rather than requiring each consumer
2395 // to spell the guard by hand.
2396 let mut p = Process::new("scratch", empty_spec());
2397 p.metadata.annotations = None;
2398 assert!(p.annotation("tatara.pleme.io/signal").is_none());
2399 assert!(p.annotation("tatara.pleme.io/pool").is_none());
2400 assert!(p.annotation("").is_none());
2401 }
2402
2403 #[test]
2404 fn annotation_returns_none_when_key_absent_from_populated_map() {
2405 // Missing-key corner: annotations block populated with OTHER
2406 // keys returns `None` for the queried key. Symmetric with the
2407 // missing-`annotations` corner — both corners collapse to the
2408 // same `None`, matching the pre-lift `.and_then(...)`
2409 // behavior every consumer relied on.
2410 let p = process_with_annotation("tatara.pleme.io/other", "value");
2411 assert!(p.annotation("tatara.pleme.io/signal").is_none());
2412 assert!(p.annotation("").is_none());
2413 }
2414
2415 #[test]
2416 fn annotation_returns_borrowed_slice_when_key_present() {
2417 // Happy path: annotations block populated + key present →
2418 // `Some(&str)` borrowed from the underlying `String` in the
2419 // map. A regression that returned an owned `String` (defeating
2420 // the primitive's role as a zero-copy projection) would
2421 // surface at the lifetime of the returned reference — the
2422 // `&str` outlives the borrow of `&p` here.
2423 let p = process_with_annotation("tatara.pleme.io/signal", "SIGHUP");
2424 assert_eq!(p.annotation("tatara.pleme.io/signal"), Some("SIGHUP"));
2425 }
2426
2427 #[test]
2428 fn annotation_returns_borrowed_empty_string_slice_when_value_is_empty() {
2429 // Edge corner between the missing-key `None` and the present-
2430 // key `Some("")` — a Process whose annotation is EXPLICITLY
2431 // set to an empty string returns `Some("")`, NOT `None`. A
2432 // regression that normalized the empty-string value to `None`
2433 // (a plausible "defensive" simplification) would silently
2434 // reshape the corner every callsite pre-lift kept distinct via
2435 // `.cloned().unwrap_or_default()` (which collapses BOTH to
2436 // `""`) or `.map(String::as_str)` (which keeps them distinct
2437 // as `None` vs `Some("")`).
2438 let p = process_with_annotation("tatara.pleme.io/signal", "");
2439 assert_eq!(p.annotation("tatara.pleme.io/signal"), Some(""));
2440 }
2441
2442 #[test]
2443 fn annotation_is_a_pure_projection() {
2444 // Purity pin — repeated calls return equal results and the
2445 // primitive does not mutate `self`. Peer to
2446 // `observed_flux_resources_is_a_pure_projection` on the
2447 // status-projection axis.
2448 let p = process_with_annotation("tatara.pleme.io/released-from", "Attested");
2449 let a = p.annotation("tatara.pleme.io/released-from");
2450 let b = p.annotation("tatara.pleme.io/released-from");
2451 assert_eq!(a, b);
2452 assert_eq!(a, Some("Attested"));
2453 }
2454
2455 #[test]
2456 fn annotation_matches_pre_lift_reconciler_chain_shape() {
2457 // Byte-identical parity pin between the borrow-form primitive
2458 // here and the pre-lift `tatara-reconciler` / `tatara-pool-
2459 // reconciler` chain shape — the exact 3-line
2460 // `.metadata.annotations.as_ref().and_then(|m| m.get(KEY))
2461 // .map(String::as_str)` incantation each pre-lift caller
2462 // spelled by hand (three variants of tail collapsed onto ONE
2463 // borrow-form primitive here; each caller reapplies its own
2464 // tail at its own site). Sweeps every corner (missing
2465 // annotations map, missing key, present key with value,
2466 // present key with empty value) so a regression that inserted
2467 // a normalization at the primitive the pre-lift chain does
2468 // NOT apply — or vice versa — surfaces here rather than as
2469 // silent drift between the ONE substrate owner and the three
2470 // consumer sites.
2471 fn pre_lift<'a>(p: &'a Process, key: &str) -> Option<&'a str> {
2472 p.metadata
2473 .annotations
2474 .as_ref()
2475 .and_then(|m| m.get(key))
2476 .map(String::as_str)
2477 }
2478 // Missing annotations map.
2479 let mut p = Process::new("x", empty_spec());
2480 p.metadata.annotations = None;
2481 assert_eq!(p.annotation("k"), pre_lift(&p, "k"));
2482 // Missing key in populated map.
2483 let p = process_with_annotation("other", "v");
2484 assert_eq!(p.annotation("k"), pre_lift(&p, "k"));
2485 // Present key with non-empty value.
2486 let p = process_with_annotation("k", "v");
2487 assert_eq!(p.annotation("k"), pre_lift(&p, "k"));
2488 // Present key with explicitly-empty value — the corner
2489 // `.cloned().unwrap_or_default()` collapses to `""` post-tail
2490 // but the primitive-level shape stays `Some("")`.
2491 let p = process_with_annotation("k", "");
2492 assert_eq!(p.annotation("k"), pre_lift(&p, "k"));
2493 }
2494
2495 #[test]
2496 fn annotation_composes_owned_tail_matching_pre_lift_signals_ingest() {
2497 // Pins the exact tail shape `tatara-reconciler::signals::
2498 // ingest` composed pre-lift: an `Option<String>` for the
2499 // downstream `let Some(raw) = raw else { ... }` guard.
2500 // Post-lift the callsite composes `.map(str::to_string)` at
2501 // its own site; this test pins the composition matches the
2502 // pre-lift `.cloned()` tail byte-for-byte on both corners the
2503 // consumer's downstream distinguishes (annotation present →
2504 // `Some(String)`; absent → `None`).
2505 let p = process_with_annotation("tatara.pleme.io/signal", "SIGUSR1");
2506 assert_eq!(
2507 p.annotation("tatara.pleme.io/signal").map(str::to_string),
2508 Some("SIGUSR1".to_string())
2509 );
2510 let mut q = Process::new("y", empty_spec());
2511 q.metadata.annotations = None;
2512 assert_eq!(
2513 q.annotation("tatara.pleme.io/signal").map(str::to_string),
2514 None
2515 );
2516 }
2517
2518 #[test]
2519 fn annotation_composes_default_tail_matching_pre_lift_released_from() {
2520 // Pins the exact tail shape
2521 // `tatara-reconciler::phase_machine::released_from_annotation`
2522 // composed pre-lift: a bare `String` via `.cloned()
2523 // .unwrap_or_default()` for the downstream
2524 // `match v.as_str()` dispatch. Post-lift the callsite matches
2525 // directly on `Option<&str>` (Some("Failed") vs _); this test
2526 // pins that the borrow-form primitive plus the `.unwrap_or("")`
2527 // fallback reproduces the pre-lift bare-string shape on both
2528 // corners.
2529 let p = process_with_annotation("tatara.pleme.io/released-from", "Failed");
2530 assert_eq!(
2531 p.annotation("tatara.pleme.io/released-from").unwrap_or(""),
2532 "Failed"
2533 );
2534 let mut q = Process::new("y", empty_spec());
2535 q.metadata.annotations = None;
2536 assert_eq!(
2537 q.annotation("tatara.pleme.io/released-from").unwrap_or(""),
2538 ""
2539 );
2540 }
2541
2542 #[test]
2543 fn annotation_composes_borrow_equality_tail_matching_pre_lift_pool() {
2544 // Pins the exact tail shape `tatara-pool-reconciler::
2545 // controller_pool::process_belongs_to_pool` composed pre-lift:
2546 // an `Option<&str>` compared with `== Some(pool_name)` for the
2547 // membership gate. Post-lift the callsite composes
2548 // `p.annotation(POOL) == Some(pool_name)` verbatim; this test
2549 // pins that the borrow-form primitive returns exactly the
2550 // shape the equality gate expects.
2551 let p = process_with_annotation("tatara.pleme.io/pool", "demo-pool");
2552 assert_eq!(
2553 p.annotation("tatara.pleme.io/pool") == Some("demo-pool"),
2554 true
2555 );
2556 assert_eq!(p.annotation("tatara.pleme.io/pool") == Some("other"), false);
2557 }
2558
2559 // ─── Process::uid_or_empty substrate pins ──────────────────────────
2560 //
2561 // Pins the borrow-form metadata-projection primitive on the
2562 // `metadata.uid` axis that owns the `.metadata.uid.as_deref()
2563 // .unwrap_or("")` chain the two hand-authored
2564 // `tatara-reconciler::render` sites (`render_routing` +
2565 // `render_export_jobs`) restated by hand pre-lift. Peer to the
2566 // sibling `namespace_or_default_*` + `name_or_placeholder_*` pin
2567 // families on the metadata-slot × fallback-shape axis; all three
2568 // primitives return borrows of an owned-metadata slot with a slot-
2569 // specific fallback baked in (`"default"` for namespace, `"unnamed"`
2570 // for name, `""` for uid — the load-bearing gate value for
2571 // `owner_references_json`'s `is_empty` check). Fail-before-pass-
2572 // after granularity: `uid_or_empty` did not exist pre-lift, so any
2573 // test invoking it fails to compile pre-lift and passes post-lift.
2574
2575 #[test]
2576 fn uid_or_empty_returns_empty_string_when_metadata_uid_is_none() {
2577 // Empty-slot corner pin: the primitive collapses the no-uid
2578 // case to `""`, matching the pre-lift `.as_deref().unwrap_or("")`
2579 // chain's `""` byte-identically at both render consumer sites.
2580 // Semantically corresponds to a Process pre-metadata (fixtured
2581 // in tests, or caught mid-Forking before the API server has
2582 // stamped a `uid`); the downstream `owner_references_json`
2583 // composer gates on this exact `""` sentinel to stamp
2584 // `metadata.ownerReferences: []` rather than emit an owner-ref
2585 // pointing at a placeholder uid.
2586 let mut p = Process::new("scratch", empty_spec());
2587 p.metadata.uid = None;
2588 assert_eq!(p.uid_or_empty(), "");
2589 }
2590
2591 #[test]
2592 fn uid_or_empty_returns_borrowed_str_when_slot_is_populated() {
2593 // Happy-path pin: with a populated `metadata.uid` slot, the
2594 // primitive returns a borrowed `&str` whose contents match the
2595 // persisted `String`. A regression that reshaped / normalized
2596 // / cross-cluster-stripped the uid without touching this pin
2597 // would surface here rather than as silent skew at the two
2598 // `owner_references_json(name, uid)` emitters on the SAME
2599 // Process.
2600 let mut p = Process::new("owned-proc", empty_spec());
2601 p.metadata.uid = Some("uid-abc-123".into());
2602 assert_eq!(p.uid_or_empty(), "uid-abc-123");
2603 }
2604
2605 #[test]
2606 fn uid_or_empty_returns_empty_string_when_slot_is_explicitly_empty_string() {
2607 // Corner between the missing-slot `None` and the explicitly-
2608 // empty-string `Some("")` — both collapse to `""` at the
2609 // primitive because the downstream gate at
2610 // `owner_references_json` treats `.is_empty()` uniformly (the
2611 // empty-slot posture is what the whole primitive family
2612 // encodes: "no admissible owner reference, stamp `[]`"). A
2613 // regression that discriminated the two corners (returning a
2614 // sentinel `"<none>"` for the missing slot but `""` for the
2615 // explicit slot) would break the composition with
2616 // `owner_references_json` at the exactly-two-corner gate.
2617 let mut p = Process::new("owned-proc", empty_spec());
2618 p.metadata.uid = Some(String::new());
2619 assert_eq!(p.uid_or_empty(), "");
2620 }
2621
2622 #[test]
2623 fn uid_or_empty_is_a_zero_copy_borrow_projection() {
2624 // Borrow-discipline pin: the returned `&str` borrows the
2625 // persisted `String`'s underlying byte buffer in place — NOT
2626 // a fresh allocation or a clone. A regression that switched
2627 // the projection to an owned `String` (via `.clone()` or a
2628 // `format!` wrap) would defeat the zero-copy contract the
2629 // lift's primary strict-widening delivers, and would surface
2630 // here via pointer-identity comparison.
2631 let mut p = Process::new("owned-proc", empty_spec());
2632 p.metadata.uid = Some("uid-borrow-pin".into());
2633 let slice = p.uid_or_empty();
2634 assert!(std::ptr::eq(
2635 slice.as_ptr(),
2636 p.metadata.uid.as_ref().unwrap().as_ptr()
2637 ));
2638 }
2639
2640 #[test]
2641 fn uid_or_empty_is_a_pure_projection() {
2642 // Purity pin — repeated calls return byte-identical slices
2643 // (same pointer, same length). A regression that introduced
2644 // state (a lazy-cached normalized slot, a first-call
2645 // canonicalization pass) would surface here rather than as
2646 // silent drift between the two render consumer sites on the
2647 // SAME Process within one render pass.
2648 let mut p = Process::new("owned-proc", empty_spec());
2649 p.metadata.uid = Some("uid-pure".into());
2650 let a = p.uid_or_empty();
2651 let b = p.uid_or_empty();
2652 assert!(std::ptr::eq(a.as_ptr(), b.as_ptr()));
2653 assert_eq!(a.len(), b.len());
2654 }
2655
2656 #[test]
2657 fn uid_or_empty_matches_pre_lift_render_chain_shape() {
2658 // Byte-identical parity pin between the borrow-form primitive
2659 // here and the pre-lift `tatara-reconciler::render` chain shape
2660 // — the exact `.metadata.uid.as_deref().unwrap_or("")`
2661 // incantation both `render_routing` (line 514) and
2662 // `render_export_jobs` (line 653) spelled by hand pre-lift.
2663 // Sweeps every corner (missing uid slot, populated uid slot,
2664 // explicitly-empty uid slot) so a regression that inserted a
2665 // normalization the pre-lift chain does NOT apply — or vice
2666 // versa — surfaces here rather than as silent drift between
2667 // the ONE substrate owner and the two consumer sites.
2668 fn pre_lift(p: &Process) -> &str {
2669 p.metadata.uid.as_deref().unwrap_or("")
2670 }
2671 // Missing slot.
2672 let mut p = Process::new("x", empty_spec());
2673 p.metadata.uid = None;
2674 assert_eq!(p.uid_or_empty(), pre_lift(&p));
2675 // Populated slot.
2676 let mut p = Process::new("x", empty_spec());
2677 p.metadata.uid = Some("uid-42".into());
2678 assert_eq!(p.uid_or_empty(), pre_lift(&p));
2679 // Explicitly-empty slot.
2680 let mut p = Process::new("x", empty_spec());
2681 p.metadata.uid = Some(String::new());
2682 assert_eq!(p.uid_or_empty(), pre_lift(&p));
2683 }
2684
2685 #[test]
2686 fn uid_or_empty_composes_with_owner_references_json_empty_gate() {
2687 // Cross-primitive composition pin — the empty-string sentinel
2688 // this primitive returns for the missing-uid corner is EXACTLY
2689 // the sentinel the sibling substrate composer
2690 // `owner_references_json(name, uid)` gates on to stamp
2691 // `metadata.ownerReferences: []`. A regression that changed
2692 // the sentinel at either end (this primitive returning
2693 // `"<none>"`, `owner_references_json` gating on `uid == "0"`
2694 // instead of `uid.is_empty()`) would break the composition
2695 // and surface here rather than as an operator-observed
2696 // orphan resource after apply.
2697 let mut p = Process::new("x", empty_spec());
2698 p.metadata.uid = None;
2699 let refs = crate::owner_references_json("some-name", p.uid_or_empty());
2700 assert!(
2701 refs.is_empty(),
2702 "empty-uid corner must produce empty owner-refs array"
2703 );
2704
2705 p.metadata.uid = Some("real-uid".into());
2706 let refs = crate::owner_references_json("some-name", p.uid_or_empty());
2707 assert_eq!(
2708 refs.len(),
2709 1,
2710 "populated-uid corner must produce one owner-ref entry"
2711 );
2712 }
2713
2714 // ─── Process::owned_name_or_empty substrate pins ─────────────────
2715 //
2716 // Pins the owned-form metadata-projection primitive on the
2717 // `metadata.name` axis that owns the
2718 // `.metadata.name.clone().unwrap_or_default()` chain the two hand-
2719 // authored `tatara-pool-reconciler::controller_pool` sites (the
2720 // `PoolMember` seed at line 68 + the `PoolMemberSnapshot` desired-
2721 // count seed at line 108) restated by hand pre-lift. Peer to the
2722 // sibling `uid_or_empty` pin family on the (return-form × fallback-
2723 // value) axis pair — `uid_or_empty` owns the BORROW + empty-sentinel
2724 // corner (`&str` for owner-ref emitters gating on `.is_empty()`);
2725 // this method owns the OWNED + empty-sentinel corner (`String` for
2726 // struct-literal / HashMap-key row-builder consumers whose
2727 // downstream fills a `String` field with the load-bearing `""`
2728 // sentinel). Fail-before-pass-after granularity: `owned_name_or_empty`
2729 // did not exist pre-lift, so any test invoking it fails to compile
2730 // pre-lift and passes post-lift.
2731
2732 #[test]
2733 fn owned_name_or_empty_returns_empty_string_when_metadata_name_is_none() {
2734 // Empty-slot corner pin: the primitive collapses the no-name
2735 // case to `String::new()`, matching the pre-lift
2736 // `.clone().unwrap_or_default()` chain's empty `String` byte-
2737 // identically at both pool-reconciler consumer sites.
2738 // Semantically corresponds to a Process pre-metadata-name (test
2739 // fixture, dynamic API response pre-name-resolution); the
2740 // downstream `PoolMember { process_name, .. }` slot then holds
2741 // `""` as a stable "no name to key by" signal rather than a
2742 // display placeholder that would silently alias distinct rows.
2743 let mut p = Process::new("scratch", empty_spec());
2744 p.metadata.name = None;
2745 assert_eq!(p.owned_name_or_empty(), String::new());
2746 }
2747
2748 #[test]
2749 fn owned_name_or_empty_returns_owned_string_when_slot_is_populated() {
2750 // Happy-path pin: with a populated `metadata.name` slot, the
2751 // primitive returns an owned `String` whose contents match the
2752 // persisted `String`. A regression that reshaped / normalized
2753 // / case-folded the name without touching this pin would surface
2754 // here rather than as silent skew between the two pool-member
2755 // seeds keying on the SAME Process's name.
2756 let p = Process::new("api", empty_spec());
2757 assert_eq!(p.owned_name_or_empty(), "api");
2758 }
2759
2760 #[test]
2761 fn owned_name_or_empty_returns_empty_string_when_slot_is_explicitly_empty_string() {
2762 // Corner between the missing-slot `None` and the explicitly-
2763 // empty-string `Some(String::new())` — both collapse to `""` at
2764 // the primitive because the downstream pool-member consumers
2765 // treat both corners uniformly (no name, no key). A regression
2766 // that discriminated the two corners (returning a sentinel
2767 // `"<none>"` for the missing slot but `""` for the explicit
2768 // slot) would break `String::is_empty` gating at the row-builder
2769 // callsites without moving this pin.
2770 let mut p = Process::new("scratch", empty_spec());
2771 p.metadata.name = Some(String::new());
2772 assert_eq!(p.owned_name_or_empty(), String::new());
2773 assert!(p.owned_name_or_empty().is_empty());
2774 }
2775
2776 #[test]
2777 fn owned_name_or_empty_is_a_pure_projection() {
2778 // Purity pin — repeated calls return byte-identical `String`
2779 // values. A regression that introduced state (a lazy-cached
2780 // normalized slot, a first-call canonicalization pass) would
2781 // surface here rather than as silent drift between the pool-
2782 // member seed and the desired-count snapshot seed on the SAME
2783 // Process within one reconcile pass.
2784 let p = Process::new("stable-name", empty_spec());
2785 assert_eq!(p.owned_name_or_empty(), p.owned_name_or_empty());
2786 }
2787
2788 #[test]
2789 fn owned_name_or_empty_returns_independent_owned_string() {
2790 // Owned-discipline pin: the returned `String` is an independent
2791 // allocation the caller may consume, `.push_str` into, or move
2792 // into a struct-literal `process_name: String` slot — NOT a
2793 // shared reference into `metadata.name`. A regression that
2794 // switched the projection to a `Cow`-shaped variant or a slice-
2795 // form projection would defeat the owned-form contract the two
2796 // pool-reconciler struct-literal consumers depend on (a slice
2797 // cannot land in a `process_name: String` slot without a re-
2798 // clone), and would surface here at compile time via the mutate-
2799 // in-place test below.
2800 let p = Process::new("owned-proc", empty_spec());
2801 let mut owned = p.owned_name_or_empty();
2802 owned.push_str("-mutated");
2803 assert_eq!(owned, "owned-proc-mutated");
2804 // The Process's own slot is unchanged — the returned String
2805 // owns its own byte buffer, disjoint from `metadata.name`.
2806 assert_eq!(p.metadata.name.as_deref(), Some("owned-proc"));
2807 }
2808
2809 #[test]
2810 fn owned_name_or_empty_matches_pre_lift_controller_pool_chain_shape() {
2811 // Byte-identical parity pin between the owned-form primitive
2812 // here and the pre-lift `tatara-pool-reconciler::controller_pool`
2813 // chain shape — the exact `.metadata.name.clone().unwrap_or_default()`
2814 // incantation both `PoolMember` seed (line 68) and
2815 // `PoolMemberSnapshot` seed (line 108) spelled by hand pre-lift.
2816 // Sweeps every corner (missing name slot, populated name slot,
2817 // explicitly-empty name slot) so a regression that inserted a
2818 // normalization the pre-lift chain does NOT apply — or vice
2819 // versa — surfaces here rather than as silent drift between
2820 // the ONE substrate owner and the two consumer sites.
2821 fn pre_lift(p: &Process) -> String {
2822 p.metadata.name.clone().unwrap_or_default()
2823 }
2824 // Missing slot.
2825 let mut p = Process::new("x", empty_spec());
2826 p.metadata.name = None;
2827 assert_eq!(p.owned_name_or_empty(), pre_lift(&p));
2828 // Populated slot.
2829 let p = Process::new("real-name", empty_spec());
2830 assert_eq!(p.owned_name_or_empty(), pre_lift(&p));
2831 // Explicitly-empty slot.
2832 let mut p = Process::new("x", empty_spec());
2833 p.metadata.name = Some(String::new());
2834 assert_eq!(p.owned_name_or_empty(), pre_lift(&p));
2835 }
2836
2837 #[test]
2838 fn owned_name_or_empty_shares_empty_sentinel_with_uid_or_empty() {
2839 // Cross-primitive coherence pin — the empty-string fallback this
2840 // primitive returns for the missing-name corner is the SAME
2841 // sentinel the sibling borrow-form primitive `uid_or_empty`
2842 // returns for the missing-uid corner. Both partition the OWNED
2843 // × BORROW corner of the metadata-slot family on identical
2844 // fallback semantics ("the slot is unset"), so a consumer that
2845 // switches between them based on downstream ownership
2846 // requirements never sees a different missing-slot spelling as
2847 // a side effect. A regression that drifted either sentinel
2848 // (this primitive returning `"<unnamed>"`, `uid_or_empty`
2849 // returning `"<none>"`) would break the partition and surface
2850 // here rather than as silent shape drift across the family.
2851 let mut p = Process::new("scratch", empty_spec());
2852 p.metadata.name = None;
2853 p.metadata.uid = None;
2854 assert_eq!(p.owned_name_or_empty(), p.uid_or_empty());
2855 assert!(p.owned_name_or_empty().is_empty());
2856 assert!(p.uid_or_empty().is_empty());
2857 }
2858
2859 #[test]
2860 fn owned_name_or_empty_returns_distinct_fallback_from_name_or_placeholder() {
2861 // Axis-partition pin — the owned + empty-sentinel primitive here
2862 // and the borrow + display-placeholder primitive
2863 // [`Self::name_or_placeholder`] MUST return distinct fallback
2864 // values on the missing-name corner. The distinction is load-
2865 // bearing: `owned_name_or_empty` is for HashMap-key / row-builder
2866 // consumers that need distinct keys for missing-name Processes
2867 // (empty string collides only with other missing-name rows,
2868 // never with a real "unnamed" Process); `name_or_placeholder`
2869 // is for log-line / display consumers that render the
2870 // `"unnamed"` word to operators. A regression that unified the
2871 // two fallbacks (either primitive returning the other's
2872 // sentinel) would silently collapse missing-name pool members
2873 // into a display-string key or expose the empty sentinel to
2874 // operator log lines. This pin catches either drift.
2875 let mut p = Process::new("scratch", empty_spec());
2876 p.metadata.name = None;
2877 assert_eq!(p.owned_name_or_empty(), "");
2878 assert_eq!(p.name_or_placeholder(), Process::UNNAMED_PLACEHOLDER);
2879 assert_ne!(p.owned_name_or_empty(), p.name_or_placeholder());
2880 }
2881
2882 // ─── Process::declared_parent_pid substrate pins ─────────────────
2883 //
2884 // Pins the borrow-form spec-projection primitive on the declared
2885 // parent-PID axis that owns the `.spec.identity.parent.as_deref()`
2886 // chain the two hand-authored `tatara-reconciler::phase_machine`
2887 // sites (`handle_forking` ALLOCATE-PID composer + `handle_exiting`
2888 // SIGTERM-cascade child-fan-out filter) restated by hand pre-lift.
2889 // Peer to the sibling `observed_pid_*` pin family on the (spec-
2890 // declared × status-observed) axis pair; both compose the same
2891 // borrow-form `Option<&str>` return-shape skeleton on distinct
2892 // slots (`spec.identity.parent` vs. `status.pid`). Fail-before-
2893 // pass-after granularity: `declared_parent_pid` did not exist
2894 // pre-lift, so any test invoking it fails to compile pre-lift and
2895 // passes post-lift.
2896 fn process_with_declared_parent(parent: Option<&str>) -> Process {
2897 let mut spec = empty_spec();
2898 spec.identity.parent = parent.map(str::to_string);
2899 Process::new("child-proc", spec)
2900 }
2901
2902 #[test]
2903 fn declared_parent_pid_returns_none_when_slot_is_none() {
2904 // Empty-slot corner pin: the primitive collapses the no-
2905 // parent case to `None`, matching the pre-lift `.as_deref()`
2906 // chain's `None` byte-identically at both reconciler consumer
2907 // sites. Semantically corresponds to a Process authored at
2908 // cluster init (PID 1) with no upstream parent — the
2909 // ALLOCATE-PID composer feeds `None` into `pid::allocate_pid`
2910 // to signal "no prefix", and the SIGTERM cascade's filter
2911 // never matches such a Process because a child's declared
2912 // parent can never equal `Some(pid)` when the slot is `None`.
2913 let p = process_with_declared_parent(None);
2914 assert!(p.declared_parent_pid().is_none());
2915 }
2916
2917 #[test]
2918 fn declared_parent_pid_returns_borrowed_str_when_slot_is_populated() {
2919 // Happy-path pin: with a populated `spec.identity.parent`
2920 // slot, the primitive returns a borrowed `&str` whose
2921 // contents match the persisted `String`. A regression that
2922 // filtered / reshaped / canonicalized the string would
2923 // surface here rather than as silent skew at the child-fan-
2924 // out filter's `.declared_parent_pid() == Some(pid)`
2925 // equality check on the SAME parent-child pair.
2926 let p = process_with_declared_parent(Some("seph.1"));
2927 assert_eq!(p.declared_parent_pid(), Some("seph.1"));
2928 }
2929
2930 #[test]
2931 fn declared_parent_pid_is_a_zero_copy_borrow_projection() {
2932 // Borrow-discipline pin: the returned `&str` borrows the
2933 // persisted `String`'s underlying byte buffer in place —
2934 // NOT a fresh allocation or a clone. A regression that
2935 // switched the projection to an owned `String` (via
2936 // `.clone()` or `.to_owned()`) would defeat the zero-copy
2937 // contract the lift's primary strict-widening delivers.
2938 // The `handle_exiting` cascade filter runs per candidate
2939 // child across the cluster-wide Process list; a per-row
2940 // `String::clone` would allocate one heap block per non-
2941 // matching row, so the borrow-form primitive is load-
2942 // bearing for large clusters. Peer to the sibling
2943 // `observed_pid_is_a_zero_copy_borrow_projection` pin on
2944 // the status-observed side of the axis pair.
2945 let p = process_with_declared_parent(Some("seph.1"));
2946 let borrowed = p.declared_parent_pid().expect("populated slot");
2947 let persisted = p.spec.identity.parent.as_ref().unwrap();
2948 assert!(std::ptr::eq(borrowed.as_ptr(), persisted.as_ptr()));
2949 }
2950
2951 #[test]
2952 fn declared_parent_pid_is_a_pure_projection() {
2953 // Purity pin: calling the projection twice on the same
2954 // `Process` returns byte-identical `&str`s (same pointer,
2955 // same length). A regression that introduced state — a
2956 // lazy-cached slice materialized on first call, a
2957 // normalization step that ran once and cached — would
2958 // surface here rather than as silent drift between the
2959 // ALLOCATE-PID composer and the SIGTERM cascade's child-
2960 // fan-out filter within one reconcile pass.
2961 let p = process_with_declared_parent(Some("seph.1.3"));
2962 let a = p.declared_parent_pid().expect("populated slot");
2963 let b = p.declared_parent_pid().expect("populated slot");
2964 assert!(std::ptr::eq(a.as_ptr(), b.as_ptr()));
2965 assert_eq!(a.len(), b.len());
2966 }
2967
2968 #[test]
2969 fn declared_parent_pid_matches_pre_lift_reconciler_chain_shape() {
2970 // Byte-identical parity pin between the borrow-form primitive
2971 // here and the pre-lift `tatara-reconciler::phase_machine`
2972 // `.spec.identity.parent.as_deref()` chain shape. Sweeps
2973 // every corner every callsite plausibly encounters (empty
2974 // slot, populated with a hierarchical PID). A regression
2975 // that inserted a normalization step at the primitive the
2976 // pre-lift chain does NOT apply — or vice versa — surfaces
2977 // here rather than as silent drift between the pre-lift
2978 // consumer sites and the ONE substrate owner they now route
2979 // through. Peer to
2980 // `observed_pid_matches_pre_lift_reconciler_chain_shape` on
2981 // the sibling axis's borrow-form primitive.
2982 fn pre_lift(p: &Process) -> Option<&str> {
2983 p.spec.identity.parent.as_deref()
2984 }
2985 // Empty slot.
2986 let p = process_with_declared_parent(None);
2987 assert_eq!(p.declared_parent_pid(), pre_lift(&p));
2988 // Populated with a hierarchical PID.
2989 let p = process_with_declared_parent(Some("seph.1"));
2990 assert_eq!(p.declared_parent_pid(), pre_lift(&p));
2991 // Populated with a deeper hierarchical PID.
2992 let p = process_with_declared_parent(Some("seph.1.7.42"));
2993 assert_eq!(p.declared_parent_pid(), pre_lift(&p));
2994 }
2995
2996 #[test]
2997 fn declared_parent_pid_preserves_hierarchical_pid_format() {
2998 // Format-preservation pin: the hierarchical PID path
2999 // (dotted-segment form `seph.1.7`, matching the ported
3000 // `convergence-controller/src/identity.rs` scheme) reaches
3001 // the caller with segments and separators byte-identical
3002 // to the persisted `String`. A regression that inserted a
3003 // canonicalization pass (a segment-count validator, a
3004 // separator swap `.` → `/`, a leading/trailing whitespace
3005 // trim) would silently misroute the SIGTERM cascade's
3006 // `declared_parent_pid() == Some(pid)` comparator against
3007 // children whose `parent` field was authored in the ported
3008 // scheme's exact form — the SAME children the observed_pid
3009 // primitive is pinned to match on the other side of the
3010 // axis pair.
3011 for parent in ["seph", "seph.1", "seph.1.7", "seph.1.7.42"] {
3012 let p = process_with_declared_parent(Some(parent));
3013 assert_eq!(p.declared_parent_pid(), Some(parent));
3014 }
3015 }
3016
3017 #[test]
3018 fn declared_parent_pid_composes_with_observed_pid_for_child_fanout_filter() {
3019 // Cross-axis coherence pin against the sibling
3020 // [`Self::observed_pid`] on the (spec-declared × status-
3021 // observed) axis pair: a child's `.declared_parent_pid()`
3022 // and its parent's `.observed_pid()` compose through the
3023 // SAME borrow-form `Option<&str>` skeleton so the
3024 // `handle_exiting` cascade filter's equality gate holds
3025 // structurally. A regression that skewed EITHER primitive's
3026 // return-form (return-shape, borrow discipline, empty-slot
3027 // collapse) would silently misroute every SIGTERM cascade
3028 // on the parent-child pair. This pin re-reads both primitives
3029 // at test time so the composition holds iff both live paths
3030 // are the current implementation.
3031 // Parent Process: has an observed PID.
3032 let mut parent = Process::new("parent-proc", empty_spec());
3033 parent.status = Some(ProcessStatus {
3034 pid: Some("seph.1".to_string()),
3035 ..Default::default()
3036 });
3037 // Child Process: declared parent matches parent's observed PID.
3038 let child = process_with_declared_parent(Some("seph.1"));
3039 // The `handle_exiting` filter's equality gate:
3040 // `child.declared_parent_pid() == Some(parent.observed_pid()?)`.
3041 let parent_pid = parent.observed_pid().expect("parent has PID");
3042 assert_eq!(child.declared_parent_pid(), Some(parent_pid));
3043 // Sibling Process with an unrelated declared parent must NOT
3044 // match the same parent — pins that the filter's SKIP branch
3045 // holds on the other side of the axis pair.
3046 let sibling = process_with_declared_parent(Some("seph.2"));
3047 assert_ne!(sibling.declared_parent_pid(), Some(parent_pid));
3048 }
3049
3050 // ─── Process::declared_name_override substrate pins ──────────────
3051 //
3052 // Pins the borrow-form spec-projection primitive on the declared
3053 // name-override sub-axis of the declared-identity axis that owns
3054 // the `.spec.identity.name_override.as_deref()` chain the two
3055 // hand-authored `tatara-reconciler::phase_machine` sites
3056 // (`handle_pending` DECLARE composer + `handle_forking` ALLOCATE-
3057 // PID rehydration branch) restated by hand pre-lift. Peer to the
3058 // sibling `declared_parent_pid_*` pin family on the (parent ×
3059 // name-override) sub-axis pair; both compose the same borrow-form
3060 // `Option<&str>` return-shape skeleton on distinct slots
3061 // (`spec.identity.name_override` vs `spec.identity.parent`).
3062 // Fail-before-pass-after granularity: `declared_name_override`
3063 // did not exist pre-lift, so any test invoking it fails to
3064 // compile pre-lift and passes post-lift.
3065 fn process_with_declared_name_override(name_override: Option<&str>) -> Process {
3066 let mut spec = empty_spec();
3067 spec.identity.name_override = name_override.map(str::to_string);
3068 Process::new("some-proc", spec)
3069 }
3070
3071 #[test]
3072 fn declared_name_override_returns_none_when_slot_is_none() {
3073 // Empty-slot corner pin: the primitive collapses the no-
3074 // override case to `None`, matching the pre-lift `.as_deref()`
3075 // chain's `None` byte-identically at both reconciler consumer
3076 // sites. Semantically corresponds to a Process authored
3077 // WITHOUT the human-name-override escape hatch — the default;
3078 // `derive_identity` then computes the name from the content
3079 // hash and stamps `name_override: false` on the resulting
3080 // [`Identity`].
3081 let p = process_with_declared_name_override(None);
3082 assert!(p.declared_name_override().is_none());
3083 }
3084
3085 #[test]
3086 fn declared_name_override_returns_borrowed_str_when_slot_is_populated() {
3087 // Happy-path pin: with a populated `spec.identity
3088 // .name_override` slot, the primitive returns a borrowed
3089 // `&str` whose contents match the persisted `String`. A
3090 // regression that filtered / reshaped / canonicalized the
3091 // string at the primitive (as opposed to inside
3092 // `derive_identity`, where the trim/empty-filter lives today)
3093 // would surface here rather than as silent skew between the
3094 // DECLARE composer and the ALLOCATE-PID rehydration branch on
3095 // the SAME Process spec.
3096 let p = process_with_declared_name_override(Some("observability-stack"));
3097 assert_eq!(p.declared_name_override(), Some("observability-stack"));
3098 }
3099
3100 #[test]
3101 fn declared_name_override_is_a_zero_copy_borrow_projection() {
3102 // Borrow-discipline pin: the returned `&str` borrows the
3103 // persisted `String`'s underlying byte buffer in place —
3104 // NOT a fresh allocation or a clone. Peer to the sibling
3105 // `declared_parent_pid_is_a_zero_copy_borrow_projection` pin
3106 // on the other side of the (parent × name-override) sub-axis
3107 // pair; the borrow discipline holds structurally on BOTH
3108 // sub-axes so a future `declared_identity` composite that
3109 // returns both halves together can compose them without
3110 // dropping into an owning form.
3111 let p = process_with_declared_name_override(Some("observability-stack"));
3112 let borrowed = p.declared_name_override().expect("populated slot");
3113 let persisted = p.spec.identity.name_override.as_ref().unwrap();
3114 assert!(std::ptr::eq(borrowed.as_ptr(), persisted.as_ptr()));
3115 }
3116
3117 #[test]
3118 fn declared_name_override_is_a_pure_projection() {
3119 // Purity pin: calling the projection twice on the same
3120 // `Process` returns byte-identical `&str`s (same pointer,
3121 // same length). A regression that introduced state — a
3122 // lazy-cached slice materialized on first call, a
3123 // normalization step that ran once and cached — would
3124 // surface here rather than as silent drift between the
3125 // DECLARE composer and the ALLOCATE-PID rehydration branch
3126 // within one reconcile pass.
3127 let p = process_with_declared_name_override(Some("gateway-primary"));
3128 let a = p.declared_name_override().expect("populated slot");
3129 let b = p.declared_name_override().expect("populated slot");
3130 assert!(std::ptr::eq(a.as_ptr(), b.as_ptr()));
3131 assert_eq!(a.len(), b.len());
3132 }
3133
3134 #[test]
3135 fn declared_name_override_matches_pre_lift_reconciler_chain_shape() {
3136 // Byte-identical parity pin between the borrow-form primitive
3137 // here and the pre-lift `tatara-reconciler::phase_machine`
3138 // `.spec.identity.name_override.as_deref()` chain shape.
3139 // Sweeps every corner every callsite plausibly encounters
3140 // (empty slot, populated with a bare name, populated with a
3141 // whitespace-containing name that `derive_identity`'s
3142 // internal trim would collapse, populated with an explicitly
3143 // empty string that `derive_identity`'s internal
3144 // `!s.is_empty()` filter would reject). A regression that
3145 // inserted a normalization step at the primitive the pre-
3146 // lift chain does NOT apply — or vice versa — surfaces here
3147 // rather than as silent drift between the pre-lift consumer
3148 // sites and the ONE substrate owner they now route through.
3149 // Peer to
3150 // `declared_parent_pid_matches_pre_lift_reconciler_chain_shape`
3151 // on the sibling sub-axis's borrow-form primitive.
3152 fn pre_lift(p: &Process) -> Option<&str> {
3153 p.spec.identity.name_override.as_deref()
3154 }
3155 // Empty slot.
3156 let p = process_with_declared_name_override(None);
3157 assert_eq!(p.declared_name_override(), pre_lift(&p));
3158 // Populated with a bare name.
3159 let p = process_with_declared_name_override(Some("observability-stack"));
3160 assert_eq!(p.declared_name_override(), pre_lift(&p));
3161 // Populated with a whitespace-containing name.
3162 let p = process_with_declared_name_override(Some(" observability-stack "));
3163 assert_eq!(p.declared_name_override(), pre_lift(&p));
3164 // Populated with an explicitly empty string. Distinct from
3165 // the missing-slot `None` corner both at the primitive here
3166 // and at the pre-lift chain (the trim/filter that collapses
3167 // these two into the same `false`-branched
3168 // `Identity { name_override: false, .. }` lives INSIDE
3169 // `derive_identity`, NOT at the borrow site) — the primitive
3170 // MUST preserve the distinction so a future lift of the trim/
3171 // filter OUT of `derive_identity` INTO the primitive is a
3172 // conscious substrate change, not a silent one.
3173 let p = process_with_declared_name_override(Some(""));
3174 assert_eq!(p.declared_name_override(), pre_lift(&p));
3175 }
3176
3177 #[test]
3178 fn declared_name_override_preserves_raw_slot_verbatim() {
3179 // Invariance-under-`derive_identity`-normalization pin: the
3180 // primitive returns the slot's raw byte contents verbatim —
3181 // no trim, no empty-string filter, no case fold, no
3182 // normalization of any kind. `derive_identity` internally
3183 // applies `.map(str::trim).filter(|s| !s.is_empty())` before
3184 // dispatching on `Some(non_empty)` vs `None | Some(empty |
3185 // whitespace)`, but that transform lives IN `derive_identity`,
3186 // NOT at the borrow site. A regression that pulled the trim/
3187 // filter forward INTO the primitive would silently collapse
3188 // three currently-distinct corners at the borrow site (bare
3189 // populated → `Some(name)`; whitespace-only → `Some(" ")`;
3190 // empty → `Some("")`) into two (bare → `Some(name)`; the
3191 // other two → `None`). That collapse might be an intentional
3192 // substrate change some future run wants to make; if so, it
3193 // lands as a conscious edit here (with this pin updated in
3194 // the same commit) rather than as silent behavior drift.
3195 for value in ["bare", " padded ", "\ttabs\t", " ", ""] {
3196 let p = process_with_declared_name_override(Some(value));
3197 assert_eq!(
3198 p.declared_name_override(),
3199 Some(value),
3200 "declared_name_override must preserve raw slot verbatim for value {value:?}"
3201 );
3202 }
3203 }
3204
3205 #[test]
3206 fn declared_name_override_composes_with_derive_identity_call_shape() {
3207 // Cross-primitive coherence pin against the [`derive_identity`]
3208 // consumer: the two live `tatara-reconciler::phase_machine`
3209 // callsites feed `p.declared_name_override()` as the second
3210 // positional argument to `derive_identity(&p.spec, …)`. This
3211 // pin exercises that exact call shape at test time so a
3212 // regression that skewed the primitive's return-form (return-
3213 // shape, borrow discipline, empty-slot collapse) surfaces
3214 // here as a shape mismatch at the [`derive_identity`] call
3215 // site rather than as silent operator-facing skew between the
3216 // DECLARE composer and the ALLOCATE-PID rehydration branch.
3217 // Populated with a bare non-empty name: `derive_identity`
3218 // dispatches on `Some(non_empty)` and stamps
3219 // `name_override: true` on the resulting [`Identity`], with
3220 // the resulting `.name` equal to the raw slot value.
3221 let p = process_with_declared_name_override(Some("gateway-primary"));
3222 let id = crate::identity::derive_identity(&p.spec, p.declared_name_override());
3223 assert!(id.name_override);
3224 assert_eq!(id.name, "gateway-primary");
3225 // Empty slot: `derive_identity` dispatches on `None` and
3226 // stamps `name_override: false` on the resulting [`Identity`],
3227 // with the resulting `.name` derived from the content hash
3228 // (NOT equal to any operator-authored slot value).
3229 let p = process_with_declared_name_override(None);
3230 let id = crate::identity::derive_identity(&p.spec, p.declared_name_override());
3231 assert!(!id.name_override);
3232 }
3233
3234 // ─── Process::observed_flux_resources substrate pins ───────────────
3235 //
3236 // Pins the borrow-form status-projection primitive that owns the
3237 // 5-line `.status.as_ref().map(|s| s.flux_resources.clone())
3238 // .unwrap_or_default()` chain the two hand-authored
3239 // `tatara-reconciler::phase_machine` sites (`handle_running` +
3240 // `handle_attested`) restated by hand pre-lift. Fail-before-pass-
3241 // after granularity: a regression that widened the missing-`status`
3242 // corner, dropped the slot, or drifted the borrow discipline
3243 // surfaces here rather than as silent operator-facing skew between
3244 // the VERIFY-phase readiness probe and the ATTEST-heartbeat drift
3245 // detector.
3246
3247 fn sample_flux_ref(name: &str) -> FluxResourceRef {
3248 // Distinct slot values so a swap between adjacent tuple
3249 // positions surfaces as an equality failure at the assertion
3250 // site — a slot-inversion regression cannot masquerade as
3251 // identity by accident. Peer to the sibling
3252 // `tatara_process::status::tests::sample_flux_ref` discipline
3253 // on the fetch-coords axis.
3254 FluxResourceRef {
3255 api_version: "kustomize.toolkit.fluxcd.io/v1".to_string(),
3256 kind: "Kustomization".to_string(),
3257 name: name.to_string(),
3258 namespace: "flux-system".to_string(),
3259 ready: false,
3260 message: None,
3261 last_check: None,
3262 }
3263 }
3264
3265 fn process_with_flux_resources(refs: Vec<FluxResourceRef>) -> Process {
3266 let mut p = Process::new("api-gateway", empty_spec());
3267 p.metadata.namespace = Some("prod".into());
3268 let mut status = ProcessStatus::default();
3269 status.flux_resources = refs;
3270 p.status = Some(status);
3271 p
3272 }
3273
3274 #[test]
3275 fn observed_flux_resources_returns_empty_slice_when_status_is_none() {
3276 // Missing-`status` corner pin: the primitive collapses the
3277 // no-status case to `&[]` so downstream `.is_empty()` /
3278 // `.len()` / iteration behave identically on a `Process`
3279 // whose status field is `None` and on one whose status
3280 // carries an empty `flux_resources` slot. Matches the
3281 // pre-lift `.unwrap_or_default()`'s empty-`Vec` corner
3282 // byte-identically at every reconciler consumer's downstream
3283 // shape.
3284 let mut p = Process::new("api", empty_spec());
3285 p.status = None;
3286 assert!(p.observed_flux_resources().is_empty());
3287 assert_eq!(p.observed_flux_resources().len(), 0);
3288 }
3289
3290 #[test]
3291 fn observed_flux_resources_returns_empty_slice_when_flux_resources_is_empty() {
3292 // Zero-refs-under-populated-status corner pin: the primitive
3293 // returns an empty slice, matching the missing-`status`
3294 // corner byte-identically. A regression that treated the two
3295 // corners differently (a `None`-vs-empty signal that
3296 // downstream consumers could grep on) would silently promote
3297 // an internal representation detail (whether the reconciler
3298 // has ever written a status subresource) into observable
3299 // behavior.
3300 let p = process_with_flux_resources(vec![]);
3301 assert!(p.observed_flux_resources().is_empty());
3302 assert_eq!(p.observed_flux_resources().len(), 0);
3303 }
3304
3305 #[test]
3306 fn observed_flux_resources_returns_slice_of_persisted_vec() {
3307 // Happy-path pin: with a populated `status.flux_resources`
3308 // slot, the primitive returns a borrowed slice whose length
3309 // and per-element identity match the persisted vector. A
3310 // regression that filtered / reshaped / deduplicated the
3311 // slice would surface here rather than as silent skew at the
3312 // downstream fetch consumers.
3313 let refs = vec![
3314 sample_flux_ref("observability-stack"),
3315 sample_flux_ref("gateway"),
3316 ];
3317 let p = process_with_flux_resources(refs.clone());
3318 let observed = p.observed_flux_resources();
3319 assert_eq!(observed.len(), 2);
3320 assert_eq!(observed[0].name, "observability-stack");
3321 assert_eq!(observed[1].name, "gateway");
3322 }
3323
3324 #[test]
3325 fn observed_flux_resources_is_a_zero_copy_borrow_projection() {
3326 // Borrow-discipline pin: the returned slice borrows the
3327 // persisted `Vec<FluxResourceRef>` in place — NOT a fresh
3328 // allocation or a clone. A regression that switched the
3329 // projection to owned refs (via `.clone()` or `.to_vec()`)
3330 // would defeat the zero-copy contract the lift's primary
3331 // strict-widening delivers (the pre-lift 5-line chain
3332 // eagerly cloned the whole vector per reconcile pass; the
3333 // post-lift primitive borrows). Peer to the sibling
3334 // `flux_resource_ref_fetch_coords_returns_borrows_of_owned_slots`
3335 // pin on the per-ref borrow-projection axis.
3336 let refs = vec![sample_flux_ref("observability-stack")];
3337 let p = process_with_flux_resources(refs);
3338 let observed = p.observed_flux_resources();
3339 let persisted = &p.status.as_ref().unwrap().flux_resources;
3340 assert!(std::ptr::eq(observed.as_ptr(), persisted.as_ptr()));
3341 }
3342
3343 #[test]
3344 fn observed_flux_resources_is_a_pure_projection() {
3345 // Purity pin: calling the projection twice on the same
3346 // `Process` returns byte-identical slices (same pointer,
3347 // same length). A regression that introduced state — a
3348 // lazy-cached slice materialized on first call, a
3349 // normalization step that ran once and cached — would
3350 // surface here rather than as silent drift between the
3351 // VERIFY-phase and ATTEST-heartbeat consumers on the SAME
3352 // `Process` within one reconcile pass.
3353 let refs = vec![sample_flux_ref("observability-stack")];
3354 let p = process_with_flux_resources(refs);
3355 let a = p.observed_flux_resources();
3356 let b = p.observed_flux_resources();
3357 assert!(std::ptr::eq(a.as_ptr(), b.as_ptr()));
3358 assert_eq!(a.len(), b.len());
3359 }
3360
3361 #[test]
3362 fn observed_flux_resources_matches_pre_lift_reconciler_chain_shape() {
3363 // Byte-identical parity pin between the borrow-form primitive
3364 // here and the pre-lift `tatara-reconciler::phase_machine`
3365 // 5-line chain shape. Sweeps every corner every callsite
3366 // plausibly encounters (missing status, empty flux_resources,
3367 // populated flux_resources with one ref, populated with
3368 // multiple refs). A regression that inserted a normalization
3369 // step at the primitive the pre-lift chain does NOT apply —
3370 // or vice versa — surfaces here rather than as silent drift
3371 // between the pre-lift consumer sites and the ONE substrate
3372 // owner they now route through. Peer to
3373 // `coordinates_or_none_matches_pre_lift_reconciler_helper_shape`
3374 // on the metadata axis's borrow-form primitive.
3375 // `FluxResourceRef` does not derive `PartialEq` — the parity
3376 // check walks the per-ref fetch-coords tuple (the same 4-slot
3377 // borrow projection every downstream fetch consumer routes
3378 // through) so a regression that reshaped ANY slot at ANY
3379 // index surfaces here through the sibling
3380 // `FluxResourceRef::fetch_coords` typed projection.
3381 fn pre_lift(p: &Process) -> Vec<FluxResourceRef> {
3382 p.status
3383 .as_ref()
3384 .map(|s| s.flux_resources.clone())
3385 .unwrap_or_default()
3386 }
3387 fn coord_shape(refs: &[FluxResourceRef]) -> Vec<(String, String, String, String)> {
3388 refs.iter()
3389 .map(|r| {
3390 let (ns, av, kind, name) = r.fetch_coords();
3391 (
3392 ns.to_string(),
3393 av.to_string(),
3394 kind.to_string(),
3395 name.to_string(),
3396 )
3397 })
3398 .collect()
3399 }
3400 // Missing status.
3401 let mut p = Process::new("api", empty_spec());
3402 p.status = None;
3403 assert_eq!(
3404 coord_shape(p.observed_flux_resources()),
3405 coord_shape(&pre_lift(&p))
3406 );
3407 // Populated status, empty slot.
3408 let p = process_with_flux_resources(vec![]);
3409 assert_eq!(
3410 coord_shape(p.observed_flux_resources()),
3411 coord_shape(&pre_lift(&p))
3412 );
3413 // Populated status, one ref.
3414 let p = process_with_flux_resources(vec![sample_flux_ref("obs")]);
3415 assert_eq!(
3416 coord_shape(p.observed_flux_resources()),
3417 coord_shape(&pre_lift(&p))
3418 );
3419 // Populated status, multiple refs.
3420 let p = process_with_flux_resources(vec![
3421 sample_flux_ref("obs"),
3422 sample_flux_ref("gw"),
3423 sample_flux_ref("api"),
3424 ]);
3425 assert_eq!(
3426 coord_shape(p.observed_flux_resources()),
3427 coord_shape(&pre_lift(&p))
3428 );
3429 }
3430
3431 #[test]
3432 fn observed_flux_resources_missing_status_and_empty_slot_collapse_to_the_same_slice_shape() {
3433 // Cross-corner coherence pin: the missing-`status` corner and
3434 // the populated-empty-slot corner return slices whose
3435 // `.is_empty()` / `.len()` observations are IDENTICAL. A
3436 // regression that promoted the missing-`status` corner to
3437 // returning `None` (via a signature change) — or that widened
3438 // the empty-slot corner to a synthetic single-element slice
3439 // — would surface here rather than as silent operator-facing
3440 // divergence between a never-status-written Process and a
3441 // status-emptied Process.
3442 let mut p_no_status = Process::new("api", empty_spec());
3443 p_no_status.status = None;
3444 let p_empty_status = process_with_flux_resources(vec![]);
3445 assert_eq!(
3446 p_no_status.observed_flux_resources().len(),
3447 p_empty_status.observed_flux_resources().len()
3448 );
3449 assert_eq!(
3450 p_no_status.observed_flux_resources().is_empty(),
3451 p_empty_status.observed_flux_resources().is_empty()
3452 );
3453 }
3454
3455 #[test]
3456 fn observed_flux_resources_slice_preserves_persisted_ordering() {
3457 // Ordering-preservation pin: the borrowed slice preserves
3458 // the exact insertion order of the persisted vector — no
3459 // sort, no dedup, no reshape. A regression that inserted a
3460 // sort or reordering would silently misroute per-ref
3461 // observations at the downstream VERIFY-phase / ATTEST-
3462 // heartbeat consumers, both of which walk the slice
3463 // positionally and correlate the position to the observed
3464 // readiness.
3465 let refs = vec![
3466 sample_flux_ref("z-last"),
3467 sample_flux_ref("a-first"),
3468 sample_flux_ref("m-middle"),
3469 ];
3470 let p = process_with_flux_resources(refs);
3471 let observed = p.observed_flux_resources();
3472 assert_eq!(observed[0].name, "z-last");
3473 assert_eq!(observed[1].name, "a-first");
3474 assert_eq!(observed[2].name, "m-middle");
3475 }
3476
3477 // ─── Process::observed_pid substrate pins ─────────────────────────
3478 //
3479 // Pins the borrow-form status-projection primitive on the PID axis
3480 // that owns the 3-line `.status.as_ref().and_then(|s| s.pid.clone())`
3481 // chain the two hand-authored `tatara-reconciler::phase_machine`
3482 // sites (`handle_forking` ALLOCATE-PID gate + `handle_exiting`
3483 // SIGTERM cascade) restated by hand pre-lift. Peer to the sibling
3484 // `observed_flux_resources_*` pin family on the flux-resources
3485 // axis; both compose the missing-`status` fallback + borrow-form
3486 // return-shape skeleton on distinct `ProcessStatus` slots. Fail-
3487 // before-pass-after granularity: `observed_pid` did not exist
3488 // pre-lift, so any test invoking it fails to compile pre-lift and
3489 // passes post-lift.
3490
3491 fn process_with_pid(pid: Option<&str>) -> Process {
3492 let mut p = Process::new("api-gateway", empty_spec());
3493 p.metadata.namespace = Some("prod".into());
3494 let mut status = ProcessStatus::default();
3495 status.pid = pid.map(str::to_string);
3496 p.status = Some(status);
3497 p
3498 }
3499
3500 #[test]
3501 fn observed_pid_returns_none_when_status_is_none() {
3502 // Missing-`status` corner pin: the primitive collapses the
3503 // no-status case to `None` so downstream `.is_some()` /
3504 // `if let Some(_)` / `.map(...)` behave identically on a
3505 // `Process` whose status field is `None` and on one whose
3506 // status carries an unpopulated `pid` slot. Matches the
3507 // pre-lift `.and_then(...)` chain's `None` byte-identically
3508 // at every reconciler consumer's downstream shape.
3509 let mut p = Process::new("api", empty_spec());
3510 p.status = None;
3511 assert!(p.observed_pid().is_none());
3512 }
3513
3514 #[test]
3515 fn observed_pid_returns_none_when_pid_slot_is_none() {
3516 // Empty-slot-under-populated-status corner pin: the
3517 // primitive returns `None`, matching the missing-`status`
3518 // corner byte-identically. A regression that treated the
3519 // two corners differently (a `None`-vs-`Some("")` signal
3520 // that downstream consumers could grep on) would silently
3521 // promote an internal representation detail (whether the
3522 // reconciler has ever written a status subresource) into
3523 // observable behavior at the ALLOCATE-PID gate.
3524 let p = process_with_pid(None);
3525 assert!(p.observed_pid().is_none());
3526 }
3527
3528 #[test]
3529 fn observed_pid_returns_borrowed_str_when_pid_slot_is_populated() {
3530 // Happy-path pin: with a populated `status.pid` slot, the
3531 // primitive returns a borrowed `&str` whose contents match
3532 // the persisted `String`. A regression that filtered /
3533 // reshaped / canonicalized the string would surface here
3534 // rather than as silent skew at the downstream cascade
3535 // comparator's `.as_deref() == Some(...)` equality check.
3536 let p = process_with_pid(Some("seph.1.7"));
3537 assert_eq!(p.observed_pid(), Some("seph.1.7"));
3538 }
3539
3540 #[test]
3541 fn observed_pid_is_a_zero_copy_borrow_projection() {
3542 // Borrow-discipline pin: the returned `&str` borrows the
3543 // persisted `String`'s underlying byte buffer in place —
3544 // NOT a fresh allocation or a clone. A regression that
3545 // switched the projection to an owned `String` (via
3546 // `.clone()` or `.to_owned()`) would defeat the zero-copy
3547 // contract the lift's primary strict-widening delivers
3548 // (the pre-lift 3-line chain eagerly cloned the `String`
3549 // per reconcile pass at BOTH call sites even though the
3550 // ALLOCATE-PID gate immediately dropped the clone and the
3551 // SIGTERM cascade only re-borrowed it via `.as_str()`; the
3552 // post-lift primitive borrows). Peer to the sibling
3553 // `observed_flux_resources_is_a_zero_copy_borrow_projection`
3554 // pin on the flux-resources borrow-projection axis.
3555 let p = process_with_pid(Some("seph.1.7"));
3556 let observed = p.observed_pid().expect("populated slot");
3557 let persisted = p.status.as_ref().unwrap().pid.as_ref().unwrap();
3558 assert!(std::ptr::eq(observed.as_ptr(), persisted.as_ptr()));
3559 }
3560
3561 #[test]
3562 fn observed_pid_is_a_pure_projection() {
3563 // Purity pin: calling the projection twice on the same
3564 // `Process` returns byte-identical `&str`s (same pointer,
3565 // same length). A regression that introduced state — a
3566 // lazy-cached slice materialized on first call, a
3567 // normalization step that ran once and cached — would
3568 // surface here rather than as silent drift between the
3569 // ALLOCATE-PID gate and the SIGTERM cascade on the SAME
3570 // `Process` within one reconcile pass.
3571 let p = process_with_pid(Some("seph.1.7"));
3572 let a = p.observed_pid().expect("populated slot");
3573 let b = p.observed_pid().expect("populated slot");
3574 assert!(std::ptr::eq(a.as_ptr(), b.as_ptr()));
3575 assert_eq!(a.len(), b.len());
3576 }
3577
3578 #[test]
3579 fn observed_pid_matches_pre_lift_reconciler_chain_shape() {
3580 // Byte-identical parity pin between the borrow-form
3581 // primitive here and the pre-lift `tatara-reconciler
3582 // ::phase_machine` 3-line chain shape. Sweeps every corner
3583 // every callsite plausibly encounters (missing status,
3584 // empty pid slot, populated pid slot). A regression that
3585 // inserted a normalization step at the primitive the pre-
3586 // lift chain does NOT apply — or vice versa — surfaces
3587 // here rather than as silent drift between the pre-lift
3588 // consumer sites and the ONE substrate owner they now
3589 // route through. Peer to
3590 // `observed_flux_resources_matches_pre_lift_reconciler_chain_shape`
3591 // on the flux-resources axis's borrow-form primitive.
3592 fn pre_lift(p: &Process) -> Option<String> {
3593 p.status.as_ref().and_then(|s| s.pid.clone())
3594 }
3595 // Missing status.
3596 let mut p = Process::new("api", empty_spec());
3597 p.status = None;
3598 assert_eq!(p.observed_pid().map(str::to_string), pre_lift(&p));
3599 // Populated status, empty pid slot.
3600 let p = process_with_pid(None);
3601 assert_eq!(p.observed_pid().map(str::to_string), pre_lift(&p));
3602 // Populated status, populated pid slot.
3603 let p = process_with_pid(Some("seph.1.7"));
3604 assert_eq!(p.observed_pid().map(str::to_string), pre_lift(&p));
3605 }
3606
3607 #[test]
3608 fn observed_pid_missing_status_and_empty_slot_collapse_to_the_same_option_shape() {
3609 // Cross-corner coherence pin: the missing-`status` corner
3610 // and the populated-empty-slot corner return `Option`s whose
3611 // `.is_none()` observations are IDENTICAL. A regression
3612 // that promoted the missing-`status` corner to returning a
3613 // typed error (via a signature change to `Result<_, _>`) —
3614 // or that widened the empty-slot corner to a synthetic
3615 // `Some("")` — would surface here rather than as silent
3616 // operator-facing divergence between a never-status-
3617 // written Process and a status-emptied Process on the
3618 // ALLOCATE-PID gate.
3619 let mut p_no_status = Process::new("api", empty_spec());
3620 p_no_status.status = None;
3621 let p_empty_slot = process_with_pid(None);
3622 assert_eq!(
3623 p_no_status.observed_pid().is_none(),
3624 p_empty_slot.observed_pid().is_none()
3625 );
3626 assert_eq!(
3627 p_no_status.observed_pid().is_some(),
3628 p_empty_slot.observed_pid().is_some()
3629 );
3630 }
3631
3632 #[test]
3633 fn observed_pid_preserves_hierarchical_pid_format() {
3634 // Format-preservation pin: the hierarchical PID path
3635 // (dotted-segment form `seph.1.7`, matching the ported
3636 // `convergence-controller/src/identity.rs` scheme) reaches
3637 // the caller with segments and separators byte-identical
3638 // to the persisted `String`. A regression that inserted a
3639 // canonicalization pass (a segment-count validator, a
3640 // separator swap `.` → `/`, a leading/trailing whitespace
3641 // trim) would silently misroute the SIGTERM cascade's
3642 // `spec.identity.parent == Some(pid)` comparator against
3643 // children whose `parent` field was authored in the ported
3644 // scheme's exact form.
3645 for pid in ["seph", "seph.1", "seph.1.7", "seph.1.7.42"] {
3646 let p = process_with_pid(Some(pid));
3647 assert_eq!(p.observed_pid(), Some(pid));
3648 }
3649 }
3650
3651 // ─── Process::observed_attestation substrate pins ─────────────────
3652 //
3653 // Pins the borrow-form status-projection primitive on the
3654 // attestation-chain axis that owns the 3-line
3655 // `.status.as_ref().and_then(|s| s.attestation.as_ref())` chain
3656 // the two hand-authored `tatara-reconciler` sites
3657 // (`phase_machine::advance_to_attested` ATTEST composer +
3658 // `render::render_export_jobs` export-Job builder) restated by
3659 // hand pre-lift. Peer to the sibling `observed_pid_*` +
3660 // `observed_flux_resources_*` pin families; all three compose
3661 // the missing-`status` fallback + borrow-form return-shape
3662 // skeleton on distinct `ProcessStatus` slots. Fail-before-pass-
3663 // after granularity: `observed_attestation` did not exist
3664 // pre-lift, so any test invoking it fails to compile pre-lift
3665 // and passes post-lift.
3666
3667 fn sample_attestation(artifact: &str, intent: &str) -> ProcessAttestation {
3668 // Distinct pillar strings so a regression that swapped the
3669 // artifact / intent pillars silently surfaces as an
3670 // equality failure at the composed-root parity pin.
3671 ProcessAttestation::initial(artifact.to_string(), None, intent.to_string())
3672 }
3673
3674 fn process_with_attestation(attestation: Option<ProcessAttestation>) -> Process {
3675 let mut p = Process::new("api-gateway", empty_spec());
3676 p.metadata.namespace = Some("prod".into());
3677 let mut status = ProcessStatus::default();
3678 status.attestation = attestation;
3679 p.status = Some(status);
3680 p
3681 }
3682
3683 #[test]
3684 fn observed_attestation_returns_none_when_status_is_none() {
3685 // Missing-`status` corner pin: the primitive collapses the
3686 // no-status case to `None` so downstream `.is_some()` /
3687 // `if let Some(_)` / `.map(...)` behave identically on a
3688 // `Process` whose status field is `None` and on one whose
3689 // status carries an unpopulated `attestation` slot.
3690 // Matches the pre-lift `.and_then(...)` chain's `None`
3691 // byte-identically at every reconciler consumer's
3692 // downstream shape.
3693 let mut p = Process::new("api", empty_spec());
3694 p.status = None;
3695 assert!(p.observed_attestation().is_none());
3696 }
3697
3698 #[test]
3699 fn observed_attestation_returns_none_when_attestation_slot_is_none() {
3700 // Empty-slot-under-populated-status corner pin: the
3701 // primitive returns `None`, matching the missing-`status`
3702 // corner byte-identically. A regression that treated the
3703 // two corners differently (a `None`-vs-`Some(_)` signal
3704 // that downstream consumers could grep on) would silently
3705 // promote an internal representation detail (whether the
3706 // reconciler has ever written a status subresource) into
3707 // observable behavior at the ATTEST composer's
3708 // seed-vs-chain branch.
3709 let p = process_with_attestation(None);
3710 assert!(p.observed_attestation().is_none());
3711 }
3712
3713 #[test]
3714 fn observed_attestation_returns_borrow_when_slot_is_populated() {
3715 // Happy-path pin: with a populated `status.attestation`
3716 // slot, the primitive returns a borrowed
3717 // `&ProcessAttestation` whose fields match the persisted
3718 // record. A regression that filtered / reshaped /
3719 // canonicalized the record would surface here rather than
3720 // as silent skew at the downstream `prior.next(pillars)`
3721 // chain composer + the ephemeral-export receipt's
3722 // `previous_root` linker.
3723 let att = sample_attestation("art-1", "int-1");
3724 let composed_root = att.composed_root.clone();
3725 let p = process_with_attestation(Some(att));
3726 let observed = p.observed_attestation().expect("populated slot");
3727 assert_eq!(observed.artifact_hash, "art-1");
3728 assert_eq!(observed.intent_hash, "int-1");
3729 assert_eq!(observed.composed_root, composed_root);
3730 assert_eq!(observed.generation, 0);
3731 assert!(observed.previous_root.is_none());
3732 }
3733
3734 #[test]
3735 fn observed_attestation_is_a_zero_copy_borrow_projection() {
3736 // Borrow-discipline pin: the returned reference points at
3737 // the persisted `ProcessAttestation` in place — NOT a fresh
3738 // allocation or a clone. A regression that switched the
3739 // projection to an owned `ProcessAttestation` (via
3740 // `.clone()`) would defeat the zero-copy contract the
3741 // lift's primary strict-widening delivers (the pre-lift
3742 // 3-line chain returned a borrow, but the export-Job
3743 // builder then cloned `composed_root` off it; the post-
3744 // lift primitive preserves the borrow all the way to the
3745 // consumer's own cloning choice). Peer to the sibling
3746 // `observed_pid_is_a_zero_copy_borrow_projection` +
3747 // `observed_flux_resources_is_a_zero_copy_borrow_projection`
3748 // pins on the PID + flux-resources borrow-projection axes.
3749 let att = sample_attestation("art-1", "int-1");
3750 let p = process_with_attestation(Some(att));
3751 let observed = p.observed_attestation().expect("populated slot") as *const _;
3752 let persisted = p.status.as_ref().unwrap().attestation.as_ref().unwrap() as *const _;
3753 assert!(std::ptr::eq(observed, persisted));
3754 }
3755
3756 #[test]
3757 fn observed_attestation_is_a_pure_projection() {
3758 // Purity pin: calling the projection twice on the same
3759 // `Process` returns byte-identical borrows (same pointer).
3760 // A regression that introduced state — a lazy-cached
3761 // reference materialized on first call, a normalization
3762 // step that ran once and cached — would surface here
3763 // rather than as silent drift between the ATTEST composer
3764 // and the ephemeral-export receipt chain on the SAME
3765 // `Process` within one reconcile pass.
3766 let att = sample_attestation("art-1", "int-1");
3767 let p = process_with_attestation(Some(att));
3768 let a = p.observed_attestation().expect("populated slot") as *const _;
3769 let b = p.observed_attestation().expect("populated slot") as *const _;
3770 assert!(std::ptr::eq(a, b));
3771 }
3772
3773 #[test]
3774 fn observed_attestation_matches_pre_lift_reconciler_chain_shape() {
3775 // Byte-identical parity pin between the borrow-form
3776 // primitive here and the pre-lift `tatara-reconciler`
3777 // 3-line chain shape. Sweeps every corner every callsite
3778 // plausibly encounters (missing status, empty attestation
3779 // slot, populated attestation slot). A regression that
3780 // inserted a normalization step at the primitive the pre-
3781 // lift chain does NOT apply — or vice versa — surfaces
3782 // here rather than as silent drift between the pre-lift
3783 // consumer sites and the ONE substrate owner they now
3784 // route through. Peer to
3785 // `observed_pid_matches_pre_lift_reconciler_chain_shape` +
3786 // `observed_flux_resources_matches_pre_lift_reconciler_chain_shape`
3787 // on the PID + flux-resources axes.
3788 // `ProcessAttestation` does not derive `PartialEq` — the
3789 // parity check walks the `composed_root` field (the
3790 // byte-string every downstream consumer keys off) so a
3791 // regression that reshaped the record without touching
3792 // the composed-root observation surfaces here through
3793 // the receipt-chain projection.
3794 fn pre_lift(p: &Process) -> Option<String> {
3795 p.status
3796 .as_ref()
3797 .and_then(|s| s.attestation.as_ref())
3798 .map(|a| a.composed_root.clone())
3799 }
3800 // Missing status.
3801 let mut p = Process::new("api", empty_spec());
3802 p.status = None;
3803 assert_eq!(
3804 p.observed_attestation().map(|a| a.composed_root.clone()),
3805 pre_lift(&p)
3806 );
3807 // Populated status, empty attestation slot.
3808 let p = process_with_attestation(None);
3809 assert_eq!(
3810 p.observed_attestation().map(|a| a.composed_root.clone()),
3811 pre_lift(&p)
3812 );
3813 // Populated status, populated attestation slot.
3814 let p = process_with_attestation(Some(sample_attestation("art-1", "int-1")));
3815 assert_eq!(
3816 p.observed_attestation().map(|a| a.composed_root.clone()),
3817 pre_lift(&p)
3818 );
3819 }
3820
3821 #[test]
3822 fn observed_attestation_missing_status_and_empty_slot_collapse_to_the_same_option_shape() {
3823 // Cross-corner coherence pin: the missing-`status` corner
3824 // and the populated-empty-slot corner return `Option`s
3825 // whose `.is_none()` observations are IDENTICAL. A
3826 // regression that promoted the missing-`status` corner to
3827 // returning a typed error (via a signature change to
3828 // `Result<_, _>`) — or that widened the empty-slot corner
3829 // to a synthetic `Some(default_attestation)` — would
3830 // surface here rather than as silent operator-facing
3831 // divergence between a never-status-written Process and
3832 // an attestation-emptied Process on the ATTEST composer's
3833 // seed-vs-chain branch.
3834 let mut p_no_status = Process::new("api", empty_spec());
3835 p_no_status.status = None;
3836 let p_empty_slot = process_with_attestation(None);
3837 assert_eq!(
3838 p_no_status.observed_attestation().is_none(),
3839 p_empty_slot.observed_attestation().is_none()
3840 );
3841 assert_eq!(
3842 p_no_status.observed_attestation().is_some(),
3843 p_empty_slot.observed_attestation().is_some()
3844 );
3845 }
3846
3847 #[test]
3848 fn observed_attestation_preserves_chain_generation_field() {
3849 // Generation-preservation pin: a chained attestation
3850 // (`prior.next(...)` at generation N ≥ 1 with a
3851 // `previous_root` linked to `prior.composed_root`) reaches
3852 // the caller with its `generation` counter + `previous_root`
3853 // link byte-identical to the persisted record. The pre-lift
3854 // ATTEST composer discriminated exactly on this borrow's
3855 // `Some(prior)` vs `None` arm; a regression that dropped
3856 // the chain's `generation` counter (say, by folding
3857 // `next(...)` into a fresh `initial(...)` on every
3858 // reconcile pass) would silently reset every chain and
3859 // orphan every downstream `previous_root` link, but that
3860 // drift is invisible to a Process CRD reader who only
3861 // observes the LATEST composed_root.
3862 let prior = sample_attestation("art-0", "int-0");
3863 let chained = prior.next("art-1".to_string(), None, "int-1".to_string());
3864 let expected_generation = chained.generation;
3865 let expected_previous = chained.previous_root.clone();
3866 let p = process_with_attestation(Some(chained));
3867 let observed = p.observed_attestation().expect("populated slot");
3868 assert_eq!(observed.generation, expected_generation);
3869 assert_eq!(observed.generation, 1);
3870 assert_eq!(observed.previous_root, expected_previous);
3871 assert_eq!(
3872 observed.previous_root.as_deref(),
3873 Some(prior.composed_root.as_str())
3874 );
3875 }
3876
3877 // ─── Process::observed_identity substrate pins ────────────────────
3878 //
3879 // The borrow-form status-projection primitive on the resolved-
3880 // identity axis. Collapses the paired 3-line `.status.as_ref()
3881 // .and_then(|s| s.identity.<clone|as_ref>())` chain every
3882 // consumer in `tatara-reconciler` restated by hand pre-lift at
3883 // TWO sites (`phase_machine::handle_forking` seed +
3884 // `ssapply::inject_annotations` content-hash annotation
3885 // composer). Peer to the sibling `observed_pid_*` +
3886 // `observed_attestation_*` + `observed_flux_resources_*` pin
3887 // families; all four compose the same missing-`status` fallback
3888 // + borrow-form return-shape skeleton on distinct
3889 // `ProcessStatus` slots. Each pin fails-before-pass-after
3890 // granularity: `observed_identity` did not exist pre-lift, so
3891 // any test invoking it fails to compile pre-lift and passes
3892 // post-lift.
3893
3894 fn sample_identity(name: &str) -> Identity {
3895 // Distinct name + content_hash + override flag so a
3896 // regression that reshaped one slot surfaces at the
3897 // populated-slot pin's field-equality check without
3898 // aliasing the sibling slots.
3899 Identity {
3900 name: name.to_string(),
3901 content_hash: "a".repeat(26),
3902 name_override: true,
3903 }
3904 }
3905
3906 fn process_with_identity(identity: Option<Identity>) -> Process {
3907 let mut p = Process::new("api-gateway", empty_spec());
3908 p.metadata.namespace = Some("prod".into());
3909 let mut status = ProcessStatus::default();
3910 status.identity = identity;
3911 p.status = Some(status);
3912 p
3913 }
3914
3915 #[test]
3916 fn observed_identity_returns_none_when_status_is_none() {
3917 // Missing-`status` corner pin: the primitive collapses the
3918 // no-status case to `None` so downstream `.is_some()` /
3919 // `if let Some(_)` / `.cloned().unwrap_or_else(...)` behave
3920 // identically on a `Process` whose status field is `None`
3921 // and on one whose status carries an unpopulated `identity`
3922 // slot. Matches the pre-lift `.and_then(...)` chain's `None`
3923 // byte-identically at every reconciler consumer's
3924 // downstream shape.
3925 let mut p = Process::new("api", empty_spec());
3926 p.status = None;
3927 assert!(p.observed_identity().is_none());
3928 }
3929
3930 #[test]
3931 fn observed_identity_returns_none_when_identity_slot_is_none() {
3932 // Empty-slot-under-populated-status corner pin: the
3933 // primitive returns `None`, matching the missing-`status`
3934 // corner byte-identically. A regression that treated the
3935 // two corners differently (a `None`-vs-`Some(_)` signal
3936 // that downstream consumers could grep on) would silently
3937 // promote an internal representation detail (whether the
3938 // reconciler has ever written a status subresource) into
3939 // observable behavior at the FORK-time `derive_identity`
3940 // fallback branch.
3941 let p = process_with_identity(None);
3942 assert!(p.observed_identity().is_none());
3943 }
3944
3945 #[test]
3946 fn observed_identity_returns_borrow_when_slot_is_populated() {
3947 // Happy-path pin: with a populated `status.identity` slot,
3948 // the primitive returns a borrowed `&Identity` whose fields
3949 // match the persisted record. A regression that filtered /
3950 // reshaped / canonicalized the record would surface here
3951 // rather than as silent skew at the FORK-time seed's
3952 // `.cloned().unwrap_or_else(derive_identity)` composition
3953 // + the SSA-time content-hash annotation stamp on the SAME
3954 // Process.
3955 let id = sample_identity("seph");
3956 let expected = id.clone();
3957 let p = process_with_identity(Some(id));
3958 let observed = p.observed_identity().expect("populated slot");
3959 assert_eq!(observed, &expected);
3960 assert_eq!(observed.name, "seph");
3961 assert_eq!(observed.content_hash, "a".repeat(26));
3962 assert!(observed.name_override);
3963 }
3964
3965 #[test]
3966 fn observed_identity_is_a_zero_copy_borrow_projection() {
3967 // Borrow-discipline pin: the returned reference points at
3968 // the persisted `Identity` in place — NOT a fresh
3969 // allocation or a clone. A regression that switched the
3970 // projection to an owned `Identity` (via `.clone()`) would
3971 // defeat the zero-copy contract the lift's primary strict-
3972 // widening delivers (the SSA-time consumer never clones the
3973 // whole `Identity`, only the `content_hash` field it stamps
3974 // onto the annotation map, so the borrow-form return
3975 // shape's happy-path allocation count is exactly ZERO).
3976 // Peer to the sibling
3977 // `observed_attestation_is_a_zero_copy_borrow_projection`
3978 // + `observed_pid_is_a_zero_copy_borrow_projection` +
3979 // `observed_flux_resources_is_a_zero_copy_borrow_projection`
3980 // pins on the attestation-chain + PID + flux-resources
3981 // borrow-projection axes.
3982 let id = sample_identity("seph");
3983 let p = process_with_identity(Some(id));
3984 let observed = p.observed_identity().expect("populated slot") as *const _;
3985 let persisted = p.status.as_ref().unwrap().identity.as_ref().unwrap() as *const _;
3986 assert!(std::ptr::eq(observed, persisted));
3987 }
3988
3989 #[test]
3990 fn observed_identity_is_a_pure_projection() {
3991 // Purity pin: calling the projection twice on the same
3992 // `Process` returns byte-identical borrows (same pointer).
3993 // A regression that introduced state — a lazy-cached
3994 // reference materialized on first call, a normalization
3995 // step that ran once and cached — would surface here
3996 // rather than as silent drift between the FORK-time
3997 // identity seed and the SSA-time content-hash annotation
3998 // stamp on the SAME `Process` within one reconcile pass.
3999 let p = process_with_identity(Some(sample_identity("seph")));
4000 let a = p.observed_identity().expect("populated slot") as *const _;
4001 let b = p.observed_identity().expect("populated slot") as *const _;
4002 assert!(std::ptr::eq(a, b));
4003 }
4004
4005 #[test]
4006 fn observed_identity_matches_pre_lift_reconciler_chain_shape() {
4007 // Byte-identical parity pin between the borrow-form
4008 // primitive here and the pre-lift `tatara-reconciler`
4009 // 3-line chain shape. Sweeps every corner every callsite
4010 // plausibly encounters (missing status, empty identity
4011 // slot, populated identity slot). A regression that
4012 // inserted a normalization step at the primitive the pre-
4013 // lift chain does NOT apply — or vice versa — surfaces
4014 // here rather than as silent drift between the pre-lift
4015 // consumer sites and the ONE substrate owner they now
4016 // route through. Peer to
4017 // `observed_attestation_matches_pre_lift_reconciler_chain_shape`
4018 // + `observed_pid_matches_pre_lift_reconciler_chain_shape`
4019 // + `observed_flux_resources_matches_pre_lift_reconciler_chain_shape`
4020 // on the attestation-chain + PID + flux-resources axes.
4021 fn pre_lift(p: &Process) -> Option<Identity> {
4022 p.status.as_ref().and_then(|s| s.identity.clone())
4023 }
4024 // Missing status.
4025 let mut p = Process::new("api", empty_spec());
4026 p.status = None;
4027 assert_eq!(p.observed_identity().cloned(), pre_lift(&p));
4028 // Populated status, empty identity slot.
4029 let p = process_with_identity(None);
4030 assert_eq!(p.observed_identity().cloned(), pre_lift(&p));
4031 // Populated status, populated identity slot.
4032 let p = process_with_identity(Some(sample_identity("seph")));
4033 assert_eq!(p.observed_identity().cloned(), pre_lift(&p));
4034 }
4035
4036 #[test]
4037 fn observed_identity_missing_status_and_empty_slot_collapse_to_the_same_option_shape() {
4038 // Cross-corner coherence pin: the missing-`status` corner
4039 // and the populated-empty-slot corner return `Option`s
4040 // whose `.is_none()` observations are IDENTICAL. A
4041 // regression that promoted the missing-`status` corner to
4042 // returning a typed error (via a signature change to
4043 // `Result<_, _>`) — or that widened the empty-slot corner
4044 // to a synthetic `Some(derive_identity(default_spec))` —
4045 // would surface here rather than as silent operator-facing
4046 // divergence between a never-status-written Process and an
4047 // identity-cleared Process on the FORK-time seed branch.
4048 let mut p_no_status = Process::new("api", empty_spec());
4049 p_no_status.status = None;
4050 let p_empty_slot = process_with_identity(None);
4051 assert_eq!(
4052 p_no_status.observed_identity().is_none(),
4053 p_empty_slot.observed_identity().is_none()
4054 );
4055 assert_eq!(
4056 p_no_status.observed_identity().is_some(),
4057 p_empty_slot.observed_identity().is_some()
4058 );
4059 }
4060
4061 #[test]
4062 fn observed_identity_cloned_composes_with_derive_identity_fallback() {
4063 // Cross-primitive composition pin: the borrow-form
4064 // primitive threaded through `.cloned().unwrap_or_else(||
4065 // derive_identity(...))` reproduces the pre-lift FORK-time
4066 // seed's owned-`Identity` shape at every corner. Binds the
4067 // exact composition the `phase_machine::handle_forking`
4068 // consumer performs: on the populated-slot corner the
4069 // reconciler-persisted `Identity` is returned verbatim (the
4070 // fallback never fires), and on both empty corners
4071 // (missing-status + empty-slot) the fallback fires
4072 // producing a fresh `derive_identity(&spec,
4073 // name_override)`. A regression that (a) swapped the
4074 // fallback direction, (b) made `.cloned()` re-derive
4075 // instead of clone, or (c) made the empty-slot corner
4076 // return a synthetic `Some(default_identity)` collides
4077 // with the fallback surfaces here rather than as silent
4078 // FORK-time PID allocator skew.
4079 let spec = empty_spec();
4080 let fallback_expected = crate::identity::derive_identity(&spec, None);
4081 // Populated-slot corner: the seed returns the persisted
4082 // identity, NOT the derive fallback.
4083 let persisted = sample_identity("seph");
4084 let p = process_with_identity(Some(persisted.clone()));
4085 let seed = p.observed_identity().cloned().unwrap_or_else(|| {
4086 crate::identity::derive_identity(&p.spec, p.declared_name_override())
4087 });
4088 assert_eq!(seed, persisted);
4089 assert_ne!(seed, fallback_expected);
4090 // Empty-slot corner: the seed fires the derive fallback.
4091 let p = process_with_identity(None);
4092 let seed = p.observed_identity().cloned().unwrap_or_else(|| {
4093 crate::identity::derive_identity(&p.spec, p.declared_name_override())
4094 });
4095 assert_eq!(seed, fallback_expected);
4096 // Missing-status corner: the seed fires the derive
4097 // fallback, byte-identical to the empty-slot corner.
4098 let mut p = Process::new("api-gateway", empty_spec());
4099 p.metadata.namespace = Some("prod".into());
4100 p.status = None;
4101 let seed = p.observed_identity().cloned().unwrap_or_else(|| {
4102 crate::identity::derive_identity(&p.spec, p.declared_name_override())
4103 });
4104 assert_eq!(seed, fallback_expected);
4105 }
4106
4107 // ─── Process::observed_phase substrate pins ───────────────────────
4108 //
4109 // The copy-form status-projection primitive on the phase axis.
4110 // Collapses the paired 3-line `.status.as_ref().map(|s| s.phase)`
4111 // chain every consumer in `tatara-reconciler` restated by hand
4112 // pre-lift at FIVE sites. Peer to the borrow-form
4113 // `observed_pid_*` + `observed_flux_resources_*` +
4114 // `observed_attestation_*` pin families; all four compose the
4115 // same missing-`status` fallback skeleton on distinct
4116 // `ProcessStatus` slots, with the phase-axis form returning
4117 // `Option<ProcessPhase>` (copy of a `Copy` scalar) rather than
4118 // `Option<&T>` (borrow) because the underlying slot is a bare
4119 // `ProcessPhase` — no allocation to borrow past, and the enum
4120 // is one byte on the wire. Each pin fails-before-pass-after
4121 // granularity: `observed_phase` did not exist pre-lift, so any
4122 // test invoking it fails to compile pre-lift and passes
4123 // post-lift.
4124
4125 fn process_with_phase(phase: Option<ProcessPhase>) -> Process {
4126 let mut p = Process::new("api-gateway", empty_spec());
4127 p.metadata.namespace = Some("prod".into());
4128 if let Some(ph) = phase {
4129 let mut status = ProcessStatus::default();
4130 status.phase = ph;
4131 p.status = Some(status);
4132 }
4133 p
4134 }
4135
4136 #[test]
4137 fn observed_phase_returns_none_when_status_is_none() {
4138 // Missing-`status` corner pin: the primitive collapses the
4139 // no-status case to `None` so downstream `.unwrap_or(...)`
4140 // at every reconciler consumer chooses the default
4141 // deliberately (`Pending` for the top-level dispatch seed
4142 // + boundary evaluator + routing groupby; `Attested` for
4143 // the released-from annotation composer). Matches the
4144 // pre-lift `.map(|s| s.phase)` chain's `None`
4145 // byte-identically at every consumer's downstream shape.
4146 let mut p = Process::new("api", empty_spec());
4147 p.status = None;
4148 assert!(p.observed_phase().is_none());
4149 }
4150
4151 #[test]
4152 fn observed_phase_returns_some_default_when_status_is_populated_with_default_phase() {
4153 // Populated-status corner pin: the primitive returns
4154 // `Some(ProcessPhase::default())` — a `ProcessStatus`
4155 // constructed via `default()` carries `phase: Pending`
4156 // because the phase field is a bare `ProcessPhase` (not
4157 // `Option<ProcessPhase>`), so there is NO "empty slot"
4158 // corner peer to the borrow-form projections' empty-slot
4159 // pins. A regression that reshaped the return type to
4160 // filter out `Pending` (treating it as "unset") would
4161 // surface here and silently break the top-level
4162 // dispatcher's Pending → Forking transition on a Process
4163 // freshly written by the reconciler.
4164 let p = process_with_phase(Some(ProcessPhase::default()));
4165 assert_eq!(p.observed_phase(), Some(ProcessPhase::Pending));
4166 assert_eq!(p.observed_phase(), Some(ProcessPhase::default()));
4167 }
4168
4169 #[test]
4170 fn observed_phase_returns_persisted_phase_when_status_is_populated() {
4171 // Happy-path pin: with a populated `status.phase` slot,
4172 // the primitive returns the persisted `ProcessPhase`.
4173 // A regression that filtered / reshaped / canonicalized
4174 // the phase would surface here rather than as silent
4175 // skew at the top-level dispatcher's phase handler
4176 // dispatch on the SAME Process.
4177 let p = process_with_phase(Some(ProcessPhase::Running));
4178 assert_eq!(p.observed_phase(), Some(ProcessPhase::Running));
4179 }
4180
4181 #[test]
4182 fn observed_phase_is_a_pure_projection() {
4183 // Purity pin: two consecutive calls return byte-identical
4184 // `Option<ProcessPhase>` values (no lazy materialization,
4185 // no interior mutation of `self`). Peer to the sibling
4186 // `observed_pid_is_a_pure_projection` +
4187 // `observed_flux_resources_is_a_pure_projection` +
4188 // `observed_attestation_is_a_pure_projection` pins; all
4189 // four bind the pure-projection discipline on the ONE
4190 // substrate accessor per status slot.
4191 let p = process_with_phase(Some(ProcessPhase::Attested));
4192 let a = p.observed_phase();
4193 let b = p.observed_phase();
4194 assert_eq!(a, b);
4195 assert_eq!(a, Some(ProcessPhase::Attested));
4196 }
4197
4198 #[test]
4199 fn observed_phase_matches_pre_lift_reconciler_chain_shape() {
4200 // Parity pin: sweeps the two corners every pre-lift
4201 // consumer plausibly encountered (missing status,
4202 // populated status with a particular phase) and compares
4203 // the substrate call against a hand-authored pre-lift
4204 // chain byte-identically. A regression that reshaped ANY
4205 // of the two corners would surface here rather than as
4206 // silent operator-facing skew between the top-level
4207 // dispatcher and any of the four other reconciler
4208 // consumers on the SAME `Process`.
4209 fn pre_lift(p: &Process) -> Option<ProcessPhase> {
4210 p.status.as_ref().map(|s| s.phase)
4211 }
4212 let mut p = Process::new("api", empty_spec());
4213 p.status = None;
4214 assert_eq!(p.observed_phase(), pre_lift(&p));
4215 let p = process_with_phase(Some(ProcessPhase::Running));
4216 assert_eq!(p.observed_phase(), pre_lift(&p));
4217 let p = process_with_phase(Some(ProcessPhase::Attested));
4218 assert_eq!(p.observed_phase(), pre_lift(&p));
4219 let p = process_with_phase(Some(ProcessPhase::Failed));
4220 assert_eq!(p.observed_phase(), pre_lift(&p));
4221 }
4222
4223 #[test]
4224 fn observed_phase_default_unwrap_matches_pre_lift_pending_default() {
4225 // Callsite-shape pin: three of the FIVE pre-lift consumers
4226 // (`controller::reconcile`, `boundary::evaluate_process_phase`,
4227 // `table_controller::stable_name_group_key`) closed the
4228 // 3-line chain with `.unwrap_or(ProcessPhase::Pending)`
4229 // (identical to `.unwrap_or_default()`). This pin binds
4230 // that call-site shape: `observed_phase().unwrap_or
4231 // (Pending)` returns `Pending` on missing status and the
4232 // persisted phase otherwise. A regression that swapped
4233 // the `None` sentinel's downstream default would surface
4234 // here rather than as silent skew at three of the five
4235 // consumer sites.
4236 let mut p = Process::new("api", empty_spec());
4237 p.status = None;
4238 assert_eq!(
4239 p.observed_phase().unwrap_or(ProcessPhase::Pending),
4240 ProcessPhase::Pending
4241 );
4242 let p = process_with_phase(Some(ProcessPhase::Running));
4243 assert_eq!(
4244 p.observed_phase().unwrap_or(ProcessPhase::Pending),
4245 ProcessPhase::Running
4246 );
4247 }
4248
4249 #[test]
4250 fn observed_phase_attested_unwrap_matches_pre_lift_released_from_default() {
4251 // Callsite-shape pin: the ONE pre-lift consumer
4252 // (`phase_machine::p_current_phase_str` — the
4253 // released-from annotation composer) closed the 3-line
4254 // chain with `.unwrap_or(ProcessPhase::Attested)` rather
4255 // than the `Default` (`Pending`). This pin binds that
4256 // call-site shape: `observed_phase().unwrap_or(Attested)`
4257 // returns `Attested` on missing status and the persisted
4258 // phase otherwise. A regression that folded the
4259 // `Attested`-default consumer into the `Pending`-default
4260 // majority would break the SIGSTOP/SIGCONT release gate's
4261 // "which annotation label to emit" branch — the pin binds
4262 // the primitive at the raw `Option<ProcessPhase>` form so
4263 // this default choice stays local at the callsite.
4264 let mut p = Process::new("api", empty_spec());
4265 p.status = None;
4266 assert_eq!(
4267 p.observed_phase().unwrap_or(ProcessPhase::Attested),
4268 ProcessPhase::Attested
4269 );
4270 let p = process_with_phase(Some(ProcessPhase::Failed));
4271 assert_eq!(
4272 p.observed_phase().unwrap_or(ProcessPhase::Attested),
4273 ProcessPhase::Failed
4274 );
4275 }
4276
4277 #[test]
4278 fn observed_phase_preserves_every_process_phase_variant() {
4279 // Round-trip pin: every `ProcessPhase` variant round-
4280 // trips through the primitive unchanged. Peer to the
4281 // sibling `observed_pid_preserves_hierarchical_pid_format`
4282 // pin's dotted-segment sweep; this pin sweeps the closed
4283 // set of `ProcessPhase` variants directly so a
4284 // canonicalization pass that dropped or reshaped one
4285 // (e.g. folded `Reconverging` back into `Execing`, or
4286 // remapped `Zombie` to `Reaped`) surfaces here rather
4287 // than as silent skew at the SIGSTOP/SIGCONT release
4288 // gate's phase-name annotation branch. Covers every
4289 // variant the `ProcessPhase::DeriveClosedSet` enumerates
4290 // so a future variant addition surfaces via the closed-
4291 // set macro rather than at a silent partial sweep.
4292 for phase in [
4293 ProcessPhase::Pending,
4294 ProcessPhase::Forking,
4295 ProcessPhase::Execing,
4296 ProcessPhase::Running,
4297 ProcessPhase::Attested,
4298 ProcessPhase::Reconverging,
4299 ProcessPhase::Releasing,
4300 ProcessPhase::Exiting,
4301 ProcessPhase::Failed,
4302 ProcessPhase::Zombie,
4303 ProcessPhase::Reaped,
4304 ] {
4305 let p = process_with_phase(Some(phase));
4306 assert_eq!(
4307 p.observed_phase(),
4308 Some(phase),
4309 "phase variant {phase:?} did not round-trip"
4310 );
4311 }
4312 }
4313
4314 // ─── Process::observed_phase_or_pending substrate pins ─────────────
4315 //
4316 // Pins the copy-form status-projection primitive on the phase
4317 // axis with the `Pending` sink applied. Sibling to the raw
4318 // `observed_phase_*` pin family on the (return-form × fallback
4319 // shape) axis pair — the raw-`Option` corner stays with the
4320 // sibling family; this pin family opens the `Pending`-defaulted
4321 // corner that four of the five pre-lift `observed_phase`
4322 // consumers wrote by hand. Fail-before-pass-after granularity:
4323 // `observed_phase_or_pending` did not exist pre-lift, so any
4324 // test invoking it fails to compile pre-lift and passes
4325 // post-lift.
4326
4327 #[test]
4328 fn observed_phase_or_pending_returns_pending_when_status_is_none() {
4329 // Missing-`status` corner pin: the primitive collapses the
4330 // no-status case to `Pending` — the sink four of the five
4331 // pre-lift `observed_phase` consumers wrote by hand
4332 // (`controller::reconcile` / `boundary::
4333 // evaluate_process_phase` / `table_controller::
4334 // stable_name_group_key` / `controller_pool::reconcile_pool`)
4335 // and the sentinel `ProcessPhase::default()` returns. A
4336 // regression that folded the `None` sink to any other phase
4337 // (e.g. `Forking` — treating "not yet observed" as "already
4338 // dispatched") would silently mis-seed the top-level
4339 // dispatcher's `Pending → Forking` transition and surface as
4340 // operator-visible reconcile-cycle skew on a freshly-forked
4341 // Process rather than at this pin.
4342 let mut p = Process::new("api", empty_spec());
4343 p.status = None;
4344 assert_eq!(p.observed_phase_or_pending(), ProcessPhase::Pending);
4345 }
4346
4347 #[test]
4348 fn observed_phase_or_pending_returns_persisted_phase_when_status_is_populated() {
4349 // Populated-status corner pin: the primitive passes through
4350 // the persisted `ProcessPhase` unchanged — the sink only
4351 // fires on missing `status`, not on a populated one carrying
4352 // a `Pending`-adjacent variant. Two variants pinned to
4353 // separate the "pass through the persisted phase" arm from
4354 // the "sink fires" arm: `Running` (mid-lifecycle) and
4355 // `Attested` (post-verify) both round-trip unchanged where
4356 // a regression that always returned `Pending` (dropped the
4357 // pass-through arm entirely) would surface here rather than
4358 // as silent skew at every reconciler's per-phase branch.
4359 let p = process_with_phase(Some(ProcessPhase::Running));
4360 assert_eq!(p.observed_phase_or_pending(), ProcessPhase::Running);
4361 let p = process_with_phase(Some(ProcessPhase::Attested));
4362 assert_eq!(p.observed_phase_or_pending(), ProcessPhase::Attested);
4363 }
4364
4365 #[test]
4366 fn observed_phase_or_pending_matches_pre_lift_unwrap_or_pending_chain_shape() {
4367 // Byte-identical parity pin: the primitive's return equals
4368 // the pre-lift two-link `.observed_phase().unwrap_or
4369 // (ProcessPhase::Pending)` chain at every one of the four
4370 // corner values (missing `status` → `Pending`, populated
4371 // with `Pending` → `Pending`, populated with a mid-lifecycle
4372 // variant → pass-through, populated with a terminal variant
4373 // → pass-through). A regression that swapped the sink to
4374 // `ProcessPhase::default()` (currently equivalent to
4375 // `Pending`) would keep this pin green until the enum's
4376 // `Default` impl drifted — the explicit `Pending` spelling
4377 // in the pin binds the operator-visible label rather than
4378 // the derived `Default`, so a future rename or reordering
4379 // of `ProcessPhase` variants that shifted `Default` off
4380 // `Pending` would surface here rather than as silent skew
4381 // at the four downstream consumer sites.
4382 let pre_lift = |p: &Process| p.observed_phase().unwrap_or(ProcessPhase::Pending);
4383 let mut p = Process::new("api", empty_spec());
4384 p.status = None;
4385 assert_eq!(p.observed_phase_or_pending(), pre_lift(&p));
4386 let p = process_with_phase(Some(ProcessPhase::Pending));
4387 assert_eq!(p.observed_phase_or_pending(), pre_lift(&p));
4388 let p = process_with_phase(Some(ProcessPhase::Running));
4389 assert_eq!(p.observed_phase_or_pending(), pre_lift(&p));
4390 let p = process_with_phase(Some(ProcessPhase::Reaped));
4391 assert_eq!(p.observed_phase_or_pending(), pre_lift(&p));
4392 }
4393
4394 #[test]
4395 fn observed_phase_or_pending_is_a_pure_projection() {
4396 // Purity pin: two back-to-back calls on the same `Process`
4397 // return the same `ProcessPhase` — the primitive stamps no
4398 // side effect (no clock read, no metadata write, no
4399 // `status` mutation) despite the sibling `observed_phase`
4400 // taking `&self` too. Peer to the sibling `observed_phase`
4401 // purity pin; a regression that folded a clock read (e.g.
4402 // "if the sink fired, stamp `phase_since = Utc::now()`")
4403 // into the primitive would surface here rather than at the
4404 // consumer sites' downstream reconcile-cycle behavior.
4405 let p = process_with_phase(Some(ProcessPhase::Running));
4406 let a = p.observed_phase_or_pending();
4407 let b = p.observed_phase_or_pending();
4408 assert_eq!(a, b);
4409 }
4410
4411 #[test]
4412 fn observed_phase_or_pending_preserves_every_process_phase_variant() {
4413 // Round-trip pin: every `ProcessPhase` variant round-trips
4414 // through the primitive unchanged when the `status` slot is
4415 // populated. Peer to the sibling `observed_phase_preserves
4416 // _every_process_phase_variant` sweep; this pin sweeps the
4417 // closed set through the `Pending`-sinked accessor rather
4418 // than the raw-`Option` accessor so a canonicalization pass
4419 // that dropped or reshaped one variant (e.g. folded
4420 // `Reconverging` back into `Execing`, remapped `Zombie` to
4421 // `Reaped`) surfaces at BOTH primitives' pin sets rather
4422 // than as silent skew at a subset of the reconciler
4423 // consumers. Covers every variant the
4424 // `ProcessPhase::DeriveClosedSet` enumerates so a future
4425 // variant addition surfaces via the closed-set macro rather
4426 // than at a silent partial sweep.
4427 for phase in [
4428 ProcessPhase::Pending,
4429 ProcessPhase::Forking,
4430 ProcessPhase::Execing,
4431 ProcessPhase::Running,
4432 ProcessPhase::Attested,
4433 ProcessPhase::Reconverging,
4434 ProcessPhase::Releasing,
4435 ProcessPhase::Exiting,
4436 ProcessPhase::Failed,
4437 ProcessPhase::Zombie,
4438 ProcessPhase::Reaped,
4439 ] {
4440 let p = process_with_phase(Some(phase));
4441 assert_eq!(
4442 p.observed_phase_or_pending(),
4443 phase,
4444 "phase variant {phase:?} did not round-trip through observed_phase_or_pending"
4445 );
4446 }
4447 }
4448
4449 // ─── Process::is_being_deleted substrate pins ───────────────────────
4450 //
4451 // Pins the copy-form metadata-projection primitive on the
4452 // deletion-tombstone axis. Peer to the borrow-form + copy-form
4453 // metadata-fallback family (`namespace_or_default`,
4454 // `name_or_placeholder`, `uid_or_empty`, `coordinates_or_defaults`,
4455 // `coordinates_or_none`, `owned_coordinates_or_err`, `annotation`);
4456 // this one opens the presence-probe corner for the tombstone slot.
4457 // Fail-before-pass-after granularity: `is_being_deleted` did not
4458 // exist pre-lift, so any test invoking it fails to compile pre-
4459 // lift and passes post-lift.
4460
4461 fn tombstoned_process() -> Process {
4462 let mut p = Process::new("api-gateway", empty_spec());
4463 p.metadata.namespace = Some("prod".into());
4464 p.metadata.deletion_timestamp = Some(k8s_openapi::apimachinery::pkg::apis::meta::v1::Time(
4465 Utc::now(),
4466 ));
4467 p
4468 }
4469
4470 #[test]
4471 fn is_being_deleted_returns_false_when_deletion_timestamp_is_absent() {
4472 // Missing-tombstone corner pin: the primitive collapses the
4473 // no-tombstone case to `false` so the SIGTERM preempt at
4474 // `controller::reconcile` skips the `→ Exiting` forcing
4475 // branch and the DELETE-skip at `handle_exiting`'s child
4476 // fan-out does NOT `continue` past a child that is still
4477 // healthy. Matches the pre-lift `.is_some()` chain's `false`
4478 // byte-identically at every consumer's downstream gate.
4479 let mut p = Process::new("api", empty_spec());
4480 p.metadata.deletion_timestamp = None;
4481 assert!(!p.is_being_deleted());
4482 }
4483
4484 #[test]
4485 fn is_being_deleted_returns_true_when_deletion_timestamp_is_present() {
4486 // Present-tombstone corner pin: the primitive returns
4487 // `true` on any populated `metadata.deletionTimestamp`
4488 // slot regardless of the timestamp payload — the two
4489 // consumers only read the tombstone's PRESENCE, never
4490 // its RFC-3339 timestamp value. A regression that gated
4491 // the `true` return on the timestamp being non-epoch, or
4492 // parsed the timestamp before returning, would surface
4493 // here rather than as silent skew at the SIGTERM preempt
4494 // or child-fan-out DELETE-skip on the SAME `Process`.
4495 let p = tombstoned_process();
4496 assert!(p.is_being_deleted());
4497 }
4498
4499 #[test]
4500 fn is_being_deleted_is_a_pure_projection() {
4501 // Purity pin: two consecutive calls return byte-identical
4502 // `bool` values (no lazy materialization, no interior
4503 // mutation of `self`). Peer to the sibling
4504 // `observed_phase_is_a_pure_projection` +
4505 // `observed_pid_is_a_pure_projection` +
4506 // `observed_flux_resources_is_a_pure_projection` +
4507 // `observed_attestation_is_a_pure_projection` pins; all
4508 // five bind the pure-projection discipline on the ONE
4509 // substrate accessor per metadata / status slot.
4510 let p = tombstoned_process();
4511 let a = p.is_being_deleted();
4512 let b = p.is_being_deleted();
4513 assert_eq!(a, b);
4514 assert!(a);
4515 }
4516
4517 #[test]
4518 fn is_being_deleted_matches_pre_lift_reconciler_chain_shape() {
4519 // Parity pin: sweeps the two corners every pre-lift
4520 // consumer plausibly encountered (missing tombstone,
4521 // present tombstone) and compares the substrate call
4522 // against a hand-authored pre-lift chain byte-identically.
4523 // A regression that reshaped either corner would surface
4524 // here rather than as silent operator-facing skew between
4525 // the top-level dispatcher's SIGTERM preempt and the
4526 // SIGTERM cascade's child-fan-out DELETE-skip on the
4527 // SAME `Process` within one reconcile pass.
4528 fn pre_lift(p: &Process) -> bool {
4529 p.metadata.deletion_timestamp.is_some()
4530 }
4531 let mut p = Process::new("api", empty_spec());
4532 p.metadata.deletion_timestamp = None;
4533 assert_eq!(p.is_being_deleted(), pre_lift(&p));
4534 let p = tombstoned_process();
4535 assert_eq!(p.is_being_deleted(), pre_lift(&p));
4536 }
4537
4538 #[test]
4539 fn is_being_deleted_composes_with_process_phase_is_alive_at_reconcile_preempt() {
4540 // Call-site-shape pin: the `controller::reconcile` SIGTERM
4541 // preempt composes `is_being_deleted() && current_phase
4542 // .is_alive()` — the tombstone-presence probe AND the
4543 // alive-phase gate must BOTH hold to force `→ Exiting`.
4544 // A dead-phase (`Zombie` / `Reaped` / `Failed`) Process
4545 // that carries a tombstone still runs its normal handler,
4546 // not the preempt. This pin binds that composition shape
4547 // at the primitive so a regression that flipped either
4548 // half of the `&&` (or that broadened the tombstone probe
4549 // to include the `is_alive` half implicitly) surfaces
4550 // here rather than as silent skew at the top-level
4551 // dispatch on the SAME `Process`.
4552 let mut p = tombstoned_process();
4553 // Alive + tombstoned → preempt fires.
4554 let mut alive = ProcessStatus::default();
4555 alive.phase = ProcessPhase::Running;
4556 p.status = Some(alive);
4557 assert!(p.is_being_deleted());
4558 assert!(p.observed_phase().unwrap_or_default().is_alive());
4559 // Dead + tombstoned → preempt does NOT fire (composition
4560 // with `is_alive` returns false).
4561 let mut dead = ProcessStatus::default();
4562 dead.phase = ProcessPhase::Reaped;
4563 p.status = Some(dead);
4564 assert!(p.is_being_deleted());
4565 assert!(!p.observed_phase().unwrap_or_default().is_alive());
4566 }
4567
4568 // ─── Process::created_at substrate pins ─────────────────────────
4569 //
4570 // Pins the copy-form metadata-projection primitive on the
4571 // `metadata.creationTimestamp` axis that owns the
4572 // `.metadata.creation_timestamp.as_ref().map(|t| t.0)` chain the
4573 // three hand-authored sites (`lifetime_clock::evaluate`,
4574 // `lifetime_clock::requeue_with_ttl`,
4575 // `tatara-reconciler::table_controller`) restated by hand pre-lift.
4576 // Peer to the sibling `is_being_deleted_*` +
4577 // `observed_phase_*` pin families — all three primitives project a
4578 // wire-format `Option<T>` slot into a `Copy` inner value at ONE
4579 // owner. Fail-before-pass-after granularity: `created_at` did not
4580 // exist pre-lift, so any test invoking it fails to compile pre-lift
4581 // and passes post-lift.
4582
4583 fn creation_stamped_process(t: DateTime<Utc>) -> Process {
4584 let mut p = Process::new("age-anchor", empty_spec());
4585 p.metadata.namespace = Some("prod".into());
4586 p.metadata.creation_timestamp =
4587 Some(k8s_openapi::apimachinery::pkg::apis::meta::v1::Time(t));
4588 p
4589 }
4590
4591 #[test]
4592 fn created_at_returns_none_when_creation_timestamp_is_absent() {
4593 // Missing-slot corner pin: the primitive collapses the
4594 // no-creation-timestamp case to `None` so the TTL-expiry gate
4595 // at `lifetime_clock::evaluate` short-circuits its inner
4596 // `if let Some(...)` branch (no elapsed computation), the
4597 // requeue-budget picker returns its default sleep, and the
4598 // stable-name arbiter's `.unwrap_or_else(Utc::now)` tail
4599 // synthesizes a "just created" anchor at its own site. Matches
4600 // the pre-lift `.as_ref().map(|t| t.0)` chain's `None`
4601 // byte-identically at every consumer's downstream tail.
4602 let mut p = Process::new("api", empty_spec());
4603 p.metadata.creation_timestamp = None;
4604 assert!(p.created_at().is_none());
4605 }
4606
4607 #[test]
4608 fn created_at_returns_some_datetime_when_slot_is_populated() {
4609 // Populated-slot corner pin: with a populated
4610 // `metadata.creationTimestamp` slot, the primitive unwraps the
4611 // wire-format `Time` newtype to its inner `DateTime<Utc>` and
4612 // returns it as `Some(datetime)` — hiding the `.0` field-access
4613 // every pre-lift consumer restated to reach the underlying
4614 // instant.
4615 let anchor = Utc::now() - chrono::Duration::seconds(300);
4616 let p = creation_stamped_process(anchor);
4617 assert_eq!(p.created_at(), Some(anchor));
4618 }
4619
4620 #[test]
4621 fn created_at_is_a_pure_projection() {
4622 // Purity pin: two consecutive calls return byte-identical
4623 // `Option<DateTime<Utc>>` values (no lazy materialization, no
4624 // interior mutation of `self`). Peer to the sibling
4625 // `is_being_deleted_is_a_pure_projection` +
4626 // `observed_phase_is_a_pure_projection` pins; all three bind
4627 // the pure-projection discipline on the ONE substrate accessor
4628 // per metadata / status slot.
4629 let anchor = Utc::now();
4630 let p = creation_stamped_process(anchor);
4631 let a = p.created_at();
4632 let b = p.created_at();
4633 assert_eq!(a, b);
4634 assert_eq!(a, Some(anchor));
4635 }
4636
4637 #[test]
4638 fn created_at_matches_pre_lift_creation_timestamp_chain_shape() {
4639 // Parity pin: sweeps the two corners every pre-lift consumer
4640 // plausibly encountered (missing slot, populated slot) and
4641 // compares the substrate call against a hand-authored pre-lift
4642 // chain byte-identically. A regression that reshaped either
4643 // corner (returning `Some(Utc::now())` on the missing slot,
4644 // returning a rounded / truncated timestamp on the populated
4645 // slot) would surface here rather than as silent operator-
4646 // facing skew between the TTL-expiry gate, the requeue-budget
4647 // picker, and the stable-name claim-arbiter tie-break on the
4648 // SAME `Process` within one reconcile pass.
4649 fn pre_lift(p: &Process) -> Option<DateTime<Utc>> {
4650 p.metadata.creation_timestamp.as_ref().map(|t| t.0)
4651 }
4652 // Missing slot.
4653 let mut p = Process::new("x", empty_spec());
4654 p.metadata.creation_timestamp = None;
4655 assert_eq!(p.created_at(), pre_lift(&p));
4656 // Populated slot.
4657 let anchor = Utc::now() - chrono::Duration::seconds(42);
4658 let p = creation_stamped_process(anchor);
4659 assert_eq!(p.created_at(), pre_lift(&p));
4660 }
4661
4662 #[test]
4663 fn created_at_composes_with_signed_duration_since_at_ttl_gate() {
4664 // Call-site-shape pin: the `lifetime_clock::evaluate` TTL-
4665 // expiry gate composes `now.signed_duration_since(creation)`
4666 // where `creation` is the `DateTime<Utc>` returned by this
4667 // primitive's `Some` corner. A regression that returned a
4668 // per-callsite `Local` timezone (or that stripped the timezone
4669 // marker) would break the arithmetic silently. This pin
4670 // computes the elapsed duration byte-identically against the
4671 // pre-lift `.map(|t| t.0)` chain so a timezone drift surfaces
4672 // here rather than as silent skew at the TTL-expiry decision
4673 // on the SAME `Process` within one reconcile pass.
4674 let now = Utc::now();
4675 let anchor = now - chrono::Duration::seconds(120);
4676 let p = creation_stamped_process(anchor);
4677 let via_primitive = p.created_at().expect("populated slot");
4678 let via_pre_lift = p
4679 .metadata
4680 .creation_timestamp
4681 .as_ref()
4682 .map(|t| t.0)
4683 .expect("populated slot");
4684 assert_eq!(
4685 now.signed_duration_since(via_primitive),
4686 now.signed_duration_since(via_pre_lift)
4687 );
4688 }
4689
4690 // ─── Process::resolved_ephemeral substrate pins ─────────────────
4691 //
4692 // Pins the compound spec-projection primitive on the
4693 // `spec.lifetime` axis that owns the ambiguity-aware
4694 // `resolved_ephemeral` chain the three hand-authored sites
4695 // (`lifetime_clock::evaluate`, `lifetime_clock::requeue_with_ttl`,
4696 // `tatara-reconciler::render::render_export_jobs`) restated by
4697 // hand pre-lift through TWO different chains that disagreed on
4698 // the ambiguous corner. Fail-before-pass-after granularity:
4699 // `resolved_ephemeral` did not exist pre-lift on `impl Process`,
4700 // so any test invoking it fails to compile pre-lift and passes
4701 // post-lift.
4702
4703 fn permanent_only_process() -> Process {
4704 let mut spec = empty_spec();
4705 spec.lifetime = crate::lifetime::Lifetime {
4706 permanent: Some(crate::lifetime::PermanentLifetime {}),
4707 ..crate::lifetime::Lifetime::default()
4708 };
4709 Process::new("perm", spec)
4710 }
4711
4712 fn ephemeral_only_process(ttl: &str) -> Process {
4713 let mut spec = empty_spec();
4714 spec.lifetime = crate::lifetime::Lifetime {
4715 ephemeral: Some(EphemeralLifetime {
4716 ttl: ttl.into(),
4717 teardown_policy: crate::lifetime::TeardownPolicy::OnAttested,
4718 max_concurrent: 3,
4719 exports: vec![],
4720 }),
4721 ..crate::lifetime::Lifetime::default()
4722 };
4723 Process::new("eph", spec)
4724 }
4725
4726 fn ambiguous_lifetime_process() -> Process {
4727 let mut spec = empty_spec();
4728 spec.lifetime = crate::lifetime::Lifetime {
4729 permanent: Some(crate::lifetime::PermanentLifetime {}),
4730 ephemeral: Some(EphemeralLifetime::default()),
4731 };
4732 Process::new("both", spec)
4733 }
4734
4735 #[test]
4736 fn resolved_ephemeral_returns_none_when_lifetime_is_default_empty() {
4737 // Empty-default corner pin: neither slot populated. The
4738 // resolver collapses to `Permanent(&DEFAULT_PERMANENT)` and
4739 // the compound projection sees no ephemeral inner. Matches
4740 // the pre-lift `lifetime_clock::evaluate` early-return to
4741 // `AutoTerminate::Skip` byte-identically.
4742 let p = Process::new("empty-lifetime", empty_spec());
4743 assert!(p.resolved_ephemeral().is_none());
4744 }
4745
4746 #[test]
4747 fn resolved_ephemeral_returns_none_for_permanent_only_process() {
4748 // Permanent-only corner pin: the `permanent:` slot is
4749 // populated, `ephemeral:` is not. Matches the pre-lift
4750 // `lifetime_clock::evaluate` outcome — the teardown/TTL
4751 // branch is never reached on a Permanent Process, and the
4752 // export-render arm now agrees at this call site (was
4753 // previously reached through the raw `.ephemeral.as_ref()`
4754 // that also returned `None` on this same corner — no drift
4755 // here; the drift is at the ambiguous corner below).
4756 let p = permanent_only_process();
4757 assert!(p.resolved_ephemeral().is_none());
4758 }
4759
4760 #[test]
4761 fn resolved_ephemeral_returns_some_for_ephemeral_only_process() {
4762 // Ephemeral-only corner pin: the ONE arm that projects. The
4763 // returned borrow carries the operator-authored `ttl` /
4764 // `teardown_policy` / `max_concurrent` verbatim. A
4765 // regression that swapped the projection to the sibling
4766 // `permanent:` slot would surface here as a type mismatch on
4767 // the `EphemeralLifetime` fields rather than as silent
4768 // operator-facing no-op teardown at the reconciler.
4769 let p = ephemeral_only_process("42m");
4770 let e = p
4771 .resolved_ephemeral()
4772 .expect("ephemeral-only Process must project");
4773 assert_eq!(e.ttl, "42m");
4774 assert_eq!(
4775 e.teardown_policy,
4776 crate::lifetime::TeardownPolicy::OnAttested
4777 );
4778 assert_eq!(e.max_concurrent, 3);
4779 }
4780
4781 #[test]
4782 fn resolved_ephemeral_returns_none_for_ambiguous_lifetime() {
4783 // DRIFT-CLOSING CONTRACT: BOTH `permanent:` AND `ephemeral:`
4784 // slots populated is an operator-authored mis-configuration.
4785 // Pre-lift, `lifetime_clock::evaluate` (via
4786 // `resolved_ephemeral()` on `Lifetime`) collapsed this
4787 // corner to `None` and yielded `AutoTerminate::Skip`, while
4788 // `tatara-reconciler::render::render_export_jobs` walked
4789 // the naked `.spec.lifetime.ephemeral.as_ref()` chain and
4790 // returned `Some(&e)` — so the reconciler would emit export
4791 // Jobs on a Process whose teardown-triggered fire semantics
4792 // the lifetime clock refused to honor. Post-lift this
4793 // primitive collapses ambiguity to `None` at ONE site so
4794 // BOTH consumers agree. A regression that broadened the
4795 // projection back to the raw field (or that silently
4796 // "preferred ephemeral" in the ambiguous case) surfaces
4797 // here rather than as export-Job noise on a mis-configured
4798 // ephemeral.
4799 let p = ambiguous_lifetime_process();
4800 assert!(p.resolved_ephemeral().is_none());
4801 // The raw field IS populated at this corner — pins the
4802 // pre-lift `.spec.lifetime.ephemeral.as_ref()` shape that
4803 // returned `Some` here.
4804 assert!(p.spec.lifetime.ephemeral.is_some());
4805 }
4806
4807 #[test]
4808 fn resolved_ephemeral_matches_spec_lifetime_forwarder() {
4809 // Byte-identity pin: the `Process` projection delegates
4810 // through the underlying `Lifetime::resolved_ephemeral`
4811 // primitive at every corner (empty, permanent-only,
4812 // ephemeral-only, ambiguous). A regression that silently
4813 // reintroduced the raw `.ephemeral.as_ref()` shortcut, or
4814 // that decided the ambiguous case by "prefer ephemeral"
4815 // at the Process layer instead of delegating, surfaces
4816 // here.
4817 for p in [
4818 Process::new("empty", empty_spec()),
4819 permanent_only_process(),
4820 ephemeral_only_process("1h"),
4821 ambiguous_lifetime_process(),
4822 ] {
4823 let via_process = p.resolved_ephemeral();
4824 let via_lifetime = p.spec.lifetime.resolved_ephemeral();
4825 // Both borrows point into the SAME `EphemeralLifetime`
4826 // slot when present — a regression that materialized a
4827 // per-call clone at the Process layer would fail the
4828 // pointer-equality gate.
4829 match (via_process, via_lifetime) {
4830 (Some(a), Some(b)) => assert!(
4831 std::ptr::eq(a, b),
4832 "Process::resolved_ephemeral must borrow the same slot as Lifetime::resolved_ephemeral"
4833 ),
4834 (None, None) => {}
4835 (a, b) => panic!(
4836 "resolved_ephemeral shape drift: process={:?}, lifetime={:?}",
4837 a.is_some(),
4838 b.is_some()
4839 ),
4840 }
4841 }
4842 }
4843
4844 #[test]
4845 fn resolved_ephemeral_is_a_pure_projection() {
4846 // Purity pin: two consecutive calls return borrows into the
4847 // same underlying slot (no lazy materialization, no interior
4848 // mutation of `self`). Peer to the sibling
4849 // `is_being_deleted_is_a_pure_projection` +
4850 // `observed_attestation_is_a_pure_projection` pins; all
4851 // three bind the pure-projection discipline on the ONE
4852 // substrate accessor per spec / metadata / status slot.
4853 let p = ephemeral_only_process("5m");
4854 let a = p.resolved_ephemeral();
4855 let b = p.resolved_ephemeral();
4856 match (a, b) {
4857 (Some(x), Some(y)) => assert!(std::ptr::eq(x, y)),
4858 other => panic!("expected two Some borrows into the same slot, got {other:?}"),
4859 }
4860 }
4861}