engineering/tpmjs
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 2.2k

fix: handle null/undefined usage object in scenario runs

Browse source 
- Make usage field nullable in TypeScript interface
- Add optional chaining for run.usage?.executionTimeMs
- Add optional chaining for run.usage?.totalTokens
- Prevents TypeError when usage object is missing
This commit is contained in:
Ajax Davis 2026-01-20 06:04:58 +10:00
parent 474e11fc13
commit 2d939ea11b
1 changed files with 5 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

10
apps/web/src/app/(profile)/[username]/collections/[slug]/scenarios/[id]/page.tsx
View file

@ -24,7 +24,7 @@ interface ScenarioRun {
totalTokens: number | null;
executionTimeMs: number | null;
estimatedCost: number | null;
};
} | null;
timestamps: {
startedAt: string | null;
completedAt: string | null;
@ -361,12 +361,12 @@ export default function CollectionScenarioDetailPage(): React.ReactElement {
<span className="text-sm text-foreground-secondary">
{formatDate(run.timestamps?.createdAt || null)}
</span>
{run.usage.executionTimeMs && (
{run.usage?.executionTimeMs && (
<span className="text-sm text-foreground-tertiary">
{formatDuration(run.usage.executionTimeMs)}
</span>
)}
{run.usage.totalTokens && (
{run.usage?.totalTokens && (
<span className="text-sm text-foreground-tertiary">
{run.usage.totalTokens.toLocaleString()} tokens
</span>
@ -413,13 +413,13 @@ export default function CollectionScenarioDetailPage(): React.ReactElement {
<div>
<span className="text-foreground-tertiary">Duration:</span>{' '}
<span className="text-foreground">
{formatDuration(run.usage.executionTimeMs)}
{formatDuration(run.usage?.executionTimeMs || null)}
</span>
</div>
<div>
<span className="text-foreground-tertiary">Tokens:</span>{' '}
<span className="text-foreground">
{run.usage.totalTokens?.toLocaleString() || '—'}
{run.usage?.totalTokens?.toLocaleString() || '—'}
</span>
</div>
<div>