refactor: add a JSON-LD deserializer symmetric with the serializer

[ehennestad]

Serialization had an architecture; deserialization had a function. This adds the missing half.

What loadInstances was doing

A single 144-line function read the files, stripped a hardcoded vocabulary prefix, dispatched each node to a type, built the instances, and then wired the references between them with a hand-rolled recursive traversal. That traversal had no cycle detection, no extension point for another format, and no way to configure any of it.

BaseDeserializer

The counterpart to BaseSerializer. Subclasses implement one method:

rawStructs = parseToStructs(obj, data)

The base class owns what does not depend on the format: dispatching each node to its type, constructing the instances, and wiring the links. JsonLdDeserializer implements the JSON-LD case and accepts several documents in one call, which is what loading a folder of one-file-per-instance needs.

Link wiring moves onto the traversal core

LinkWiringVisitor replaces the recursive resolveLinks local function. Building it on BaseVisitor means it inherits cycle detection and positional replacement rather than reimplementing traversal for the third time in the codebase.

Reporting instead of dropping

A node that could not be turned into an instance was previously handled like this:

catch ME
    warning(ME.message)
end

That loses the error identifier, makes the warning unsuppressable by identifier, garbles the message if it contains a %, and produces one warning per node with no indication of which node was lost. A node with no @type was skipped even more quietly, under a continue % Todo: Why skip?.

Unreadable nodes are now collected and reported once, by identifier, under openMINDS:Deserializer:UnreadableNodes, so it is possible to tell how much of a document did not survive. A caller that cannot work with a partial result can construct the deserializer with UnreadableNodePolicy = "error".

Resolving the type is deliberately left outside that handling. A type in an unknown namespace means the whole document was written for a different model version. Treating that as a per-node problem would hand the caller an empty collection and a warning where a hard failure is more useful, and the fixture test for legacy namespace documents asserts exactly that.

Result

loadInstances is now file reading and deserializer selection, 48 lines instead of 144.

Tests

New DeserializerTest covers reading a collection document, wiring links between nodes, wiring links across separate documents, terminating on a circular document, reporting an untyped node while still returning the readable ones, and the error policy.

Not addressed here

jsonld2struct still strips a hardcoded openminds.ebrains.eu vocabulary prefix, which is wrong for documents written in expanded form under a v4 model. That belongs with the version-tolerant loading work later in this stack.

🤖 Generated with Claude Code

[github-actions]

Test Results (R2022a)

756 tests   755 ✅  2m 36s ⏱️
 20 suites    1 💤
  1 files      0 ❌

Results for commit 69fe48b.

♻️ This comment has been updated with latest results.

[codecov]

Codecov Report

❌ Patch coverage is 82.05128% with 14 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (fix-controlled-term-array-construction@818b5de). Learn more about missing BASE report.

Files with missing lines	Patch %	Lines
...openminds/+internal/+serializer/BaseDeserializer.m	80.00%	8 Missing ⚠️
...ternal/+openminds/+internal/+store/loadInstances.m	69.23%	4 Missing ⚠️
...penminds/+internal/+serializer/LinkWiringVisitor.m	88.23%	2 Missing ⚠️

Additional details and impacted files

@@                            Coverage Diff                            @@
##             fix-controlled-term-array-construction     #110   +/-   ##
=========================================================================
  Coverage                                          ?   79.31%           
=========================================================================
  Files                                             ?      422           
  Lines                                             ?     4095           
  Branches                                          ?        0           
=========================================================================
  Hits                                              ?     3248           
  Misses                                            ?      847           
  Partials                                          ?        0           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[github-actions]

Test Results (R2026a)

756 tests  +6   755 ✅ +6   2m 35s ⏱️ +5s
 20 suites +1     1 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit 69fe48b. ± Comparison against base commit 818b5de.

♻️ This comment has been updated with latest results.