<feature>[errorcode]: fix i18n gaps in copy ctor and SDK

- ErrorCode copy constructor: add missing cost and opaque fields
- ZSClient: deserialize message field from JSON response
- Add ErrorCodeI18nCase gate test (12 cases)

Change-Id: I0002errorcode0i18n0gaps0fix

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: AlanJager

header/src/main/java/org/zstack/header/errorcode/ErrorCode.java

@@ -104,6 +104,8 @@ public ErrorCode(ErrorCode other) {
         this.message = other.message;
         this.formatArgs = other.formatArgs == null ? null : other.formatArgs.clone();
         this.globalErrorCode = other.globalErrorCode;
+        this.cost = other.cost;
+        this.opaque = other.opaque;
     }
 
     public void setCode(String code) {

sdk/src/main/java/org/zstack/sdk/ZSClient.java

@@ -121,6 +121,10 @@ public ErrorCode deserialize(JsonElement jsonElement, Type type, JsonDeserializa
             if (item != null && item.isJsonPrimitive()) {
                 wrapper.setGlobalErrorCode(item.getAsString());
             }
+            item = object.get("message");
+            if (item != null && item.isJsonPrimitive()) {
+                wrapper.setMessage(item.getAsString());
+            }
             return wrapper;
         }
     }

test/src/test/groovy/org/zstack/test/integration/core/ErrorCodeI18nCase.groovy

@@ -0,0 +1,130 @@
+package org.zstack.test.integration.core
+
+import org.zstack.core.errorcode.LocaleUtils
+import org.zstack.header.errorcode.ErrorCode
+import org.zstack.testlib.SubCase
+
+class ErrorCodeI18nCase extends SubCase {
+
+    @Override
+    void setup() {
+        INCLUDE_CORE_SERVICES = false
+    }
+
+    @Override
+    void environment() {
+    }
+
+    @Override
+    void test() {
+        testLocaleUtilsExactMatch()
+        testLocaleUtilsBaseLanguageFallback()
+        testLocaleUtilsQValueSorting()
+        testLocaleUtilsNullAndEmpty()
+        testLocaleUtilsNoMatch()
+        testLocaleUtilsCaseInsensitive()
+        testLocaleUtilsMalformedHeader()
+        testErrorCodeCopyConstructor()
+        testErrorCodeCopyConstructorWithNulls()
+    }
+
+    @Override
+    void clean() {
+    }
+
+    // ---- LocaleUtils ----
+
+    void testLocaleUtilsExactMatch() {
+        def available = ["zh_CN", "en_US"] as Set
+        assert LocaleUtils.resolveLocale("zh-CN", available) == "zh_CN"
+        assert LocaleUtils.resolveLocale("en-US", available) == "en_US"
+    }
+
+    void testLocaleUtilsBaseLanguageFallback() {
+        def available = ["zh_CN", "en_US"] as Set
+        assert LocaleUtils.resolveLocale("en", available) == "en_US"
+        assert LocaleUtils.resolveLocale("zh", available) == "zh_CN"
+    }
+
+    void testLocaleUtilsQValueSorting() {
+        def available = ["zh_CN", "en_US"] as Set
+        assert LocaleUtils.resolveLocale("zh-CN,en;q=0.8", available) == "zh_CN"
+        assert LocaleUtils.resolveLocale("en-US,zh-CN;q=0.5", available) == "en_US"
+        // q-value should override header order
+        assert LocaleUtils.resolveLocale("en;q=0.8,zh-CN;q=1.0", available) == "zh_CN"
+    }
+
+    void testLocaleUtilsNullAndEmpty() {
+        def available = ["zh_CN", "en_US"] as Set
+        assert LocaleUtils.resolveLocale(null, available) == "en_US"
+        assert LocaleUtils.resolveLocale("", available) == "en_US"
+        assert LocaleUtils.resolveLocale("  ", available) == "en_US"
+    }
+
+    void testLocaleUtilsNoMatch() {
+        def available = ["zh_CN", "en_US"] as Set
+        assert LocaleUtils.resolveLocale("ja-JP,ko-KR", available) == "en_US"
+    }
+
+    void testLocaleUtilsCaseInsensitive() {
+        def available = ["zh_CN", "en_US"] as Set
+        assert LocaleUtils.resolveLocale("ZH-CN", available) == "zh_CN"
+        assert LocaleUtils.resolveLocale("EN-US", available) == "en_US"
+    }
+
+    void testLocaleUtilsMalformedHeader() {
+        def available = ["zh_CN", "en_US"] as Set
+        // malformed header should fall back to default
+        assert LocaleUtils.resolveLocale(";;;,,,", available) == "en_US"
+    }
+
+    // ---- ErrorCode copy constructor ----
+
+    void testErrorCodeCopyConstructor() {
+        def original = new ErrorCode("SYS.1000", "System Error", "something failed")
+        original.setElaboration("elaboration text")
+        original.setLocation("org.zstack.Foo:123")
+        original.setCost("50ms")
+        original.setGlobalErrorCode("ORG_ZSTACK_FOO_10000")
+        original.setMessage("系统错误")
+        original.setFormatArgs(["arg1", "arg2"] as String[])
+
+        def opaque = new LinkedHashMap()
+        opaque.put("key1", "value1")
+        original.setOpaque(opaque)
+
+        def cause = new ErrorCode("INTERNAL.1001", "Internal Error")
+        original.setCause(cause)
+
+        def copy = new ErrorCode(original)
+
+        assert copy.code == original.code
+        assert copy.description == original.description
+        assert copy.details == original.details
+        assert copy.elaboration == original.elaboration
+        assert copy.location == original.location
+        assert copy.cost == original.cost
+        assert copy.globalErrorCode == original.globalErrorCode
+        assert copy.message == original.message
+        assert copy.opaque.is(original.opaque)
+        assert copy.cause.is(original.cause)
+        // formatArgs should be cloned, not shared
+        assert copy.formatArgs == original.formatArgs
+        assert !copy.formatArgs.is(original.formatArgs)
+    }
+
+    void testErrorCodeCopyConstructorWithNulls() {
+        def original = new ErrorCode("SYS.1000", "System Error")
+        def copy = new ErrorCode(original)
+
+        assert copy.code == original.code
+        assert copy.description == original.description
+        assert copy.details == null
+        assert copy.cost == null
+        assert copy.opaque == null
+        assert copy.message == null
+        assert copy.globalErrorCode == null
+        assert copy.formatArgs == null
+    }
+
+}