Fix locale default, retention performance, console logs, and test cleanup

[Copilot]

Addresses 6 code quality and consistency issues identified in PR review.

Changes

• Store locale default: Changed CreateStoreSchema locale from 'en' to 'bn' to match Prisma schema default
• Retention calculation performance: Replaced O(n²) array includes() with O(1) Set.has() lookup

  // Before: O(n²)
  const retained = customerIdsInPeriod.filter(id => previousPeriodCustomerIds.includes(id));
  
  // After: O(n)
  const previousPeriodSet = new Set(previousPeriodCustomerIds);
  const retained = customerIdsInPeriod.filter(id => previousPeriodSet.has(id));

• Production console cleanup: Removed debug console.log statements from analytics dashboard client component
• React Compiler pattern: Restored eslint-disable directive for useReactTable to match established pattern in data-table.tsx
• Playwright resource cleanup: Changed test teardown to close browser context (not just page) to prevent resource leaks
• Free shipping threshold: Corrected threshold from 50 to 5000 BDT to match UI documentation

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
stormcomui	Ready	Preview, Comment	Jan 30, 2026 6:59pm