From bcca8ca8c1d847dc5064c37ef8a790c415a451b4 Mon Sep 17 00:00:00 2001
From: Roy Nieterau <roy_nieterau@hotmail.com>
Date: Tue, 17 Jun 2025 22:41:31 +0200
Subject: [PATCH] Rename `post_*` method kwarg `container` to `result` to not
 clash with `container` argument on `update` and `remove` and make it clearer
 that it's the "result" of something

---
 client/ayon_core/pipeline/load/plugins.py | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/client/ayon_core/pipeline/load/plugins.py b/client/ayon_core/pipeline/load/plugins.py
index 7176bd82ac..0fc2718190 100644
--- a/client/ayon_core/pipeline/load/plugins.py
+++ b/client/ayon_core/pipeline/load/plugins.py
@@ -1,6 +1,7 @@
 from __future__ import annotations
 import os
 import logging
+from typing import Any
 from abc import abstractmethod
 
 from ayon_core.settings import get_project_settings
@@ -259,6 +260,9 @@ class LoaderHookPlugin:
     Should be used as non-invasive method to enrich core loading process.
     Any studio might want to modify loaded data before or after
     they are loaded without need to override existing core plugins.
+
+    The post methods are called after the loader's methods and receive the
+    return value of the loader's method as `result` argument.
     """
     order = 0
 
@@ -284,7 +288,7 @@ class LoaderHookPlugin:
         name: str | None = None,
         namespace: str | None = None,
         options: dict | None = None,
-        container: dict | None = None,  # (ayon:container-3.0)
+        result: Any = None,
     ):
         pass
 
@@ -301,6 +305,7 @@ class LoaderHookPlugin:
         self,
         container: dict,  # (ayon:container-3.0)
         context: dict,
+        result: Any = None,
     ):
         pass
 
@@ -315,6 +320,7 @@ class LoaderHookPlugin:
     def post_remove(
         self,
         container: dict,  # (ayon:container-3.0)
+        result: Any = None,
     ):
         pass
 
@@ -374,7 +380,7 @@ def add_hooks_to_loader(
 
             # Add result to kwargs if needed
             # Assuming container-like result for load, update, remove
-            kwargs["container"] = result
+            kwargs["result"] = result
 
             # Call post_<method_name> on all hooks
             post_hook_name = f"post_{method_name}"