fix(clinical-scheduler): VPR-40 Use local time for audit timestamps

README.md

@@ -194,8 +194,8 @@ The project includes VS Code launch configurations for debugging both frontend a
 ### Code Quality
 
 - `npm run lint` - Check code style for specified files or directories
-- `npm run lint:staged` - Run pre-commit checks manually
-- `npm run lint:precommit` - Run lint-staged (used by pre-commit hooks)
+- `npm run lint:staged` - Lint only git-staged files
+- `npm run precommit` - Run full pre-commit checks manually (lint, test, build verify)
 
 ## Troubleshooting
 

scripts/lint-staged-dotnet.js

@@ -143,9 +143,9 @@ const runBuild = (projectPath) => {
         logger.info(`Force flag enabled, skipping cache for ${projectPath}/ project`)
     } else {
         try {
-            if (!needsBuild(projectPath, effectiveProjectName)) {
+            if (!needsBuild(projectPath, projectName)) {
                 // Get cached analyzer output instead of skipping analysis
-                const cachedOutput = getCachedBuildOutput(effectiveProjectName)
+                const cachedOutput = getCachedBuildOutput(projectName)
                 logger.success(`Build not needed for ${projectPath}/ project (using cached results)`)
 
                 // Check for analyzer warnings in cached output
@@ -171,7 +171,7 @@ const runBuild = (projectPath) => {
 
         // Store successful build in cache
         try {
-            markAsBuilt(projectPath, effectiveProjectName, result)
+            markAsBuilt(projectPath, projectName, result)
         } catch (error) {
             logger.warning(`Failed to cache build result: ${error.message}`)
         }
@@ -188,7 +188,7 @@ const runBuild = (projectPath) => {
 
             // Store build output even for failed builds so analyzers can process it
             try {
-                markAsBuilt(projectPath, effectiveProjectName, output)
+                markAsBuilt(projectPath, projectName, output)
             } catch (error) {
                 logger.warning(`Failed to cache build output: ${error.message}`)
             }

scripts/test-dotnet.js

@@ -87,7 +87,7 @@ function ensureBuild() {
 function runTests() {
     logger.info("Running tests...")
     try {
-        execFileSync("dotnet", ["test", precommitDll, "--no-build", "--verbosity=normal", "--nologo"], {
+        execFileSync("dotnet", ["test", precommitDll, "--verbosity=normal", "--nologo"], {
             encoding: "utf8",
             timeout: 300_000, // 5 minute timeout for tests
             stdio: "inherit",

test/ClinicalScheduler/EmailNotificationTest.cs

@@ -695,7 +695,7 @@ public async Task AddInstructorAsync_AsPrimaryEvaluator_DoesNotSendReplacementEm
             var rotationId = 50; // Use unique rotation ID to avoid conflict
             var weekIds = new[] { 50 }; // Use unique week ID to avoid conflict
             var weekNum = 15;
-            var testYear = 2025;
+            var testYear = DateTime.Now.Year;
 
             // Add test data
             await AddTestPersonAsync(oldPrimaryMothraId, "Alice", "Johnson");

web/Areas/ClinicalScheduler/Services/ScheduleAuditService.cs

@@ -167,7 +167,7 @@ private async Task<ScheduleAudit> CreateAuditEntryAsync(
                     WeekId = weekId,
                     Action = action,
                     Area = ScheduleAuditAreas.Clinicians,
-                    TimeStamp = DateTime.UtcNow,
+                    TimeStamp = DateTime.Now,
                     ModifiedBy = modifiedByMothraId
                 };
 

web/Areas/ClinicalScheduler/Services/ScheduleEditService.cs

@@ -168,7 +168,7 @@ public async Task<List<InstructorSchedule>> AddInstructorAsync(
                             WeekId = weekId,
                             Evaluator = isPrimaryEvaluator,
                             ModifiedBy = currentUser.MothraId,
-                            ModifiedDate = DateTime.UtcNow
+                            ModifiedDate = DateTime.Now
                         };
 
                         _context.InstructorSchedules.Add(schedule);
@@ -410,13 +410,13 @@ await _auditService.LogInstructorRemovedAsync(
 
                         schedule.Evaluator = true;
                         schedule.ModifiedBy = currentUser.MothraId;
-                        schedule.ModifiedDate = DateTime.UtcNow;
+                        schedule.ModifiedDate = DateTime.Now;
                     }
                     else
                     {
                         schedule.Evaluator = false;
                         schedule.ModifiedBy = currentUser.MothraId;
-                        schedule.ModifiedDate = DateTime.UtcNow;
+                        schedule.ModifiedDate = DateTime.Now;
                     }
 
                     await _context.SaveChangesAsync(cancellationToken);
@@ -587,7 +587,7 @@ private async Task<List<InstructorSchedule>> ClearPrimaryEvaluatorAsync(int rota
                 cancellationToken.ThrowIfCancellationRequested();
                 schedule.Evaluator = false;
                 schedule.ModifiedBy = modifiedByMothraId;
-                schedule.ModifiedDate = DateTime.UtcNow;
+                schedule.ModifiedDate = DateTime.Now;
             }
 
             if (existingPrimary.Any())