search

r-sugi / nextjs-tdd-template

https://nextjs-tdd-templatestorybook-rsugis-projects.vercel.app
0 stars 0 forks source link

fix: 本番環境ビルド時にdata-testitを除外する #155

Closed r-sugi closed 3 months ago

r-sugi commented 3 months ago

User description

154

124

PR Type

enhancement

Description

Changes walkthrough 📝

Relevant files
Enhancement
next.config.mjs
Enhance production build configuration in Next.js               
next.config.mjs
  • Added a check for production environment.
  • Enabled reactStrictMode.
  • Configured removal of data-testid properties in production builds.
    		•  +11/-1   

    💡 PR-Agent usage: Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

    vercel[bot] commented 3 months ago

    The latest updates on your projects. Learn more about Vercel for Git ↗︎

    Name Status Preview Comments Updated (UTC)
    nextjs-tdd-template_storybook ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 22, 2024 9:41am
    github-actions[bot] commented 3 months ago

    PR Reviewer Guide 🔍

    ⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ Key issues to review

    コードの最適化
    `isProd` 変数の定義が単一のファイルに閉じられていますが、他の設定やコンポーネントでも使用される可能性があるため、より中央で管理されるべきかもしれません。
    github-actions[bot] commented 3 months ago

    PR Code Suggestions ✨

    CategorySuggestion                                                                                                                                    Score
    Maintainability
    条件式を直接 compiler プロパティに設定することで、コードの明瞭さを向上させます。 ___ **isProd 条件を使用して compiler オプションを設定する際に、... 演算子を使用していますが、これにより nextConfig
    オブジェクトが不必要に複雑になります。条件が false の場合、空のオブジェクトがマージされるため、条件式を直接 compiler
    プロパティに設定することをお勧めします。** [next.config.mjs [9-14]](https://github.com/r-sugi/nextjs-tdd-template/pull/155/files#diff-18c049b08c4a0f5ab451c598aeb2c4848bb9d7877b51ca3e5effb94a225814d2R9-R14) ```diff -...(isProd && { - compiler: { - reactRemoveProperties: { - properties: ["^data-testid$"], - }, +compiler: isProd ? { + reactRemoveProperties: { + properties: ["^data-testid$"], }, -}), +} : undefined, ```
    Suggestion importance[1-10]: 7 Why: The suggestion improves code clarity by directly assigning the `compiler` property based on the `isProd` condition, avoiding unnecessary complexity from merging an empty object when the condition is false.
    		7