# Step 3: Foundation Google Style

## 담당 역할과 필수 스킬

- 담당 역할: `implementation-agent`
- 필수 스킬: `harness`, `fesa-cpp-msvc-tdd`
- 이 Step만 `RED -> observed failure -> minimal GREEN -> focused/full VERIFY`로 수행한다.

## 읽어야 할 파일

- `/AGENTS.md`
- `/docs/CODINGSTYLE.md`
- `/docs/ARCHITECTURE.md`
- `/docs/ADR.md`
- `/docs/cpp-object-oriented-modular-refactoring/implementation-plan.md`
- `/include/fesa/core/diagnostic.hpp`
- `/include/fesa/core/source_identity.hpp`
- `/include/fesa/core/status.hpp`
- `/include/fesa/math/vector.hpp`
- `/include/fesa/math/matrix.hpp`
- `/include/fesa/math/sparse_matrix.hpp`
- `/include/fesa/solvers/linear/linear_solver.hpp`
- `/include/fesa/solvers/linear/mkl_pardiso_solver.hpp`
- `/include/fesa/build_info.hpp`
- `/src/fesa/core/diagnostic.cpp`, `/src/fesa/core/status.cpp`
- `/src/fesa/math/vector.cpp`, `/src/fesa/math/matrix.cpp`,
  `/src/fesa/math/sparse_matrix.cpp`
- `/src/fesa/solvers/linear/mkl_pardiso_solver.cpp`, `/src/fesa/build_info.cpp`
- `/tests/unit/core/diagnostic_test.cpp`, `/tests/unit/core/source_identity_test.cpp`,
  `/tests/unit/core/status_test.cpp`
- `/tests/unit/math/vector_test.cpp`, `/tests/unit/math/matrix_test.cpp`,
  `/tests/unit/math/sparse_matrix_test.cpp`
- `/tests/unit/solvers/linear/linear_solver_test.cpp`,
  `/tests/unit/solvers/linear/mkl_pardiso_solver_test.cpp`
- `/tests/unit/build_info_test.cpp`
- `/src/fesa/CMakeLists.txt`, `/tests/CMakeLists.txt`
- `/phases/cpp-object-oriented-modular-refactoring/index.json`
- `/phases/cpp-object-oriented-modular-refactoring/step3.md`

## 작업

Requirement `R-STYLE-001`의 foundation slice와 `R-DOC-001`을 구현한다. 동작 또는
수치식은 변경하지 않는다.

1. Focused tests의 include를 `.h`로, 모든 function/accessor call을 PascalCase로 먼저
   변경하고 missing header/member compile failure를 RED로 기록한다.
2. 다음 production headers를 같은 basename의 `.h`로 이동하고 full-path include guard를
   사용한다: core headers, `vector`, `matrix`, `sparse_matrix`, `linear_solver`,
   `mkl_pardiso_solver`, `build_info`.
3. 이 slice가 소유하는 public/protected function과 production internal function 이름을
   PascalCase로 바꾼다. Parameters/local variables는 snake_case, class members는 trailing
   underscore, enum values/constants는 `kPascalCase`로 바꾼다.
4. Public/protected declarations에 `@brief`와 필요한 ownership/failure/backend contract를
   Doxygen으로 기록한다. Tests에는 Doxygen를 추가하지 않는다.
5. 모든 repository consumer include/callsite를 기계적으로 갱신하되 다른 모듈의 함수
   정의나 책임은 바꾸지 않는다.
6. Matrix/Vector 연산 순서, MKL calls, Status category, diagnostic text는 보존한다.

## Acceptance Criteria

RED 후 GREEN focused/full verification:

```powershell
cmake -S . -B .harness/build -G "Visual Studio 18 2026" -A x64 `
  "-DFESA_GTEST_SOURCE_DIR=C:/git/googletest" `
  "-DMKL_DIR=C:/Program Files (x86)/Intel/oneAPI/mkl/2026.1/lib/cmake/mkl" `
  "-DTBB_DIR=C:/Program Files (x86)/Intel/oneAPI/tbb/2023.1/lib/cmake/tbb" `
  "-DHDF5_DIR=C:/Program Files/HDF_Group/HDF5/2.1.1/cmake"
cmake --build .harness/build --config Debug --target fesa_unit_tests
ctest --test-dir .harness/build -C Debug `
  -R "DenseMath|SparseAssembly|Status|Diagnostic|LinearSolver|MklPardiso|BuildInfo" `
  --output-on-failure
$files = @(
  (rg --files include/fesa/core include/fesa/math include/fesa/solvers/linear `
    -g "*.h"),
  "include/fesa/build_info.h",
  (rg --files src/fesa/core src/fesa/math src/fesa/solvers/linear -g "*.cpp"),
  "src/fesa/build_info.cpp",
  (rg --files tests/unit/core tests/unit/math tests/unit/solvers/linear -g "*.cpp"),
  "tests/unit/build_info_test.cpp"
)
& "C:/Program Files/LLVM/bin/clang-format.exe" --dry-run --Werror $files
cmake --build .harness/build --config Debug
ctest --test-dir .harness/build -C Debug --show-only=json-v1
ctest --test-dir .harness/build -C Debug --output-on-failure
```

## 검증 및 상태 갱신

- RED compile error, focused GREEN, formatting, full CTest를 summary에 남긴다.
- 성공 시 현재 Step만 `completed`; 반복 compile failure는 `error`로 기록한다.
- timestamp, retry, commit, advancement는 Executor 소유다.

## 금지사항

- `Vector3`나 BLAS adapter를 추가하지 마라. 이유: Step 7과 10의 semantic work다.
- 수치 expression/reduction 순서를 바꾸지 마라. 이유: style-only review unit이다.
- Test에 Doxygen를 추가하지 마라. 이유: 승인 문서화 범위는 production뿐이다.
- 직접 commit하거나 hook script를 수동 실행하지 마라.
