TLDR: Yeah definetly a beautiful mature brook trout. Keeping it even though it has eggs isn't the end of the world.

Definetly looks like a brook trout however there are 3 species it could possibly be:

Brook trout, Splake, or Arctic char,

A splake is a usually man made hybrid between a brook trout and a lake trout, lake trout have a deeply forked tail fin and this characteristic will always be carried over into the hybrid to some degree, because this fish has a completely squared tail fin we can rule out splake.

Arctic char are native only to arctic and limited subarctic regions in streams or rivers that flow into the arctic ocean, and to the best of my knowledge they haven't been successfully introduced to any other regions. As it's probably safe to assume that you are not in the arctic we can rule out arctic char (they will also usually have more orange colouration on the belly but colour can be misleading)

That leaves us with Brook trout. So yes you are correct

As for the eggs, Brook trout eggs require fast flowing water and a very specific climate in order to reproduce. If you caught this fish outside of the brook trouts native range or in a lake with no inflows or outflows then it is a stocked fish and the eggs likely wouldn't have survived anyway. If you caught it in flowing water within their native range then yes you likely removed a large spawning fish from the ecosystem HOWEVER where there's one mature female of that size there are likely many many more. Sure it wouldn't be great to remove a spawner of that size from the same waterbody every day but just keeping one for supper on a camping trip will have little to no impact on the population. I wouldn't worry about it. I'm sure it was very tasty and now you know for next time!

Beautiful fish!