Why didn't the minister who changed the fate of Qin come to a good end?

Shang Yang made A Qin rich and strong through political reform, which is called "Shang Yang Political Reform" in history. Politically, Shang Yang reformed Qin's household registration, rank, land system, administrative divisions, taxation, weights and measures, and folk customs, and formulated harsh laws. Economically, Shang Yang advocated attaching importance to agriculture and restraining commerce, and rewarding agriculture and weaving. Militarily, Qin Jun, commander-in-chief of Shang Yang, recovered Hexi. In 338 BC, Qin Xiaogong died and his son Qin Huiwen succeeded him. In the same year of Qin Xiaogong's death, Shang Yang was framed for rebellion by Childe Qian, and he was defeated and died in his younger brother. His body was brought back to Xianyang and was publicly displayed after being cracked.

Sima Qian's comments: Shang Jun is cruel and ungrateful. He used the emperor Wang Zhidao to lobby for filial piety, relying on ostentation and extravagance, which he was not qualified to pursue. Besides, relying on the recommendation of the monarch's favorite eunuch, he punished his son as soon as he was appointed, cheated Wei's son and didn't listen to Zhao Liang's advice, which was enough to prove cruelty and ingratitude. I once read a book about Shang Jun's The Battle of Tillage, which is similar to his own works. But in the end, his rebellion in Qin was notorious for a reason!