eclipse-langium / langium

Next-gen language engineering / DSL framework
https://langium.org/
MIT License
665 stars 61 forks source link

langium-cli: improved error reporting in case of langium config loading failures #1377

Closed sailingKieler closed 4 months ago

sailingKieler commented 4 months ago

Just ran into an erroneous invocation of the langium cli giving me error messages like

Reading config from package.json
Failed to read config file. TypeError: Cannot set properties of undefined (setting 'Symbol(RelativePath)')
    at loadConfig (file://<projectPath>/node_modules/langium-cli/lib/package.js:34:30)
    at async generate (file://<projectPath>/node_modules/langium-cli/lib/generate.js:27:20)

This change improves the error reporting to

Reading config from <projectPath>/package.json
Failed to read Langium config: Didn't find <projectPath>/langium-config.json or an object named 'langium' in <projectPath>package.json.
 Error: Langium config is missing.
    at loadConfig (file://<projectPath>/node_modules/langium-cli/lib/package.js:39:19)
    at async generate (file://<projectPath>/node_modules/langium-cli/lib/generate.js:27:20)