Skip to content

Commit fccf1b7

Browse files
david-crespoclaude
andauthored
Upgrade oxlint and oxfmt (#3148)
Did this from the mobile app, which is fun. I saw https://oxc.rs/blog/2026-03-11-oxlint-js-plugins-alpha.html and was wondering if the React Hook Form linter plugin works now. The answer is 2/3 of the way. One rule still has to be disabled. --------- Co-authored-by: Claude <noreply@anthropic.com>
1 parent 5768e1e commit fccf1b7

5 files changed

Lines changed: 230 additions & 196 deletions

File tree

.oxlintrc.json

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,11 @@
44
{
55
"name": "playwright",
66
"specifier": "eslint-plugin-playwright"
7+
},
8+
{
9+
"name": "react-hook-form",
10+
"specifier": "eslint-plugin-react-hook-form"
711
}
8-
// eslint-plugin-react-hook-form won't work — all its rules use context.getScope()
912
],
1013
"plugins": [
1114
"import",
@@ -38,6 +41,9 @@
3841

3942
"react-hooks/exhaustive-deps": ["error", { "additionalHooks": "useQuickActions" }],
4043
"react-hooks/rules-of-hooks": "error",
44+
"react-hook-form/destructuring-formstate": "error",
45+
// no-access-control uses context.getScope() which oxlint doesn't support
46+
"react-hook-form/no-nested-object-setvalue": "error",
4147
"import/no-default-export": "error",
4248
"consistent-type-imports": "error",
4349

app/ui/lib/Spinner.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,7 @@ type Props = {
106106
/** Loading spinner that shows for a minimum of `minTime` */
107107
export const SpinnerLoader = ({ isLoading, children = null, minTime = 500 }: Props) => {
108108
const [isVisible, setIsVisible] = useState(isLoading)
109-
const hideTimeout = useRef<NodeJS.Timeout | null>(null)
109+
const hideTimeout = useRef<ReturnType<typeof setTimeout> | null>(null)
110110
const loadingStartTime = useRef<number>(0)
111111

112112
useEffect(() => {

0 commit comments

Comments
 (0)