Merge branch 'main' into patch-5

src/history/item.rs

@@ -6,9 +6,10 @@ use std::{fmt::Display, time::Duration};
 
 /// Unique ID for the [`HistoryItem`]. More recent items have higher ids than older ones.
 #[derive(Debug, Copy, Clone, PartialEq, Eq, PartialOrd, Ord, Hash, Serialize, Deserialize)]
-pub struct HistoryItemId(pub(crate) i64);
+pub struct HistoryItemId(pub i64);
 impl HistoryItemId {
-    pub(crate) const fn new(i: i64) -> HistoryItemId {
+    /// Create a new `HistoryItemId` value
+    pub const fn new(i: i64) -> HistoryItemId {
         HistoryItemId(i)
     }
 }

src/result.rs

@@ -3,7 +3,7 @@ use thiserror::Error;
 
 /// non-public (for now)
 #[derive(Error, Debug)]
-pub(crate) enum ReedlineErrorVariants {
+pub enum ReedlineErrorVariants {
     // todo: we should probably be more specific here
     #[cfg(any(feature = "sqlite", feature = "sqlite-dynlib"))]
     #[error("error within history database: {0}")]
@@ -21,7 +21,7 @@ pub(crate) enum ReedlineErrorVariants {
 
 /// separate struct to not expose anything to the public (for now)
 #[derive(Debug)]
-pub struct ReedlineError(pub(crate) ReedlineErrorVariants);
+pub struct ReedlineError(pub ReedlineErrorVariants);
 
 impl Display for ReedlineError {
     fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {