Reset typed accessor cache with MemberFilter changes

xoofx · Copilot · xoofx · commit 8180fb6cd1f6 · 2026-03-21T07:19:24.000+01:00
Co-authored-by: Copilot &lt;223556219+Copilot@users.noreply.github.com&gt;
diff --git a/src/Scriban.Tests/TestRuntime.cs b/src/Scriban.Tests/TestRuntime.cs
@@ -1071,6 +1071,37 @@ public void TestScriptObjectAccessor()
                 Assert.False(accessor.HasMember(context, new SourceSpan(), obj, nameof(MyStaticObject.StaticPropertyA)));
                 Assert.False(accessor.HasMember(context, new SourceSpan(), obj, nameof(MyStaticObject.StaticPropertyB)));
             }
+
+            // Test Reset clears cached typed accessors that captured the previous MemberFilter
+            {
+                var context = new TemplateContext
+                {
+                    EnableRelaxedMemberAccess = false,
+                    MemberFilter = _ => true
+                };
+                var obj = new MyObject { PropertyA = "allowed", PropertyB = "blocked" };
+                var template = Template.Parse("{{ model.property_b }}");
+                var accessor = context.GetMemberAccessor(obj);
+                var globals = new ScriptObject();
+                globals["model"] = obj;
+                context.PushGlobal(globals);
+
+                Assert.True(accessor.HasMember(context, new SourceSpan(), obj, "property_b"));
+                Assert.AreEqual("blocked", template.Render(context));
+
+                context.Reset();
+                context.MemberFilter = member => member.Name == nameof(MyObject.PropertyA);
+
+                accessor = context.GetMemberAccessor(obj);
+                Assert.False(accessor.HasMember(context, new SourceSpan(), obj, "property_b"));
+
+                globals = new ScriptObject();
+                globals["model"] = obj;
+                context.PushGlobal(globals);
+
+                var exception = Assert.Throws<ScriptRuntimeException>(() => template.Render(context));
+                StringAssert.Contains("Cannot get member", exception!.Message);
+            }
         }
 
         [Test]
diff --git a/src/Scriban/TemplateContext.cs b/src/Scriban/TemplateContext.cs
@@ -901,6 +901,7 @@ public virtual void Reset()
             }
 
             CachedTemplates.Clear();
+            _memberAccessors.Clear();
         }
 
         /// <summary>

Original file line number	Diff line number	Diff line change
`@@ -901,6 +901,7 @@ public virtual void Reset()`
`901`	`901`	`}`
`902`	`902`
`903`	`903`	`CachedTemplates.Clear();`
	`904`	`+ _memberAccessors.Clear();`
`904`	`905`	`}`
`905`	`906`
`906`	`907`	`/// <summary>`