fix: spot-perp netPnl now includes spot PnL (was perp-only)

Hiksang · claude · Hiksang · commit eab035b3506f · 2026-03-19T12:02:48.000+09:00
netPnl was only counting perp unrealizedPnl, ignoring spot price change.
For delta-neutral positions this inflated PnL by the perp leg's gain.
Now correctly: spotPnL + perpPnL + funding - fees.

Co-Authored-By: Claude Opus 4.6 (1M context) &lt;noreply@anthropic.com&gt;
diff --git a/src/commands/arb/index.ts b/src/commands/arb/index.ts
@@ -475,7 +475,7 @@ export function registerArbManageCommands(
               holdDurationMs,
               estimatedFundingIncome,
               estimatedFees: totalFees,
-              netPnl: shortPos.unrealizedPnl + estimatedFundingIncome - totalFees,
+              netPnl: (spotValueUsd - shortPos.entryPrice * spotAmount) + shortPos.unrealizedPnl + estimatedFundingIncome - totalFees,
               dailyFundingEstimate,
             });
           }

Original file line number	Diff line number	Diff line change
`@@ -475,7 +475,7 @@ export function registerArbManageCommands(`
`475`	`475`	`holdDurationMs,`
`476`	`476`	`estimatedFundingIncome,`
`477`	`477`	`estimatedFees: totalFees,`
`478`		`- netPnl: shortPos.unrealizedPnl + estimatedFundingIncome - totalFees,`
	`478`	`+ netPnl: (spotValueUsd - shortPos.entryPrice * spotAmount) + shortPos.unrealizedPnl + estimatedFundingIncome - totalFees,`
`479`	`479`	`dailyFundingEstimate,`
`480`	`480`	`});`
`481`	`481`	`}`