Move test case recorder into Cursorless engine #1269
pokey
started this conversation in
Architectural decision record
Replies: 0 comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Context
As we move to a language server (#946), we need to decide which components will live server side, and which will live in the VSCode extension. The test case recorder presents a challenge, because it references several VSCode apis, but also references many core pieces of our Cursorless engine, such as targets. The test case recorder is also arguably useful beyond Cursorless as a general-purpose VSCode test case recorder, but it is also useful beyond VSCode as a general-purpose Cursorless test case recorder.
Decision
We have decided to have the test case recorder live in the server, rather than in the extension. Whenever we initiate test case recording, VSCode will just send a command to the Cursorless LSP to initiate recording.
Consequences
that
mark, nextthat
mark, hat locations, etc. To capture such state with the recorder living client side would require exposing more internals to the client than we would likeSpyIDE
, which currently is designed to live server sideBeta Was this translation helpful? Give feedback.
All reactions