matterviz 0.4.0 → 0.4.1

package/dist/trajectory/Trajectory.svelte

@@ -1,15 +1,17 @@
 <script lang="ts">
-  import type { ShowControlsProp } from '../controls'
-  import { normalize_show_controls } from '../controls'
+  import { normalize_show_controls, type ShowControlsProp } from '../controls'
   import type { ElementSymbol } from '../element'
   import EmptyState from '../EmptyState.svelte'
+  import { StatusMessage } from '../feedback'
   import Spinner from '../feedback/Spinner.svelte'
   import Icon from '../Icon.svelte'
-  import { handle_url_drop, load_from_url } from '../io'
+  import { drag_over_handlers, handle_url_drop, load_from_url } from '../io'
   import { forward_window_keydown, handle_and_prevent } from '../keyboard'
   import { format_num, trajectory_property_config } from '../labels'
+  import type { Vec2 } from '../math'
   import { sanitize_html } from '../sanitize'
-  import { toggle_fullscreen } from '../layout'
+  import { FullscreenButton, type FullscreenToggleProp, toggle_fullscreen } from '../layout'
+  import { sync_fullscreen } from '../layout/fullscreen.svelte'
   import type { ControlsConfig, DataSeries, Orientation, Point } from '../plot'
   import type { ScatterHandlerProps } from '../plot/core/types'
   import { Histogram, ScatterPlot } from '../plot'
@@ -99,6 +101,10 @@
     atom_type_mapping,
     plot_skimming = true,
     hovered = $bindable(false),
+    controls_open = $bindable(false),
+    info_pane_open = $bindable(false),
+    wrapper = $bindable(),
+    fullscreen = $bindable(false),
     ...rest
   }: EventHandlers & HTMLAttributes<HTMLDivElement> & {
     // trajectory data - can be provided directly or loaded from file
@@ -134,7 +140,7 @@
     // Control names: 'filename', 'nav', 'step', 'fps', 'info-pane', 'export-pane', 'view-mode', 'fullscreen'
     show_controls?: ShowControlsProp
     // show/hide the fullscreen button
-    fullscreen_toggle?: Snippet<[{ fullscreen: boolean }]> | boolean
+    fullscreen_toggle?: FullscreenToggleProp
     // automatically start playing when trajectory data is loaded
     auto_play?: boolean
     // display mode: 'structure+scatter' (default), 'structure' (only structure), 'scatter' (only scatter), 'histogram' (only histogram), 'structure+histogram' (structure with histogram)
@@ -160,24 +166,7 @@
     // - e.g. {energy: 'Total Energy', volume: 'Cell Volume', force_max: 'Max Force'}
     // - merged with built-in trajectory_property_config
     ELEM_PROPERTY_LABELS?: Record<string, string>
-    // units configuration - developers can override these (deprecated - use ELEM_PROPERTY_LABELS instead)
-    units?: {
-      energy?: string
-      energy_per_atom?: string
-      force_max?: string
-      force_norm?: string
-      stress_max?: string
-      volume?: string
-      density?: string
-      temperature?: string
-      pressure?: string
-      length?: string
-      a?: string
-      b?: string
-      c?: string
-      [key: string]: string | undefined
-    }
-    fps_range?: [number, number] // allowed FPS range [min_fps, max_fps]
+    fps_range?: Vec2 // allowed FPS range [min_fps, max_fps]
     fps?: number // frame rate for playback
     // Loading options for large files
     loading_options?: LoadingOptions
@@ -187,11 +176,29 @@
     plot_skimming?: boolean
     // bindable: true while the pointer is over the viewer (drives hover-scoped shortcuts)
     hovered?: boolean
+    // bindable: whether the (structure) controls pane is currently open
+    controls_open?: boolean
+    // bindable: whether the trajectory info pane is currently open
+    info_pane_open?: boolean
+    // bindable: top-level wrapper element
+    wrapper?: HTMLDivElement
+    // bindable: fullscreen state
+    fullscreen?: boolean
   } = $props()
 
   let dragover = $state(false)
   let loading = $state(false)
   let error_msg = $state<string | null>(null)
