Skip to main content

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    /// The copy-form status-projection primitive on the
1367    /// `status.phase_since` axis: returns the [`DateTime<Utc>`] the
1368    /// reconciler stamped when this Process last transitioned into its
1369    /// current [`ProcessPhase`], wrapped in an `Option` so BOTH the
1370    /// missing-`status` corner AND the empty-slot corner
1371    /// (`ProcessStatus.phase_since == None` — a freshly-forked Process
1372    /// whose reconciler has not yet stamped a first transition) collapse
1373    /// to `None` — the ONE-liner collapse of the paired
1374    /// `self.status.as_ref().and_then(|s| s.phase_since)` incantation
1375    /// the pool reconciler's per-owned-Process member-seed builder
1376    /// restated by hand pre-lift.
1377    ///
1378    /// Pre-lift the 5-line
1379    /// ```rust,ignore
1380    /// p.status
1381    ///     .as_ref()
1382    ///     .and_then(|s| s.phase_since)
1383    ///     .unwrap_or_else(Utc::now)
1384    /// ```
1385    /// chain was hand-authored at
1386    /// `tatara-pool-reconciler::controller_pool::reconcile_inner`'s
1387    /// per-owned-Process `PoolMember { entered_state_at: … }` seed —
1388    /// the row-builder that feeds `pool_phase_from_members` +
1389    /// `apply_pool_reconcile_decision` with each owned Process's
1390    /// last-observed transition instant. Post-lift the callsite reads
1391    /// `p.observed_phase_since().unwrap_or_else(Utc::now)`, a
1392    /// one-liner symmetric to the peer `p.created_at()
1393    /// .unwrap_or_else(Utc::now)` chain the sibling
1394    /// [`PoolMemberSnapshot`] `created_at` seed two branches below
1395    /// already routes through — closing the last raw
1396    /// `.status.as_ref()` chain on `Process` at that reconciler site.
1397    ///
1398    /// Return-form axis: `Option<DateTime<Utc>>` matches the copy-form
1399    /// discipline of the sibling metadata-projection primitive
1400    /// [`Self::created_at`] (both return `Option<DateTime<Utc>>` and
1401    /// hide the wire-format wrapper — `ProcessStatus` on the status
1402    /// side, `k8s_openapi::…::v1::Time` on the metadata side) so the
1403    /// two timestamp-projection primitives compose byte-uniformly at
1404    /// the pool reconciler's `PoolMember` / `PoolMemberSnapshot`
1405    /// seeds. Returning owned `DateTime<Utc>` with a
1406    /// substrate-injected `Utc::now()` fallback would fold an impure
1407    /// wall-clock read into the primitive, breaking the pure-
1408    /// projection discipline every peer `observed_*` accessor
1409    /// follows; the sink stays at the callsite where it composes with
1410    /// [`Self::created_at`]'s identical `.unwrap_or_else(Utc::now)`
1411    /// tail.
1412    ///
1413    /// Peer to the copy-form status-projection primitive
1414    /// [`Self::observed_phase`] on the (return-shape × status-slot)
1415    /// axis pair — both walk the paired `.status.as_ref().<map|and_then>
1416    /// (|s| s.<slot>)` chain and both project a `Copy` inner from a
1417    /// wire slot whose "not yet observed" corner collapses to `None`.
1418    /// [`Self::observed_phase`] projects the `phase` slot (a bare
1419    /// [`ProcessPhase`] with a `Default` sentinel — collapses only on
1420    /// missing `status`); this method projects the `phase_since` slot
1421    /// (an `Option<DateTime<Utc>>` with no sentinel — collapses on
1422    /// missing `status` OR on empty slot). The paired
1423    /// `.map` vs `.and_then` choice tracks the difference: the raw
1424    /// slot is `Option<DateTime<Utc>>` here so the closure returns an
1425    /// `Option` and the outer combinator flattens through `.and_then`,
1426    /// where `observed_phase`'s raw slot is a bare `ProcessPhase` so
1427    /// the closure returns a bare value and the outer combinator maps
1428    /// through `.map`. Future status-timestamp projections (an
1429    /// `observed_last_boundary_check` on
1430    /// [`crate::status::BoundaryStatus.last_check`], an
1431    /// `observed_last_export_receipt` on a future receipt-observation
1432    /// slot) land as peer methods on this same axis.
1433    ///
1434    /// A future normalization step (a per-cluster clock-skew guard
1435    /// that offsets the returned timestamp by the observing controller's
1436    /// measured skew, a canonicalization pass that maps a suspiciously-
1437    /// zero `phase_since` to `None` so consumers' `.unwrap_or_else
1438    /// (Utc::now)` tails synthesize a fresh anchor, a staleness gate
1439    /// that drops a `phase_since` predating a reconcile deadline) lands
1440    /// at ONE substrate method here and every downstream consumer
1441    /// picks up the upgrade mechanically — no per-callsite hand-edit
1442    /// at `reconcile_inner`'s member-seed builder.
1443    ///
1444    /// Theory anchor: THEORY.md §VI.1 (generation over composition —
1445    /// the paired `.status.as_ref().and_then(|s| s.phase_since)` chain
1446    /// closes the last raw `.status.as_ref()` chain in
1447    /// `tatara-pool-reconciler`'s production reconciler code on
1448    /// `Process`, and is lifted to ONE substrate owner here alongside
1449    /// the sibling `observed_phase` / `observed_phase_or_pending` /
1450    /// `observed_identity` / `observed_pid` / `observed_attestation` /
1451    /// `observed_flux_resources` primitives that closed their axes
1452    /// previously). THEORY.md §II.1 invariant 5 (composition preserves
1453    /// proofs — the pins bind the missing-`status` corner + the empty-
1454    /// slot corner + the populated-slot pass-through + the pure-
1455    /// projection discipline + the byte-identical parity with the pre-
1456    /// lift `.status.as_ref().and_then(|s| s.phase_since)` chain + the
1457    /// composition-shape agreement with [`Self::created_at`]'s
1458    /// identical `.unwrap_or_else(Utc::now)` tail at the peer
1459    /// pool-reconciler seed, so a regression that drifted any surface
1460    /// at `tests::observed_phase_since_*` rather than as silent
1461    /// operator-facing skew between the `PoolMember` row's observed-
1462    /// transition anchor and the `PoolMemberSnapshot`'s creation-
1463    /// timestamp anchor on the SAME owned `Process` within one
1464    /// reconcile pass).
1465    #[must_use]
1466    pub fn observed_phase_since(&self) -> Option<DateTime<Utc>> {
1467        self.status.as_ref().and_then(|s| s.phase_since)
1468    }
1469
1470    /// Copy-form metadata-projection primitive on the deletion-tombstone
1471    /// axis: returns `true` iff the K8s API server has stamped a
1472    /// `metadata.deletionTimestamp` on this Process (the moment the
1473    /// object entered the "being deleted" corner of its lifecycle,
1474    /// after which further mutating writes are refused and finalizers
1475    /// are drained before the object is actually removed) — the ONE-
1476    /// liner collapse of the paired `self.metadata.deletion_timestamp
1477    /// .is_some()` incantation every consumer restated by hand
1478    /// pre-lift.
1479    ///
1480    /// Pre-lift the `.metadata.deletion_timestamp.is_some()` chain
1481    /// was hand-authored at TWO sites past the ★★ PRIME-DIRECTIVE
1482    /// ≥ 2 duplication threshold in `tatara-reconciler`, both
1483    /// projecting the SAME tombstone-presence predicate on a
1484    /// `Process` value:
1485    /// * `controller::reconcile` — the top-level dispatcher's
1486    ///   deletion-preempt gate that forces the SIGTERM cascade
1487    ///   (`→ Exiting`) as soon as the API server stamps the
1488    ///   tombstone, before the phase handler for the current
1489    ///   [`ProcessPhase`] gets a chance to run. Composed with
1490    ///   [`ProcessPhase::is_alive`] so the preempt only fires on a
1491    ///   Process still in an alive phase — a Process already in
1492    ///   `Zombie` / `Reaped` / `Failed` runs its normal handler.
1493    /// * `phase_machine::handle_exiting` — the SIGTERM cascade's
1494    ///   child-fan-out loop that enumerates every child Process and
1495    ///   skips ones the API server has already tombstoned (so the
1496    ///   reconciler does not re-issue a `DELETE` against a child
1497    ///   whose deletion the API server is already draining through
1498    ///   its own finalizer). The skip composes with
1499    ///   [`Self::coordinates_or_none`]'s name-required probe so a
1500    ///   child missing either its tombstone-absent gate or its
1501    ///   `metadata.name` slot is a clean `continue` rather than an
1502    ///   attempted `child_api.delete("")` no-op.
1503    ///
1504    /// Both sites walked the SAME `.metadata.deletion_timestamp
1505    /// .is_some()` chain and both wanted the `bool` form the
1506    /// primitive returns — the `controller::reconcile` site to gate
1507    /// the SIGTERM preempt with `&& current_phase.is_alive()` and
1508    /// the `handle_exiting` site to gate the DELETE-skip with a
1509    /// bare `if child.is_being_deleted() { continue; }`. Post-lift
1510    /// each callsite reads `process.is_being_deleted()` and the
1511    /// produced `bool` feeds the same downstream gate unchanged.
1512    ///
1513    /// Return-form axis: `bool` matches the copy-form discipline of
1514    /// [`Self::observed_phase`] (an `Option<Copy>` scalar) — the
1515    /// underlying slot is a wire-format `Option<Time>` that carries
1516    /// only presence information at this axis (the RFC-3339 timestamp
1517    /// payload itself is not what the two consumers read; both only
1518    /// probe presence to detect the tombstone-stamped state).
1519    /// Returning the raw `Option<&Time>` would push the `.is_some()`
1520    /// probe back to every callsite, restating the pre-lift chain
1521    /// one link shorter without collapsing the primitive.
1522    ///
1523    /// Peer to the metadata-fallback primitives
1524    /// [`Self::namespace_or_default`], [`Self::name_or_placeholder`],
1525    /// [`Self::uid_or_empty`], [`Self::coordinates_or_defaults`],
1526    /// [`Self::coordinates_or_none`], [`Self::owned_coordinates_or_err`],
1527    /// [`Self::annotation`] on the metadata axis; this method opens
1528    /// the copy-form peer for the presence-probe corner. Future
1529    /// metadata-presence projections (an `is_being_finalized`
1530    /// projection on `metadata.finalizers.is_empty()`'s negation,
1531    /// a `has_owner` projection on `metadata.owner_references.is_empty()`'s
1532    /// negation) land as peer methods on this same axis.
1533    ///
1534    /// A future normalization step (a per-tombstone staleness gate
1535    /// that returns `false` for a tombstone older than the reconciler's
1536    /// grace-period budget, a canonicalization pass that treats a
1537    /// tombstone from a paused controller as absent, a cross-cluster
1538    /// tombstone-observation clock skew guard) lands at ONE substrate
1539    /// method here and both downstream consumers pick up the upgrade
1540    /// mechanically — no per-callsite hand-edit at
1541    /// `controller::reconcile` / `phase_machine::handle_exiting`.
1542    ///
1543    /// Theory anchor: THEORY.md §VI.1 (generation over composition —
1544    /// the `.metadata.deletion_timestamp.is_some()` chain recurred at
1545    /// two hand-authored sites past the ★★ PRIME-DIRECTIVE ≥ 2
1546    /// duplication trigger, and is lifted to ONE owner here).
1547    /// THEORY.md §II.1 invariant 5 (composition preserves proofs —
1548    /// the pins bind the missing-tombstone corner + the present-
1549    /// tombstone corner + the copy-form `bool` return + the byte-
1550    /// identical parity with the pre-lift `.is_some()` chain, so a
1551    /// regression that drifted any surface at
1552    /// `tests::is_being_deleted_*` rather than as silent operator-
1553    /// facing skew between the top-level dispatcher's SIGTERM
1554    /// preempt and the SIGTERM cascade's child-fan-out DELETE-skip
1555    /// on the SAME `Process` within one reconcile pass).
1556    pub fn is_being_deleted(&self) -> bool {
1557        self.metadata.deletion_timestamp.is_some()
1558    }
1559
1560    /// Copy-form metadata-projection primitive on the
1561    /// `metadata.creationTimestamp` axis: returns the K8s-API-server-
1562    /// assigned creation moment as a `DateTime<Utc>`, hiding the wire-
1563    /// format `k8s_openapi::apimachinery::pkg::apis::meta::v1::Time`
1564    /// newtype behind an inherent projection — the ONE-liner collapse
1565    /// of the paired `self.metadata.creation_timestamp.as_ref().map(|t|
1566    /// t.0)` incantation every timestamp-driven consumer restated by
1567    /// hand pre-lift.
1568    ///
1569    /// Pre-lift the paired `.metadata.creation_timestamp.as_ref()` +
1570    /// `t.0` unwrap chain was hand-authored at THREE sites past the
1571    /// ★★ PRIME-DIRECTIVE ≥ 2 duplication threshold across the
1572    /// workspace, all projecting the SAME creation-moment `DateTime<Utc>`
1573    /// on a `Process`:
1574    /// * `tatara-process::lifetime_clock::evaluate` — TTL-expiry gate
1575    ///   in the ephemeral-lifetime decision (`elapsed = now
1576    ///   .signed_duration_since(creation.0)`), inside the non-terminal-
1577    ///   phase guard that fires the `AutoTerminate::Now { TtlExpired }`
1578    ///   branch. Pre-lift the site read `if let Some(creation) = process
1579    ///   .metadata.creation_timestamp.as_ref() { ... creation.0 ... }`.
1580    /// * `tatara-process::lifetime_clock::requeue_with_ttl` — sleep-
1581    ///   budget picker for the reconciler's next requeue, choosing the
1582    ///   smaller of HEARTBEAT and TTL-remaining so the reconciler
1583    ///   doesn't oversleep past a TTL boundary. Pre-lift the site read
1584    ///   `let Some(creation) = process.metadata.creation_timestamp
1585    ///   .as_ref() else { return default; };` + `creation.0`.
1586    /// * `tatara-reconciler::table_controller::reconcile_process_table`
1587    ///   — stable-name claim-arbiter row builder, seeding each
1588    ///   candidate row's `created_at` for the tie-break ordering
1589    ///   (oldest wins). Pre-lift the site read `p.metadata
1590    ///   .creation_timestamp.as_ref().map(|t| t.0).unwrap_or_else(Utc
1591    ///   ::now)`.
1592    ///
1593    /// All THREE sites walked the SAME two-link chain — read the
1594    /// `Option<Time>` slot as a borrow, then unwrap the `Time` newtype
1595    /// to its inner `DateTime<Utc>` — differing only in the tail
1596    /// (`if-let-Some` guard, `let-else` short-circuit, `Utc::now`
1597    /// fallback). Post-lift each callsite reads
1598    /// `process.created_at()` and applies its own tail at its own site
1599    /// (`if let Some(creation) = ...`, `let Some(creation) = ... else`,
1600    /// `.unwrap_or_else(Utc::now)`).
1601    ///
1602    /// Return-form axis: `Option<DateTime<Utc>>` matches the copy-form
1603    /// discipline of the sibling status-projection primitive
1604    /// [`Self::observed_phase`] — both return `Option<T>` where `T:
1605    /// Copy` and hide the wire-format wrapper (`ProcessStatus` on the
1606    /// status side; `Time` on the metadata side). Returning the raw
1607    /// `Option<&Time>` would push the `.0` unwrap back to every
1608    /// callsite, restating the pre-lift chain one link shorter without
1609    /// collapsing the primitive; returning owned `Option<Time>` would
1610    /// force a `Time` import at every consumer for a projection every
1611    /// consumer immediately discards past `.0`.
1612    ///
1613    /// Peer to the metadata-fallback + presence-probe primitives
1614    /// [`Self::namespace_or_default`], [`Self::name_or_placeholder`],
1615    /// [`Self::uid_or_empty`], [`Self::coordinates_or_defaults`],
1616    /// [`Self::coordinates_or_none`], [`Self::owned_coordinates_or_err`],
1617    /// [`Self::annotation`], [`Self::is_being_deleted`] on the metadata
1618    /// axis; this method opens the copy-form timestamp corner. Future
1619    /// metadata-timestamp projections (a
1620    /// `deletion_at() -> Option<DateTime<Utc>>` peer on the
1621    /// tombstone-payload axis for staleness gates that need the
1622    /// timestamp value alongside the presence bit) land as peer
1623    /// methods on this same axis.
1624    ///
1625    /// A future normalization step (a per-cluster clock-skew guard
1626    /// that offsets the returned timestamp by the observing controller's
1627    /// measured skew, a canonicalization pass that maps a suspiciously-
1628    /// zero creation moment to `None`, a per-namespace override that
1629    /// substitutes a `spec.identity`-declared creation anchor for the
1630    /// metadata slot on adopted resources) lands at ONE substrate
1631    /// method here and all three downstream consumers pick up the
1632    /// upgrade mechanically — no per-callsite hand-edit at `evaluate`
1633    /// / `requeue_with_ttl` / `reconcile_process_table`.
1634    ///
1635    /// Theory anchor: THEORY.md §VI.1 (generation over composition —
1636    /// the `.metadata.creation_timestamp.as_ref().map(|t| t.0)` chain
1637    /// recurred at three hand-authored sites past the ★★
1638    /// PRIME-DIRECTIVE ≥ 2 duplication trigger, and is lifted to ONE
1639    /// owner here). THEORY.md §II.1 invariant 5 (composition preserves
1640    /// proofs — the pins bind the missing-timestamp corner + the
1641    /// present-timestamp corner + the copy-form `DateTime<Utc>` return
1642    /// + the byte-identical parity with the pre-lift `.as_ref().map(|t|
1643    /// t.0)` chain, so a regression that drifted any surface at
1644    /// `tests::created_at_*` rather than as silent operator-facing
1645    /// skew between the TTL-expiry gate, the requeue-budget picker,
1646    /// and the stable-name claim-arbiter tie-break on the SAME
1647    /// `Process` within one reconcile pass).
1648    pub fn created_at(&self) -> Option<DateTime<Utc>> {
1649        self.metadata.creation_timestamp.as_ref().map(|t| t.0)
1650    }
1651
1652    /// Pure composer over [`Self::created_at`] that folds the paired
1653    /// `.unwrap_or(fallback)` sink into ONE substrate owner — the
1654    /// ONE-liner collapse of the paired
1655    /// `p.created_at().unwrap_or_else(Utc::now)` incantation the two
1656    /// production consumers restated by hand pre-lift, with the
1657    /// wall-clock read kept at the callsite (as `Utc::now()` passed in
1658    /// positionally) so the primitive itself stays pure — matching the
1659    /// discipline every peer `observed_*` / `created_at` copy-form
1660    /// projection follows and the explicit warning against a
1661    /// substrate-injected `Utc::now()` fallback that
1662    /// [`Self::observed_phase_since`]'s doc already spelled out.
1663    ///
1664    /// Pre-lift the paired 2-step
1665    /// `.created_at().unwrap_or_else(Utc::now)` chain was hand-authored
1666    /// at TWO production sites past the ★★ PRIME-DIRECTIVE ≥ 2
1667    /// duplication threshold, both stamping the SAME wall-clock
1668    /// fallback on the same missing-timestamp corner:
1669    /// * `tatara-reconciler::table_controller::reconcile_process_table` —
1670    ///   the per-Process claim-row's `created_at` anchor that feeds
1671    ///   the stable-name group's tie-break comparator; a freshly-forked
1672    ///   Process whose API server has not yet stamped
1673    ///   `metadata.creationTimestamp` gets `Utc::now()` synthesized so
1674    ///   the tie-break sorts by "just-created" order rather than
1675    ///   short-circuiting on the missing slot.
1676    /// * `tatara-pool-reconciler::controller_pool::reconcile_inner`'s
1677    ///   desired-count `PoolMemberSnapshot { created_at, .. }` seed —
1678    ///   the per-owned-Process snapshot fed to
1679    ///   `decide_pool_convergence`, whose stability arithmetic
1680    ///   subtracts the anchor from `now` to compute the observed dwell
1681    ///   time; the same "just-created" fallback keeps a freshly-spawned
1682    ///   pool member from being reaped as if it were a stale zombie.
1683    ///
1684    /// Both sites walked the SAME `.unwrap_or_else(Utc::now)` tail on
1685    /// the SAME [`Self::created_at`] pure projection and both wanted
1686    /// the resolved `DateTime<Utc>` the composer returns. Post-lift
1687    /// each callsite reads `p.created_at_or(Utc::now())` and the
1688    /// produced value feeds the same downstream slot unchanged.
1689    ///
1690    /// The `fallback: DateTime<Utc>` parameter (rather than a
1691    /// substrate-injected `Utc::now()`) keeps the composer pure — a
1692    /// test with a fixed-clock harness passes its own frozen anchor, a
1693    /// production consumer passes `Utc::now()`, both go through the
1694    /// same primitive without the composer itself reaching for the
1695    /// wall clock. This resolves the tension the sibling
1696    /// [`Self::observed_phase_since`]'s doc spelled out (a buried
1697    /// `Utc::now()` fallback "would fold an impure wall-clock read
1698    /// into the primitive, breaking the pure-projection discipline
1699    /// every peer `observed_*` accessor follows") by lifting the
1700    /// composition shape, not the wall-clock read.
1701    ///
1702    /// Return-form axis: `DateTime<Utc>` matches the `unwrap_or`-style
1703    /// composer discipline of `Option::unwrap_or` in std — takes the
1704    /// pure projection, an owned fallback, returns the resolved owned
1705    /// value. Peer to the substrate composers
1706    /// [`Self::observed_phase_or_pending`] on the status-phase axis and
1707    /// [`Self::coordinates_or_defaults`] on the metadata-coordinate
1708    /// axis; all three lift a `.unwrap_or(<fallback>)` tail into ONE
1709    /// substrate site so the fallback-shape decision lives at ONE
1710    /// owner per axis.
1711    ///
1712    /// A future normalization step (a per-cluster clock-skew guard
1713    /// that offsets the returned timestamp by the observing
1714    /// controller's measured skew before applying the fallback, a
1715    /// canonicalization pass that folds a suspiciously-zero
1716    /// `creationTimestamp` to the fallback rather than accepting it,
1717    /// a per-namespace override that substitutes a `spec.identity`-
1718    /// declared creation anchor for the metadata slot on adopted
1719    /// resources) lands at ONE substrate method here and both
1720    /// downstream consumers pick up the upgrade mechanically — no
1721    /// per-callsite hand-edit at `reconcile_process_table` /
1722    /// `reconcile_inner`.
1723    ///
1724    /// Theory anchor: THEORY.md §VI.1 (generation over composition —
1725    /// the paired `.created_at().unwrap_or_else(Utc::now)` chain
1726    /// recurred at two hand-authored sites past the ★★
1727    /// PRIME-DIRECTIVE ≥ 2 duplication trigger, and is lifted to ONE
1728    /// owner here). THEORY.md §II.1 invariant 5 (composition
1729    /// preserves proofs — the pins bind the missing-slot fallback
1730    /// corner + the populated-slot pass-through + the pure-composer
1731    /// discipline + the byte-identical parity with the pre-lift
1732    /// `.unwrap_or(fallback)` chain, so a regression that drifted
1733    /// any surface at `tests::created_at_or_*` rather than as
1734    /// silent operator-facing skew between the claim-arbiter's
1735    /// tie-break anchor and the pool convergence snapshot's dwell-time
1736    /// anchor on the SAME `Process` within one reconcile pass).
1737    #[must_use]
1738    pub fn created_at_or(&self, fallback: DateTime<Utc>) -> DateTime<Utc> {
1739        self.created_at().unwrap_or(fallback)
1740    }
1741
1742    /// Compound spec-projection primitive on the `spec.lifetime` axis:
1743    /// returns `Some(&e)` iff the resolver unambiguously picks the
1744    /// `Ephemeral` slot, `None` otherwise — the ONE-liner collapse of
1745    /// the 4-step `self.spec.lifetime.resolved_ephemeral()` chain the
1746    /// two `lifetime_clock` consumers previously reached through and
1747    /// the coherence-tightening lift of the naked
1748    /// `self.spec.lifetime.ephemeral.as_ref()` raw-field access
1749    /// `tatara_reconciler::render::render_export_jobs` previously
1750    /// walked past.
1751    ///
1752    /// Pre-lift THREE consumer sites past the ★★ PRIME-DIRECTIVE ≥ 2
1753    /// duplication threshold reached the ephemeral inner through TWO
1754    /// different chains that disagreed on the ambiguous corner:
1755    /// * `tatara_process::lifetime_clock::evaluate` — ambiguity-aware:
1756    ///   `process.spec.lifetime.resolved_ephemeral()` collapses BOTH-
1757    ///   slots-set to `None`, matching the "no ephemeral action"
1758    ///   outcome (`AutoTerminate::Skip`) the ambiguous case must yield.
1759    /// * `tatara_process::lifetime_clock::requeue_with_ttl` —
1760    ///   ambiguity-aware peer of `evaluate`; both share the SAME
1761    ///   `resolved_ephemeral()` gate and MUST agree on the ambiguous
1762    ///   corner or the reconciler's teardown decision and requeue-
1763    ///   budget picker drift apart on the SAME `Process` within one
1764    ///   reconcile pass.
1765    /// * `tatara_reconciler::render::render_export_jobs` — RAW field
1766    ///   access: `process.spec.lifetime.ephemeral.as_ref()` returned
1767    ///   `Some(&e)` on the ambiguous corner, so an operator-authored
1768    ///   `Process` with BOTH `permanent:` AND `ephemeral:` slots
1769    ///   populated would emit export Jobs whose teardown-triggered
1770    ///   fire semantics `lifetime_clock` refused to honor. The two
1771    ///   consumers drifted at the mis-configuration corner.
1772    ///
1773    /// Post-lift ALL THREE consumers reach through ONE `Process` method
1774    /// that composes `self.spec.lifetime.resolved_ephemeral()` — the
1775    /// ambiguity-aware `variant().ok() + as_ephemeral` chain
1776    /// [`crate::lifetime::Lifetime::resolved_ephemeral`] owns — and
1777    /// the drift between the reconciler's export-render arm and the
1778    /// lifetime clock's teardown/TTL arm CLOSES at ONE substrate site.
1779    ///
1780    /// Return-form axis: `Option<&EphemeralLifetime>` matches the
1781    /// borrow-form discipline of the underlying
1782    /// [`crate::lifetime::Lifetime::resolved_ephemeral`] projection so
1783    /// the borrow carries the `'_self` lifetime through directly
1784    /// without a temporary `LifetimeVariant` binding. Peer to the
1785    /// borrow-form status-projection primitives
1786    /// [`Self::observed_attestation`], [`Self::observed_identity`] and
1787    /// the borrow-form metadata-projection primitive
1788    /// [`Self::uid_or_empty`] — all four hide a wrapping `Option`-
1789    /// carrying wire slot behind an inherent projection.
1790    ///
1791    /// A future normalization step (a canonicalization pass that maps
1792    /// a suspiciously-zero `ttl` to a per-cluster default, a per-
1793    /// namespace override that substitutes an operator-declared
1794    /// teardown policy on adopted resources, a wire-schema migration
1795    /// that renames `spec.lifetime.ephemeral` to `spec.lifetime.timed`
1796    /// with a bridging `From` shim) lands at ONE substrate method here
1797    /// and all three downstream consumers pick up the upgrade
1798    /// mechanically — no per-callsite hand-edit at `evaluate` /
1799    /// `requeue_with_ttl` / `render_export_jobs`.
1800    ///
1801    /// Theory anchor: THEORY.md §II.1 invariant 5 (composition
1802    /// preserves proofs — the pins bind the Permanent-only corner, the
1803    /// Ephemeral-only corner, the Both-set-ambiguous corner, the
1804    /// empty-default corner, and the byte-identity parity with the
1805    /// underlying `self.spec.lifetime.resolved_ephemeral()` delegate,
1806    /// so a regression that silently swapped the projection back to
1807    /// the raw `.ephemeral.as_ref()` field would surface here rather
1808    /// than as operator-facing drift between the export-render arm
1809    /// and the teardown/TTL arm on the SAME `Process`). THEORY.md
1810    /// §VI.1 (generation over composition — the ambiguity-aware
1811    /// projection recurred at three hand-authored sites past the ★★
1812    /// PRIME-DIRECTIVE ≥ 2 duplication trigger, and is lifted to ONE
1813    /// owner here).
1814    pub fn resolved_ephemeral(&self) -> Option<&EphemeralLifetime> {
1815        self.spec.lifetime.resolved_ephemeral()
1816    }
1817}
1818
1819impl ProcessSpec {
1820    /// Canonical minimum [`ProcessSpec`] — a [`Classification::gate_compute`]
1821    /// classification with every other field parked at its [`Default`] —
1822    /// the workspace-baseline spec every consumer that needed "a
1823    /// `ProcessSpec` that just exists, with no domain-specific claim on
1824    /// intent / boundary / lifetime / routing / encapsulates" hand-authored
1825    /// as a 12-line struct-literal at scattered sites across the workspace.
1826    ///
1827    /// The composition is the two-primitive product of
1828    /// [`Classification::gate_compute`] (the two axes with no `Default` — a
1829    /// `Gate` point on the `Compute` substrate) with the `Default` impl on
1830    /// every other slot: [`IdentitySpec`], [`Intent`], [`Boundary`],
1831    /// [`ComplianceSpec`], `Vec<DependsOn>`, [`SignalPolicy`], [`Lifetime`],
1832    /// `Option<RoutingSpec>`, `Option<EncapsulatesSpec>`, `bool`. The 11
1833    /// defaulted axes ride at the sibling closed-set + `#[serde(default)]`
1834    /// defaults the CRD already owns; the two `_or_default` /
1835    /// `_or_placeholder` corners on the metadata axis stay closed at the
1836    /// substrate ([`Process::coordinates_or_defaults`],
1837    /// [`Process::name_or_placeholder`]) since this primitive builds the
1838    /// `spec` half, not the `metadata` half.
1839    ///
1840    /// Pre-lift the 12-line `ProcessSpec { identity: <Default>,
1841    /// classification: Classification::gate_compute(), intent: <Default>,
1842    /// boundary: Default::default(), compliance: Default::default(),
1843    /// depends_on: vec![], signals: Default::default(), lifetime:
1844    /// Default::default(), routing: None, encapsulates: None, suspended:
1845    /// false }` struct-literal recurred at EIGHT hand-authored sites past
1846    /// the ★★ PRIME-DIRECTIVE ≥ 2 duplication threshold across four
1847    /// crates, each restating the SAME 12-slot verbatim:
1848    /// * `tatara-process::crd::tests::empty_spec` — the substrate test
1849    ///   fixture that pins every `Process::*_or_*` metadata-projection
1850    ///   primitive on the (return-form × fallback-shape) axis;
1851    /// * `tatara-process::lib::tests::empty_process_spec` (×2) — the
1852    ///   sibling fixture inside the `qualified_process_ref` +
1853    ///   `DeletionTombstoned` / `Annotated` trait pin modules;
1854    /// * `tatara-process::lib::tests` (one inline site in the
1855    ///   `qualified_process_ref_composes_from_process_coordinates_or_defaults`
1856    ///   pin) — restated the SAME 12-line block inside the test body;
1857    /// * `tatara-reconciler::claim::tests::empty_process` — the claim-
1858    ///   arbiter row-builder pin fixture;
1859    /// * `tatara-pool-reconciler::controller_pool::tests` (×3) — the
1860    ///   `empty_spec` fixture + two inline `process_to_member_state_*` pin
1861    ///   sites that hand-composed the same 12-slot spec inline.
1862    ///
1863    /// Five more sites walked the SAME 12-slot shape but overrode ONE
1864    /// field (intent, lifetime, or routing) inline and are lifted onto
1865    /// the primitive via struct-update syntax
1866    /// (`..ProcessSpec::gate_compute_defaults()`): the three
1867    /// `tatara-reconciler::render` test-fixture helpers
1868    /// (`render_through_top_level_intent_dispatch`, `process_with`,
1869    /// `demo_process`) and the two `tatara-process::lifetime_clock`
1870    /// helpers (`ephemeral_process`, `permanent_process`).
1871    ///
1872    /// Post-lift each callsite reads `ProcessSpec::gate_compute_defaults()`
1873    /// (or `ProcessSpec { <slot>: <value>,
1874    /// ..ProcessSpec::gate_compute_defaults() }` for the override sites);
1875    /// a future workspace-wide baseline shift (a new `#[serde(default)]`
1876    /// on a promoted [`Intent`] variant, a rename of a defaulted slot, a
1877    /// per-baseline compliance overlay stamping through the spec) lands
1878    /// at ONE substrate function here and every downstream consumer
1879    /// inherits the upgrade mechanically. The current pin ties the
1880    /// classification axis to the sibling [`Classification::gate_compute`]
1881    /// primitive so a future change to that baseline surfaces at this
1882    /// primitive's tests rather than as silent drift across thirteen
1883    /// independent callsites.
1884    ///
1885    /// Sibling to [`Classification::gate_compute`] on the composition-
1886    /// depth axis — that primitive owns the ONE-axis-slice construction
1887    /// (the 5-slot [`Classification`] value); this primitive owns the
1888    /// FULL-spec construction (the 11-slot [`ProcessSpec`] value that
1889    /// wraps the classification-slice plus every other slot at
1890    /// `Default`). A future peer `ProcessSpec::observability_stack()` or
1891    /// similar named variant lands as a sibling method here when a
1892    /// second unremarkable-baseline shape opens.
1893    ///
1894    /// Theory anchor: THEORY.md §VI.1 (generation over composition — the
1895    /// 12-line struct-literal shape recurred at EIGHT hand-authored sites
1896    /// past the ★★ PRIME-DIRECTIVE ≥ 2 duplication trigger and is lifted
1897    /// onto ONE workspace-wide owner here). THEORY.md §II.1 invariant 5
1898    /// (composition preserves proofs — a regression that drifted the
1899    /// baseline axis choice at only one consumer, or that broke the
1900    /// sibling-default correspondence with [`Classification::gate_compute`],
1901    /// surfaces at this primitive's tests rather than as silent operator-
1902    /// visible skew between the eight exact-match test-fixtures + the
1903    /// five override sites whose struct-update composition depends on the
1904    /// shape).
1905    #[must_use]
1906    pub fn gate_compute_defaults() -> Self {
1907        Self {
1908            identity: IdentitySpec::default(),
1909            classification: Classification::gate_compute(),
1910            intent: Intent::default(),
1911            boundary: Boundary::default(),
1912            compliance: ComplianceSpec::default(),
1913            depends_on: Vec::new(),
1914            signals: SignalPolicy::default(),
1915            lifetime: Lifetime::default(),
1916            routing: None,
1917            encapsulates: None,
1918            suspended: false,
1919        }
1920    }
1921}
1922
1923/// Process status — every field optional until the reconciler writes it.
1924#[derive(Clone, Debug, Default, Deserialize, Serialize, JsonSchema)]
1925#[serde(rename_all = "camelCase")]
1926pub struct ProcessStatus {
1927    /// Hierarchical PID path — e.g., `"seph.1.7"`.
1928    #[serde(default, skip_serializing_if = "Option::is_none")]
1929    pub pid: Option<String>,
1930
1931    /// Parent PID path (mirror of `spec.identity.parent`, resolved at fork).
1932    #[serde(default, skip_serializing_if = "Option::is_none")]
1933    pub parent: Option<String>,
1934
1935    /// Direct children's PID paths.
1936    #[serde(default)]
1937    pub children: Vec<String>,
1938
1939    /// Resolved identity (name + content hash).
1940    #[serde(default, skip_serializing_if = "Option::is_none")]
1941    pub identity: Option<Identity>,
1942
1943    /// Current phase.
1944    #[serde(default)]
1945    pub phase: ProcessPhase,
1946
1947    /// When the process entered the current phase.
1948    #[serde(default, skip_serializing_if = "Option::is_none")]
1949    pub phase_since: Option<DateTime<Utc>>,
1950
1951    /// Three-pillar attestation (written at end of every successful cycle).
1952    #[serde(default, skip_serializing_if = "Option::is_none")]
1953    pub attestation: Option<ProcessAttestation>,
1954
1955    /// FluxCD resources currently owned by this Process.
1956    #[serde(default)]
1957    pub flux_resources: Vec<FluxResourceRef>,
1958
1959    /// Boundary verification state.
1960    #[serde(default)]
1961    pub boundary: BoundaryStatus,
1962
1963    /// Compliance summary at the latest attestation.
1964    #[serde(default)]
1965    pub compliance: ComplianceStatus,
1966
1967    /// Pending signals (delivered, not yet handled).
1968    #[serde(default)]
1969    pub signal_queue: Vec<ProcessSignal>,
1970
1971    /// Standard K8s Conditions.
1972    #[serde(default)]
1973    pub conditions: Vec<ProcessCondition>,
1974
1975    /// Human-readable last status message.
1976    #[serde(default, skip_serializing_if = "Option::is_none")]
1977    pub message: Option<String>,
1978
1979    /// Exit code (only set on Failed / Reaped).
1980    #[serde(default, skip_serializing_if = "Option::is_none")]
1981    pub exit_code: Option<i32>,
1982}
1983
1984#[cfg(test)]
1985mod tests {
1986    use super::*;
1987    use crate::classification::{ConvergencePointType, SubstrateType};
1988    use crate::intent::NixIntent;
1989
1990    #[test]
1991    fn minimal_spec_serializes() {
1992        let spec = ProcessSpec {
1993            identity: IdentitySpec::default(),
1994            classification: Classification {
1995                point_type: ConvergencePointType::Gate,
1996                substrate: SubstrateType::Observability,
1997                horizon: Default::default(),
1998                calm: Default::default(),
1999                data_classification: Default::default(),
2000            },
2001            intent: Intent {
2002                nix: Some(NixIntent {
2003                    flake_ref: "github:pleme-io/k8s".into(),
2004                    attribute: "obs".into(),
2005                    system: None,
2006                    attic_cache: None,
2007                    extra_args: vec![],
2008                    delegate_to_nix_build: false,
2009                }),
2010                ..Intent::default()
2011            },
2012            boundary: Default::default(),
2013            compliance: Default::default(),
2014            depends_on: vec![],
2015            signals: Default::default(),
2016            lifetime: Default::default(),
2017            routing: None,
2018            encapsulates: None,
2019            suspended: false,
2020        };
2021        let yaml = serde_yaml::to_string(&spec).unwrap();
2022        assert!(yaml.contains("pointType: Gate"));
2023        assert!(yaml.contains("substrate: Observability"));
2024        assert!(yaml.contains("flakeRef: github:pleme-io/k8s"));
2025    }
2026
2027    // ─── Process::coordinates_or_defaults substrate pins ────────────────
2028    //
2029    // Pins the (namespace, name) coordinate-primitive family on the
2030    // (metadata slot × fallback shape) axis. Fail-before-pass-after
2031    // granularity: a regression that flipped either fallback string,
2032    // swapped the return-tuple axis order, or dropped the
2033    // `Option::as_deref` unwrap surfaces here rather than as silent
2034    // drift at every downstream annotation writer / claim-arbiter row
2035    // builder / render owner-metadata seed.
2036
2037    fn empty_spec() -> ProcessSpec {
2038        // Routes through the ONE substrate composer
2039        // `ProcessSpec::gate_compute_defaults` — pre-lift this was the
2040        // 12-line struct-literal restated verbatim at every fixture in
2041        // this pin family, one of EIGHT hand-authored exact-match sites
2042        // past the ★★ PRIME-DIRECTIVE ≥ 2 duplication threshold across
2043        // four crates.
2044        ProcessSpec::gate_compute_defaults()
2045    }
2046
2047    #[test]
2048    fn default_namespace_constant_is_k8s_canonical_default() {
2049        // Pins the load-bearing convention that this primitive's
2050        // namespace fallback matches K8s's own implicit-namespace
2051        // spelling. A regression that renamed this to "kube-system"
2052        // or any other K8s-reserved name would silently misroute
2053        // every downstream namespaced-Api call on a Process without
2054        // a metadata.namespace.
2055        assert_eq!(Process::DEFAULT_NAMESPACE, "default");
2056    }
2057
2058    #[test]
2059    fn unnamed_placeholder_constant_matches_prior_annotation_writer_fallback() {
2060        // Pins the load-bearing convention that this primitive's name
2061        // fallback matches the exact spelling every annotation writer
2062        // (tatara-reconciler::ssapply::inject_annotations,
2063        // tatara-reconciler::render::render, and
2064        // tatara-reconciler::table_controller's claim-row builder)
2065        // was hand-authoring pre-lift ("unnamed", NOT "<unnamed>" or
2066        // ""). A regression that renamed this would break the
2067        // annotation-writer / claim-arbiter grep contract silently.
2068        assert_eq!(Process::UNNAMED_PLACEHOLDER, "unnamed");
2069    }
2070
2071    #[test]
2072    fn namespace_or_default_falls_back_when_metadata_namespace_is_none() {
2073        let mut p = Process::new("some-proc", empty_spec());
2074        p.metadata.namespace = None;
2075        assert_eq!(p.namespace_or_default(), Process::DEFAULT_NAMESPACE);
2076    }
2077
2078    #[test]
2079    fn namespace_or_default_returns_metadata_slice_when_some() {
2080        let mut p = Process::new("some-proc", empty_spec());
2081        p.metadata.namespace = Some("prod-app".into());
2082        assert_eq!(p.namespace_or_default(), "prod-app");
2083    }
2084
2085    #[test]
2086    fn name_or_placeholder_falls_back_when_metadata_name_is_none() {
2087        let mut p = Process::new("real-name", empty_spec());
2088        p.metadata.name = None;
2089        assert_eq!(p.name_or_placeholder(), Process::UNNAMED_PLACEHOLDER);
2090    }
2091
2092    #[test]
2093    fn name_or_placeholder_returns_metadata_slice_when_some() {
2094        let p = Process::new("api-gateway", empty_spec());
2095        assert_eq!(p.name_or_placeholder(), "api-gateway");
2096    }
2097
2098    #[test]
2099    fn coordinates_or_defaults_composes_both_halves() {
2100        // Both slots present — returns metadata slices in
2101        // (namespace, name) axis order.
2102        let mut p = Process::new("api", empty_spec());
2103        p.metadata.namespace = Some("staging".into());
2104        assert_eq!(p.coordinates_or_defaults(), ("staging", "api"));
2105    }
2106
2107    #[test]
2108    fn coordinates_or_defaults_falls_back_on_both_slots() {
2109        // Both slots None — returns (DEFAULT_NAMESPACE,
2110        // UNNAMED_PLACEHOLDER) in axis order.
2111        let mut p = Process::new("scratch", empty_spec());
2112        p.metadata.name = None;
2113        p.metadata.namespace = None;
2114        assert_eq!(
2115            p.coordinates_or_defaults(),
2116            (Process::DEFAULT_NAMESPACE, Process::UNNAMED_PLACEHOLDER)
2117        );
2118    }
2119
2120    #[test]
2121    fn coordinates_or_defaults_mixes_slotted_and_fallback_halves() {
2122        // Namespace set, name missing — the (namespace, name) tuple
2123        // pins each half independently. A regression that returned
2124        // BOTH fallbacks when EITHER metadata slot was None would
2125        // surface here rather than at every downstream reader.
2126        let mut p = Process::new("kept-name", empty_spec());
2127        p.metadata.namespace = Some("prod".into());
2128        assert_eq!(p.coordinates_or_defaults(), ("prod", "kept-name"));
2129
2130        // Name set, namespace missing — the peer corner.
2131        let mut q = Process::new("api", empty_spec());
2132        q.metadata.namespace = None;
2133        assert_eq!(
2134            q.coordinates_or_defaults(),
2135            (Process::DEFAULT_NAMESPACE, "api")
2136        );
2137    }
2138
2139    // ─── Process::qualified_ref substrate pins ─────────────────────────
2140    //
2141    // Pins the paired-projection + shape-composer chain
2142    // `coordinates_or_defaults() → qualified_process_ref(ns, name)` on
2143    // the (return-form × composition-depth) axis pair. Fail-before-
2144    // pass-after granularity: a regression that swapped the `<ns>/<name>`
2145    // axis order, dropped either half, drifted the fallback strings
2146    // between the paired-projection primitive and the shape composer, or
2147    // inserted a normalization step at only the composed site and not
2148    // the pair-returning primitive (or vice versa) surfaces here rather
2149    // than as silent operator-visible skew across the three pre-lift
2150    // `tatara-reconciler` sites (`render::render_routing`,
2151    // `render::render_export_jobs`, `table_controller::reconcile`)
2152    // whose downstream greps the reference shape verbatim (the
2153    // `PROCESS=<ref>` annotation seed on every emitted Ingress /
2154    // DNSEndpoint / export Job, the `ClaimRecord.holder` slot on the
2155    // stable-name claim registry).
2156
2157    #[test]
2158    fn qualified_ref_composes_ns_and_name_with_slash_when_both_slots_present() {
2159        // Happy path — both metadata slots populated. The composed
2160        // reference is EXACTLY `<ns>/<name>`, in that order, joined by
2161        // a single `/`. A regression that swapped the two axes at
2162        // this primitive would silently break every downstream
2163        // `PROCESS=<ref>` annotation grep + claim-registry lookup.
2164        let mut p = Process::new("api-gateway", empty_spec());
2165        p.metadata.namespace = Some("prod-app".into());
2166        assert_eq!(p.qualified_ref(), "prod-app/api-gateway");
2167    }
2168
2169    #[test]
2170    fn qualified_ref_falls_back_to_default_namespace_when_metadata_namespace_is_none() {
2171        // Namespace-fallback pin: an absent `metadata.namespace` rides
2172        // through `namespace_or_default()` → `DEFAULT_NAMESPACE`, so
2173        // the composed reference lands as `default/<name>`. Matches
2174        // what a pre-lift `qualified_process_ref(process.
2175        // coordinates_or_defaults())` composition produced.
2176        let mut p = Process::new("api-gateway", empty_spec());
2177        p.metadata.namespace = None;
2178        assert_eq!(p.qualified_ref(), "default/api-gateway");
2179    }
2180
2181    #[test]
2182    fn qualified_ref_falls_back_to_unnamed_placeholder_when_metadata_name_is_none() {
2183        // Name-fallback pin: an absent `metadata.name` rides through
2184        // `name_or_placeholder()` → `UNNAMED_PLACEHOLDER`, so the
2185        // composed reference lands as `<ns>/unnamed`. A pre-lift
2186        // consumer whose paired projection returned the placeholder
2187        // (annotation writer, render owner-metadata seed) sees the
2188        // exact same `<ns>/unnamed` shape post-lift, so downstream
2189        // greps keyed on the pre-metadata Process's reference match
2190        // bytewise.
2191        let mut p = Process::new("ignored", empty_spec());
2192        p.metadata.namespace = Some("staging".into());
2193        p.metadata.name = None;
2194        assert_eq!(p.qualified_ref(), "staging/unnamed");
2195    }
2196
2197    #[test]
2198    fn qualified_ref_falls_back_on_both_slots_when_both_metadata_are_none() {
2199        // Both slots absent → both fallbacks land in the composed
2200        // reference. The `default/unnamed` shape is what every pre-
2201        // lift caller produced when a Process fixture (test or
2202        // dynamic API response) surfaced without populated metadata;
2203        // pinning it here holds the primitive's contract against a
2204        // regression that dropped either fallback at only the
2205        // composed site.
2206        let mut p = Process::new("ignored", empty_spec());
2207        p.metadata.namespace = None;
2208        p.metadata.name = None;
2209        assert_eq!(
2210            p.qualified_ref(),
2211            format!(
2212                "{}/{}",
2213                Process::DEFAULT_NAMESPACE,
2214                Process::UNNAMED_PLACEHOLDER
2215            )
2216        );
2217    }
2218
2219    #[test]
2220    fn qualified_ref_matches_pre_lift_paired_composition_bytewise() {
2221        // Byte-identical parity with the exact pre-lift 2-step
2222        // composition every `tatara-reconciler` site hand-authored:
2223        // `let (ns, name) = process.coordinates_or_defaults(); let r
2224        // = qualified_process_ref(ns, name);`. Sweeps every metadata-
2225        // slot combination the three pre-lift consumers plausibly
2226        // encountered — both slots populated (steady state), one
2227        // slot absent (Process mid-fork before API-server metadata
2228        // stamp), both slots absent (dynamic API response / test
2229        // fixture) — so a regression that reshaped the composition at
2230        // the substrate primitive would surface here rather than as
2231        // silent drift at the three consumer sites.
2232        let fixtures: [(Option<&str>, Option<&str>); 4] = [
2233            (Some("prod-app"), Some("api-gateway")),
2234            (None, Some("api-gateway")),
2235            (Some("staging"), None),
2236            (None, None),
2237        ];
2238        for (ns_slot, name_slot) in fixtures {
2239            let mut p = Process::new(name_slot.unwrap_or("seed"), empty_spec());
2240            p.metadata.namespace = ns_slot.map(str::to_string);
2241            p.metadata.name = name_slot.map(str::to_string);
2242            let via_primitive = p.qualified_ref();
2243            let (ns, name) = p.coordinates_or_defaults();
2244            let via_paired = crate::qualified_process_ref(ns, name);
2245            assert_eq!(
2246                via_primitive, via_paired,
2247                "qualified_ref must be byte-identical to the pre-lift \
2248                 paired composition on (ns={ns_slot:?}, name={name_slot:?})"
2249            );
2250        }
2251    }
2252
2253    #[test]
2254    fn qualified_ref_composes_from_the_shared_coordinates_or_defaults_owner() {
2255        // Composition invariant: the composed reference decomposes at
2256        // the single `/` separator into EXACTLY the (ns, name) pair
2257        // `coordinates_or_defaults` returns. A regression that
2258        // introduced a per-callsite normalization at the shape
2259        // composer (URL-escape, case-fold, path-normalize) or that
2260        // pulled the pair from a different metadata source than the
2261        // paired-projection primitive would surface here rather than
2262        // at every downstream reference-shape grep.
2263        let mut p = Process::new("api-gateway", empty_spec());
2264        p.metadata.namespace = Some("prod-app".into());
2265        let composed = p.qualified_ref();
2266        let (ns, name) = p.coordinates_or_defaults();
2267        let (composed_ns, composed_name) = composed.split_once('/').unwrap();
2268        assert_eq!(composed_ns, ns);
2269        assert_eq!(composed_name, name);
2270    }
2271
2272    // ─── Process::owned_coordinates_or_err substrate pins ──────────────
2273    //
2274    // Pins the owned + name-required peer of the coordinate-primitive
2275    // family on the (return-form × name gate) axis pair. Fail-before-
2276    // pass-after granularity: a regression that flipped the namespace
2277    // fallback string, dropped the `Option::clone` unwrap, changed the
2278    // return-tuple axis order, or altered the "Process has no
2279    // metadata.name" error wording surfaces here rather than as silent
2280    // drift at every pre-lift caller (10 sites in
2281    // `tatara-reconciler::phase_machine` + 2 sites in
2282    // `tatara-reconciler::signals` pre-lift).
2283
2284    #[test]
2285    fn owned_coordinates_or_err_returns_owned_strings_when_both_slots_present() {
2286        // Happy path — both slots populated, method returns owned
2287        // Strings in (namespace, name) axis order.
2288        let mut p = Process::new("api-gateway", empty_spec());
2289        p.metadata.namespace = Some("prod-app".into());
2290        let (ns, name) = p.owned_coordinates_or_err().unwrap();
2291        assert_eq!(ns, "prod-app");
2292        assert_eq!(name, "api-gateway");
2293        // Ownership pin: type inference above binds ns/name as
2294        // owned Strings — a regression that returned &str would
2295        // fail to compile at the following .push() call. This
2296        // holds the "owned" half of the primitive's contract.
2297        let mut owned_ns = ns;
2298        owned_ns.push_str("-mutated");
2299        assert_eq!(owned_ns, "prod-app-mutated");
2300    }
2301
2302    #[test]
2303    fn owned_coordinates_or_err_falls_back_on_namespace_but_returns_owned_name() {
2304        // Namespace absent → DEFAULT_NAMESPACE. Name present → owned.
2305        let p = Process::new("api", empty_spec());
2306        // Process::new leaves metadata.namespace = None by default.
2307        let (ns, name) = p.owned_coordinates_or_err().unwrap();
2308        assert_eq!(ns, Process::DEFAULT_NAMESPACE);
2309        assert_eq!(name, "api");
2310    }
2311
2312    #[test]
2313    fn owned_coordinates_or_err_errors_when_metadata_name_absent_regardless_of_namespace() {
2314        // Name absent → Err, REGARDLESS of whether the namespace is
2315        // populated. The name gate is strictly on `metadata.name` and
2316        // does NOT fall back to `Self::UNNAMED_PLACEHOLDER` (that
2317        // fallback is on the peer `coordinates_or_defaults`, which
2318        // exists precisely for consumers that can tolerate a
2319        // display placeholder).
2320        for ns_slot in [None, Some("prod".to_string())] {
2321            let mut p = Process::new("scratch", empty_spec());
2322            p.metadata.name = None;
2323            p.metadata.namespace = ns_slot.clone();
2324            let err = p.owned_coordinates_or_err().unwrap_err();
2325            assert!(
2326                err.to_string().contains("metadata.name"),
2327                "err on missing name (ns={ns_slot:?}) should mention metadata.name; got {err}"
2328            );
2329        }
2330    }
2331
2332    #[test]
2333    fn owned_coordinates_or_err_error_message_matches_pre_lift_reconciler_wording() {
2334        // Load-bearing wording pin — every pre-lift `tatara-reconciler`
2335        // helper (`phase_machine::namespace_and_name`,
2336        // `signals::ingest`, `signals::consume_effect`) errored with
2337        // EXACTLY this wording. Post-lift the substrate owner produces
2338        // the same wording so log-line / test greps that anchored on
2339        // it keep matching, and no operator-visible message drift
2340        // lands as a side effect of the substrate move.
2341        let mut p = Process::new("scratch", empty_spec());
2342        p.metadata.name = None;
2343        let err = p.owned_coordinates_or_err().unwrap_err();
2344        assert_eq!(err.to_string(), "Process has no metadata.name");
2345    }
2346
2347    #[test]
2348    fn owned_coordinates_or_err_namespace_fallback_matches_default_namespace_const() {
2349        // Byte-identity pin between the owned form's namespace
2350        // fallback and the workspace-wide `DEFAULT_NAMESPACE` const.
2351        // A regression that spelled this fallback as any other
2352        // string ("kube-system", "", "default-ns") would silently
2353        // misroute every downstream namespaced-Api call on a
2354        // Process without a metadata.namespace — surfaces here
2355        // rather than at every kube-rs API caller.
2356        let mut p = Process::new("api", empty_spec());
2357        p.metadata.namespace = None;
2358        let (ns, _) = p.owned_coordinates_or_err().unwrap();
2359        assert_eq!(ns, Process::DEFAULT_NAMESPACE);
2360    }
2361
2362    #[test]
2363    fn owned_coordinates_or_err_matches_pre_lift_reconciler_helper_shape() {
2364        // Byte-identical parity pin between the owned + name-required
2365        // primitive here and the pre-lift `tatara-reconciler` helper
2366        // shape — the exact 2-slot unwrap chain each pre-lift caller
2367        // spelled by hand:
2368        //
2369        //   let ns = p.metadata.namespace.clone().unwrap_or_else(|| "default".into());
2370        //   let name = p.metadata.name.clone().ok_or_else(|| anyhow!(...))?;
2371        //   Ok((ns, name))
2372        //
2373        // Sweeps every corner every callsite plausibly encounters
2374        // (both slots present, namespace absent, name absent, both
2375        // absent). A regression that inserted a normalization step
2376        // at the primitive that the pre-lift chain does NOT apply —
2377        // or vice versa — surfaces here rather than as silent drift
2378        // between the 12 pre-lift consumer callsites and the ONE
2379        // substrate owner they now route through.
2380        fn pre_lift(p: &Process) -> anyhow::Result<(String, String)> {
2381            let ns = p
2382                .metadata
2383                .namespace
2384                .clone()
2385                .unwrap_or_else(|| "default".into());
2386            let name = p
2387                .metadata
2388                .name
2389                .clone()
2390                .ok_or_else(|| anyhow::anyhow!("Process has no metadata.name"))?;
2391            Ok((ns, name))
2392        }
2393        // Both present.
2394        let mut p = Process::new("api", empty_spec());
2395        p.metadata.namespace = Some("prod".into());
2396        assert_eq!(p.owned_coordinates_or_err().unwrap(), pre_lift(&p).unwrap());
2397        // Namespace absent.
2398        let p = Process::new("api", empty_spec());
2399        assert_eq!(p.owned_coordinates_or_err().unwrap(), pre_lift(&p).unwrap());
2400        // Name absent → both variants error with the same wording.
2401        let mut p = Process::new("api", empty_spec());
2402        p.metadata.name = None;
2403        p.metadata.namespace = Some("prod".into());
2404        assert_eq!(
2405            p.owned_coordinates_or_err().unwrap_err().to_string(),
2406            pre_lift(&p).unwrap_err().to_string(),
2407        );
2408        // Both absent → still errors on the name gate.
2409        let mut p = Process::new("api", empty_spec());
2410        p.metadata.name = None;
2411        p.metadata.namespace = None;
2412        assert_eq!(
2413            p.owned_coordinates_or_err().unwrap_err().to_string(),
2414            pre_lift(&p).unwrap_err().to_string(),
2415        );
2416    }
2417
2418    #[test]
2419    fn owned_coordinates_or_err_axis_order_matches_coordinates_or_defaults() {
2420        // Cross-primitive coherence pin between the owned + name-
2421        // required form and the borrow + name-defaulted peer:
2422        // (namespace, name) axis order is IDENTICAL across both
2423        // return-forms. A regression that swapped the tuple slots on
2424        // only ONE of the two primitives would silently misroute
2425        // every consumer that picked between the two forms based on
2426        // its callsite's ownership needs. The pin re-reads both
2427        // primitives at test time so the equality holds iff both
2428        // live paths are the current implementation.
2429        let mut p = Process::new("app", empty_spec());
2430        p.metadata.namespace = Some("infra".into());
2431        let (borrow_ns, borrow_name) = p.coordinates_or_defaults();
2432        let (owned_ns, owned_name) = p.owned_coordinates_or_err().unwrap();
2433        assert_eq!(owned_ns, borrow_ns);
2434        assert_eq!(owned_name, borrow_name);
2435        // Explicit slot labels — pins the (namespace, name) axis
2436        // order as opposed to (name, namespace).
2437        assert_eq!(owned_ns, "infra"); // NOT "app"
2438        assert_eq!(owned_name, "app"); // NOT "infra"
2439    }
2440
2441    // ─── Process::coordinates_or_none substrate pins ──────────────────
2442    //
2443    // Pins the borrow + name-required peer of the coordinate-primitive
2444    // family on the (return-form × name-gate) axis pair. Closes the
2445    // corner previously left open (borrow + name-required) so the
2446    // three consumer shapes (child-Process delete-fan-out at
2447    // `phase_machine::handle_exiting`, claim-arbiter probe at
2448    // `phase_machine::process_holds_any_claim`, any future non-fatal
2449    // skip site) route through ONE primitive rather than three hand-
2450    // authored empty-string / `unwrap_or_default()` sentinel chains.
2451    // Fail-before-pass-after granularity: a regression that flipped
2452    // the namespace fallback, swapped the return-tuple axis order,
2453    // returned an owned form, or promoted a missing name to an error
2454    // rather than `None` surfaces here rather than as silent drift at
2455    // every borrow + name-required consumer.
2456
2457    #[test]
2458    fn coordinates_or_none_returns_slices_when_both_slots_present() {
2459        // Happy path — both slots populated, method returns borrowed
2460        // (&str, &str) in (namespace, name) axis order wrapped in
2461        // `Some`.
2462        let mut p = Process::new("api-gateway", empty_spec());
2463        p.metadata.namespace = Some("prod-app".into());
2464        let (ns, name) = p.coordinates_or_none().expect("Some when name set");
2465        assert_eq!(ns, "prod-app");
2466        assert_eq!(name, "api-gateway");
2467    }
2468
2469    #[test]
2470    fn coordinates_or_none_falls_back_on_namespace_but_returns_name_slice() {
2471        // Namespace absent → DEFAULT_NAMESPACE (shared with the peer
2472        // `coordinates_or_defaults` + `namespace_or_default`). Name
2473        // present → the metadata slice, wrapped in `Some`.
2474        let mut p = Process::new("api", empty_spec());
2475        p.metadata.namespace = None;
2476        let (ns, name) = p.coordinates_or_none().expect("Some when name set");
2477        assert_eq!(ns, Process::DEFAULT_NAMESPACE);
2478        assert_eq!(name, "api");
2479    }
2480
2481    #[test]
2482    fn coordinates_or_none_returns_none_when_metadata_name_absent_regardless_of_namespace() {
2483        // Name absent → `None`, REGARDLESS of whether the namespace
2484        // slot is populated. The name gate is strictly on
2485        // `metadata.name` and does NOT fall back to
2486        // `Self::UNNAMED_PLACEHOLDER` (that fallback is on the peer
2487        // `coordinates_or_defaults`, which exists precisely for
2488        // consumers that tolerate a display placeholder). Peer to
2489        // `owned_coordinates_or_err_errors_when_metadata_name_absent_regardless_of_namespace`
2490        // on the sibling primitive; a regression that widened THIS
2491        // form to substitute the placeholder while leaving the owned
2492        // form strict would silently drift the two borrow-form
2493        // primitives out of the coherence the family carries.
2494        for ns_slot in [None, Some("prod".to_string())] {
2495            let mut p = Process::new("scratch", empty_spec());
2496            p.metadata.name = None;
2497            p.metadata.namespace = ns_slot.clone();
2498            assert!(
2499                p.coordinates_or_none().is_none(),
2500                "coordinates_or_none must be None on missing name (ns={ns_slot:?})",
2501            );
2502        }
2503    }
2504
2505    #[test]
2506    fn coordinates_or_none_namespace_fallback_matches_default_namespace_const() {
2507        // Byte-identity pin between the borrow + name-required form's
2508        // namespace fallback and the workspace-wide `DEFAULT_NAMESPACE`
2509        // const. Sibling to
2510        // `owned_coordinates_or_err_namespace_fallback_matches_default_namespace_const`
2511        // on the peer primitive — the two forms MUST substitute the
2512        // same fallback string, else a consumer that switches between
2513        // them based on its ownership need silently observes a
2514        // different namespace-fallback shape as a side effect.
2515        let mut p = Process::new("api", empty_spec());
2516        p.metadata.namespace = None;
2517        let (ns, _) = p.coordinates_or_none().unwrap();
2518        assert_eq!(ns, Process::DEFAULT_NAMESPACE);
2519    }
2520
2521    #[test]
2522    fn coordinates_or_none_axis_order_matches_coordinates_or_defaults_when_name_present() {
2523        // Cross-primitive coherence pin between the two borrow-form
2524        // primitives: when the name is present, the (namespace, name)
2525        // return-tuple axis order is IDENTICAL across the two forms,
2526        // and the returned slices are the SAME `&str` view onto the
2527        // same metadata slots. A regression that swapped the tuple
2528        // slots on ONE form would silently misroute every consumer
2529        // that picked between the two forms based on its name-gate
2530        // need. The pin re-reads both primitives at test time so the
2531        // equality holds iff both live paths are the current
2532        // implementation.
2533        let mut p = Process::new("app", empty_spec());
2534        p.metadata.namespace = Some("infra".into());
2535        let (defaulted_ns, defaulted_name) = p.coordinates_or_defaults();
2536        let (required_ns, required_name) = p.coordinates_or_none().unwrap();
2537        assert_eq!(defaulted_ns, required_ns);
2538        assert_eq!(defaulted_name, required_name);
2539        // Explicit slot labels — pins the (namespace, name) axis order
2540        // as opposed to (name, namespace).
2541        assert_eq!(required_ns, "infra"); // NOT "app"
2542        assert_eq!(required_name, "app"); // NOT "infra"
2543    }
2544
2545    #[test]
2546    fn coordinates_or_none_axis_pair_diverges_from_coordinates_or_defaults_on_missing_name() {
2547        // Divergence pin between the two borrow-form primitives when
2548        // the name gate fires: `coordinates_or_defaults` substitutes
2549        // the display placeholder AND still returns a tuple;
2550        // `coordinates_or_none` returns `None`. A regression that
2551        // collapsed the two behaviors (either by dropping the gate
2552        // from the required form or by adding a `None` corner to the
2553        // defaulted form) would blur the axis pair's whole reason to
2554        // exist as two peer primitives.
2555        let mut p = Process::new("scratch", empty_spec());
2556        p.metadata.name = None;
2557        p.metadata.namespace = Some("prod".into());
2558        // Defaulted form: substitutes placeholder, no gate.
2559        assert_eq!(
2560            p.coordinates_or_defaults(),
2561            ("prod", Process::UNNAMED_PLACEHOLDER)
2562        );
2563        // Required form: gate fires, `None`.
2564        assert!(p.coordinates_or_none().is_none());
2565    }
2566
2567    #[test]
2568    fn coordinates_or_none_matches_pre_lift_reconciler_helper_shape() {
2569        // Byte-identical parity pin between the borrow + name-required
2570        // primitive here and the pre-lift `tatara-reconciler` helper
2571        // shapes — the exact 2-slot unwrap + gate chains each pre-lift
2572        // caller spelled by hand (`phase_machine::process_holds_any_claim`
2573        // spelled it as `unwrap_or("")` + `is_empty` early-return;
2574        // `phase_machine::handle_exiting`'s child-fan-out spelled it
2575        // as `unwrap_or_default()` + implicit no-op delete on the
2576        // empty API-path). Sweeps every corner every callsite plausibly
2577        // encounters (both slots present, namespace absent, name
2578        // absent + ns present, both absent). A regression that
2579        // inserted a normalization step at the primitive the pre-lift
2580        // chain does NOT apply — or vice versa — surfaces here rather
2581        // than as silent drift between the pre-lift consumer sites
2582        // and the ONE substrate owner they now route through.
2583        fn pre_lift_holds_any_claim(p: &Process) -> Option<(&str, &str)> {
2584            let ns = p.metadata.namespace.as_deref().unwrap_or("default");
2585            let name = p.metadata.name.as_deref().unwrap_or("");
2586            if name.is_empty() {
2587                return None;
2588            }
2589            Some((ns, name))
2590        }
2591        // Both present.
2592        let mut p = Process::new("api", empty_spec());
2593        p.metadata.namespace = Some("prod".into());
2594        assert_eq!(p.coordinates_or_none(), pre_lift_holds_any_claim(&p));
2595        // Namespace absent.
2596        let p = Process::new("api", empty_spec());
2597        assert_eq!(p.coordinates_or_none(), pre_lift_holds_any_claim(&p));
2598        // Name absent → both variants return `None` regardless of ns.
2599        let mut p = Process::new("api", empty_spec());
2600        p.metadata.name = None;
2601        p.metadata.namespace = Some("prod".into());
2602        assert_eq!(p.coordinates_or_none(), pre_lift_holds_any_claim(&p));
2603        // Both absent → still `None` on the name gate.
2604        let mut p = Process::new("api", empty_spec());
2605        p.metadata.name = None;
2606        p.metadata.namespace = None;
2607        assert_eq!(p.coordinates_or_none(), pre_lift_holds_any_claim(&p));
2608    }
2609
2610    #[test]
2611    fn coordinates_or_none_axis_order_matches_owned_coordinates_or_err_on_happy_path() {
2612        // Cross-primitive coherence pin at the sibling corner: when
2613        // BOTH slots are present, the borrow + name-required form
2614        // (this method) and the owned + name-required peer
2615        // (`owned_coordinates_or_err`) return the SAME `(ns, name)`
2616        // pair — the axis order is IDENTICAL and neither primitive
2617        // silently applies a normalization the other omits. A
2618        // regression that skewed one form's normalization would
2619        // surface here rather than as silent drift between the two
2620        // name-required corners of the primitive family.
2621        let mut p = Process::new("app", empty_spec());
2622        p.metadata.namespace = Some("infra".into());
2623        let (borrow_ns, borrow_name) = p.coordinates_or_none().unwrap();
2624        let (owned_ns, owned_name) = p.owned_coordinates_or_err().unwrap();
2625        assert_eq!(borrow_ns, owned_ns.as_str());
2626        assert_eq!(borrow_name, owned_name.as_str());
2627    }
2628
2629    #[test]
2630    fn coordinates_or_defaults_axis_order_matches_qualified_process_ref() {
2631        // Pins the load-bearing convention that the return-tuple
2632        // axis order is (namespace, name) — the exact positional
2633        // argument order the substrate's paired-composer primitive
2634        // `tatara_reconciler::ssapply::qualified_process_ref(ns,
2635        // name)` consumes. A regression that swapped the tuple
2636        // slots would silently misroute every annotation writer /
2637        // claim-arbiter row / owner-metadata seed built by feeding
2638        // this pair into the composer — every downstream `<ns>/
2639        // <name>` grep would suddenly see `<name>/<ns>`. The test
2640        // verifies the tuple's first slot is what a hand-authored
2641        // `.metadata.namespace.as_deref()...` produced pre-lift, and
2642        // the second slot is what `.metadata.name.as_deref()...`
2643        // produced.
2644        let mut p = Process::new("app", empty_spec());
2645        p.metadata.namespace = Some("infra".into());
2646        let (ns, name) = p.coordinates_or_defaults();
2647        assert_eq!(ns, "infra"); // NOT "app"
2648        assert_eq!(name, "app"); // NOT "infra"
2649    }
2650
2651    // ─── Process::annotation substrate pins ────────────────────────────
2652    //
2653    // Pins the borrow-form annotation-lookup primitive that owns the
2654    // 3-line `.metadata.annotations.as_ref().and_then(|m| m.get(KEY))`
2655    // chain three hand-authored sites restated by hand pre-lift:
2656    // `tatara-reconciler::signals::ingest` (SIGNAL),
2657    // `tatara-reconciler::phase_machine::released_from_annotation`
2658    // (RELEASED_FROM), and
2659    // `tatara-pool-reconciler::controller_pool::process_belongs_to_pool`
2660    // (POOL). Fail-before-pass-after granularity: a regression that
2661    // widened the missing-`annotations` corner (returning `Some("")`
2662    // instead of `None`), promoted a missing key to an error, dropped
2663    // the borrow-form return, or changed the two swallowed corners'
2664    // shared collapse to `None` surfaces here rather than as silent
2665    // drift at the three consumer sites.
2666    fn process_with_annotation(key: &str, value: &str) -> Process {
2667        let mut p = Process::new("some-proc", empty_spec());
2668        let mut anns = std::collections::BTreeMap::new();
2669        anns.insert(key.to_string(), value.to_string());
2670        p.metadata.annotations = Some(anns);
2671        p
2672    }
2673
2674    #[test]
2675    fn annotation_returns_none_when_metadata_annotations_is_none() {
2676        // Missing-`annotations` corner: a Process with no annotations
2677        // block at all returns `None` for every key. Peer to
2678        // `observed_flux_resources_returns_empty_slice_when_status_is_none`
2679        // on the status-projection axis; both primitives collapse the
2680        // outer `Option` corner rather than requiring each consumer
2681        // to spell the guard by hand.
2682        let mut p = Process::new("scratch", empty_spec());
2683        p.metadata.annotations = None;
2684        assert!(p.annotation("tatara.pleme.io/signal").is_none());
2685        assert!(p.annotation("tatara.pleme.io/pool").is_none());
2686        assert!(p.annotation("").is_none());
2687    }
2688
2689    #[test]
2690    fn annotation_returns_none_when_key_absent_from_populated_map() {
2691        // Missing-key corner: annotations block populated with OTHER
2692        // keys returns `None` for the queried key. Symmetric with the
2693        // missing-`annotations` corner — both corners collapse to the
2694        // same `None`, matching the pre-lift `.and_then(...)`
2695        // behavior every consumer relied on.
2696        let p = process_with_annotation("tatara.pleme.io/other", "value");
2697        assert!(p.annotation("tatara.pleme.io/signal").is_none());
2698        assert!(p.annotation("").is_none());
2699    }
2700
2701    #[test]
2702    fn annotation_returns_borrowed_slice_when_key_present() {
2703        // Happy path: annotations block populated + key present →
2704        // `Some(&str)` borrowed from the underlying `String` in the
2705        // map. A regression that returned an owned `String` (defeating
2706        // the primitive's role as a zero-copy projection) would
2707        // surface at the lifetime of the returned reference — the
2708        // `&str` outlives the borrow of `&p` here.
2709        let p = process_with_annotation("tatara.pleme.io/signal", "SIGHUP");
2710        assert_eq!(p.annotation("tatara.pleme.io/signal"), Some("SIGHUP"));
2711    }
2712
2713    #[test]
2714    fn annotation_returns_borrowed_empty_string_slice_when_value_is_empty() {
2715        // Edge corner between the missing-key `None` and the present-
2716        // key `Some("")` — a Process whose annotation is EXPLICITLY
2717        // set to an empty string returns `Some("")`, NOT `None`. A
2718        // regression that normalized the empty-string value to `None`
2719        // (a plausible "defensive" simplification) would silently
2720        // reshape the corner every callsite pre-lift kept distinct via
2721        // `.cloned().unwrap_or_default()` (which collapses BOTH to
2722        // `""`) or `.map(String::as_str)` (which keeps them distinct
2723        // as `None` vs `Some("")`).
2724        let p = process_with_annotation("tatara.pleme.io/signal", "");
2725        assert_eq!(p.annotation("tatara.pleme.io/signal"), Some(""));
2726    }
2727
2728    #[test]
2729    fn annotation_is_a_pure_projection() {
2730        // Purity pin — repeated calls return equal results and the
2731        // primitive does not mutate `self`. Peer to
2732        // `observed_flux_resources_is_a_pure_projection` on the
2733        // status-projection axis.
2734        let p = process_with_annotation("tatara.pleme.io/released-from", "Attested");
2735        let a = p.annotation("tatara.pleme.io/released-from");
2736        let b = p.annotation("tatara.pleme.io/released-from");
2737        assert_eq!(a, b);
2738        assert_eq!(a, Some("Attested"));
2739    }
2740
2741    #[test]
2742    fn annotation_matches_pre_lift_reconciler_chain_shape() {
2743        // Byte-identical parity pin between the borrow-form primitive
2744        // here and the pre-lift `tatara-reconciler` / `tatara-pool-
2745        // reconciler` chain shape — the exact 3-line
2746        // `.metadata.annotations.as_ref().and_then(|m| m.get(KEY))
2747        // .map(String::as_str)` incantation each pre-lift caller
2748        // spelled by hand (three variants of tail collapsed onto ONE
2749        // borrow-form primitive here; each caller reapplies its own
2750        // tail at its own site). Sweeps every corner (missing
2751        // annotations map, missing key, present key with value,
2752        // present key with empty value) so a regression that inserted
2753        // a normalization at the primitive the pre-lift chain does
2754        // NOT apply — or vice versa — surfaces here rather than as
2755        // silent drift between the ONE substrate owner and the three
2756        // consumer sites.
2757        fn pre_lift<'a>(p: &'a Process, key: &str) -> Option<&'a str> {
2758            p.metadata
2759                .annotations
2760                .as_ref()
2761                .and_then(|m| m.get(key))
2762                .map(String::as_str)
2763        }
2764        // Missing annotations map.
2765        let mut p = Process::new("x", empty_spec());
2766        p.metadata.annotations = None;
2767        assert_eq!(p.annotation("k"), pre_lift(&p, "k"));
2768        // Missing key in populated map.
2769        let p = process_with_annotation("other", "v");
2770        assert_eq!(p.annotation("k"), pre_lift(&p, "k"));
2771        // Present key with non-empty value.
2772        let p = process_with_annotation("k", "v");
2773        assert_eq!(p.annotation("k"), pre_lift(&p, "k"));
2774        // Present key with explicitly-empty value — the corner
2775        // `.cloned().unwrap_or_default()` collapses to `""` post-tail
2776        // but the primitive-level shape stays `Some("")`.
2777        let p = process_with_annotation("k", "");
2778        assert_eq!(p.annotation("k"), pre_lift(&p, "k"));
2779    }
2780
2781    #[test]
2782    fn annotation_composes_owned_tail_matching_pre_lift_signals_ingest() {
2783        // Pins the exact tail shape `tatara-reconciler::signals::
2784        // ingest` composed pre-lift: an `Option<String>` for the
2785        // downstream `let Some(raw) = raw else { ... }` guard.
2786        // Post-lift the callsite composes `.map(str::to_string)` at
2787        // its own site; this test pins the composition matches the
2788        // pre-lift `.cloned()` tail byte-for-byte on both corners the
2789        // consumer's downstream distinguishes (annotation present →
2790        // `Some(String)`; absent → `None`).
2791        let p = process_with_annotation("tatara.pleme.io/signal", "SIGUSR1");
2792        assert_eq!(
2793            p.annotation("tatara.pleme.io/signal").map(str::to_string),
2794            Some("SIGUSR1".to_string())
2795        );
2796        let mut q = Process::new("y", empty_spec());
2797        q.metadata.annotations = None;
2798        assert_eq!(
2799            q.annotation("tatara.pleme.io/signal").map(str::to_string),
2800            None
2801        );
2802    }
2803
2804    #[test]
2805    fn annotation_composes_default_tail_matching_pre_lift_released_from() {
2806        // Pins the exact tail shape
2807        // `tatara-reconciler::phase_machine::released_from_annotation`
2808        // composed pre-lift: a bare `String` via `.cloned()
2809        // .unwrap_or_default()` for the downstream
2810        // `match v.as_str()` dispatch. Post-lift the callsite matches
2811        // directly on `Option<&str>` (Some("Failed") vs _); this test
2812        // pins that the borrow-form primitive plus the `.unwrap_or("")`
2813        // fallback reproduces the pre-lift bare-string shape on both
2814        // corners.
2815        let p = process_with_annotation("tatara.pleme.io/released-from", "Failed");
2816        assert_eq!(
2817            p.annotation("tatara.pleme.io/released-from").unwrap_or(""),
2818            "Failed"
2819        );
2820        let mut q = Process::new("y", empty_spec());
2821        q.metadata.annotations = None;
2822        assert_eq!(
2823            q.annotation("tatara.pleme.io/released-from").unwrap_or(""),
2824            ""
2825        );
2826    }
2827
2828    #[test]
2829    fn annotation_composes_borrow_equality_tail_matching_pre_lift_pool() {
2830        // Pins the exact tail shape `tatara-pool-reconciler::
2831        // controller_pool::process_belongs_to_pool` composed pre-lift:
2832        // an `Option<&str>` compared with `== Some(pool_name)` for the
2833        // membership gate. Post-lift the callsite composes
2834        // `p.annotation(POOL) == Some(pool_name)` verbatim; this test
2835        // pins that the borrow-form primitive returns exactly the
2836        // shape the equality gate expects.
2837        let p = process_with_annotation("tatara.pleme.io/pool", "demo-pool");
2838        assert_eq!(
2839            p.annotation("tatara.pleme.io/pool") == Some("demo-pool"),
2840            true
2841        );
2842        assert_eq!(p.annotation("tatara.pleme.io/pool") == Some("other"), false);
2843    }
2844
2845    // ─── Process::uid_or_empty substrate pins ──────────────────────────
2846    //
2847    // Pins the borrow-form metadata-projection primitive on the
2848    // `metadata.uid` axis that owns the `.metadata.uid.as_deref()
2849    // .unwrap_or("")` chain the two hand-authored
2850    // `tatara-reconciler::render` sites (`render_routing` +
2851    // `render_export_jobs`) restated by hand pre-lift. Peer to the
2852    // sibling `namespace_or_default_*` + `name_or_placeholder_*` pin
2853    // families on the metadata-slot × fallback-shape axis; all three
2854    // primitives return borrows of an owned-metadata slot with a slot-
2855    // specific fallback baked in (`"default"` for namespace, `"unnamed"`
2856    // for name, `""` for uid — the load-bearing gate value for
2857    // `owner_references_json`'s `is_empty` check). Fail-before-pass-
2858    // after granularity: `uid_or_empty` did not exist pre-lift, so any
2859    // test invoking it fails to compile pre-lift and passes post-lift.
2860
2861    #[test]
2862    fn uid_or_empty_returns_empty_string_when_metadata_uid_is_none() {
2863        // Empty-slot corner pin: the primitive collapses the no-uid
2864        // case to `""`, matching the pre-lift `.as_deref().unwrap_or("")`
2865        // chain's `""` byte-identically at both render consumer sites.
2866        // Semantically corresponds to a Process pre-metadata (fixtured
2867        // in tests, or caught mid-Forking before the API server has
2868        // stamped a `uid`); the downstream `owner_references_json`
2869        // composer gates on this exact `""` sentinel to stamp
2870        // `metadata.ownerReferences: []` rather than emit an owner-ref
2871        // pointing at a placeholder uid.
2872        let mut p = Process::new("scratch", empty_spec());
2873        p.metadata.uid = None;
2874        assert_eq!(p.uid_or_empty(), "");
2875    }
2876
2877    #[test]
2878    fn uid_or_empty_returns_borrowed_str_when_slot_is_populated() {
2879        // Happy-path pin: with a populated `metadata.uid` slot, the
2880        // primitive returns a borrowed `&str` whose contents match the
2881        // persisted `String`. A regression that reshaped / normalized
2882        // / cross-cluster-stripped the uid without touching this pin
2883        // would surface here rather than as silent skew at the two
2884        // `owner_references_json(name, uid)` emitters on the SAME
2885        // Process.
2886        let mut p = Process::new("owned-proc", empty_spec());
2887        p.metadata.uid = Some("uid-abc-123".into());
2888        assert_eq!(p.uid_or_empty(), "uid-abc-123");
2889    }
2890
2891    #[test]
2892    fn uid_or_empty_returns_empty_string_when_slot_is_explicitly_empty_string() {
2893        // Corner between the missing-slot `None` and the explicitly-
2894        // empty-string `Some("")` — both collapse to `""` at the
2895        // primitive because the downstream gate at
2896        // `owner_references_json` treats `.is_empty()` uniformly (the
2897        // empty-slot posture is what the whole primitive family
2898        // encodes: "no admissible owner reference, stamp `[]`"). A
2899        // regression that discriminated the two corners (returning a
2900        // sentinel `"<none>"` for the missing slot but `""` for the
2901        // explicit slot) would break the composition with
2902        // `owner_references_json` at the exactly-two-corner gate.
2903        let mut p = Process::new("owned-proc", empty_spec());
2904        p.metadata.uid = Some(String::new());
2905        assert_eq!(p.uid_or_empty(), "");
2906    }
2907
2908    #[test]
2909    fn uid_or_empty_is_a_zero_copy_borrow_projection() {
2910        // Borrow-discipline pin: the returned `&str` borrows the
2911        // persisted `String`'s underlying byte buffer in place — NOT
2912        // a fresh allocation or a clone. A regression that switched
2913        // the projection to an owned `String` (via `.clone()` or a
2914        // `format!` wrap) would defeat the zero-copy contract the
2915        // lift's primary strict-widening delivers, and would surface
2916        // here via pointer-identity comparison.
2917        let mut p = Process::new("owned-proc", empty_spec());
2918        p.metadata.uid = Some("uid-borrow-pin".into());
2919        let slice = p.uid_or_empty();
2920        assert!(std::ptr::eq(
2921            slice.as_ptr(),
2922            p.metadata.uid.as_ref().unwrap().as_ptr()
2923        ));
2924    }
2925
2926    #[test]
2927    fn uid_or_empty_is_a_pure_projection() {
2928        // Purity pin — repeated calls return byte-identical slices
2929        // (same pointer, same length). A regression that introduced
2930        // state (a lazy-cached normalized slot, a first-call
2931        // canonicalization pass) would surface here rather than as
2932        // silent drift between the two render consumer sites on the
2933        // SAME Process within one render pass.
2934        let mut p = Process::new("owned-proc", empty_spec());
2935        p.metadata.uid = Some("uid-pure".into());
2936        let a = p.uid_or_empty();
2937        let b = p.uid_or_empty();
2938        assert!(std::ptr::eq(a.as_ptr(), b.as_ptr()));
2939        assert_eq!(a.len(), b.len());
2940    }
2941
2942    #[test]
2943    fn uid_or_empty_matches_pre_lift_render_chain_shape() {
2944        // Byte-identical parity pin between the borrow-form primitive
2945        // here and the pre-lift `tatara-reconciler::render` chain shape
2946        // — the exact `.metadata.uid.as_deref().unwrap_or("")`
2947        // incantation both `render_routing` (line 514) and
2948        // `render_export_jobs` (line 653) spelled by hand pre-lift.
2949        // Sweeps every corner (missing uid slot, populated uid slot,
2950        // explicitly-empty uid slot) so a regression that inserted a
2951        // normalization the pre-lift chain does NOT apply — or vice
2952        // versa — surfaces here rather than as silent drift between
2953        // the ONE substrate owner and the two consumer sites.
2954        fn pre_lift(p: &Process) -> &str {
2955            p.metadata.uid.as_deref().unwrap_or("")
2956        }
2957        // Missing slot.
2958        let mut p = Process::new("x", empty_spec());
2959        p.metadata.uid = None;
2960        assert_eq!(p.uid_or_empty(), pre_lift(&p));
2961        // Populated slot.
2962        let mut p = Process::new("x", empty_spec());
2963        p.metadata.uid = Some("uid-42".into());
2964        assert_eq!(p.uid_or_empty(), pre_lift(&p));
2965        // Explicitly-empty slot.
2966        let mut p = Process::new("x", empty_spec());
2967        p.metadata.uid = Some(String::new());
2968        assert_eq!(p.uid_or_empty(), pre_lift(&p));
2969    }
2970
2971    #[test]
2972    fn uid_or_empty_composes_with_owner_references_json_empty_gate() {
2973        // Cross-primitive composition pin — the empty-string sentinel
2974        // this primitive returns for the missing-uid corner is EXACTLY
2975        // the sentinel the sibling substrate composer
2976        // `owner_references_json(name, uid)` gates on to stamp
2977        // `metadata.ownerReferences: []`. A regression that changed
2978        // the sentinel at either end (this primitive returning
2979        // `"<none>"`, `owner_references_json` gating on `uid == "0"`
2980        // instead of `uid.is_empty()`) would break the composition
2981        // and surface here rather than as an operator-observed
2982        // orphan resource after apply.
2983        let mut p = Process::new("x", empty_spec());
2984        p.metadata.uid = None;
2985        let refs = crate::owner_references_json("some-name", p.uid_or_empty());
2986        assert!(
2987            refs.is_empty(),
2988            "empty-uid corner must produce empty owner-refs array"
2989        );
2990
2991        p.metadata.uid = Some("real-uid".into());
2992        let refs = crate::owner_references_json("some-name", p.uid_or_empty());
2993        assert_eq!(
2994            refs.len(),
2995            1,
2996            "populated-uid corner must produce one owner-ref entry"
2997        );
2998    }
2999
3000    // ─── Process::owned_name_or_empty substrate pins ─────────────────
3001    //
3002    // Pins the owned-form metadata-projection primitive on the
3003    // `metadata.name` axis that owns the
3004    // `.metadata.name.clone().unwrap_or_default()` chain the two hand-
3005    // authored `tatara-pool-reconciler::controller_pool` sites (the
3006    // `PoolMember` seed at line 68 + the `PoolMemberSnapshot` desired-
3007    // count seed at line 108) restated by hand pre-lift. Peer to the
3008    // sibling `uid_or_empty` pin family on the (return-form × fallback-
3009    // value) axis pair — `uid_or_empty` owns the BORROW + empty-sentinel
3010    // corner (`&str` for owner-ref emitters gating on `.is_empty()`);
3011    // this method owns the OWNED + empty-sentinel corner (`String` for
3012    // struct-literal / HashMap-key row-builder consumers whose
3013    // downstream fills a `String` field with the load-bearing `""`
3014    // sentinel). Fail-before-pass-after granularity: `owned_name_or_empty`
3015    // did not exist pre-lift, so any test invoking it fails to compile
3016    // pre-lift and passes post-lift.
3017
3018    #[test]
3019    fn owned_name_or_empty_returns_empty_string_when_metadata_name_is_none() {
3020        // Empty-slot corner pin: the primitive collapses the no-name
3021        // case to `String::new()`, matching the pre-lift
3022        // `.clone().unwrap_or_default()` chain's empty `String` byte-
3023        // identically at both pool-reconciler consumer sites.
3024        // Semantically corresponds to a Process pre-metadata-name (test
3025        // fixture, dynamic API response pre-name-resolution); the
3026        // downstream `PoolMember { process_name, .. }` slot then holds
3027        // `""` as a stable "no name to key by" signal rather than a
3028        // display placeholder that would silently alias distinct rows.
3029        let mut p = Process::new("scratch", empty_spec());
3030        p.metadata.name = None;
3031        assert_eq!(p.owned_name_or_empty(), String::new());
3032    }
3033
3034    #[test]
3035    fn owned_name_or_empty_returns_owned_string_when_slot_is_populated() {
3036        // Happy-path pin: with a populated `metadata.name` slot, the
3037        // primitive returns an owned `String` whose contents match the
3038        // persisted `String`. A regression that reshaped / normalized
3039        // / case-folded the name without touching this pin would surface
3040        // here rather than as silent skew between the two pool-member
3041        // seeds keying on the SAME Process's name.
3042        let p = Process::new("api", empty_spec());
3043        assert_eq!(p.owned_name_or_empty(), "api");
3044    }
3045
3046    #[test]
3047    fn owned_name_or_empty_returns_empty_string_when_slot_is_explicitly_empty_string() {
3048        // Corner between the missing-slot `None` and the explicitly-
3049        // empty-string `Some(String::new())` — both collapse to `""` at
3050        // the primitive because the downstream pool-member consumers
3051        // treat both corners uniformly (no name, no key). A regression
3052        // that discriminated the two corners (returning a sentinel
3053        // `"<none>"` for the missing slot but `""` for the explicit
3054        // slot) would break `String::is_empty` gating at the row-builder
3055        // callsites without moving this pin.
3056        let mut p = Process::new("scratch", empty_spec());
3057        p.metadata.name = Some(String::new());
3058        assert_eq!(p.owned_name_or_empty(), String::new());
3059        assert!(p.owned_name_or_empty().is_empty());
3060    }
3061
3062    #[test]
3063    fn owned_name_or_empty_is_a_pure_projection() {
3064        // Purity pin — repeated calls return byte-identical `String`
3065        // values. A regression that introduced state (a lazy-cached
3066        // normalized slot, a first-call canonicalization pass) would
3067        // surface here rather than as silent drift between the pool-
3068        // member seed and the desired-count snapshot seed on the SAME
3069        // Process within one reconcile pass.
3070        let p = Process::new("stable-name", empty_spec());
3071        assert_eq!(p.owned_name_or_empty(), p.owned_name_or_empty());
3072    }
3073
3074    #[test]
3075    fn owned_name_or_empty_returns_independent_owned_string() {
3076        // Owned-discipline pin: the returned `String` is an independent
3077        // allocation the caller may consume, `.push_str` into, or move
3078        // into a struct-literal `process_name: String` slot — NOT a
3079        // shared reference into `metadata.name`. A regression that
3080        // switched the projection to a `Cow`-shaped variant or a slice-
3081        // form projection would defeat the owned-form contract the two
3082        // pool-reconciler struct-literal consumers depend on (a slice
3083        // cannot land in a `process_name: String` slot without a re-
3084        // clone), and would surface here at compile time via the mutate-
3085        // in-place test below.
3086        let p = Process::new("owned-proc", empty_spec());
3087        let mut owned = p.owned_name_or_empty();
3088        owned.push_str("-mutated");
3089        assert_eq!(owned, "owned-proc-mutated");
3090        // The Process's own slot is unchanged — the returned String
3091        // owns its own byte buffer, disjoint from `metadata.name`.
3092        assert_eq!(p.metadata.name.as_deref(), Some("owned-proc"));
3093    }
3094
3095    #[test]
3096    fn owned_name_or_empty_matches_pre_lift_controller_pool_chain_shape() {
3097        // Byte-identical parity pin between the owned-form primitive
3098        // here and the pre-lift `tatara-pool-reconciler::controller_pool`
3099        // chain shape — the exact `.metadata.name.clone().unwrap_or_default()`
3100        // incantation both `PoolMember` seed (line 68) and
3101        // `PoolMemberSnapshot` seed (line 108) spelled by hand pre-lift.
3102        // Sweeps every corner (missing name slot, populated name slot,
3103        // explicitly-empty name slot) so a regression that inserted a
3104        // normalization the pre-lift chain does NOT apply — or vice
3105        // versa — surfaces here rather than as silent drift between
3106        // the ONE substrate owner and the two consumer sites.
3107        fn pre_lift(p: &Process) -> String {
3108            p.metadata.name.clone().unwrap_or_default()
3109        }
3110        // Missing slot.
3111        let mut p = Process::new("x", empty_spec());
3112        p.metadata.name = None;
3113        assert_eq!(p.owned_name_or_empty(), pre_lift(&p));
3114        // Populated slot.
3115        let p = Process::new("real-name", empty_spec());
3116        assert_eq!(p.owned_name_or_empty(), pre_lift(&p));
3117        // Explicitly-empty slot.
3118        let mut p = Process::new("x", empty_spec());
3119        p.metadata.name = Some(String::new());
3120        assert_eq!(p.owned_name_or_empty(), pre_lift(&p));
3121    }
3122
3123    #[test]
3124    fn owned_name_or_empty_shares_empty_sentinel_with_uid_or_empty() {
3125        // Cross-primitive coherence pin — the empty-string fallback this
3126        // primitive returns for the missing-name corner is the SAME
3127        // sentinel the sibling borrow-form primitive `uid_or_empty`
3128        // returns for the missing-uid corner. Both partition the OWNED
3129        // × BORROW corner of the metadata-slot family on identical
3130        // fallback semantics ("the slot is unset"), so a consumer that
3131        // switches between them based on downstream ownership
3132        // requirements never sees a different missing-slot spelling as
3133        // a side effect. A regression that drifted either sentinel
3134        // (this primitive returning `"<unnamed>"`, `uid_or_empty`
3135        // returning `"<none>"`) would break the partition and surface
3136        // here rather than as silent shape drift across the family.
3137        let mut p = Process::new("scratch", empty_spec());
3138        p.metadata.name = None;
3139        p.metadata.uid = None;
3140        assert_eq!(p.owned_name_or_empty(), p.uid_or_empty());
3141        assert!(p.owned_name_or_empty().is_empty());
3142        assert!(p.uid_or_empty().is_empty());
3143    }
3144
3145    #[test]
3146    fn owned_name_or_empty_returns_distinct_fallback_from_name_or_placeholder() {
3147        // Axis-partition pin — the owned + empty-sentinel primitive here
3148        // and the borrow + display-placeholder primitive
3149        // [`Self::name_or_placeholder`] MUST return distinct fallback
3150        // values on the missing-name corner. The distinction is load-
3151        // bearing: `owned_name_or_empty` is for HashMap-key / row-builder
3152        // consumers that need distinct keys for missing-name Processes
3153        // (empty string collides only with other missing-name rows,
3154        // never with a real "unnamed" Process); `name_or_placeholder`
3155        // is for log-line / display consumers that render the
3156        // `"unnamed"` word to operators. A regression that unified the
3157        // two fallbacks (either primitive returning the other's
3158        // sentinel) would silently collapse missing-name pool members
3159        // into a display-string key or expose the empty sentinel to
3160        // operator log lines. This pin catches either drift.
3161        let mut p = Process::new("scratch", empty_spec());
3162        p.metadata.name = None;
3163        assert_eq!(p.owned_name_or_empty(), "");
3164        assert_eq!(p.name_or_placeholder(), Process::UNNAMED_PLACEHOLDER);
3165        assert_ne!(p.owned_name_or_empty(), p.name_or_placeholder());
3166    }
3167
3168    // ─── Process::declared_parent_pid substrate pins ─────────────────
3169    //
3170    // Pins the borrow-form spec-projection primitive on the declared
3171    // parent-PID axis that owns the `.spec.identity.parent.as_deref()`
3172    // chain the two hand-authored `tatara-reconciler::phase_machine`
3173    // sites (`handle_forking` ALLOCATE-PID composer + `handle_exiting`
3174    // SIGTERM-cascade child-fan-out filter) restated by hand pre-lift.
3175    // Peer to the sibling `observed_pid_*` pin family on the (spec-
3176    // declared × status-observed) axis pair; both compose the same
3177    // borrow-form `Option<&str>` return-shape skeleton on distinct
3178    // slots (`spec.identity.parent` vs. `status.pid`). Fail-before-
3179    // pass-after granularity: `declared_parent_pid` did not exist
3180    // pre-lift, so any test invoking it fails to compile pre-lift and
3181    // passes post-lift.
3182    fn process_with_declared_parent(parent: Option<&str>) -> Process {
3183        let mut spec = empty_spec();
3184        spec.identity.parent = parent.map(str::to_string);
3185        Process::new("child-proc", spec)
3186    }
3187
3188    #[test]
3189    fn declared_parent_pid_returns_none_when_slot_is_none() {
3190        // Empty-slot corner pin: the primitive collapses the no-
3191        // parent case to `None`, matching the pre-lift `.as_deref()`
3192        // chain's `None` byte-identically at both reconciler consumer
3193        // sites. Semantically corresponds to a Process authored at
3194        // cluster init (PID 1) with no upstream parent — the
3195        // ALLOCATE-PID composer feeds `None` into `pid::allocate_pid`
3196        // to signal "no prefix", and the SIGTERM cascade's filter
3197        // never matches such a Process because a child's declared
3198        // parent can never equal `Some(pid)` when the slot is `None`.
3199        let p = process_with_declared_parent(None);
3200        assert!(p.declared_parent_pid().is_none());
3201    }
3202
3203    #[test]
3204    fn declared_parent_pid_returns_borrowed_str_when_slot_is_populated() {
3205        // Happy-path pin: with a populated `spec.identity.parent`
3206        // slot, the primitive returns a borrowed `&str` whose
3207        // contents match the persisted `String`. A regression that
3208        // filtered / reshaped / canonicalized the string would
3209        // surface here rather than as silent skew at the child-fan-
3210        // out filter's `.declared_parent_pid() == Some(pid)`
3211        // equality check on the SAME parent-child pair.
3212        let p = process_with_declared_parent(Some("seph.1"));
3213        assert_eq!(p.declared_parent_pid(), Some("seph.1"));
3214    }
3215
3216    #[test]
3217    fn declared_parent_pid_is_a_zero_copy_borrow_projection() {
3218        // Borrow-discipline pin: the returned `&str` borrows the
3219        // persisted `String`'s underlying byte buffer in place —
3220        // NOT a fresh allocation or a clone. A regression that
3221        // switched the projection to an owned `String` (via
3222        // `.clone()` or `.to_owned()`) would defeat the zero-copy
3223        // contract the lift's primary strict-widening delivers.
3224        // The `handle_exiting` cascade filter runs per candidate
3225        // child across the cluster-wide Process list; a per-row
3226        // `String::clone` would allocate one heap block per non-
3227        // matching row, so the borrow-form primitive is load-
3228        // bearing for large clusters. Peer to the sibling
3229        // `observed_pid_is_a_zero_copy_borrow_projection` pin on
3230        // the status-observed side of the axis pair.
3231        let p = process_with_declared_parent(Some("seph.1"));
3232        let borrowed = p.declared_parent_pid().expect("populated slot");
3233        let persisted = p.spec.identity.parent.as_ref().unwrap();
3234        assert!(std::ptr::eq(borrowed.as_ptr(), persisted.as_ptr()));
3235    }
3236
3237    #[test]
3238    fn declared_parent_pid_is_a_pure_projection() {
3239        // Purity pin: calling the projection twice on the same
3240        // `Process` returns byte-identical `&str`s (same pointer,
3241        // same length). A regression that introduced state — a
3242        // lazy-cached slice materialized on first call, a
3243        // normalization step that ran once and cached — would
3244        // surface here rather than as silent drift between the
3245        // ALLOCATE-PID composer and the SIGTERM cascade's child-
3246        // fan-out filter within one reconcile pass.
3247        let p = process_with_declared_parent(Some("seph.1.3"));
3248        let a = p.declared_parent_pid().expect("populated slot");
3249        let b = p.declared_parent_pid().expect("populated slot");
3250        assert!(std::ptr::eq(a.as_ptr(), b.as_ptr()));
3251        assert_eq!(a.len(), b.len());
3252    }
3253
3254    #[test]
3255    fn declared_parent_pid_matches_pre_lift_reconciler_chain_shape() {
3256        // Byte-identical parity pin between the borrow-form primitive
3257        // here and the pre-lift `tatara-reconciler::phase_machine`
3258        // `.spec.identity.parent.as_deref()` chain shape. Sweeps
3259        // every corner every callsite plausibly encounters (empty
3260        // slot, populated with a hierarchical PID). A regression
3261        // that inserted a normalization step at the primitive the
3262        // pre-lift chain does NOT apply — or vice versa — surfaces
3263        // here rather than as silent drift between the pre-lift
3264        // consumer sites and the ONE substrate owner they now route
3265        // through. Peer to
3266        // `observed_pid_matches_pre_lift_reconciler_chain_shape` on
3267        // the sibling axis's borrow-form primitive.
3268        fn pre_lift(p: &Process) -> Option<&str> {
3269            p.spec.identity.parent.as_deref()
3270        }
3271        // Empty slot.
3272        let p = process_with_declared_parent(None);
3273        assert_eq!(p.declared_parent_pid(), pre_lift(&p));
3274        // Populated with a hierarchical PID.
3275        let p = process_with_declared_parent(Some("seph.1"));
3276        assert_eq!(p.declared_parent_pid(), pre_lift(&p));
3277        // Populated with a deeper hierarchical PID.
3278        let p = process_with_declared_parent(Some("seph.1.7.42"));
3279        assert_eq!(p.declared_parent_pid(), pre_lift(&p));
3280    }
3281
3282    #[test]
3283    fn declared_parent_pid_preserves_hierarchical_pid_format() {
3284        // Format-preservation pin: the hierarchical PID path
3285        // (dotted-segment form `seph.1.7`, matching the ported
3286        // `convergence-controller/src/identity.rs` scheme) reaches
3287        // the caller with segments and separators byte-identical
3288        // to the persisted `String`. A regression that inserted a
3289        // canonicalization pass (a segment-count validator, a
3290        // separator swap `.` → `/`, a leading/trailing whitespace
3291        // trim) would silently misroute the SIGTERM cascade's
3292        // `declared_parent_pid() == Some(pid)` comparator against
3293        // children whose `parent` field was authored in the ported
3294        // scheme's exact form — the SAME children the observed_pid
3295        // primitive is pinned to match on the other side of the
3296        // axis pair.
3297        for parent in ["seph", "seph.1", "seph.1.7", "seph.1.7.42"] {
3298            let p = process_with_declared_parent(Some(parent));
3299            assert_eq!(p.declared_parent_pid(), Some(parent));
3300        }
3301    }
3302
3303    #[test]
3304    fn declared_parent_pid_composes_with_observed_pid_for_child_fanout_filter() {
3305        // Cross-axis coherence pin against the sibling
3306        // [`Self::observed_pid`] on the (spec-declared × status-
3307        // observed) axis pair: a child's `.declared_parent_pid()`
3308        // and its parent's `.observed_pid()` compose through the
3309        // SAME borrow-form `Option<&str>` skeleton so the
3310        // `handle_exiting` cascade filter's equality gate holds
3311        // structurally. A regression that skewed EITHER primitive's
3312        // return-form (return-shape, borrow discipline, empty-slot
3313        // collapse) would silently misroute every SIGTERM cascade
3314        // on the parent-child pair. This pin re-reads both primitives
3315        // at test time so the composition holds iff both live paths
3316        // are the current implementation.
3317        // Parent Process: has an observed PID.
3318        let mut parent = Process::new("parent-proc", empty_spec());
3319        parent.status = Some(ProcessStatus {
3320            pid: Some("seph.1".to_string()),
3321            ..Default::default()
3322        });
3323        // Child Process: declared parent matches parent's observed PID.
3324        let child = process_with_declared_parent(Some("seph.1"));
3325        // The `handle_exiting` filter's equality gate:
3326        // `child.declared_parent_pid() == Some(parent.observed_pid()?)`.
3327        let parent_pid = parent.observed_pid().expect("parent has PID");
3328        assert_eq!(child.declared_parent_pid(), Some(parent_pid));
3329        // Sibling Process with an unrelated declared parent must NOT
3330        // match the same parent — pins that the filter's SKIP branch
3331        // holds on the other side of the axis pair.
3332        let sibling = process_with_declared_parent(Some("seph.2"));
3333        assert_ne!(sibling.declared_parent_pid(), Some(parent_pid));
3334    }
3335
3336    // ─── Process::declared_name_override substrate pins ──────────────
3337    //
3338    // Pins the borrow-form spec-projection primitive on the declared
3339    // name-override sub-axis of the declared-identity axis that owns
3340    // the `.spec.identity.name_override.as_deref()` chain the two
3341    // hand-authored `tatara-reconciler::phase_machine` sites
3342    // (`handle_pending` DECLARE composer + `handle_forking` ALLOCATE-
3343    // PID rehydration branch) restated by hand pre-lift. Peer to the
3344    // sibling `declared_parent_pid_*` pin family on the (parent ×
3345    // name-override) sub-axis pair; both compose the same borrow-form
3346    // `Option<&str>` return-shape skeleton on distinct slots
3347    // (`spec.identity.name_override` vs `spec.identity.parent`).
3348    // Fail-before-pass-after granularity: `declared_name_override`
3349    // did not exist pre-lift, so any test invoking it fails to
3350    // compile pre-lift and passes post-lift.
3351    fn process_with_declared_name_override(name_override: Option<&str>) -> Process {
3352        let mut spec = empty_spec();
3353        spec.identity.name_override = name_override.map(str::to_string);
3354        Process::new("some-proc", spec)
3355    }
3356
3357    #[test]
3358    fn declared_name_override_returns_none_when_slot_is_none() {
3359        // Empty-slot corner pin: the primitive collapses the no-
3360        // override case to `None`, matching the pre-lift `.as_deref()`
3361        // chain's `None` byte-identically at both reconciler consumer
3362        // sites. Semantically corresponds to a Process authored
3363        // WITHOUT the human-name-override escape hatch — the default;
3364        // `derive_identity` then computes the name from the content
3365        // hash and stamps `name_override: false` on the resulting
3366        // [`Identity`].
3367        let p = process_with_declared_name_override(None);
3368        assert!(p.declared_name_override().is_none());
3369    }
3370
3371    #[test]
3372    fn declared_name_override_returns_borrowed_str_when_slot_is_populated() {
3373        // Happy-path pin: with a populated `spec.identity
3374        // .name_override` slot, the primitive returns a borrowed
3375        // `&str` whose contents match the persisted `String`. A
3376        // regression that filtered / reshaped / canonicalized the
3377        // string at the primitive (as opposed to inside
3378        // `derive_identity`, where the trim/empty-filter lives today)
3379        // would surface here rather than as silent skew between the
3380        // DECLARE composer and the ALLOCATE-PID rehydration branch on
3381        // the SAME Process spec.
3382        let p = process_with_declared_name_override(Some("observability-stack"));
3383        assert_eq!(p.declared_name_override(), Some("observability-stack"));
3384    }
3385
3386    #[test]
3387    fn declared_name_override_is_a_zero_copy_borrow_projection() {
3388        // Borrow-discipline pin: the returned `&str` borrows the
3389        // persisted `String`'s underlying byte buffer in place —
3390        // NOT a fresh allocation or a clone. Peer to the sibling
3391        // `declared_parent_pid_is_a_zero_copy_borrow_projection` pin
3392        // on the other side of the (parent × name-override) sub-axis
3393        // pair; the borrow discipline holds structurally on BOTH
3394        // sub-axes so a future `declared_identity` composite that
3395        // returns both halves together can compose them without
3396        // dropping into an owning form.
3397        let p = process_with_declared_name_override(Some("observability-stack"));
3398        let borrowed = p.declared_name_override().expect("populated slot");
3399        let persisted = p.spec.identity.name_override.as_ref().unwrap();
3400        assert!(std::ptr::eq(borrowed.as_ptr(), persisted.as_ptr()));
3401    }
3402
3403    #[test]
3404    fn declared_name_override_is_a_pure_projection() {
3405        // Purity pin: calling the projection twice on the same
3406        // `Process` returns byte-identical `&str`s (same pointer,
3407        // same length). A regression that introduced state — a
3408        // lazy-cached slice materialized on first call, a
3409        // normalization step that ran once and cached — would
3410        // surface here rather than as silent drift between the
3411        // DECLARE composer and the ALLOCATE-PID rehydration branch
3412        // within one reconcile pass.
3413        let p = process_with_declared_name_override(Some("gateway-primary"));
3414        let a = p.declared_name_override().expect("populated slot");
3415        let b = p.declared_name_override().expect("populated slot");
3416        assert!(std::ptr::eq(a.as_ptr(), b.as_ptr()));
3417        assert_eq!(a.len(), b.len());
3418    }
3419
3420    #[test]
3421    fn declared_name_override_matches_pre_lift_reconciler_chain_shape() {
3422        // Byte-identical parity pin between the borrow-form primitive
3423        // here and the pre-lift `tatara-reconciler::phase_machine`
3424        // `.spec.identity.name_override.as_deref()` chain shape.
3425        // Sweeps every corner every callsite plausibly encounters
3426        // (empty slot, populated with a bare name, populated with a
3427        // whitespace-containing name that `derive_identity`'s
3428        // internal trim would collapse, populated with an explicitly
3429        // empty string that `derive_identity`'s internal
3430        // `!s.is_empty()` filter would reject). A regression that
3431        // inserted a normalization step at the primitive the pre-
3432        // lift chain does NOT apply — or vice versa — surfaces here
3433        // rather than as silent drift between the pre-lift consumer
3434        // sites and the ONE substrate owner they now route through.
3435        // Peer to
3436        // `declared_parent_pid_matches_pre_lift_reconciler_chain_shape`
3437        // on the sibling sub-axis's borrow-form primitive.
3438        fn pre_lift(p: &Process) -> Option<&str> {
3439            p.spec.identity.name_override.as_deref()
3440        }
3441        // Empty slot.
3442        let p = process_with_declared_name_override(None);
3443        assert_eq!(p.declared_name_override(), pre_lift(&p));
3444        // Populated with a bare name.
3445        let p = process_with_declared_name_override(Some("observability-stack"));
3446        assert_eq!(p.declared_name_override(), pre_lift(&p));
3447        // Populated with a whitespace-containing name.
3448        let p = process_with_declared_name_override(Some("  observability-stack  "));
3449        assert_eq!(p.declared_name_override(), pre_lift(&p));
3450        // Populated with an explicitly empty string. Distinct from
3451        // the missing-slot `None` corner both at the primitive here
3452        // and at the pre-lift chain (the trim/filter that collapses
3453        // these two into the same `false`-branched
3454        // `Identity { name_override: false, .. }` lives INSIDE
3455        // `derive_identity`, NOT at the borrow site) — the primitive
3456        // MUST preserve the distinction so a future lift of the trim/
3457        // filter OUT of `derive_identity` INTO the primitive is a
3458        // conscious substrate change, not a silent one.
3459        let p = process_with_declared_name_override(Some(""));
3460        assert_eq!(p.declared_name_override(), pre_lift(&p));
3461    }
3462
3463    #[test]
3464    fn declared_name_override_preserves_raw_slot_verbatim() {
3465        // Invariance-under-`derive_identity`-normalization pin: the
3466        // primitive returns the slot's raw byte contents verbatim —
3467        // no trim, no empty-string filter, no case fold, no
3468        // normalization of any kind. `derive_identity` internally
3469        // applies `.map(str::trim).filter(|s| !s.is_empty())` before
3470        // dispatching on `Some(non_empty)` vs `None | Some(empty |
3471        // whitespace)`, but that transform lives IN `derive_identity`,
3472        // NOT at the borrow site. A regression that pulled the trim/
3473        // filter forward INTO the primitive would silently collapse
3474        // three currently-distinct corners at the borrow site (bare
3475        // populated → `Some(name)`; whitespace-only → `Some("   ")`;
3476        // empty → `Some("")`) into two (bare → `Some(name)`; the
3477        // other two → `None`). That collapse might be an intentional
3478        // substrate change some future run wants to make; if so, it
3479        // lands as a conscious edit here (with this pin updated in
3480        // the same commit) rather than as silent behavior drift.
3481        for value in ["bare", "  padded  ", "\ttabs\t", "   ", ""] {
3482            let p = process_with_declared_name_override(Some(value));
3483            assert_eq!(
3484                p.declared_name_override(),
3485                Some(value),
3486                "declared_name_override must preserve raw slot verbatim for value {value:?}"
3487            );
3488        }
3489    }
3490
3491    #[test]
3492    fn declared_name_override_composes_with_derive_identity_call_shape() {
3493        // Cross-primitive coherence pin against the [`derive_identity`]
3494        // consumer: the two live `tatara-reconciler::phase_machine`
3495        // callsites feed `p.declared_name_override()` as the second
3496        // positional argument to `derive_identity(&p.spec, …)`. This
3497        // pin exercises that exact call shape at test time so a
3498        // regression that skewed the primitive's return-form (return-
3499        // shape, borrow discipline, empty-slot collapse) surfaces
3500        // here as a shape mismatch at the [`derive_identity`] call
3501        // site rather than as silent operator-facing skew between the
3502        // DECLARE composer and the ALLOCATE-PID rehydration branch.
3503        // Populated with a bare non-empty name: `derive_identity`
3504        // dispatches on `Some(non_empty)` and stamps
3505        // `name_override: true` on the resulting [`Identity`], with
3506        // the resulting `.name` equal to the raw slot value.
3507        let p = process_with_declared_name_override(Some("gateway-primary"));
3508        let id = crate::identity::derive_identity(&p.spec, p.declared_name_override());
3509        assert!(id.name_override);
3510        assert_eq!(id.name, "gateway-primary");
3511        // Empty slot: `derive_identity` dispatches on `None` and
3512        // stamps `name_override: false` on the resulting [`Identity`],
3513        // with the resulting `.name` derived from the content hash
3514        // (NOT equal to any operator-authored slot value).
3515        let p = process_with_declared_name_override(None);
3516        let id = crate::identity::derive_identity(&p.spec, p.declared_name_override());
3517        assert!(!id.name_override);
3518    }
3519
3520    // ─── Process::observed_flux_resources substrate pins ───────────────
3521    //
3522    // Pins the borrow-form status-projection primitive that owns the
3523    // 5-line `.status.as_ref().map(|s| s.flux_resources.clone())
3524    // .unwrap_or_default()` chain the two hand-authored
3525    // `tatara-reconciler::phase_machine` sites (`handle_running` +
3526    // `handle_attested`) restated by hand pre-lift. Fail-before-pass-
3527    // after granularity: a regression that widened the missing-`status`
3528    // corner, dropped the slot, or drifted the borrow discipline
3529    // surfaces here rather than as silent operator-facing skew between
3530    // the VERIFY-phase readiness probe and the ATTEST-heartbeat drift
3531    // detector.
3532
3533    fn sample_flux_ref(name: &str) -> FluxResourceRef {
3534        // Distinct slot values so a swap between adjacent tuple
3535        // positions surfaces as an equality failure at the assertion
3536        // site — a slot-inversion regression cannot masquerade as
3537        // identity by accident. Peer to the sibling
3538        // `tatara_process::status::tests::sample_flux_ref` discipline
3539        // on the fetch-coords axis.
3540        FluxResourceRef {
3541            api_version: "kustomize.toolkit.fluxcd.io/v1".to_string(),
3542            kind: "Kustomization".to_string(),
3543            name: name.to_string(),
3544            namespace: "flux-system".to_string(),
3545            ready: false,
3546            message: None,
3547            last_check: None,
3548        }
3549    }
3550
3551    fn process_with_flux_resources(refs: Vec<FluxResourceRef>) -> Process {
3552        let mut p = Process::new("api-gateway", empty_spec());
3553        p.metadata.namespace = Some("prod".into());
3554        let mut status = ProcessStatus::default();
3555        status.flux_resources = refs;
3556        p.status = Some(status);
3557        p
3558    }
3559
3560    #[test]
3561    fn observed_flux_resources_returns_empty_slice_when_status_is_none() {
3562        // Missing-`status` corner pin: the primitive collapses the
3563        // no-status case to `&[]` so downstream `.is_empty()` /
3564        // `.len()` / iteration behave identically on a `Process`
3565        // whose status field is `None` and on one whose status
3566        // carries an empty `flux_resources` slot. Matches the
3567        // pre-lift `.unwrap_or_default()`'s empty-`Vec` corner
3568        // byte-identically at every reconciler consumer's downstream
3569        // shape.
3570        let mut p = Process::new("api", empty_spec());
3571        p.status = None;
3572        assert!(p.observed_flux_resources().is_empty());
3573        assert_eq!(p.observed_flux_resources().len(), 0);
3574    }
3575
3576    #[test]
3577    fn observed_flux_resources_returns_empty_slice_when_flux_resources_is_empty() {
3578        // Zero-refs-under-populated-status corner pin: the primitive
3579        // returns an empty slice, matching the missing-`status`
3580        // corner byte-identically. A regression that treated the two
3581        // corners differently (a `None`-vs-empty signal that
3582        // downstream consumers could grep on) would silently promote
3583        // an internal representation detail (whether the reconciler
3584        // has ever written a status subresource) into observable
3585        // behavior.
3586        let p = process_with_flux_resources(vec![]);
3587        assert!(p.observed_flux_resources().is_empty());
3588        assert_eq!(p.observed_flux_resources().len(), 0);
3589    }
3590
3591    #[test]
3592    fn observed_flux_resources_returns_slice_of_persisted_vec() {
3593        // Happy-path pin: with a populated `status.flux_resources`
3594        // slot, the primitive returns a borrowed slice whose length
3595        // and per-element identity match the persisted vector. A
3596        // regression that filtered / reshaped / deduplicated the
3597        // slice would surface here rather than as silent skew at the
3598        // downstream fetch consumers.
3599        let refs = vec![
3600            sample_flux_ref("observability-stack"),
3601            sample_flux_ref("gateway"),
3602        ];
3603        let p = process_with_flux_resources(refs.clone());
3604        let observed = p.observed_flux_resources();
3605        assert_eq!(observed.len(), 2);
3606        assert_eq!(observed[0].name, "observability-stack");
3607        assert_eq!(observed[1].name, "gateway");
3608    }
3609
3610    #[test]
3611    fn observed_flux_resources_is_a_zero_copy_borrow_projection() {
3612        // Borrow-discipline pin: the returned slice borrows the
3613        // persisted `Vec<FluxResourceRef>` in place — NOT a fresh
3614        // allocation or a clone. A regression that switched the
3615        // projection to owned refs (via `.clone()` or `.to_vec()`)
3616        // would defeat the zero-copy contract the lift's primary
3617        // strict-widening delivers (the pre-lift 5-line chain
3618        // eagerly cloned the whole vector per reconcile pass; the
3619        // post-lift primitive borrows). Peer to the sibling
3620        // `flux_resource_ref_fetch_coords_returns_borrows_of_owned_slots`
3621        // pin on the per-ref borrow-projection axis.
3622        let refs = vec![sample_flux_ref("observability-stack")];
3623        let p = process_with_flux_resources(refs);
3624        let observed = p.observed_flux_resources();
3625        let persisted = &p.status.as_ref().unwrap().flux_resources;
3626        assert!(std::ptr::eq(observed.as_ptr(), persisted.as_ptr()));
3627    }
3628
3629    #[test]
3630    fn observed_flux_resources_is_a_pure_projection() {
3631        // Purity pin: calling the projection twice on the same
3632        // `Process` returns byte-identical slices (same pointer,
3633        // same length). A regression that introduced state — a
3634        // lazy-cached slice materialized on first call, a
3635        // normalization step that ran once and cached — would
3636        // surface here rather than as silent drift between the
3637        // VERIFY-phase and ATTEST-heartbeat consumers on the SAME
3638        // `Process` within one reconcile pass.
3639        let refs = vec![sample_flux_ref("observability-stack")];
3640        let p = process_with_flux_resources(refs);
3641        let a = p.observed_flux_resources();
3642        let b = p.observed_flux_resources();
3643        assert!(std::ptr::eq(a.as_ptr(), b.as_ptr()));
3644        assert_eq!(a.len(), b.len());
3645    }
3646
3647    #[test]
3648    fn observed_flux_resources_matches_pre_lift_reconciler_chain_shape() {
3649        // Byte-identical parity pin between the borrow-form primitive
3650        // here and the pre-lift `tatara-reconciler::phase_machine`
3651        // 5-line chain shape. Sweeps every corner every callsite
3652        // plausibly encounters (missing status, empty flux_resources,
3653        // populated flux_resources with one ref, populated with
3654        // multiple refs). A regression that inserted a normalization
3655        // step at the primitive the pre-lift chain does NOT apply —
3656        // or vice versa — surfaces here rather than as silent drift
3657        // between the pre-lift consumer sites and the ONE substrate
3658        // owner they now route through. Peer to
3659        // `coordinates_or_none_matches_pre_lift_reconciler_helper_shape`
3660        // on the metadata axis's borrow-form primitive.
3661        // `FluxResourceRef` does not derive `PartialEq` — the parity
3662        // check walks the per-ref fetch-coords tuple (the same 4-slot
3663        // borrow projection every downstream fetch consumer routes
3664        // through) so a regression that reshaped ANY slot at ANY
3665        // index surfaces here through the sibling
3666        // `FluxResourceRef::fetch_coords` typed projection.
3667        fn pre_lift(p: &Process) -> Vec<FluxResourceRef> {
3668            p.status
3669                .as_ref()
3670                .map(|s| s.flux_resources.clone())
3671                .unwrap_or_default()
3672        }
3673        fn coord_shape(refs: &[FluxResourceRef]) -> Vec<(String, String, String, String)> {
3674            refs.iter()
3675                .map(|r| {
3676                    let (ns, av, kind, name) = r.fetch_coords();
3677                    (
3678                        ns.to_string(),
3679                        av.to_string(),
3680                        kind.to_string(),
3681                        name.to_string(),
3682                    )
3683                })
3684                .collect()
3685        }
3686        // Missing status.
3687        let mut p = Process::new("api", empty_spec());
3688        p.status = None;
3689        assert_eq!(
3690            coord_shape(p.observed_flux_resources()),
3691            coord_shape(&pre_lift(&p))
3692        );
3693        // Populated status, empty slot.
3694        let p = process_with_flux_resources(vec![]);
3695        assert_eq!(
3696            coord_shape(p.observed_flux_resources()),
3697            coord_shape(&pre_lift(&p))
3698        );
3699        // Populated status, one ref.
3700        let p = process_with_flux_resources(vec![sample_flux_ref("obs")]);
3701        assert_eq!(
3702            coord_shape(p.observed_flux_resources()),
3703            coord_shape(&pre_lift(&p))
3704        );
3705        // Populated status, multiple refs.
3706        let p = process_with_flux_resources(vec![
3707            sample_flux_ref("obs"),
3708            sample_flux_ref("gw"),
3709            sample_flux_ref("api"),
3710        ]);
3711        assert_eq!(
3712            coord_shape(p.observed_flux_resources()),
3713            coord_shape(&pre_lift(&p))
3714        );
3715    }
3716
3717    #[test]
3718    fn observed_flux_resources_missing_status_and_empty_slot_collapse_to_the_same_slice_shape() {
3719        // Cross-corner coherence pin: the missing-`status` corner and
3720        // the populated-empty-slot corner return slices whose
3721        // `.is_empty()` / `.len()` observations are IDENTICAL. A
3722        // regression that promoted the missing-`status` corner to
3723        // returning `None` (via a signature change) — or that widened
3724        // the empty-slot corner to a synthetic single-element slice
3725        // — would surface here rather than as silent operator-facing
3726        // divergence between a never-status-written Process and a
3727        // status-emptied Process.
3728        let mut p_no_status = Process::new("api", empty_spec());
3729        p_no_status.status = None;
3730        let p_empty_status = process_with_flux_resources(vec![]);
3731        assert_eq!(
3732            p_no_status.observed_flux_resources().len(),
3733            p_empty_status.observed_flux_resources().len()
3734        );
3735        assert_eq!(
3736            p_no_status.observed_flux_resources().is_empty(),
3737            p_empty_status.observed_flux_resources().is_empty()
3738        );
3739    }
3740
3741    #[test]
3742    fn observed_flux_resources_slice_preserves_persisted_ordering() {
3743        // Ordering-preservation pin: the borrowed slice preserves
3744        // the exact insertion order of the persisted vector — no
3745        // sort, no dedup, no reshape. A regression that inserted a
3746        // sort or reordering would silently misroute per-ref
3747        // observations at the downstream VERIFY-phase / ATTEST-
3748        // heartbeat consumers, both of which walk the slice
3749        // positionally and correlate the position to the observed
3750        // readiness.
3751        let refs = vec![
3752            sample_flux_ref("z-last"),
3753            sample_flux_ref("a-first"),
3754            sample_flux_ref("m-middle"),
3755        ];
3756        let p = process_with_flux_resources(refs);
3757        let observed = p.observed_flux_resources();
3758        assert_eq!(observed[0].name, "z-last");
3759        assert_eq!(observed[1].name, "a-first");
3760        assert_eq!(observed[2].name, "m-middle");
3761    }
3762
3763    // ─── Process::observed_pid substrate pins ─────────────────────────
3764    //
3765    // Pins the borrow-form status-projection primitive on the PID axis
3766    // that owns the 3-line `.status.as_ref().and_then(|s| s.pid.clone())`
3767    // chain the two hand-authored `tatara-reconciler::phase_machine`
3768    // sites (`handle_forking` ALLOCATE-PID gate + `handle_exiting`
3769    // SIGTERM cascade) restated by hand pre-lift. Peer to the sibling
3770    // `observed_flux_resources_*` pin family on the flux-resources
3771    // axis; both compose the missing-`status` fallback + borrow-form
3772    // return-shape skeleton on distinct `ProcessStatus` slots. Fail-
3773    // before-pass-after granularity: `observed_pid` did not exist
3774    // pre-lift, so any test invoking it fails to compile pre-lift and
3775    // passes post-lift.
3776
3777    fn process_with_pid(pid: Option<&str>) -> Process {
3778        let mut p = Process::new("api-gateway", empty_spec());
3779        p.metadata.namespace = Some("prod".into());
3780        let mut status = ProcessStatus::default();
3781        status.pid = pid.map(str::to_string);
3782        p.status = Some(status);
3783        p
3784    }
3785
3786    #[test]
3787    fn observed_pid_returns_none_when_status_is_none() {
3788        // Missing-`status` corner pin: the primitive collapses the
3789        // no-status case to `None` so downstream `.is_some()` /
3790        // `if let Some(_)` / `.map(...)` behave identically on a
3791        // `Process` whose status field is `None` and on one whose
3792        // status carries an unpopulated `pid` slot. Matches the
3793        // pre-lift `.and_then(...)` chain's `None` byte-identically
3794        // at every reconciler consumer's downstream shape.
3795        let mut p = Process::new("api", empty_spec());
3796        p.status = None;
3797        assert!(p.observed_pid().is_none());
3798    }
3799
3800    #[test]
3801    fn observed_pid_returns_none_when_pid_slot_is_none() {
3802        // Empty-slot-under-populated-status corner pin: the
3803        // primitive returns `None`, matching the missing-`status`
3804        // corner byte-identically. A regression that treated the
3805        // two corners differently (a `None`-vs-`Some("")` signal
3806        // that downstream consumers could grep on) would silently
3807        // promote an internal representation detail (whether the
3808        // reconciler has ever written a status subresource) into
3809        // observable behavior at the ALLOCATE-PID gate.
3810        let p = process_with_pid(None);
3811        assert!(p.observed_pid().is_none());
3812    }
3813
3814    #[test]
3815    fn observed_pid_returns_borrowed_str_when_pid_slot_is_populated() {
3816        // Happy-path pin: with a populated `status.pid` slot, the
3817        // primitive returns a borrowed `&str` whose contents match
3818        // the persisted `String`. A regression that filtered /
3819        // reshaped / canonicalized the string would surface here
3820        // rather than as silent skew at the downstream cascade
3821        // comparator's `.as_deref() == Some(...)` equality check.
3822        let p = process_with_pid(Some("seph.1.7"));
3823        assert_eq!(p.observed_pid(), Some("seph.1.7"));
3824    }
3825
3826    #[test]
3827    fn observed_pid_is_a_zero_copy_borrow_projection() {
3828        // Borrow-discipline pin: the returned `&str` borrows the
3829        // persisted `String`'s underlying byte buffer in place —
3830        // NOT a fresh allocation or a clone. A regression that
3831        // switched the projection to an owned `String` (via
3832        // `.clone()` or `.to_owned()`) would defeat the zero-copy
3833        // contract the lift's primary strict-widening delivers
3834        // (the pre-lift 3-line chain eagerly cloned the `String`
3835        // per reconcile pass at BOTH call sites even though the
3836        // ALLOCATE-PID gate immediately dropped the clone and the
3837        // SIGTERM cascade only re-borrowed it via `.as_str()`; the
3838        // post-lift primitive borrows). Peer to the sibling
3839        // `observed_flux_resources_is_a_zero_copy_borrow_projection`
3840        // pin on the flux-resources borrow-projection axis.
3841        let p = process_with_pid(Some("seph.1.7"));
3842        let observed = p.observed_pid().expect("populated slot");
3843        let persisted = p.status.as_ref().unwrap().pid.as_ref().unwrap();
3844        assert!(std::ptr::eq(observed.as_ptr(), persisted.as_ptr()));
3845    }
3846
3847    #[test]
3848    fn observed_pid_is_a_pure_projection() {
3849        // Purity pin: calling the projection twice on the same
3850        // `Process` returns byte-identical `&str`s (same pointer,
3851        // same length). A regression that introduced state — a
3852        // lazy-cached slice materialized on first call, a
3853        // normalization step that ran once and cached — would
3854        // surface here rather than as silent drift between the
3855        // ALLOCATE-PID gate and the SIGTERM cascade on the SAME
3856        // `Process` within one reconcile pass.
3857        let p = process_with_pid(Some("seph.1.7"));
3858        let a = p.observed_pid().expect("populated slot");
3859        let b = p.observed_pid().expect("populated slot");
3860        assert!(std::ptr::eq(a.as_ptr(), b.as_ptr()));
3861        assert_eq!(a.len(), b.len());
3862    }
3863
3864    #[test]
3865    fn observed_pid_matches_pre_lift_reconciler_chain_shape() {
3866        // Byte-identical parity pin between the borrow-form
3867        // primitive here and the pre-lift `tatara-reconciler
3868        // ::phase_machine` 3-line chain shape. Sweeps every corner
3869        // every callsite plausibly encounters (missing status,
3870        // empty pid slot, populated pid slot). A regression that
3871        // inserted a normalization step at the primitive the pre-
3872        // lift chain does NOT apply — or vice versa — surfaces
3873        // here rather than as silent drift between the pre-lift
3874        // consumer sites and the ONE substrate owner they now
3875        // route through. Peer to
3876        // `observed_flux_resources_matches_pre_lift_reconciler_chain_shape`
3877        // on the flux-resources axis's borrow-form primitive.
3878        fn pre_lift(p: &Process) -> Option<String> {
3879            p.status.as_ref().and_then(|s| s.pid.clone())
3880        }
3881        // Missing status.
3882        let mut p = Process::new("api", empty_spec());
3883        p.status = None;
3884        assert_eq!(p.observed_pid().map(str::to_string), pre_lift(&p));
3885        // Populated status, empty pid slot.
3886        let p = process_with_pid(None);
3887        assert_eq!(p.observed_pid().map(str::to_string), pre_lift(&p));
3888        // Populated status, populated pid slot.
3889        let p = process_with_pid(Some("seph.1.7"));
3890        assert_eq!(p.observed_pid().map(str::to_string), pre_lift(&p));
3891    }
3892
3893    #[test]
3894    fn observed_pid_missing_status_and_empty_slot_collapse_to_the_same_option_shape() {
3895        // Cross-corner coherence pin: the missing-`status` corner
3896        // and the populated-empty-slot corner return `Option`s whose
3897        // `.is_none()` observations are IDENTICAL. A regression
3898        // that promoted the missing-`status` corner to returning a
3899        // typed error (via a signature change to `Result<_, _>`) —
3900        // or that widened the empty-slot corner to a synthetic
3901        // `Some("")` — would surface here rather than as silent
3902        // operator-facing divergence between a never-status-
3903        // written Process and a status-emptied Process on the
3904        // ALLOCATE-PID gate.
3905        let mut p_no_status = Process::new("api", empty_spec());
3906        p_no_status.status = None;
3907        let p_empty_slot = process_with_pid(None);
3908        assert_eq!(
3909            p_no_status.observed_pid().is_none(),
3910            p_empty_slot.observed_pid().is_none()
3911        );
3912        assert_eq!(
3913            p_no_status.observed_pid().is_some(),
3914            p_empty_slot.observed_pid().is_some()
3915        );
3916    }
3917
3918    #[test]
3919    fn observed_pid_preserves_hierarchical_pid_format() {
3920        // Format-preservation pin: the hierarchical PID path
3921        // (dotted-segment form `seph.1.7`, matching the ported
3922        // `convergence-controller/src/identity.rs` scheme) reaches
3923        // the caller with segments and separators byte-identical
3924        // to the persisted `String`. A regression that inserted a
3925        // canonicalization pass (a segment-count validator, a
3926        // separator swap `.` → `/`, a leading/trailing whitespace
3927        // trim) would silently misroute the SIGTERM cascade's
3928        // `spec.identity.parent == Some(pid)` comparator against
3929        // children whose `parent` field was authored in the ported
3930        // scheme's exact form.
3931        for pid in ["seph", "seph.1", "seph.1.7", "seph.1.7.42"] {
3932            let p = process_with_pid(Some(pid));
3933            assert_eq!(p.observed_pid(), Some(pid));
3934        }
3935    }
3936
3937    // ─── Process::observed_attestation substrate pins ─────────────────
3938    //
3939    // Pins the borrow-form status-projection primitive on the
3940    // attestation-chain axis that owns the 3-line
3941    // `.status.as_ref().and_then(|s| s.attestation.as_ref())` chain
3942    // the two hand-authored `tatara-reconciler` sites
3943    // (`phase_machine::advance_to_attested` ATTEST composer +
3944    // `render::render_export_jobs` export-Job builder) restated by
3945    // hand pre-lift. Peer to the sibling `observed_pid_*` +
3946    // `observed_flux_resources_*` pin families; all three compose
3947    // the missing-`status` fallback + borrow-form return-shape
3948    // skeleton on distinct `ProcessStatus` slots. Fail-before-pass-
3949    // after granularity: `observed_attestation` did not exist
3950    // pre-lift, so any test invoking it fails to compile pre-lift
3951    // and passes post-lift.
3952
3953    fn sample_attestation(artifact: &str, intent: &str) -> ProcessAttestation {
3954        // Distinct pillar strings so a regression that swapped the
3955        // artifact / intent pillars silently surfaces as an
3956        // equality failure at the composed-root parity pin.
3957        ProcessAttestation::initial(artifact.to_string(), None, intent.to_string())
3958    }
3959
3960    fn process_with_attestation(attestation: Option<ProcessAttestation>) -> Process {
3961        let mut p = Process::new("api-gateway", empty_spec());
3962        p.metadata.namespace = Some("prod".into());
3963        let mut status = ProcessStatus::default();
3964        status.attestation = attestation;
3965        p.status = Some(status);
3966        p
3967    }
3968
3969    #[test]
3970    fn observed_attestation_returns_none_when_status_is_none() {
3971        // Missing-`status` corner pin: the primitive collapses the
3972        // no-status case to `None` so downstream `.is_some()` /
3973        // `if let Some(_)` / `.map(...)` behave identically on a
3974        // `Process` whose status field is `None` and on one whose
3975        // status carries an unpopulated `attestation` slot.
3976        // Matches the pre-lift `.and_then(...)` chain's `None`
3977        // byte-identically at every reconciler consumer's
3978        // downstream shape.
3979        let mut p = Process::new("api", empty_spec());
3980        p.status = None;
3981        assert!(p.observed_attestation().is_none());
3982    }
3983
3984    #[test]
3985    fn observed_attestation_returns_none_when_attestation_slot_is_none() {
3986        // Empty-slot-under-populated-status corner pin: the
3987        // primitive returns `None`, matching the missing-`status`
3988        // corner byte-identically. A regression that treated the
3989        // two corners differently (a `None`-vs-`Some(_)` signal
3990        // that downstream consumers could grep on) would silently
3991        // promote an internal representation detail (whether the
3992        // reconciler has ever written a status subresource) into
3993        // observable behavior at the ATTEST composer's
3994        // seed-vs-chain branch.
3995        let p = process_with_attestation(None);
3996        assert!(p.observed_attestation().is_none());
3997    }
3998
3999    #[test]
4000    fn observed_attestation_returns_borrow_when_slot_is_populated() {
4001        // Happy-path pin: with a populated `status.attestation`
4002        // slot, the primitive returns a borrowed
4003        // `&ProcessAttestation` whose fields match the persisted
4004        // record. A regression that filtered / reshaped /
4005        // canonicalized the record would surface here rather than
4006        // as silent skew at the downstream `prior.next(pillars)`
4007        // chain composer + the ephemeral-export receipt's
4008        // `previous_root` linker.
4009        let att = sample_attestation("art-1", "int-1");
4010        let composed_root = att.composed_root.clone();
4011        let p = process_with_attestation(Some(att));
4012        let observed = p.observed_attestation().expect("populated slot");
4013        assert_eq!(observed.artifact_hash, "art-1");
4014        assert_eq!(observed.intent_hash, "int-1");
4015        assert_eq!(observed.composed_root, composed_root);
4016        assert_eq!(observed.generation, 0);
4017        assert!(observed.previous_root.is_none());
4018    }
4019
4020    #[test]
4021    fn observed_attestation_is_a_zero_copy_borrow_projection() {
4022        // Borrow-discipline pin: the returned reference points at
4023        // the persisted `ProcessAttestation` in place — NOT a fresh
4024        // allocation or a clone. A regression that switched the
4025        // projection to an owned `ProcessAttestation` (via
4026        // `.clone()`) would defeat the zero-copy contract the
4027        // lift's primary strict-widening delivers (the pre-lift
4028        // 3-line chain returned a borrow, but the export-Job
4029        // builder then cloned `composed_root` off it; the post-
4030        // lift primitive preserves the borrow all the way to the
4031        // consumer's own cloning choice). Peer to the sibling
4032        // `observed_pid_is_a_zero_copy_borrow_projection` +
4033        // `observed_flux_resources_is_a_zero_copy_borrow_projection`
4034        // pins on the PID + flux-resources borrow-projection axes.
4035        let att = sample_attestation("art-1", "int-1");
4036        let p = process_with_attestation(Some(att));
4037        let observed = p.observed_attestation().expect("populated slot") as *const _;
4038        let persisted = p.status.as_ref().unwrap().attestation.as_ref().unwrap() as *const _;
4039        assert!(std::ptr::eq(observed, persisted));
4040    }
4041
4042    #[test]
4043    fn observed_attestation_is_a_pure_projection() {
4044        // Purity pin: calling the projection twice on the same
4045        // `Process` returns byte-identical borrows (same pointer).
4046        // A regression that introduced state — a lazy-cached
4047        // reference materialized on first call, a normalization
4048        // step that ran once and cached — would surface here
4049        // rather than as silent drift between the ATTEST composer
4050        // and the ephemeral-export receipt chain on the SAME
4051        // `Process` within one reconcile pass.
4052        let att = sample_attestation("art-1", "int-1");
4053        let p = process_with_attestation(Some(att));
4054        let a = p.observed_attestation().expect("populated slot") as *const _;
4055        let b = p.observed_attestation().expect("populated slot") as *const _;
4056        assert!(std::ptr::eq(a, b));
4057    }
4058
4059    #[test]
4060    fn observed_attestation_matches_pre_lift_reconciler_chain_shape() {
4061        // Byte-identical parity pin between the borrow-form
4062        // primitive here and the pre-lift `tatara-reconciler`
4063        // 3-line chain shape. Sweeps every corner every callsite
4064        // plausibly encounters (missing status, empty attestation
4065        // slot, populated attestation slot). A regression that
4066        // inserted a normalization step at the primitive the pre-
4067        // lift chain does NOT apply — or vice versa — surfaces
4068        // here rather than as silent drift between the pre-lift
4069        // consumer sites and the ONE substrate owner they now
4070        // route through. Peer to
4071        // `observed_pid_matches_pre_lift_reconciler_chain_shape` +
4072        // `observed_flux_resources_matches_pre_lift_reconciler_chain_shape`
4073        // on the PID + flux-resources axes.
4074        // `ProcessAttestation` does not derive `PartialEq` — the
4075        // parity check walks the `composed_root` field (the
4076        // byte-string every downstream consumer keys off) so a
4077        // regression that reshaped the record without touching
4078        // the composed-root observation surfaces here through
4079        // the receipt-chain projection.
4080        fn pre_lift(p: &Process) -> Option<String> {
4081            p.status
4082                .as_ref()
4083                .and_then(|s| s.attestation.as_ref())
4084                .map(|a| a.composed_root.clone())
4085        }
4086        // Missing status.
4087        let mut p = Process::new("api", empty_spec());
4088        p.status = None;
4089        assert_eq!(
4090            p.observed_attestation().map(|a| a.composed_root.clone()),
4091            pre_lift(&p)
4092        );
4093        // Populated status, empty attestation slot.
4094        let p = process_with_attestation(None);
4095        assert_eq!(
4096            p.observed_attestation().map(|a| a.composed_root.clone()),
4097            pre_lift(&p)
4098        );
4099        // Populated status, populated attestation slot.
4100        let p = process_with_attestation(Some(sample_attestation("art-1", "int-1")));
4101        assert_eq!(
4102            p.observed_attestation().map(|a| a.composed_root.clone()),
4103            pre_lift(&p)
4104        );
4105    }
4106
4107    #[test]
4108    fn observed_attestation_missing_status_and_empty_slot_collapse_to_the_same_option_shape() {
4109        // Cross-corner coherence pin: the missing-`status` corner
4110        // and the populated-empty-slot corner return `Option`s
4111        // whose `.is_none()` observations are IDENTICAL. A
4112        // regression that promoted the missing-`status` corner to
4113        // returning a typed error (via a signature change to
4114        // `Result<_, _>`) — or that widened the empty-slot corner
4115        // to a synthetic `Some(default_attestation)` — would
4116        // surface here rather than as silent operator-facing
4117        // divergence between a never-status-written Process and
4118        // an attestation-emptied Process on the ATTEST composer's
4119        // seed-vs-chain branch.
4120        let mut p_no_status = Process::new("api", empty_spec());
4121        p_no_status.status = None;
4122        let p_empty_slot = process_with_attestation(None);
4123        assert_eq!(
4124            p_no_status.observed_attestation().is_none(),
4125            p_empty_slot.observed_attestation().is_none()
4126        );
4127        assert_eq!(
4128            p_no_status.observed_attestation().is_some(),
4129            p_empty_slot.observed_attestation().is_some()
4130        );
4131    }
4132
4133    #[test]
4134    fn observed_attestation_preserves_chain_generation_field() {
4135        // Generation-preservation pin: a chained attestation
4136        // (`prior.next(...)` at generation N ≥ 1 with a
4137        // `previous_root` linked to `prior.composed_root`) reaches
4138        // the caller with its `generation` counter + `previous_root`
4139        // link byte-identical to the persisted record. The pre-lift
4140        // ATTEST composer discriminated exactly on this borrow's
4141        // `Some(prior)` vs `None` arm; a regression that dropped
4142        // the chain's `generation` counter (say, by folding
4143        // `next(...)` into a fresh `initial(...)` on every
4144        // reconcile pass) would silently reset every chain and
4145        // orphan every downstream `previous_root` link, but that
4146        // drift is invisible to a Process CRD reader who only
4147        // observes the LATEST composed_root.
4148        let prior = sample_attestation("art-0", "int-0");
4149        let chained = prior.next("art-1".to_string(), None, "int-1".to_string());
4150        let expected_generation = chained.generation;
4151        let expected_previous = chained.previous_root.clone();
4152        let p = process_with_attestation(Some(chained));
4153        let observed = p.observed_attestation().expect("populated slot");
4154        assert_eq!(observed.generation, expected_generation);
4155        assert_eq!(observed.generation, 1);
4156        assert_eq!(observed.previous_root, expected_previous);
4157        assert_eq!(
4158            observed.previous_root.as_deref(),
4159            Some(prior.composed_root.as_str())
4160        );
4161    }
4162
4163    // ─── Process::observed_identity substrate pins ────────────────────
4164    //
4165    // The borrow-form status-projection primitive on the resolved-
4166    // identity axis. Collapses the paired 3-line `.status.as_ref()
4167    // .and_then(|s| s.identity.<clone|as_ref>())` chain every
4168    // consumer in `tatara-reconciler` restated by hand pre-lift at
4169    // TWO sites (`phase_machine::handle_forking` seed +
4170    // `ssapply::inject_annotations` content-hash annotation
4171    // composer). Peer to the sibling `observed_pid_*` +
4172    // `observed_attestation_*` + `observed_flux_resources_*` pin
4173    // families; all four compose the same missing-`status` fallback
4174    // + borrow-form return-shape skeleton on distinct
4175    // `ProcessStatus` slots. Each pin fails-before-pass-after
4176    // granularity: `observed_identity` did not exist pre-lift, so
4177    // any test invoking it fails to compile pre-lift and passes
4178    // post-lift.
4179
4180    fn sample_identity(name: &str) -> Identity {
4181        // Distinct name + content_hash + override flag so a
4182        // regression that reshaped one slot surfaces at the
4183        // populated-slot pin's field-equality check without
4184        // aliasing the sibling slots.
4185        Identity {
4186            name: name.to_string(),
4187            content_hash: "a".repeat(26),
4188            name_override: true,
4189        }
4190    }
4191
4192    fn process_with_identity(identity: Option<Identity>) -> Process {
4193        let mut p = Process::new("api-gateway", empty_spec());
4194        p.metadata.namespace = Some("prod".into());
4195        let mut status = ProcessStatus::default();
4196        status.identity = identity;
4197        p.status = Some(status);
4198        p
4199    }
4200
4201    #[test]
4202    fn observed_identity_returns_none_when_status_is_none() {
4203        // Missing-`status` corner pin: the primitive collapses the
4204        // no-status case to `None` so downstream `.is_some()` /
4205        // `if let Some(_)` / `.cloned().unwrap_or_else(...)` behave
4206        // identically on a `Process` whose status field is `None`
4207        // and on one whose status carries an unpopulated `identity`
4208        // slot. Matches the pre-lift `.and_then(...)` chain's `None`
4209        // byte-identically at every reconciler consumer's
4210        // downstream shape.
4211        let mut p = Process::new("api", empty_spec());
4212        p.status = None;
4213        assert!(p.observed_identity().is_none());
4214    }
4215
4216    #[test]
4217    fn observed_identity_returns_none_when_identity_slot_is_none() {
4218        // Empty-slot-under-populated-status corner pin: the
4219        // primitive returns `None`, matching the missing-`status`
4220        // corner byte-identically. A regression that treated the
4221        // two corners differently (a `None`-vs-`Some(_)` signal
4222        // that downstream consumers could grep on) would silently
4223        // promote an internal representation detail (whether the
4224        // reconciler has ever written a status subresource) into
4225        // observable behavior at the FORK-time `derive_identity`
4226        // fallback branch.
4227        let p = process_with_identity(None);
4228        assert!(p.observed_identity().is_none());
4229    }
4230
4231    #[test]
4232    fn observed_identity_returns_borrow_when_slot_is_populated() {
4233        // Happy-path pin: with a populated `status.identity` slot,
4234        // the primitive returns a borrowed `&Identity` whose fields
4235        // match the persisted record. A regression that filtered /
4236        // reshaped / canonicalized the record would surface here
4237        // rather than as silent skew at the FORK-time seed's
4238        // `.cloned().unwrap_or_else(derive_identity)` composition
4239        // + the SSA-time content-hash annotation stamp on the SAME
4240        // Process.
4241        let id = sample_identity("seph");
4242        let expected = id.clone();
4243        let p = process_with_identity(Some(id));
4244        let observed = p.observed_identity().expect("populated slot");
4245        assert_eq!(observed, &expected);
4246        assert_eq!(observed.name, "seph");
4247        assert_eq!(observed.content_hash, "a".repeat(26));
4248        assert!(observed.name_override);
4249    }
4250
4251    #[test]
4252    fn observed_identity_is_a_zero_copy_borrow_projection() {
4253        // Borrow-discipline pin: the returned reference points at
4254        // the persisted `Identity` in place — NOT a fresh
4255        // allocation or a clone. A regression that switched the
4256        // projection to an owned `Identity` (via `.clone()`) would
4257        // defeat the zero-copy contract the lift's primary strict-
4258        // widening delivers (the SSA-time consumer never clones the
4259        // whole `Identity`, only the `content_hash` field it stamps
4260        // onto the annotation map, so the borrow-form return
4261        // shape's happy-path allocation count is exactly ZERO).
4262        // Peer to the sibling
4263        // `observed_attestation_is_a_zero_copy_borrow_projection`
4264        // + `observed_pid_is_a_zero_copy_borrow_projection` +
4265        // `observed_flux_resources_is_a_zero_copy_borrow_projection`
4266        // pins on the attestation-chain + PID + flux-resources
4267        // borrow-projection axes.
4268        let id = sample_identity("seph");
4269        let p = process_with_identity(Some(id));
4270        let observed = p.observed_identity().expect("populated slot") as *const _;
4271        let persisted = p.status.as_ref().unwrap().identity.as_ref().unwrap() as *const _;
4272        assert!(std::ptr::eq(observed, persisted));
4273    }
4274
4275    #[test]
4276    fn observed_identity_is_a_pure_projection() {
4277        // Purity pin: calling the projection twice on the same
4278        // `Process` returns byte-identical borrows (same pointer).
4279        // A regression that introduced state — a lazy-cached
4280        // reference materialized on first call, a normalization
4281        // step that ran once and cached — would surface here
4282        // rather than as silent drift between the FORK-time
4283        // identity seed and the SSA-time content-hash annotation
4284        // stamp on the SAME `Process` within one reconcile pass.
4285        let p = process_with_identity(Some(sample_identity("seph")));
4286        let a = p.observed_identity().expect("populated slot") as *const _;
4287        let b = p.observed_identity().expect("populated slot") as *const _;
4288        assert!(std::ptr::eq(a, b));
4289    }
4290
4291    #[test]
4292    fn observed_identity_matches_pre_lift_reconciler_chain_shape() {
4293        // Byte-identical parity pin between the borrow-form
4294        // primitive here and the pre-lift `tatara-reconciler`
4295        // 3-line chain shape. Sweeps every corner every callsite
4296        // plausibly encounters (missing status, empty identity
4297        // slot, populated identity slot). A regression that
4298        // inserted a normalization step at the primitive the pre-
4299        // lift chain does NOT apply — or vice versa — surfaces
4300        // here rather than as silent drift between the pre-lift
4301        // consumer sites and the ONE substrate owner they now
4302        // route through. Peer to
4303        // `observed_attestation_matches_pre_lift_reconciler_chain_shape`
4304        // + `observed_pid_matches_pre_lift_reconciler_chain_shape`
4305        // + `observed_flux_resources_matches_pre_lift_reconciler_chain_shape`
4306        // on the attestation-chain + PID + flux-resources axes.
4307        fn pre_lift(p: &Process) -> Option<Identity> {
4308            p.status.as_ref().and_then(|s| s.identity.clone())
4309        }
4310        // Missing status.
4311        let mut p = Process::new("api", empty_spec());
4312        p.status = None;
4313        assert_eq!(p.observed_identity().cloned(), pre_lift(&p));
4314        // Populated status, empty identity slot.
4315        let p = process_with_identity(None);
4316        assert_eq!(p.observed_identity().cloned(), pre_lift(&p));
4317        // Populated status, populated identity slot.
4318        let p = process_with_identity(Some(sample_identity("seph")));
4319        assert_eq!(p.observed_identity().cloned(), pre_lift(&p));
4320    }
4321
4322    #[test]
4323    fn observed_identity_missing_status_and_empty_slot_collapse_to_the_same_option_shape() {
4324        // Cross-corner coherence pin: the missing-`status` corner
4325        // and the populated-empty-slot corner return `Option`s
4326        // whose `.is_none()` observations are IDENTICAL. A
4327        // regression that promoted the missing-`status` corner to
4328        // returning a typed error (via a signature change to
4329        // `Result<_, _>`) — or that widened the empty-slot corner
4330        // to a synthetic `Some(derive_identity(default_spec))` —
4331        // would surface here rather than as silent operator-facing
4332        // divergence between a never-status-written Process and an
4333        // identity-cleared Process on the FORK-time seed branch.
4334        let mut p_no_status = Process::new("api", empty_spec());
4335        p_no_status.status = None;
4336        let p_empty_slot = process_with_identity(None);
4337        assert_eq!(
4338            p_no_status.observed_identity().is_none(),
4339            p_empty_slot.observed_identity().is_none()
4340        );
4341        assert_eq!(
4342            p_no_status.observed_identity().is_some(),
4343            p_empty_slot.observed_identity().is_some()
4344        );
4345    }
4346
4347    #[test]
4348    fn observed_identity_cloned_composes_with_derive_identity_fallback() {
4349        // Cross-primitive composition pin: the borrow-form
4350        // primitive threaded through `.cloned().unwrap_or_else(||
4351        // derive_identity(...))` reproduces the pre-lift FORK-time
4352        // seed's owned-`Identity` shape at every corner. Binds the
4353        // exact composition the `phase_machine::handle_forking`
4354        // consumer performs: on the populated-slot corner the
4355        // reconciler-persisted `Identity` is returned verbatim (the
4356        // fallback never fires), and on both empty corners
4357        // (missing-status + empty-slot) the fallback fires
4358        // producing a fresh `derive_identity(&spec,
4359        // name_override)`. A regression that (a) swapped the
4360        // fallback direction, (b) made `.cloned()` re-derive
4361        // instead of clone, or (c) made the empty-slot corner
4362        // return a synthetic `Some(default_identity)` collides
4363        // with the fallback surfaces here rather than as silent
4364        // FORK-time PID allocator skew.
4365        let spec = empty_spec();
4366        let fallback_expected = crate::identity::derive_identity(&spec, None);
4367        // Populated-slot corner: the seed returns the persisted
4368        // identity, NOT the derive fallback.
4369        let persisted = sample_identity("seph");
4370        let p = process_with_identity(Some(persisted.clone()));
4371        let seed = p.observed_identity().cloned().unwrap_or_else(|| {
4372            crate::identity::derive_identity(&p.spec, p.declared_name_override())
4373        });
4374        assert_eq!(seed, persisted);
4375        assert_ne!(seed, fallback_expected);
4376        // Empty-slot corner: the seed fires the derive fallback.
4377        let p = process_with_identity(None);
4378        let seed = p.observed_identity().cloned().unwrap_or_else(|| {
4379            crate::identity::derive_identity(&p.spec, p.declared_name_override())
4380        });
4381        assert_eq!(seed, fallback_expected);
4382        // Missing-status corner: the seed fires the derive
4383        // fallback, byte-identical to the empty-slot corner.
4384        let mut p = Process::new("api-gateway", empty_spec());
4385        p.metadata.namespace = Some("prod".into());
4386        p.status = None;
4387        let seed = p.observed_identity().cloned().unwrap_or_else(|| {
4388            crate::identity::derive_identity(&p.spec, p.declared_name_override())
4389        });
4390        assert_eq!(seed, fallback_expected);
4391    }
4392
4393    // ─── Process::observed_phase substrate pins ───────────────────────
4394    //
4395    // The copy-form status-projection primitive on the phase axis.
4396    // Collapses the paired 3-line `.status.as_ref().map(|s| s.phase)`
4397    // chain every consumer in `tatara-reconciler` restated by hand
4398    // pre-lift at FIVE sites. Peer to the borrow-form
4399    // `observed_pid_*` + `observed_flux_resources_*` +
4400    // `observed_attestation_*` pin families; all four compose the
4401    // same missing-`status` fallback skeleton on distinct
4402    // `ProcessStatus` slots, with the phase-axis form returning
4403    // `Option<ProcessPhase>` (copy of a `Copy` scalar) rather than
4404    // `Option<&T>` (borrow) because the underlying slot is a bare
4405    // `ProcessPhase` — no allocation to borrow past, and the enum
4406    // is one byte on the wire. Each pin fails-before-pass-after
4407    // granularity: `observed_phase` did not exist pre-lift, so any
4408    // test invoking it fails to compile pre-lift and passes
4409    // post-lift.
4410
4411    fn process_with_phase(phase: Option<ProcessPhase>) -> Process {
4412        let mut p = Process::new("api-gateway", empty_spec());
4413        p.metadata.namespace = Some("prod".into());
4414        if let Some(ph) = phase {
4415            let mut status = ProcessStatus::default();
4416            status.phase = ph;
4417            p.status = Some(status);
4418        }
4419        p
4420    }
4421
4422    #[test]
4423    fn observed_phase_returns_none_when_status_is_none() {
4424        // Missing-`status` corner pin: the primitive collapses the
4425        // no-status case to `None` so downstream `.unwrap_or(...)`
4426        // at every reconciler consumer chooses the default
4427        // deliberately (`Pending` for the top-level dispatch seed
4428        // + boundary evaluator + routing groupby; `Attested` for
4429        // the released-from annotation composer). Matches the
4430        // pre-lift `.map(|s| s.phase)` chain's `None`
4431        // byte-identically at every consumer's downstream shape.
4432        let mut p = Process::new("api", empty_spec());
4433        p.status = None;
4434        assert!(p.observed_phase().is_none());
4435    }
4436
4437    #[test]
4438    fn observed_phase_returns_some_default_when_status_is_populated_with_default_phase() {
4439        // Populated-status corner pin: the primitive returns
4440        // `Some(ProcessPhase::default())` — a `ProcessStatus`
4441        // constructed via `default()` carries `phase: Pending`
4442        // because the phase field is a bare `ProcessPhase` (not
4443        // `Option<ProcessPhase>`), so there is NO "empty slot"
4444        // corner peer to the borrow-form projections' empty-slot
4445        // pins. A regression that reshaped the return type to
4446        // filter out `Pending` (treating it as "unset") would
4447        // surface here and silently break the top-level
4448        // dispatcher's Pending → Forking transition on a Process
4449        // freshly written by the reconciler.
4450        let p = process_with_phase(Some(ProcessPhase::default()));
4451        assert_eq!(p.observed_phase(), Some(ProcessPhase::Pending));
4452        assert_eq!(p.observed_phase(), Some(ProcessPhase::default()));
4453    }
4454
4455    #[test]
4456    fn observed_phase_returns_persisted_phase_when_status_is_populated() {
4457        // Happy-path pin: with a populated `status.phase` slot,
4458        // the primitive returns the persisted `ProcessPhase`.
4459        // A regression that filtered / reshaped / canonicalized
4460        // the phase would surface here rather than as silent
4461        // skew at the top-level dispatcher's phase handler
4462        // dispatch on the SAME Process.
4463        let p = process_with_phase(Some(ProcessPhase::Running));
4464        assert_eq!(p.observed_phase(), Some(ProcessPhase::Running));
4465    }
4466
4467    #[test]
4468    fn observed_phase_is_a_pure_projection() {
4469        // Purity pin: two consecutive calls return byte-identical
4470        // `Option<ProcessPhase>` values (no lazy materialization,
4471        // no interior mutation of `self`). Peer to the sibling
4472        // `observed_pid_is_a_pure_projection` +
4473        // `observed_flux_resources_is_a_pure_projection` +
4474        // `observed_attestation_is_a_pure_projection` pins; all
4475        // four bind the pure-projection discipline on the ONE
4476        // substrate accessor per status slot.
4477        let p = process_with_phase(Some(ProcessPhase::Attested));
4478        let a = p.observed_phase();
4479        let b = p.observed_phase();
4480        assert_eq!(a, b);
4481        assert_eq!(a, Some(ProcessPhase::Attested));
4482    }
4483
4484    #[test]
4485    fn observed_phase_matches_pre_lift_reconciler_chain_shape() {
4486        // Parity pin: sweeps the two corners every pre-lift
4487        // consumer plausibly encountered (missing status,
4488        // populated status with a particular phase) and compares
4489        // the substrate call against a hand-authored pre-lift
4490        // chain byte-identically. A regression that reshaped ANY
4491        // of the two corners would surface here rather than as
4492        // silent operator-facing skew between the top-level
4493        // dispatcher and any of the four other reconciler
4494        // consumers on the SAME `Process`.
4495        fn pre_lift(p: &Process) -> Option<ProcessPhase> {
4496            p.status.as_ref().map(|s| s.phase)
4497        }
4498        let mut p = Process::new("api", empty_spec());
4499        p.status = None;
4500        assert_eq!(p.observed_phase(), pre_lift(&p));
4501        let p = process_with_phase(Some(ProcessPhase::Running));
4502        assert_eq!(p.observed_phase(), pre_lift(&p));
4503        let p = process_with_phase(Some(ProcessPhase::Attested));
4504        assert_eq!(p.observed_phase(), pre_lift(&p));
4505        let p = process_with_phase(Some(ProcessPhase::Failed));
4506        assert_eq!(p.observed_phase(), pre_lift(&p));
4507    }
4508
4509    #[test]
4510    fn observed_phase_default_unwrap_matches_pre_lift_pending_default() {
4511        // Callsite-shape pin: three of the FIVE pre-lift consumers
4512        // (`controller::reconcile`, `boundary::evaluate_process_phase`,
4513        // `table_controller::stable_name_group_key`) closed the
4514        // 3-line chain with `.unwrap_or(ProcessPhase::Pending)`
4515        // (identical to `.unwrap_or_default()`). This pin binds
4516        // that call-site shape: `observed_phase().unwrap_or
4517        // (Pending)` returns `Pending` on missing status and the
4518        // persisted phase otherwise. A regression that swapped
4519        // the `None` sentinel's downstream default would surface
4520        // here rather than as silent skew at three of the five
4521        // consumer sites.
4522        let mut p = Process::new("api", empty_spec());
4523        p.status = None;
4524        assert_eq!(
4525            p.observed_phase().unwrap_or(ProcessPhase::Pending),
4526            ProcessPhase::Pending
4527        );
4528        let p = process_with_phase(Some(ProcessPhase::Running));
4529        assert_eq!(
4530            p.observed_phase().unwrap_or(ProcessPhase::Pending),
4531            ProcessPhase::Running
4532        );
4533    }
4534
4535    #[test]
4536    fn observed_phase_attested_unwrap_matches_pre_lift_released_from_default() {
4537        // Callsite-shape pin: the ONE pre-lift consumer
4538        // (`phase_machine::p_current_phase_str` — the
4539        // released-from annotation composer) closed the 3-line
4540        // chain with `.unwrap_or(ProcessPhase::Attested)` rather
4541        // than the `Default` (`Pending`). This pin binds that
4542        // call-site shape: `observed_phase().unwrap_or(Attested)`
4543        // returns `Attested` on missing status and the persisted
4544        // phase otherwise. A regression that folded the
4545        // `Attested`-default consumer into the `Pending`-default
4546        // majority would break the SIGSTOP/SIGCONT release gate's
4547        // "which annotation label to emit" branch — the pin binds
4548        // the primitive at the raw `Option<ProcessPhase>` form so
4549        // this default choice stays local at the callsite.
4550        let mut p = Process::new("api", empty_spec());
4551        p.status = None;
4552        assert_eq!(
4553            p.observed_phase().unwrap_or(ProcessPhase::Attested),
4554            ProcessPhase::Attested
4555        );
4556        let p = process_with_phase(Some(ProcessPhase::Failed));
4557        assert_eq!(
4558            p.observed_phase().unwrap_or(ProcessPhase::Attested),
4559            ProcessPhase::Failed
4560        );
4561    }
4562
4563    #[test]
4564    fn observed_phase_preserves_every_process_phase_variant() {
4565        // Round-trip pin: every `ProcessPhase` variant round-
4566        // trips through the primitive unchanged. Peer to the
4567        // sibling `observed_pid_preserves_hierarchical_pid_format`
4568        // pin's dotted-segment sweep; this pin sweeps the closed
4569        // set of `ProcessPhase` variants directly so a
4570        // canonicalization pass that dropped or reshaped one
4571        // (e.g. folded `Reconverging` back into `Execing`, or
4572        // remapped `Zombie` to `Reaped`) surfaces here rather
4573        // than as silent skew at the SIGSTOP/SIGCONT release
4574        // gate's phase-name annotation branch. Covers every
4575        // variant the `ProcessPhase::DeriveClosedSet` enumerates
4576        // so a future variant addition surfaces via the closed-
4577        // set macro rather than at a silent partial sweep.
4578        for phase in [
4579            ProcessPhase::Pending,
4580            ProcessPhase::Forking,
4581            ProcessPhase::Execing,
4582            ProcessPhase::Running,
4583            ProcessPhase::Attested,
4584            ProcessPhase::Reconverging,
4585            ProcessPhase::Releasing,
4586            ProcessPhase::Exiting,
4587            ProcessPhase::Failed,
4588            ProcessPhase::Zombie,
4589            ProcessPhase::Reaped,
4590        ] {
4591            let p = process_with_phase(Some(phase));
4592            assert_eq!(
4593                p.observed_phase(),
4594                Some(phase),
4595                "phase variant {phase:?} did not round-trip"
4596            );
4597        }
4598    }
4599
4600    // ─── Process::observed_phase_or_pending substrate pins ─────────────
4601    //
4602    // Pins the copy-form status-projection primitive on the phase
4603    // axis with the `Pending` sink applied. Sibling to the raw
4604    // `observed_phase_*` pin family on the (return-form × fallback
4605    // shape) axis pair — the raw-`Option` corner stays with the
4606    // sibling family; this pin family opens the `Pending`-defaulted
4607    // corner that four of the five pre-lift `observed_phase`
4608    // consumers wrote by hand. Fail-before-pass-after granularity:
4609    // `observed_phase_or_pending` did not exist pre-lift, so any
4610    // test invoking it fails to compile pre-lift and passes
4611    // post-lift.
4612
4613    #[test]
4614    fn observed_phase_or_pending_returns_pending_when_status_is_none() {
4615        // Missing-`status` corner pin: the primitive collapses the
4616        // no-status case to `Pending` — the sink four of the five
4617        // pre-lift `observed_phase` consumers wrote by hand
4618        // (`controller::reconcile` / `boundary::
4619        // evaluate_process_phase` / `table_controller::
4620        // stable_name_group_key` / `controller_pool::reconcile_pool`)
4621        // and the sentinel `ProcessPhase::default()` returns. A
4622        // regression that folded the `None` sink to any other phase
4623        // (e.g. `Forking` — treating "not yet observed" as "already
4624        // dispatched") would silently mis-seed the top-level
4625        // dispatcher's `Pending → Forking` transition and surface as
4626        // operator-visible reconcile-cycle skew on a freshly-forked
4627        // Process rather than at this pin.
4628        let mut p = Process::new("api", empty_spec());
4629        p.status = None;
4630        assert_eq!(p.observed_phase_or_pending(), ProcessPhase::Pending);
4631    }
4632
4633    #[test]
4634    fn observed_phase_or_pending_returns_persisted_phase_when_status_is_populated() {
4635        // Populated-status corner pin: the primitive passes through
4636        // the persisted `ProcessPhase` unchanged — the sink only
4637        // fires on missing `status`, not on a populated one carrying
4638        // a `Pending`-adjacent variant. Two variants pinned to
4639        // separate the "pass through the persisted phase" arm from
4640        // the "sink fires" arm: `Running` (mid-lifecycle) and
4641        // `Attested` (post-verify) both round-trip unchanged where
4642        // a regression that always returned `Pending` (dropped the
4643        // pass-through arm entirely) would surface here rather than
4644        // as silent skew at every reconciler's per-phase branch.
4645        let p = process_with_phase(Some(ProcessPhase::Running));
4646        assert_eq!(p.observed_phase_or_pending(), ProcessPhase::Running);
4647        let p = process_with_phase(Some(ProcessPhase::Attested));
4648        assert_eq!(p.observed_phase_or_pending(), ProcessPhase::Attested);
4649    }
4650
4651    #[test]
4652    fn observed_phase_or_pending_matches_pre_lift_unwrap_or_pending_chain_shape() {
4653        // Byte-identical parity pin: the primitive's return equals
4654        // the pre-lift two-link `.observed_phase().unwrap_or
4655        // (ProcessPhase::Pending)` chain at every one of the four
4656        // corner values (missing `status` → `Pending`, populated
4657        // with `Pending` → `Pending`, populated with a mid-lifecycle
4658        // variant → pass-through, populated with a terminal variant
4659        // → pass-through). A regression that swapped the sink to
4660        // `ProcessPhase::default()` (currently equivalent to
4661        // `Pending`) would keep this pin green until the enum's
4662        // `Default` impl drifted — the explicit `Pending` spelling
4663        // in the pin binds the operator-visible label rather than
4664        // the derived `Default`, so a future rename or reordering
4665        // of `ProcessPhase` variants that shifted `Default` off
4666        // `Pending` would surface here rather than as silent skew
4667        // at the four downstream consumer sites.
4668        let pre_lift = |p: &Process| p.observed_phase().unwrap_or(ProcessPhase::Pending);
4669        let mut p = Process::new("api", empty_spec());
4670        p.status = None;
4671        assert_eq!(p.observed_phase_or_pending(), pre_lift(&p));
4672        let p = process_with_phase(Some(ProcessPhase::Pending));
4673        assert_eq!(p.observed_phase_or_pending(), pre_lift(&p));
4674        let p = process_with_phase(Some(ProcessPhase::Running));
4675        assert_eq!(p.observed_phase_or_pending(), pre_lift(&p));
4676        let p = process_with_phase(Some(ProcessPhase::Reaped));
4677        assert_eq!(p.observed_phase_or_pending(), pre_lift(&p));
4678    }
4679
4680    #[test]
4681    fn observed_phase_or_pending_is_a_pure_projection() {
4682        // Purity pin: two back-to-back calls on the same `Process`
4683        // return the same `ProcessPhase` — the primitive stamps no
4684        // side effect (no clock read, no metadata write, no
4685        // `status` mutation) despite the sibling `observed_phase`
4686        // taking `&self` too. Peer to the sibling `observed_phase`
4687        // purity pin; a regression that folded a clock read (e.g.
4688        // "if the sink fired, stamp `phase_since = Utc::now()`")
4689        // into the primitive would surface here rather than at the
4690        // consumer sites' downstream reconcile-cycle behavior.
4691        let p = process_with_phase(Some(ProcessPhase::Running));
4692        let a = p.observed_phase_or_pending();
4693        let b = p.observed_phase_or_pending();
4694        assert_eq!(a, b);
4695    }
4696
4697    #[test]
4698    fn observed_phase_or_pending_preserves_every_process_phase_variant() {
4699        // Round-trip pin: every `ProcessPhase` variant round-trips
4700        // through the primitive unchanged when the `status` slot is
4701        // populated. Peer to the sibling `observed_phase_preserves
4702        // _every_process_phase_variant` sweep; this pin sweeps the
4703        // closed set through the `Pending`-sinked accessor rather
4704        // than the raw-`Option` accessor so a canonicalization pass
4705        // that dropped or reshaped one variant (e.g. folded
4706        // `Reconverging` back into `Execing`, remapped `Zombie` to
4707        // `Reaped`) surfaces at BOTH primitives' pin sets rather
4708        // than as silent skew at a subset of the reconciler
4709        // consumers. Covers every variant the
4710        // `ProcessPhase::DeriveClosedSet` enumerates so a future
4711        // variant addition surfaces via the closed-set macro rather
4712        // than at a silent partial sweep.
4713        for phase in [
4714            ProcessPhase::Pending,
4715            ProcessPhase::Forking,
4716            ProcessPhase::Execing,
4717            ProcessPhase::Running,
4718            ProcessPhase::Attested,
4719            ProcessPhase::Reconverging,
4720            ProcessPhase::Releasing,
4721            ProcessPhase::Exiting,
4722            ProcessPhase::Failed,
4723            ProcessPhase::Zombie,
4724            ProcessPhase::Reaped,
4725        ] {
4726            let p = process_with_phase(Some(phase));
4727            assert_eq!(
4728                p.observed_phase_or_pending(),
4729                phase,
4730                "phase variant {phase:?} did not round-trip through observed_phase_or_pending"
4731            );
4732        }
4733    }
4734
4735    // ─── Process::observed_phase_since substrate pins ──────────────────
4736    //
4737    // Pins the copy-form status-projection primitive on the
4738    // `status.phase_since` axis that owns the paired 5-line
4739    // `.status.as_ref().and_then(|s| s.phase_since).unwrap_or_else
4740    // (Utc::now)` chain the pool reconciler's per-owned-Process
4741    // `PoolMember { entered_state_at: … }` seed restated by hand pre-
4742    // lift. Peer to the sibling `observed_phase_*` +
4743    // `observed_identity_*` + `observed_attestation_*` +
4744    // `observed_flux_resources_*` + `observed_pid_*` + `created_at_*`
4745    // pin families — all six / seven primitives project a wire-format
4746    // `Option<T>` slot into a `Copy`-or-borrow inner value at ONE
4747    // owner. Fail-before-pass-after granularity: `observed_phase_since`
4748    // did not exist pre-lift, so any test invoking it fails to
4749    // compile pre-lift and passes post-lift.
4750
4751    fn process_with_phase_since(phase_since: Option<DateTime<Utc>>) -> Process {
4752        let mut p = Process::new("api-gateway", empty_spec());
4753        p.metadata.namespace = Some("prod".into());
4754        let mut status = ProcessStatus::default();
4755        status.phase_since = phase_since;
4756        p.status = Some(status);
4757        p
4758    }
4759
4760    #[test]
4761    fn observed_phase_since_returns_none_when_status_is_none() {
4762        // Missing-`status` corner pin: the primitive collapses the
4763        // no-status case to `None` so the pool reconciler's `PoolMember
4764        // { entered_state_at: p.observed_phase_since().unwrap_or_else
4765        // (Utc::now), .. }` seed synthesizes a "just entered" anchor
4766        // at its own tail rather than materializing a stale timestamp
4767        // at the substrate. Matches the pre-lift `.and_then(|s| s
4768        // .phase_since)` chain's `None` byte-identically at the
4769        // consumer's downstream tail.
4770        let mut p = Process::new("api", empty_spec());
4771        p.status = None;
4772        assert!(p.observed_phase_since().is_none());
4773    }
4774
4775    #[test]
4776    fn observed_phase_since_returns_none_when_slot_is_empty() {
4777        // Populated-status + empty-slot corner pin: a `ProcessStatus`
4778        // whose `phase_since` slot is `None` (a freshly-forked
4779        // Process whose reconciler has not yet stamped a first
4780        // transition) collapses to `None` at the primitive. The
4781        // paired-corner collapse with the missing-`status` corner
4782        // (both → `None`) matches what `.and_then` produces
4783        // structurally — one `None` cannot recover into a `Some` at
4784        // the flat outer wrapper. A regression that swapped the outer
4785        // combinator to `.map(|s| s.phase_since)` would flatten to
4786        // `Option<Option<_>>` and the compiler would reject the
4787        // signature, but a regression that "synthesized" a default
4788        // anchor at the substrate (e.g. `Utc::now()` on the empty
4789        // slot) would silently break the callsite's own
4790        // `.unwrap_or_else(Utc::now)` tail's semantics — the sink
4791        // fires ONCE at the callsite, not twice.
4792        let p = process_with_phase_since(None);
4793        assert!(p.observed_phase_since().is_none());
4794    }
4795
4796    #[test]
4797    fn observed_phase_since_returns_populated_timestamp_verbatim() {
4798        // Populated-slot corner pin: with a populated `status
4799        // .phase_since` slot, the primitive returns the persisted
4800        // `DateTime<Utc>` verbatim — no rounding, no timezone
4801        // stripping, no `Time` wrapper leaked. A regression that
4802        // canonicalized the timestamp (e.g. truncated to the second,
4803        // stripped the timezone marker) would surface here rather
4804        // than as silent skew at the pool reconciler's per-member
4805        // entered-state-at seed comparison against `Utc::now()`
4806        // downstream at `pool_phase_from_members`.
4807        let anchor = crate::time::seconds_ago(720);
4808        let p = process_with_phase_since(Some(anchor));
4809        assert_eq!(p.observed_phase_since(), Some(anchor));
4810    }
4811
4812    #[test]
4813    fn observed_phase_since_is_a_pure_projection() {
4814        // Purity pin: two consecutive calls return byte-identical
4815        // `Option<DateTime<Utc>>` values (no lazy materialization,
4816        // no interior mutation of `self`, no wall-clock read on the
4817        // empty corner). Peer to the sibling
4818        // `is_being_deleted_is_a_pure_projection` +
4819        // `created_at_is_a_pure_projection` +
4820        // `observed_phase_is_a_pure_projection` +
4821        // `observed_phase_or_pending_is_a_pure_projection` pins; all
4822        // five bind the pure-projection discipline on the ONE
4823        // substrate accessor per metadata / status slot. A
4824        // regression that folded the impure `Utc::now()` sink into
4825        // this primitive (rather than keeping it at the callsite's
4826        // `.unwrap_or_else(Utc::now)` tail alongside the sibling
4827        // `created_at` seed) would surface here as two consecutive
4828        // calls that returned distinct `Some(now_1)` /
4829        // `Some(now_2)` values.
4830        let anchor = crate::time::seconds_ago(5);
4831        let p = process_with_phase_since(Some(anchor));
4832        let a = p.observed_phase_since();
4833        let b = p.observed_phase_since();
4834        assert_eq!(a, b);
4835        assert_eq!(a, Some(anchor));
4836        // Empty-slot corner: pure `None`, not a fresh `Utc::now()`.
4837        let p_empty = process_with_phase_since(None);
4838        let a = p_empty.observed_phase_since();
4839        let b = p_empty.observed_phase_since();
4840        assert_eq!(a, b);
4841        assert!(a.is_none());
4842    }
4843
4844    #[test]
4845    fn observed_phase_since_matches_pre_lift_pool_reconciler_chain_shape() {
4846        // Byte-identical parity pin between the copy-form primitive
4847        // here and the pre-lift `tatara-pool-reconciler::
4848        // controller_pool::reconcile_inner` 5-line chain shape
4849        // (without the callsite's `.unwrap_or_else(Utc::now)` tail —
4850        // that tail stays at the callsite). Sweeps every corner
4851        // every pre-lift callsite plausibly encountered: missing
4852        // `status`, populated `status` + empty `phase_since` slot,
4853        // populated `status` + populated `phase_since` slot. A
4854        // regression that inserted a normalization step at the
4855        // primitive the pre-lift chain does NOT apply — or vice
4856        // versa — surfaces here rather than as silent drift between
4857        // the pre-lift consumer site and the ONE substrate owner it
4858        // now routes through.
4859        fn pre_lift(p: &Process) -> Option<DateTime<Utc>> {
4860            p.status.as_ref().and_then(|s| s.phase_since)
4861        }
4862        // Missing status.
4863        let mut p = Process::new("x", empty_spec());
4864        p.status = None;
4865        assert_eq!(p.observed_phase_since(), pre_lift(&p));
4866        // Populated status, empty slot.
4867        let p = process_with_phase_since(None);
4868        assert_eq!(p.observed_phase_since(), pre_lift(&p));
4869        // Populated status, populated slot.
4870        let anchor = crate::time::seconds_ago(90);
4871        let p = process_with_phase_since(Some(anchor));
4872        assert_eq!(p.observed_phase_since(), pre_lift(&p));
4873    }
4874
4875    #[test]
4876    fn observed_phase_since_missing_status_and_empty_slot_collapse_to_the_same_option_shape() {
4877        // Cross-corner coherence pin: the missing-`status` corner
4878        // AND the populated-empty-slot corner return `Option`s
4879        // whose `.is_none()` observations are IDENTICAL — a
4880        // shape peer to `observed_identity_missing_status_and_empty
4881        // _slot_collapse_to_the_same_option_shape`. A regression
4882        // that promoted the missing-`status` corner to returning a
4883        // typed error (via a signature change to `Result<_, _>`) —
4884        // or that widened the empty-slot corner to a synthetic
4885        // `Some(Utc::now())` at the substrate — would surface here
4886        // rather than as silent operator-facing divergence between
4887        // a never-status-written Process and a phase-since-cleared
4888        // Process at the pool reconciler's per-member row builder.
4889        let mut p_no_status = Process::new("api", empty_spec());
4890        p_no_status.status = None;
4891        let p_empty_slot = process_with_phase_since(None);
4892        assert_eq!(
4893            p_no_status.observed_phase_since().is_none(),
4894            p_empty_slot.observed_phase_since().is_none()
4895        );
4896        assert_eq!(
4897            p_no_status.observed_phase_since().is_some(),
4898            p_empty_slot.observed_phase_since().is_some()
4899        );
4900    }
4901
4902    #[test]
4903    fn observed_phase_since_composes_with_unwrap_or_else_utc_now_tail_at_pool_seed() {
4904        // Call-site-shape pin: the `tatara-pool-reconciler::
4905        // controller_pool::reconcile_inner` per-owned-Process
4906        // `PoolMember { entered_state_at: … }` seed composes
4907        // `p.observed_phase_since().unwrap_or_else(Utc::now)`. A
4908        // regression that returned `Some(Utc::now())` on the empty
4909        // corner (folding the sink into the primitive) would break
4910        // the observable contract that a caller with a distinct
4911        // now-source (e.g. an injected `time_source: impl Fn() ->
4912        // DateTime<Utc>`, or a test-time frozen clock) could
4913        // substitute at the tail — this pin binds the empty-corner
4914        // shape by observing that the substrate returns `None` (so
4915        // the `.unwrap_or_else` runs at the callsite) and that the
4916        // populated-corner shape is byte-identical between the
4917        // substrate `Some(anchor)` and the composed
4918        // `Some(anchor).unwrap_or_else(...)` (the fallback never
4919        // fires when the corner is populated). Peer to
4920        // `created_at_composes_with_signed_duration_since_at_ttl_gate`
4921        // on the metadata-timestamp side — both bind the
4922        // composition shape at the callsite so a substrate-side
4923        // refactor cannot silently break the tail semantics.
4924        let anchor = crate::time::seconds_ago(30);
4925        // Populated corner: substrate returns `Some(anchor)` and
4926        // the composed tail returns `anchor` (fallback silent).
4927        let p = process_with_phase_since(Some(anchor));
4928        let composed = p.observed_phase_since().unwrap_or_else(Utc::now);
4929        assert_eq!(composed, anchor);
4930        // Empty corner: substrate returns `None` and the composed
4931        // tail fires `Utc::now()` at the callsite (observed as a
4932        // timestamp >= a `before` sample AND close to now).
4933        let before = Utc::now();
4934        let p = process_with_phase_since(None);
4935        assert!(p.observed_phase_since().is_none());
4936        let composed = p.observed_phase_since().unwrap_or_else(Utc::now);
4937        assert!(composed >= before);
4938        assert!(composed <= Utc::now() + chrono::Duration::seconds(1));
4939    }
4940
4941    // ─── Process::is_being_deleted substrate pins ───────────────────────
4942    //
4943    // Pins the copy-form metadata-projection primitive on the
4944    // deletion-tombstone axis. Peer to the borrow-form + copy-form
4945    // metadata-fallback family (`namespace_or_default`,
4946    // `name_or_placeholder`, `uid_or_empty`, `coordinates_or_defaults`,
4947    // `coordinates_or_none`, `owned_coordinates_or_err`, `annotation`);
4948    // this one opens the presence-probe corner for the tombstone slot.
4949    // Fail-before-pass-after granularity: `is_being_deleted` did not
4950    // exist pre-lift, so any test invoking it fails to compile pre-
4951    // lift and passes post-lift.
4952
4953    fn tombstoned_process() -> Process {
4954        let mut p = Process::new("api-gateway", empty_spec());
4955        p.metadata.namespace = Some("prod".into());
4956        // Routes through the ONE substrate composer
4957        // `tatara_process::time::tombstone_now` — one of 12 pre-lift
4958        // exact-match sites past the ★★ PRIME-DIRECTIVE ≥ 2 threshold
4959        // for the `Some(Time(Utc::now()))` wire shape.
4960        p.metadata.deletion_timestamp = crate::time::tombstone_now();
4961        p
4962    }
4963
4964    #[test]
4965    fn is_being_deleted_returns_false_when_deletion_timestamp_is_absent() {
4966        // Missing-tombstone corner pin: the primitive collapses the
4967        // no-tombstone case to `false` so the SIGTERM preempt at
4968        // `controller::reconcile` skips the `→ Exiting` forcing
4969        // branch and the DELETE-skip at `handle_exiting`'s child
4970        // fan-out does NOT `continue` past a child that is still
4971        // healthy. Matches the pre-lift `.is_some()` chain's `false`
4972        // byte-identically at every consumer's downstream gate.
4973        let mut p = Process::new("api", empty_spec());
4974        p.metadata.deletion_timestamp = None;
4975        assert!(!p.is_being_deleted());
4976    }
4977
4978    #[test]
4979    fn is_being_deleted_returns_true_when_deletion_timestamp_is_present() {
4980        // Present-tombstone corner pin: the primitive returns
4981        // `true` on any populated `metadata.deletionTimestamp`
4982        // slot regardless of the timestamp payload — the two
4983        // consumers only read the tombstone's PRESENCE, never
4984        // its RFC-3339 timestamp value. A regression that gated
4985        // the `true` return on the timestamp being non-epoch, or
4986        // parsed the timestamp before returning, would surface
4987        // here rather than as silent skew at the SIGTERM preempt
4988        // or child-fan-out DELETE-skip on the SAME `Process`.
4989        let p = tombstoned_process();
4990        assert!(p.is_being_deleted());
4991    }
4992
4993    #[test]
4994    fn is_being_deleted_is_a_pure_projection() {
4995        // Purity pin: two consecutive calls return byte-identical
4996        // `bool` values (no lazy materialization, no interior
4997        // mutation of `self`). Peer to the sibling
4998        // `observed_phase_is_a_pure_projection` +
4999        // `observed_pid_is_a_pure_projection` +
5000        // `observed_flux_resources_is_a_pure_projection` +
5001        // `observed_attestation_is_a_pure_projection` pins; all
5002        // five bind the pure-projection discipline on the ONE
5003        // substrate accessor per metadata / status slot.
5004        let p = tombstoned_process();
5005        let a = p.is_being_deleted();
5006        let b = p.is_being_deleted();
5007        assert_eq!(a, b);
5008        assert!(a);
5009    }
5010
5011    #[test]
5012    fn is_being_deleted_matches_pre_lift_reconciler_chain_shape() {
5013        // Parity pin: sweeps the two corners every pre-lift
5014        // consumer plausibly encountered (missing tombstone,
5015        // present tombstone) and compares the substrate call
5016        // against a hand-authored pre-lift chain byte-identically.
5017        // A regression that reshaped either corner would surface
5018        // here rather than as silent operator-facing skew between
5019        // the top-level dispatcher's SIGTERM preempt and the
5020        // SIGTERM cascade's child-fan-out DELETE-skip on the
5021        // SAME `Process` within one reconcile pass.
5022        fn pre_lift(p: &Process) -> bool {
5023            p.metadata.deletion_timestamp.is_some()
5024        }
5025        let mut p = Process::new("api", empty_spec());
5026        p.metadata.deletion_timestamp = None;
5027        assert_eq!(p.is_being_deleted(), pre_lift(&p));
5028        let p = tombstoned_process();
5029        assert_eq!(p.is_being_deleted(), pre_lift(&p));
5030    }
5031
5032    #[test]
5033    fn is_being_deleted_composes_with_process_phase_is_alive_at_reconcile_preempt() {
5034        // Call-site-shape pin: the `controller::reconcile` SIGTERM
5035        // preempt composes `is_being_deleted() && current_phase
5036        // .is_alive()` — the tombstone-presence probe AND the
5037        // alive-phase gate must BOTH hold to force `→ Exiting`.
5038        // A dead-phase (`Zombie` / `Reaped` / `Failed`) Process
5039        // that carries a tombstone still runs its normal handler,
5040        // not the preempt. This pin binds that composition shape
5041        // at the primitive so a regression that flipped either
5042        // half of the `&&` (or that broadened the tombstone probe
5043        // to include the `is_alive` half implicitly) surfaces
5044        // here rather than as silent skew at the top-level
5045        // dispatch on the SAME `Process`.
5046        let mut p = tombstoned_process();
5047        // Alive + tombstoned → preempt fires.
5048        let mut alive = ProcessStatus::default();
5049        alive.phase = ProcessPhase::Running;
5050        p.status = Some(alive);
5051        assert!(p.is_being_deleted());
5052        assert!(p.observed_phase().unwrap_or_default().is_alive());
5053        // Dead + tombstoned → preempt does NOT fire (composition
5054        // with `is_alive` returns false).
5055        let mut dead = ProcessStatus::default();
5056        dead.phase = ProcessPhase::Reaped;
5057        p.status = Some(dead);
5058        assert!(p.is_being_deleted());
5059        assert!(!p.observed_phase().unwrap_or_default().is_alive());
5060    }
5061
5062    // ─── Process::created_at substrate pins ─────────────────────────
5063    //
5064    // Pins the copy-form metadata-projection primitive on the
5065    // `metadata.creationTimestamp` axis that owns the
5066    // `.metadata.creation_timestamp.as_ref().map(|t| t.0)` chain the
5067    // three hand-authored sites (`lifetime_clock::evaluate`,
5068    // `lifetime_clock::requeue_with_ttl`,
5069    // `tatara-reconciler::table_controller`) restated by hand pre-lift.
5070    // Peer to the sibling `is_being_deleted_*` +
5071    // `observed_phase_*` pin families — all three primitives project a
5072    // wire-format `Option<T>` slot into a `Copy` inner value at ONE
5073    // owner. Fail-before-pass-after granularity: `created_at` did not
5074    // exist pre-lift, so any test invoking it fails to compile pre-lift
5075    // and passes post-lift.
5076
5077    fn creation_stamped_process(t: DateTime<Utc>) -> Process {
5078        let mut p = Process::new("age-anchor", empty_spec());
5079        p.metadata.namespace = Some("prod".into());
5080        p.metadata.creation_timestamp =
5081            Some(k8s_openapi::apimachinery::pkg::apis::meta::v1::Time(t));
5082        p
5083    }
5084
5085    #[test]
5086    fn created_at_returns_none_when_creation_timestamp_is_absent() {
5087        // Missing-slot corner pin: the primitive collapses the
5088        // no-creation-timestamp case to `None` so the TTL-expiry gate
5089        // at `lifetime_clock::evaluate` short-circuits its inner
5090        // `if let Some(...)` branch (no elapsed computation), the
5091        // requeue-budget picker returns its default sleep, and the
5092        // stable-name arbiter's `.unwrap_or_else(Utc::now)` tail
5093        // synthesizes a "just created" anchor at its own site. Matches
5094        // the pre-lift `.as_ref().map(|t| t.0)` chain's `None`
5095        // byte-identically at every consumer's downstream tail.
5096        let mut p = Process::new("api", empty_spec());
5097        p.metadata.creation_timestamp = None;
5098        assert!(p.created_at().is_none());
5099    }
5100
5101    #[test]
5102    fn created_at_returns_some_datetime_when_slot_is_populated() {
5103        // Populated-slot corner pin: with a populated
5104        // `metadata.creationTimestamp` slot, the primitive unwraps the
5105        // wire-format `Time` newtype to its inner `DateTime<Utc>` and
5106        // returns it as `Some(datetime)` — hiding the `.0` field-access
5107        // every pre-lift consumer restated to reach the underlying
5108        // instant.
5109        let anchor = crate::time::seconds_ago(300);
5110        let p = creation_stamped_process(anchor);
5111        assert_eq!(p.created_at(), Some(anchor));
5112    }
5113
5114    #[test]
5115    fn created_at_is_a_pure_projection() {
5116        // Purity pin: two consecutive calls return byte-identical
5117        // `Option<DateTime<Utc>>` values (no lazy materialization, no
5118        // interior mutation of `self`). Peer to the sibling
5119        // `is_being_deleted_is_a_pure_projection` +
5120        // `observed_phase_is_a_pure_projection` pins; all three bind
5121        // the pure-projection discipline on the ONE substrate accessor
5122        // per metadata / status slot.
5123        let anchor = Utc::now();
5124        let p = creation_stamped_process(anchor);
5125        let a = p.created_at();
5126        let b = p.created_at();
5127        assert_eq!(a, b);
5128        assert_eq!(a, Some(anchor));
5129    }
5130
5131    #[test]
5132    fn created_at_matches_pre_lift_creation_timestamp_chain_shape() {
5133        // Parity pin: sweeps the two corners every pre-lift consumer
5134        // plausibly encountered (missing slot, populated slot) and
5135        // compares the substrate call against a hand-authored pre-lift
5136        // chain byte-identically. A regression that reshaped either
5137        // corner (returning `Some(Utc::now())` on the missing slot,
5138        // returning a rounded / truncated timestamp on the populated
5139        // slot) would surface here rather than as silent operator-
5140        // facing skew between the TTL-expiry gate, the requeue-budget
5141        // picker, and the stable-name claim-arbiter tie-break on the
5142        // SAME `Process` within one reconcile pass.
5143        fn pre_lift(p: &Process) -> Option<DateTime<Utc>> {
5144            p.metadata.creation_timestamp.as_ref().map(|t| t.0)
5145        }
5146        // Missing slot.
5147        let mut p = Process::new("x", empty_spec());
5148        p.metadata.creation_timestamp = None;
5149        assert_eq!(p.created_at(), pre_lift(&p));
5150        // Populated slot.
5151        let anchor = crate::time::seconds_ago(42);
5152        let p = creation_stamped_process(anchor);
5153        assert_eq!(p.created_at(), pre_lift(&p));
5154    }
5155
5156    #[test]
5157    fn created_at_composes_with_signed_duration_since_at_ttl_gate() {
5158        // Call-site-shape pin: the `lifetime_clock::evaluate` TTL-
5159        // expiry gate composes `now.signed_duration_since(creation)`
5160        // where `creation` is the `DateTime<Utc>` returned by this
5161        // primitive's `Some` corner. A regression that returned a
5162        // per-callsite `Local` timezone (or that stripped the timezone
5163        // marker) would break the arithmetic silently. This pin
5164        // computes the elapsed duration byte-identically against the
5165        // pre-lift `.map(|t| t.0)` chain so a timezone drift surfaces
5166        // here rather than as silent skew at the TTL-expiry decision
5167        // on the SAME `Process` within one reconcile pass.
5168        let now = Utc::now();
5169        let anchor = now - chrono::Duration::seconds(120);
5170        let p = creation_stamped_process(anchor);
5171        let via_primitive = p.created_at().expect("populated slot");
5172        let via_pre_lift = p
5173            .metadata
5174            .creation_timestamp
5175            .as_ref()
5176            .map(|t| t.0)
5177            .expect("populated slot");
5178        assert_eq!(
5179            now.signed_duration_since(via_primitive),
5180            now.signed_duration_since(via_pre_lift)
5181        );
5182    }
5183
5184    // ─── Process::created_at_or substrate pins ──────────────────────
5185    //
5186    // Pins the pure composer over `Process::created_at` that owns the
5187    // paired `.created_at().unwrap_or_else(Utc::now)` chain the two
5188    // production consumers restated by hand pre-lift
5189    // (`tatara-reconciler::table_controller::reconcile_process_table`
5190    // + `tatara-pool-reconciler::controller_pool::reconcile_inner`).
5191    // Fail-before-pass-after granularity: `created_at_or` did not
5192    // exist pre-lift, so any test invoking it fails to compile
5193    // pre-lift and passes post-lift.
5194
5195    #[test]
5196    fn created_at_or_returns_fallback_when_creation_timestamp_is_absent() {
5197        // Missing-slot corner pin: the composer collapses the
5198        // no-creation-timestamp case to the caller's fallback anchor
5199        // byte-identically to the pre-lift `.unwrap_or(fallback)`
5200        // tail. A freshly-forked Process whose API server has not yet
5201        // stamped `metadata.creationTimestamp` gets the caller's
5202        // wall-clock read (or a test's frozen anchor) synthesized so
5203        // downstream dwell-time / tie-break arithmetic proceeds
5204        // without a special-case branch at each consumer.
5205        let mut p = Process::new("api", empty_spec());
5206        p.metadata.creation_timestamp = None;
5207        let fallback = crate::time::seconds_ago(42);
5208        assert_eq!(p.created_at_or(fallback), fallback);
5209    }
5210
5211    #[test]
5212    fn created_at_or_returns_anchor_when_slot_is_populated() {
5213        // Populated-slot corner pin: with a populated
5214        // `metadata.creationTimestamp` slot, the composer ignores the
5215        // caller's fallback and returns the observed anchor
5216        // byte-identically to the pre-lift `.unwrap_or(fallback)`
5217        // pass-through. Sibling to `created_at_returns_some_datetime_
5218        // when_slot_is_populated` — that pin binds the pure projection,
5219        // this pin binds the composer's pass-through on the same
5220        // populated corner.
5221        let anchor = crate::time::seconds_ago(300);
5222        let p = creation_stamped_process(anchor);
5223        let unrelated_fallback = Utc::now() + chrono::Duration::seconds(9_999);
5224        assert_eq!(p.created_at_or(unrelated_fallback), anchor);
5225    }
5226
5227    #[test]
5228    fn created_at_or_is_pure_over_the_fallback_argument() {
5229        // Purity pin: the composer itself never reads the wall clock —
5230        // two consecutive calls with the SAME fallback return
5231        // byte-identical `DateTime<Utc>` values on both the missing-
5232        // slot corner (both calls return the caller's fallback) and
5233        // the populated-slot corner (both calls return the observed
5234        // anchor). Peer to the sibling
5235        // `created_at_is_a_pure_projection` pin; both bind the pure-
5236        // projection / pure-composer discipline on the ONE substrate
5237        // accessor per axis.
5238        let fallback = crate::time::seconds_ago(7);
5239        // Missing slot.
5240        let mut p = Process::new("x", empty_spec());
5241        p.metadata.creation_timestamp = None;
5242        assert_eq!(p.created_at_or(fallback), p.created_at_or(fallback));
5243        // Populated slot.
5244        let anchor = crate::time::seconds_ago(120);
5245        let p = creation_stamped_process(anchor);
5246        assert_eq!(p.created_at_or(fallback), p.created_at_or(fallback));
5247    }
5248
5249    #[test]
5250    fn created_at_or_matches_pre_lift_unwrap_or_chain_shape() {
5251        // Parity pin: sweeps the two corners every pre-lift consumer
5252        // encountered (missing slot, populated slot) and compares the
5253        // substrate call against the hand-authored pre-lift
5254        // `.created_at().unwrap_or(fallback)` chain byte-identically.
5255        // A regression that reshaped either corner (returning the
5256        // fallback on a populated slot, returning `Utc::now()` on the
5257        // missing slot regardless of the caller's fallback) would
5258        // surface here rather than as silent operator-facing skew
5259        // between the claim-arbiter's tie-break anchor and the pool
5260        // convergence snapshot's dwell-time anchor on the SAME
5261        // `Process` within one reconcile pass.
5262        fn pre_lift(p: &Process, fallback: DateTime<Utc>) -> DateTime<Utc> {
5263            p.created_at().unwrap_or(fallback)
5264        }
5265        let fallback = crate::time::seconds_ago(13);
5266        // Missing slot.
5267        let mut p = Process::new("x", empty_spec());
5268        p.metadata.creation_timestamp = None;
5269        assert_eq!(p.created_at_or(fallback), pre_lift(&p, fallback));
5270        // Populated slot.
5271        let anchor = crate::time::seconds_ago(42);
5272        let p = creation_stamped_process(anchor);
5273        assert_eq!(p.created_at_or(fallback), pre_lift(&p, fallback));
5274    }
5275
5276    #[test]
5277    fn created_at_or_composes_with_utc_now_at_reconciler_callsites() {
5278        // Call-site-shape pin: the two production consumers
5279        // (`table_controller::reconcile_process_table` +
5280        // `controller_pool::reconcile_inner`) both call
5281        // `p.created_at_or(Utc::now())`. On the populated corner the
5282        // wall-clock fallback is irrelevant (the observed anchor
5283        // wins); on the missing corner the fallback becomes the
5284        // resolved value within the sub-second window between the
5285        // caller's `Utc::now()` read and the assertion below. This
5286        // pin binds that the callsite composition returns the
5287        // observed anchor exactly on the populated corner (the
5288        // stable, drift-free assertion) and a "recent" wall-clock
5289        // read on the missing corner (bounded within a two-second
5290        // window to absorb scheduler jitter). A regression that
5291        // silently substituted a different fallback (`DateTime::MIN`,
5292        // a per-cluster prefix offset, a hardcoded epoch) would
5293        // surface at the second half of this pin.
5294        // Populated corner: byte-identical to the observed anchor.
5295        let anchor = crate::time::seconds_ago(600);
5296        let p = creation_stamped_process(anchor);
5297        assert_eq!(p.created_at_or(Utc::now()), anchor);
5298        // Missing corner: within a two-second wall-clock window.
5299        let mut p = Process::new("x", empty_spec());
5300        p.metadata.creation_timestamp = None;
5301        let before = Utc::now();
5302        let resolved = p.created_at_or(Utc::now());
5303        let after = Utc::now();
5304        assert!(
5305            resolved >= before - chrono::Duration::seconds(2),
5306            "resolved {resolved} is before window start {before}"
5307        );
5308        assert!(
5309            resolved <= after + chrono::Duration::seconds(2),
5310            "resolved {resolved} is after window end {after}"
5311        );
5312    }
5313
5314    // ─── Process::resolved_ephemeral substrate pins ─────────────────
5315    //
5316    // Pins the compound spec-projection primitive on the
5317    // `spec.lifetime` axis that owns the ambiguity-aware
5318    // `resolved_ephemeral` chain the three hand-authored sites
5319    // (`lifetime_clock::evaluate`, `lifetime_clock::requeue_with_ttl`,
5320    // `tatara-reconciler::render::render_export_jobs`) restated by
5321    // hand pre-lift through TWO different chains that disagreed on
5322    // the ambiguous corner. Fail-before-pass-after granularity:
5323    // `resolved_ephemeral` did not exist pre-lift on `impl Process`,
5324    // so any test invoking it fails to compile pre-lift and passes
5325    // post-lift.
5326
5327    fn permanent_only_process() -> Process {
5328        let mut spec = empty_spec();
5329        // Routes through the ONE substrate composer
5330        // [`crate::lifetime::Lifetime::permanent`] — one of FOUR
5331        // pre-lift exact-match sites past the ★★ PRIME-DIRECTIVE ≥ 2
5332        // threshold; see the composer's doc-comment for the full
5333        // migration rationale.
5334        spec.lifetime = crate::lifetime::Lifetime::permanent();
5335        Process::new("perm", spec)
5336    }
5337
5338    fn ephemeral_only_process(ttl: &str) -> Process {
5339        let mut spec = empty_spec();
5340        // Routes through the ONE substrate composer
5341        // [`crate::lifetime::Lifetime::ephemeral`] — one of ELEVEN+
5342        // pre-lift exact-match sites past the ★★ PRIME-DIRECTIVE ≥ 2
5343        // threshold; see the composer's doc-comment for the full
5344        // migration rationale.
5345        spec.lifetime = crate::lifetime::Lifetime::ephemeral(EphemeralLifetime {
5346            ttl: ttl.into(),
5347            teardown_policy: crate::lifetime::TeardownPolicy::OnAttested,
5348            max_concurrent: 3,
5349            exports: vec![],
5350        });
5351        Process::new("eph", spec)
5352    }
5353
5354    fn ambiguous_lifetime_process() -> Process {
5355        let mut spec = empty_spec();
5356        spec.lifetime = crate::lifetime::Lifetime {
5357            permanent: Some(crate::lifetime::PermanentLifetime {}),
5358            ephemeral: Some(EphemeralLifetime::default()),
5359        };
5360        Process::new("both", spec)
5361    }
5362
5363    #[test]
5364    fn resolved_ephemeral_returns_none_when_lifetime_is_default_empty() {
5365        // Empty-default corner pin: neither slot populated. The
5366        // resolver collapses to `Permanent(&DEFAULT_PERMANENT)` and
5367        // the compound projection sees no ephemeral inner. Matches
5368        // the pre-lift `lifetime_clock::evaluate` early-return to
5369        // `AutoTerminate::Skip` byte-identically.
5370        let p = Process::new("empty-lifetime", empty_spec());
5371        assert!(p.resolved_ephemeral().is_none());
5372    }
5373
5374    #[test]
5375    fn resolved_ephemeral_returns_none_for_permanent_only_process() {
5376        // Permanent-only corner pin: the `permanent:` slot is
5377        // populated, `ephemeral:` is not. Matches the pre-lift
5378        // `lifetime_clock::evaluate` outcome — the teardown/TTL
5379        // branch is never reached on a Permanent Process, and the
5380        // export-render arm now agrees at this call site (was
5381        // previously reached through the raw `.ephemeral.as_ref()`
5382        // that also returned `None` on this same corner — no drift
5383        // here; the drift is at the ambiguous corner below).
5384        let p = permanent_only_process();
5385        assert!(p.resolved_ephemeral().is_none());
5386    }
5387
5388    #[test]
5389    fn resolved_ephemeral_returns_some_for_ephemeral_only_process() {
5390        // Ephemeral-only corner pin: the ONE arm that projects. The
5391        // returned borrow carries the operator-authored `ttl` /
5392        // `teardown_policy` / `max_concurrent` verbatim. A
5393        // regression that swapped the projection to the sibling
5394        // `permanent:` slot would surface here as a type mismatch on
5395        // the `EphemeralLifetime` fields rather than as silent
5396        // operator-facing no-op teardown at the reconciler.
5397        let p = ephemeral_only_process("42m");
5398        let e = p
5399            .resolved_ephemeral()
5400            .expect("ephemeral-only Process must project");
5401        assert_eq!(e.ttl, "42m");
5402        assert_eq!(
5403            e.teardown_policy,
5404            crate::lifetime::TeardownPolicy::OnAttested
5405        );
5406        assert_eq!(e.max_concurrent, 3);
5407    }
5408
5409    #[test]
5410    fn resolved_ephemeral_returns_none_for_ambiguous_lifetime() {
5411        // DRIFT-CLOSING CONTRACT: BOTH `permanent:` AND `ephemeral:`
5412        // slots populated is an operator-authored mis-configuration.
5413        // Pre-lift, `lifetime_clock::evaluate` (via
5414        // `resolved_ephemeral()` on `Lifetime`) collapsed this
5415        // corner to `None` and yielded `AutoTerminate::Skip`, while
5416        // `tatara-reconciler::render::render_export_jobs` walked
5417        // the naked `.spec.lifetime.ephemeral.as_ref()` chain and
5418        // returned `Some(&e)` — so the reconciler would emit export
5419        // Jobs on a Process whose teardown-triggered fire semantics
5420        // the lifetime clock refused to honor. Post-lift this
5421        // primitive collapses ambiguity to `None` at ONE site so
5422        // BOTH consumers agree. A regression that broadened the
5423        // projection back to the raw field (or that silently
5424        // "preferred ephemeral" in the ambiguous case) surfaces
5425        // here rather than as export-Job noise on a mis-configured
5426        // ephemeral.
5427        let p = ambiguous_lifetime_process();
5428        assert!(p.resolved_ephemeral().is_none());
5429        // The raw field IS populated at this corner — pins the
5430        // pre-lift `.spec.lifetime.ephemeral.as_ref()` shape that
5431        // returned `Some` here.
5432        assert!(p.spec.lifetime.ephemeral.is_some());
5433    }
5434
5435    #[test]
5436    fn resolved_ephemeral_matches_spec_lifetime_forwarder() {
5437        // Byte-identity pin: the `Process` projection delegates
5438        // through the underlying `Lifetime::resolved_ephemeral`
5439        // primitive at every corner (empty, permanent-only,
5440        // ephemeral-only, ambiguous). A regression that silently
5441        // reintroduced the raw `.ephemeral.as_ref()` shortcut, or
5442        // that decided the ambiguous case by "prefer ephemeral"
5443        // at the Process layer instead of delegating, surfaces
5444        // here.
5445        for p in [
5446            Process::new("empty", empty_spec()),
5447            permanent_only_process(),
5448            ephemeral_only_process("1h"),
5449            ambiguous_lifetime_process(),
5450        ] {
5451            let via_process = p.resolved_ephemeral();
5452            let via_lifetime = p.spec.lifetime.resolved_ephemeral();
5453            // Both borrows point into the SAME `EphemeralLifetime`
5454            // slot when present — a regression that materialized a
5455            // per-call clone at the Process layer would fail the
5456            // pointer-equality gate.
5457            match (via_process, via_lifetime) {
5458                (Some(a), Some(b)) => assert!(
5459                    std::ptr::eq(a, b),
5460                    "Process::resolved_ephemeral must borrow the same slot as Lifetime::resolved_ephemeral"
5461                ),
5462                (None, None) => {}
5463                (a, b) => panic!(
5464                    "resolved_ephemeral shape drift: process={:?}, lifetime={:?}",
5465                    a.is_some(),
5466                    b.is_some()
5467                ),
5468            }
5469        }
5470    }
5471
5472    #[test]
5473    fn resolved_ephemeral_is_a_pure_projection() {
5474        // Purity pin: two consecutive calls return borrows into the
5475        // same underlying slot (no lazy materialization, no interior
5476        // mutation of `self`). Peer to the sibling
5477        // `is_being_deleted_is_a_pure_projection` +
5478        // `observed_attestation_is_a_pure_projection` pins; all
5479        // three bind the pure-projection discipline on the ONE
5480        // substrate accessor per spec / metadata / status slot.
5481        let p = ephemeral_only_process("5m");
5482        let a = p.resolved_ephemeral();
5483        let b = p.resolved_ephemeral();
5484        match (a, b) {
5485            (Some(x), Some(y)) => assert!(std::ptr::eq(x, y)),
5486            other => panic!("expected two Some borrows into the same slot, got {other:?}"),
5487        }
5488    }
5489
5490    // ── ProcessSpec::gate_compute_defaults substrate pins ───────────────
5491    //
5492    // The 12-line `ProcessSpec { identity: <Default>, classification:
5493    // Classification::gate_compute(), intent: <Default>, boundary:
5494    // Default::default(), compliance: Default::default(), depends_on:
5495    // vec![], signals: Default::default(), lifetime: Default::default(),
5496    // routing: None, encapsulates: None, suspended: false }` struct-
5497    // literal was open-coded verbatim at eight hand-authored callsites
5498    // before this primitive closed it. These pins bind the composed
5499    // shape at fail-before-pass-after granularity so a regression that
5500    // drifted the classification baseline, promoted a defaulted slot to
5501    // a non-default, or leaked a non-baseline slot into the substrate
5502    // composer surfaces HERE rather than as silent operator-visible
5503    // drift across every test fixture that keys assertions on the
5504    // shape.
5505    fn hand_authored_pre_lift() -> ProcessSpec {
5506        ProcessSpec {
5507            identity: IdentitySpec::default(),
5508            classification: Classification::gate_compute(),
5509            intent: Intent::default(),
5510            boundary: Default::default(),
5511            compliance: Default::default(),
5512            depends_on: vec![],
5513            signals: Default::default(),
5514            lifetime: Default::default(),
5515            routing: None,
5516            encapsulates: None,
5517            suspended: false,
5518        }
5519    }
5520
5521    #[test]
5522    fn gate_compute_defaults_composes_the_classification_baseline() {
5523        // Primary shape: the classification axis rides the sibling
5524        // `Classification::gate_compute` primitive verbatim. A
5525        // regression that flipped the classification baseline (a new
5526        // `#[default]` on the sibling closed-set, a re-import through a
5527        // different composer) surfaces HERE rather than at every
5528        // downstream fixture whose assertions key on
5529        // `spec.classification`.
5530        let s = ProcessSpec::gate_compute_defaults();
5531        assert_eq!(s.classification, Classification::gate_compute());
5532    }
5533
5534    #[test]
5535    fn gate_compute_defaults_defaulted_slots_ride_sibling_defaults() {
5536        // Pins the sibling-default correspondence the doc comment
5537        // names — a regression that promoted any defaulted slot to a
5538        // non-default (a new `#[default]` on `Intent`, a `Lifetime`
5539        // baseline shift, a per-field overlay stamping through the
5540        // primitive) would move the baseline HERE rather than at every
5541        // downstream consumer.
5542        let s = ProcessSpec::gate_compute_defaults();
5543        assert_eq!(
5544            serde_json::to_value(&s.identity).unwrap(),
5545            serde_json::to_value(IdentitySpec::default()).unwrap()
5546        );
5547        assert_eq!(
5548            serde_json::to_value(&s.intent).unwrap(),
5549            serde_json::to_value(Intent::default()).unwrap()
5550        );
5551        assert_eq!(
5552            serde_json::to_value(&s.boundary).unwrap(),
5553            serde_json::to_value(Boundary::default()).unwrap()
5554        );
5555        assert_eq!(
5556            serde_json::to_value(&s.compliance).unwrap(),
5557            serde_json::to_value(ComplianceSpec::default()).unwrap()
5558        );
5559        assert!(s.depends_on.is_empty());
5560        assert_eq!(
5561            serde_json::to_value(&s.signals).unwrap(),
5562            serde_json::to_value(SignalPolicy::default()).unwrap()
5563        );
5564        assert_eq!(
5565            serde_json::to_value(&s.lifetime).unwrap(),
5566            serde_json::to_value(Lifetime::default()).unwrap()
5567        );
5568        assert!(s.routing.is_none());
5569        assert!(s.encapsulates.is_none());
5570        assert!(!s.suspended);
5571    }
5572
5573    #[test]
5574    fn gate_compute_defaults_matches_hand_authored_pre_lift_bytewise() {
5575        // Byte-identical parity pin between the substrate primitive
5576        // and the pre-lift 12-line struct-literal that recurred at
5577        // eight hand-authored sites. Compares via `serde_json` value
5578        // equality — `ProcessSpec` does not derive `PartialEq` (the
5579        // typed fields it composes over do not uniformly derive it),
5580        // so a serialize round-trip is the shape-equality currency the
5581        // pin family already uses for `ProcessSpec`-shaped assertions
5582        // elsewhere in this test module. A regression that reshaped
5583        // the primitive would diverge from the pre-lift block HERE
5584        // rather than at every downstream fixture that keys on the
5585        // shape.
5586        let composed = ProcessSpec::gate_compute_defaults();
5587        let hand_authored = hand_authored_pre_lift();
5588        assert_eq!(
5589            serde_json::to_value(&composed).unwrap(),
5590            serde_json::to_value(&hand_authored).unwrap(),
5591        );
5592    }
5593
5594    #[test]
5595    fn gate_compute_defaults_supports_struct_update_override() {
5596        // The five override sites (three `render.rs` fixtures + two
5597        // `lifetime_clock.rs` fixtures) rely on struct-update syntax
5598        // to override a single slot while the primitive supplies the
5599        // other eleven. Pin the composition here so a regression that
5600        // broke the struct-update path (e.g. a `#[non_exhaustive]`
5601        // attribute added to `ProcessSpec` that would refuse struct-
5602        // update syntax across crate boundaries) surfaces at compile
5603        // time HERE rather than as a five-site downstream break.
5604        let base = ProcessSpec::gate_compute_defaults();
5605        let overridden = ProcessSpec {
5606            suspended: true,
5607            ..ProcessSpec::gate_compute_defaults()
5608        };
5609        assert!(!base.suspended);
5610        assert!(overridden.suspended);
5611        // Every other slot rides the same default as the base.
5612        assert_eq!(
5613            serde_json::to_value(&overridden.classification).unwrap(),
5614            serde_json::to_value(&base.classification).unwrap(),
5615        );
5616        assert_eq!(
5617            serde_json::to_value(&overridden.lifetime).unwrap(),
5618            serde_json::to_value(&base.lifetime).unwrap(),
5619        );
5620    }
5621
5622    #[test]
5623    fn gate_compute_defaults_is_call_time_construction_not_a_shared_singleton() {
5624        // Two independent calls produce structurally-equal but
5625        // distinct values — pins that the primitive is a plain
5626        // constructor rather than a `lazy_static` clone whose in-
5627        // place mutation at one consumer would silently mutate the
5628        // shape at every other consumer. Mirrors the sibling
5629        // `gate_compute_is_call_time_construction_not_a_shared_singleton`
5630        // pin on `Classification::gate_compute`.
5631        let a = ProcessSpec::gate_compute_defaults();
5632        let b = ProcessSpec::gate_compute_defaults();
5633        assert_eq!(
5634            serde_json::to_value(&a).unwrap(),
5635            serde_json::to_value(&b).unwrap(),
5636        );
5637        assert!(!std::ptr::eq(&a, &b));
5638    }
5639}