refactor(web): remove barrelsby tool and configuration

- Remove barrelsby dependency from package.json
- Remove barrelsby scripts from root package.json
- Remove barrelsby task from apps/web/project.json
- Update dev target to remove barrelsby dependency
- Remove barrelsby-header ESLint rule and file
- Update tools/eslint-rules/index.ts to remove barrelsby-header
- Update eslint.config.base.ts to remove barrelsby-header reference

Fixes issue where pnpm dev command was failing due to missing .barrelsby.json

Author: Mearman

apps/web/project.json

@@ -11,25 +11,15 @@
         "cwd": "{projectRoot}"
       }
     },
-    "barrelsby": {
-      "executor": "nx:run-commands",
-      "options": {
-        "command": "barrelsby --config ../../.barrelsby.json --directory ./src/components",
-        "cwd": "{projectRoot}"
-      },
-      "cache": true
-    },
     "dev": {
       "executor": "nx:run-commands",
       "options": {
         "commands": [
-          "echo '[barrels:watch] Starting...' && exec nx watch --projects=types,utils,client,ui,algorithms,web -- nx run \\$NX_PROJECT_NAME:barrelsby",
           "nx run web:serve"
         ],
         "parallel": true
       },
-      "continuous": true,
-      "dependsOn": ["barrelsby"]
+      "continuous": true
     },
     "test:e2e": {
       "executor": "@nx/playwright:playwright",

eslint.config.base.ts

@@ -171,8 +171,7 @@ export default tseslint.config([
             "barrel-files/avoid-re-export-all": "off", // Turned off - using custom rule instead for more control
             "barrel-files/avoid-namespace-import": "off", // Turned off - namespace imports have valid use cases
 
-            // Custom rules - prevent re-exports from non-barrel files but make barrels optional
-            "custom/barrelsby-header": "off", // Turned off - barrels are optional
+            // Custom rules - prevent re-exports from non-barrel files
             "custom/no-deprecated": "error",
             "custom/no-duplicate-reexports": "error",
             "custom/no-reexport-from-non-barrel": "error", // Keep - prevents re-exports from non-barrels

package.json

@@ -10,8 +10,6 @@
   ],
   "repository": "https://github.com/Mearman/BibGraph.git",
   "scripts": {
-    "barrels": "nx run-many -t barrelsby",
-    "barrels:dry": "barrelsby --config .barrelsby.json --delete false",
     "build": "nx run-many -t build",
     "clean": "nx reset && rm -rf dist coverage .nx/cache .tanstack node_modules/.vite && pnpm --filter='*' clean",
     "cli": "tsx apps/cli/src/openalex-cli.ts",
@@ -104,7 +102,6 @@
     "@vitest/coverage-v8": "4.0.14",
     "@vitest/eslint-plugin": "1.5.1",
     "@vitest/ui": "4.0.14",
-    "barrelsby": "2.8.1",
     "chokidar": "5.0.0",
     "eslint": "9.39.1",
     "eslint-config-xo": "0.49.0",

tools/eslint-rules/barrelsby-header.ts

@@ -7,15 +7,13 @@
 
 import { noDeprecated } from "./no-deprecated.js";
 import { noRedundantAssignment } from "./no-redundant-assignment.js";
-import { barrelsbyHeader } from "./barrelsby-header.js";
 import { noDuplicateReexports } from "./no-duplicate-reexports.js";
 import { noReexportFromNonBarrel } from "./no-reexport-from-non-barrel.js";
 
 export const customRulesPlugin = {
   rules: {
     "no-deprecated": noDeprecated,
     "no-redundant-assignment": noRedundantAssignment,
-    "barrelsby-header": barrelsbyHeader,
     "no-duplicate-reexports": noDuplicateReexports,
     "no-reexport-from-non-barrel": noReexportFromNonBarrel,
   },
@@ -24,7 +22,6 @@ export const customRulesPlugin = {
 export {
   noDeprecated,
   noRedundantAssignment,
-  barrelsbyHeader,
   noDuplicateReexports,
   noReexportFromNonBarrel,
 };