fix(example-crm): author SalesByAccountReport via defineReport (lint #2035) (#2108)

The fixture added in #2107 used a bare `: UI.ReportInput` typed literal, which
the `no-restricted-syntax` rule (#2035) disallows — metadata must go through its
`defineX` factory so it validates at parse time. The sibling sales-by-stage
report already uses `defineReport`; this aligns the new one. The CI lint only
checks changed files, so #2107 landed with the violation — this fixes it on main.

No behavior change: same exported `SalesByAccountReport`, same fields.

Co-authored-by: Jack Zhuang <277994282+os-zhuang@users.noreply.github.com>
Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>

Author: 3 people

examples/app-crm/src/reports/sales-by-account.report.ts

@@ -1,6 +1,6 @@
 // Copyright (c) 2025 ObjectStack. Licensed under the Apache-2.0 license.
 
-import type * as UI from '@objectstack/spec/ui';
+import { defineReport } from '@objectstack/spec/ui';
 
 /**
  * Example report — total opportunity amount grouped by ACCOUNT.
@@ -13,12 +13,12 @@ import type * as UI from '@objectstack/spec/ui';
  * `total_amount` measure (USD), this exercises both render paths — Intl
  * currency formatting AND raw-value lookup drill — end to end.
  */
-export const SalesByAccountReport: UI.ReportInput = {
+export const SalesByAccountReport = defineReport({
   name: 'crm_sales_by_account',
   label: 'Sales by Account',
   description: 'Total opportunity amount grouped by account (lookup-dimension drill).',
   type: 'summary',
   dataset: 'opportunity_metrics',
   rows: ['account'],
   values: ['total_amount'],
-};
+});