Strip phase/PR process labels from docstrings and comments

Remove opaque development-history tags (PR-C, PR-B, Phase 6.0/6.1) and
rephrase the "Phase N scope/deferred" mentions to preserve their
covered-vs-deferred meaning without the process vocabulary. Rename the
langfuse resume test's internal "Phase 1/2/3" step labels to "Step" to
avoid confusion with build phases.

Leaves the harness skip registry untouched: its DIRECTIVE_PHASE dict
maps directives to phase integers as functional data, and its comments
document that live mapping. Fixture-number references are kept
throughout (concrete in-repo identifiers, not spec refs).

Author: chris-colinsky

src/openarmature/graph/compiled.py

@@ -1521,7 +1521,7 @@ async def innermost(s: Any) -> Mapping[str, Any]:
             innermost,
         )
 
-        # Spec observability §3 / Phase 6 LLM-span hook: capability
+        # Spec observability §3 LLM-span hook: capability
         # backends emitting from inside a node body (the
         # llm-provider span instrumentation in OpenAIProvider) need
         # to find the observers active for THIS invocation, which
@@ -2503,7 +2503,7 @@ async def _maybe_save_checkpoint(
         # before/after distinction for a save like there is for a
         # node attempt. The field is repurposed because a save
         # event represents "the state was persisted" rather than
-        # "the state transitioned." Phase 6 OTel mapping reads
+        # "the state transitioned." The OTel mapping reads
         # ``pre_state`` as the save's state.
         _dispatch(
             context,

src/openarmature/graph/middleware/_core.py

@@ -109,7 +109,7 @@ def compose_chain(
     ``CompiledGraph._step_function_node``, producing one closure layer
     per middleware on every node step. For typical workloads
     (single-digit middleware × hundreds of node activations) this is
-    negligible. Under heavy fan-out (Phase 3+), e.g. 10K instances × 5
+    negligible. Under heavy fan-out, e.g. 10K instances × 5
     inner nodes × 3 middlewares = 150K closure constructions per
     invocation; worth measuring with realistic workloads when the
     fan-out runtime lands. The optimization shape (cache the chain at

src/openarmature/observability/correlation.py

@@ -163,7 +163,7 @@ def validate_invocation_id(value: object) -> str:
 
 # ---------------------------------------------------------------------------
 # Active observer set — for capability backends emitting from outside the
-# engine's per-step path (llm-provider span hook in Phase 6, future
+# engine's per-step path (llm-provider span hook, future
 # Langfuse/Datadog backends, user-written instrumented capabilities).
 # ---------------------------------------------------------------------------
 
@@ -206,7 +206,7 @@ def _reset_active_observers(token: Token[tuple[SubscribedObserver, ...]]) -> Non
 # Active dispatch hook — queue-mediated event emission from outside the
 # engine's per-step path. The engine sets this ContextVar to a closure
 # over the current invocation's delivery queue + observer chain;
-# capability backends (the LLM provider span hook in Phase 6, future
+# capability backends (the LLM provider span hook, future
 # Langfuse/Datadog instrumentations) call ``current_dispatch()(event)``
 # to enqueue an event for the same delivery worker the engine uses.
 #

src/openarmature/observability/otel/observer.py

@@ -2283,7 +2283,7 @@ def close_invocation(self, invocation_id: str) -> None:
         invocation in one call without needing to track ids
         externally. A first-class engine-level signal that lets
         observers auto-drain per-invocation state on completion is
-        tracked as Phase 6.1+ follow-up work in
+        tracked as follow-up work in
         ``openarmature-coord/docs/phase-6-1-conformance-fillin.md``.
         """
         inv_state = self._inv_states.pop(invocation_id, None)

tests/conformance/adapter.py

@@ -991,7 +991,7 @@ def _add_fan_out_node(
     - ``state_field_read`` — read an int from a parent state field.
     - ``queue_chunk`` — ``max(1, len(state.<field>) // chunk_size)``.
 
-    These are the only callable shapes the in-scope Phase 3 fixtures
+    These are the only callable shapes the in-scope fan-out fixtures
     use. Adding more is straightforward.
     """
     sub_name = cfg["subgraph"]

tests/conformance/harness/__init__.py

@@ -1,8 +1,8 @@
 """Conformance fixture harness — typed parsing for the four spec capabilities.
 
-Phase 0 (per the implementation plan): every fixture under
+Every fixture under
 ``openarmature-spec/spec/<capability>/conformance/`` lands as a typed pydantic
-config. Phases 1–6 add runtime interpretation under ``harness/runtime/``;
+config. Later stages add runtime interpretation under ``harness/runtime/``;
 they never re-touch parsing.
 
 Public surface:

tests/conformance/harness/directives.py

@@ -1,6 +1,6 @@
 """Typed directive sub-models — the shapes referenced inside fixtures.
 
-Phase 0 typing strategy: model every key in every fixture, but use
+Typing strategy: model every key in every fixture, but use
 ``dict[str, Any]`` for genuinely polymorphic payloads (notably the inner
 ``state``/``nodes``/``edges`` of recursive subgraph definitions, and the
 update payloads themselves which are arbitrary state-shaped dicts). The
@@ -38,7 +38,7 @@ class _AllowExtras(BaseModel):
     middleware-specific params, flaky/fan-out config). Validates KNOWN
     keys' types but doesn't reject unknown ones — the spec evolves these
     payloads frequently and modelling every parameter exhaustively
-    creates churn without proportional value. The Phase 0 strictness
+    creates churn without proportional value. The strictness
     contract sits at the directive STRUCTURE level (above), not the
     parameter-bag level (here)."""
 
@@ -90,7 +90,7 @@ class EdgeSpec(_AllowExtras):
     The spec defines static (``from``/``to``) and conditional
     (``from``/``condition``) edges; observability/011 also uses a
     ``when``-shaped predicate. Schema is permissive here so all forms
-    parse — Phase 1 (engine retrofit) interprets each shape against
+    parse — the engine retrofit interprets each shape against
     the engine's edge model.
     """
 
@@ -208,7 +208,7 @@ class UpdateFromFieldSpec(_ForbidExtras):
     """
 
     # Free-form: some fixtures use ``{result: x, multiplier: 2}``, others
-    # ``{score: item}`` with no multiplier. Phase 4 (fan-out runtime) reads
+    # ``{score: item}`` with no multiplier. The fan-out runtime reads
     # whichever keys are present.
     model_config = ConfigDict(extra="allow")
 

tests/conformance/harness/expectations.py

@@ -7,12 +7,11 @@
 fixture authors mixing keys across capabilities, and gives runtime code
 in :mod:`runtime` typed access to the assertion payload it needs.
 
-Phase 0 typing depth: TOP-LEVEL keys per capability are exhaustively
+Typing depth: TOP-LEVEL keys per capability are exhaustively
 typed (catches new directives the spec adds). The nested payload values
 underneath (e.g., individual span tree entries, observer event details)
 are kept loose as ``list[Any]`` / ``dict[str, Any]`` because the runtime
-phases that consume them are the right place to tighten — Phase 1
-will type observer-event entries, Phase 5 will type span_tree, etc.
+code that consumes them is the right place to tighten.
 """
 
 from __future__ import annotations
@@ -44,12 +43,12 @@ class GraphEngineExpected(_ForbidExtras):
     # Two shapes seen in fixtures:
     # - dict[observer_name, list[event_dict]] — most fixtures
     # - list[event_dict] flat — pipeline-utilities/011 (single-observer)
-    # Permissive ``Any`` until Phase 1 (engine retrofit) tightens.
+    # Permissive ``Any`` until the engine retrofit tightens.
     observer_events: Any = None
     delivery_order: list[dict[str, Any]] | None = None
     observer_event_invariants: dict[str, Any] | None = None
     # 020 — proposal-0012 fixture: assertions about edge-resolution
-    # failure event shapes. Permissive dict until Phase 1.
+    # failure event shapes. Permissive dict until the engine retrofit.
     # 022–024 (proposal 0010 §6 Drain) — drain-summary invariants
     # (drain_returned_within_timeout, graph_state_intact_after_timeout,
     # drain_waited_for_all_events) ride on the same field.
@@ -90,7 +89,7 @@ class LlmProviderRaisesAssertion(BaseModel):
 
     Permissive — fixtures attach assertion-specific knobs like
     ``retry_after_seconds`` (rate-limit fixture) without restructuring
-    the type. The runtime in Phase 2 validates the keys it reads.
+    the type. The runtime validates the keys it reads.
     """
 
     model_config = ConfigDict(extra="allow")
@@ -130,7 +129,7 @@ class PipelineUtilitiesExpected(_ForbidExtras):
     # Two shapes seen in fixtures:
     # - dict[observer_name, list[event_dict]] — most fixtures
     # - list[event_dict] flat — pipeline-utilities/011 (single-observer)
-    # Permissive ``Any`` until Phase 1 (engine retrofit) tightens.
+    # Permissive ``Any`` until the engine retrofit tightens.
     observer_events: Any = None
     observer_event_invariants: dict[str, Any] | None = None
     # Singular form used by 015 — assert one specific event shape.

tests/conformance/harness/fixtures.py

@@ -1,6 +1,6 @@
 """Typed fixture root models.
 
-Per the Phase 0 plan: every YAML fixture under
+Every YAML fixture under
 ``openarmature-spec/spec/<capability>/conformance/`` lands as one of three
 typed shapes. The shape is chosen by a callable discriminator that inspects
 the raw dict's top-level keys (no tag field is present in the YAML).

tests/conformance/test_checkpoint.py

@@ -24,7 +24,7 @@
 - 030 checkpoint-not-found — supported.
 - 031 correlation-id-preserved-across-resume — record-level
   assertions supported here; the OTel span/log assertions are
-  gated until Phase 6 lands the observability mapping.
+  gated until the observability mapping lands.
 - 048-054 per-instance fan-out resume contract — supported.
 """
 
@@ -818,7 +818,7 @@ async def _run_one_case(spec: Mapping[str, Any], *, top_level: Mapping[str, Any]
         _assert_resume_invariants(invariants_block, final_resume, flaky_per_index_recorders)
 
     # Fixture 031: assert correlation_id preserved + invocation_id
-    # changed. Span/log assertions deferred to Phase 6 — observability
+    # changed. Span/log assertions deferred — observability
     # isn't wired yet. Skip those cleanly here.
     if "correlation_id_assertions" in resume_expected:
         cid_block = cast("Mapping[str, Any]", resume_expected["correlation_id_assertions"])