feat(web): add tree shaking for lucide-icons

Author: leonardcser

apps/api/src/db/queries.ts

@@ -29,11 +29,9 @@ import {
   executions,
   type ExecutionStatusType,
   type MembershipInsert,
-  type MembershipRow,
   memberships,
   type OrganizationInsert,
   OrganizationRole,
-  type OrganizationRoleType,
   organizations,
   Plan,
   type PlanType,
@@ -1703,204 +1701,3 @@ export async function deleteOrganization(
 
   return !!deletedOrganization;
 }
-
-/**
- * Add or update a user's membership in an organization (only admins and owners can do this)
- *
- * @param db Database instance
- * @param organizationIdOrHandle Organization ID or handle
- * @param targetUserId User ID to add/update membership for
- * @param role Role to assign (member, admin, owner)
- * @param adminUserId User ID of the admin/owner making the change
- * @returns The created or updated membership record, or null if permission denied
- */
-export async function addOrUpdateMembership(
-  db: ReturnType<typeof createDatabase>,
-  organizationIdOrHandle: string,
-  targetUserId: string,
-  role: OrganizationRoleType,
-  adminUserId: string
-): Promise<MembershipRow | null> {
-  // First, verify the admin user has permission (admin or owner)
-  const [adminMembership] = await db
-    .select()
-    .from(memberships)
-    .innerJoin(organizations, eq(memberships.organizationId, organizations.id))
-    .where(
-      and(
-        eq(memberships.userId, adminUserId),
-        getOrganizationCondition(organizationIdOrHandle),
-        inArray(memberships.role, [
-          OrganizationRole.ADMIN,
-          OrganizationRole.OWNER,
-        ])
-      )
-    )
-    .limit(1);
-
-  if (!adminMembership) {
-    return null; // Admin user doesn't have permission
-  }
-
-  // Additional check: only owners can assign the owner role
-  if (
-    role === OrganizationRole.OWNER &&
-    adminMembership.memberships.role !== OrganizationRole.OWNER
-  ) {
-    return null; // Only owners can assign owner role
-  }
-
-  const organizationId = adminMembership.organizations.id;
-  const now = new Date();
-
-  // Check if the target user is already a member
-  const [existingMembership] = await db
-    .select()
-    .from(memberships)
-    .where(
-      and(
-        eq(memberships.userId, targetUserId),
-        eq(memberships.organizationId, organizationId)
-      )
-    )
-    .limit(1);
-
-  if (existingMembership) {
-    // Update existing membership
-    const [updatedMembership] = await db
-      .update(memberships)
-      .set({
-        role,
-        updatedAt: now,
-      })
-      .where(
-        and(
-          eq(memberships.userId, targetUserId),
-          eq(memberships.organizationId, organizationId)
-        )
-      )
-      .returning();
-
-    return updatedMembership;
-  } else {
-    // Create new membership
-    const newMembership: MembershipInsert = {
-      userId: targetUserId,
-      organizationId,
-      role,
-      createdAt: now,
-      updatedAt: now,
-    };
-
-    const [createdMembership] = await db
-      .insert(memberships)
-      .values(newMembership)
-      .returning();
-
-    return createdMembership;
-  }
-}
-
-/**
- * Delete a user's membership from an organization (only admins and owners can do this)
- *
- * @param db Database instance
- * @param organizationIdOrHandle Organization ID or handle
- * @param targetUserId User ID to remove from the organization
- * @param adminUserId User ID of the admin/owner making the change
- * @returns True if membership was deleted, false if permission denied or not found
- */
-export async function deleteMembership(
-  db: ReturnType<typeof createDatabase>,
-  organizationIdOrHandle: string,
-  targetUserId: string,
-  adminUserId: string
-): Promise<boolean> {
-  // First, verify the admin user has permission (admin or owner)
-  const [adminMembership] = await db
-    .select()
-    .from(memberships)
-    .innerJoin(organizations, eq(memberships.organizationId, organizations.id))
-    .where(
-      and(
-        eq(memberships.userId, adminUserId),
-        getOrganizationCondition(organizationIdOrHandle),
-        inArray(memberships.role, [
-          OrganizationRole.ADMIN,
-          OrganizationRole.OWNER,
-        ])
-      )
-    )
-    .limit(1);
-
-  if (!adminMembership) {
-    return false; // Admin user doesn't have permission
-  }
-
-  // Get the target user's membership to check their role
-  const [targetMembership] = await db
-    .select()
-    .from(memberships)
-    .where(
-      and(
-        eq(memberships.userId, targetUserId),
-        eq(memberships.organizationId, adminMembership.organizations.id)
-      )
-    )
-    .limit(1);
-
-  if (!targetMembership) {
-    return false; // Target user is not a member
-  }
-
-  // Additional check: only owners can delete other owners
-  if (
-    targetMembership.role === OrganizationRole.OWNER &&
-    adminMembership.memberships.role !== OrganizationRole.OWNER
-  ) {
-    return false; // Only owners can delete other owners
-  }
-
-  // Prevent users from deleting themselves
-  if (targetUserId === adminUserId) {
-    return false; // Users cannot delete their own membership
-  }
-
-  // Delete the membership
-  const [deletedMembership] = await db
-    .delete(memberships)
-    .where(
-      and(
-        eq(memberships.userId, targetUserId),
-        eq(memberships.organizationId, adminMembership.organizations.id)
-      )
-    )
-    .returning({ id: memberships.userId });
-
-  return !!deletedMembership;
-}
-
-/**
- * List all memberships for an organization
- *
- * @param db Database instance
- * @param organizationIdOrHandle Organization ID or handle
- * @returns Array of membership records
- */
-export async function getOrganizationMemberships(
-  db: ReturnType<typeof createDatabase>,
-  organizationIdOrHandle: string
-) {
-  return await db
-    .select({
-      userId: memberships.userId,
-      organizationId: memberships.organizationId,
-      role: memberships.role,
-      createdAt: memberships.createdAt,
-      updatedAt: memberships.updatedAt,
-    })
-    .from(memberships)
-    .innerJoin(organizations, eq(memberships.organizationId, organizations.id))
-    .where(getOrganizationCondition(organizationIdOrHandle))
-    .orderBy(memberships.createdAt);
-}

apps/web/server.ts

@@ -51,14 +51,14 @@ const vite = await createServer({
 });
 app.use(vite.middlewares);
 
-let template = await fs.readFile("./index.html", "utf-8");
+const originalTemplate = await fs.readFile("./index.html", "utf-8");
 const render = (await vite.ssrLoadModule("/src/entry-server.tsx")).render;
 
 // Serve HTML
 app.use("*all", async (req, res) => {
   try {
     const pathname = req.originalUrl;
-    template = await vite.transformIndexHtml(pathname, template);
+    let html = await vite.transformIndexHtml(pathname, originalTemplate);
 
     // Construct a Fetch API Request object from the Express request
     const url = new URL(
@@ -72,7 +72,7 @@ app.use("*all", async (req, res) => {
 
     const rendered = await render(fetchRequest);
 
-    let html = template
+    html = html
       .replace(`<!--app-head-->`, rendered.headHtml ?? "")
       .replace(`<!--app-html-->`, "");
 

apps/web/src/components/app-header.tsx

@@ -1,4 +1,4 @@
-import { Bot } from "lucide-react";
+import Bot from "lucide-react/icons/bot";
 import { Link } from "react-router";
 
 import { OrganizationSwitcher } from "@/components/organization-switcher";

apps/web/src/components/deployments/deployment-info-card.tsx

@@ -1,10 +1,8 @@
 import { format } from "date-fns";
-import {
-  ArrowUpToLine,
-  Clock,
-  GitCommitHorizontal,
-  IdCard,
-} from "lucide-react";
+import ArrowUpToLine from "lucide-react/icons/arrow-up-to-line";
+import Clock from "lucide-react/icons/clock";
+import GitCommitHorizontal from "lucide-react/icons/git-commit-horizontal";
+import IdCard from "lucide-react/icons/id-card";
 import { Link } from "react-router";
 
 import { Badge } from "@/components/ui/badge";

apps/web/src/components/deployments/workflow-info-card.tsx

@@ -1,4 +1,5 @@
-import { IdCard, Workflow } from "lucide-react";
+import IdCard from "lucide-react/icons/id-card";
+import Workflow from "lucide-react/icons/workflow";
 import { Link } from "react-router";
 
 import {

apps/web/src/components/docs/code-block.tsx

@@ -1,4 +1,5 @@
-import { Check, Copy } from "lucide-react";
+import Check from "lucide-react/icons/check";
+import Copy from "lucide-react/icons/copy";
 import { useEffect, useState } from "react";
 
 import { Button } from "@/components/ui/button";

apps/web/src/components/docs/nodes-browser.tsx

@@ -1,4 +1,6 @@
-import { Grid, List, Loader2 } from "lucide-react";
+import Grid from "lucide-react/icons/grid";
+import List from "lucide-react/icons/list";
+import Loader2 from "lucide-react/icons/loader-2";
 import { useMemo, useState } from "react";
 
 import { Button } from "@/components/ui/button";

apps/web/src/components/docs/ready-to-build.tsx

@@ -1,12 +1,10 @@
 import { CreateWorkflowRequest, WorkflowType } from "@dafthunk/types";
-import {
-  Activity,
-  CloudUpload,
-  LayoutDashboard,
-  PlusCircle,
-  Workflow,
-  Zap,
-} from "lucide-react";
+import Activity from "lucide-react/icons/activity";
+import CloudUpload from "lucide-react/icons/cloud-upload";
+import LayoutDashboard from "lucide-react/icons/layout-dashboard";
+import PlusCircle from "lucide-react/icons/plus-circle";
+import Workflow from "lucide-react/icons/workflow";
+import Zap from "lucide-react/icons/zap";
 import { useState } from "react";
 import { Link, useNavigate } from "react-router";
 

apps/web/src/components/executions/execution-info-card.tsx

@@ -1,13 +1,11 @@
 import { WorkflowExecutionStatus } from "@dafthunk/types";
 import { format } from "date-fns";
-import {
-  AlertCircle,
-  Clock,
-  Eye,
-  EyeOff,
-  IdCard,
-  Workflow,
-} from "lucide-react";
+import AlertCircle from "lucide-react/icons/alert-circle";
+import Clock from "lucide-react/icons/clock";
+import Eye from "lucide-react/icons/eye";
+import EyeOff from "lucide-react/icons/eye-off";
+import IdCard from "lucide-react/icons/id-card";
+import Workflow from "lucide-react/icons/workflow";
 import { Link } from "react-router";
 
 import {

apps/web/src/components/executions/execution-status-badge.tsx

@@ -1,12 +1,10 @@
 import type { WorkflowExecution } from "@dafthunk/types";
 import type { VariantProps } from "class-variance-authority";
-import {
-  AlertCircle,
-  CheckCircle2,
-  CircleDashed,
-  CircleSlash,
-  Zap,
-} from "lucide-react";
+import AlertCircle from "lucide-react/icons/alert-circle";
+import CheckCircle2 from "lucide-react/icons/check-circle-2";
+import CircleDashed from "lucide-react/icons/circle-dashed";
+import CircleSlash from "lucide-react/icons/circle-slash";
+import Zap from "lucide-react/icons/zap";
 
 import { Badge, badgeVariants } from "@/components/ui/badge";
 import { cn } from "@/utils/utils";