fix: resolve all biome lint warnings

[coji]

Summary

• Remove unnecessary non-null assertions (!) on organization.slug where the type is already string (6 files)
• Refactor SizeBadgePopover to accept orgSlug as an explicit prop instead of using useParams() internally — eliminates implicit routing context dependency and prevents silently generating malformed URLs
• Refactor ExportDataSection to accept orgSlug as prop instead of useParams()
• Replace MemberLink's silent orgSlug = '' default with an explicit invariant throw
• Remove unused biome-ignore suppression comment in pagination.tsx
• Update biome schema version 2.3.13 → 2.4.7

Test plan

• pnpm lint passes with 0 warnings
• pnpm typecheck passes
• Verify throughput pages (merged/deployed/ongoing) render size badges correctly
• Verify data management export download works
• Verify workload member links navigate correctly

🤖 Generated with Claude Code

[coderabbitai]

📝 Walkthrough

Walkthrough

The PR removes non-null assertions throughout the codebase, primarily on slug properties, and replaces them with either direct property access or nullish coalescing fallbacks. Additionally, the biome.json schema reference is updated and a lint comment is removed.

Changes

Cohort / File(s)	Summary
Non-null assertion replacements app/components/size-badge-popover.tsx	Replaced orgSlug! with orgSlug ?? '' in two action calls, providing an empty string fallback when undefined.
Non-null assertion removals app/routes/$orgSlug/settings/repositories/$repository/delete/index.tsx, app/routes/$orgSlug/settings/repositories/$repository/settings/index.tsx, app/routes/_index.tsx, app/routes/admin/_index/index.tsx, app/routes/resources/organization.tsx	Removed non-null assertions from slug property accesses in redirect URLs and href constructions, allowing properties to remain potentially undefined.
Configuration and comment cleanup biome.json, app/components/ui/pagination.tsx	Updated biome schema version from 2.3.13 to 2.4.7 and removed a biome-ignore lint comment.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• feat: improve size feedback popover UX #154: Modifies app/components/size-badge-popover.tsx to adjust hasFeedback logic, touching the same component as this PR.
• refactor: promote shared modules to app/libs and app/components #185: Modifies app/components/size-badge-popover.tsx imports, overlapping with this PR's changes to the same file.

Poem

🐰 Assertions gone, the code runs free,
No more exclamation marks to see!
Nullish coalescing saves the day,
While biome schemas update their way. ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title directly matches the main objective: resolving all biome lint warnings by removing non-null assertions, replacing unsafe patterns, and updating schema version.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

• 📝 Generate docstrings (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/lint-warnings

📝 Coding Plan

• Generate coding plan for human review comments

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}