fix: ci failure

Author: 0xNinx

.github/workflows/ci.yml

@@ -22,9 +22,14 @@ jobs:
       - name: Checkout code
         uses: actions/checkout@v4
 
+      # Pinned to 1.96.0: Rust 1.97 changed the size of core::num::TryFromIntError,
+      # which breaks the (unmaintained, transitive) ethnum 1.5.2 crate with E0512
+      # ("cannot transmute between types of different sizes"). No fixed ethnum
+      # release exists yet, so pin below 1.97 until soroban's dep tree drops it.
       - name: Install Rust
-        uses: dtolnay/rust-toolchain@stable
+        uses: dtolnay/rust-toolchain@1.96.0
         with:
+          components: rustfmt, clippy
           targets: wasm32-unknown-unknown, wasm32v1-none
 
       - name: Ensure wasm32v1-none target is available

.github/workflows/deploy.yml

@@ -36,8 +36,11 @@ jobs:
       - name: Checkout code
         uses: actions/checkout@v4
 
+      # Pinned to 1.96.0: Rust 1.97 changed core::num::TryFromIntError's size,
+      # breaking the transitive ethnum 1.5.2 crate (E0512). Keep below 1.97 until
+      # soroban's dependency tree no longer pulls the broken ethnum.
       - name: Install Rust
-        uses: dtolnay/rust-toolchain@stable
+        uses: dtolnay/rust-toolchain@1.96.0
         with:
           targets: wasm32-unknown-unknown
 

.github/workflows/release.yml

@@ -24,8 +24,9 @@ jobs:
           node-version: "20"
 
       # Install Rust
+      # Pinned to 1.96.0: Rust 1.97+ breaks the transitive ethnum 1.5.2 crate (E0512).
       - name: Install Rust toolchain
-        uses: dtolnay/rust-toolchain@stable
+        uses: dtolnay/rust-toolchain@1.96.0
 
       # Add WASM target
       - name: Add wasm target

frontend/components/digital_twin/HealthGauge.tsx

@@ -1,6 +1,6 @@
 'use client';
 
-import React, { useEffect, useState } from 'react';
+import React, { useCallback, useEffect, useState } from 'react';
 import { Card } from '@/components/ui/card';
 import { Badge } from '@/components/ui/badge';
 import { Skeleton } from '@/components/ui/skeleton';
