feat!: upgrade to Concerto v4.0.3 (breaking change)

[rockaxorb13]

fixes #136

This PR fixes the first and second problems listed in #136, as directed by @mttrbrts, these changes did not break the v3 build and as per my understanding, the strict : true parameter for modelManager was mostly redundant.

For the serializer API changes I updated Serializer.fromJSON() calls, adding an option:
({ validate: true, acceptResourcesForRelationships: false });
This satisfies v3's strict TypeScript requirements for the SerializerOptions interface while preparing the calls for v4's updated signature where these options are made optional.

Updated snapshots to adjust these changes.

[Copilot]

Pull request overview

Updates template-engine to better align with Concerto v4 API changes by removing deprecated ModelManager strict-mode usage and adding explicit serializer option objects to relevant deserialization calls.

Changes:

• Remove { strict: true } from ModelManager instantiations (tests + interpreter internal validation helpers).
• Update Serializer.fromJSON(...) call sites to pass an explicit SerializerOptions object.
• Update Jest snapshots to match the new diagnostics output.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
src/TemplateMarkInterpreter.ts	Removes ModelManager strict mode and adds serializer options for validation/deserialization.
test/TemplateMarkInterpreter.test.ts	Removes strict mode from ModelManager in interpreter integration tests.
test/HelloWorld.test.ts	Removes strict mode from ModelManager in hello-world test setup.
test/GenerateOptions.test.ts	Removes strict mode from ModelManager in generation options tests.
test/CustomTemplateModel.test.ts	Removes strict mode from ModelManager in custom model test setup.
test/__snapshots__/TemplateMarkInterpreter.test.ts.snap	Snapshot updates reflecting changed error locations/output.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[mttrbrts]

Thanks for working on this! However, adding explicit options like { validate: true, acceptResourcesForRelationships: false } everywhere shouldn't be necessary — these should be the defaults.

The root cause is that the TypeScript type definitions in Concerto v4 appear to incorrectly require these options. I've opened an upstream issue to fix this: accordproject/concerto#1192

Once that's fixed, this PR can be simplified to just remove the strict: true from ModelManager (which is the correct fix), without needing to add redundant options to every Serializer call.

Let's fix the type definitions upstream first, then revisit this PR.

This comment was generated by AI on behalf of @mttrbrts.

[rockaxorb13]

Understood! Sorry for the delay, I will get to this asap.

[rockaxorb13]

@mttrbrts done, ready for review

[coveralls]

Coverage Report for CI Build 25486640833

Warning

Build has drifted: This PR's base is out of sync with its target branch, so coverage data may include unrelated changes.
Quick fix: rebase this PR. Learn more →

Coverage remained the same at 63.844%

Details

• Coverage remained the same as the base build.
• Patch coverage: 1 uncovered change across 1 file (4 of 5 lines covered, 80.0%).
• No coverage regressions found.

Uncovered Changes

File	Changed	Covered	%
src/TemplateMarkInterpreter.ts	3	2	66.67%

Coverage Regressions

No coverage regressions found.

---

Coverage Stats

Relevant Lines:	1128
Covered Lines:	719
Line Coverage:	63.74%
Relevant Branches:	537
Covered Branches:	344
Branch Coverage:	64.06%
Branches in Coverage %:	Yes
Coverage Strength:	5700.41 hits per line

---

💛 - Coveralls