fix(docs): Update outdated path references from packages/ to src/

- Fix code path references: packages/codegen -> src/codegen
- Fix code path references: packages/leetcode_datasource -> src/leetcode_datasource
- Fix code path references: packages/practice_workspace -> src/practice_workspace
- Update script comments to reflect correct package location
- Replace non-existent validate-tests/validate-e2e commands with check
- Mark unimplemented commands in in-progress docs

Files updated:
- src/codegen/analyzer.py
- scripts/new_problem.sh, new_problem.bat, new_practice.sh, new_practice.bat
- docs/contracts/test-file-format.md
- docs/contributors/docs-directory-organization.md
- docs/contributors/package-documentation-strategy.md
- docs/in-progress/new-problem-tests-autogen/migration-plan.md
- docs/in-progress/new-problem-tests-autogen/specification.md
- docs/in-progress/tiered-problem-generation/migration-plan.md
- docs/in-progress/tiered-problem-generation/specification.md
- docs/packages/leetcode_datasource/README.md
- runner/utils/codec/README.md
- tools/review-code/compare_html_parsers.py

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: lufftw

docs/contracts/test-file-format.md

@@ -274,7 +274,7 @@ false
 All test files must pass:
 
 ```bash
-python -m codegen validate-tests --all
+python -m codegen check --all
 ```
 
 Checks:

docs/contributors/docs-directory-organization.md

@@ -41,7 +41,7 @@ This document defines the organization of the `docs/` directory, including folde
 |------|------------|---------|
 | Folders | `snake_case` or `kebab-case` | `leetcode_datasource/`, `two_pointers/` |
 | Files | `kebab-case.md` | `solution-contract.md` |
-| Package folders | Match package name exactly | `docs/packages/codegen/` matches `packages/codegen/` |
+| Package folders | Match package name exactly | `docs/packages/codegen/` matches `src/codegen/` |
 
 ### 1.3 Root-level Files
 
