Refactor agent and enterprise settings tabs#568
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: b21b1b4bb2
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| proxy: { | ||
| '/api': { | ||
| target: 'http://localhost:8008', | ||
| target: 'http://localhost:8000', |
There was a problem hiding this comment.
Pointing the Vite /api and /ws proxies to port 8000 breaks local source-mode development, because restart.sh starts the backend on 8008 (BACKEND_PORT=8008). In that common flow (bash restart.sh / --source), frontend requests now proxy to a non-listening port and fail with connection errors, so the UI cannot reach the API unless users manually change their backend port.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: a2022a862d
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| const imgRef = useRef<HTMLImageElement>(null); | ||
| const [naturalSize, setNaturalSize] = useState({ width: 1, height: 1 }); | ||
| const [zoom, setZoom] = useState(1); |
There was a problem hiding this comment.
This file now includes copied company-identity editor code after the SkillsTab export, but its imports only bring in useMemo/useState; the appended section uses useRef, useEffect, useQueryClient, fetchJson, and other symbols that are not imported here. Because the frontend build runs tsc (frontend/package.json), this introduces TypeScript compile failures and blocks production builds even though the extra functions are unrelated to the Skills tab.
Useful? React with 👍 / 👎.
1 participant
Summary
Testing