FE exercise patch (exercism#7853)

* Avoid double adjusting col numbers

* Remove unused files, console logs

Author: dem4ron

app/javascript/components/bootcamp/FrontendExercisePage/FrontendExercisePage.tsx

@@ -8,7 +8,6 @@ import { useSetupEditors } from './hooks/useSetupEditors'
 import { useSetupIFrames } from './hooks/useSetupIFrames'
 import { LHS } from './LHS/LHS'
 import { RHS } from './RHS/RHS'
-import { useLogger } from '../common/hooks/useLogger'
 import { useRestoreIframeScrollAfterResize } from './hooks/useRestoreIframeScrollAfterResize'
 
 export default function FrontendExercisePage(data: FrontendExercisePageProps) {
@@ -17,8 +16,6 @@ export default function FrontendExercisePage(data: FrontendExercisePageProps) {
 
   const { handleWidthChangeMouseDown } = useInitResizablePanels()
 
-  useLogger('data', data)
-
   const {
     htmlEditorViewRef,
     cssEditorViewRef,

app/javascript/components/bootcamp/FrontendExercisePage/LHS/showJsError.ts

@@ -22,7 +22,7 @@ export function showJsError(
 
   const editorLength = view.state.doc.length
   const errorLine = view.state.doc.line(error.lineNumber)
-  const basePos = errorLine.from + error.colNumber
+  const basePos = errorLine.from + error.colNumber - 1
 
   // range rules to avoid breaking:
   // from < to
@@ -31,8 +31,6 @@ export function showJsError(
   const from = Math.max(1, Math.min(basePos, editorLength - 1))
   const to = Math.min(editorLength, from + 1)
 
-  console.log('editorleng', editorLength)
-
   view.dispatch({
     effects: [
       showInfoWidgetEffect.of(true),

app/javascript/components/bootcamp/FrontendExercisePage/LHS/useHandleJsErrorMessage.ts

@@ -23,14 +23,15 @@ export function useHandleJsErrorMessage({
 
         const { column, line } = extractLineAndColumnFromStack(data.stack)
 
-        const adjustedLineNumber = line - jsLineOffset
-        const lineNumberAbsolutePosition =
-          jsViewRef.current?.state.doc.line(adjustedLineNumber).from ?? 0
-        const adjustedColumnNumber = lineNumberAbsolutePosition + column - 1
+        const adjustedLineNumber = Math.min(
+          Math.max(1, line - jsLineOffset),
+          jsViewRef.current?.state.doc.lines || 1
+        )
 
         showJsError(jsViewRef.current, {
           message: data.message,
-          colNumber: adjustedColumnNumber,
+          // column gets adjusted + positioned in showJsError
+          colNumber: column,
           lineNumber: adjustedLineNumber,
         })
       }