search

flynneva / react-ros

Connect your website to your robot!
https://flynneva.github.io/react-ros/
54 stars 11 forks source link

Build/Deprecation warnings #48

Open jaguardo opened 3 years ago

jaguardo commented 3 years ago

Started looking at all the Warning when building with react-ros:

seems there are currently 5 components: docz, roslib, react-scripts, microbundle-crl, uws: Docz:

warning docz > docz-core > wait-on > request@2.88.2: request has been deprecated, see https://github.com/request/request/issues/3142                                                                
warning docz > docz-core > wait-on > @hapi/joi@15.1.1: Switch to 'npm install joi'                                                                                                                  
warning docz > gatsby > @hapi/joi@15.1.1: Switch to 'npm install joi'                                                                                                                               
warning docz > gatsby > babel-eslint@10.1.0: babel-eslint is now @babel/eslint-parser. This package will no longer receive updates.                                                                
warning docz > gatsby > eslint-loader@2.2.1: This loader has been deprecated. Please use eslint-webpack-plugin                                                                                      
warning docz > gatsby > gatsby-cli > @hapi/joi@15.1.1: Switch to 'npm install joi'                                                                                                                  
warning docz > gatsby > uuid@3.4.0: Please upgrade  to version 7 or higher.  Older versions may use Math.random() in certain circumstances, which is known to be problematic.  See https://v8.dev/blog/math-random for details.                                                                                                                                                                         
warning docz > gatsby-theme-docz > gatsby-source-filesystem > better-queue > uuid@3.4.0: Please upgrade  to version 7 or higher.  Older versions may use Math.random() in certain circumstances, which is known to be problematic.  See https://v8.dev/blog/math-random for details.                                                                                                                    
warning docz > docz-core > wait-on > request > uuid@3.4.0: Please upgrade  to version 7 or higher.  Older versions may use Math.random() in certain circumstances, which is known to be problematic.  See https://v8.dev/blog/math-random for details.                                                                                                                                                  
warning docz > gatsby > gatsby-cli > uuid@3.4.0: Please upgrade  to version 7 or higher.  Older versions may use Math.random() in certain circumstances, which is known to be problematic.  See https://v8.dev/blog/math-random for details.                                                                                                                                                            
warning docz > gatsby > gatsby-telemetry > uuid@3.4.0: Please upgrade  to version 7 or higher.  Older versions may use Math.random() in certain circumstances, which is known to be problematic.  See https://v8.dev/blog/math-random for details.                                                                                                                                                      
warning docz > gatsby > webpack-dev-server > chokidar@2.1.8: Chokidar 2 will break on node v14+. Upgrade to chokidar 3 with 15x less dependencies.                                                  
warning docz > gatsby > webpack-dev-server > chokidar > fsevents@1.2.13: fsevents 1 will break on node v14+ and could be using insecure binaries. Upgrade to fsevents 2.                            
warning docz > docz-core > wait-on > request > har-validator@5.1.5: this library is no longer supported                                                                                             
warning docz > docz-core > wait-on > @hapi/joi > @hapi/address@2.1.4: Moved to 'npm install @sideway/address'                                                                                       
warning docz > docz-core > wait-on > @hapi/joi > @hapi/bourne@1.3.2: This version has been deprecated and is no longer supported or maintained                                                      
warning docz > docz-core > wait-on > @hapi/joi > @hapi/topo@3.1.6: This version has been deprecated and is no longer supported or maintained                                                        
warning docz > docz-core > wait-on > @hapi/joi > @hapi/hoek@8.5.1: This version has been deprecated and is no longer supported or maintained                                                        
warning docz > docz-core > wait-on > @hapi/joi > @hapi/topo > @hapi/hoek@8.5.1: This version has been deprecated and is no longer supported or maintained                                           
warning docz > gatsby > gatsby-cli > gatsby-recipes > @hapi/hoek@8.5.1: This version has been deprecated and is no longer supported or maintained                                                   
warning docz > gatsby > gatsby-cli > gatsby-recipes > @hapi/joi@15.1.1: Switch to 'npm install joi'                                                                                                 
warning docz > gatsby > gatsby-cli > gatsby-recipes > uuid@3.4.0: Please upgrade  to version 7 or higher.  Older versions may use Math.random() in certain circumstances, which is known to be problematic.  See https://v8.dev/blog/math-random for details.                                                                                                                                           
warning docz > gatsby > webpack-hot-middleware > querystring@0.2.1: The querystring API is considered Legacy. new code should use the URLSearchParams API instead.                                  
warning docz > gatsby-theme-docz > gatsby-plugin-mdx > static-site-generator-webpack-plugin > url > querystring@0.2.0: The querystring API is considered Legacy. new code should use the URLSearchParams API instead.                                                                                                                                                                                   
warning docz > gatsby > @pmmmwh/react-refresh-webpack-plugin > native-url > querystring@0.2.1: The querystring API is considered Legacy. new code should use the URLSearchParams API instead.       
warning docz > gatsby > webpack-dev-server > sockjs > uuid@3.4.0: Please upgrade  to version 7 or higher.  Older versions may use Math.random() in certain circumstances, which is known to be problematic.  See https://v8.dev/blog/math-random for details.                                                                                                                                           
warning docz > gatsby > webpack-dev-server > webpack-log > uuid@3.4.0: Please upgrade  to version 7 or higher.  Older versions may use Math.random() in certain circumstances, which is known to be problematic.  See https://v8.dev/blog/math-random for details.                                                                                                                                      
warning docz > gatsby-theme-docz > gatsby-plugin-mdx > remark > unified > @types/vfile > @types/vfile-message@2.0.0: This is a stub types definition. vfile-message provides its own type definitions, so you do not need this installed.                                                                                                                                                               
warning docz > gatsby > webpack > watchpack > watchpack-chokidar2 > chokidar@2.1.8: Chokidar 2 will break on node v14+. Upgrade to chokidar 3 with 15x less dependencies.                           
warning docz > gatsby-theme-docz > babel-plugin-export-metadata > @babel/cli > @nicolo-ribaudo/chokidar-2 > braces > snapdragon > source-map-resolve > resolve-url@0.2.1: https://github.com/lydell/resolve-url#deprecated                                                                                                                                                                              
warning docz > gatsby-theme-docz > babel-plugin-export-metadata > @babel/cli > @nicolo-ribaudo/chokidar-2 > braces > snapdragon > source-map-resolve > urix@0.1.0: Please see https://github.com/lydell/urix#deprecated

