Format: Run yarn format to ensure code follows project standards

Co-Authored-By: Francisco Javier Arceo <arceofrancisco@gmail.com>
feast-dev · franciscojavierarceo · May 4, 2025 · May 3, 2025 · May 4, 2025 · May 4, 2025
commit c7dfb245d14e1e63286dfb96010446567f2d52f0
@@ -102,8 +102,8 @@ const ProjectOverviewPage = () => {
                 <EuiText>
                   <p>
                     Welcome to your new Feast project. In this UI, you can see
-                    Data Sources, Entities, Features, Feature Views, and
-                    Feature Services registered in Feast.
+                    Data Sources, Entities, Features, Feature Views, and Feature
+                    Services registered in Feast.
                   </p>
                   <p>
                     It looks like this project already has some objects

@@ -76,9 +76,7 @@ const SideNav = () => {
           name: "Lineage",
           id: htmlIdGenerator("lineage")(),
           icon: <EuiIcon type="graphApp" />,
-          renderItem: (props) => (
-            <Link {...props} to={`${baseUrl}/lineage`} />
-          ),
+          renderItem: (props) => <Link {...props} to={`${baseUrl}/lineage`} />,
           isSelected: useMatchSubpath(`${baseUrl}/lineage`),
         },
         {
@@ -133,7 +131,9 @@ const SideNav = () => {
           name: "Permissions",
           id: htmlIdGenerator("permissions")(),
           icon: <EuiIcon type="lock" />,
-          renderItem: (props) => <Link {...props} to={`${baseUrl}/permissions`} />,
+          renderItem: (props) => (
+            <Link {...props} to={`${baseUrl}/permissions`} />
+          ),
           isSelected: useMatchSubpath(`${baseUrl}/permissions`),
         },
       ],

@@ -124,15 +124,17 @@ const DataSourceOverviewTab = () => {
                 </EuiTitle>
                 <EuiHorizontalRule margin="xs"></EuiHorizontalRule>
                 {registryQuery.data?.permissions ? (
-                  <PermissionsDisplay 
+                  <PermissionsDisplay
                     permissions={getEntityPermissions(
                       registryQuery.data.permissions,
                       FEAST_FCO_TYPES.dataSource,
-                      dsName
+                      dsName,
                     )}
                   />
                 ) : (
-                  <EuiText>No permissions defined for this data source.</EuiText>
+                  <EuiText>
+                    No permissions defined for this data source.
+                  </EuiText>
                 )}
               </EuiPanel>
             </EuiFlexItem>

@@ -147,11 +147,11 @@ const EntityOverviewTab = () => {
                 </EuiTitle>
                 <EuiHorizontalRule margin="xs" />
                 {registryQuery.data?.permissions ? (
-                  <PermissionsDisplay 
+                  <PermissionsDisplay
                     permissions={getEntityPermissions(
                       registryQuery.data.permissions,
                       FEAST_FCO_TYPES.entity,
-                      eName
+                      eName,
                     )}
                   />
                 ) : (

@@ -175,15 +175,17 @@ const FeatureServiceOverviewTab = () => {
                 </EuiTitle>
                 <EuiHorizontalRule margin="xs" />
                 {data?.permissions ? (
-                  <PermissionsDisplay 
+                  <PermissionsDisplay
                     permissions={getEntityPermissions(
                       data.permissions,
                       FEAST_FCO_TYPES.featureService,
-                      fsName
+                      fsName,
                     )}
                   />
                 ) : (
-                  <EuiText>No permissions defined for this feature service.</EuiText>
+                  <EuiText>
+                    No permissions defined for this feature service.
+                  </EuiText>
                 )}
               </EuiPanel>
             </EuiFlexItem>

@@ -40,10 +40,12 @@ const useLoadFeatureService = (featureServiceName: string) => {
   }
   return {
     ...registryQuery,
-    data: data ? {
-      ...data,
-      permissions: registryQuery.data?.permissions
-    } : undefined,
+    data: data
+      ? {
+          ...data,
+          permissions: registryQuery.data?.permissions,
+        }
+      : undefined,
     entities,
   };
 };

@@ -42,7 +42,12 @@ const FeatureViewInstance = () => {
     if (data.type === FEAST_FV_TYPES.regular) {
       const fv: feast.core.IFeatureView = data.object;
 
-      return <RegularFeatureInstance data={fv} permissions={registryQuery.data?.permissions} />;
+      return (
+        <RegularFeatureInstance
+          data={fv}
+          permissions={registryQuery.data?.permissions}
+        />
+      );
     }
 
     if (data.type === FEAST_FV_TYPES.ondemand) {

@@ -20,7 +20,10 @@ interface RegularFeatureInstanceProps {
   permissions?: any[];
 }
 
-const RegularFeatureInstance = ({ data, permissions }: RegularFeatureInstanceProps) => {
+const RegularFeatureInstance = ({
+  data,
+  permissions,
+}: RegularFeatureInstanceProps) => {
   const { enabledFeatureStatistics } = useContext(FeatureFlagsContext);
   const navigate = useNavigate();
 
@@ -70,7 +73,12 @@ const RegularFeatureInstance = ({ data, permissions }: RegularFeatureInstancePro
         <Routes>
           <Route
             path="/"
-            element={<RegularFeatureViewOverviewTab data={data} permissions={permissions} />}
+            element={
+              <RegularFeatureViewOverviewTab
+                data={data}
+                permissions={permissions}
+              />
+            }
           />
           <Route
             path="/lineage"

@@ -156,11 +156,11 @@ const RegularFeatureViewOverviewTab = ({
             </EuiTitle>
             <EuiHorizontalRule margin="xs" />
             {permissions ? (
-              <PermissionsDisplay 
+              <PermissionsDisplay
                 permissions={getEntityPermissions(
                   permissions,
                   FEAST_FCO_TYPES.featureView,
-                  data?.spec?.name
+                  data?.spec?.name,
                 )}
               />
             ) : (

@@ -23,7 +23,11 @@ import useLoadRegistry from "../../queries/useLoadRegistry";
 import RegistryPathContext from "../../contexts/RegistryPathContext";
 import { FeatureIcon } from "../../graphics/FeatureIcon";
 import { FEAST_FCO_TYPES } from "../../parsers/types";
-import { getEntityPermissions, formatPermissions, filterPermissionsByAction } from "../../utils/permissionUtils";
+import {
+  getEntityPermissions,
+  formatPermissions,
+  filterPermissionsByAction,
+} from "../../utils/permissionUtils";
 
 interface Feature {
   name: string;
@@ -49,18 +53,23 @@ const FeatureListPage = () => {
   const [pageIndex, setPageIndex] = useState(0);
   const [pageSize, setPageSize] = useState(100);
 
-  const featuresWithPermissions: Feature[] = (data?.allFeatures || []).map(feature => {
-    return {
-      ...feature,
-      permissions: getEntityPermissions(
-        selectedPermissionAction 
-          ? filterPermissionsByAction(data?.permissions, selectedPermissionAction)
-          : data?.permissions,
-        FEAST_FCO_TYPES.featureView,
-        feature.featureView
-      )
-    };
-  });
+  const featuresWithPermissions: Feature[] = (data?.allFeatures || []).map(
+    (feature) => {
+      return {
+        ...feature,
+        permissions: getEntityPermissions(
+          selectedPermissionAction
+            ? filterPermissionsByAction(
+                data?.permissions,
+                selectedPermissionAction,
+              )
+            : data?.permissions,
+          FEAST_FCO_TYPES.featureView,
+          feature.featureView,
+        ),
+      };
+    },
+  );
 
   const features: Feature[] = featuresWithPermissions;
 
@@ -114,17 +123,22 @@ const FeatureListPage = () => {
         return hasPermissions ? (
           <EuiToolTip
             position="top"
-            content={<pre style={{ margin: 0 }}>{formatPermissions(permissions)}</pre>}
+            content={
+              <pre style={{ margin: 0 }}>{formatPermissions(permissions)}</pre>
+            }
           >
             <div style={{ display: "flex", alignItems: "center" }}>
               <EuiIcon type="lock" color="#5a7be0" />
               <EuiText size="xs" style={{ marginLeft: "4px" }}>
-                {permissions.length} permission{permissions.length !== 1 ? "s" : ""}
+                {permissions.length} permission
+                {permissions.length !== 1 ? "s" : ""}
               </EuiText>
             </div>
           </EuiToolTip>
         ) : (
-          <EuiText size="xs" color="subdued">None</EuiText>
+          <EuiText size="xs" color="subdued">
+            None
+          </EuiText>
         );
       },
     },
@@ -193,7 +207,9 @@ const FeatureListPage = () => {
                       { value: "WRITE_OFFLINE", text: "WRITE_OFFLINE" },
                     ]}
                     value={selectedPermissionAction}
-                    onChange={(e) => setSelectedPermissionAction(e.target.value)}
+                    onChange={(e) =>
+                      setSelectedPermissionAction(e.target.value)
+                    }
                     aria-label="Filter by permission action"
                   />
                 </EuiFormRow>

@@ -37,9 +37,9 @@ const LineagePage = () => {
             title={<h2>Error Loading Project Configs</h2>}
             body={
               <p>
-                There was an error loading the Project Configurations.
-                Please check that <code>feature_store.yaml</code> file is
-                available and well-formed.
+                There was an error loading the Project Configurations. Please
+                check that <code>feature_store.yaml</code> file is available and
+                well-formed.
               </p>
             }
           />

@@ -54,7 +54,9 @@ const PermissionsIndex = () => {
                       { value: "WRITE_OFFLINE", text: "WRITE_OFFLINE" },
                     ]}
                     value={selectedPermissionAction}
-                    onChange={(e) => setSelectedPermissionAction(e.target.value)}
+                    onChange={(e) =>
+                      setSelectedPermissionAction(e.target.value)
+                    }
                     aria-label="Filter by action"
                   />
                 </EuiFormRow>
@@ -72,7 +74,7 @@ const PermissionsIndex = () => {
                     selectedPermissionAction
                       ? filterPermissionsByAction(
                           data.permissions,
-                          selectedPermissionAction
+                          selectedPermissionAction,
                         )
                       : data.permissions
                   }

@@ -80,47 +80,48 @@ const useLoadRegistry = (url: string) => {
             relationships,
             indirectRelationships,
             allFeatures,
-            permissions: objects.permissions && objects.permissions.length > 0 
-              ? objects.permissions 
-              : [
-                  {
-                    spec: {
-                      name: "zipcode-features-reader",
-                      types: [2], // FeatureView
-                      name_patterns: ["zipcode_features"],
-                      policy: { roles: ["analyst", "data_scientist"] },
-                      actions: [1, 4, 5] // DESCRIBE, READ_ONLINE, READ_OFFLINE
-                    }
-                  },
-                  {
-                    spec: {
-                      name: "zipcode-source-writer",
-                      types: [7], // FileSource
-                      name_patterns: ["zipcode"],
-                      policy: { roles: ["admin", "data_engineer"] },
-                      actions: [0, 2, 7] // CREATE, UPDATE, WRITE_OFFLINE
-                    }
-                  },
-                  {
-                    spec: {
-                      name: "credit-score-v1-reader",
-                      types: [6], // FeatureService
-                      name_patterns: ["credit_score_v1"],
-                      policy: { roles: ["model_user", "data_scientist"] },
-                      actions: [1, 4] // DESCRIBE, READ_ONLINE
-                    }
-                  },
-                  {
-                    spec: {
-                      name: "risky-features-reader",
-                      types: [2, 6], // FeatureView, FeatureService
-                      name_patterns: [],
-                      required_tags: { "stage": "prod" },
-                      policy: { roles: ["trusted_analyst"] },
-                      actions: [5] // READ_OFFLINE
-                    }
-                  }
-                ],
+            permissions:
+              objects.permissions && objects.permissions.length > 0
+                ? objects.permissions
+                : [
+                    {
+                      spec: {
+                        name: "zipcode-features-reader",
+                        types: [2], // FeatureView
+                        name_patterns: ["zipcode_features"],
+                        policy: { roles: ["analyst", "data_scientist"] },
+                        actions: [1, 4, 5], // DESCRIBE, READ_ONLINE, READ_OFFLINE
+                      },
+                    },
+                    {
+                      spec: {
+                        name: "zipcode-source-writer",
+                        types: [7], // FileSource
+                        name_patterns: ["zipcode"],
+                        policy: { roles: ["admin", "data_engineer"] },
+                        actions: [0, 2, 7], // CREATE, UPDATE, WRITE_OFFLINE
+                      },
+                    },
+                    {
+                      spec: {
+                        name: "credit-score-v1-reader",
+                        types: [6], // FeatureService
+                        name_patterns: ["credit_score_v1"],
+                        policy: { roles: ["model_user", "data_scientist"] },
+                        actions: [1, 4], // DESCRIBE, READ_ONLINE
+                      },
+                    },
+                    {
+                      spec: {
+                        name: "risky-features-reader",
+                        types: [2, 6], // FeatureView, FeatureService
+                        name_patterns: [],
+                        required_tags: { stage: "prod" },
+                        policy: { roles: ["trusted_analyst"] },
+                        actions: [5], // READ_OFFLINE
+                      },
+                    },
+                  ],
           };
         });
     },