Frontend: allow using outdated param sets

Author: Williangalvani

core/frontend/src/components/vehiclesetup/overview/ParamSets.vue

@@ -44,16 +44,37 @@
           These are the recommended parameter sets for your vehicle and firmware version. Curated by Blue Robotics
         </p>
       </v-card-text>
-      <v-card-actions>
-        <v-btn
-          v-for="(paramSet, name) in filtered_param_sets"
-          :key="name"
-          color="primary"
-          @click="loadParams(name, paramSet)"
+      <v-card-actions class="flex-wrap">
+        <v-tooltip
+          v-for="item in filtered_param_sets"
+          :key="item.name"
+          bottom
+          :disabled="!item.outdated"
         >
-          {{ name.split('/').pop() }}
-        </v-btn>
-        <p v-if="(Object.keys(filtered_param_sets).length === 0)">
+          <template #activator="{ on }">
+            <v-btn
+              :color="item.outdated ? 'warning' : 'primary'"
+              :outlined="item.outdated"
+              @click="loadParams(item.name, item.paramset)"
+              v-on="on"
+            >
+              <v-icon
+                v-if="item.outdated"
+                left
+                small
+              >
+                mdi-alert
+              </v-icon>
+              {{ item.name.split('/').pop()?.replace(/\.params$/i, '') }}
+              <span class="version-tag ml-2">v{{ item.version_label }}</span>
+            </v-btn>
+          </template>
+          <span>
+            Outdated: built for firmware {{ item.version_label }} (current is
+            {{ version ? `${version.major}.${version.minor}` : 'unknown' }})
+          </span>
+        </v-tooltip>
+        <p v-if="filtered_param_sets.length === 0">
           No parameters available for this setup
         </p>
       </v-card-actions>
@@ -95,6 +116,18 @@ import { frontend_service } from '@/types/frontend_services'
 const notifier = new Notifier(frontend_service)
 const REPOSITORY_URL = 'https://docs.bluerobotics.com/Blueos-Parameter-Repository/params_v1.json'
 
+interface FilteredParamSet {
+  name: string
+  paramset: Dictionary<number>
+  version: SemVer
+  version_label: string
+  outdated: boolean
+}
+
+function escapeRegex(value: string): string {
+  return value.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')
+}
+
 export default Vue.extend({
   name: 'ParamSets',
   components: {
@@ -122,30 +155,73 @@ export default Vue.extend({
     version(): SemVer | undefined {
       return autopilot.firmware_info?.version
     },
-    filtered_param_sets(): Dictionary<Dictionary<number>> | undefined {
-      const fw_patch = `${this.vehicle}/${this.version}/${this.board}`
-      const fw_minor = `${this.vehicle}/${this.version?.major}.${this.version?.minor}/${this.board}`
-      const fw_major = `${this.vehicle}/${this.version?.major}/${this.board}`
-
-      // returns a new dict where the keys start with the fullname
-      // e.g. "ArduSub/BlueROV2/4.0.3" -> "ArduSub/BlueROV2/4.0.3/BlueROV2"
-
-      let fw_params = {}
-      // try to find a paramset that matches the firmware version, starting from patch and walking up to major
-      for (const string of [fw_patch, fw_minor, fw_major]) {
-        fw_params = Object.fromEntries(
-          Object.entries(this.all_param_sets).filter(
-            // We add a trailing slash to avoid matching Navigator and Navigator64, or any board with suffix
-            ([name]) => name.toLocaleLowerCase().includes(`${string.toLowerCase()}/`),
-          ),
-        )
-        if (Object.keys(fw_params).length > 0) {
-          break
+    filtered_param_sets(): FilteredParamSet[] {
+      if (!this.vehicle || !this.board || !this.version) {
+        return []
+      }
+
+      // Match keys shaped like ".../<vehicle>/<X.Y[.Z]>/<board>/..."
+      // Trailing slash on the board avoids matching Navigator vs Navigator64, etc.
+      const pattern = new RegExp(
+        `/${escapeRegex(this.vehicle)}/(\\d+\\.\\d+(?:\\.\\d+)?)/${escapeRegex(this.board)}/`,
+        'i',
+      )
+
+      const current = this.version
+      const candidates: { name: string, paramset: Dictionary<number>, version: SemVer,
+        version_label: string, has_patch: boolean }[] = []
+
+      for (const [name, paramset] of Object.entries(this.all_param_sets)) {
+        const match = name.match(pattern)
+        if (!match) {
+          continue
+        }
+
+        const version_label = match[1]
+        const has_patch = version_label.split('.').length === 3
+        let version: SemVer
+        try {
+          // Normalize "4.5" -> "4.5.0" so SemVer can parse it
+          version = new SemVer(has_patch ? version_label : `${version_label}.0`)
+        } catch {
+          continue
         }
+
+        // Skip paramsets targeting a newer firmware than the one currently installed
+        if (version.compare(current) > 0) {
+          continue
+        }
+
+        candidates.push({
+          name, paramset, version, version_label, has_patch,
+        })
       }
-      return {
-        ...fw_params,
+
+      // Specificity: 2 = exact patch match, 1 = same major.minor, 0 = older
+      function specificity(c: typeof candidates[number]): number {
+        if (c.has_patch && c.version.compare(current) === 0) return 2
+        if (c.version.major === current.major && c.version.minor === current.minor) return 1
+        return 0
       }
+      const best = candidates.reduce((max, c) => Math.max(max, specificity(c)), 0)
+
+      const result: FilteredParamSet[] = candidates.map((c) => ({
+        name: c.name,
+        paramset: c.paramset,
+        version: c.version,
+        version_label: c.version_label,
+        outdated: best === 0 || specificity(c) < best,
+      }))
+
+      // Current paramsets first, then outdated ones sorted newest-first
+      result.sort((a, b) => {
+        if (a.outdated !== b.outdated) {
+          return a.outdated ? 1 : -1
+        }
+        return b.version.compare(a.version)
+      })
+
+      return result
     },
     warningMessage(): string {
       return 'You will lose ALL your parameters, vehicle setup, and calibrations. Are you sure you want to reset?'