fix(archives): restore get_result error logs + accept @graph shape in E2E test

Two test-driven fixes after running the full local test suite:

1. archives-plugin.unit.php expected the regex pattern
   `function fetchArchivalUnitsForAuthority(...) ... get_result() ...
   [Archives] fetchArchivalUnitsForAuthority get_result failed:` —
   i.e. an explicit error log on get_result failure. A prior fix batch
   had simplified the code to silently degrade. Restore the explicit
   `if (!(\$result instanceof \\mysqli_result)) { SecureLogger::error(...); }`
   branch on BOTH fetchArchivalUnitsForAuthority and
   fetchAuthoritiesForArchivalUnit so the test's regex matches and
   the public error contract (logged failure on get_result error)
   holds.

2. tests/archives-ric-jsonld.spec.js: 'unit ric.json @type' and 'agent
   ric.json @type' assertions failed when relations exist on the
   entity (the response shape is now `{@context, @graph: [...]}` after
   F006 wired fetchRelationsForEntity into the ric handlers). Updated
   both tests to extract the entity node from `@graph` when present,
   falling back to flat `body` when not. Both shapes are valid post-
   F006 — the test just needed to accept the wrapped form.

Result: unit tests 60/60, archives ric-jsonld E2E 41/41.

Author: fabiodalez-dev

storage/plugins/archives/ArchivesPlugin.php

@@ -2811,12 +2811,15 @@ private function fetchArchivalUnitsForAuthority(int $authorityId): array
             return $rows;
         }
         $result = $stmt->get_result();
-        if ($result instanceof \mysqli_result) {
-            while ($row = $result->fetch_assoc()) {
-                $rows[] = $row;
-            }
-            $result->free();
+        if (!($result instanceof \mysqli_result)) {
+            SecureLogger::error('[Archives] fetchArchivalUnitsForAuthority get_result failed: ' . $this->db->error);
+            $stmt->close();
+            return $rows;
+        }
+        while ($row = $result->fetch_assoc()) {
+            $rows[] = $row;
         }
+        $result->free();
         $stmt->close();
         return $rows;
     }
@@ -2852,12 +2855,15 @@ public function fetchAuthoritiesForArchivalUnit(int $archivalUnitId): array
             return $rows;
         }
         $result = $stmt->get_result();
-        if ($result instanceof \mysqli_result) {
-            while ($row = $result->fetch_assoc()) {
-                $rows[] = $row;
-            }
-            $result->free();
+        if (!($result instanceof \mysqli_result)) {
+            SecureLogger::error('[Archives] fetchAuthoritiesForArchivalUnit get_result failed: ' . $this->db->error);
+            $stmt->close();
+            return $rows;
+        }
+        while ($row = $result->fetch_assoc()) {
+            $rows[] = $row;
         }
+        $result->free();
         $stmt->close();
         return $rows;
     }

tests/archives-ric-jsonld.spec.js

@@ -130,17 +130,28 @@ test.describe.serial('RiC-O JSON-LD endpoints — v0.7.7', () => {
         test.skip(!hasArchives, 'No archival_units seeded');
         const res = await request.get(`${BASE}/archives/${unitId}/ric.json`);
         const body = await res.json();
-        expect(body['@type']).toMatch(/^ric:(Record|RecordSet)$/);
-        expect(body['@id']).toMatch(/\/archives\/\d+$/);
+        // Post-F006: @graph wrapper when relations exist. Accept both shapes.
+        const unitNode = Array.isArray(body['@graph'])
+            ? body['@graph'].find(n => /\/archives\/\d+$/.test(n['@id'] ?? ''))
+            : body;
+        expect(unitNode, 'unit node present (flat or in @graph)').toBeDefined();
+        expect(unitNode['@type']).toMatch(/^ric:(Record|RecordSet)$/);
+        expect(unitNode['@id']).toMatch(/\/archives\/\d+$/);
     });
 
     test('GET /archives/agents/{id}/ric.json responds 200 with an agent type', async ({ request }) => {
         test.skip(agentId === 0, 'No authority_records seeded');
         const res = await request.get(`${BASE}/archives/agents/${agentId}/ric.json`);
         expect(res.status()).toBe(200);
         const body = await res.json();
-        expect(body['@type']).toMatch(/^ric:(Person|CorporateBody|Family)$/);
-        expect(body['@id']).toMatch(/\/archives\/agents\/\d+$/);
+        // Post-F006: when polymorphic relations exist for the agent the document
+        // is wrapped in @graph, otherwise it stays flat. Both shapes are valid.
+        const agentNode = Array.isArray(body['@graph'])
+            ? body['@graph'].find(n => /\/archives\/agents\/\d+$/.test(n['@id'] ?? ''))
+            : body;
+        expect(agentNode, 'agent node must be present (flat or in @graph)').toBeDefined();
+        expect(agentNode['@type']).toMatch(/^ric:(Person|CorporateBody|Family)$/);
+        expect(agentNode['@id']).toMatch(/\/archives\/agents\/\d+$/);
     });
 
     // ── 2. Headers — Cache-Control, Link rel=canonical, CORS, no Vary ─