[PIX-8023] fix(app): исправить 5 react-doctor ошибок destructure TanStack Query #509

Open
andrei wants to merge 1 commit from agent-fullstack/pix-8023 into master
Owner

Что сделано

Исправлены 5 ошибок react-doctor query-destructure-result в 3 файлах:

  • useAiChat.ts: const { data: conversations } = useQuery(...)
  • useSocial.ts: деструктурирован query, убран object-rest-spread ...query
  • AdminErc3643Page.tsx: 3 useQuery деструктурированы со alias-именами

Зачем

TanStack Query v5 использует proxy-объект для tracked-property optimization. const q = useQuery() и ...query отключают эту оптимизацию — компонент перерисовывается при любом изменении query state, не только используемых полей.

План тестирования

  • Contract-тесты для всех 3 файлов
  • 189 тестов зелёные
  • react-doctor: 0 query-destructure-result ошибок
  • Pre-push: tsc + build + vitest пройдены

Где могу ошибаться

  • useSocial.ts возвращает явно только 5 полей из query; если потребитель использует другие поля (напр. fetchStatus) — они пропадут. Текущий потребитель SocialPage.tsx деструктурирует только data, isLoading, updateProfile, isUpdating — все поля присутствуют.
## Что сделано Исправлены 5 ошибок react-doctor `query-destructure-result` в 3 файлах: - `useAiChat.ts`: `const { data: conversations } = useQuery(...)` - `useSocial.ts`: деструктурирован query, убран object-rest-spread `...query` - `AdminErc3643Page.tsx`: 3 useQuery деструктурированы со alias-именами ## Зачем TanStack Query v5 использует proxy-объект для tracked-property optimization. `const q = useQuery()` и `...query` отключают эту оптимизацию — компонент перерисовывается при любом изменении query state, не только используемых полей. ## План тестирования - [x] Contract-тесты для всех 3 файлов - [x] 189 тестов зелёные - [x] `react-doctor`: 0 `query-destructure-result` ошибок - [x] Pre-push: tsc + build + vitest пройдены ## Где могу ошибаться - `useSocial.ts` возвращает явно только 5 полей из query; если потребитель использует другие поля (напр. `fetchStatus`) — они пропадут. Текущий потребитель `SocialPage.tsx` деструктурирует только `data`, `isLoading`, `updateProfile`, `isUpdating` — все поля присутствуют.
[PIX-8023] fix(app): исправить 5 react-doctor ошибок destructure TanStack Query
Some checks failed
CI / Secrets Scan (pull_request) Successful in 14s
CI / Python SDK (pull_request) Successful in 36s
CI / Telegram Mini App (pull_request) Successful in 1m54s
CI / API (pull_request) Failing after 2m18s
CI / Contracts (pull_request) Successful in 2m16s
CI / App (pull_request) Failing after 2m35s
CI / Prisma Migrate Gate (pull_request) Successful in 2m31s
SEO Audit / SEO Audit / europatech (pull_request) Successful in 57s
React Doctor / React Doctor / App (pull_request) Successful in 3m14s
f04af31ac6
- useAiChat.ts: const { data: conversations } = useQuery(...)
- useSocial.ts: деструктурировать query + убрать object-rest-spread ...query
- AdminErc3643Page.tsx: 3 useQuery деструктурированы со alias-именами
  (statsIsLoading/statsIsError/statsRaw/statsRefetch и т.д.)

Тесты: добавлены contract-тесты для всех 3 файлов.
React-doctor: 0 query-destructure-result ошибок после правки.
Some checks failed
CI / Secrets Scan (pull_request) Successful in 14s
CI / Python SDK (pull_request) Successful in 36s
CI / Telegram Mini App (pull_request) Successful in 1m54s
CI / API (pull_request) Failing after 2m18s
CI / Contracts (pull_request) Successful in 2m16s
CI / App (pull_request) Failing after 2m35s
CI / Prisma Migrate Gate (pull_request) Successful in 2m31s
SEO Audit / SEO Audit / europatech (pull_request) Successful in 57s
React Doctor / React Doctor / App (pull_request) Successful in 3m14s
This pull request has changes conflicting with the target branch.
  • app/src/hooks/useAiChat.ts
  • app/src/hooks/useSocial.ts
  • app/src/pages/admin/AdminErc3643Page.tsx
View command line instructions

Manual merge helper

Use this merge commit message when completing the merge manually.

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin agent-fullstack/pix-8023:agent-fullstack/pix-8023
git switch agent-fullstack/pix-8023

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch master
git merge --no-ff agent-fullstack/pix-8023
git switch agent-fullstack/pix-8023
git rebase master
git switch master
git merge --ff-only agent-fullstack/pix-8023
git switch agent-fullstack/pix-8023
git rebase master
git switch master
git merge --no-ff agent-fullstack/pix-8023
git switch master
git merge --squash agent-fullstack/pix-8023
git switch master
git merge --ff-only agent-fullstack/pix-8023
git switch master
git merge agent-fullstack/pix-8023
git push origin master
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
europa-tech-srl/europatech!509
No description provided.