Sort keys in all JSON dumps

Author: davegaeddert

importmap/core.py

@@ -33,7 +33,7 @@ class LockfileSchema(Schema):
 
 
 def hash_for_data(data):
-    return hashlib.md5(json.dumps(data).encode("utf-8")).hexdigest()
+    return hashlib.md5(json.dumps(data, sort_keys=True).encode("utf-8")).hexdigest()
 
 
 class Importmap:
@@ -102,7 +102,7 @@ def load_lockfile(self):
 
     def save_lockfile(self, lockfile):
         with open(self.lock_filename, "w+") as f:
-            json.dump(lockfile, f, indent=2)
+            json.dump(lockfile, f, indent=2, sort_keys=True)
 
     def delete_lockfile(self):
         if os.path.exists(self.lock_filename):

importmap/templatetags/importmap.py

@@ -14,6 +14,6 @@ def importmap_scripts():
     importmap.load()
 
     if settings.DEBUG:
-        return {"importmap": json.dumps(importmap.map_dev, indent=2)}
+        return {"importmap": json.dumps(importmap.map_dev, indent=2, sort_keys=True)}
     else:
-        return {"importmap": json.dumps(importmap.map)}
+        return {"importmap": json.dumps(importmap.map, sort_keys=True)}

test_project/importmap.lock

@@ -1,23 +1,23 @@
 {
-  "config_hash": "09d6237cdd891aad07de60f54689d130",
-  "importmap": {
+  "importmap_dev": {
     "imports": {
-      "react": "https://ga.jspm.io/npm:react@17.0.2/index.js"
+      "react": "https://ga.jspm.io/npm:react@17.0.2/dev.index.js"
     },
     "scopes": {
       "https://ga.jspm.io/": {
         "object-assign": "https://ga.jspm.io/npm:object-assign@4.1.1/index.js"
       }
     }
   },
-  "importmap_dev": {
+  "importmap": {
     "imports": {
-      "react": "https://ga.jspm.io/npm:react@17.0.2/dev.index.js"
+      "react": "https://ga.jspm.io/npm:react@17.0.2/index.js"
     },
     "scopes": {
       "https://ga.jspm.io/": {
         "object-assign": "https://ga.jspm.io/npm:object-assign@4.1.1/index.js"
       }
     }
-  }
+  },
+  "config_hash": "09d6237cdd891aad07de60f54689d130"
 }