Fix HMR for action files during development#16932
Conversation
Merging this PR will improve performance by 37.93%
|
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ⚡ | Simulation | Rendering: streaming [false], .md file |
1.7 ms | 1.2 ms | +37.93% |
Tip
Curious why this is faster? Comment @codspeedbot explain why this is faster on this PR, or directly use the CodSpeed MCP with your agent.
Comparing fkatsuhiro:fix/actions-hmr-invalidate (3a5bc27) with main (dc45246)1
Footnotes
ematipico
left a comment
There was a problem hiding this comment.
The clearing of the actions is correct, but it shouldn't happen in prroduction
There was a problem hiding this comment.
This something that shouldn't happen in production. Clearing actions in production is incorrect.
There was a problem hiding this comment.
I understood. That's make sense. Let me update it. Thank you.
There was a problem hiding this comment.
I've updated this.
Moved clearActions() from Pipeline to NonRunnablePipeline and RunnablePipeline to ensure it isn't used in the production environment.
8f52610 to
052731d
Compare
dbbbc4e to
4cfc87d
Compare
4cfc87d to
3a5bc27
Compare

This PR fix ci error which cause type error from #16929
-------- Following content is cloned from #16929 --------
Changes
clearActions()method to dev-specific pipelines (RunnablePipeline and NonRunnablePipeline) to reset the cached resolved actions, matching the existingclearMiddleware()pattern while ensuring production paths remain completely untouched.Testing
Docs
Closes #16913