ci: fix nine ways the selection under-selected or mismatched

Every one of these dropped coverage silently - the worst failure mode here,
because the PR still goes green. Found by review, each reproduced first.

Selection rules:

* class_macros derived the config macro from the class DIRECTORY, so a change to
  src/class/midi/midi2_device.c selected the midi_test examples (which do not
  compile it) and never examples/device/midi2_device (the only one that enables
  CFG_TUD_MIDI2, and the only one that does). The file's own macro is unioned in
  where it differs - union, never replace: over-selecting costs a build,
  under-selecting merges a break.
* the ${FAMILY_MCUS} fallback added for espressif fired on any family whose
  _family_mcus came back empty, and _cmake_sets is if()-blind and keeps the FIRST
  definition - so mcx/frdm_mcxn947 answered MCXA15, a token six examples' skip.txt
  names, dropping 12 firmware images CMake builds. Limited now to families that
  never spell set(FAMILY_MCUS ...) at all.
* lib_examples read only an example's top-level CMakeLists.txt/Makefile;
  host/msc_file_explorer_freertos names lib/embedded-cli in src/CMakeLists.txt and
  survived by luck. The whole example tree is scanned. (SEGGER_RTT and rt-thread
  still resolve to nothing: all three references sit inside a LOGGER=rtt guard no
  CI build sets - the documented ruling, not a miss.)
* get_family_boards applied ci_skip_boards/ci_preferred_boards only under
  GITHUB_ACTIONS/CIRCLECI, so the selector answered differently on a laptop than
  on a runner; _prune_buildable forces CI semantics. Its one-board pick also
  abandoned the whole preferred list when entry one could not build the -e set,
  and asked skip_example without the build's -D tokens.
* _config_enables and lib_examples still read with the locale encoding - under
  LC_ALL=C the selector tracebacked on three tracked tusb_config.h files. The
  whole selector and its suite run clean there now.

Workflows:

* the Membrowse Upload step omitted $EX_ARGS, but --one-first now picks the board
  from the -e set, so it configured a different, empty build dir and uploaded
  --identical for a board never compiled. It takes $EX_ARGS for the BOARD; the
  target stays the aggregate, which has no DEPENDS and still records every example.
* blanking FAM_REGEX reset only build_filtered, leaving the build scoped while
  code-metrics took the UNSCOPED branch and diffed a 1-family run against the full
  averaged baseline. All three drop together now, as CircleCI's fall-open does.
* CircleCI's EX_ARGS had no character screen and is used unquoted, and its
  code-metrics job still exit 1'd on an empty metrics set - which a scoped build
  makes a legitimate outcome.
* a `ci-full` PR label now turns the scoping off for one PR. A selector bug
  under-selects silently, and without a label the only ways back to a full matrix
  are accidental.

Performance, since the selector gates every other job: family.cmake texts are read
once rather than per changed directory (a 6,000-file dep bump re-read 84 files
99,892 times) and _scrape_mcu is cached: 2.2s -> 0.29s there, 0.8s -> 0.33s on a
class diff.

Tests: a drift guard for hw/bsp families absent from ci_set_matrix.family_list
(they select zero legs now, where they used to ride the full matrix); the rule-4
port test asserted a SUBSET, which set() satisfies, so it could not fail on the
empty selection it exists to catch; the GITHUB_ENV guard test counted a SUM of two
guards. Drops metrics.py's --only-examples, which nothing called, and applies the
TOTAL scrub to the by-example branch that skipped it.

1 files changed, 24 insertions, 5 deletions

diff --git a/tools/build.py b/tools/build.py
index e7ca1c839..eeefca22d 100755
--- a/tools/build.py
+++ b/tools/build.py
@@ -299,7 +299,8 @@ def build_boards_list(boards, build_defines, build_system, build_name, build_cfl
     return ret
 
 
-def get_family_boards(family, one_random, one_first, examples=None, build_system='cmake'):
+def get_family_boards(family, one_random, one_first, examples=None, build_system='cmake',
+                      extra_defines=(), ci=None):
     """Get list of boards for a family.
 
     Args:
@@ -314,13 +315,23 @@ def get_family_boards(family, one_random, one_first, examples=None, build_system
             which every one of those examples skips - and the leg runs to green having
             compiled nothing and uploaded no metrics.
         build_system: which skip answer to ask for; the two differ (build_utils)
+        extra_defines: this build's -D tokens, so a board whose only.txt match comes
+            from -DMAX3421_HOST=1 is not judged unbuildable here and buildable in
+            cmake_board
+        ci: force the ci_skip_boards / ci_preferred_boards lists on or off. Default
+            None reads the environment, which is right for a build but NOT for a caller
+            asking what CI would do: ci_select must answer the same on a laptop as on a
+            runner, or /pre-pr and the code-size skill report a family list CI will not
+            reproduce.
 
     Returns:
         List of board names
     """
+    if ci is None:
+        ci = bool(os.getenv('GITHUB_ACTIONS') or os.getenv('CIRCLECI'))
     skip_list = []
     preferred_list = []
-    if os.getenv('GITHUB_ACTIONS') or os.getenv('CIRCLECI'):
+    if ci:
         skip_list = ci_skip_boards.get(family, [])
         preferred_list = ci_preferred_boards.get(family, [])
 
@@ -339,9 +350,16 @@ def get_family_boards(family, one_random, one_first, examples=None, build_system
             # no filter, or nothing in the filter is buildable anywhere: keep today's
             # answer rather than inventing a different board
             return examples is None or any(
-                not build_utils.skip_example(e, board, (), build_system) for e in examples)
+                not build_utils.skip_example(e, board, extra_defines, build_system)
+                for e in examples)
 
-        if preferred_list and buildable(preferred_list[0]):
+        # the WHOLE preferred list, in order - stopping at entry one would abandon a
+        # curated list for the raw alphabetical order the moment its first board cannot
+        # build the filter, which also moves the board the metrics baseline is keyed on
+        for b in preferred_list:
+            if buildable(b):
+                return [b]
+        if preferred_list and examples is None:
             return [preferred_list[0]]
         candidates = [b for b in all_boards if buildable(b)] or all_boards
         if one_first:
@@ -434,7 +452,8 @@ def main():
     # get boards from families and append to boards list
     all_boards = list(boards)
     for f in all_families:
-        all_boards.extend(get_family_boards(f, one_random, one_first, examples, build_system))
+        all_boards.extend(get_family_boards(f, one_random, one_first, examples,
+                                            build_system, tuple(build_defines)))
 
     # build all boards
     result = build_boards_list(all_boards, build_defines, build_system, build_name, build_cflags, build_targets,