The problem is that Carol Danvers' arc in Captain Marvel is being emotionally repressed by the Kree. She's a soldier from a regressive society that reduces people to mere weapons. She wins the movie when she embraces her emotions and gains full control of her powers. This is a perfectly fine movie plot on a logical level. Character has a flaw, character overcomes flaw in order to become self actualised and resolve the plot. It's even socially relevant, because women's emotions being treated as a taboo is a political issue in our own society. This is classic sci-fi writing. It's even the plot of five different Halo games.

Unfortunately, it necessitates that Brie Larson must spend 90% of the movie showing absolutely no emotion or feeling. Which is a TERRIBLE directing decision especially for the new flagship of your franchise.

There are ways to use framing to play on this kind of thing and make a good movie out of an emotionless character. Look at Spock and Data. Look at Blade Runner. But Captain Marvel chose not to do that, and to play the superhero genre straight, for... some reason, which resulted in a bad movie.