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

Author: bnusunny

samcli/commands/build/build_context.py

@@ -919,6 +919,7 @@ def _copy_artifact_paths(self, original_resource: Dict, modified_resource: Dict)
             The modified resource with updated artifact paths
         """
         from samcli.lib.cfn_language_extensions.models import PACKAGEABLE_RESOURCE_ARTIFACT_PROPERTIES
+        from samcli.lib.package.language_extensions_packaging import _get_prop_value, _set_prop_value
 
         original_props = original_resource.get("Properties", {})
         modified_props = modified_resource.get("Properties", {})
@@ -929,8 +930,9 @@ def _copy_artifact_paths(self, original_resource: Dict, modified_resource: Dict)
             return
 
         for prop_name in prop_names:
-            if prop_name in modified_props:
-                original_props[prop_name] = modified_props[prop_name]
+            value = _get_prop_value(modified_props, prop_name)
+            if value is not None:
+                _set_prop_value(original_props, prop_name, value)
 
     def _gen_success_msg(self, artifacts_dir: str, output_template_path: str, is_default_build_dir: bool) -> str:
         """

tests/unit/commands/buildcmd/test_build_context_language_extensions.py

@@ -91,6 +91,21 @@ def test_no_matching_property_no_copy(self):
         ctx._copy_artifact_paths(original, modified)
         self.assertNotIn("CodeUri", original["Properties"])
 
+    def test_glue_job_dotted_path(self):
+        """Regression: dotted property path (Command.ScriptLocation) must be copied."""
+        ctx = self._make_context()
+        original = {
+            "Type": "AWS::Glue::Job",
+            "Properties": {"Command": {"Name": "glueetl", "ScriptLocation": "./script.py"}},
+        }
+        modified = {
+            "Type": "AWS::Glue::Job",
+            "Properties": {"Command": {"Name": "glueetl", "ScriptLocation": "s3://b/k.py"}},
+        }
+        ctx._copy_artifact_paths(original, modified)
+        self.assertEqual(original["Properties"]["Command"]["ScriptLocation"], "s3://b/k.py")
+        self.assertEqual(original["Properties"]["Command"]["Name"], "glueetl")
+
 
 class TestGetTemplateForOutputForEachExploration(TestCase):
     """