[radhikaa06]

Closes #289

Changes

• AvroVisitor now writes @namespace("org.acme.hr") instead of @namespace("org.acme.hr@1.0.0"). It uses ModelUtil.parseNamespace, the same as the Java and OData visitors.
• Removed the Avro skip from 7 cases in test/verification/cases.js and from the metamodel case in verification/corpus/manifest.json. They now compile with avro-tools 1.12.0.
• hr_integration stays skipped for Avro. I replaced its reason with the failure it actually hits now.
• Updated the 4 Avro snapshots and added a unit test for the namespace line.

Flags

• Two versions of one namespace now map to the same Avro namespace. A single model file can't import both (Concerto rejects that), so a clash only happens if a consumer loads both protocols side by side. If you'd rather keep the version in, something like org.acme.hr.v1_0_0 would work and I'm happy to switch.
• The newly enabled cases pass on exit code, but avro-tools still drops the leading underscore from _identifier and _timestamp. I left that out of this PR and described it in the issue.
• Locally npm test fails the branch coverage gate at 95.91%. main gives the same number on my machine, and CI on main reports 96.08%. This PR adds no branches.
• fix(graphql): generate valid GraphQL for empty enums #273 and fix(rust): resolve scalar maps and recursion #282 also edit cases.js and the snapshot file. Whichever lands second will need a small rebase.

Screenshots or Video

Before:

After:

Related Issues

• Issue bug: Avro IDL generated from a versioned namespace fails avro-tools #289
• Issue bug: GraphQL verification bugs (generated output fails graphql-js) #248, Issue bug: Protobuf verification bugs (generated output fails protobufjs) #249 (same verification suite, other targets)

Author Checklist

• Ensure you provide a DCO sign-off for your commits using the --signoff option of git commit.
• Vital features and changes captured in unit and/or integration tests
• Commits messages follow AP format
• Extend the documentation, if necessary
• Merging to main from radhikaa06:fix/avro-versioned-namespace

[radhikaa06]

@mttrbrts PTAL.

[mttrbrts]

Welcome to concerto-codegen, @radhikaa06, and thanks for this! 🎉

This is a great fix. A one-line change in AvroVisitor turns Avro verification from skipping every case to compiling 7 of 8, and it now handles namespaces the same way as the Java and OData generators. The issue (#289) was a model bug report: clear repro steps, before/after evidence, and you'd already worked out the follow-ups (imported types written by short name, and the leading-underscore fields). Updating the hr_integration skip reason to the failure it actually hits now was a nice touch too.

I checked the snapshot updates and the new unit test, and the codegen suite passes locally. Avro verification in CI will confirm the newly enabled cases compile with avro-tools.

After concerto#1582 last week, it's great to see you contributing across the Accord Project repos. The follow-ups you noted in #289 would make good next PRs if you're interested. Thanks again!

---

Generated by Claude Code

[radhikaa06]

Thank you so much for the detailed feedback and guidance! I'll continue exploring the follow-ups from #289.