On Tue, Oct 13, 2026 at 6:37 AM Shubhankar Anand Kulkarni < [email protected]> wrote:
> Since this touches both the extension-state API and > eager aggregation, are you OK with resetting the fields at each copy > site, or would you rather have a shared helper that does the flat copy > and the resets in one place? > I'm okay with the per-site resets for this fix, alongside the existing duplicated path resets. > Also can we turn the reproducer into a src/test/modules test if you'd like > regression coverage for it? > Yes, I believe regression coverage under src/test/modules would be useful. I'll hold off on updating the status until I've reviewed the revised patch and run the new test. Thanks, Sho Ito -- ＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝ *Sho Ito (Sean)* E-mail: [email protected] LinkedIn: *https://www.linkedin.com/in/sho-ito-0628/ <https://www.linkedin.com/in/sho-ito-0628/>* GitHub: https://github.com/sean0628/ ＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝
