Review item D from PR #153: RenderPageWithTimeout is implemented and tested in the ssr package, but BuildPhase2 exec mode calls ssr.RenderPage() (no timeout) instead. ssrCfg.Timeout is parsed from config but never used. Needs spec decision: should exec mode always enforce timeout? What is the default? Should builds hang indefinitely if an SSR command stalls?
Review item D from PR #153:
RenderPageWithTimeoutis implemented and tested in the ssr package, butBuildPhase2exec mode callsssr.RenderPage()(no timeout) instead.ssrCfg.Timeoutis parsed from config but never used. Needs spec decision: should exec mode always enforce timeout? What is the default? Should builds hang indefinitely if an SSR command stalls?