fix(e2e): waitForTable waits for 'N companies' count text

The previous spinner-wait approach (c7e8953) had a budget problem:
30s spinner wait + 30s selector wait could total 60s inside a 30s
test timeout. Tests died in beforeEach before assertions ran.

Replace with a single wait for the "N companies" count text inside
[data-testid="company-table"]. This text only renders after
isLoading becomes false and data is available. Single 30s budget,
no multi-stage timeout stacking.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: YourGitHubUsername

tests/e2e/pages/CompaniesPage.ts

@@ -64,23 +64,13 @@ export class CompaniesPage extends BasePage {
    * Wait for the companies table to load
    */
   async waitForTable() {
-    // Wait for the table container to appear
-    await this.page.waitForSelector(this.selectors.table, {
-      state: 'visible',
-      timeout: 30000,
-    });
-    // CompanyTable renders the same data-testid on both the loading spinner
-    // and the loaded table. Wait for the loading spinner inside the table to
-    // disappear — that means the Supabase query completed and real rows are
-    // rendered (or "No companies yet" if the user has zero companies).
+    // CompanyTable renders [data-testid="company-table"] on both the loading
+    // spinner and the loaded table. Wait for the "N companies" count text
+    // which only appears after isLoading becomes false and data is rendered.
     await this.page
-      .locator(`${this.selectors.table} .loading`)
-      .first()
-      .waitFor({ state: 'hidden', timeout: 30000 })
-      .catch(() => {
-        // Spinner may never appear if data loads before the selector check
-      });
-    await this.waitForLoad();
+      .locator(`${this.selectors.table}`)
+      .getByText(/\d+ compan/i)
+      .waitFor({ state: 'visible', timeout: 30000 });
   }
 
   /**