dengliming opened a new issue, #634:
URL: https://github.com/apache/shenyu-dashboard/issues/634

   ## Description
   `src/routes/System/Instance/index.js` imports `format` from `date-fns`, but 
`date-fns` is not listed in `dependencies`. It only resolves because a dev 
transitive dependency hoists `[email protected]` into `node_modules`. The format 
tokens used (`YYYY`, `DD`) are also v1-specific; under date-fns v2+ they throw 
a `RangeError`.
   
   ## Location
   (Lines refer to `master @ 83969a5`.)
   - `src/routes/System/Instance/index.js:32,404,420`
   - `package.json` (missing dependency)
   
   ## Impact
   Any change in the dev-dependency tree can remove or upgrade `date-fns` and 
break the Instance page at build or run time.
   
   ## Suggested fix
   Use the already-declared `dayjs`/`moment` for formatting, or add `date-fns` 
to `dependencies` with the correct version and tokens.
   
   ## Related existing
   None
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to