Skip to content

refactor(render): centralize execution ownership#2158

Open
jrusso1020 wants to merge 1 commit into
07-10-refactor_core_unify_composition_contractfrom
07-10-refactor_render_add_execution_context
Open

refactor(render): centralize execution ownership#2158
jrusso1020 wants to merge 1 commit into
07-10-refactor_core_unify_composition_contractfrom
07-10-refactor_render_add_execution_context

Conversation

@jrusso1020

@jrusso1020 jrusso1020 commented Jul 10, 2026

Copy link
Copy Markdown
Collaborator

What

Centralize render-scoped cancellation, logging, event publication, and cleanup ownership.

Why

Independent cleanup paths and global logging made failure ordering and resource disposal difficult to reason about.

How

Introduce RenderExecutionContext with immutable request state, disposer stack, scoped logger, ordered publisher, and one outer cleanup path.

Test plan

  • render execution context and cleanup tests; producer orchestrator tests
  • Stack-wide lint, format, build, typecheck, and relevant integration gates

jrusso1020 commented Jul 10, 2026

Copy link
Copy Markdown
Collaborator Author

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_core_unify_composition_contract branch 2 times, most recently from 1030990 to bb1d745 Compare July 11, 2026 08:17
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_render_add_execution_context branch from 333782f to b108d4f Compare July 11, 2026 08:17
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_core_unify_composition_contract branch from bb1d745 to b5df3de Compare July 11, 2026 08:21
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_render_add_execution_context branch from b108d4f to 96c989c Compare July 11, 2026 08:21
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_core_unify_composition_contract branch from b5df3de to eebd587 Compare July 11, 2026 08:53
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_render_add_execution_context branch from 96c989c to 99bec1e Compare July 11, 2026 08:53
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_core_unify_composition_contract branch from eebd587 to ab5e15b Compare July 11, 2026 17:05
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_render_add_execution_context branch from 99bec1e to 1878b29 Compare July 11, 2026 17:05
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_core_unify_composition_contract branch from ab5e15b to e873a3a Compare July 11, 2026 17:30
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_render_add_execution_context branch 2 times, most recently from efd1c86 to b75de2d Compare July 11, 2026 17:38
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_core_unify_composition_contract branch from 095fba2 to ece039a Compare July 11, 2026 18:04
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_render_add_execution_context branch 2 times, most recently from 98358c3 to cf33ce4 Compare July 11, 2026 18:11
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_core_unify_composition_contract branch from 63b042e to 9793d33 Compare July 11, 2026 18:16
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_render_add_execution_context branch from cf33ce4 to 5686104 Compare July 11, 2026 18:16
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_core_unify_composition_contract branch from 9793d33 to c4086c0 Compare July 11, 2026 18:21
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_render_add_execution_context branch 2 times, most recently from 04cdfef to e2b7c9d Compare July 12, 2026 04:01
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_core_unify_composition_contract branch 2 times, most recently from 8ed5d2d to a659ef4 Compare July 12, 2026 04:32
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_render_add_execution_context branch 2 times, most recently from 8c537d2 to 77f7246 Compare July 12, 2026 04:58
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_core_unify_composition_contract branch from a659ef4 to 5f053c9 Compare July 12, 2026 04:58
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_render_add_execution_context branch from 77f7246 to c65c003 Compare July 12, 2026 08:54
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_core_unify_composition_contract branch from 5f053c9 to bb0d3c2 Compare July 12, 2026 08:54
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_render_add_execution_context branch from c65c003 to d6188b9 Compare July 12, 2026 09:11
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_core_unify_composition_contract branch from bb0d3c2 to d961f3a Compare July 12, 2026 09:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant