check for source in transaction planner request (#1499)

* assert validity of source in transaction request

* pass tests

packages/services/src/view-service/transaction-planner/assert-transaction-source.ts

@@ -0,0 +1,12 @@
+import { TransactionPlannerRequest } from '@buf/penumbra-zone_penumbra.bufbuild_es/penumbra/view/v1/view_pb.js';
+import { Code, ConnectError } from '@connectrpc/connect';
+
+export const assertTransactionSource = (transactionPlannerRequest: TransactionPlannerRequest) => {
+  // Ensure that a source is provided in the transaction request.
+  if (!transactionPlannerRequest.source) {
+    throw new ConnectError(
+      'Source is required in the TransactionPlannerRequest',
+      Code.InvalidArgument,
+    );
+  }
+};

packages/services/src/view-service/transaction-planner/index.test.ts

@@ -18,6 +18,7 @@ import {
   AssetId,
   Value,
 } from '@buf/penumbra-zone_penumbra.bufbuild_es/penumbra/core/asset/v1/asset_pb.js';
+import { AddressIndex } from '@buf/penumbra-zone_penumbra.bufbuild_es/penumbra/core/keys/v1/keys_pb.js';
 
 const mockPlanTransaction = vi.hoisted(() => vi.fn());
 vi.mock('@penumbra-zone/wasm/planner', () => ({
@@ -60,7 +61,9 @@ describe('TransactionPlanner request handler', () => {
         .set(fvkCtx, () => Promise.resolve(testFullViewingKey)),
     });
 
-    req = new TransactionPlannerRequest({});
+    req = new TransactionPlannerRequest({
+      source: new AddressIndex({ account: 0 }),
+    });
   });
 
   test('should throw if request is not valid', async () => {

packages/services/src/view-service/transaction-planner/index.ts

@@ -6,6 +6,7 @@ import { assertSwapAssetsAreNotTheSame } from './assert-swap-assets-are-not-the-
 import { TransactionPlannerRequest } from '@buf/penumbra-zone_penumbra.bufbuild_es/penumbra/view/v1/view_pb.js';
 import { fvkCtx } from '../../ctx/full-viewing-key.js';
 import { extractAltFee } from '../fees.js';
+import { assertTransactionSource } from './assert-transaction-source.js';
 
 export const transactionPlanner: Impl['transactionPlanner'] = async (req, ctx) => {
   assertValidRequest(req);
@@ -66,4 +67,5 @@ export const transactionPlanner: Impl['transactionPlanner'] = async (req, ctx) =
  */
 const assertValidRequest = (req: TransactionPlannerRequest): void => {
   assertSwapAssetsAreNotTheSame(req);
+  assertTransactionSource(req);
 };