Fix _add_excel_table: gate warnings behind verbose, ensure workbook-wide unique table names

[Copilot]

_add_excel_table was unconditionally printing on exception (ignoring verbose) and table names were generated per-sheet without collision detection, risking duplicate names after sanitization.

Changes

• Structured warning collection: Added warnings: Optional[List[str]] parameter to _add_excel_table. Exceptions now append to the caller-supplied list instead of printing directly.

• Verbose-gated output: _apply_formatting collects table warnings in a local list and only emits them when verbose=True.

• Workbook-wide name uniqueness: _apply_formatting tracks assigned names in a Set[str] and appends _1, _2, … on collision, truncating the base name to stay within Excel's 255-character limit.

# Before – always printed, no deduplication
except Exception as e:
    print(f"    Warning: Could not create table {table_name}: {e}")

# After – collected, output gated on verbose, names guaranteed unique
except Exception as e:
    if warnings is not None:
        warnings.append(f"Could not create table {table_name}: {e}")

# In _apply_formatting
used_table_names: Set[str] = set()
...
while table_name in used_table_names:
    suffix = f"_{counter}"
    table_name = base_name[: 255 - len(suffix)] + suffix
    counter += 1
used_table_names.add(table_name)

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[Copilot]

Pull request overview

This PR improves the HW Queue Excel report generator’s table creation behavior by collecting table-creation failures as warnings (instead of printing unconditionally) and ensuring Excel table names are unique across the entire workbook after sanitization.

Changes:

• Add an optional warnings collector parameter to _add_excel_table and stop unconditional printing on exceptions.
• In _apply_formatting, collect table warnings and only print them when verbose=True.
• Ensure workbook-wide unique table names by tracking used names and suffixing on collisions while respecting Excel’s 255-character limit.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.