TheAngryByrd / TheAngryByrd/MrBuild
Prove editor diagnostics and hover
Nobody has claimed this yet.
- Dominant language
- F#
- Stars
- 1
- Forks
- 0
- PR merge metrics
- No merged PRs in 30d
Description
Parent
#10 — Add cross-platform VS Code editor adapter for MrBuild LSP proof
What to build
Use the shared fixture and real Coordinator session to prove that standard editor diagnostics and hover results reach the VS Code surface through the Language Workers.
Acceptance criteria
- Opening the fixture displays the intentional diagnostic in the editor diagnostics UI.
- The diagnostic is produced by the real LSP/worker path and not by adapter-local logic.
- Hovering a known F# symbol returns a visible hover result.
- Hovering a known C# symbol returns a visible hover result.
- The proof tolerates normal editor document-open and synchronization timing without fixed sleeps as its only readiness mechanism.
- The acceptance test fails when the Coordinator or relevant worker is not serving the request.
Blocked by
- #13 — Add the shared fixture and isolated VS Code proof launcher
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the shared fixture and isolated VS Code proof launcher from #13, then trace the Coordinator and Language Workers used by the acceptance test. Verify diagnostics and hover results through the real editor surface, using readiness based on document synchronization rather than fixed sleeps. Done means the test observes the intentional diagnostic and visible F# and C# hover results, and fails when the Coordinator or worker is unavailable.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- fsharp, vscode
- Domain
- developer-experience, testing
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 48/100