ci: add project quality checks#4
Conversation
There was a problem hiding this comment.
Code Review
This pull request adds a CI workflow badge to both the English and Chinese README files, introduces a new ESLint configuration file for the frontend, and removes the unused vite-node dependency from the frontend package configuration. There are no review comments, and I have no feedback to provide.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
There was a problem hiding this comment.
Pull request overview
Adds a GitHub Actions CI pipeline and supporting frontend tooling updates to enforce baseline project quality checks (backend tests/compile, frontend lint/unit/build, and Playwright smoke), while keeping external/secret-dependent checks non-blocking.
Changes:
- Introduces a CI workflow that runs backend pytest/compileall, frontend lint/unit/build, and a Playwright smoke test, with report upload.
- Adds an ESLint 9 flat config (
eslint.config.js) so the existingnpm run lintworks under ESLint 9. - Removes
vite-nodefrom frontend dependencies (previously declared version not available), and adds CI badges to both READMEs.
Reviewed changes
Copilot reviewed 5 out of 6 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| README.md | Adds CI badge pointing to the new workflow. |
| README_zh.md | Adds CI badge pointing to the new workflow. |
| frontend/package.json | Removes vite-node devDependency (nonexistent version). |
| frontend/package-lock.json | Removes vite-node from the lockfile. |
| frontend/eslint.config.js | Adds ESLint 9 flat config for the frontend. |
| .github/workflows/ci.yml | Adds CI workflow for backend + frontend + Playwright smoke tests. |
Files not reviewed (1)
- frontend/package-lock.json: Generated file
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| permissions: | ||
| contents: read | ||
|
|
Summary
Local verification
npm ciinfrontend/python -m pytest -q-> 92 passed, 4 warningspython -m compileall -q backendnpm run lint-> 0 errors, 27 warnings from existing frontend any/hook/Fast Refresh rulesnpm test -- --run-> 25 passednpm run buildnpm run test:e2e -- interactive-artifacts.spec.ts-> 1 passedgit diff --checkNotes
npm auditis intentionally not blocking yet; the current dependency tree reports existing vulnerabilities unrelated to this CI change.