magical instant shared inventories is a huge nono
Well, this "nono" is allready there, was allways ben there, and Larian never, ever, expressed any desire to remove it ...
On the contrary, this particular feature (called "magic pockets") was presented as one of first, if not THE first ever QoL feature in game.
So ...
I dare to presume this is one of prime examples of Larian Sandbox style ... its there ... now its up to you if you wish to use it, or ignore it.
why have caster classes when you can just have a endless potion scroll supply around the corner
There is few reasons ...
1) Caster healing is stronger
2) Scrolls cant be upcasted, and there is not *that much* (even tho still a lot) stronger potions.
3) Drinking potions on your turn takes your Bonus Action ... that may not seem like a big isue, but if you are Berserker barbarian, this means minus one attack ... if you are using Heat Metal, this means you are holding concentration for nothing ... etc, you get the idea.
4) Not all classes are casters, or healers ... so saying classes are meaningless with scrolls and potions is just not true.
Show me potion or scroll that gives you expertise, that gives you wildshape, that gives you bardic inspiration, or rage ... see? Classes have their meaning even with unlimited suplies of scrolls and potions.
5) Some people tend to forget this, but ... having classes, is "fun" ... you know, the very reason games exist in the first place.
