refactor(semantic): remove more-asserts dependency (#5739)

Remove `more-asserts` dependency which was added in #5710.

As suggested by @Boshen in #5710 (review).

Cargo.toml

@@ -147,7 +147,6 @@ memoffset = "0.9.1"
 miette = { version = "7.2.0", features = ["fancy-no-syscall"] }
 mimalloc = "0.1.43"
 mime_guess = "2.0.5"
-more-asserts = "0.3.1"
 nonmax = "0.5.5"
 num-bigint = "0.4.6"
 num-traits = "0.2.19"

crates/oxc_semantic/Cargo.toml

@@ -30,7 +30,6 @@ oxc_syntax = { workspace = true }
 assert-unchecked = { workspace = true }
 indexmap = { workspace = true }
 itertools = { workspace = true }
-more-asserts = { workspace = true }
 phf = { workspace = true, features = ["macros"] }
 rustc-hash = { workspace = true }
 serde = { workspace = true, features = ["derive"], optional = true }

crates/oxc_semantic/src/counter.rs

@@ -4,8 +4,6 @@
 
 use std::cell::Cell;
 
-use more_asserts::assert_le;
-
 use oxc_ast::{
     ast::{
         BindingIdentifier, IdentifierReference, Program, TSEnumMemberName, TSModuleDeclarationName,
@@ -41,7 +39,12 @@ impl Counts {
         // This is not a big problem - allocating a `Vec` with excess capacity is cheap.
         // It's allocating with *not enough* capacity which is costly, as then the `Vec`
         // will grow and reallocate.
-        assert_le!(actual.symbols, estimated.symbols, "symbols count mismatch");
+        assert!(
+            actual.symbols <= estimated.symbols,
+            "symbols count mismatch {} <= {}",
+            actual.symbols,
+            estimated.symbols
+        );
     }
 }