Which seems to jiv with that packages issues... (https://github.com/doczjs/docz/issues/1568) so, either start to PR changes to that repo, or find another package that does something similar? not familiar, but might not have a long life span...

roslib:

warning roslib > socket.io > debug@4.1.1: Debug versions >=3.2.0 <3.2.7 || >=4 <4.3.1 have a low-severity ReDos regression when used in a Node.js environment. It is recommended you upgrade to 3.2.7 or 4.3.1. (https://github.com/visionmedia/debug/issues/797)                                                                                                                                       
warning roslib > xmldom@0.1.31: Deprecated due to CVE-2021-21366 resolved in 0.5.0

socket.io (https://github.com/RobotWebTools/roslibjs/issues/322) looks like they depend on 4.0.0 which is what the warning says not to use... maybe a PR with them?

microbundle-crl > rollup-plugin-babel@4.4.0: This package has been deprecated and is no longer maintained. Please use @rollup/plugin-babel.

no idea

react-scripts

warning react-scripts > babel-eslint@10.1.0: babel-eslint is now @babel/eslint-parser. This package will no longer receive updates.                                                                 
warning react-scripts > eslint-loader@3.0.3: This loader has been deprecated. Please use eslint-webpack-plugin                                                                                      
warning react-scripts > fsevents@2.1.2: "Please update to latest v2.3 or v2.2"                                                                                                                      
warning react-scripts > webpack-dev-server > chokidar@2.1.8: Chokidar 2 will break on node v14+. Upgrade to chokidar 3 with 15x less dependencies.                                                  
warning react-scripts > webpack-dev-server > sockjs > uuid@3.4.0: Please upgrade  to version 7 or higher.  Older versions may use Math.random() in certain circumstances, which is known to be problematic.  See https://v8.dev/blog/math-random for details.                                                                                                                                           
warning react-scripts > html-webpack-plugin@4.0.0-beta.11: please switch to a stable version                                                                                                        
warning react-scripts > jest-environment-jsdom-fourteen > jsdom > request@2.88.2: request has been deprecated, see https://github.com/request/request/issues/3142                                   
warning react-scripts > workbox-webpack-plugin > workbox-build > @hapi/joi@15.1.1: Switch to 'npm install joi'                                                                                      
warning react-scripts > babel-jest > @jest/transform > jest-haste-map > fsevents@1.2.13: fsevents 1 will break on node v14+ and could be using insecure binaries. Upgrade to fsevents 2.            
warning react-scripts > jest-environment-jsdom-fourteen > jsdom > request-promise-native@1.0.9: request-promise-native has been deprecated because it extends the now deprecated request package, see https://github.com/request/request/issues/3142                                                                                                                                                    
warning react-scripts > resolve-url-loader > rework > css > urix@0.1.0: Please see https://github.com/lydell/urix#deprecated                                                                        
warning react-scripts > babel-jest > @jest/transform > jest-haste-map > sane@4.1.0: some dependency vulnerabilities fixed, support for node < 10 dropped, and newer ECMAScript syntax/features addedwarning react-scripts > jest > jest-cli > jest-config > jest-environment-jsdom > jsdom > request@2.88.2: request has been deprecated, see https://github.com/request/request/issues/3142            
warning react-scripts > jest > jest-cli > jest-config > jest-environment-jsdom > jsdom > request-promise-native@1.0.9: request-promise-native has been deprecated because it extends the now deprecated request package, see https://github.com/request/request/issues/3142                                                                                                                             
warning react-scripts > jest > jest-cli > jest-config > jest-environment-jsdom > jsdom > left-pad@1.3.0: use String.prototype.padStart()                                                            
warning react-scripts > workbox-webpack-plugin > workbox-build > strip-comments > babel-plugin-transform-object-rest-spread > babel-runtime > core-js@2.6.12: core-js@<3.3 is no longer maintained and not recommended for usage due to the number of issues. Because of the V8 engine whims, feature detection in old core-js versions could cause a slowdown up to 100x even if nothing is polyfilled. Please, upgrade your dependencies to the actual version of core-js.

might just have to accept these, as there are about 1000 issues (https://github.com/facebook/create-react-app/issues) and this one

warning uws@10.148.1: New code is available at github.com/uNetworking/uWebSockets.js

looks like they are up to version 19ish now (https://github.com/uNetworking/uWebSockets.js)

this is more just to document them and see if there are good alternatives or ways to address them

jaguardo commented 3 years ago

Docz might be back on track soon:? https://github.com/doczjs/docz/issues/1634 however, is it a package that needs to be in the release?

jaguardo commented 3 years ago

looks like 

"uws": "^10.148.2"

in package.json should work

flynneva commented 3 years ago

i should hve some time next week to work on updating the react-ros package. good job at tracking this though! if you find anything else that needs updating let me know and ill add it to the list for next week

jaguardo commented 3 years ago

will do... not sure how much you have looked at performance (google Chrome f12), but I've never really looked too hard at "dropped frames" until now. I think it is the way the messages are handled, I'm trying to minimize the demand, but I still get dropped frames every now and then... a little judder here and there. I have a feeling it is within roslibjs and the way the listener.subscribe is created. Interested if other folks have dived in that far, or "if it works, it works!" which is about where I am at the moment!

flynneva commented 3 years ago

keep an eye at what Foxglove studios are doing. they are doing some really good work IMO with regards to web visualization stuff. https://github.com/foxglove

flynneva commented 3 years ago

what im trying to say is if you are seeing a problem most likely they are too. ill bet money they are either working on a fix or coming up with a new way around the problem

jaguardo commented 3 years ago

oh nice... I'll give them a look!

flynneva commented 3 years ago

@jaguardo I just pushed an update to fix some of these warnings. a lot of them are sort of out of my hands though....so if you'd like to keep pushing for updates on other repos feel free 😄 I respect a lot the work you are doing tbh, it takes someone very determined to keep pushing for these sorts of updates on all these repos! 😅

flynneva commented 3 years ago

going to leave this open to track all these updates. docz build is failing due to some missing deps i think.

Can't resolve 'uws' in '/home/flynn/code/web/ros/react-ros/node_modules/engine.io/lib'