fix(core): harden routing reliability and add quiet-hours controls - #375
21 fail, 1 pass in 0s
Annotations
Check warning on line 0 in eslint
github-actions / 🔒 Security: Scan Summary
All 11 runs failed: no-duplicate-imports (eslint)
combined-reports/eslint-security.xml
Raw output
'@prisma/client' import is duplicated.
Check warning on line 0 in eslint
github-actions / 🔒 Security: Scan Summary
All 243 runs failed: @typescript-eslint/no-unused-vars (eslint)
combined-reports/eslint-security.xml
Raw output
'NotificationChannel' is defined but never used. Allowed unused vars must match /^_/u.
Check warning on line 0 in eslint
github-actions / 🔒 Security: Scan Summary
All 236 runs failed: security/detect-object-injection (eslint)
combined-reports/eslint-security.xml
Raw output
Generic Object Injection Sink
Check warning on line 0 in eslint
github-actions / 🔒 Security: Scan Summary
All 834 runs failed: @typescript-eslint/no-explicit-any (eslint)
combined-reports/eslint-security.xml
Raw output
Unexpected any. Specify a different type.
Check warning on line 0 in eslint
github-actions / 🔒 Security: Scan Summary
All 38 runs failed: react/no-unescaped-entities (eslint)
combined-reports/eslint-security.xml
Raw output
`'` can be escaped with `'`, `‘`, `'`, `’`.
Check warning on line 0 in eslint
github-actions / 🔒 Security: Scan Summary
All 31 runs failed: no-alert (eslint)
combined-reports/eslint-security.xml
Raw output
Unexpected alert.
Check warning on line 0 in eslint
github-actions / 🔒 Security: Scan Summary
All 7 runs failed: @next/next/no-location-assign-relative-destination (eslint)
combined-reports/eslint-security.xml
Raw output
Do not use `window.location.href` to navigate to internal Next.js pages. Use `redirect()` in the render phase, or `useRouter().push()` in Client Components' event handlers instead. See: https://nextjs.org/docs/messages/no-location-assign-relative-destination
Check warning on line 0 in eslint
github-actions / 🔒 Security: Scan Summary
All 13 runs failed: @next/next/no-img-element (eslint)
combined-reports/eslint-security.xml
Raw output
Using `<img>` could result in slower LCP and higher bandwidth. Consider using `<Image />` from `next/image` or a custom image loader to automatically optimize images. This may incur additional usage or cost from your provider. See: https://nextjs.org/docs/messages/no-img-element
Check warning on line 0 in eslint
github-actions / 🔒 Security: Scan Summary
All 53 runs failed: react-hooks/set-state-in-effect (eslint)
combined-reports/eslint-security.xml
Raw output
Error: Calling setState synchronously within an effect can trigger cascading renders Effects are intended to synchronize state between React and external systems such as manually updating the DOM, state management libraries, or other platform APIs. In general, the body of an effect should do one or both of the following: * Update external systems with the latest state from React. * Subscribe for updates from some external system, calling setState in a callback function when external state changes. Calling setState synchronously within an effect body causes cascading renders that can hurt performance, and is not recommended. (https://react.dev/learn/you-might-not-need-an-effect). /home/runner/work/OpsKnight/OpsKnight/src/app/(public)/m/login/MobileLoginClient.tsx:107:20 105 | 106 | useEffect(() => { > 107 | if (errorCode) setError(formatError(errorCode)); | ^^^^^^^^ Avoid calling setState() directly within an effect 108 | }, [errorCode]); 109 | 110 | useEffect(() => {
Check warning on line 0 in eslint
github-actions / 🔒 Security: Scan Summary
All 2 runs failed: security/detect-possible-timing-attacks (eslint)
combined-reports/eslint-security.xml
Raw output
Potential timing attack, left side: true
Check warning on line 0 in eslint
github-actions / 🔒 Security: Scan Summary
prefer-const (eslint) failed
combined-reports/eslint-security.xml
Raw output
'handleSecondKey' is never reassigned. Use 'const' instead.
Check warning on line 0 in eslint
github-actions / 🔒 Security: Scan Summary
All 6 runs failed: react-hooks/exhaustive-deps (eslint)
combined-reports/eslint-security.xml
Raw output
React Hook useEffect has missing dependencies: 'isMobileMenuOpen' and 'setIsMobileMenuOpen'. Either include them or remove the dependency array.
Check warning on line 0 in eslint
github-actions / 🔒 Security: Scan Summary
security/detect-non-literal-regexp (eslint) failed
combined-reports/eslint-security.xml
Raw output
Found non-literal argument to RegExp Constructor
Check warning on line 0 in eslint
github-actions / 🔒 Security: Scan Summary
All 2 runs failed: @next/next/no-html-link-for-pages (eslint)
combined-reports/eslint-security.xml
Raw output
Do not use an `<a>` element to navigate to `/incidents/`. Use `<Link />` from `next/link` instead. See: https://nextjs.org/docs/messages/no-html-link-for-pages
Check warning on line 0 in eslint
github-actions / 🔒 Security: Scan Summary
react-hooks/incompatible-library (eslint) failed
combined-reports/eslint-security.xml
Raw output
Compilation Skipped: Use of incompatible library This API returns functions which cannot be memoized without leading to stale UI. To prevent this, by default React Compiler will skip memoizing this component/hook. However, you may see issues if values from this API are passed to other components/hooks that are memoized. /home/runner/work/OpsKnight/OpsKnight/src/components/settings/forms/AutosaveForm.tsx:44:23 42 | 43 | const { watch, formState } = form; > 44 | const watchedData = watch(); | ^^^^^ React Hook Form's `useForm()` API returns a `watch()` function which cannot be memoized safely. 45 | 46 | const { status, error, isSaving } = useAutosave({ 47 | data: watchedData,
Check warning on line 0 in eslint
github-actions / 🔒 Security: Scan Summary
jsx-a11y/role-supports-aria-props (eslint) failed
combined-reports/eslint-security.xml
Raw output
The attribute aria-invalid is not supported by the role radio. This role is implicit on the element input.
Check warning on line 0 in eslint
github-actions / 🔒 Security: Scan Summary
All 4 runs failed: security/detect-unsafe-regex (eslint)
combined-reports/eslint-security.xml
Raw output
Unsafe Regular Expression
Check warning on line 0 in eslint
github-actions / 🔒 Security: Scan Summary
no-throw-literal (eslint) failed
combined-reports/eslint-security.xml
Raw output
Expected an error object to be thrown.
Check warning on line 0 in eslint
github-actions / 🔒 Security: Scan Summary
@typescript-eslint/no-unused-expressions (eslint) failed
combined-reports/eslint-security.xml
Raw output
Expected an assignment or function call and instead saw an expression.
Check warning on line 0 in eslint
github-actions / 🔒 Security: Scan Summary
All 3 runs failed: @typescript-eslint/no-require-imports (eslint)
combined-reports/eslint-security.xml
Raw output
A `require()` style import is forbidden.
Check warning on line 0 in eslint
github-actions / 🔒 Security: Scan Summary
security/detect-non-literal-fs-filename (eslint) failed
combined-reports/eslint-security.xml
Raw output
Found readFileSync from package "node:fs" with non literal argument at index 0