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