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_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
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_render_add_execution_context branch 3 times, most recently from d9d5892 to 82a2f69 Compare July 13, 2026 18:09
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_core_unify_composition_contract branch from 70a22dc to 5f10ce8 Compare July 13, 2026 18:09
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_render_add_execution_context branch from 82a2f69 to ebdc7d4 Compare July 13, 2026 18:35
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_core_unify_composition_contract branch 2 times, most recently from 9160e18 to d266322 Compare July 13, 2026 20:27
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_render_add_execution_context branch from ebdc7d4 to b53c113 Compare July 13, 2026 20:27
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_core_unify_composition_contract branch from d266322 to e1a0e21 Compare July 13, 2026 20:33
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_render_add_execution_context branch 4 times, most recently from a9e5a31 to 8233fbc Compare July 13, 2026 23:08
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_core_unify_composition_contract branch 2 times, most recently from 43cf344 to b4a3e92 Compare July 14, 2026 00:03
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_render_add_execution_context branch from 8233fbc to 164a96f Compare July 14, 2026 00:03
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_core_unify_composition_contract branch from b4a3e92 to feb7e4c Compare July 14, 2026 03:20
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_render_add_execution_context branch from 164a96f to da86e33 Compare July 14, 2026 03:20
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