@geode/opengeodeweb-front 10.34.0-rc.8 → 10.34.0

package/app/components/Viewer/Surface/PolygonsOptions.vue

@@ -1,18 +1,14 @@
 <script setup lang="ts">
-// Not auto-fixable (eslint's sort-imports core rule has no autofixer) and this file's import order doesn't match its syntax-kind-then-alphabetical requirement - left as-is rather than manually reordered across the codebase for a purely cosmetic rule.
-// oxlint-disable eslint/sort-imports
-import PolygonalSurfacePolygons from "@ogw_front/assets/viewer_svgs/surface_triangles.svg";
-// oxlint-disable import/consistent-type-specifier-style -- combining the default import with the type import avoids a duplicate-imports violation on this same module; using disable-next-line here is fragile because a formatter can re-wrap the import onto multiple lines and shift the flagged line
 import ViewerContextMenuItem, {
   type ItemProps,
 } from "@ogw_front/components/Viewer/ContextMenu/ContextMenuItem.vue";
+import PolygonalSurfacePolygons from "@ogw_front/assets/viewer_svgs/surface_triangles.svg";
+import type { RGBAColor } from "@ogw_front/utils/default_styles/constants";
 import ViewerOptionsColoringTypeSelector from "@ogw_front/components/Viewer/Options/ColoringTypeSelector.vue";
 import ViewerOptionsVisibilitySwitch from "@ogw_front/components/Viewer/Options/VisibilitySwitch.vue";
-
 import { useBatchStyle } from "@ogw_front/composables/batch_style";
 import { useDataStyleStore } from "@ogw_front/stores/data_style";
 import { useHybridViewerStore } from "@ogw_front/stores/hybrid_viewer";
-import type { RGBAColor } from "@ogw_front/utils/default_styles/constants";
 
 const dataStyleStore = useDataStyleStore();
 const hybridViewerStore = useHybridViewerStore();
@@ -32,7 +28,7 @@ const { itemProps, tooltip = "Polygons options" } = defineProps<Props>();
 
 const id = toRef(() => itemProps.id);
 
