Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request introduces documentation for an AI-assisted development pipeline and a corresponding prompt template for the repository. The feedback provided addresses a non-standard glob pattern for test files and a conflict in the verification steps where the --frozen-lockfile flag would prevent necessary lockfile updates when dependencies are modified.
| 5. Do not change the public API (exported names, type signatures, default exports) | ||
| unless the issue explicitly requires it. If you must, call it out in the PR body. | ||
| 6. When behavior changes, add or update tests in the same package | ||
| (`packages/<pkg>/src/**/*.test.ts(x)` or the package's existing test layout). |
There was a problem hiding this comment.
테스트 파일 경로 패턴에서 ts(x)는 표준적인 글로브(glob) 패턴이 아닙니다. AI가 정확하게 파일을 찾을 수 있도록 {ts,tsx} 또는 ts*와 같은 표준 형식을 사용하는 것이 좋습니다.
| (`packages/<pkg>/src/**/*.test.ts(x)` or the package's existing test layout). | |
| (packages/<pkg>/src/**/*.test.{ts,tsx} or the package's existing test layout). |
|
/gemini review |
There was a problem hiding this comment.
Code Review
This pull request introduces documentation for an AI-assisted development pipeline, including a prompt template and a detailed architectural overview involving n8n and GitHub Actions. The feedback focuses on refining security wildcard patterns to avoid over-blocking files, updating the installation command to allow lockfile updates when dependencies change, ensuring branch naming consistency, and synchronizing security rules across the documentation files.
📊 Test Coverage Report (vitest)
|
|
/gemini review |
📊 Test Coverage Report (vitest)
|
There was a problem hiding this comment.
Code Review
This pull request introduces documentation for the AI-assisted development pipeline, including a reusable prompt template and a detailed architecture overview for n8n and GitHub Actions integration. The review feedback suggests adding 'area:shared' to the label list in the pipeline documentation for consistency and updating the 'pnpm install' command in the prompt template to use '--frozen-lockfile' by default to prevent unintended lockfile changes.
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
📊 Test Coverage Report (vitest)
|
📊 Test Coverage Report (vitest)
|
1 participant
No description provided.