Plaid v2 writeback: three independent canary parity fixes

One bundle, three logically independent diffs uncovered by the same canary-run-15 forensic trace — approval ceilings, full parent bodies, and a soft-skip that stops one bad product from blanking entire quotes.

1. Why this exists

Canary-run-15 surfaced a cluster of v1↔v2 writeback diffs on real Plaid quotes. A single forensic trace identified four independent root causes that all live in (or feed) PlaidWriteBack.ts. Each is logically separable; they're bundled because they share a touch surface and were uncovered together.

2. The four fixes at a glance

[
  { "name": "l1price" }
]

[
  { "name": "l1price" },
  { "name": "l2price" },
  { "name": "l3price" },
  { "name": "l4price" }
]

if (activeEntries.length === 0) {
  throw new CrmWriteBackError(
    `... no active pricebook entries ...`,
    'PLAID_PIPELINE_PRICEBOOK_ENTRY_NOT_FOUND',
  );
}

if (activeEntries.length === 0) {
  errorNotificationService.logPagingError(
    `... skipping product ${product.id}`,
  );
  skippedProductIds.add(product.id);
  continue;
}

// Transactions Refresh
"additionalCrmProductIds": ["01t1Q0000088VwCQAU", "01t1Q0000088VwDQAU"]

// Investments T&H
"additionalCrmProductIds": [
  "01t1Q000007UFPFQA4",
  "01t1Q000007UFStQAO",
  "01t1Q000007UFTLQA4"
]

3. How FA2 wires the parent body together

The parent-body field set is the largest single change. It requires plumbing data that wasn't previously in scope at the emitParentIfNeeded call site.

Resolver side — only pay the cost when needed

if (parentProduct2IdsToResolve.size > 0) {
  const parentSfRows = await this.loadSalesforceProductsByProduct2Ids({
    organizationId,
    product2Ids: Array.from(parentProduct2IdsToResolve),
    quoteCurrency,
  });
  for (const [product2Id, row] of parentSfRows) {
    parentSfRowsBySfProductId.set(product2Id, {
      Product2Id: row.Product2Id,
      UnitPrice: row.UnitPrice,
      CurrencyIsoCode: row.CurrencyIsoCode,
      Description: row.Description,
      extraFields: row.extraFields,
    });
  }
}

The size > 0 guard means non-bundle quotes pay zero extra DB cost. The map is keyed by parent SF Product2Id — same key as the existing parentPricebookEntryIdBySfProductId.

Build side — spread order matters

body: {
  // Hardcoded defaults FIRST so extraFields can override on collision
  // (mirrors v1's constructPenguinQuoteLineItemData spread order).
  SBQQ__PriceEditable__c: true,
  SBQQ__SubscriptionType__c: 'Renewable',
  SBQQ__BillingType__c: 'Monthly Usage',
  // ... 5 more constants
  ...safeParentExtraFields,
  // Structural fields LAST so they always win
  SBQQ__Quote__c: quoteRefPlaceholder,
  SBQQ__Product__c: parentSfProductId,
  SBQQ__ListPrice__c: listPriceOverride,
  SBQQ__OriginalPrice__c: parentSfRow?.UnitPrice ?? 0,
  CurrencyIsoCode: parentSfRow?.CurrencyIsoCode ?? pricingQuote.input.targetCurrency,
  SBCF_Approval_Level__c: 0,  // hardcoded, not via resolver
  // ...
}

4. What this PR explicitly does NOT change

5. Tests

6. Intentionally deferred

7. Reviewer attention points