make integrity field optional

Author: dsherret

src/graphs.rs

@@ -74,7 +74,7 @@ enum LockfileGraphPackage {
 struct LockfileNpmGraphPackage {
   /// Root ids that transitively reference this package.
   root_ids: HashSet<LockfilePkgId>,
-  integrity: String,
+  integrity: Option<String>,
   dependencies: BTreeMap<StackString, LockfileNpmPackageId>,
 }
 

src/lib.rs

@@ -75,7 +75,8 @@ pub struct NpmPackageDependencyLockfileInfo {
 
 #[derive(Debug, Clone, Serialize, Deserialize, Hash, PartialEq, Eq)]
 pub struct NpmPackageInfo {
-  pub integrity: String,
+  /// Will be `None` for patch packages.
+  pub integrity: Option<String>,
   #[serde(default)]
   pub dependencies: BTreeMap<StackString, StackString>,
 }
@@ -226,7 +227,7 @@ impl LockfileContent {
 
     #[derive(Debug, Deserialize)]
     struct RawNpmPackageInfo {
-      pub integrity: String,
+      pub integrity: Option<String>,
       #[serde(default)]
       pub dependencies: Vec<StackString>,
     }
@@ -761,7 +762,7 @@ impl Lockfile {
 
     let entry = self.content.packages.npm.entry(package_info.serialized_id);
     let package_info = NpmPackageInfo {
-      integrity: package_info.integrity,
+      integrity: Some(package_info.integrity),
       dependencies,
     };
     match entry {

src/printer.rs

@@ -28,7 +28,9 @@ struct SerializedJsrPkg<'a> {
 
 #[derive(Serialize)]
 struct SerializedNpmPkg<'a> {
-  integrity: &'a str,
+  /// Will be `None` for patch packages.
+  #[serde(skip_serializing_if = "Option::is_none")]
+  integrity: Option<&'a str>,
   #[serde(skip_serializing_if = "Vec::is_empty")]
   dependencies: Vec<Cow<'a, str>>,
 }
@@ -220,7 +222,7 @@ pub fn print_v4_content(content: &LockfileContent) -> String {
         (
           key.as_str(),
           SerializedNpmPkg {
-            integrity: &value.integrity,
+            integrity: value.integrity.as_deref(),
             dependencies: value
               .dependencies
               .iter()

tests/specs/config_changes/PatchNotChangingKeepsNpmDeps.txt

@@ -139,7 +139,6 @@
       ]
     },
     "[email protected]": {
-      "integrity": "sha512-dbDtVdEAncKctzrVZ+Nr7kHpHkv+0JDJb2MjjpBaj8bFeCkePU9rHfMklmhuLFnpeq/EJZk2IhStY6NzqgjOkg==",
       "dependencies": [
         "@ts-morph/common",
         "code-block-writer"
@@ -309,7 +308,6 @@
       ]
     },
     "[email protected]": {
-      "integrity": "sha512-dbDtVdEAncKctzrVZ+Nr7kHpHkv+0JDJb2MjjpBaj8bFeCkePU9rHfMklmhuLFnpeq/EJZk2IhStY6NzqgjOkg==",
       "dependencies": [
         "@ts-morph/common",
         "code-block-writer"