Back to all reviewers

configure build tools properly

remix-run/react-router
Based on 2 comments
JavaScript

Ensure build configurations use proper package.json exports fields and TypeScript settings to avoid hard-coded paths and module format inconsistencies. Build tools should be configured to handle module resolution and output formats correctly to prevent runtime issues.

CI/CD JavaScript

Reviewer Prompt

Ensure build configurations use proper package.json exports fields and TypeScript settings to avoid hard-coded paths and module format inconsistencies. Build tools should be configured to handle module resolution and output formats correctly to prevent runtime issues.

Use package.json exports field instead of hard-coding file paths:

// Instead of hard-coded paths
input: `${SOURCE_DIR}/index.ts`,

// Use multiple inputs with proper exports configuration
input: [`${SOURCE_DIR}/index.ts`, `${SOURCE_DIR}/install.ts`],

Configure TypeScript and Babel properly for consistent module formats:

typescript({
  tsconfig: path.join(__dirname, "tsconfig.json"),
  exclude: ["__tests__"],
  noEmitOnError: !WATCH,
  noForceEmit: true, // Prevents incorrect CJS exports in ESM builds
}),
babel({
  babelHelpers: "bundled",
  exclude: /node_modules/,
  extensions: [".ts"],
  ...remixBabelConfig, // Use shared configuration
})

This prevents build-time issues that can cause deployment failures and ensures consistent module handling across different environments.

2
Comments Analyzed
JavaScript
Primary Language
CI/CD
Category

Source Discussions