internal: add virtual pkg "moonbitlang/core/panic"

[Young-Flash]

No description provided.

[peter-jerry-ye-code-review]

Message parameter in abort() is ignored

Category
Correctness
Code Snippet
pub fn abort[T](msg : String) -> T {
let _ = msg
panic()
}
Recommendation
The error message should be logged or displayed before panicking. Consider using a system logging function or stderr output before calling panic()
Reasoning
Currently the error message parameter is completely ignored (bound to _ and never used). This defeats the purpose of having a message parameter since the error context will be lost.

Missing documentation for panic() function

Category
Maintainability
Code Snippet
pub fn panicT -> T {
@intrinsics.unreachable()
}
Recommendation
Add documentation comments explaining the purpose and behavior of the panic() function, similar to how abort() is documented
Reasoning
Consistent documentation helps maintainers and users understand the purpose and behavior of public APIs. The panic() function is public but lacks explanatory comments.

Package organization could be improved

Category
Maintainability
Code Snippet
"import": [
"moonbitlang/core/panic"
],
Recommendation
Consider moving panic-related functionality to a more general error handling module that could also include other error handling mechanisms in the future
Reasoning
While separating panic into its own module is good, it might make more sense as part of a broader error handling module that could include other error handling patterns in the future. This would provide a more cohesive organization of error-related functionality.