From 33d92077d6c20609935b33cb5a8940e4e6f36fc4 Mon Sep 17 00:00:00 2001
From: forhappy <haipingf@gmail.com>
Date: Thu, 27 Aug 2026 16:35:14 -0700
Subject: [PATCH 1/2] chore(deps): update yanked chacha20 lock entries

---
 Cargo.lock      | 4 ++--
 fuzz/Cargo.lock | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/Cargo.lock b/Cargo.lock
index b1443541..7eee5715 100644
--- a/Cargo.lock
+++ b/Cargo.lock
@@ -1043,9 +1043,9 @@ checksum = "f079e83a288787bcd14a6aea84cee5c87a67c5a3e660c30f557a3d24761b3527"
 
 [[package]]
 name = "chacha20"
-version = "0.10.1"
+version = "0.10.2"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "d524456ba66e72eb8b115ff89e01e497f8e6d11d78b70b1aa13c0fbd97540a81"
+checksum = "65c35e4b699c7e15ccbe7ee35c005e4fc0a278d22238a2857e6ce2dadeda1b06"
 dependencies = [
  "cfg-if",
  "cpufeatures 0.3.0",
diff --git a/fuzz/Cargo.lock b/fuzz/Cargo.lock
index 271076b5..7196d90e 100644
--- a/fuzz/Cargo.lock
+++ b/fuzz/Cargo.lock
@@ -763,9 +763,9 @@ checksum = "f079e83a288787bcd14a6aea84cee5c87a67c5a3e660c30f557a3d24761b3527"
 
 [[package]]
 name = "chacha20"
-version = "0.10.1"
+version = "0.10.2"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "d524456ba66e72eb8b115ff89e01e497f8e6d11d78b70b1aa13c0fbd97540a81"
+checksum = "65c35e4b699c7e15ccbe7ee35c005e4fc0a278d22238a2857e6ce2dadeda1b06"
 dependencies = [
  "cfg-if",
  "cpufeatures 0.3.0",

From a5dd1538da992d9da52f4c5209a6b19a0569b7f7 Mon Sep 17 00:00:00 2001
From: forhappy <haipingf@gmail.com>
Date: Fri, 11 Sep 2026 00:37:07 -0700
Subject: [PATCH 2/2] fix(review): preserve strict URI node records

---
 crates/compass-cli/tests/review_cli.rs    |  24 ++++-
 crates/compass-history/src/artifacts.rs   |  24 ++++-
 crates/compass-history/tests/roundtrip.rs | 119 +++++++++++++++-------
 3 files changed, 130 insertions(+), 37 deletions(-)

diff --git a/crates/compass-cli/tests/review_cli.rs b/crates/compass-cli/tests/review_cli.rs
index 040cad91..8e2407eb 100644
--- a/crates/compass-cli/tests/review_cli.rs
+++ b/crates/compass-cli/tests/review_cli.rs
@@ -92,7 +92,11 @@ fn local_review_writes_round_trippable_exact_report() -> Result<(), Box<dyn std:
         directory.path().join("feature.rs"),
         "pub fn feature() -> u8 { 2 }\n",
     )?;
-    git(directory.path(), &["add", "feature.rs"])?;
+    std::fs::write(
+        directory.path().join("package.json"),
+        "{\"dependencies\":{\"fixture-package\":\"1.0.0\"}}\n",
+    )?;
+    git(directory.path(), &["add", "feature.rs", "package.json"])?;
     git(directory.path(), &["commit", "--quiet", "-m", "feature"])?;
     let head = git(directory.path(), &["rev-parse", "HEAD"])?;
     git(directory.path(), &["checkout", "--quiet", "main"])?;
@@ -137,6 +141,24 @@ fn local_review_writes_round_trippable_exact_report() -> Result<(), Box<dyn std:
     assert_eq!(report.identity.revisions.target_head, base);
     assert_eq!(report.identity.revisions.pull_request_head, head);
     assert!(report.identity.revisions.merge_result.is_clean());
+    let repository = Repository::discover(directory.path())?;
+    let comparison = report
+        .identity
+        .revisions
+        .merge_result
+        .object_id()
+        .ok_or("clean review has no merge result")?;
+    let history = HistoryStore::open_existing(&repository)?.ok_or("review history store")?;
+    let realization = history
+        .preferred(&repository.resolve(comparison)?)?
+        .ok_or("review comparison realization")?;
+    let graph = history.reader(&realization.id)?.graph_document()?;
+    assert!(
+        graph
+            .nodes
+            .iter()
+            .any(|node| node.string("uri") == "pkg:fixture-package")
+    );
 
     let preserved_path = directory.path().join("bounded-review.md");
     std::fs::write(&preserved_path, "preserve-me")?;
diff --git a/crates/compass-history/src/artifacts.rs b/crates/compass-history/src/artifacts.rs
index df9dbeba..9d8077c9 100644
--- a/crates/compass-history/src/artifacts.rs
+++ b/crates/compass-history/src/artifacts.rs
@@ -26,10 +26,30 @@ const ANALYSIS_KIND: &[u8] = &[4];
 const METADATA_SCHEMA: &[u8] = &[1];
 const METADATA_KIND: &[u8] = &[5];
 const MOVED_NODE_FIELDS: [&str; 3] = ["community", "community_name", "norm_label"];
-const NODE_COMPATIBILITY_FIELDS: [&str; 14] = [
+const NODE_COMPATIBILITY_FIELDS: &[&str] = &[
     "label",
     "qualified_name",
     "file_type",
+    "document_format",
+    "document_kind",
+    "document_role",
+    "block_index",
+    "document_ordinal",
+    "document_section",
+    "uri",
+    "document_content",
+    "document_significance",
+    "table_columns",
+    "table_headers",
+    "table_alignments",
+    "table_body_row_count",
+    "table_omitted_row_count",
+    "table_omitted_column_count",
+    "table_truncated",
+    "table_row_index",
+    "table_identity_cell_index",
+    "table_cells",
+    "document_references",
     "source_file",
     "source_location",
     "line_start",
@@ -1162,7 +1182,7 @@ fn compat_edge(edge: &compass_model::code_graph::EdgeRecord) -> Result<EdgeRecor
 fn trusted_node_value(node: &NodeRecord) -> Value {
     let mut object = node.attributes.clone();
     for field in NODE_COMPATIBILITY_FIELDS {
-        object.remove(field);
+        object.remove(*field);
     }
     object.insert("id".to_owned(), Value::String(node.id.clone()));
     Value::Object(object)
diff --git a/crates/compass-history/tests/roundtrip.rs b/crates/compass-history/tests/roundtrip.rs
index 97e445a8..048ae586 100644
--- a/crates/compass-history/tests/roundtrip.rs
+++ b/crates/compass-history/tests/roundtrip.rs
@@ -57,25 +57,60 @@ fn trusted_graph_partitions_store_full_typed_node_records() -> Result<(), Box<dy
         "generated":false,
         "extractionStatus":"extracted"
     }]);
-    graph["nodes"] = json!([{
-        "id":"symbol:test",
-        "kind":"function",
-        "name":"test",
-        "qualifiedName":"fixture.test",
-        "source":{
-            "file":"src/lib.rs",
-            "startByte":0,
-            "endByte":4,
-            "startLine":1,
-            "startColumn":0,
-            "endLine":1,
-            "endColumn":4
+    graph["nodes"] = json!([
+        {
+            "id":"symbol:test",
+            "kind":"function",
+            "name":"test",
+            "qualifiedName":"fixture.test",
+            "source":{
+                "file":"src/lib.rs",
+                "startByte":0,
+                "endByte":4,
+                "startLine":1,
+                "startColumn":0,
+                "endLine":1,
+                "endColumn":4
+            },
+            "evidence":[{
+                "origin":"config",
+                "extractor":"fixture",
+                "confidence":"exact",
+                "anchors":[{
+                    "file":"src/lib.rs",
+                    "startByte":0,
+                    "endByte":4,
+                    "startLine":1,
+                    "startColumn":0,
+                    "endLine":1,
+                    "endColumn":4
+                }]
+            }],
+            "coverage":[{
+                "capability":"node:function",
+                "producer":"fixture",
+                "status":"partial",
+                "reason":"fixture"
+            }],
+            "diagnostics":[{
+                "severity":"warning",
+                "code":"fixture",
+                "message":"fixture warning"
+            }]
         },
-        "evidence":[{
-            "origin":"config",
-            "extractor":"fixture",
-            "confidence":"exact",
-            "anchors":[{
+        {
+            "id":"resource:guide",
+            "kind":"resource",
+            "name":"Guide",
+            "qualifiedName":"Guide",
+            "details":{
+                "type":"resource",
+                "data":{
+                    "resourceKind":"document",
+                    "uri":"#guide"
+                }
+            },
+            "source":{
                 "file":"src/lib.rs",
                 "startByte":0,
                 "endByte":4,
@@ -83,28 +118,44 @@ fn trusted_graph_partitions_store_full_typed_node_records() -> Result<(), Box<dy
                 "startColumn":0,
                 "endLine":1,
                 "endColumn":4
+            },
+            "evidence":[{
+                "origin":"config",
+                "extractor":"fixture",
+                "confidence":"exact",
+                "anchors":[{
+                    "file":"src/lib.rs",
+                    "startByte":0,
+                    "endByte":4,
+                    "startLine":1,
+                    "startColumn":0,
+                    "endLine":1,
+                    "endColumn":4
+                }]
             }]
-        }],
-        "coverage":[{
-            "capability":"node:function",
-            "producer":"fixture",
-            "status":"partial",
-            "reason":"fixture"
-        }],
-        "diagnostics":[{
-            "severity":"warning",
-            "code":"fixture",
-            "message":"fixture warning"
-        }]
-    }]);
+        }
+    ]);
     std::fs::write(
         directory.path().join("graph.json"),
         canonical_json_bytes(&graph)?,
     )?;
     let partition = GraphArtifacts::load(directory.path())?.partition(&completion())?;
-    let record = VersionedValue::from_bytes(&partition.nodes[0].1)?;
-    assert_eq!(record.schema, "compass.graph.node.v1");
-    let payload: Value = serde_json::from_slice(&record.payload)?;
+    for (_, encoded) in &partition.nodes {
+        let record = VersionedValue::from_bytes(encoded)?;
+        assert_eq!(record.schema, "compass.graph.node.v1");
+        let _: compass_model::code_graph::NodeRecord = serde_json::from_slice(&record.payload)?;
+    }
+    let payload = partition
+        .nodes
+        .iter()
+        .map(|(_, encoded)| VersionedValue::from_bytes(encoded))
+        .collect::<Result<Vec<_>, _>>()?
+        .into_iter()
+        .map(|record| serde_json::from_slice::<Value>(&record.payload))
+        .collect::<Result<Vec<_>, _>>()?
+        .into_iter()
+        .find(|payload| payload["id"] == "symbol:test")
+        .ok_or("missing symbol payload")?;
     assert_eq!(payload["evidence"][0]["origin"], "config");
     assert_eq!(payload["coverage"][0]["status"], "partial");
     assert_eq!(payload["diagnostics"][0]["code"], "fixture");