fix: trivial cleanup (#298)

* fix: trivial cleanup * fix: linting
pezzolabs · Apr 28, 2024 · dc56c85 · dc56c85
1 parent bbd6e20
commit dc56c85
Show file tree

Hide file tree

Showing 10 changed files with 28 additions and 21 deletions.
diff --git a/.vscode/settings.json b/.vscode/settings.json
@@ -1,16 +1,3 @@
 {
-  "sqltools.connections": [
-    {
-      "server": "localhost",
-      "port": 8123,
-      "useHTTPS": false,
-      "database": "default",
-      "username": "default",
-      "password": "",
-      "tls": {},
-      "previewLimit": 50,
-      "driver": "ClickHouse",
-      "name": "Local"
-    }
-  ]
+  "css.customData": [".vscode/tailwind.json"]
 }
diff --git a/apps/console/src/components/common/Drawer.tsx b/apps/console/src/components/common/Drawer.tsx
@@ -26,6 +26,13 @@ export const Drawer = ({ children, className, onClose, open }: Props) => {
           !open && "hidden"
         )}
         onClick={onClose}
+        onKeyDown={(e) => {
+          if (e.key === "Enter") {
+            onClose();
+          }
+        }}
+        role="button"
+        aria-label="Close"
       ></div>
       <motion.div
         initial={{ width: 0, opacity: 0 }}

diff --git a/apps/console/src/components/layout/OrgSubHeader.tsx b/apps/console/src/components/layout/OrgSubHeader.tsx
@@ -40,6 +40,13 @@ export const OrgSubHeader = () => {
               key={nav.href}
               onClick={() => navigate(nav.href)}
               className={getClassName(nav.href)}
+              onKeyDown={(e) => {
+                if (e.key === "Enter") {
+                  navigate(nav.href);
+                }
+              }}
+              role="button"
+              aria-label={nav.name}
             >
               {nav.name}
             </li>

diff --git a/apps/console/src/components/layout/SideNavigation.tsx b/apps/console/src/components/layout/SideNavigation.tsx
@@ -49,6 +49,7 @@ export const SideNavigation = () => {
     <div
       onMouseOver={() => setCollapsed(false)}
       onMouseLeave={() => setCollapsed(true)}
+      onFocus={() => setCollapsed(false)}
       className={cn(
         "z-50 flex h-full grow flex-col gap-y-4 overflow-y-auto overflow-x-hidden border-r border-border bg-stone-900 px-3 pt-2"
       )}

diff --git a/apps/console/src/components/prompts/ConsumePromptModal.tsx b/apps/console/src/components/prompts/ConsumePromptModal.tsx
@@ -21,8 +21,6 @@ interface Props {
 export const ConsumePromptModal = ({ open, onClose }: Props) => {
   const [integration, setIntegration] = useState("typescript");
 
-  <TypeScriptOpenAIIntegrationTutorial />;
-
   const renderInsructions = () => {
     switch (integration) {
       case "typescript":

diff --git a/apps/console/src/pages/prompts/PromptNavigation.tsx b/apps/console/src/pages/prompts/PromptNavigation.tsx
@@ -55,6 +55,13 @@ export const PromptNavigation = () => {
               key={nav.name}
               onClick={() => navigate(nav.href)}
               className={getClassName(nav)}
+              onKeyDown={(e) => {
+                if (e.key === "Enter") {
+                  navigate(nav.href);
+                }
+              }}
+              role="button"
+              aria-label={nav.name}
             >
               <nav.icon className="h-4 w-4" />
               {nav.name}

diff --git a/apps/server/src/app/identity/projects.resolver.ts b/apps/server/src/app/identity/projects.resolver.ts
@@ -61,7 +61,7 @@ export class ProjectsResolver {
   }
 
   @Query(() => [Project])
-  async projects(
+  projects(
     @Args("data") data: GetProjectsInput,
     @CurrentUser() user: RequestUser
   ) {

diff --git a/apps/server/src/app/prompts/prompts.resolver.ts b/apps/server/src/app/prompts/prompts.resolver.ts
@@ -266,7 +266,7 @@ export class PromptsResolver {
   }
 
   @ResolveField(() => [PromptVersion])
-  async versions(@Parent() prompt: PrismaPrompt) {
+  versions(@Parent() prompt: PrismaPrompt) {
     this.logger.info("Resolving prompt versions");
 
     try {
@@ -278,7 +278,7 @@ export class PromptsResolver {
   }
 
   @ResolveField(() => PromptVersion, { nullable: true })
-  async latestVersion(@Parent() prompt: PrismaPrompt) {
+  latestVersion(@Parent() prompt: PrismaPrompt) {
     this.logger.info("Resolving prompt latest version");
 
     try {

diff --git a/apps/server/src/app/reporting/reporting.controller.ts b/apps/server/src/app/reporting/reporting.controller.ts
@@ -31,7 +31,7 @@ export class ReportingController {
     description: "Report has been reported successfully",
   })
   @ApiResponse({ status: 500, description: "Internal server error" })
-  async reportRequest(
+  reportRequest(
     @Body() dto: CreateReportDto,
     @ApiKeyOrgId() organizationId: string,
     @ProjectId() projectId: string

diff --git a/libs/common/src/utils/slugify.ts b/libs/common/src/utils/slugify.ts
@@ -1,5 +1,5 @@
 export function slugify(str: string) {
-  str = str.replace(/^\s+|\s+$/g, ""); // trim leading/trailing white space
+  str = str.replace(/(^\s+|\s+$)/g, ""); // trim leading/trailing white space
   str = str.toLowerCase(); // convert string to lowercase
   str = str
     .replace(/[^a-z0-9 -]/g, "") // remove any non-alphanumeric characters