Skip to content

remove hidden window abstraction#8918

Draft
jackkav wants to merge 6 commits intoKong:developfrom
jackkav:spike/remove-hidden-window
Draft

remove hidden window abstraction#8918
jackkav wants to merge 6 commits intoKong:developfrom
jackkav:spike/remove-hidden-window

Conversation

@jackkav
Copy link
Contributor

@jackkav jackkav commented Jul 29, 2025

follow up from #7790

we used hidden window as a stop gap so we could move fast but it causes possible execution difference with inso, so this change will align implementations

todo:

  • remove forks
  • remove unused code
  • remove unused config
  • decide what to do about concurrency
  • decide whether to run in utility process or elsewhere
  • decide what to do about waiting until promises have settled
  • environment caching issue that causes set env in pre req script to re-render without context

notes:

concurrency doesn't appear to have a test
runs in main now, but working on running elsewhere

whats next:

  • keep trying the utility process approach
  • we should document the reasons for change and results of the investigation before moving this forward.

@jackkav jackkav changed the title removes hidden window abstraction remove hidden window abstraction Jul 30, 2025
@jackkav jackkav marked this pull request as draft July 30, 2025 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