fix: log environment API user only once per session

- Store user in static variable for reuse
- Only debugPrint on first successful load

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

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

CHANGELOG.md

@@ -7,13 +7,29 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [1.5.6] - 2026-01-07
+
+### Changed
+
+- **Environment user logging** - Now logs API user only once per session instead of on every refresh
+
+## [1.5.5] - 2026-01-07
+
+### Fixed
+
+- **Sun position arc overflow** - Arc now constrained to fit within card boundaries
+  - Prevents arc and sun/moon from overflowing on wider displays
+  - Scales radius down when arc height exceeds available space
+
+### Added
+
+- **Debug logging for environment user** - Logs authenticated user on environment data load
+
 ## [1.5.4] - 2026-01-07
 
 ### Added
 
-- **User display in environment section** - Shows authenticated user in section header for debugging
-  - Displays `user: {username}` next to "Environment" header when data loads
-  - Helps diagnose user resolution issues with OIDC authentication
+- User display in environment section header (reverted in 1.5.5)
 
 ## [1.5.3] - 2026-01-07
 

lib/features/front_hall/presentation/widgets/dashboard_content.dart

@@ -26,6 +26,13 @@ class _DashboardContentState extends ConsumerState<DashboardContent> {
   Timer? _systemStatsTimer;
   Timer? _environmentTimer;
 
+  /// Tracks if we've logged the environment API user (log once per session)
+  static bool _hasLoggedEnvUser = false;
+  static String? _envApiUser;
+
+  /// Get the environment API user (available after first load)
+  static String? get envApiUser => _envApiUser;
+
   @override
   void initState() {
     super.initState();
@@ -129,62 +136,47 @@ class _DashboardContentState extends ConsumerState<DashboardContent> {
         const SizedBox(height: 24),
 
         // Environment - Sun, Weather, Forecast, Air Quality
+        const _SectionHeader(title: 'Environment', icon: Icons.eco),
+        const SizedBox(height: 8),
         environmentAsync.when(
-          data: (envData) => Column(
-            crossAxisAlignment: CrossAxisAlignment.start,
-            children: [
-              _SectionHeader(
-                title: 'Environment',
-                icon: Icons.eco,
-                subtitle: envData.user != null ? 'user: ${envData.user}' : null,
-              ),
-              const SizedBox(height: 8),
-              _EnvironmentSection(
-                envData: envData,
-                onRefresh: () => ref.invalidate(environmentProvider),
-              ),
-            ],
+          data: (envData) {
+            // Store and log user once per session
+            if (!_hasLoggedEnvUser && envData.user != null) {
+              _envApiUser = envData.user;
+              _hasLoggedEnvUser = true;
+              debugPrint('Environment API user: ${envData.user}');
+            }
+            return _EnvironmentSection(
+              envData: envData,
+              onRefresh: () => ref.invalidate(environmentProvider),
+            );
+          },
+          loading: () => const Card(
+            child: Padding(
+              padding: EdgeInsets.all(32),
+              child: Center(child: CircularProgressIndicator()),
+            ),
           ),
-          loading: () => Column(
-            crossAxisAlignment: CrossAxisAlignment.start,
-            children: [
-              const _SectionHeader(title: 'Environment', icon: Icons.eco),
-              const SizedBox(height: 8),
-              const Card(
-                child: Padding(
-                  padding: EdgeInsets.all(32),
-                  child: Center(child: CircularProgressIndicator()),
-                ),
-              ),
-            ],
-          ),
-          error: (error, _) => Column(
-            crossAxisAlignment: CrossAxisAlignment.start,
-            children: [
-              const _SectionHeader(title: 'Environment', icon: Icons.eco),
-              const SizedBox(height: 8),
-              Card(
-                child: Padding(
-                  padding: const EdgeInsets.all(16),
-                  child: Row(
-                    children: [
-                      Icon(Icons.error_outline, color: colorScheme.error),
-                      const SizedBox(width: 12),
-                      Expanded(
-                        child: Text(
-                          'Failed to load environment data',
-                          style: TextStyle(color: colorScheme.error),
-                        ),
-                      ),
-                      IconButton(
-                        icon: const Icon(Icons.refresh),
-                        onPressed: () => ref.invalidate(environmentProvider),
-                      ),
-                    ],
+          error: (error, _) => Card(
+            child: Padding(
+              padding: const EdgeInsets.all(16),
+              child: Row(
+                children: [
+                  Icon(Icons.error_outline, color: colorScheme.error),
+                  const SizedBox(width: 12),
+                  Expanded(
+                    child: Text(
+                      'Failed to load environment data',
+                      style: TextStyle(color: colorScheme.error),
+                    ),
                   ),
-                ),
+                  IconButton(
+                    icon: const Icon(Icons.refresh),
+                    onPressed: () => ref.invalidate(environmentProvider),
+                  ),
+                ],
               ),
-            ],
+            ),
           ),
         ),
         const SizedBox(height: 24),
@@ -208,12 +200,10 @@ class _SectionHeader extends StatelessWidget {
   const _SectionHeader({
     required this.title,
     required this.icon,
-    this.subtitle,
   });
 
   final String title;
   final IconData icon;
-  final String? subtitle;
 
   @override
   Widget build(BuildContext context) {
@@ -234,15 +224,6 @@ class _SectionHeader extends StatelessWidget {
                 fontWeight: FontWeight.w500,
               ),
         ),
-        if (subtitle != null) ...[
-          const SizedBox(width: 8),
-          Text(
-            subtitle!,
-            style: Theme.of(context).textTheme.labelSmall?.copyWith(
-                  color: colorScheme.outline,
-                ),
-          ),
-        ],
       ],
     );
   }

lib/shared/widgets/sun_position_widget.dart

@@ -386,7 +386,15 @@ class _SunArcPainter extends CustomPainter {
 
     // Calculate radius so arc endpoints touch horizon
     // For a chord of width W and arc angle θ: R = W / (2 * sin(θ/2))
-    final radius = horizonWidth / (2 * math.sin(clampedAngle / 2));
+    var radius = horizonWidth / (2 * math.sin(clampedAngle / 2));
+
+    // Constrain arc height to fit within available space (leave 25px margin for sun)
+    final maxArcHeight = size.height - horizonY - 25;
+    final arcHeight = radius * (1 - math.cos(clampedAngle / 2));
+    if (arcHeight > maxArcHeight) {
+      // Scale radius down to fit
+      radius = maxArcHeight / (1 - math.cos(clampedAngle / 2));
+    }
 
     // Arc center is below the horizon for an upward-bulging arc
     // Distance from chord to center = R * cos(θ/2)
@@ -491,7 +499,15 @@ class _SunArcPainter extends CustomPainter {
     final clampedAngle = arcAngle.clamp(math.pi / 6, math.pi);
 
     // Calculate radius so arc endpoints touch horizon
-    final radius = horizonWidth / (2 * math.sin(clampedAngle / 2));
+    var radius = horizonWidth / (2 * math.sin(clampedAngle / 2));
+
+    // Constrain arc height to fit within available space (leave 20px margin for moon)
+    final maxArcHeight = size.height - horizonY - 20;
+    final arcHeight = radius * (1 - math.cos(clampedAngle / 2));
+    if (arcHeight > maxArcHeight) {
+      // Scale radius down to fit
+      radius = maxArcHeight / (1 - math.cos(clampedAngle / 2));
+    }
 
     // Arc center is below the horizon for an upward-bulging arc
     final horizonYPos = size.height - horizonY;

pubspec.yaml

@@ -16,7 +16,7 @@ publish_to: 'none' # Remove this line if you wish to publish to pub.dev
 # https://developer.apple.com/library/archive/documentation/General/Reference/InfoPlistKeyReference/Articles/CoreFoundationKeys.html
 # In Windows, build-name is used as the major, minor, and patch parts
 # of the product and file versions while build-number is used as the build suffix.
-version: 1.5.4+1
+version: 1.5.6+1
 
 environment:
   sdk: ^3.10.4