TSQL don't comment lines last line if the cursor is at the beginning

Removed unused scopes from shortcuts

Author: mpcgrid

apps/webapp/app/components/code/TSQLEditor.tsx

@@ -64,7 +64,10 @@ const defaultProps: TSQLEditorDefaultProps = {
 const toggleLineComment = (view: EditorView): boolean => {
   const { from, to } = view.state.selection.main;
   const startLine = view.state.doc.lineAt(from);
-  const endLine = view.state.doc.lineAt(to);
+  // When `to` is exactly at the start of a line and there's an actual selection,
+  // the caret sits before that line — so exclude it by stepping back one position.
+  const adjustedTo = to > from && view.state.doc.lineAt(to).from === to ? to - 1 : to;
+  const endLine = view.state.doc.lineAt(adjustedTo);
 
   // Collect all lines in the selection
   const lines: { from: number; to: number; text: string }[] = [];

apps/webapp/app/hooks/useShortcutKeys.tsx

@@ -23,15 +23,13 @@ type useShortcutKeysProps = {
   action: (event: KeyboardEvent) => void;
   disabled?: boolean;
   enabledOnInputElements?: boolean;
-  scopes?: string | string[];
 };
 
 export function useShortcutKeys({
   shortcut,
   action,
   disabled = false,
   enabledOnInputElements,
-  scopes = "global",
 }: useShortcutKeysProps) {
   const { platform } = useOperatingSystem();
   const { areShortcutsEnabled } = useShortcuts();
@@ -52,7 +50,6 @@ export function useShortcutKeys({
     },
     {
       enabled: isEnabled,
-      scopes,
       enableOnFormTags:
         isEnabled && (enabledOnInputElements ?? relevantShortcut?.enabledOnInputElements),
       enableOnContentEditable:

apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.logs/route.tsx

@@ -402,24 +402,25 @@ function LogsList({
         return;
       }
 
-      const existingIds = new Set(accumulatedLogs.map((log) => log.id));
-      const newLogs = fetcher.data.logs.filter((log) => !existingIds.has(log.id));
-
       if (isCheckingForNewRef.current) {
         // "Check for new" - prepend new logs, don't update cursor
-        if (newLogs.length > 0) {
-          setAccumulatedLogs((prev) => [...newLogs, ...prev]);
-        }
+        setAccumulatedLogs((prev) => {
+          const existingIds = new Set(prev.map((log) => log.id));
+          const newLogs = fetcher.data!.logs.filter((log) => !existingIds.has(log.id));
+          return newLogs.length > 0 ? [...newLogs, ...prev] : prev;
+        });
         isCheckingForNewRef.current = false;
       } else {
         // "Load more" - append logs and update cursor
-        if (newLogs.length > 0) {
-          setAccumulatedLogs((prev) => [...prev, ...newLogs]);
-        }
+        setAccumulatedLogs((prev) => {
+          const existingIds = new Set(prev.map((log) => log.id));
+          const newLogs = fetcher.data!.logs.filter((log) => !existingIds.has(log.id));
+          return newLogs.length > 0 ? [...prev, ...newLogs] : prev;
+        });
         setNextCursor(fetcher.data.pagination.next);
       }
     }
-  }, [fetcher.data, fetcher.state, accumulatedLogs, location.search]);
+  }, [fetcher.data, fetcher.state, location.search]);
 
   // Build resource URL for loading more
   const loadMoreUrl = useMemo(() => {

apps/webapp/app/routes/resources.orgs.$organizationSlug.projects.$projectParam.env.$envParam.logs.ts

@@ -8,7 +8,6 @@ import { LogsListPresenter, type LogLevel, LogsListOptionsSchema } from "~/prese
 import { $replica } from "~/db.server";
 import { clickhouseClient } from "~/services/clickhouseInstance.server";
 import { getCurrentPlan } from "~/services/platform.v3.server";
-import { getTimezonePreference } from "~/services/preferences/uiPreferences.server";
 
 // Valid log levels for filtering
 const validLevels: LogLevel[] = ["DEBUG", "INFO", "WARN", "ERROR"];