[enilsen16]

Stacked on #2.

Summary

• objectidproto.Definition(enum, base) and MustDefinition(enum, base) read ObjectType and Separator from the annotation over a base Definition. Each type is then one package-level variable, the same way the Elixir use Trogon.ObjectId, proto: {...}, validate: :uuid declares it. A new Definition field, such as Aliases in feat: accept earlier prefixes with Definition.Aliases #8, works with no change here. This breaks v0.1.0's Definition(enum).
• One generic cache (cache.go) stores each type's spec by reflect.Type, as a pointer, for both IDs and unions. Definition must return the same value every time, and the docs say so.
• ObjectTypeOf[T]() and PrefixOf[T]() read the type's object type and prefix without an ID. They are the Go form of the Elixir module functions object_type/0 and prefix/0. The ID methods call them.

Benchmarks (Apple M5 Max)

	v0.1.0	Now
String	about 45 ns, 2 allocs	20 ns, 1 alloc
Parse	1 alloc	22 ns, 0 allocs

Tests

The tests cover the base overlay, a check that Definition runs once across String and Parse, the type-level accessors, and the benchmarks. CI runs only on PRs to main. Locally, go test -race, go vet and golangci-lint pass.

[coderabbitai]

Important

• 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

• Source mode: Auto
• Repository: Unavailable
• Branch: Unavailable
• File: No configuration file from this repository applied
• Commit: Unavailable
• Layers: Organization UI
• Review profile: CHILL
• Plan: Advanced
• Run ID: e40e5b13-e74a-4904-9253-2062fd761b8f

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}