add bounds checks to highlight ranges

Author: brendan-kellam

packages/web/src/app/[domain]/browse/[...path]/components/pureCodePreviewPanel.tsx

@@ -114,6 +114,11 @@ export const PureCodePreviewPanel = ({
         const doc = editorRef.state.doc;
         const { start, end } = highlightRange;
 
+        if (start.lineNumber > doc.lines || end.lineNumber > doc.lines) {
+            console.warn(`Highlight range is out of bounds: start.lineNumber=${start.lineNumber}, end.lineNumber=${end.lineNumber}, doc.lines=${doc.lines}`);
+            return;
+        }
+
         const from = doc.line(start.lineNumber).from;
         const to = doc.line(end.lineNumber).to;
         const selection = EditorSelection.range(from, to);

packages/web/src/app/[domain]/browse/[...path]/components/rangeHighlightingExtension.ts

@@ -16,6 +16,11 @@ export const rangeHighlightingExtension = (range: BrowseHighlightRange) => State
     create(state) {
         const { start, end } = range;
 
+        if (start.lineNumber > state.doc.lines || end.lineNumber > state.doc.lines) {
+            console.warn(`Highlight range is out of bounds: start.lineNumber=${start.lineNumber}, end.lineNumber=${end.lineNumber}, doc.lines=${state.doc.lines}`);
+            return Decoration.none;
+        }
+
         if ('column' in start && 'column' in end) {
             const from = state.doc.line(start.lineNumber).from + start.column - 1;
             const to = state.doc.line(end.lineNumber).from + end.column - 1;