Refactor ESP service interface instructions and update DFU query sort order mappings

Author: GordonSmith

.github/instructions/esp-service-interface.instructions.md

@@ -11,11 +11,6 @@ applyTo: '**/*.ecm'
 - Include the `exceptions_inline` attribute in any new `ESPresponse` element.
 - Include the `exceptions_inline` attribute in any new `ESPmethod` element only if its parent `ESPservice` element doesn't have an `exceptions_inline` attribute.
 
-## Version Bumping
-- Always bump the `ESPservice` `version` and `default_client_version` when changing request/response schemas or behavior.
-- Use the next sequential version string (for example, if current is 1.67, bump to 1.68).
-- Keep changes backward compatible; do not remove or change existing fields without a version gate.
-
 ## Naming Conventions
 - Use `PascalCase` for element names.
 - Use lower case for native types such as `string`, `integer`, `boolean`, etc.

dali/base/dadfs.cpp

@@ -14211,7 +14211,8 @@ static IPropertyTreeIterator *deserializeFileAttrIterator(MemoryBuffer& mb, unsi
             if (options.includeField(DFUQResultField::origsize))
             {
                 const char *propName = getDFUQResultFieldName(DFUQResultField::origsize);
-                attr->setPropInt64(propName, attr->getPropInt64(getDFUQResultFieldName(DFUQResultField::origsize), -1));
+                if (!attr->hasProp(propName))  
+                    attr->setPropInt64(propName, -1); 
             }
 
             if (options.includeField(DFUQResultField::recordcount))

esp/services/ws_dfu/ws_dfuService.cpp

@@ -3769,7 +3769,6 @@ void CWsDfuEx::setDFUQuerySortOrder(IEspDFUQueryRequest& req, StringBuffer& sort
     static const std::unordered_map<std::string_view, std::string_view> legacyMappings =
     {
         {"FileSize", "@DFUSFsize"},
-        {"Size", "@origsize"},
         {"CompressedFileSize", "@compressedSize"},
         {"ContentType", "@kind"},
         {"IsCompressed", "@compressed"},