Replace a registry on registration instead of writing into it

[ericproulx]

A registry is written when a formatter, parser, params builder, validator or versioner class is defined, and from then on it is only ever looked up — on the request path, at that: once per response by Grape::Formatter, once per parsed body by Grape::Parser, once per params build by Grape::ParamsBuilder.

Have register build the next Hash and freeze it, rather than writing into the one readers hold. Nothing can find a registry changed underneath it, and nothing can write to one without going through register.

Notes:

• Registration still lands under both the String and the Symbol spelling, and still warns when it overrides an existing short name — the registry is replaced, not appended to, so a re-registration overrides exactly as before.
• spec/support/deregister.rb, the spec-only helper that undoes a registration, replaces the registry in turn. No unregister is added to the library.
• Two examples pin that a registry is frozen both before and after anything is registered.
• Also a prerequisite for serving a compiled API from non-main Ractors, which cannot read a class ivar holding an unshareable value — a frozen Hash of classes they can read.

🤖 Generated with Claude Code

[github-actions]

Danger Report

No issues found.

View run