[sdk generation pipeline] rename _meta.json to _metadata.json (#41187)

* rename _meta.json to _metadata.json

* format

* Update tools/azure-sdk-tools/packaging_tools/generate_utils.py

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* update

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Author: msyyc

tools/azure-sdk-tools/packaging_tools/generate_utils.py

@@ -132,8 +132,14 @@ def update_servicemetadata(sdk_folder, data, config, folder_name, package_name,
         _LOGGER.info(f"Fail to save metadata since package folder doesn't exist: {package_folder}")
         return
     for_swagger_gen = "meta" in config
-    metadata_folder = package_folder / "_meta.json"
-    if metadata_folder.exists() and for_swagger_gen:
+    # remove old _meta.json
+    old_metadata_folder = package_folder / "_meta.json"
+    if old_metadata_folder.exists():
+        os.remove(old_metadata_folder)
+        _LOGGER.info(f"Remove old metadata file: {old_metadata_folder}")
+
+    metadata_folder = package_folder / "_metadata.json"
+    if metadata_folder.exists():
         with open(metadata_folder, "r") as file_in:
             metadata = json.load(file_in)
     else:
@@ -172,31 +178,11 @@ def update_servicemetadata(sdk_folder, data, config, folder_name, package_name,
 
     _LOGGER.info("Metadata json:\n {}".format(json.dumps(metadata, indent=2)))
 
-    metadata_file_path = os.path.join(package_folder, "_meta.json")
-    with open(metadata_file_path, "w") as writer:
+    metadata_file_path = package_folder / "_metadata.json"
+    with metadata_file_path.open("w") as writer:
         json.dump(metadata, writer, indent=2)
     _LOGGER.info(f"Saved metadata to {metadata_file_path}")
 
-    # Check whether MANIFEST.in includes _meta.json
-    if "resource-manager" in input_readme:
-        require_meta = "include _meta.json\n"
-        manifest_file = os.path.join(package_folder, "MANIFEST.in")
-        if not os.path.exists(manifest_file):
-            _LOGGER.info(f"MANIFEST.in doesn't exist: {manifest_file}")
-            return
-
-        includes = []
-        write_flag = False
-        with open(manifest_file, "r") as f:
-            includes = f.readlines()
-            if require_meta not in includes:
-                includes = [require_meta] + includes
-                write_flag = True
-
-        if write_flag:
-            with open(manifest_file, "w") as f:
-                f.write("".join(includes))
-
 
 @return_origin_path
 def judge_tag_preview(path: str, package_name: str) -> bool:
@@ -462,8 +448,8 @@ def gen_typespec(
 
     with open(Path("eng/emitter-package.json"), "r") as file_in:
         data = json.load(file_in)
-        npm_package_verstion = {
-            typespec_python: data["dependencies"][typespec_python],
+        npm_package_version = {
+            "emitterVersion": data["dependencies"][typespec_python],
         }
 
-    return npm_package_verstion
+    return npm_package_version

tools/azure-sdk-tools/packaging_tools/templates/packaging_files/MANIFEST.in

@@ -1,4 +1,3 @@
-include _meta.json
 recursive-include tests *.py *.json
 recursive-include samples *.py *.md
 include *.md

tools/azure-sdk-tools/tests/test_servicemetadata.py

@@ -72,7 +72,7 @@ def test_metadata(self):
             self.config["projects"][readme_file] = {}
 
             package_folder = Path(self.sdk_folder, self.folder_name, self.package_name).expanduser()
-            metadata_file_path = os.path.join(package_folder, "_meta.json")
+            metadata_file_path = os.path.join(package_folder, "_metadata.json")
             manifest_file = os.path.join(package_folder, "MANIFEST.in")
             os.makedirs(package_folder)
 
@@ -125,10 +125,6 @@ def test_metadata(self):
 
             assert os.path.isfile(metadata_file_path) == True
             assert os.path.isfile(manifest_file) == True
-            with open(manifest_file, "r") as f:
-                meta_line = "include _meta.json\n"
-                line = f.readline()
-                assert meta_line == line
 
             # Test update MANIFEST.in again
             update_servicemetadata(
@@ -143,9 +139,3 @@ def test_metadata(self):
 
             assert os.path.isfile(metadata_file_path) == True
             assert os.path.isfile(manifest_file) == True
-            with open(manifest_file, "r") as f:
-                meta_line = "include _meta.json\n"
-                line = f.readline()
-                second_line = f.readline()
-                assert meta_line == line
-                assert meta_line != second_line