Mass update for all lucide icon imports to use the component with Icon suffix (#51659)

## Context

This is largely just a mechanical change here:
Opting to use lucide icons with the `Icon` suffix across all icon
imports within `studio`
- so that it’s clearer which components are icons + reduces the
possibility of name collision with our own components
- e.g <User /> vs <UserIcon />

Also added a eslint rule for this:
<img width="1069" height="167" alt="image"
src="https://github.com/user-attachments/assets/7f36f443-5ca2-4bf6-8d22-86e774513bfc"
/>

And updated `AGENTS.md` as such:
```
Icons come from `lucide-react`, always imported with the `Icon` suffix (`import { CheckIcon, Trash2Icon } from 'lucide-react'`) — never the bare names (`Check`) or `Check as CheckIcon` aliases (lint-enforced).
```

apps/studio/components/ui/DataTable/DataTableColumn/DataTableColumnStatusCode.tsx

@@ -1,4 +1,4 @@
-import { Minus } from 'lucide-react'
+import { MinusIcon } from 'lucide-react'
 import { cn } from 'ui'
 
 export const DataTableColumnStatusCode = ({
@@ -33,7 +33,7 @@ export const DataTableColumnStatusCode = ({
   }
 
   if (!value) {
-    return <Minus className="h-4 w-4 text-muted-foreground/50" />
+    return <MinusIcon className="h-4 w-4 text-muted-foreground/50" />
   }
 
   return (