Merge 'Nicer parse errors using miette' from Samyak Sarnayak

I noticed that the parse errors were a bit hard to read - only the
nearest token and the line/col offsets were printed.
I made a first attempt at improving the errors using
[miette](https://github.com/zkat/miette).
- Added derive for `miette::Diagnostic` to both the parser's error type
and LimboError.
- Added miette dependency to both sqlite3_parser and core. The `fancy`
feature is only enabled for the CLI. So the overhead on the libraries
(core, parser) should be minimal.
Some future improvements that can be made further:
- Add spans to AST nodes so that errors can better point to the correct
token. See upstream issue: gwenn/lemon-rs#33
- Construct more errors with offset information. I noticed that most
parser errors are constructed with `None` as the offset.
- The messages are a bit redundant (example "syntax error at (1, 6)").
This can improved.
Comparisons.
Before:
```
❯ cargo run --package limbo --bin limbo database.db --output-mode pretty
...
limbo> selet * from a;
[2025-01-05T11:22:55Z ERROR sqlite3Parser] near "Token([115, 101, 108, 101, 116])": syntax error
Parse error: near "selet": syntax error at (1, 6)
```
<img width="969" alt="image" src="https://github.com/user-
attachments/assets/82651a77-f5ac-4eee-b208-88c6ea7fc9b7" />
After:
```
❯ cargo run --package limbo --bin limbo database.db --output-mode pretty
...
limbo> selet * from a;
[2025-01-05T12:25:52Z ERROR sqlite3Parser] near "Token([115, 101, 108, 101, 116])": syntax error

  × near "selet": syntax error at (1, 6)
   ╭────
 1 │ selet * from a
   ·     ▲
   ·     ╰── syntax error
   ╰────

```
<img width="980" alt="image" src="https://github.com/user-
attachments/assets/747a90e5-5085-41f9-b0fe-25864179ca35" />

Closes #618

cli/Cargo.toml

@@ -28,3 +28,4 @@ limbo_core = { path = "../core" }
 rustyline = "12.0.0"
 ctrlc = "3.4.4"
 csv = "1.3.1"
+miette = { version = "7.4.0", features = ["fancy"] }

cli/app.rs

@@ -581,7 +581,10 @@ impl Limbo {
                                 break;
                             }
                             Err(err) => {
-                                let _ = self.write_fmt(format_args!("{}", err));
+                                let _ = self.write_fmt(format_args!(
+                                    "{:?}",
+                                    miette::Error::from(err).with_source_code(sql.to_owned())
+                                ));
                                 break;
                             }
                         }
@@ -595,7 +598,10 @@ impl Limbo {
             },
             Ok(None) => {}
             Err(err) => {
-                let _ = self.write_fmt(format_args!("{}", err));
+                let _ = self.write_fmt(format_args!(
+                    "{:?}",
+                    miette::Error::from(err).with_source_code(sql.to_owned())
+                ));
             }
         }
         // for now let's cache flush always

core/Cargo.toml

@@ -57,6 +57,7 @@ rand = "0.8.5"
 bumpalo = { version = "3.16.0", features = ["collections", "boxed"] }
 limbo_macros = { path = "../macros" }
 uuid = { version = "1.11.0", features = ["v4", "v7"], optional = true }
+miette = "7.4.0"
 
 [target.'cfg(not(target_family = "windows"))'.dev-dependencies]
 pprof = { version = "0.14.0", features = ["criterion", "flamegraph"] }

core/error.rs

@@ -1,6 +1,6 @@
 use thiserror::Error;
 
-#[derive(Debug, Error)]
+#[derive(Debug, Error, miette::Diagnostic)]
 pub enum LimboError {
     #[error("Corrupt database: {0}")]
     Corrupt(String),
@@ -10,7 +10,8 @@ pub enum LimboError {
     InternalError(String),
     #[error("Parse error: {0}")]
     ParseError(String),
-    #[error("Parse error: {0}")]
+    #[error(transparent)]
+    #[diagnostic(transparent)]
     LexerError(#[from] sqlite3_parser::lexer::sql::Error),
     #[error("Conversion error: {0}")]
     ConversionError(String),

vendored/sqlite3-parser/Cargo.toml

@@ -31,6 +31,7 @@ fallible-iterator = "0.3"
 bitflags = "2.0"
 uncased = "0.9.10"
 indexmap = "2.0"
+miette = "7.4.0"
 
 [dev-dependencies]
 env_logger = { version = "0.11", default-features = false }

vendored/sqlite3-parser/src/lexer/scan.rs

@@ -79,6 +79,12 @@ impl<S: Splitter> Scanner<S> {
     pub fn column(&self) -> usize {
         self.column
     }
+
+    /// Current byte offset in the source string
+    pub fn offset(&self) -> usize {
+        self.offset
+    }
+
     /// Associated splitter
     pub fn splitter(&self) -> &S {
         &self.splitter