@@ -253,9 +253,9 @@ System-level documentation for core modules. See [Package Documentation Strategy
 | Folder | Corresponds to |
 |--------|----------------|
 | `docs/runner/` | `runner/` |
-| `docs/packages/codegen/` | `packages/codegen/` |
-| `docs/packages/leetcode_datasource/` | `packages/leetcode_datasource/` |
-| `docs/packages/practice_workspace/` | `packages/practice_workspace/` |
+| `docs/packages/codegen/` | `src/codegen/` |
+| `docs/packages/leetcode_datasource/` | `src/leetcode_datasource/` |
+| `docs/packages/practice_workspace/` | `src/practice_workspace/` |
 
 > ⚠️ **Important**: Package docs must be under `docs/packages/<pkg>/`, not `docs/<pkg>/`.
 > Scattered files like `docs/codegen.md` are NOT allowed.

docs/contributors/package-documentation-strategy.md

@@ -335,11 +335,11 @@ The `runner/` module is the **reference implementation** for this documentation
 
 ```markdown
 <!-- BAD: Same content in both files -->
-packages/codegen/README.md: "## How It Works: Step 1, Step 2..."
+src/codegen/README.md: "## How It Works: Step 1, Step 2..."
 docs/packages/codegen/README.md: "## How It Works: Step 1, Step 2..."
 
 <!-- GOOD: Different perspectives -->
-packages/codegen/README.md: "## Public API" (what to call)
+src/codegen/README.md: "## Public API" (what to call)
 docs/packages/codegen/README.md: "## Typical Workflows" (system-level flow)
 ```
 

docs/in-progress/new-problem-tests-autogen/migration-plan.md

@@ -462,7 +462,9 @@ CI automation (when enabled) does not replace manual responsibility.
 #### Command
 
 ```bash
-python -m codegen validate-tests --all
+# Note: validate-tests was planned but not implemented
+# Using check command instead:
+python -m codegen check --all
 ```
 
 #### Pass Condition
@@ -660,7 +662,7 @@ Phase 6: solve_generator ◀── Phase 5: generators/ ◀── Phase 4: codeg
 
 2. **Run Gate 0 validation**
    ```bash
-   python -m codegen validate-tests --all
+   python -m codegen check --all
    ```
 
 3. **Review and commit**
@@ -745,8 +747,9 @@ Phase 6: solve_generator ◀── Phase 5: generators/ ◀── Phase 4: codeg
 
 2. **Validate all Tier-0 problems**
    ```bash
-   # Run E2E validation script (to be created)
-   python -m codegen validate-e2e --tier 0
+   # Note: validate-e2e was planned but not implemented
+   # Use check command instead:
+   python -m codegen check --all
    ```
 
 3. **Verify unsupported types fail gracefully**
@@ -853,7 +856,7 @@ python -m codegen migrate --all --dry-run
 python -m codegen migrate --all
 
 # Step 3: Validate
-python -m codegen validate-tests --all
+python -m codegen check --all
 
 # Step 4: Run regression to ensure nothing broke
 python runner/test_runner.py --all
@@ -1017,7 +1020,7 @@ Prioritize by complexity:
 | Task | Command |
 |------|---------|
 | Migrate tests | `python -m codegen migrate --all` |
-| Validate tests | `python -m codegen validate-tests --all` |
+| Validate tests | `python -m codegen check --all` |
 | Run regression | `python runner/test_runner.py --all` |
 | New problem + tests | `python -m codegen new <id> --with-tests` |
 | Check consistency | `python -m codegen check <id>` |
@@ -1088,16 +1091,18 @@ python -m codegen migrate 0001_two_sum
 python -m codegen migrate --all --no-backup
 ```
 
-#### `codegen validate-tests`
+#### `codegen validate-tests` (Not Implemented)
+
+> **Note**: This command was planned but not implemented. Use `codegen check --all` instead for test file validation.
 
 Validate test files against canonical format (Gate 0).
 
 ```bash
-python -m codegen validate-tests [options]
+# Planned command (not implemented):
+# python -m codegen validate-tests [options]
 
-Options:
-  --all             Validate all test files
-  --verbose         Show detailed output
+# Use this instead:
+python -m codegen check --all
 ```
 
 #### `codegen check`
@@ -1260,7 +1265,7 @@ git checkout HEAD~1 -- generators/
 
 #### Scenario 1: Gate 0 fails after migration
 
-**Symptom:** `validate-tests` reports parse errors
+**Symptom:** `codegen check` reports parse errors
 
 **Recovery:**
 1. Identify failing files from error output

docs/in-progress/new-problem-tests-autogen/specification.md

@@ -20,7 +20,7 @@ This feature has been fully implemented with the following capabilities:
 
 For complete documentation, see:
 - [CodeGen Package Docs](../../packages/codegen/README.md)
-- [Package README](../../../packages/codegen/README.md)
+- [Package README](../../../src/codegen/README.md)
 
 ---
 
@@ -57,7 +57,7 @@ For complete documentation, see:
 @echo off
 setlocal EnableExtensions
 
-REM Pass-through wrapper: all logic lives in packages/codegen
+REM Pass-through wrapper: all logic lives in src/codegen
 python -m codegen new %*
 exit /b %ERRORLEVEL%
 ```
@@ -338,14 +338,14 @@ Summary: 1 solution, 0 test cases created
 
 | Component | Location |
 |-----------|----------|
-| CLI entry point | `packages/codegen/cli.py` |
-| IO Schema inference | `packages/codegen/core/io_schema.py` |
-| Example parser | `packages/codegen/core/example_parser.py` |
-| Stub parser | `packages/codegen/core/stub_parser.py` |
-| solve() generator | `packages/codegen/core/solve_generator.py` |
-| Test generator | `packages/codegen/core/test_generator.py` |
-| Consistency checker | `packages/codegen/checker.py` |
-| Format migrator | `packages/codegen/migrator.py` |
+| CLI entry point | `src/codegen/cli.py` |
+| IO Schema inference | `src/codegen/core/io_schema.py` |
+| Example parser | `src/codegen/core/example_parser.py` |
+| Stub parser | `src/codegen/core/stub_parser.py` |
+| solve() generator | `src/codegen/core/solve_generator.py` |
+| Test generator | `src/codegen/core/test_generator.py` |
+| Consistency checker | `src/codegen/checker.py` |
+| Format migrator | `src/codegen/migrator.py` |
 | Windows wrapper | `scripts/new_problem.bat` |
 
 ---
@@ -571,7 +571,7 @@ python -m codegen.analyzer
 
 | Component | Location |
 |-----------|----------|
-| Analyzer | `packages/codegen/analyzer.py` |
+| Analyzer | `src/codegen/analyzer.py` |
 | Report output | `docs/in-progress/new-problem-tests-autogen/mismatch-report.json` |
 
 ---

docs/in-progress/tiered-problem-generation/migration-plan.md

@@ -52,7 +52,7 @@ encode_random_pointer_list(head) -> list
 
 **New Structure**:
 ```
-packages/codegen/core/catalog/
+src/codegen/core/catalog/
 ├── __init__.py              # Unified API
 ├── registry.py              # Metadata registry
 └── templates/
@@ -81,7 +81,7 @@ packages/codegen/core/catalog/
 
 ### 3.1 Create config reader utility ✅
 
-**Location**: `packages/codegen/core/problem_support.py` (implemented)
+**Location**: `src/codegen/core/problem_support.py` (implemented)
 
 All required functions are implemented:
 - ✅ `load_problem_config(problem_id: str) -> ProblemConfig`
@@ -92,7 +92,7 @@ All required functions are implemented:
 
 ### 3.2 Integrate with codegen ✅
 
-**Updated**: `packages/codegen/reference/generator.py`
+**Updated**: `src/codegen/reference/generator.py`
 - ✅ Auto-detects tier from config
 - ✅ Automatically uses tiered mode for Tier-1/1.5 problems
 - ✅ Applies codec_mode logic (import vs inline)

docs/in-progress/tiered-problem-generation/specification.md

@@ -211,9 +211,9 @@ IF tier == "1.5":
 | `docs/contracts/codec.md` | Codec package contract |
 | `config/problem-support.yaml` | Living registry |
 | `runner/utils/codec/` | **Single Source**: Runtime + inline templates |
-| `packages/codegen/core/catalog/` | AST extraction from codec/ |
-| `packages/codegen/core/problem_support.py` | Config reader utility |
-| `packages/codegen/core/tiered_solve_generator.py` | Tiered solve() generation |
+| `src/codegen/core/catalog/` | AST extraction from codec/ |
+| `src/codegen/core/problem_support.py` | Config reader utility |
+| `src/codegen/core/tiered_solve_generator.py` | Tiered solve() generation |
 
 ---
 
@@ -231,8 +231,8 @@ IF tier == "1.5":
 - [x] Update `catalog/__init__.py` with AST extraction
 
 ### Phase 3: codegen Integration ✅
-- [x] Create `packages/codegen/core/problem_support.py` (config reader)
-- [x] Create `packages/codegen/core/tiered_solve_generator.py`
+- [x] Create `src/codegen/core/problem_support.py` (config reader)
+- [x] Create `src/codegen/core/tiered_solve_generator.py`
 - [x] Implement `codec_mode` logic (import vs inline)
 - [x] Add `--solve-mode=tiered` CLI option
 - [x] Add `--codec-mode` CLI option (override config)

docs/packages/leetcode_datasource/README.md

@@ -128,7 +128,7 @@ High-level summary of public APIs. For complete API reference, see [Package READ
 ### Module Structure
 
 ```
-packages/leetcode_datasource/
+src/leetcode_datasource/
 ├── __init__.py              # Public API exports
 ├── datasource.py            # LeetCodeDataSource main class
 ├── config.py                # DataSourceConfig

runner/utils/codec/README.md

@@ -20,7 +20,7 @@ This package provides codec functions to convert between simple `.in/.out` forma
 
 - ❌ Problem-specific logic (handled by `solve()`)
 - ❌ Test comparison (handled by `runner/compare.py`)
-- ❌ Code generation (handled by `packages/codegen/`)
+- ❌ Code generation (handled by `src/codegen/`)
 
 ## Public API
 
@@ -85,7 +85,7 @@ codec/
 | Direction | Module | Purpose |
 |-----------|--------|---------|
 | Used by → | `solutions/*.py` | Runtime codec |
-| Used by → | `packages/codegen/` | AST extraction for inline mode |
+| Used by → | `src/codegen/` | AST extraction for inline mode |
 
 ## Related Documentation
 
@@ -100,5 +100,5 @@ codec/
 
 1. Update this README (quick reference)
 2. Update `docs/contracts/codec.md` (contract)
-3. Update `packages/codegen/core/catalog/__init__.py` if adding new classes/functions
+3. Update `src/codegen/core/catalog/__init__.py` if adding new classes/functions
 

scripts/new_practice.bat

@@ -3,7 +3,7 @@ setlocal EnableExtensions
 
 REM ============================================
 REM  Create Practice File (pass-through wrapper)
-REM  All logic lives in packages/codegen
+REM  All logic lives in src/codegen
 REM  
 REM  Usage:
 REM    new_practice.bat 23                    Generate practice skeleton