@@ -14,7 +14,7 @@ interface HealthMetric {
   threshold_max?: number;
   severity?: string;
   calculated_at: string;
-  metadata: any;
+  metadata: Record<string, unknown>;
 }
 
 interface HealthGaugeProps {
@@ -27,13 +27,7 @@ export function HealthGauge({ twinId, className = '' }: HealthGaugeProps) {
   const [loading, setLoading] = useState(true);
   const [error, setError] = useState<string | null>(null);
 
-  useEffect(() => {
-    fetchHealthMetrics();
-    const interval = setInterval(fetchHealthMetrics, 30000); // Refresh every 30s
-    return () => clearInterval(interval);
-  }, [twinId]);
-
-  const fetchHealthMetrics = async () => {
+  const fetchHealthMetrics = useCallback(async () => {
     try {
       const response = await fetch(`/api/v1/physics/twins/${twinId}/health-metrics`);
       if (!response.ok) throw new Error('Failed to fetch health metrics');
@@ -44,20 +38,16 @@ export function HealthGauge({ twinId, className = '' }: HealthGaugeProps) {
       setError(err instanceof Error ? err.message : 'Unknown error');
       setLoading(false);
     }
-  };
+  }, [twinId]);
 
-  const getSeverityColor = (severity?: string) => {
-    switch (severity) {
-      case 'normal':
-        return 'bg-green-500';
-      case 'warning':
-        return 'bg-yellow-500';
-      case 'critical':
-        return 'bg-red-500';
-      default:
-        return 'bg-gray-500';
-    }
-  };
+  useEffect(() => {
+    // setState happens only after an await inside fetchHealthMetrics, so this is
+    // an async data fetch, not a synchronous cascading render.
+    // eslint-disable-next-line react-hooks/set-state-in-effect
+    fetchHealthMetrics();
+    const interval = setInterval(fetchHealthMetrics, 30000); // Refresh every 30s
+    return () => clearInterval(interval);
+  }, [fetchHealthMetrics]);
 
   const getSeverityIcon = (severity?: string) => {
     switch (severity) {

frontend/components/digital_twin/TwinView.tsx

@@ -1,28 +1,45 @@
 'use client';
 
-import React, { useEffect, useState } from 'react';
+import React, { useCallback, useEffect, useState } from 'react';
 import { Card } from '@/components/ui/card';
 import { Badge } from '@/components/ui/badge';
 import { Button } from '@/components/ui/button';
 import { Skeleton } from '@/components/ui/skeleton';
 import { LineChart, Line, XAxis, YAxis, CartesianGrid, Tooltip, Legend, ResponsiveContainer, AreaChart, Area, BarChart, Bar } from 'recharts';
-import { Activity, TrendingUp, Calendar, Clock, AlertTriangle, Play, Thermometer } from 'lucide-react';
+import { Activity, Calendar, Clock, AlertTriangle, Play, Thermometer } from 'lucide-react';
+
+interface TwinStateData {
+  temperature?: number;
+  humidity?: number;
+}
+
+interface TwinMetrics {
+  health_score?: number;
+  decay_rate?: number;
+  temperature_stress?: number;
+  humidity_stress?: number;
+}
+
+interface HealthHistoryEntry {
+  timestamp: string;
+  score: number;
+}
 
 interface TwinState {
   id: string;
-  state_data: any;
-  metrics: any;
+  state_data: TwinStateData;
+  metrics: TwinMetrics;
   timestamp: string;
   source: string;
 }
 
 interface DigitalTwin {
   id: string;
   name: string;
-  current_state: any;
+  current_state: TwinStateData;
   current_health_score?: number;
   predicted_expiry_date?: string;
-  health_history?: any[];
+  health_history?: HealthHistoryEntry[];
 }
 
 interface TwinViewProps {
@@ -37,13 +54,7 @@ export function TwinView({ twinId, className = '' }: TwinViewProps) {
   const [error, setError] = useState<string | null>(null);
   const [viewMode, setViewMode] = useState<'health' | 'temperature' | 'decay' | 'metrics'>('health');
 
-  useEffect(() => {
-    fetchTwinData();
-    const interval = setInterval(fetchTwinData, 15000); // Refresh every 15s
-    return () => clearInterval(interval);
-  }, [twinId]);
-
-  const fetchTwinData = async () => {
+  const fetchTwinData = useCallback(async () => {
     try {
       const [twinRes, historyRes] = await Promise.all([
         fetch(`/api/v1/digital-twins/${twinId}`),
@@ -62,12 +73,21 @@ export function TwinView({ twinId, className = '' }: TwinViewProps) {
       setError(err instanceof Error ? err.message : 'Unknown error');
       setLoading(false);
     }
-  };
+  }, [twinId]);
+
+  useEffect(() => {
+    // setState happens only after an await inside fetchTwinData, so this is an
+    // async data fetch, not a synchronous cascading render.
+    // eslint-disable-next-line react-hooks/set-state-in-effect
+    fetchTwinData();
+    const interval = setInterval(fetchTwinData, 15000); // Refresh every 15s
+    return () => clearInterval(interval);
+  }, [fetchTwinData]);
 
   const getHealthChartData = () => {
     if (!twin?.health_history) return [];
-    
-    return twin.health_history.map((entry: any) => ({
+
+    return twin.health_history.map((entry) => ({
       timestamp: new Date(entry.timestamp).toLocaleTimeString(),
       score: Math.round(entry.score * 100),
     }));

frontend/components/layouts/NavBar.test.tsx

@@ -1,5 +1,5 @@
 import { render, screen, fireEvent } from '@testing-library/react';
-import { describe, it, expect, vi } from 'vitest';
+import { describe, it, expect } from 'vitest';
 import { NavBar } from './NavBar';
 
 // WalletStatus is mocked via setup.ts global mock