feat: Implement some useful traits on Location

Signed-off-by: Dmitry Dygalo <[email protected]>

Author: Stranger6667

.github/workflows/ci.yml

@@ -29,6 +29,16 @@ jobs:
 
       - uses: Swatinem/rust-cache@v2
 
+      # (dd): These packages bumped their MSRV with a patch release.
+      #       As per my understanding cargo will be able to automatically
+      #       resolve the proper version with the resolver v3 which is MSRV-aware.
+      #       With the current MSRV it is not possible to force the resolver to find
+      #       proper versions, hence they are pinned manually here. The end users
+      #       will be able to do the same on their side (which is a bad UX though),
+      #       but apparently there is nothing can be done on the `jsonschema` side
+      #       beside bumping MSRV to 1.85 and upgrading to the new resolver.
+      - run: cargo update -p [email protected] --precise 0.7.4
+      - run: cargo update -p [email protected] --precise 0.1.5
       - run: cargo build -p jsonschema
 
   test-stable:

CHANGELOG.md

@@ -2,6 +2,10 @@
 
 ## [Unreleased]
 
+### Added
+
+- `Hash`, `PartialOrd`, `Ord` and `serde::Serialize` for `Location`.
+
 ## [0.29.0] - 2025-02-08
 
 ### Breaking Changes

crates/jsonschema-referencing/Cargo.toml

@@ -29,7 +29,7 @@ workspace = true
 
 [dev-dependencies]
 benchmark = { path = "../benchmark/" }
-codspeed-criterion-compat = { version = "2.7", default-features = false }
+codspeed-criterion-compat = { version = "2.9", default-features = false }
 criterion = { version = "0.5", default-features = false }
 referencing_testsuite = { package = "jsonschema-referencing-testsuite", path = "../jsonschema-referencing-testsuite/" }
 test-case = "3.3.1"

crates/jsonschema-referencing/benches/registry.rs

@@ -1,4 +1,6 @@
-use codspeed_criterion_compat::{criterion_group, criterion_main, BenchmarkId, Criterion};
+use codspeed_criterion_compat::{
+    criterion_group, criterion_main, BatchSize, BenchmarkId, Criterion,
+};
 use referencing::{Draft, Registry, SPECIFICATIONS};
 
 static DRAFT4: &[u8] = include_bytes!("../../benchmark/data/subresources/draft4.json");
@@ -28,7 +30,7 @@ fn bench_subresources(c: &mut Criterion) {
                     Registry::try_new("http://example.com/schema.json", resource)
                         .expect("Invalid registry input")
                 },
-                criterion::BatchSize::SmallInput,
+                BatchSize::SmallInput,
             );
         });
     }
@@ -57,7 +59,7 @@ fn bench_subresources(c: &mut Criterion) {
                     |(resource, registry)| {
                         registry.try_with_resource("http://example.com/schema.json", resource)
                     },
-                    criterion::BatchSize::SmallInput,
+                    BatchSize::SmallInput,
                 );
             },
         );

crates/jsonschema/src/paths.rs

@@ -136,9 +136,18 @@ impl From<usize> for LocationSegment<'_> {
 }
 
 /// A cheap to clone JSON pointer that represents location with a JSON value.
-#[derive(Debug, Clone, PartialEq, Eq)]
+#[derive(Debug, Clone, PartialEq, Eq, Hash, PartialOrd, Ord)]
 pub struct Location(Arc<String>);
 
+impl serde::Serialize for Location {
+    fn serialize<S>(&self, serializer: S) -> Result<S::Ok, S::Error>
+    where
+        S: serde::Serializer,
+    {
+        serializer.serialize_str(&self.0)
+    }
+}
+
 impl Location {
     /// Create a new, empty `Location`.
     pub fn new() -> Self {