Document context merging order for before hook ctx (#125)

Signed-off-by: Justin Abrahms <[email protected]> Co-authored-by: Todd Baert <[email protected]> Signed-off-by: Justin Abrahms <[email protected]>
open-feature · Aug 9, 2022 · 3891551 · 3891551
1 parent 0b272d5
commit 3891551
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/specification.json b/specification.json
@@ -400,7 +400,7 @@
         {
             "id": "Requirement 4.3.4",
             "machine_id": "requirement_4_3_4",
-            "content": "When `before` hooks have finished executing, any resulting `evaluation context` MUST be merged with the invocation `evaluation context` with the invocation `evaluation context` taking precedence in the case of any conflicts.",
+            "content": "When `before` hooks have finished executing, any resulting `evaluation context` MUST be merged with the existing `evaluation context` in the following order: before-hook (highest precedence), invocation, client, api (lowest precedence).",
             "RFC 2119 keyword": "MUST",
             "children": []
         },

diff --git a/specification/sections/04-hooks.md b/specification/sections/04-hooks.md
@@ -87,7 +87,7 @@ EvaluationContext | void before(HookContext, HookHints);
 
 #### Requirement 4.3.4
 
-> When `before` hooks have finished executing, any resulting `evaluation context` **MUST** be merged with the invocation `evaluation context` with the invocation `evaluation context` taking precedence in the case of any conflicts.
+> When `before` hooks have finished executing, any resulting `evaluation context` **MUST** be merged with the existing `evaluation context` in the following order: before-hook (highest precedence), invocation, client, api (lowest precedence).
 
 #### Requirement 4.3.5