@open-mercato/core 0.10.1-develop.7954.1.48da4a7d13 → 0.10.1-develop.7972.1.03f813eaaf

package/src/modules/warranty_claims/backend/warranty_claims/[id]/page.tsx

@@ -6,6 +6,7 @@ import { useRouter } from 'next/navigation'
 import type { LegacyColumnDef as ColumnDef } from '@tanstack/react-table/legacy'
 import { Copy, Hash, Info, MessageSquare, Pencil, RefreshCw, Trash2, UserRound } from 'lucide-react'
 import { hasFeature } from '@open-mercato/shared/security/features'
+import { getTelemetryRuntime } from '@open-mercato/shared/lib/telemetry/runtime'
 import { parseBooleanFromUnknown } from '@open-mercato/shared/lib/boolean'
 import type { TranslateFn, TranslateParams } from '@open-mercato/shared/lib/i18n/context'
 import { useLocale, useT } from '@open-mercato/shared/lib/i18n/context'
@@ -783,6 +784,7 @@ export default function WarrantyClaimDetailPage({ params }: { params?: { id?: st
   const [lineDialog, setLineDialog] = React.useState<{ mode: 'create' } | { mode: 'edit'; line: ClaimLine } | null>(null)
   const [defaultWarrantyMonths, setDefaultWarrantyMonths] = React.useState<number | null>(null)
   const [featureAccess, setFeatureAccess] = React.useState({
+    attachmentsView: false,
     claimManage: false,
     claimDelete: false,
     receivingManage: false,
@@ -814,6 +816,7 @@ export default function WarrantyClaimDetailPage({ params }: { params?: { id?: st
         headers: { 'content-type': 'application/json' },
         body: JSON.stringify({
           features: [
+            'attachments.view',
             'warranty_claims.claim.manage',
             'warranty_claims.claim.delete',
             'warranty_claims.receiving.manage',
@@ -830,6 +833,7 @@ export default function WarrantyClaimDetailPage({ params }: { params?: { id?: st
         const granted = Array.isArray(call.result?.granted) ? call.result.granted : []
         const allGranted = call.result?.ok === true
         setFeatureAccess({
+          attachmentsView: allGranted || hasFeature(granted, 'attachments.view'),
           claimManage: allGranted || hasFeature(granted, 'warranty_claims.claim.manage'),
           claimDelete: allGranted || hasFeature(granted, 'warranty_claims.claim.delete'),
           receivingManage: allGranted || hasFeature(granted, 'warranty_claims.receiving.manage'),
@@ -841,6 +845,7 @@ export default function WarrantyClaimDetailPage({ params }: { params?: { id?: st
       .catch(() => {
         if (!cancelled) {
           setFeatureAccess({
+            attachmentsView: false,
             claimManage: false,
             claimDelete: false,
             receivingManage: false,
@@ -925,7 +930,7 @@ export default function WarrantyClaimDetailPage({ params }: { params?: { id?: st
     setError(null)
     setNotFound(false)
     try {
-      const [claimPayload, linesPayload, eventsPayload, riskPayload, statsPayload, attachmentsPayload] = await Promise.all([
+      const [claimPayload, linesPayload, eventsPayload, riskPayload, statsPayload] = await Promise.all([
         readApiResultOrThrow<{ items?: unknown[] }>(
           `/api/warranty_claims?ids=${encodeURIComponent(id)}&page=1&pageSize=1`,
           undefined,
@@ -950,11 +955,6 @@ export default function WarrantyClaimDetailPage({ params }: { params?: { id?: st
           undefined,
           { fallback: { ok: true, result: {} } },
         ),
-        readApiResultOrThrow<{ items?: unknown[] }>(
-          `/api/attachments?entityId=${encodeURIComponent('warranty_claims:warranty_claim')}&recordId=${encodeURIComponent(id)}`,
-          undefined,
-          { fallback: { items: [] } },
-        ),
       ])
       const nextClaim = (claimPayload.items ?? []).map(normalizeClaim).find((item): item is ClaimRecord => item !== null) ?? null
       if (!nextClaim) {
@@ -969,7 +969,6 @@ export default function WarrantyClaimDetailPage({ params }: { params?: { id?: st
       setClaim(nextClaim)
       setLines((linesPayload.items ?? []).map(normalizeLine).filter((item): item is ClaimLine => item !== null))
       setEvents((eventsPayload.items ?? []).map(normalizeEvent).filter((item): item is ClaimEvent => item !== null))
-      setAttachmentCount(Array.isArray(attachmentsPayload.items) ? attachmentsPayload.items.length : 0)
       setRiskAssessment(normalizeRiskResponse(riskPayload))
       const nextThreshold = statsPayload?.result?.slaAtRiskThresholdPct
       setSlaAtRiskThresholdPct(typeof nextThreshold === 'number' ? nextThreshold : undefined)
@@ -984,6 +983,30 @@ export default function WarrantyClaimDetailPage({ params }: { params?: { id?: st
     void loadData()
   }, [loadData])
 
+  React.useEffect(() => {
+    let cancelled = false
+    setAttachmentCount(0)
+    if (!claim || claim.id !== id || !featureAccess.attachmentsView) return
+    void apiCall<{ items?: unknown[] }>(
+      `/api/attachments?entityId=${encodeURIComponent('warranty_claims:warranty_claim')}&recordId=${encodeURIComponent(claim.id)}`,
+      { headers: { 'x-om-forbidden-redirect': '0' } },
+      { fallback: { items: [] } },
+    )
+      .then((call) => {
+        if (cancelled || !call.ok) return
+        setAttachmentCount(Array.isArray(call.result?.items) ? call.result.items.length : 0)
+      })
+      .catch((error) => {
+        getTelemetryRuntime()?.reportError(error, {
+          module: 'warranty_claims',
+          code: 'warranty_claims.attachment_count_failed',
+        })
+      })
+    return () => {
+      cancelled = true
+    }
+  }, [claim, featureAccess.attachmentsView, id, scopeVersion])
+
   useAppEvent('warranty_claims.claim.*', (event) => {
     const eventClaimId = toStringOrNull(event.payload.claimId)
     if (eventClaimId === id) void loadData()
@@ -1908,7 +1931,9 @@ export default function WarrantyClaimDetailPage({ params }: { params?: { id?: st
   const tabs: Array<{ id: TabId; label: string; count?: number }> = [
     { id: 'lines', label: t('warranty_claims.detail.tabs.lines'), count: lines.length },
     { id: 'timeline', label: t('warranty_claims.detail.tabs.timeline'), count: events.length },
-    { id: 'attachments', label: t('warranty_claims.detail.tabs.attachments'), count: attachmentCount },
+    ...(featureAccess.attachmentsView
+      ? [{ id: 'attachments' as const, label: t('warranty_claims.detail.tabs.attachments'), count: attachmentCount }]
+      : []),
     { id: 'ai', label: t('warranty_claims.detail.tabs.ai') },
   ]
   const approvedLineCount = lines.filter((line) => line.lineStatus === 'approved').length
@@ -2046,7 +2071,7 @@ export default function WarrantyClaimDetailPage({ params }: { params?: { id?: st
             </div>
             <ClaimStageProgress status={claim.status} />
           <Tabs
-            value={activeTab}
+            value={activeTab === 'attachments' && !featureAccess.attachmentsView ? 'lines' : activeTab}
             onValueChange={(value) => setActiveTab(value as TabId)}
             variant="underline"
           >
@@ -2249,25 +2274,27 @@ export default function WarrantyClaimDetailPage({ params }: { params?: { id?: st
               </div>
             </TabsContent>
 
-            <TabsContent value="attachments" className="mt-0">
-              <div className="px-8 pb-8 pt-6">
-                <AttachmentInput
-                  entityId="warranty_claims:warranty_claim"
-                  recordId={claim.id}
-                  allowDelete={featureAccess.claimManage}
-                  allowReplace={featureAccess.claimManage}
-                  disabled={!featureAccess.claimManage}
-                  onCountChange={setAttachmentCount}
-                  renderItemMeta={(item) => (
-                    isCustomerVisibleAttachment(item.tags) ? (
-                      <StatusBadge variant="info">
-                        {t('warranty_claims.detail.attachments.customerVisible', 'Visible to customer')}
-                      </StatusBadge>
-                    ) : null
-                  )}
-                />
-              </div>
-            </TabsContent>
+            {featureAccess.attachmentsView ? (
+              <TabsContent value="attachments" className="mt-0">
+                <div className="px-8 pb-8 pt-6">
+                  <AttachmentInput
+                    entityId="warranty_claims:warranty_claim"
+                    recordId={claim.id}
+                    allowDelete={featureAccess.claimManage}
+                    allowReplace={featureAccess.claimManage}
+                    disabled={!featureAccess.claimManage}
+                    onCountChange={setAttachmentCount}
+                    renderItemMeta={(item) => (
+                      isCustomerVisibleAttachment(item.tags) ? (
+                        <StatusBadge variant="info">
+                          {t('warranty_claims.detail.attachments.customerVisible', 'Visible to customer')}
+                        </StatusBadge>
+                      ) : null
+                    )}
+                  />
+                </div>
+              </TabsContent>
+            ) : null}
 
             <TabsContent value="ai" className="mt-0">
               <div className="grid gap-4 px-8 pb-8 pt-6 xl:grid-cols-2">

package/src/modules/warranty_claims/setup.ts

@@ -76,6 +76,42 @@ function stableSlaSweepScheduleId(organizationId: string): string {
   return stableScheduleUuid(`warranty_claims:sla-sweep:${organizationId}`)
 }
 
+async function ensureSlaSweepSchedule(ctx: {
+  container: import('awilix').AwilixContainer
+  tenantId: string
+  organizationId: string
+}): Promise<void> {
+  const cradle = ctx.container as { hasRegistration?: (name: string) => boolean }
+  if (typeof cradle.hasRegistration !== 'function' || !cradle.hasRegistration('schedulerService')) return
+
+  const schedulerService = ctx.container.resolve('schedulerService') as SchedulerServiceLike
+  try {
+    await schedulerService.register({
+      id: stableSlaSweepScheduleId(ctx.organizationId),
+      name: 'Warranty claims SLA sweep',
+      description: 'Enqueues tenant-scoped warranty claim SLA escalation sweeps every 900 seconds.',
+      scopeType: 'organization',
+      organizationId: ctx.organizationId,
+      tenantId: ctx.tenantId,
+      scheduleType: 'interval',
+      scheduleValue: '900s',
+      timezone: 'UTC',
+      targetType: 'queue',
+      targetQueue: 'warranty_claims.sla_sweep',
+      targetPayload: {
+        scope: { tenantId: ctx.tenantId, organizationId: ctx.organizationId },
+      },
+      sourceType: 'module',
+      sourceModule: 'warranty_claims',
+      isEnabled: true,
+    })
+  } catch (error) {
+    logger.warn('[warranty_claims] Failed to register SLA sweep schedule', {
+      error: error instanceof Error ? error.message : error,
+    })
+  }
+}
+
 export const setup: ModuleSetupConfig = {
   defaultRoleFeatures: {
     admin: adminFeatures,
@@ -117,37 +153,11 @@ export const setup: ModuleSetupConfig = {
 
   async seedDefaults({ em, tenantId, organizationId, container }) {
     await seedWarrantyClaimDictionaries(em, { tenantId, organizationId })
-    const cradle = container as { hasRegistration?: (name: string) => boolean }
-    if (typeof cradle.hasRegistration !== 'function' || !cradle.hasRegistration('schedulerService')) {
-      return
-    }
+    await ensureSlaSweepSchedule({ container, tenantId, organizationId })
+  },
 
-    const schedulerService = container.resolve('schedulerService') as SchedulerServiceLike
-    try {
-      await schedulerService.register({
-        id: stableSlaSweepScheduleId(organizationId),
-        name: 'Warranty claims SLA sweep',
-        description: 'Enqueues tenant-scoped warranty claim SLA escalation sweeps every 900 seconds.',
-        scopeType: 'organization',
-        organizationId,
-        tenantId,
-        scheduleType: 'interval',
-        scheduleValue: '900s',
-        timezone: 'UTC',
-        targetType: 'queue',
-        targetQueue: 'warranty_claims.sla_sweep',
-        targetPayload: {
-          scope: { tenantId, organizationId },
-        },
-        sourceType: 'module',
-        sourceModule: 'warranty_claims',
-        isEnabled: true,
-      })
-    } catch (error) {
-      logger.warn('[warranty_claims] Failed to register SLA sweep schedule', {
-        error: error instanceof Error ? error.message : error,
-      })
-    }
+  async onOrganizationCreated({ container, tenantId, organizationId }) {
+    await ensureSlaSweepSchedule({ container, tenantId, organizationId })
   },
 }
 

package/src/modules/workflows/setup.ts

@@ -95,6 +95,14 @@ export const setup: ModuleSetupConfig = {
     })
   },
 
+  onOrganizationCreated: async (ctx) => {
+    await ensureMetricRollupSchedule({
+      container: ctx.container as never,
+      tenantId: ctx.tenantId,
+      organizationId: ctx.organizationId,
+    })
+  },
+
   /**
    * Portal grants for the §6.4 portal task surface.
    *