refactor: use PrimitiveEntry named fields instead of positional destructuring

Access prim.path, prim.frontmatter, and prim.body by name instead of
unpacking to (entry, frontmatter, body). This makes field access
self-documenting — prim.path.name is clearly Path.name, whereas the
old entry.name required tracing through discover_primitives to understand.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: 2 people

src/ralphify/checks.py

@@ -57,23 +57,23 @@ def discover_checks(root: Path = Path(".")) -> list[Check]:
     are skipped with a warning.  Defaults: ``timeout=_DEFAULT_TIMEOUT``, ``enabled=True``.
     """
     checks = []
-    for entry, frontmatter, body in discover_primitives(root, "checks", CHECK_MARKER):
-        script = find_run_script(entry)
-        command = frontmatter.get("command")
+    for prim in discover_primitives(root, "checks", CHECK_MARKER):
+        script = find_run_script(prim.path)
+        command = prim.frontmatter.get("command")
 
         if not script and not command:
-            warnings.warn(f"Check '{entry.name}' has neither a run.* script nor a command — skipping")
+            warnings.warn(f"Check '{prim.path.name}' has neither a run.* script nor a command — skipping")
             continue
 
         checks.append(
             Check(
-                name=entry.name,
-                path=entry,
+                name=prim.path.name,
+                path=prim.path,
                 command=command,
                 script=script,
-                timeout=frontmatter.get("timeout", _DEFAULT_TIMEOUT),
-                enabled=frontmatter.get("enabled", True),
-                failure_instruction=body,
+                timeout=prim.frontmatter.get("timeout", _DEFAULT_TIMEOUT),
+                enabled=prim.frontmatter.get("enabled", True),
+                failure_instruction=prim.body,
             )
         )
 

src/ralphify/contexts.py

@@ -53,15 +53,15 @@ def discover_contexts(root: Path = Path(".")) -> list[Context]:
     """Discover contexts in root/.ralph/contexts/ directories."""
     return [
         Context(
-            name=entry.name,
-            path=entry,
-            command=frontmatter.get("command"),
-            script=find_run_script(entry),
-            timeout=frontmatter.get("timeout", _DEFAULT_TIMEOUT),
-            enabled=frontmatter.get("enabled", True),
-            static_content=body,
+            name=prim.path.name,
+            path=prim.path,
+            command=prim.frontmatter.get("command"),
+            script=find_run_script(prim.path),
+            timeout=prim.frontmatter.get("timeout", _DEFAULT_TIMEOUT),
+            enabled=prim.frontmatter.get("enabled", True),
+            static_content=prim.body,
         )
-        for entry, frontmatter, body in discover_primitives(root, "contexts", CONTEXT_MARKER)
+        for prim in discover_primitives(root, "contexts", CONTEXT_MARKER)
     ]
 
 

src/ralphify/instructions.py

@@ -35,12 +35,12 @@ def discover_instructions(root: Path = Path(".")) -> list[Instruction]:
     """
     return [
         Instruction(
-            name=entry.name,
-            path=entry,
-            enabled=frontmatter.get("enabled", True),
-            content=body,
+            name=prim.path.name,
+            path=prim.path,
+            enabled=prim.frontmatter.get("enabled", True),
+            content=prim.body,
         )
-        for entry, frontmatter, body in discover_primitives(root, "instructions", INSTRUCTION_MARKER)
+        for prim in discover_primitives(root, "instructions", INSTRUCTION_MARKER)
     ]
 
 

src/ralphify/prompts.py

@@ -32,13 +32,13 @@ def discover_prompts(root: Path = Path(".")) -> list[Prompt]:
     """
     return [
         Prompt(
-            name=entry.name,
-            path=entry,
-            description=frontmatter.get("description", ""),
-            enabled=frontmatter.get("enabled", True),
-            content=body,
+            name=prim.path.name,
+            path=prim.path,
+            description=prim.frontmatter.get("description", ""),
+            enabled=prim.frontmatter.get("enabled", True),
+            content=prim.body,
         )
-        for entry, frontmatter, body in discover_primitives(root, "prompts", PROMPT_MARKER)
+        for prim in discover_primitives(root, "prompts", PROMPT_MARKER)
     ]