refactor(cfn-lang-ext): use jmespath for artifact property copy

Author: bnusunny

samcli/lib/package/language_extensions_packaging.py

@@ -271,11 +271,12 @@ def _copy_artifact_uris_for_type(
 
     copied = False
     for prop_name in prop_names:
-        if prop_name not in exported_props:
+        exported_value = _get_prop_value(exported_props, prop_name)
+        if exported_value is None:
             continue
         if dynamic_prop_keys and foreach_key and (foreach_key, prop_name) in dynamic_prop_keys:
             continue
-        original_props[prop_name] = exported_props[prop_name]
+        _set_prop_value(original_props, prop_name, exported_value)
         copied = True
 
     return copied

tests/unit/commands/package/test_package_context.py

@@ -1002,6 +1002,18 @@ def test_set_prop_value_dotted_key_creates_intermediate(self):
         # Existing keys preserved
         self.assertEqual(props["Command"]["Name"], "glueetl")
 
+    def test_copy_artifact_uris_for_type_glue_job_dotted_path(self):
+        """Regression: dotted property path (Command.ScriptLocation) must be copied."""
+        original_props = {"Command": {"Name": "glueetl", "ScriptLocation": "./script.py"}}
+        exported_props = {"Command": {"Name": "glueetl", "ScriptLocation": "s3://b/k.py"}}
+
+        result = _copy_artifact_uris_for_type(original_props, exported_props, "AWS::Glue::Job")
+
+        self.assertTrue(result)
+        self.assertEqual(original_props["Command"]["ScriptLocation"], "s3://b/k.py")
+        # Sibling keys preserved
+        self.assertEqual(original_props["Command"]["Name"], "glueetl")
+
 
 class TestPackageContextMappingsIntegration(TestCase):
     """Test cases for the complete Mappings transformation integration in _export()"""