+  // Non-fatal parse warnings from trajectory metadata (set by parser via attach_parse_warnings); dismissible
+  let parse_warning_msg = $state<string | undefined>(undefined)
+  $effect(() => {
+    const warnings = trajectory?.metadata?.parse_warnings
+    parse_warning_msg = Array.isArray(warnings) && warnings.length > 0
+      ? `${warnings.length} parse warning${warnings.length > 1 ? `s` : ``}: ${
+        warnings.join(`; `)
+      }`
+      : undefined
+  })
   let is_playing = $state(false)
   let play_interval: ReturnType<typeof setInterval> | undefined = $state(undefined)
 
@@ -207,8 +214,6 @@
   let current_file_path = $state<string | null>(null)
   let file_size = $state<number | undefined>(undefined)
   let file_object = $state<File | null>(null)
-  let wrapper = $state<HTMLDivElement | undefined>(undefined)
-  let info_pane_open = $state(false)
   let parsing_progress = $state<ParseProgress | null>(null)
   let element_size = $state({ width: 0, height: 0 })
   let filename_copied = $state(false)
@@ -868,24 +873,23 @@
 
   // Separate state variables for each pane to match component prop types
   let structure_info_open = $state(false)
-  let structure_controls_open = $state(false)
   let scatter_controls = $state<ControlsConfig>({ open: false })
   let trajectory_export_open = $state(false)
-  let fullscreen = $state(false)
-</script>
 
-<svelte:document
-  onfullscreenchange={() => {
-    fullscreen = !!document.fullscreenElement
-    on_fullscreen_change?.({ trajectory, fullscreen })
-  }}
-/>
+  sync_fullscreen({
+    get_wrapper: () => wrapper,
+    get_fullscreen: () => fullscreen,
+    set_fullscreen: (val) => (fullscreen = val),
+    bg_css_var: `--traj-bg-fullscreen`,
+    on_change: (val) => on_fullscreen_change?.({ trajectory, fullscreen: val }),
+  })
+</script>
 
 <svelte:window onkeydown={forward_window_keydown(() => hovered, onkeydown)} />
 
 <div
   class:dragover
