[NetworkDeployer] Node Mangling to avoid duplication (#93)

* [NetworkDeployer] Mangle these n...odes

* [DeeployTypes] Only mangle on duplicates

* Update changelog

---------

Co-authored-by: viv-eth <[email protected]>

Author: viv-eth

CHANGELOG.md

@@ -3,16 +3,19 @@ This file contains the changelog for the Deeploy project. The changelog is divid
 
 ## Unreleased (Planned Release Target: v0.2.1)
 ### List of Pull Requests
+- Node Mangling to avoid duplication [#93](https://github.com/pulp-platform/Deeploy/pull/93)
 - Prepare Post v0.2.0 Release [#104](https://github.com/pulp-platform/Deeploy/pull/104)
 - Use Docker digests instead of arch-specific tags [#106](https://github.com/pulp-platform/Deeploy/pull/106)
 
 ### Added
+- Added `_mangleNodeNames` function to avoid duplicate node mappings
 - Output Docker image digests per platform (`amd64`, `arm64`) after build, which is used to construct the multi-arch Docker manifest. This preventes registry clutter caused by unnecessary per-architecture Docker tags.
 
 ### Changed
 - Replaced platform-specific tags (`*-amd64`, `*-arm64`) with direct digest references in `Noelware/docker-manifest-action`.
 
 ### Fixed
+- Prevent node duplication for graphs generated via GraphSurgeon
 - Resolved issue with missing `id` in the `Build Cache for Docker` step, used in the `Inject build-cache` step.
 
 ### Removed

Deeploy/DeeployTypes.py

@@ -3303,6 +3303,24 @@ def _mangleTensorNames(self):
         for tensor in self.graph.tensors().values():
             tensor.name = f"{tensor.name}_tensor"
 
+    # Don't override this
+    def _mangleNodeNames(self):
+        """Mangle node names only if duplicates exist. Unique names are preserved."""
+        # Count occurrences of each original name
+        counts: Dict[str, int] = {}
+        for node in self.graph.nodes:
+            counts[node.name] = counts.get(node.name, 0) + 1
+
+        # For any name that appears more than once, append a counter suffix
+        seen: Dict[str, int] = {}
+        for node in self.graph.nodes:
+            orig = node.name
+            if counts[orig] > 1:
+                idx = seen.get(orig, 0)
+                node.name = f"{orig}_{idx}"
+                seen[orig] = idx + 1
+            # else: unique name, leave it unchanged
+
     # Don't override this
     def _removeIdentityNodes(self):
         for node in filter(lambda x: x.op == "Identity", self.graph.nodes):
@@ -3316,6 +3334,8 @@ def frontEnd(self):
 
         self._mangleTensorNames()
 
+        self._mangleNodeNames()
+
         # Rename graph inputs and outputs:
         for idx, inputNode in enumerate(self.graph.inputs):
             inputNode.name = "input_" + str(idx)