refactor: complete Clean Architecture reorganization

Reorganize remaining src root files and tests according to Clean Architecture:

**Domain Layer:**
- Move src/error.rs → src/domain/error.rs (domain errors)
- Move tests/unit/error_tests.rs → tests/unit/domain/error_tests.rs

**Infrastructure Layer:**
- Move src/logging.rs → src/infrastructure/logging.rs (file logging)
- Move src/repository_manager.rs → src/infrastructure/repository_manager.rs (git operations)

**Presentation Layer:**
- Move src/signal_handler.rs → src/presentation/signal_handler.rs (UI signal handling)
- Move src/sharing.rs → src/presentation/sharing.rs (social media sharing)
- Move tests/unit/sharing_tests.rs → tests/unit/presentation/sharing_tests.rs
- Move tests/unit/extractor_unit_tests.rs → tests/unit/domain/services/extractor_unit_tests.rs
- Move tests/unit/game_unit_tests.rs → tests/unit/presentation/game/game_unit_tests.rs

**Other Changes:**
- Update all import paths to reflect new module locations
- Update module declarations in lib.rs and test modules
- Clean up tests/unit/mod.rs to match src structure
- Update external imports from gittype::* paths
- No implementation changes, pure refactoring
- All 589 tests passing

The codebase now follows Clean Architecture with clear separation of concerns:
- Domain: Business logic and domain models
- Infrastructure: External system integrations
- Presentation: UI and external interfaces

🤖 Generated with [Claude Code](https://claude.ai/code)

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

Author: unhappychoice

src/error.rs src/domain/error.rssrc/error.rs renamed to src/domain/error.rs

@@ -1,3 +1,4 @@
+pub mod error;
 pub mod models;
 pub mod repositories;
 pub mod services;

src/domain/mod.rs

@@ -5,7 +5,7 @@ use crate::infrastructure::storage::{
 use crate::domain::models::storage::{SaveStageParams, SessionResultData, SessionStageResult, StoredRepository, StoredSession};
 use crate::domain::models::{Challenge, GitRepository, SessionResult};
 use crate::domain::services::scoring::{StageResult, StageTracker};
-use crate::{error::GitTypeError, Result};
+use crate::{domain::error::GitTypeError, Result};
 use std::sync::{Arc, Mutex};
 
 type StageResultTuple = (String, StageResult, usize, Option<Challenge>);

src/domain/repositories/session_repository.rs

@@ -1,4 +1,4 @@
-use crate::{error::GitTypeError, Result};
+use crate::{domain::error::GitTypeError, Result};
 use chrono;
 use log4rs::{
     append::{console::ConsoleAppender, file::FileAppender},

src/logging.rs src/infrastructure/logging.rssrc/logging.rs renamed to src/infrastructure/logging.rs

@@ -1,5 +1,7 @@
 pub mod cache;
 pub mod config;
 pub mod external;
+pub mod logging;
+pub mod repository_manager;
 pub mod storage;
 pub mod version;

src/infrastructure/mod.rs

@@ -1,6 +1,6 @@
 use super::super::database::Database;
 use crate::domain::models::Challenge;
-use crate::{error::GitTypeError, Result};
+use crate::{domain::error::GitTypeError, Result};
 use rusqlite::{params, Transaction};
 use serde_json;
 

src/repository_manager.rs src/infrastructure/repository_manager.rssrc/repository_manager.rs renamed to src/infrastructure/repository_manager.rs

@@ -1,7 +1,7 @@
 use super::super::database::Database;
 use crate::domain::models::GitRepository;
 use crate::domain::models::storage::{StoredRepository, StoredRepositoryWithLanguages};
-use crate::{error::GitTypeError, Result};
+use crate::{domain::error::GitTypeError, Result};
 use rusqlite::{params, Transaction};
 
 pub struct RepositoryDao<'a> {

src/infrastructure/storage/daos/challenge_dao.rs

@@ -1,7 +1,7 @@
 use super::super::database::Database;
 use crate::domain::models::{GitRepository, SessionResult};
 use crate::domain::models::storage::{SaveStageParams, SessionResultData, SessionStageResult, StoredSession};
-use crate::{error::GitTypeError, Result};
+use crate::{domain::error::GitTypeError, Result};
 use chrono::{DateTime, Utc};
 use rusqlite::{params, OptionalExtension, Transaction};
 use std::time::{SystemTime, UNIX_EPOCH};

src/infrastructure/storage/daos/repository_dao.rs

@@ -1,6 +1,6 @@
 use super::super::database::Database;
 use crate::domain::models::storage::{DifficultyStats, LanguageStats, StageStatistics, StoredStageResult};
-use crate::{error::GitTypeError, Result};
+use crate::{domain::error::GitTypeError, Result};
 use chrono::{DateTime, Utc};
 use rusqlite::params;