Azure / Azure/monaco-kusto

@kusto/language service includes `require("fs")` calls

Open
#388 2 comments 0 reactions 0 assignees View on GitHub
Dominant language
TypeScript
Stars
84
Forks
36
PR merge metrics
No merged PRs in 30d

Description

@kusto/language service includes `require("fs")` calls which causes bundlers to complain, requiring additional configuration to ignore these calls.

Webpack config looks like this:
```js
export default {
resolve: {
fallback: {
fs: false,
},
}
};
```
https://github.com/Azure/monaco-kusto/blob/134ff03184264a9900737e4b5cf93f69540e1c8e/samples/esm-webpack-react/webpack.config.js#L18

Parcel config looks like this:
```json
{
"alias": {
"process": false
}
}
```
https://github.com/Azure/monaco-kusto/blob/134ff03184264a9900737e4b5cf93f69540e1c8e/samples/parcel/package.json#L13

This is especially problematic for Create React App users because it doesn't expose this configuration.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by comparing the referenced samples/esm-webpack-react/webpack.config.js and samples/parcel/package.json with how @kusto/language is bundled. Trace where the require("fs") calls enter the published service, then verify that common bundlers no longer require extra fs configuration, including the Create React App case.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript, webpack
Domain
build-system
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.