Now that read storage class being different to write
storage class no longer directly means that a component
is being used, the logic in FileFormatter to extract
a component has to change.

1. The extraction must happen from the original python
   type that was written and not the, possibly, generic
   python type read from disk. This requires we do a
   coercion before extracting the component.
2. Built-in types must always be converted to the
   original storage class before conversion to the
   requested read storage class.
3. Remove duplicate logic from JSON and YAML that
   was trying to convert built-in types to the
   requested type. This now has moved to the
   parent class.

Remove the requirement that we only do this if we did not
write it originally as a built-in type -- it didn't really
affect the code because of the presence of the isinstance
check later on (which has now moved into the main check).

Test that we can round trip Pydantic models, dataclasses,
classes that get created from dicts, and tuples using JSON
and YAML formatters.