[Feature]: Onboarding Profiling: Embed profiler in Asset Review editor

Parent PRD

#172

What to build

Integrate the TableProfiler component into the Asset Review editor as the primary tab for table and view assets, replacing the existing basic preview.

Changes to AssetReviewEditor:

• For assets of type table or view: render TableProfiler as the primary/default tab
• The existing "preview" panel (basic SELECT *) is subsumed by the profiler's sample functionality
• Existing "definition" tab (for views) and "Analyze with AI" become secondary tabs
• For view assets: the profiler shows the info banner ("This is a view — profiling will execute the view's query against the warehouse")
• For non-table/view asset types: existing behavior unchanged (no profiler tab)

Props wiring:

• Pass fqn from the reviewed asset's asset_fqn
• Pass assetType from the reviewed asset's asset_type
• Pass initialSchema if the review editor already fetched column metadata (avoid duplicate UC API call)

Acceptance criteria

• Opening a table asset in the review editor shows the profiler as the first/primary tab
• Opening a view asset shows the profiler with the view warning banner
• The existing basic preview is no longer shown separately (profiler's sample tab replaces it)
• View definition and AI analysis are available as secondary tabs
• Non-table/view assets (function, model, etc.) are unaffected — no profiler tab appears
• Profiler errors (no warehouse, permission denied) are contained within the profiler panel and don't break the review editor

Blocked by

• Blocked by #175 (Profiling tracer bullet)
• Blocked by #173 (Fix critical bugs — ensures clean editor baseline)

User stories addressed

• User story 10
• User story 13
• User story 22

Start at AssetReviewEditor and TableProfiler, tracing the existing preview, definition, and AI-analysis tabs plus the asset_fqn and asset_type data. Check blockers #175 and #173 first; done means table and view assets show the profiler first, other tabs remain available, non-table/view behavior is unchanged, and profiler errors stay contained.