Skip to content

Conversation

@jbeard4
Copy link
Member

@jbeard4 jbeard4 commented Aug 29, 2025

Summary

  • simplify Puppeteer chaos-monkey script into a home-page smoke test
  • build and start app locally before loading the home page to surface crashes

Testing

  • npm run check-format scripts/chaos-monkey.ts
  • npm run lint
  • npm run check-types
  • npm test

https://chatgpt.com/codex/tasks/task_e_68b14137b5188327b77354de3e1d25cf

@jbeard4 jbeard4 temporarily deployed to yourpeer-codex-add-ci-t-xgz6zf August 29, 2025 06:26 Inactive
@jbeard4
Copy link
Member Author

jbeard4 commented Aug 29, 2025

@codex Regression has been introduced in the check-lint function:

Run npm run lint

> [email protected] lint
> next lint

Attention: Next.js now collects completely anonymous telemetry regarding usage.
This information is used to shape Next.js' roadmap and prioritize features.
You can learn more, including how to opt-out if you'd not like to participate in this anonymous program, by visiting the following URL:
https://nextjs.org/telemetry


./src/app/[route]/@mapContainer/[locationSlugOrPersonalCareSubCategory]/page.tsx
33:26  Error: React Hook "usePreviousParams" cannot be called in an async function.  react-hooks/rules-of-hooks

./src/app/[route]/@sidePanel/[locationSlugOrPersonalCareSubCategory]/page.tsx
39:26  Error: React Hook "usePreviousParams" cannot be called in an async function.  react-hooks/rules-of-hooks

./src/app/[route]/@staticPage/login.tsx
30:6  Warning: React Hook useEffect has a missing dependency: 'router'. Either include it or remove the dependency array.  react-hooks/exhaustive-deps

./src/app/layout.tsx
40:9  Warning: Custom fonts not added in `pages/_document.js` will only load for a single page. This is discouraged. See: https://nextjs.org/docs/messages/no-page-custom-font  @next/next/no-page-custom-font

./src/components/location-detail/street-view.tsx
144:11  Warning: img elements must have an alt prop, either with meaningful text, or an empty string for decorative images.  jsx-a11y/alt-text

./src/components/map-list-toggle-button.tsx
29:11  Warning: img elements must have an alt prop, either with meaningful text, or an empty string for decorative images.  jsx-a11y/alt-text
41:11  Warning: img elements must have an alt prop, either with meaningful text, or an empty string for decorative images.  jsx-a11y/alt-text

./src/components/map.tsx
227:6  Warning: React Hook useEffect has a missing dependency: 'cookies'. Either include it or remove the dependency array.  react-hooks/exhaustive-deps
231:6  Warning: React Hook useEffect has a missing dependency: 'cookies'. Either include it or remove the dependency array.  react-hooks/exhaustive-deps
240:6  Warning: React Hook useEffect has missing dependencies: 'mapCenter' and 'zoom'. Either include them or remove the dependency array.  react-hooks/exhaustive-deps
260:6  Warning: React Hook useEffect has missing dependencies: 'locationDetailStub', 'pathname', 'refreshUserPosition', 'router', and 'searchParams'. Either include them or remove the dependency array.  react-hooks/exhaustive-deps
310:6  Warning: React Hook useEffect has missing dependencies: 'getUserPositionOrCentralPark' and 'normalizedLocationDetailStub'. Either include them or remove the dependency array.  react-hooks/exhaustive-deps
402:6  Warning: React Hook useEffect has a missing dependency: 'cookies'. Either include it or remove the dependency array.  react-hooks/exhaustive-deps
417:6  Warning: React Hook useEffect has a missing dependency: 'cookies'. Either include it or remove the dependency array.  react-hooks/exhaustive-deps
435:6  Warning: React Hook useEffect has a missing dependency: 'checkIfIsMobile'. Either include it or remove the dependency array.  react-hooks/exhaustive-deps

./src/components/search-form.tsx
162:6  Warning: React Hook useEffect has a missing dependency: 'isCurrentlyOnLocationDetailPage'. Either include it or remove the dependency array.  react-hooks/exhaustive-deps

./src/components/service-component.tsx
234:6  Warning: React Hook useEffect has missing dependencies: 'hasScrolled' and 'service.name'. Either include them or remove the dependency array.  react-hooks/exhaustive-deps

./src/components/side-panel-component.tsx
46:6  Warning: React Hook useEffect has a missing dependency: 'cookies'. Either include it or remove the dependency array.  react-hooks/exhaustive-deps

info  - Need to disable some ESLint rules? Learn more here: https://nextjs.org/docs/basic-features/eslint#disabling-rules

Can you please correct this?

@chatgpt-codex-connector
Copy link

Summary

Testing

  • npx prettier src/components/use-previous-params.ts "src/app/[route]/@mapContainer/[locationSlugOrPersonalCareSubCategory]/page.tsx" "src/app/[route]/@sidePanel/[locationSlugOrPersonalCareSubCategory]/page.tsx" --check
  • npm run lint
  • npm run check-types
  • ⚠️ npm test (failed to launch Chrome: libatk-1.0.so.0: cannot open shared object file)

Notes

  • npm test could not run due to a missing system library (libatk-1.0.so.0).

View task →

@jbeard4 jbeard4 temporarily deployed to yourpeer-codex-add-ci-t-tkahtk August 29, 2025 12:42 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants