Skip to content

Commit 2e4e9f5

Browse files
committed
fix: address telemetry inspector review
1 parent 67e8e40 commit 2e4e9f5

4 files changed

Lines changed: 10 additions & 28 deletions

File tree

src/app/bridge/iracingSdk/iracingSdkBridge.ts

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -243,7 +243,6 @@ export async function publishIRacingSDKEvents(
243243

244244
let shouldStop = false;
245245
let lastRunningState: boolean | undefined = undefined;
246-
let latestTelemetry: Telemetry | null = null;
247246
let latestSession: Session | null = null;
248247

249248
const telemetryCallbacks = new Set<(value: Telemetry) => void>();
@@ -261,12 +260,6 @@ export async function publishIRacingSDKEvents(
261260
'runningState',
262261
lastRunningState
263262
);
264-
if (latestTelemetry && perfTelemetryDeliveryEnabled)
265-
overlayManager.publishMessageToOverlay(
266-
id,
267-
'telemetryInspector:telemetry',
268-
telemetryForRenderer(latestTelemetry)
269-
);
270263
if (latestSession)
271264
overlayManager.publishMessageToOverlay(id, 'sessionData', latestSession);
272265
});
@@ -333,7 +326,6 @@ export async function publishIRacingSDKEvents(
333326
}
334327

335328
if (telemetry) {
336-
latestTelemetry = telemetry;
337329
perfMetrics.markStart('lifecycleTelemetry');
338330
lifecycle?._onTelemetry(telemetry);
339331
perfMetrics.markEnd('lifecycleTelemetry');
@@ -416,7 +408,6 @@ export async function publishIRacingSDKEvents(
416408
// opened during a disconnect don't get re-seeded with stale data, and
417409
// so the references don't sit in main-process memory indefinitely.
418410
// They get repopulated on the next successful waitForData tick.
419-
latestTelemetry = null;
420411
latestSession = null;
421412
overlayManager.clearLatestSessionData?.();
422413
lifecycle?._onDisconnect();

src/frontend/components/Standings/Standings.stories.tsx

Lines changed: 8 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,6 @@ import {
1212
} from '@irdashies/storybook';
1313
import {
1414
DashboardProvider,
15-
SessionProvider,
16-
StoryTelemetryProvider,
1715
useLapTimesStoreUpdater,
1816
usePitLapStoreUpdater,
1917
useDrivingState,
@@ -25,7 +23,6 @@ import {
2523
useSessionDrivers,
2624
type P2PDisplayState,
2725
} from '@irdashies/context';
28-
import { generateMockDataFromPath } from '../../../app/bridge/iracingSdk/mock-data/generateMockData';
2926
import type {
3027
DashboardBridge,
3128
StandingsConfig,
@@ -1144,28 +1141,22 @@ export const HeaderOnlyAllVisible: Story = {
11441141

11451142
export const CompactMode: Story = {
11461143
decorators: [
1144+
TelemetryDecorator(),
11471145
(Story) => (
1148-
<>
1149-
<SessionProvider bridge={generateMockDataFromPath()} />
1150-
<StoryTelemetryProvider bridge={generateMockDataFromPath()} />
1151-
<DashboardProvider bridge={createMockBridgeWithCompactMode()}>
1152-
<Story />
1153-
</DashboardProvider>
1154-
</>
1146+
<DashboardProvider bridge={createMockBridgeWithCompactMode()}>
1147+
<Story />
1148+
</DashboardProvider>
11551149
),
11561150
],
11571151
};
11581152

11591153
export const CompactUltraMode: Story = {
11601154
decorators: [
1155+
TelemetryDecorator(),
11611156
(Story) => (
1162-
<>
1163-
<SessionProvider bridge={generateMockDataFromPath()} />
1164-
<StoryTelemetryProvider bridge={generateMockDataFromPath()} />
1165-
<DashboardProvider bridge={createMockBridgeWithCompactUltraMode()}>
1166-
<Story />
1167-
</DashboardProvider>
1168-
</>
1157+
<DashboardProvider bridge={createMockBridgeWithCompactUltraMode()}>
1158+
<Story />
1159+
</DashboardProvider>
11691160
),
11701161
],
11711162
};

src/frontend/components/TelemetryInspector/TelemetryInspector.stories.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import type { Meta, StoryObj } from '@storybook/react-vite';
2-
import { TelemetryDecorator } from '../../../../.storybook/telemetryDecorator';
2+
import { TelemetryDecorator } from '@irdashies/storybook';
33
import { TelemetryInspector } from './TelemetryInspector';
44

55
const InspectorStory = () => (

src/frontend/context/DashboardContext/DashboardContext.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -286,7 +286,7 @@ export const DashboardProvider: React.FC<{
286286
export const useDashboard = (): DashboardContextProps => {
287287
const context = useContext(DashboardContext);
288288
if (!context) {
289-
throw new Error('useTelemetry requires a raw telemetry store');
289+
throw new Error('useDashboard must be used within a DashboardProvider');
290290
}
291291
return context;
292292
};

0 commit comments

Comments
 (0)