feat: show error if library cannot be connected

[tilmx]

If the user tries to connect a library folder that does not work, we should give him a warning about that.

should be squashed into feat: show error if library cannot be connected

[marionebl]

Let's avoid the let binding by defining a getAnalysis function that returns a AnalysisResult:

type AnalysisResult =
  | AnalysisError
  | AnalysisSuccess;

enum AnalysisState {
   Error,
   Success
}

interface AnalysisError {
   state: AnalysisState.Error;
   error: Error;
}

interface AnalysisSuccess {
   state: AnalysisState.Success;
   analysis: Analysis;
}

const getAnalysis = async (path: string; context: AnalysisContext): Promise<AnalysisResult> => {
  try {
    return { state: AnalysisState.Sucess, analysis: Analyzer.analyze(path, context) };
  } catch (error) {
    return { state: AnalysisState.Error, error };
  }
};

This can be used like this then:

const analysisResult = await getAnalysis(/* omitted */);


if (analysisResult.state === AnalysisState.Success) {
   // send 
}

if (analysisResult.state === AnalysisState.Error) {
   // show dialog
}

[tilmx]

@marionebl I changed it to try both the analysis and the Sender.Send to avoid any let: analysis; thingies.