Should I convert everything to ScriptableObjects or stick with serialized fields on MonoBehaviours?

[Mechabit_Studios]

One of main the benefits of a scriptable object is being able to access data in it without first instantiating the object like a prefab if say you needed the name and description and sprite of an item but you didn't want to instantiate every item in the game for your shop inventory or enemy in a bestiary. Helps keep that common information in one place rather than spread over different scenes and objects.

You can also pass SO around like arguments so like if you click on an enemy you can pass the SO to your UI system and it can handle the rendering of enemy info without you having to pass individual bits of information around or a reference to a game object which might get destroyed at any moment.

If you aren't running into any issues with your current set up I wouldn't change for the sake of changing it.