BUG : Source map for react and typescript

[Ciberdead]

When running blockly on a typescript and react app their is 17 warning triggered by source map : i've disabled it by using the option sourceMap : false on tsconfig.json

[Maribeth Bottorff]

Hey there, unfortunately this comes from one of our dependencies (closure compiler) and we can't avoid it for now. You can see if your build tool has the option to ignore warnings. e.g. we use webpack and disable the warnings like this.

Maribeth

[Christopher Allen]

Hello Ciberdead & Maribeth,

On Thursday, May 4, 2023 at 12:26:22 AM UTC-7 Ciberdead wrote:

When running blockly on a typescript and react app their is 17 warning triggered by source map : i've disabled it by using the option sourceMap : false on tsconfig.json

On Thu, 4 May 2023 at 22:54, 'Maribeth Bottorff' via Blockly <blo...@googlegroups.com> wrote:

Hey there, unfortunately this comes from one of our dependencies (closure compiler) and we can't avoid it for now. You can see if your build tool has the option to ignore warnings. e.g. we use webpack and disable the warnings like this.

Since I think I have most recently been poking at our sourcemap generation code, and was not aware of this issue, I'd be interested in more information about the specific warnings that are being generated.  I'm not exactly sure why there would be errors in the sourcemaps or (as suggested by the comments in the code Maribeth references) missing sourcemaps.

Christopher

[Ciberdead]

Hi Christopher, here are the errors that are printed when compiling my app : 
https://pastecode.io/s/x71te6wj

[Ciberdead]

Are their some news ? Is it from my side ?

[Maribeth Bottorff]

Hello, unfortunately we don't have an update on this. I would recommend just silencing these errors as in my first message until we can remove the dependency creating these missing sourcemaps.

Maribeth

[Ciberdead]

I fount that the errors came when i'm start importing the inject function from blockly.

[Christopher Allen]

Hello Ciberdead,

 

I've filed issue #7124 about this.  I don't know that we will be able to fix it soon, but I did want to make sure we didn't forget about it.

We don't currently have a minimal reproduction.  If you can give us steps to reproduce (ideally with as little extraneous code/config as possible) that would be helpful—please feel free to comment on the issue directly.

Christopher