I’m a bit confused on your approach here. Whats the point of creating the module if you’re just going to include it into the top level namespace? At that point you could just define all these as methods on the top level and not even use a module in the first place. Also having a module just to hold a class variable that is used by another module is also kinda weird. You could just use a constant on the top level and get the same effect.
Namespacing is a good practice as it avoid name conflicts, especially if you release this code as a shard. Otherwise, depending on what your end goal is, I’m not sure this is the best approach.