codegen: remove unnecessary sorting of keys in ID Maps

map-ordering.cwl

@@ -0,0 +1,37 @@
+cwlVersion: v1.2
+class: Workflow
+inputs:
+ 09first_input: str
+ 05second_input: int
+ 01third_input: File
+steps:
+ zz_step_one:
+   run:
+     class: ExpressionTool
+     inputs: []
+     outputs: []
+     expression: ${return {}; }
+     requirements:
+       InlineJavascriptRequirement: {}
+   in: []
+   out: []
+ 00_step_two:
+   out: []
+   run:
+     inputs: []
+     requirements:
+       InlineJavascriptRequirement: {}
+     outputs: []
+     expression: ${return; }
+     class: ExpressionTool
+   in: []
+outputs:
+  zz_first_output:
+    type: File
+    outputSource: 01third_input
+  ll_second_output:
+    type: str
+    outputSource: 09first_input
+  aa_third_output:
+    type: int
+    outputSource: 09first_input

schema_salad/dotnet/util/Loaders/IdMapLoader.cs

@@ -22,7 +22,7 @@ public object Load(in object doc_, in string baseuri, in LoadingOptions loadingO
         {
             List<object> r = new();
             Dictionary<string, object> d = docDict.Cast<dynamic>().ToDictionary(entry => (string)entry.Key, entry => entry.Value);
-            foreach (string? k in d.Keys.OrderBy(p => p))
+            foreach (string? k in d.Keys)
             {
                 object val = d[k];
                 if (val is IDictionary dictionary)

schema_salad/java/main_utils/IdMapLoader.java

@@ -27,9 +27,7 @@ public T load(
     if (doc instanceof Map) {
       final Map<String, Object> docMap = (Map<String, Object>) doc;
       final List<Object> asList = new ArrayList();
-      final TreeSet<String> sortedKeys = new TreeSet<String>();
-      sortedKeys.addAll(docMap.keySet());
-      for (final String key : sortedKeys) {
+      for (final String key : docMap.keySet()) {
         final Object el = docMap.get(key);
         if (el instanceof Map) {
           final Map<String, Object> v2 = new HashMap<String, Object>((Map<String, Object>) el);

schema_salad/metaschema.py

@@ -950,7 +950,7 @@ def load(
     ) -> Any:
         if isinstance(doc, MutableMapping):
             r: list[Any] = []
-            for k in sorted(doc.keys()):
+            for k in doc.keys():
                 val = doc[k]
                 if isinstance(val, CommentedMap):
                     v = copy.copy(val)

schema_salad/python_codegen_support.py

@@ -947,7 +947,7 @@ def load(
     ) -> Any:
         if isinstance(doc, MutableMapping):
             r: list[Any] = []
-            for k in sorted(doc.keys()):
+            for k in doc.keys():
                 val = doc[k]
                 if isinstance(val, CommentedMap):
                     v = copy.copy(val)

schema_salad/typescript/util/loaders/IdMapLoader.ts

@@ -14,7 +14,7 @@ export class _IdMapLoader implements Loader {
   async load (doc: any, baseuri: string, loadingOptions: LoadingOptions, docRoot?: string): Promise<any> {
     if (TypeGuards.isDictionary(doc)) {
       const r: any[] = []
-      for (var k of Object.keys(doc).sort(undefined)) {
+      for (var k of Object.keys(doc)) {
         const val = doc[k]
         if (TypeGuards.isDictionary(val)) {
           const v2 = Object.assign({}, val)