atdd 4.2.2__tar.gz → 4.2.4__tar.gz

{atdd-4.2.2/src/atdd.egg-info → atdd-4.2.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: atdd
-Version: 4.2.2
+Version: 4.2.4
 Summary: ATDD Platform - Acceptance Test Driven Development toolkit
 License: MIT
 Project-URL: Homepage, https://github.com/afokapu/atdd

{atdd-4.2.2 → atdd-4.2.4}/pyproject.toml

@@ -79,6 +79,10 @@ exclude = ["atdd.*.validators.fixtures*"]
 # protocol-flow edges in consumer repos (graph/ is not a package; the glob
 # reaches into it from the atdd.coach package root).
 "atdd.coach" = ["graph/*.yaml"]
+# #1421: the URN grammar convention (urn_grammar.yaml) is read at import by the
+# URNGrammar engine, so it must ship in the wheel or a non-editable install
+# fails on first import.
+"atdd.coach.utils.graph" = ["*.yaml"]
 "atdd.planner.conventions" = ["*.yaml", "nodes/*.yaml"]
 "atdd.planner.schemas" = ["*.json", "author/*.json"]
 "atdd.tester.conventions" = ["*.yaml"]

{atdd-4.2.2 → atdd-4.2.4}/src/atdd/coach/commands/registry.py

@@ -69,12 +69,12 @@ from atdd.coach.utils.theme_map import get_theme_map
 
 _logger = logging.getLogger(__name__)
 
-# Import URNBuilder for URN generation (following conventions)
+# Import URNGrammar for URN generation (following conventions)
 try:
-    from atdd.coach.utils.graph.urn import URNBuilder
+    from atdd.coach.utils.graph.urn import URNGrammar
 except ImportError:
-    # Fallback if URNBuilder not available
-    class URNBuilder:
+    # Fallback if URNGrammar not available
+    class URNGrammar:
         @staticmethod
         def test(wagon: str, file: str, func: str) -> str:
             return f"test:{wagon}:{file}::{func}"

{atdd-4.2.2 → atdd-4.2.4}/src/atdd/coach/graph/relationships.yaml

@@ -1,78 +1,4 @@
 edges:
-# Foundational verb-object naming enforcement (#1276)
-- confidence: 1.0
-  constraint: mandatory
-  control: internal
-  foundation: finish_to_start
-  reason: the verb-object name rule enforces the verb-object preference of the wagon URN grammar
-  source_ref: planner.wagon.name-is-verb-object
-  strength: important
-  target_ref: planner.wagon.urn-naming
-  type: refines
-- confidence: 1.0
-  constraint: mandatory
-  control: internal
-  foundation: start_to_start
-  reason: the wagon name rule draws its leading-verb lexicon from the verb-selection table
-  source_ref: planner.wagon.name-is-verb-object
-  strength: important
-  target_ref: planner.feature.verb-selection-by-artifact-type
-  type: requires
-- confidence: 1.0
-  constraint: mandatory
-  control: internal
-  foundation: finish_to_start
-  reason: the verb-object name rule enforces the verb-object preference of the feature URN grammar
-  source_ref: planner.feature.name-is-verb-object
-  strength: important
-  target_ref: planner.feature.urn-naming
-  type: refines
-- confidence: 1.0
-  constraint: mandatory
-  control: internal
-  foundation: start_to_start
-  reason: the feature name rule draws its leading-verb lexicon from the verb-selection table
-  source_ref: planner.feature.name-is-verb-object
-  strength: important
-  target_ref: planner.feature.verb-selection-by-artifact-type
-  type: requires
-# Foundational artifact/contract naming enforcement (#1329)
-- confidence: 1.0
-  constraint: mandatory
-  control: internal
-  foundation: finish_to_start
-  reason: the theme-first identity rule enforces the artifact naming-pattern grammar at confirm
-  source_ref: planner.artifact-naming.theme-first-identity
-  strength: important
-  target_ref: planner.artifact-naming.naming-pattern
-  type: refines
-- confidence: 1.0
-  constraint: mandatory
-  control: internal
-  foundation: start_to_start
-  reason: the theme-first identity rule resolves its allowed themes from the same canonical theme map
-  source_ref: planner.artifact-naming.theme-first-identity
-  strength: important
-  target_ref: planner.theme.must-be-canonical
-  type: requires
-- confidence: 1.0
-  constraint: mandatory
-  control: internal
-  foundation: finish_to_start
-  reason: the path-mirrors-identity rule enforces the contract-file-mapping algorithm at confirm
-  source_ref: planner.artifact-naming.path-mirrors-identity
-  strength: important
-  target_ref: planner.artifact-naming.contract-file-mapping
-  type: refines
-- confidence: 1.0
-  constraint: mandatory
-  control: internal
-  foundation: finish_to_start
-  reason: a contract path can only mirror an identity that is a valid theme-first identity
-  source_ref: planner.artifact-naming.path-mirrors-identity
-  strength: important
-  target_ref: planner.artifact-naming.theme-first-identity
-  type: requires
 - confidence: 1.0
   constraint: discretionary
   control: internal
@@ -575,6 +501,15 @@ edges:
   strength: important
   target_ref: coder.state-store.no-raw-sql-at-call-sites
   type: runs_alongside
+- confidence: 1.0
+  constraint: mandatory
+  control: internal
+  foundation: start_to_start
+  reason: "both define what the operational State Store layer may hold \u2014 the SoT boundary runs alongside the foundational layering rule"
+  source_ref: coder.state-store.operational-vs-definition-sot
+  strength: important
+  target_ref: coder.state-store.core-imports-no-providers
+  type: runs_alongside
 - confidence: 1.0
   constraint: mandatory
   control: internal
@@ -899,6 +834,24 @@ edges:
   strength: important
   target_ref: planner.artifact-naming.theme-taxonomy
   type: requires
+- confidence: 1.0
+  constraint: mandatory
+  control: internal
+  foundation: finish_to_start
+  reason: the path-mirrors-identity rule enforces the contract-file-mapping algorithm at confirm
+  source_ref: planner.artifact-naming.path-mirrors-identity
+  strength: important
+  target_ref: planner.artifact-naming.contract-file-mapping
+  type: refines
+- confidence: 1.0
+  constraint: mandatory
+  control: internal
+  foundation: finish_to_start
+  reason: a contract path can only mirror an identity that is a valid theme-first identity
+  source_ref: planner.artifact-naming.path-mirrors-identity
+  strength: important
+  target_ref: planner.artifact-naming.theme-first-identity
+  type: requires
 - confidence: 1.0
   constraint: mandatory
   control: internal
@@ -926,6 +879,24 @@ edges:
   strength: important
   target_ref: planner.artifact-naming.urn-structure
   type: runs_alongside
+- confidence: 1.0
+  constraint: mandatory
+  control: internal
+  foundation: finish_to_start
+  reason: the theme-first identity rule enforces the artifact naming-pattern grammar at confirm
+  source_ref: planner.artifact-naming.theme-first-identity
+  strength: important
+  target_ref: planner.artifact-naming.naming-pattern
+  type: refines
+- confidence: 1.0
+  constraint: mandatory
+  control: internal
+  foundation: start_to_start
+  reason: the theme-first identity rule resolves its allowed themes from the same canonical theme map
+  source_ref: planner.artifact-naming.theme-first-identity
+  strength: important
+  target_ref: planner.theme.must-be-canonical
+  type: requires
 - confidence: 1.0
   constraint: mandatory
   control: internal
@@ -944,6 +915,15 @@ edges:
   strength: important
   target_ref: planner.artifact-naming.naming-pattern
   type: requires
+- confidence: 1.0
+  constraint: mandatory
+  control: internal
+  foundation: finish_to_start
+  reason: contract coherence is checked over the produce/consume artifact shape it constrains (#1332)
+  source_ref: planner.contract.registry-coherence
+  strength: important
+  target_ref: planner.wagon.produce-consume-artifacts
+  type: requires
 - confidence: 1.0
   constraint: discretionary
   control: internal
@@ -1187,6 +1167,24 @@ edges:
   strength: minor
   target_ref: planner.feature.size-max-rule
   type: runs_alongside
+- confidence: 1.0
+  constraint: mandatory
+  control: internal
+  foundation: finish_to_start
+  reason: the verb-object name rule enforces the verb-object preference of the feature URN grammar
+  source_ref: planner.feature.name-is-verb-object
+  strength: important
+  target_ref: planner.feature.urn-naming
+  type: refines
+- confidence: 1.0
+  constraint: mandatory
+  control: internal
+  foundation: start_to_start
+  reason: the feature name rule draws its leading-verb lexicon from the verb-selection table
+  source_ref: planner.feature.name-is-verb-object
+  strength: important
+  target_ref: planner.feature.verb-selection-by-artifact-type
+  type: requires
 - confidence: 1.0
   constraint: mandatory
   control: internal
@@ -1404,15 +1402,6 @@ edges:
   strength: minor
   target_ref: planner.train.registry
   type: runs_alongside
-- confidence: 1.0
-  constraint: mandatory
-  control: internal
-  foundation: finish_to_start
-  reason: the category digit is part of the numbering scheme
-  source_ref: planner.train.categories
-  strength: important
-  target_ref: planner.train.numbering
-  type: requires
 - confidence: 1.0
   constraint: discretionary
   control: external
@@ -1561,22 +1550,19 @@ edges:
   constraint: mandatory
   control: internal
   foundation: finish_to_start
-  reason: registry membership of a payload contract presupposes a typed payload
+  reason: "binding an interlocking payload.contract to an authored/registered contract (#1314 item E) ties the route/interlocking model to the contract layer \u2014 the declared identity must be a registered contract, resolved via the contract identity->file mapping the contract layer maintains"
   source_ref: planner.train.interlocking-payload-contract-registered
   strength: important
-  target_ref: planner.train.interlocking-message-payload-typed
+  target_ref: planner.artifact-naming.contract-file-mapping
   type: requires
 - confidence: 1.0
   constraint: mandatory
   control: internal
   foundation: finish_to_start
-  reason: binding an interlocking payload.contract to an authored/registered contract
-    (#1314 item E) ties the route/interlocking model to the contract layer — the
-    declared identity must be a registered contract, resolved via the contract
-    identity->file mapping the contract layer maintains
+  reason: registry membership of a payload contract presupposes a typed payload
   source_ref: planner.train.interlocking-payload-contract-registered
   strength: important
-  target_ref: planner.artifact-naming.contract-file-mapping
+  target_ref: planner.train.interlocking-message-payload-typed
   type: requires
 - confidence: 1.0
   constraint: mandatory
@@ -1632,15 +1618,6 @@ edges:
   strength: critical
   target_ref: planner.train.sequence
   type: runs_alongside
-- confidence: 1.0
-  constraint: mandatory
-  control: internal
-  foundation: finish_to_start
-  reason: the train_id name follows the numbering prefix
-  source_ref: planner.train.naming
-  strength: important
-  target_ref: planner.train.numbering
-  type: requires
 - confidence: 0.9
   constraint: discretionary
   control: internal
@@ -1650,6 +1627,15 @@ edges:
   strength: minor
   target_ref: planner.train.artifact-flow
   type: runs_alongside
+- confidence: 1.0
+  constraint: mandatory
+  control: internal
+  foundation: finish_to_start
+  reason: the subject-bucketed train registry can only key a typed train:<subject>:<slug> entry on a subject already registered in plan/_subjects.yaml
+  source_ref: planner.train.registry
+  strength: important
+  target_ref: planner.subject.invariants
+  type: requires
 - confidence: 1.0
   constraint: mandatory
   control: internal
@@ -1726,10 +1712,10 @@ edges:
   constraint: mandatory
   control: internal
   foundation: finish_to_start
-  reason: the orchestrator URN extends the train URN with a theme prefix
+  reason: the train URN carried on a theme orchestrator must reference a train registered in the train registry (registry entry + spec file)
   source_ref: planner.train.theme-orchestrator-urn
-  strength: minor
-  target_ref: planner.train.urn-naming
+  strength: important
+  target_ref: planner.train.registry
   type: requires
 - confidence: 1.0
   constraint: mandatory
@@ -1740,33 +1726,6 @@ edges:
   strength: minor
   target_ref: planner.train.categories
   type: requires
-- confidence: 1.0
-  constraint: mandatory
-  control: internal
-  foundation: finish_to_start
-  reason: the theme digit is part of the numbering scheme
-  source_ref: planner.train.theme-ranges
-  strength: important
-  target_ref: planner.train.numbering
-  type: requires
-- confidence: 1.0
-  constraint: mandatory
-  control: internal
-  foundation: finish_to_start
-  reason: the train URN embeds the numbered train_id
-  source_ref: planner.train.urn-naming
-  strength: important
-  target_ref: planner.train.numbering
-  type: requires
-- confidence: 1.0
-  constraint: mandatory
-  control: internal
-  foundation: finish_to_start
-  reason: validation enforces the train_id numbering
-  source_ref: planner.train.validation-rules
-  strength: important
-  target_ref: planner.train.numbering
-  type: requires
 - confidence: 1.0
   constraint: mandatory
   control: internal
@@ -1834,19 +1793,28 @@ edges:
   constraint: mandatory
   control: internal
   foundation: finish_to_start
-  reason: the strict no-consume-cycle invariant is the exact (acyclicity) case of the complexity model
-  source_ref: planner.wagon.no-consume-cycle
+  reason: the verb-object name rule enforces the verb-object preference of the wagon URN grammar
+  source_ref: planner.wagon.name-is-verb-object
   strength: important
-  target_ref: planner.wagon.complexity-model
+  target_ref: planner.wagon.urn-naming
+  type: refines
+- confidence: 1.0
+  constraint: mandatory
+  control: internal
+  foundation: start_to_start
+  reason: the wagon name rule draws its leading-verb lexicon from the verb-selection table
+  source_ref: planner.wagon.name-is-verb-object
+  strength: important
+  target_ref: planner.feature.verb-selection-by-artifact-type
   type: requires
 - confidence: 1.0
   constraint: mandatory
   control: internal
   foundation: finish_to_start
-  reason: contract coherence is checked over the produce/consume artifact shape it constrains (#1332)
-  source_ref: planner.contract.registry-coherence
+  reason: the strict no-consume-cycle invariant is the exact (acyclicity) case of the complexity model
+  source_ref: planner.wagon.no-consume-cycle
   strength: important
-  target_ref: planner.wagon.produce-consume-artifacts
+  target_ref: planner.wagon.complexity-model
   type: requires
 - confidence: 1.0
   constraint: mandatory
@@ -2343,16 +2311,6 @@ edges:
   strength: minor
   target_ref: tester.smoke.operator-observable-assertion
   type: runs_alongside
-- confidence: 1.0
-  constraint: mandatory
-  control: internal
-  foundation: start_to_start
-  reason: both define what the operational State Store layer may hold — the SoT boundary runs
-    alongside the foundational layering rule
-  source_ref: coder.state-store.operational-vs-definition-sot
-  strength: important
-  target_ref: coder.state-store.core-imports-no-providers
-  type: runs_alongside
 graph_id: atdd.convention.relationships
 kind: relationship_graph
 schema_version: 1.0.0

{atdd-4.2.2 → atdd-4.2.4}/src/atdd/coach/utils/graph/edge_validator.py

@@ -28,7 +28,7 @@ from atdd.coach.utils.graph.graph_builder import (
     EdgeType,
     URNNode,
 )
-from atdd.coach.utils.graph.urn import URNBuilder
+from atdd.coach.utils.graph.urn import URNGrammar
 
 
 class IssueSeverity(Enum):
@@ -172,13 +172,13 @@ class EdgeValidator:
     def __init__(self, graph: TraceabilityGraph):
         self.graph = graph
 
-        # Root families: derived from URNBuilder.SEGMENT_COUNTS (parent-it-belongs-to,
+        # Root families: derived from URNGrammar.SEGMENT_COUNTS (parent-it-belongs-to,
         # spec v12 §3.2). A family is a root when its segment count after the prefix
         # is 1 (no parent coordinates). Adding a new top-level family in PATTERNS +
         # SEGMENT_COUNTS automatically extends this set — no edits here required.
         self._root_families = {
             family
-            for family, count in URNBuilder.SEGMENT_COUNTS.items()
+            for family, count in URNGrammar.SEGMENT_COUNTS.items()
             if count == 1
         }
 

{atdd-4.2.2 → atdd-4.2.4}/src/atdd/coach/utils/graph/graph_builder.py

@@ -31,7 +31,7 @@ from atdd.coach.utils.graph.resolver import (
     URNDeclaration,
     URNResolution,
 )
-from atdd.coach.utils.graph.urn import URNBuilder
+from atdd.coach.utils.graph.urn import URNGrammar
 
 
 class EdgeType(Enum):
@@ -470,14 +470,14 @@ class TraceabilityGraph:
 
         # ── gaps ───────────────────────────────────────────────
         # Nodes with zero incoming edges (excluding root families).
-        # Root families derived from URNBuilder.SEGMENT_COUNTS (parent-it-belongs-to,
+        # Root families derived from URNGrammar.SEGMENT_COUNTS (parent-it-belongs-to,
         # spec v12 §3.2): a family is a root when its segment count after the
         # prefix is 1 (no parent coordinates). Adding a new top-level family in
         # PATTERNS + SEGMENT_COUNTS automatically extends this set.
         # Audit reference: docs/urn-prefix-audit-2026.md (finding #2).
         root_families = {
             family
-            for family, count in URNBuilder.SEGMENT_COUNTS.items()
+            for family, count in URNGrammar.SEGMENT_COUNTS.items()
             if count == 1
         }
         all_targets = set()
@@ -793,6 +793,7 @@ class GraphBuilder:
         self._build_containment_edges(graph)
         self._build_produce_consume_edges(graph)
         self._build_train_edges(graph)
+        self._build_subject_edges(graph)
         self._build_component_edges(graph)
         self._build_security_edges(graph)
         # Phase 3: pass content cache to edge builders that read files
@@ -1150,6 +1151,35 @@ class GraphBuilder:
             except Exception:
                 continue
 
+    def _build_subject_edges(self, graph: TraceabilityGraph) -> None:
+        """Build subject -> train (CONTAINS) edges for typed trains (#1421).
+
+        A typed ``train:<subject>:<slug>`` is a 2-token URN parented by its
+        ``subject:<subject>`` root (grammar: ``train.parent == subject``). This
+        edge makes the subject a real parent node so the typed train is not a
+        topological orphan. The subject node is auto-synthesized by ``add_edge``
+        if the registry has not declared it yet.
+
+        Legacy ``train:NNNN-slug`` (a single token) has no subject parent and is
+        skipped — dual-resolution keeps it resolving during the migration
+        window (see ``TrainResolver``).
+        """
+        for urn, node in graph.nodes.items():
+            if node.family != "train":
+                continue
+            tokens = urn[len("train:"):].split(":")
+            if len(tokens) != 2 or not all(tokens):
+                continue
+            subject_urn = f"subject:{tokens[0]}"
+            graph.add_edge(
+                URNEdge(
+                    source_urn=subject_urn,
+                    target_urn=urn,
+                    edge_type=EdgeType.CONTAINS,
+                    metadata={"source": "urn-structure"},
+                )
+            )
+
     def _build_security_edges(self, graph: TraceabilityGraph) -> None:
         """
         Build security URN edges: