chore(server): remove dead runner code

[emilyrohrbough]

chore: remove dead code -- lives in data-context/src/sources/HtmlDataSource.ts

[cypress-bot]

Thanks for taking the time to open a PR!

• Create a Draft Pull Request if your PR is not ready for review. Mark the PR as Ready for Review when you're ready for a Cypress team member to review the PR.
• Become familiar with the Code Review Checklist for guidelines on coding standards and what needs to be done before a PR can be merged.

[lmiller1990]

Good to delete dead code - definitely need to

1. get server properly type safe
2. figure out what exactly goes in data-context and server, and how the seam between the two should be decided

[lmiller1990]

Did not realize this was unused... I wonder if this should live in data-context? Server seems like a better place for, well, serving things. The line between server/data-context is really blurred, though. I think the goal was incrementally moving things so we could strongly type them, but a lot of things ended up just getting dumped there (probably since it has more strict types, so it's easier to work in).

[emilyrohrbough]

yeah the line between the data-context & server is very blurred. I think intention was to separate server vs config/setup type concepts but we didn't get all the way there how intertwined some of this is

[lmiller1990]

Going to go ahead and merge this up -- it's dead code, all CI is passing, I don't see much risk and there's not any additional manual testing required.

As an aside, we definitely need a process for getting more :eyes on these easy to review but high value, spontaneous PRs.