fix(output): small fixup for output and docs

[bdeitte]

As I went through the latest (so many good things in here), I noticed two things:

1. The tests are faster with the parallelization, but there's a new prefix ([gw1] and similar) that doesn't make the output as clean
2. I never documented the "--filter" flag

[statik]

LGTM

[Copilot]

Pull request overview

This PR improves the usability of VIP’s pytest output under xdist parallel runs and updates the Getting Started docs to include the --filter/-f flag.

Changes:

• Suppress pytest-xdist’s verbose [gw<N>] worker prefix in terminal output by stripping the report.node attribute early in reporting.
• Add documentation examples for vip verify --filter/-f and adjust the “extra pytest flags” example accordingly.
• Add a selftest asserting the [gw<N>] prefix does not appear in verbose output under xdist.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File	Description
website/src/pages/getting-started.astro	Documents --filter/-f usage and updates example commands.
src/vip/plugin.py	Adds tryfirst hook ordering and strips report.node to suppress xdist worker prefixes.
selftests/test_plugin.py	Adds regression test ensuring [gw<N>] prefixes are not present in stdout under xdist + -v.

Comments suppressed due to low confidence (1)

src/vip/plugin.py:702

• report.node is stripped before the xdist worker guard (hasattr(_active_config, "workerinput")). This means workers would also have the attribute deleted if it ever exists there, which contradicts the docstring’s “skip processing on workers” intent and could interfere with other plugins that might set report.node on workers. Consider moving the report.node stripping below the worker guard and restricting it to the controller path (and optionally wrapping the deletion in a small try/except AttributeError for robustness).

    # Strip the xdist worker attribute so pytest's built-in TerminalReporter
    # does not prefix each verbose line with ``[gw<N>]``.  Runs before the
    # terminal reporter thanks to ``tryfirst=True``.
    if hasattr(report, "node"):
        del report.node

    if _active_config is None:
        return

    # On xdist workers, skip all processing — the controller handles it.
    # _active_config is a pytest.Config; xdist sets workerinput on worker configs.
    if hasattr(_active_config, "workerinput"):
        return

---

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

[github-actions]

PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-04-22 18:38 UTC