Skip to content

fix: restore AdminPanel lint/format config after repo split#28

Open
juliusknorr wants to merge 3 commits into
mainfrom
fix/adminpanel-lint-format
Open

fix: restore AdminPanel lint/format config after repo split#28
juliusknorr wants to merge 3 commits into
mainfrom
fix/adminpanel-lint-format

Conversation

@juliusknorr

Copy link
Copy Markdown
Member

Summary

npm run lint:check was failing on every JSX file in AdminPanel/client with parse errors. The upstream repo-split commit removed the React/JSX ESLint config and the AiIntegration prettier ignores on the assumption AdminPanel had moved out, but the directory still lives in this fork.

  • Restore the JSX ESLint config block (parser + eslint-plugin-react/-react-hooks) and AiIntegration ignores; re-add the two plugin devDeps
  • Re-ignore the vendored AiIntegration files in prettier (they broke format:check, including a no-parser error on vocabulary.bpe)
  • Format the remaining first-party AdminPanel css/scss/webpack files

After this, lint:check and format:check both pass.

The upstream repo-split commit removed the React/JSX parser config for
AdminPanel/client, but that directory still lives in this fork, so lint
failed with parse errors on every JSX file. Re-add the JSX config block,
the AiIntegration ignores and the eslint-plugin-react(-hooks) deps.

AI-assistant: Claude Code 2.1.169 (Claude Opus 4.8)
Signed-off-by: Julius Knorr <jus@bitgrid.net>
These vendored files were dropped from .prettierignore by the upstream
repo-split commit but still exist in this fork, breaking format:check
(including a no-parser error on vocabulary.bpe).

AI-assistant: Claude Code 2.1.169 (Claude Opus 4.8)
Signed-off-by: Julius Knorr <jus@bitgrid.net>
AI-assistant: Claude Code 2.1.169 (Claude Opus 4.8)
Signed-off-by: Julius Knorr <jus@bitgrid.net>
@juliusknorr juliusknorr requested review from moodyjmz and rikled June 9, 2026 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant