refactor: move Node/Edge/ReadableGraph types to @bibgraph/types and keep evaluation in algorithms

Move core graph types to @bibgraph/types to break circular dependency:
- Create packages/types/src/graph-algorithms.ts with Node, Edge, NodeBase, EdgeBase, ReadableGraph
- Export core graph types from @bibgraph/types package
- Update algorithms to re-export Node/Edge from @bibgraph/types (backwards compatibility)
- Update graph-expansion to import ReadableGraph, NodeBase, EdgeBase from @bibgraph/types
- Update all evaluation files to import Node/Edge from @bibgraph/types

Move BFS back to algorithms package:
- Copy BFS implementation from graph-expansion to algorithms/traversal/bfs.ts
- Update BFS to import ReadableGraph from @bibgraph/types
- Create algorithms/traversal/index.ts to export BFS
- Update algorithms/src/index.ts to export bfs function (not types to avoid conflicts)
- Update algorithms/src/extraction/path.ts to import bfs from local traversal
- Update algorithms/src/graph-adapter.ts to import ReadableGraph from @bibgraph/types

Keep evaluation framework in algorithms package:
- Move evaluation framework from graph-expansion back to algorithms (where it belongs)
- Update evaluation imports to use relative paths (../.. for Graph/Path types)
- Export all evaluation functions from algorithms/src/index.ts
- Remove evaluation exports from graph-expansion/src/index.ts
- Remove evaluation export configuration from graph-expansion/package.json

This breaks the circular dependency:
- Previously: graph-expansion → algorithms (Node/Edge) → graph-expansion (bfs/ReadableGraph)
- Now: Both packages import from @bibgraph/types (no circular dependency)
- Evaluation stays in algorithms (depends on Graph/Path types from same package)

fix(lint-staged): wrap Nx commands in sh -c for environment variable support
- Fix NX_DAEMON=false spawn error by using shell wrapper
- Enables proper environment variable parsing in pre-commit hook

Author: Mearman

.lintstagedrc.js

@@ -12,12 +12,12 @@ export default {
   // TypeScript and JavaScript files in packages and apps
   '{packages,apps}/**/*.{ts,tsx,js,jsx}': [
     // Run cached lint:fix for affected projects using Nx (no daemon for pre-commit)
-    () => 'NX_DAEMON=false nx affected --target=lint:fix'
+    () => 'sh -c "NX_DAEMON=false nx affected --target=lint:fix"'
   ],
 
   // Type check all affected projects (no daemon for pre-commit)
   '*.{ts,tsx}': [
-    () => 'NX_DAEMON=false nx affected --target=typecheck'
+    () => 'sh -c "NX_DAEMON=false nx affected --target=typecheck"'
   ],
 
   // GitHub workflow files (actionlint only validates workflow files in .github/workflows/)

packages/algorithms/__tests__/evaluation/mi-experiment.integration.test.ts

@@ -14,8 +14,8 @@ import {
   generateMarkdownReport,
   generateLatexTable,
   generateJSONSummary,
-} from '@bibgraph/algorithms';
-import { randomRanker } from '@bibgraph/algorithms';
+} from '@bibgraph/graph-expansion';
+import { randomRanker } from '@bibgraph/graph-expansion';
 import {
   createSmallCitationNetwork,
   createMediumCitationNetwork,
@@ -27,7 +27,7 @@ import {
   validateGraphTypePerformance,
   getEffectSizeCategory,
 } from './fixtures/expected-results';
-import type { ExperimentConfig, MetricType, StatisticalTestType } from '@bibgraph/algorithms';
+import type { ExperimentConfig, MetricType, StatisticalTestType } from '@bibgraph/graph-expansion';
 
 /**
  * Simple MI ranker for testing (ranks by total path weight)

packages/algorithms/__tests__/evaluation/path-planting.unit.test.ts

@@ -13,7 +13,7 @@ import {
   pathFollowsTemplate,
   type PlantedPathConfig,
   type CitationPathType,
-} from '@bibgraph/algorithms';
+} from '@bibgraph/graph-expansion';
 
 /** Simple graph node */
 interface TestNode extends Node {