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

[Young-Flash]

No description provided.

[peter-jerry-ye-code-review]

Message parameter in abort() is ignored without logging

Category
Correctness
Code Snippet
pub fn abort[T](msg : String) -> T {
let _ = msg
panic()
}
Recommendation
Consider logging the message before panicking:

pub fn abort[T](msg : String) -> T {
  // Add logging here if available
  print(msg) // or equivalent logging mechanism
  panic()
}

Reasoning
The current implementation silently ignores the error message parameter. Error messages are crucial for debugging, and discarding them reduces the utility of the abort function.

Documentation needs updating after refactoring

Category
Maintainability
Code Snippet
///|
/// Aborts the program. Always causes a panic, regardless of the message provided.
Recommendation
Add back the complete documentation including parameters section and example usage as it was in the original code
Reasoning
The documentation was significantly reduced during the refactoring. While the code is simpler, maintaining comprehensive documentation is important for API users.

Function alias declarations lack documentation

Category
Maintainability
Code Snippet
pub fnalias @panic.panic
pub fnalias @panic.abort
Recommendation
Add documentation comments explaining that these are aliases to the new panic module implementations:

///|
/// Alias to panic module's panic function
pub fnalias @panic.panic

Reasoning
Function aliases should be documented to maintain code clarity and help developers understand the relationship between the old and new implementations.