Update utils/block_relation_builder.py

utils/block_relation_builder.py

@@ -2514,9 +2514,36 @@ def generate_plan(generated_input, opcode_keys, pseudo_code):
 #--------------------------------------------------[Security key id generation for the better understanding of keys]---------------------------------------------
 #################################################################################################################################################################
 
-def generate_secure_token(length=20):
-    charset = string.ascii_letters + string.digits + "!@#$%^&*()[]{}=+-_~"
-    return ''.join(secrets.choice(charset) for _ in range(length))
+# def generate_secure_token(length=20):
+#     charset = string.ascii_letters + string.digits + "!@#$%^&*()[]{}=+-_~"
+#     return ''.join(secrets.choice(charset) for _ in range(length))
+# conservative, safe alphabet derived from Scratch-style IDs
+_DEFAULT_SCRATCH_ALPHABET = string.ascii_letters + string.digits + "_-"
+_DEFAULT_LENGTH = 20
+
+# small in-memory cache to avoid duplicates in a single run
+_generated_ids_cache = set()
+
+def generate_secure_token() -> str:
+    """
+    Return a single Scratch-style ID string (20 chars long) using a safe alphabet.
+    No input required.
+
+    Notes:
+    - This does NOT check for collisions inside an existing project.json.
+      If you need to guarantee uniqueness with an existing file, use the
+      project-aware generator shared earlier.
+    - Collisions are astronomically unlikely for random 20-char tokens, but
+      the function still avoids duplicates within the same Python process.
+    """
+    alphabet = _DEFAULT_SCRATCH_ALPHABET
+    length = _DEFAULT_LENGTH
+
+    while True:
+        token = "".join(secrets.choice(alphabet) for _ in range(length))
+        if token not in _generated_ids_cache:
+            _generated_ids_cache.add(token)
+            return token
 
 #################################################################################################################################################################
 #--------------------------------------------------[Processed the two Skelton as input and generate refined skelton json]----------------------------------------
@@ -2527,8 +2554,8 @@ def process_scratch_blocks(all_generated_blocks, generated_output_json):
 
     # Initialize dictionaries to store and reuse generated unique IDs
     # This prevents creating multiple unique IDs for the same variable/broadcast across different blocks
-    variable_id_map = defaultdict(lambda: generate_secure_token(20))
-    broadcast_id_map = defaultdict(lambda: generate_secure_token(20))
+    variable_id_map = defaultdict(lambda: generate_secure_token())
+    broadcast_id_map = defaultdict(lambda: generate_secure_token())
 
     # Define the mapping for input field names to their required integer types for shadows
     input_type_mapping = {
@@ -2590,7 +2617,7 @@ def process_scratch_blocks(all_generated_blocks, generated_output_json):
                         processed_block["inputs"][input_name] = input_data
                     else:
                         # Fallback for unexpected formats, try to use the original if possible
-                        processed_block["inputs"][input_name] = gen_block_data["inputs"].get(input_name, [1, [11, "message1", generate_secure_token(20)]])
+                        processed_block["inputs"][input_name] = gen_block_data["inputs"].get(input_name, [1, [11, "message1", generate_secure_token()]])
 
                 elif isinstance(input_data, dict):
                     if input_data.get("kind") == "value":