Refactor: Replace barrel exports with explicit imports

[YusukeHirao]

Summary

This PR eliminates barrel export files (index.ts) across the codebase and replaces them with explicit imports from individual modules. This improves tree-shaking, reduces circular dependency risks, and makes dependencies more explicit and traceable.

closes: #15

Key Changes

Core Refactoring

• Removed barrel exports: Deleted index.ts files from:

  • packages/@nitpicker/crawler/src/utils/ (types, array, error, object subdirectories)
  • packages/@nitpicker/crawler/src/archive/filesystem/
  • packages/@nitpicker/crawler/src/archive/
  • packages/@nitpicker/crawler/src/crawler/
  • packages/@nitpicker/core/src/hooks/

• Updated all imports: Changed from barrel imports to explicit module imports across 30+ files:

  • import { foo } from './utils/index.js' → import { foo } from './utils/types/types.js'
  • import { bar, baz } from './filesystem/index.js' → import { bar } from './filesystem/bar.js'; import { baz } from './filesystem/baz.js'

Package Entry Points

• Renamed main entry files to be more descriptive:
  • index.ts → crawler.ts, cli.ts, core.ts, types.ts, axe-plugin.ts, lighthouse-plugin.ts, etc.
  • Updated package.json exports to reference new entry point names
  • Updated bin/nitpicker.js to import from new path

Documentation & Tests

• Updated ARCHITECTURE.md to reflect new file structure
• Updated test imports and mocks to use explicit module paths
• Updated test server imports from index.js to server.js

Implementation Details

• All functionality remains unchanged; this is purely a structural refactoring
• Explicit imports make it clearer which modules depend on which, improving code maintainability
• Better support for tree-shaking in bundlers since unused exports are no longer re-exported through barrel files
• Reduced potential for circular dependency issues by removing intermediate aggregation layers

https://claude.ai/code/session_01Bkk5HRFJFjAviGpaT94yDn