-const visibility = computed({
+const visibility = computed<boolean>({
   get: () => dataStyleStore.meshPolygonsVisibility(id.value),
   set: async (newValue) => {
     await applyBatchStyle(id.value, (targetId: string) =>
@@ -41,7 +37,7 @@ const visibility = computed({
     hybridViewerStore.remoteRender();
   },
 });
-const coloring_style_key = computed({
+const coloring_style_key = computed<string>({
   get: () => dataStyleStore.meshPolygonsActiveColoring(id.value),
   set: async (newValue) => {
     await applyBatchStyle(id.value, (targetId: string) =>
@@ -68,7 +64,7 @@ const textures = computed<Texture[] | undefined>({
     hybridViewerStore.remoteRender();
   },
 });
-const vertex_attribute_name = computed({
+const vertex_attribute_name = computed<string | undefined>({
   get: () => dataStyleStore.meshPolygonsVertexAttributeName(id.value),
   set: async (newValue) => {
     if (newValue === undefined) {
@@ -80,7 +76,7 @@ const vertex_attribute_name = computed({
     hybridViewerStore.remoteRender();
   },
 });
-const vertex_attribute_item = computed({
+const vertex_attribute_item = computed<string | undefined>({
   get: () => dataStyleStore.meshPolygonsVertexAttributeItem(id.value),
   set: async (newValue) => {
     await applyBatchStyle(id.value, (targetId: string) =>
@@ -89,7 +85,7 @@ const vertex_attribute_item = computed({
     hybridViewerStore.remoteRender();
   },
 });
-const vertex_attribute_range = computed({
+const vertex_attribute_range = computed<[number, number] | undefined>({
   get: () => dataStyleStore.meshPolygonsVertexAttributeRange(id.value),
   set: async (newValue) => {
     const [minimum, maximum] = newValue;
@@ -104,7 +100,7 @@ const vertex_attribute_range = computed({
     hybridViewerStore.remoteRender();
   },
 });
-const vertex_attribute_color_map = computed({
+const vertex_attribute_color_map = computed<Map<string, RGBAColor> | undefined>({
   get: () => dataStyleStore.meshPolygonsVertexAttributeColorMap(id.value),
   set: async (newValue) => {
     await applyBatchStyle(id.value, (targetId: string) =>
@@ -123,7 +119,7 @@ const vertex_attribute_no_data_color = computed<RGBAColor | undefined>({
     hybridViewerStore.remoteRender();
   },
 });
-const polygon_attribute_name = computed({
+const polygon_attribute_name = computed<string | undefined>({
   get: () => dataStyleStore.meshPolygonsPolygonAttributeName(id.value),
   set: async (newValue) => {
     if (newValue === undefined) {
@@ -135,7 +131,7 @@ const polygon_attribute_name = computed({
     hybridViewerStore.remoteRender();
   },
 });
-const polygon_attribute_item = computed({
+const polygon_attribute_item = computed<string | undefined>({
   get: () => dataStyleStore.meshPolygonsPolygonAttributeItem(id.value),
   set: async (newValue) => {
     await applyBatchStyle(id.value, (targetId: string) =>
@@ -144,7 +140,7 @@ const polygon_attribute_item = computed({
     hybridViewerStore.remoteRender();
   },
 });
-const polygon_attribute_range = computed({
+const polygon_attribute_range = computed<[number, number] | undefined>({
   get: () => dataStyleStore.meshPolygonsPolygonAttributeRange(id.value),
   set: async (newValue) => {
     const [minimum, maximum] = newValue;
@@ -159,7 +155,7 @@ const polygon_attribute_range = computed({
     hybridViewerStore.remoteRender();
   },
 });
-const polygon_attribute_color_map = computed({
+const polygon_attribute_color_map = computed<Map<string, RGBAColor> | undefined>({
   get: () => dataStyleStore.meshPolygonsPolygonAttributeColorMap(id.value),
   set: async (newValue) => {
     await applyBatchStyle(id.value, (targetId: string) =>

package/app/components/Viewer/Ui.vue

@@ -23,7 +23,7 @@ const menuStore = useMenuStore();
 const viewerStore = useViewerStore();
 const hybridViewerStore = useHybridViewerStore();
 
-function stopHoverHighlight() {
+function stopHoverHighlight(): void {
   hybridViewerStore.is_hover_highlight = false;
   hybridViewerStore.clearHoverHighlight();
 }
@@ -57,7 +57,7 @@ const {
   get_viewer_id: trigger_picker,
 } = useOverlappingPicker();
 
-function get_viewer_id(x: number, y: number) {
+function get_viewer_id(x: number, y: number): string {
   const instance = getCurrentInstance();
   const containerRect = instance?.proxy?.$el
     ?.closest?.('[data-testid="hybridViewer"]')

package/app/components/Wrapper.vue

@@ -1,13 +1,10 @@
 <script setup lang="ts">
-// Not auto-fixable (eslint's sort-imports core rule has no autofixer) and this file's import order doesn't match its syntax-kind-then-alphabetical requirement - left as-is rather than manually reordered across the codebase for a purely cosmetic rule.
-// oxlint-disable eslint/sort-imports
+import type { JsonRpcSchema } from "@ogw_shared/utils/types.js";
 import Launcher from "@ogw_front/components/Launcher.vue";
 import PackagesVersions from "@ogw_front/components/PackagesVersions.vue";
 import Stepper from "@ogw_front/components/Stepper.vue";
-
 import { useInfraStore } from "@ogw_front/stores/infra";
 import { useStepperTree } from "@ogw_front/composables/stepper_tree";
-import type { JsonRpcSchema } from "@ogw_shared/utils/types.js";
 
 const infraStore = useInfraStore();
 

package/app/components/ZScaling.vue

@@ -13,11 +13,13 @@ interface Props {
 
 const { width = DEFAULT_PANEL_WIDTH, escapeFunction = undefined } = defineProps<Props>();
 
-const emit = defineEmits<{
+interface Emits {
   apply: [];
-}>();
+}
+
+const emit = defineEmits<Emits>();
 
-function apply() {
+function apply(): void {
   emit("apply");
   show.value = false;
 }

package/app/components/csv-preview/CsvPreviewer.vue

@@ -33,26 +33,28 @@ const MIN_AVG_COUNT = 1.5;
 const MAX_VARIANCE = 0.5;
 const PREVIEW_ROWS_LIMIT = 101;
 
-const emit = defineEmits<{
+interface Emits {
   "update:modelValue": [value: boolean];
   confirm: [result: CsvParseResult];
-}>();
+}
+
+const emit = defineEmits<Emits>();
 
-const separator = ref(",");
-const headerRow = ref(0);
-const firstRow = ref(1);
+const separator = ref<string>(",");
+const headerRow = ref<number>(0);
+const firstRow = ref<number>(1);
 
 const xColumn = ref<string | undefined>(undefined);
 const yColumn = ref<string | undefined>(undefined);
 const zColumn = ref<string | undefined>(undefined);
 
-const rawContent = ref("");
+const rawContent = ref<string>("");
 const previewRows = ref<CsvRow[]>([]);
 const previewHeaders = ref<CsvHeader[]>([]);
-const loading = ref(false);
+const loading = ref<boolean>(false);
 const toggleLoading = useToggle(loading);
 
-function autoDetectSeparator(content: string) {
+function autoDetectSeparator(content: string): string {
   const lines = content
     .slice(0, MAX_CONTENT_SLICE)
     .split(/\r?\n/u)
@@ -75,35 +77,33 @@ function autoDetectSeparator(content: string) {
   return best;
 }
 
-function parseContent() {
-  if (!rawContent.value) {
-    return;
+function splitLine(line: string): string[] {
+  if (!separator.value) {
+    return [line];
   }
-
-  const allLines = rawContent.value.split(/\r?\n/u).filter((line) => line.trim() !== "");
-
-  function splitLine(line: string): string[] {
-    if (!separator.value) {
-      return [line];
+  const result = [];
+  let current = "";
+  let inQuotes = false;
+  for (const char of line) {
+    if (char === '"') {
+      inQuotes = !inQuotes;
+    } else if (char === separator.value && !inQuotes) {
+      result.push(current.trim());
+      current = "";
+    } else {
+      current += char;
     }
-    const result = [];
-    let current = "";
-    let inQuotes = false;
-    for (let index = 0; index < line.length; index += 1) {
-      const char = line[index];
-      if (char === '"') {
-        inQuotes = !inQuotes;
-      } else if (char === separator.value && !inQuotes) {
-        result.push(current.trim());
-        current = "";
-      } else {
-        current += char;
-      }
-    }
-    result.push(current.trim());
-    return result;
   }
+  result.push(current.trim());
+  return result;
+}
 
+function parseContent(): string[] {
+  if (!rawContent.value) {
+    return [];
+  }
+
+  const allLines = rawContent.value.split(/\r?\n/u).filter((line) => line.trim() !== "");
   const headerLine = allLines[headerRow.value];
   const rawHeaders = headerLine ? splitLine(headerLine) : [];
 
@@ -125,7 +125,7 @@ function parseContent() {
   });
 }
 
-function readAndParse() {
+function readAndParse(): void {
   if (!file) {
     return;
   }
@@ -145,7 +145,8 @@ function readAndParse() {
   });
   reader.readAsText(file, "utf8");
 }
-const computedResult = computed(() => {
+
+const computedResult = computed<CsvParseResult>(() => {
   const xIndex = previewHeaders.value.findIndex((header) => header.key === xColumn.value);
   const yIndex = previewHeaders.value.findIndex((header) => header.key === yColumn.value);
   const zIndex = previewHeaders.value.findIndex((header) => header.key === zColumn.value);
@@ -160,7 +161,7 @@ const computedResult = computed(() => {
   };
 });
 
-const isFormValid = computed(
+const isFormValid = computed<boolean>(
   () =>
     separator.value !== "" &&
     separator.value !== undefined &&
@@ -197,7 +198,7 @@ watch(
   },
 );
 
-function onConfirm() {
+function onConfirm(): void {
   emit("confirm", computedResult.value);
   emit("update:modelValue", false);
 }

package/app/components/csv-preview/CsvSettings.vue

@@ -17,9 +17,11 @@ const xColumn = defineModel<string>("xColumn");
 const yColumn = defineModel<string>("yColumn");
 const zColumn = defineModel<string>("zColumn");
 
-const emit = defineEmits<{
+interface Emits {
   "update:separator": [value: string];
-}>();
+}
+
+const emit = defineEmits<Emits>();
 
 const separators = [
   { title: "Comma (,)", value: "," },
@@ -30,8 +32,8 @@ const separators = [
   { title: "Custom", value: "custom" },
 ];
 
-const selectedType = ref(",");
-const customValue = ref("");
+const selectedType = ref<string>(",");
+const customValue = ref<string>("");
 
 watch(
   () => separator,

package/app/components/csv-preview/CsvTable.vue

@@ -32,7 +32,7 @@ const {
   firstRow = 1,
 } = defineProps<Props>();
 
-function getColumnClass(key: string) {
+function getColumnClass(key: string): string {
   if (key === coordinates.x) {
     return "x-col-highlight";
   }

package/app/composables/batch_style.ts

@@ -1,7 +1,7 @@
 import { useDataStore } from "@ogw_front/stores/data";
 import { useTreeviewStore } from "@ogw_front/stores/treeview";
 
-export function useBatchStyle() {
+export function useBatchStyle(): { applyBatchStyle: typeof applyBatchStyle } {
   const treeviewStore = useTreeviewStore();
   const dataStore = useDataStore();
 
@@ -22,17 +22,16 @@ export function useBatchStyle() {
       const promises = treeviewStore.activeItems.map(async (selectedId) => {
         try {
           const item = await dataStore.item(selectedId);
-          if (item && item.geode_object_type === targetType) {
+          if (item.geode_object_type === targetType) {
             await action(selectedId);
           }
-        } catch (error) {
-          console.error("Failed to apply batch style to item", selectedId, error);
+        } catch {
+          // Ignore items that fail to load; batch style continues for the rest.
         }
       });
 
       await Promise.all(promises);
-    } catch (error) {
-      console.error("Failed to fetch current item for batch style", id, error);
+    } catch {
       await action(id);
     }
   }