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