-  class:active={is_playing || structure_info_open || structure_controls_open ||
+  class:active={is_playing || structure_info_open || controls_open ||
   scatter_controls.open || trajectory_export_open || info_pane_open}
   bind:this={wrapper}
   bind:clientWidth={element_size.width}
@@ -896,15 +900,7 @@
   onmouseenter={() => (hovered = true)}
   onmouseleave={() => (hovered = false)}
   ondrop={handle_file_drop}
-  ondragover={(event) => {
-    event.preventDefault()
-    if (!allow_file_drop) return
-    dragover = true
-  }}
-  ondragleave={(event) => {
-    event.preventDefault()
-    dragover = false
-  }}
+  {...drag_over_handlers({ allow: () => allow_file_drop, set_dragover: (over) => dragover = over })}
   onclick={handle_click_outside}
   onkeydown={handle_and_prevent(onkeydown)}
   {...rest}
@@ -928,6 +924,14 @@
       {error_snippet}
     />
   {:else if trajectory}
+    {#if parse_warning_msg}
+      <StatusMessage
+        bind:message={parse_warning_msg}
+        type="warning"
+        dismissible
+        style="position: absolute; bottom: 4pt; left: 4pt; right: 4pt; z-index: 2; font-size: 0.85em"
+      />
+    {/if}
     <!-- Trajectory Controls -->
     {#if controls_config.mode !== `never`}
       <div
@@ -1150,22 +1154,14 @@
               </div>
             {/if}
             <!-- Fullscreen button - rightmost position -->
-            {#if fullscreen_toggle &&
-          controls_config.visible(`fullscreen`)}
-              <button
-                type="button"
-                onclick={() => fullscreen_toggle && toggle_fullscreen(wrapper)}
-                title="{fullscreen ? `Exit` : `Enter`} fullscreen"
+            {#if fullscreen_toggle && controls_config.visible(`fullscreen`)}
+              <FullscreenButton
+                {fullscreen}
+                toggle={fullscreen_toggle}
+                {wrapper}
                 aria-label="{fullscreen ? `Exit` : `Enter`} fullscreen"
-                aria-pressed={fullscreen}
                 class="fullscreen-button"
-              >
-                {#if typeof fullscreen_toggle === `function`}
-                  {@render fullscreen_toggle({ fullscreen })}
-                {:else}
-                  <Icon icon="{fullscreen ? `Exit` : ``}Fullscreen" />
-                {/if}
-              </button>
+              />
             {/if}
           </div>
         {/if}
@@ -1189,7 +1185,7 @@
             show_image_atoms: false, // Default to false to avoid atoms popping in/out at cell edges
             ...structure_props,
           }}
-          bind:controls_open={structure_controls_open}
+          bind:controls_open
           bind:info_pane_open={structure_info_open}
           bind:hidden_elements
         />
@@ -1316,7 +1312,7 @@
       height: 100vh !important;
       width: 100vw !important;
       border-radius: 0 !important;
-      background: var(--surface-bg);
+      background: var(--traj-bg-fullscreen, var(--surface-bg));
       overflow: hidden;
     }
     &.horizontal .content-area {
@@ -1458,7 +1454,8 @@
       opacity: 0;
     }
   }
-  .fullscreen-button {
+  /* :global pierces into FullscreenButton's markup (anchored on local .info-section) */
+  .info-section :global(.fullscreen-button) {
     background: transparent !important;
     padding: 0;
     &:hover:not(:disabled) {

package/dist/trajectory/Trajectory.svelte.d.ts

@@ -1,5 +1,7 @@
-import type { ShowControlsProp } from '../controls';
+import { type ShowControlsProp } from '../controls';
 import Spinner from '../feedback/Spinner.svelte';
+import type { Vec2 } from '../math';
+import { type FullscreenToggleProp } from '../layout';
 import type { Orientation } from '../plot';
 import { Histogram, ScatterPlot } from '../plot';
 import Structure from '../structure/Structure.svelte';
@@ -42,37 +44,23 @@ type $$ComponentProps = EventHandlers & HTMLAttributes<HTMLDivElement> & {
         on_dismiss: () => void;
     }]>;
     show_controls?: ShowControlsProp;
-    fullscreen_toggle?: Snippet<[{
-        fullscreen: boolean;
-    }]> | boolean;
+    fullscreen_toggle?: FullscreenToggleProp;
     auto_play?: boolean;
     display_mode?: `structure+scatter` | `structure` | `scatter` | `histogram` | `structure+histogram`;
     step_labels?: number | number[];
     visible_properties?: string[];
     ELEM_PROPERTY_LABELS?: Record<string, string>;
-    units?: {
-        energy?: string;
-        energy_per_atom?: string;
-        force_max?: string;
-        force_norm?: string;
-        stress_max?: string;
-        volume?: string;
-        density?: string;
-        temperature?: string;
-        pressure?: string;
-        length?: string;
-        a?: string;
-        b?: string;
-        c?: string;
-        [key: string]: string | undefined;
-    };
-    fps_range?: [number, number];
+    fps_range?: Vec2;
     fps?: number;
     loading_options?: LoadingOptions;
     atom_type_mapping?: AtomTypeMapping;
     plot_skimming?: boolean;
     hovered?: boolean;
+    controls_open?: boolean;
+    info_pane_open?: boolean;
+    wrapper?: HTMLDivElement;
+    fullscreen?: boolean;
 };
-declare const Trajectory: import("svelte").Component<$$ComponentProps, {}, "trajectory" | "hovered" | "fps" | "display_mode" | "current_step_idx" | "visible_properties">;
+declare const Trajectory: import("svelte").Component<$$ComponentProps, {}, "trajectory" | "fullscreen" | "wrapper" | "hovered" | "controls_open" | "fps" | "info_pane_open" | "display_mode" | "current_step_idx" | "visible_properties">;
 type Trajectory = ReturnType<typeof Trajectory>;
 export default Trajectory;

package/dist/trajectory/TrajectoryExportPane.svelte

@@ -1,10 +1,13 @@
 <script lang="ts">
+  import type { PaneProps, PaneToggleProps } from '../overlays'
   import {
+    estimate_video_bitrate,
     export_trajectory_video,
     get_ffmpeg_conversion_command,
+    observe_canvas_presence,
   } from '../io/export'
+  import ExportPane from '../io/ExportPane.svelte'
   import SettingsSection from '../layout/SettingsSection.svelte'
-  import DraggablePane from '../overlays/DraggablePane.svelte'
   import type { TrajectoryType } from './'
   import type { ComponentProps } from 'svelte'
   import { tooltip } from 'svelte-multiselect/attachments'
@@ -36,8 +39,8 @@
     // Function to change trajectory step during export
     on_step_change?: (step_idx: number) => Promise<void> | void
     // Pane customization
-    pane_props?: ComponentProps<typeof DraggablePane>[`pane_props`]
-    toggle_props?: ComponentProps<typeof DraggablePane>[`toggle_props`]
+    pane_props?: PaneProps
+    toggle_props?: PaneToggleProps
   } = $props()
 
   let is_exporting = $state(false)
@@ -58,7 +61,7 @@
   let file_size_mb = $derived.by(() => {
     if (!canvas) return 0
     const pixels = canvas.width * canvas.height * resolution_multiplier ** 2
-    const bitrate = Math.max(1e6, Math.min(pixels * video_fps * 0.1, 2e8))
+    const bitrate = estimate_video_bitrate(pixels, video_fps)
     return (bitrate * export_frame_count / video_fps) / 8 / 1024 / 1024
   })
 
@@ -138,24 +141,12 @@
 
   let has_canvas = $state(false)
 
-  $effect(() => {
-    if (!wrapper) {
-      has_canvas = false
-      return
-    }
-    const check = () => (has_canvas = Boolean(wrapper.querySelector(`canvas`)))
-    check()
-    const observer = new MutationObserver(check)
-    observer.observe(wrapper, { childList: true, subtree: true })
-    return () => observer.disconnect()
-  })
+  $effect(() => observe_canvas_presence(wrapper, (val) => (has_canvas = val)))
 </script>
 
-<DraggablePane
-  bind:show={export_pane_open}
-  open_icon="Cross"
-  closed_icon="Export"
-  pane_props={{ ...pane_props, class: `export-pane ${pane_props?.class ?? ``}` }}
+<ExportPane
+  bind:export_pane_open
+  {pane_props}
   toggle_props={{
     title: export_pane_open ? `` : `Export Trajectory`,
     ...toggle_props,
@@ -194,14 +185,14 @@
         Resolution
         <div class="resolution-buttons">
           {#each [0.5, 1, 2, 4, 8] as multiplier (multiplier)}
-            {@const w = canvas ? Math.round(canvas.width * multiplier) : 0}
-            {@const h = canvas ? Math.round(canvas.height * multiplier) : 0}
+            {@const width_px = canvas ? Math.round(canvas.width * multiplier) : 0}
+            {@const height_px = canvas ? Math.round(canvas.height * multiplier) : 0}
             <button
               type="button"
               class:active={resolution_multiplier === multiplier}
               onclick={() => (resolution_multiplier = multiplier)}
               {@attach tooltip({
-                content: canvas ? `${multiplier}x (${w}×${h})` : `${multiplier}x`,
+                content: canvas ? `${multiplier}x (${width_px}×${height_px})` : `${multiplier}x`,
               })}
             >
               {multiplier}x
@@ -292,7 +283,7 @@
       <div class="warning">Waiting for canvas...</div>
     {/if}
   {/if}
-</DraggablePane>
+</ExportPane>
 
 <style>
   .field-label {

package/dist/trajectory/TrajectoryExportPane.svelte.d.ts

@@ -1,6 +1,5 @@
-import DraggablePane from '../overlays/DraggablePane.svelte';
+import type { PaneProps, PaneToggleProps } from '../overlays';
 import type { TrajectoryType } from './';
-import type { ComponentProps } from 'svelte';
 type $$ComponentProps = {
     export_pane_open?: boolean;
     trajectory?: TrajectoryType;
@@ -9,9 +8,9 @@ type $$ComponentProps = {
     video_fps?: number;
     resolution_multiplier?: number;
     on_step_change?: (step_idx: number) => Promise<void> | void;
-    pane_props?: ComponentProps<typeof DraggablePane>[`pane_props`];
-    toggle_props?: ComponentProps<typeof DraggablePane>[`toggle_props`];
+    pane_props?: PaneProps;
+    toggle_props?: PaneToggleProps;
 };
-declare const TrajectoryExportPane: import("svelte").Component<$$ComponentProps, {}, "export_pane_open" | "resolution_multiplier" | "video_fps">;
+declare const TrajectoryExportPane: import("svelte").Component<$$ComponentProps, {}, "resolution_multiplier" | "export_pane_open" | "video_fps">;
 type TrajectoryExportPane = ReturnType<typeof TrajectoryExportPane>;
 export default TrajectoryExportPane;

package/dist/trajectory/TrajectoryInfoPane.svelte

@@ -1,4 +1,5 @@
 <script lang="ts">
+  import type { PaneProps, PaneToggleProps } from '../overlays'
   import type { InfoItem } from '../layout'
   import InfoPaneCards from '../overlays/InfoPaneCards.svelte'
   import { format_num } from '../labels'
@@ -28,8 +29,8 @@
     file_size?: number | null
     file_object?: File | null
     pane_open?: boolean
-    toggle_props?: ComponentProps<typeof DraggablePane>[`toggle_props`]
-    pane_props?: ComponentProps<typeof DraggablePane>[`pane_props`]
+    toggle_props?: PaneToggleProps
+    pane_props?: PaneProps
   } = $props()
 
   // Helper functions

package/dist/trajectory/TrajectoryInfoPane.svelte.d.ts

@@ -1,3 +1,4 @@
+import type { PaneProps, PaneToggleProps } from '../overlays';
 import DraggablePane from '../overlays/DraggablePane.svelte';
 import type { ComponentProps } from 'svelte';
 import type { TrajectoryType } from './index';
@@ -9,8 +10,8 @@ type $$ComponentProps = Omit<ComponentProps<typeof DraggablePane>, `children`> &
     file_size?: number | null;
     file_object?: File | null;
     pane_open?: boolean;
-    toggle_props?: ComponentProps<typeof DraggablePane>[`toggle_props`];
-    pane_props?: ComponentProps<typeof DraggablePane>[`pane_props`];
+    toggle_props?: PaneToggleProps;
+    pane_props?: PaneProps;
 };
 declare const TrajectoryInfoPane: import("svelte").Component<$$ComponentProps, {}, "pane_open">;
 type TrajectoryInfoPane = ReturnType<typeof TrajectoryInfoPane>;

package/dist/trajectory/constants.js

@@ -1,7 +1,11 @@
 // Constants for trajectory parsing and large file handling
+import { DEFAULTS } from '../settings';
 export const MAX_SAFE_STRING_LENGTH = 0x1fffffe8 * 0.5; // 50% of JS max string length as safety
 export const MAX_METADATA_SIZE = 50 * 1024 * 1024; // 50MB limit for metadata
 export const LARGE_FILE_THRESHOLD = 400 * 1024 * 1024; // 400MB
 export const INDEX_SAMPLE_RATE = 100; // Default sample rate for frame indexing
-export const MAX_BIN_FILE_SIZE = 100 * 1024 * 1024; // 100MB default for ArrayBuffer files
-export const MAX_TEXT_FILE_SIZE = 50 * 1024 * 1024; // 50MB default for string files
+// Fallback thresholds for component usage without loading_options, derived from the
+// settings schema so settings-driven contexts (e.g. the VSCode extension) and direct
+// component use agree on when large-file/indexed loading kicks in.
+export const MAX_BIN_FILE_SIZE = DEFAULTS.trajectory.bin_file_threshold; // 50MB
+export const MAX_TEXT_FILE_SIZE = DEFAULTS.trajectory.text_file_threshold; // 25MB

package/dist/trajectory/extract.js

@@ -1,5 +1,6 @@
 // Data extraction functions for trajectory analysis and plotting
 import { get_density } from '../structure/index';
+import { calc_force_stats, copy_numeric_fields } from './helpers';
 // Common data extractor that extracts energy and structural properties
 export const energy_data_extractor = (frame) => {
     const data = {
@@ -7,18 +8,13 @@ export const energy_data_extractor = (frame) => {
     };
     if (frame.metadata) {
         // Extract energy-related properties
-        const energy_fields = [
+        copy_numeric_fields(data, frame.metadata, [
             `energy`,
             `energy_per_atom`,
             `potential_energy`,
             `kinetic_energy`,
             `total_energy`,
-        ];
-        for (const field of energy_fields) {
-            if (field in frame.metadata && typeof frame.metadata[field] === `number`) {
-                data[field] = frame.metadata[field];
-            }
-        }
+        ]);
     }
     return data;
 };
@@ -30,13 +26,8 @@ export const force_stress_data_extractor = (frame) => {
     if (frame.metadata) {
         // Calculate force properties from forces array if available (preferred)
         if (frame.metadata.forces && Array.isArray(frame.metadata.forces)) {
-            const forces = frame.metadata.forces;
-            if (forces.length > 0) {
-                const force_magnitudes = forces.map((force) => Math.hypot(...force));
-                data.force_max = Math.max(...force_magnitudes);
-                // Calculate RMS (root mean square) of force magnitudes
-                data.force_norm = Math.sqrt(force_magnitudes.reduce((sum, f) => sum + f ** 2, 0) / force_magnitudes.length);
-            }
+            // Object.assign ignores the null calc_force_stats returns for empty forces
+            Object.assign(data, calc_force_stats(frame.metadata.forces));
         }
         else {
             // Fallback to metadata values if forces array not available
@@ -52,12 +43,12 @@ export const force_stress_data_extractor = (frame) => {
             }
         }
         // Extract other stress and pressure properties (no duplicates expected)
-        const other_stress_fields = [`stress_max`, `stress_frobenius`, `stress_trace`, `pressure`];
-        for (const field of other_stress_fields) {
-            if (field in frame.metadata && typeof frame.metadata[field] === `number`) {
-                data[field] = frame.metadata[field];
-            }
-        }
+        copy_numeric_fields(data, frame.metadata, [
+            `stress_max`,
+            `stress_frobenius`,
+            `stress_trace`,
+            `pressure`,
+        ]);
     }
     return data;
 };
@@ -79,31 +70,20 @@ export const structural_data_extractor = (frame) => {
     }
     if (frame.metadata) {
         // Extract other structural properties, avoiding volume duplicate
-        const structural_fields = [`temperature`];
-        for (const field of structural_fields) {
-            if (field in frame.metadata && typeof frame.metadata[field] === `number`)
-                data[field] = frame.metadata[field];
-        }
-        // Handle density separately - prefer metadata, but calculate if not available
-        if (frame.metadata.density && typeof frame.metadata.density === `number`) {
+        copy_numeric_fields(data, frame.metadata, [`temperature`]);
+        // Prefer metadata density (fall back to calculating from structure below).
+        // Finite-number check (not truthiness) so a legitimate density of 0 is kept.
+        if (typeof frame.metadata.density === `number` &&
+            Number.isFinite(frame.metadata.density)) {
             data.density = frame.metadata.density;
         }
-        else if (`lattice` in frame.structure) {
-            try {
-                data.density = get_density(frame.structure);
-            }
-            catch (error) {
-                console.warn(`Failed to calculate density for frame ${frame.step}:`, error);
-            }
-        }
         // Only use metadata volume if lattice volume is not available
         if (!data.volume && frame.metadata.volume && typeof frame.metadata.volume === `number`) {
             data.volume = frame.metadata.volume;
         }
         // Note: pressure is handled by force_stress_data_extractor to avoid duplication
     }
-    else if (`lattice` in frame.structure) {
-        // Calculate density even when no metadata is available
+    if (data.density === undefined && `lattice` in frame.structure) {
         try {
             data.density = get_density(frame.structure);
         }

package/dist/trajectory/format-detect.d.ts

@@ -1,4 +1,3 @@
-export declare function strip_compression_extensions(filename: string): string;
 export declare function ext_hint(filename: string | undefined, ext_regex: RegExp): boolean | null;
 export declare const FORMAT_PATTERNS: {
     readonly ase: (data: unknown, filename?: string) => boolean;

package/dist/trajectory/format-detect.js

@@ -1,13 +1,7 @@
 // Format detection for trajectory files
-import { COMPRESSION_EXTENSIONS_REGEX, CONFIG_DIRS_REGEX, MD_SIM_EXCLUDE_REGEX, TRAJ_EXTENSIONS_REGEX, TRAJ_FALLBACK_EXTENSIONS_REGEX, TRAJ_KEYWORDS_SIMPLE_REGEX, XDATCAR_REGEX, } from '../constants';
+import { CONFIG_DIRS_REGEX, MD_SIM_EXCLUDE_REGEX, TRAJ_EXTENSIONS_REGEX, TRAJ_FALLBACK_EXTENSIONS_REGEX, TRAJ_KEYWORDS_SIMPLE_REGEX, XDATCAR_REGEX, } from '../constants';
+import { strip_compression_extensions } from '../io';
 import { count_xyz_frames } from './helpers';
-export function strip_compression_extensions(filename) {
-    let base_name = filename.toLowerCase();
-    while (COMPRESSION_EXTENSIONS_REGEX.test(base_name)) {
-        base_name = base_name.replace(COMPRESSION_EXTENSIONS_REGEX, ``);
-    }
-    return base_name;
-}
 // Extensions that explicitly identify a format — when present, format detection trusts
 // the extension instead of sniffing content
 const KNOWN_FORMAT_EXT_REGEX = /\.(xyz|extxyz|traj|h5|hdf5|lammpstrj|json|cif|poscar|vasp|yaml|yml|xml|csv)$/;
@@ -40,7 +34,7 @@ export const FORMAT_PATTERNS = {
         if (!(data instanceof ArrayBuffer) || data.byteLength < 8)
             return false;
         const signature = new Uint8Array(data.slice(0, 8));
-        return [0x89, 0x48, 0x44, 0x46, 0x0d, 0x0a, 0x1a, 0x0a].every((b, idx) => signature[idx] === b);
+        return [0x89, 0x48, 0x44, 0x46, 0x0d, 0x0a, 0x1a, 0x0a].every((byte, idx) => signature[idx] === byte);
     },
     vasp: (data, filename) => {
         const basename = filename?.toLowerCase().split(`/`).pop() ?? ``;

package/dist/trajectory/frame-reader.d.ts

@@ -12,6 +12,5 @@ export declare class TrajFrameReader implements FrameLoader {
     }, on_progress?: (progress: ParseProgress) => void): Promise<TrajectoryMetadata[]>;
     private load_xyz_frame;
     private load_ase_frame;
-    private parse_xyz_metadata;
     private parse_ase_metadata;
 }