VSCode: Refactor cairols.ts to use Context and levelled logging

[mkaput]

Stack:

• VSCode: Move isScarbProject logic to separate file #5015
• VSCode: Slightly simplify LS selection logic #5014
• VSCode: Extract standalone CairoLS logic to StandaloneLS class #5013
• VSCode: Move Scarb searching logic to the Scarb class #5012
• VSCode: Add several handy onEnterRules #5011
• VSCode: Encapsulate all configuration magic into Config class #4976
• VSCode: Move away and simplify tool finding functions #4975
• VSCode: Add Scarb class and move Scarb's LS discovery there #4974
• VSCode: Do not spawn LS process manually and use VSC idioms instead #4973
• VSCode: Refactor cairols.ts to use Context and levelled logging #4959 ⬅
• VSCode: Use log output channel #4935
• VSCode: Create Context class #4934

⚠️ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

---

This change is 

[mkaput]

Reviewable status: 0 of 2 files reviewed, 2 unresolved discussions (waiting on @Arcticae and @orizi)

---

vscode-cairo/src/cairols.ts line 31 at r1 (raw file):

Previously, Arcticae (Tomasz Rejowski) wrote…

Isn't this more like a context function since it uses it?

It kinda is, but I would rather not touch this code in this PR. In #5013 this function is removed entirely.

---

vscode-cairo/src/cairols.ts line 146 at r1 (raw file):

Previously, Arcticae (Tomasz Rejowski) wrote…

Does this show up in 2 different places?

Yes, the warning is an alert in UI, while ctx.log.error outputs a line in Output > Cairo Extension panel.

[Arcticae]

Reviewed 2 of 2 files at r1, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @orizi)

[mkaput]

Closing pull request: commit has gone away

[orizi]

Reviewed all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @mkaput)