roschema 0.2.1__tar.gz → 0.3.0__tar.gz

{roschema-0.2.1 → roschema-0.3.0}/.github/workflows/ci.yml

@@ -15,7 +15,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
-      - uses: astral-sh/setup-uv@v5
+      - uses: astral-sh/setup-uv@v10.3.3
         with:
           python-version: "3.12"
           enable-cache: true
@@ -36,10 +36,10 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        python-version: ["3.10", "3.11", "3.12", "3.13"]
+        python-version: ["3.10", "3.11", "3.12", "3.13", "3.14"]
     steps:
       - uses: actions/checkout@v4
-      - uses: astral-sh/setup-uv@v5
+      - uses: astral-sh/setup-uv@v10.3.3
         with:
           python-version: ${{ matrix.python-version }}
           enable-cache: true

{roschema-0.2.1 → roschema-0.3.0}/.github/workflows/release.yml

@@ -10,7 +10,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
-      - uses: astral-sh/setup-uv@v5
+      - uses: astral-sh/setup-uv@v10.3.3
         with:
           python-version: "3.12"
           enable-cache: true

{roschema-0.2.1 → roschema-0.3.0}/CHANGELOG.md

@@ -3,6 +3,29 @@
 The format follows [Keep a Changelog](https://keepachangelog.com/), and the
 project adheres to [Semantic Versioning](https://semver.org/).
 
+## [0.3.0] - 2026-11-03
+
+### Added
+
+- Reports say how many interfaces were compared, and on which side the rest
+  exist only (JSON: `coverage`).
+- `--warnings-as-errors` for `check`, `bag-check` and `lint`, and the Action's
+  `warnings-as-errors` input.
+
+### Changed
+
+- Python 3.14 is supported and tested.
+- An empty baseline is an error (exit `2`) instead of a silent pass; pass
+  `--allow-empty` (Action: `allow-empty`) when that is expected.
+
+### Fixed
+
+- `--format github` attaches findings with no new-side location, such as a
+  removed field or a reorder, to the interface's file instead of leaving GitHub
+  to show them against the workflow file.
+- The Action uses `astral-sh/setup-uv@v10.3.3` (Node 24) with its cache off,
+  removing the Node 20 deprecation and missing `uv.lock` warnings.
+
 ## [0.2.1] - 2026-11-03
 
 ### Added

{roschema-0.2.1 → roschema-0.3.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: roschema
-Version: 0.2.1
+Version: 0.3.0
 Summary: Breaking-change detection, linting and formatting for ROS 2 interface definitions
 Project-URL: Homepage, https://github.com/justagist/roschema
 Project-URL: Documentation, https://github.com/justagist/roschema#readme
@@ -17,6 +17,7 @@ Classifier: Programming Language :: Python :: 3.10
 Classifier: Programming Language :: Python :: 3.11
 Classifier: Programming Language :: Python :: 3.12
 Classifier: Programming Language :: Python :: 3.13
+Classifier: Programming Language :: Python :: 3.14
 Classifier: Topic :: Software Development :: Quality Assurance
 Classifier: Typing :: Typed
 Requires-Python: >=3.10
@@ -153,7 +154,8 @@ roschema ls
 
 ```
 roschema check [TARGET] --against BASELINE [--level LEVEL] [--format text|json|github]
-               [--deep | --no-deep] [--exit-zero] [--config PATH] [--exclude GLOB]
+               [--deep | --no-deep] [--exit-zero] [--warnings-as-errors] [--allow-empty]
+               [--config PATH] [--exclude GLOB]
 ```
 
 `TARGET` selects what to check; omit it to check every interface package below
@@ -176,6 +178,13 @@ MCAP bag (`.mcap`) or a lockfile (`roschema.lock`); the type is detected
 automatically. Initialised git submodules are compared at the commit the
 baseline records for them.
 
+Every report says how much was compared, for example `compared 12 interface(s)
+in 3 package(s); 1 only in the target`. A baseline with no interfaces at all is
+an error (exit `2`): every current interface would look newly added, so the
+check would pass without comparing anything. That usually means the wrong
+baseline or target; on the pull request that adds the first interfaces, pass
+`--allow-empty`.
+
 Limitations:
 
 - When a git ref and a directory share a name, the git ref wins; prefix the
@@ -199,7 +208,7 @@ followed.
 
 ```
 roschema lint [TARGET] [--format text|json|github] [--rules RULE=SEV] [--exit-zero]
-              [--config PATH] [--exclude GLOB]
+              [--warnings-as-errors] [--config PATH] [--exclude GLOB]
 roschema fmt  [TARGET] [--check] [--exclude GLOB]
 ```
 
@@ -238,7 +247,8 @@ same. Comparison ignores it.
 
 ```
 roschema bag-check BAG.mcap [TARGETS...] [--level LEVEL] [--config PATH]
-                   [--format text|json|github] [--exit-zero] [--exclude GLOB]
+                   [--format text|json|github] [--exit-zero] [--warnings-as-errors]
+                   [--exclude GLOB]
 ```
 
 Needs the `bag` extra (`uv tool install 'roschema[bag]'`).
@@ -327,8 +337,8 @@ allow every rule for that target.
 
 ## Output
 
-**Text** (the default) groups findings by package and ends with a summary and a
-semver advice per package:
+**Text** (the default) starts with what was compared, groups findings by
+package and ends with a summary and a semver advice per package:
 
 - **MAJOR** when the package has a wire- or source-level error;
 - **MINOR** when its only notable change is an added field (RS002) whose
@@ -338,8 +348,8 @@ semver advice per package:
 The advice only covers reported findings. Additions such as a new interface or
 constant are not findings, so they never raise the advice to MINOR on their own.
 
-**`--format github`** emits `::error`/`::warning` workflow commands, which GitHub
-shows as annotations on the PR diff.
+**`--format github`** prints what was compared, then emits `::error`/`::warning`
+workflow commands, which GitHub shows as annotations on the PR diff.
 
 **`--format json`** emits a stable document (`version: 1`):
 
@@ -367,12 +377,19 @@ shows as annotations on the PR diff.
     "warnings": 0,
     "suppressed": 0,
     "advisories": { "my_msgs": "MAJOR" }
+  },
+  "coverage": {
+    "packages": 1,
+    "compared": 1,
+    "target_only": 0,
+    "baseline_only": 0
   }
 }
 ```
 
 `old`/`new` are `null` when there is no corresponding side (a removed interface
-has no `new`). `lint` output has no `advisories`.
+has no `new`). `coverage` counts interfaces present on both sides (`compared`)
+and on one side only. `lint` output has no `advisories` or `coverage`.
 
 ## Exit codes
 
@@ -380,10 +397,11 @@ has no `new`). `lint` output has no `advisories`.
 |---|---|---|---|
 | `0` | no unsuppressed errors (warnings allowed) | every file formatted | success |
 | `1` | at least one unsuppressed error | a file would change | n/a |
-| `2` | tool or usage error (bad target, baseline, configuration or file) | tool or usage error | tool or usage error |
+| `2` | tool or usage error (bad target, baseline, configuration or file, or an empty baseline) | tool or usage error | tool or usage error |
 
-`--exit-zero` turns `1` into `0` for `check`, `bag-check` and `lint`; tool
-errors still exit `2`.
+`--warnings-as-errors` makes unsuppressed warnings exit `1` too. `--exit-zero`
+turns `1` into `0` for `check`, `bag-check` and `lint`; tool errors still exit
+`2`.
 
 ## GitHub Action
 
@@ -403,9 +421,10 @@ jobs:
       - uses: justagist/roschema/action@main
         with:
           against: origin/${{ github.base_ref }}
-          # version: "roschema==0.2.1"   # pin a release for reproducible results
+          # version: "roschema==0.3.0"   # pin a release for reproducible results
           # level: source
           # deep: "true"
+          # warnings-as-errors: "true"
           # exclude: "vendor_msgs* legacy_msgs/msg/Old"
 ```
 
@@ -416,6 +435,8 @@ jobs:
 | `level` | (empty) | reporting level; empty uses the configured level, else `source` |
 | `format` | `github` | `text`, `json` or `github` |
 | `deep` | `"false"` | `"true"` adds `--deep` |
+| `allow-empty` | `"false"` | `"true"` adds `--allow-empty` |
+| `warnings-as-errors` | `"false"` | `"true"` adds `--warnings-as-errors` |
 | `exclude` | (empty) | whitespace-separated globs passed as `--exclude` |
 | `version` | `roschema` | the roschema requirement installed with `uvx` |
 
@@ -426,9 +447,9 @@ because it is part of the checkout.
 
 ## Troubleshooting
 
-**"no breaking changes found", but you expected some.** Run `roschema ls` with
-the same target to see which packages and interfaces are discovered, then check
-that:
+**"no breaking changes found", but you expected some.** The line under the
+header says how many interfaces were compared. Run `roschema ls` with the same
+target to see which packages and interfaces are discovered, then check that:
 
 - the packages are not below a `COLCON_IGNORE` or `AMENT_IGNORE` marker;
 - a package-name target did not fall back to the installed copy (the warning

{roschema-0.2.1 → roschema-0.3.0}/README.md

@@ -119,7 +119,8 @@ roschema ls
 
 ```
 roschema check [TARGET] --against BASELINE [--level LEVEL] [--format text|json|github]
-               [--deep | --no-deep] [--exit-zero] [--config PATH] [--exclude GLOB]
+               [--deep | --no-deep] [--exit-zero] [--warnings-as-errors] [--allow-empty]
+               [--config PATH] [--exclude GLOB]
 ```
 
 `TARGET` selects what to check; omit it to check every interface package below
@@ -142,6 +143,13 @@ MCAP bag (`.mcap`) or a lockfile (`roschema.lock`); the type is detected
 automatically. Initialised git submodules are compared at the commit the
 baseline records for them.
 
+Every report says how much was compared, for example `compared 12 interface(s)
+in 3 package(s); 1 only in the target`. A baseline with no interfaces at all is
+an error (exit `2`): every current interface would look newly added, so the
+check would pass without comparing anything. That usually means the wrong
+baseline or target; on the pull request that adds the first interfaces, pass
+`--allow-empty`.
+
 Limitations:
 
 - When a git ref and a directory share a name, the git ref wins; prefix the
@@ -165,7 +173,7 @@ followed.
 
 ```
 roschema lint [TARGET] [--format text|json|github] [--rules RULE=SEV] [--exit-zero]
-              [--config PATH] [--exclude GLOB]
+              [--warnings-as-errors] [--config PATH] [--exclude GLOB]
 roschema fmt  [TARGET] [--check] [--exclude GLOB]
 ```
 
@@ -204,7 +212,8 @@ same. Comparison ignores it.
 
 ```
 roschema bag-check BAG.mcap [TARGETS...] [--level LEVEL] [--config PATH]
-                   [--format text|json|github] [--exit-zero] [--exclude GLOB]
+                   [--format text|json|github] [--exit-zero] [--warnings-as-errors]
+                   [--exclude GLOB]
 ```
 
 Needs the `bag` extra (`uv tool install 'roschema[bag]'`).
@@ -293,8 +302,8 @@ allow every rule for that target.
 
 ## Output
 
-**Text** (the default) groups findings by package and ends with a summary and a
-semver advice per package:
+**Text** (the default) starts with what was compared, groups findings by
+package and ends with a summary and a semver advice per package:
 
 - **MAJOR** when the package has a wire- or source-level error;
 - **MINOR** when its only notable change is an added field (RS002) whose
@@ -304,8 +313,8 @@ semver advice per package:
 The advice only covers reported findings. Additions such as a new interface or
 constant are not findings, so they never raise the advice to MINOR on their own.
 
-**`--format github`** emits `::error`/`::warning` workflow commands, which GitHub
-shows as annotations on the PR diff.
+**`--format github`** prints what was compared, then emits `::error`/`::warning`
+workflow commands, which GitHub shows as annotations on the PR diff.
 
 **`--format json`** emits a stable document (`version: 1`):
 
@@ -333,12 +342,19 @@ shows as annotations on the PR diff.
     "warnings": 0,
     "suppressed": 0,
     "advisories": { "my_msgs": "MAJOR" }
+  },
+  "coverage": {
+    "packages": 1,
+    "compared": 1,
+    "target_only": 0,
+    "baseline_only": 0
   }
 }
 ```
 
 `old`/`new` are `null` when there is no corresponding side (a removed interface
-has no `new`). `lint` output has no `advisories`.
+has no `new`). `coverage` counts interfaces present on both sides (`compared`)
+and on one side only. `lint` output has no `advisories` or `coverage`.
 
 ## Exit codes
 
@@ -346,10 +362,11 @@ has no `new`). `lint` output has no `advisories`.
 |---|---|---|---|
 | `0` | no unsuppressed errors (warnings allowed) | every file formatted | success |
 | `1` | at least one unsuppressed error | a file would change | n/a |
-| `2` | tool or usage error (bad target, baseline, configuration or file) | tool or usage error | tool or usage error |
+| `2` | tool or usage error (bad target, baseline, configuration or file, or an empty baseline) | tool or usage error | tool or usage error |
 
-`--exit-zero` turns `1` into `0` for `check`, `bag-check` and `lint`; tool
-errors still exit `2`.
+`--warnings-as-errors` makes unsuppressed warnings exit `1` too. `--exit-zero`
+turns `1` into `0` for `check`, `bag-check` and `lint`; tool errors still exit
+`2`.
 
 ## GitHub Action
 
@@ -369,9 +386,10 @@ jobs:
       - uses: justagist/roschema/action@main
         with:
           against: origin/${{ github.base_ref }}
-          # version: "roschema==0.2.1"   # pin a release for reproducible results
+          # version: "roschema==0.3.0"   # pin a release for reproducible results
           # level: source
           # deep: "true"
+          # warnings-as-errors: "true"
           # exclude: "vendor_msgs* legacy_msgs/msg/Old"
 ```
 
@@ -382,6 +400,8 @@ jobs:
 | `level` | (empty) | reporting level; empty uses the configured level, else `source` |
 | `format` | `github` | `text`, `json` or `github` |
 | `deep` | `"false"` | `"true"` adds `--deep` |
+| `allow-empty` | `"false"` | `"true"` adds `--allow-empty` |
+| `warnings-as-errors` | `"false"` | `"true"` adds `--warnings-as-errors` |
 | `exclude` | (empty) | whitespace-separated globs passed as `--exclude` |
 | `version` | `roschema` | the roschema requirement installed with `uvx` |
 
@@ -392,9 +412,9 @@ because it is part of the checkout.
 
 ## Troubleshooting
 
-**"no breaking changes found", but you expected some.** Run `roschema ls` with
-the same target to see which packages and interfaces are discovered, then check
-that:
+**"no breaking changes found", but you expected some.** The line under the
+header says how many interfaces were compared. Run `roschema ls` with the same
+target to see which packages and interfaces are discovered, then check that:
 
 - the packages are not below a `COLCON_IGNORE` or `AMENT_IGNORE` marker;
 - a package-name target did not fall back to the installed copy (the warning

{roschema-0.2.1 → roschema-0.3.0}/action/action.yml

@@ -22,6 +22,14 @@ inputs:
   deep:
     description: Resolve nested types and report transitive breakage (RS110).
     default: "false"
+  allow-empty:
+    description: >-
+      Pass when the baseline has no interfaces, e.g. on the pull request that
+      adds the first ones. Otherwise an empty baseline fails the step.
+    default: "false"
+  warnings-as-errors:
+    description: Fail the step on warnings as well as errors.
+    default: "false"
   exclude:
     description: >-
       Whitespace-separated globs of packages/interfaces to skip. Split on
@@ -30,14 +38,18 @@ inputs:
   version:
     description: >-
       roschema requirement for uvx. Unpinned runs the latest release (refreshed
-      on every run, even with a uv cache); pin it (e.g. "roschema==0.2.1") for
+      on every run, even with a uv cache); pin it (e.g. "roschema==0.3.0") for
       reproducible results.
     default: roschema
 
 runs:
   using: composite
   steps:
-    - uses: astral-sh/setup-uv@v5
+    - uses: astral-sh/setup-uv@v10.3.3
+      with:
+        # Nothing to key a cache on in the caller's repo, and uvx refreshes
+        # roschema on every run anyway.
+        enable-cache: false
     - shell: bash
       env:
         ROSCHEMA_TARGET: ${{ inputs.target }}
@@ -45,6 +57,8 @@ runs:
         ROSCHEMA_LEVEL: ${{ inputs.level }}
         ROSCHEMA_FORMAT: ${{ inputs.format }}
         ROSCHEMA_DEEP: ${{ inputs.deep }}
+        ROSCHEMA_ALLOW_EMPTY: ${{ inputs.allow-empty }}
+        ROSCHEMA_WARNINGS_AS_ERRORS: ${{ inputs.warnings-as-errors }}
         ROSCHEMA_EXCLUDE: ${{ inputs.exclude }}
         ROSCHEMA_VERSION: ${{ inputs.version }}
       run: |
@@ -58,6 +72,12 @@ runs:
         if [ "$ROSCHEMA_DEEP" = "true" ]; then
           args+=(--deep)
         fi
+        if [ "$ROSCHEMA_ALLOW_EMPTY" = "true" ]; then
+          args+=(--allow-empty)
+        fi
+        if [ "$ROSCHEMA_WARNINGS_AS_ERRORS" = "true" ]; then
+          args+=(--warnings-as-errors)
+        fi
         read -r -a excludes <<< "${ROSCHEMA_EXCLUDE//$'\n'/ }"
         for glob in "${excludes[@]}"; do
           args+=(--exclude "$glob")

{roschema-0.2.1 → roschema-0.3.0}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "roschema"
-version = "0.2.1"
+version = "0.3.0"
 description = "Breaking-change detection, linting and formatting for ROS 2 interface definitions"
 readme = "README.md"
 license = "Apache-2.0"
@@ -19,6 +19,7 @@ classifiers = [
     "Programming Language :: Python :: 3.11",
     "Programming Language :: Python :: 3.12",
     "Programming Language :: Python :: 3.13",
+    "Programming Language :: Python :: 3.14",
     "Topic :: Software Development :: Quality Assurance",
     "Typing :: Typed",
 ]

{roschema-0.2.1 → roschema-0.3.0}/src/roschema/cli.py

@@ -38,7 +38,7 @@ from .ir import InterfaceFile
 from .loader import load_interfaces
 from .mcap_fs import McapTypeFS
 from .parser import ParseError
-from .report import build_github, build_json, render_lint, render_text
+from .report import Coverage, build_github, build_json, coverage, render_lint, render_text
 from .rules import Finding, all_lint_rules, evaluate, lint_interfaces
 from .vfs import GitTreeFS, RealFS, VfsError, git_toplevel
 
@@ -113,6 +113,17 @@ def check(
     exit_zero: Annotated[
         bool, typer.Option("--exit-zero", help="Exit 0 even when findings exist.")
     ] = False,
+    warnings_as_errors: Annotated[
+        bool,
+        typer.Option("--warnings-as-errors", help="Exit 1 on warnings as well as errors."),
+    ] = False,
+    allow_empty: Annotated[
+        bool,
+        typer.Option(
+            "--allow-empty",
+            help="Pass when the baseline has no interfaces (e.g. before any exist).",
+        ),
+    ] = False,
     exclude: Annotated[
         list[str] | None,
         typer.Option("--exclude", help="Glob of packages/interfaces to skip."),
@@ -127,7 +138,7 @@ def check(
         raise typer.Exit(code=2)
 
     try:
-        findings, new_source, old_source, prefix = _compute(
+        findings, new_source, old_source, prefix, files, cov = _compute(
             target, lambda root: resolve_baseline(against, root), level, config_path, exclude, deep
         )
     except (VfsError, ParseError, ConfigError) as exc:
@@ -139,8 +150,12 @@ def check(
             f"[dim]note: {against!r} matched a git ref; "
             f"use './{against}' to compare against the directory instead.[/dim]"
         )
-    _emit(findings, fmt, new_source, old_source, prefix)
-    _exit_for(findings, exit_zero)
+    if not allow_empty:
+        _require_baseline(
+            cov, f"the baseline ({old_source})", "pass --allow-empty if that is expected"
+        )
+    _emit(findings, fmt, new_source, old_source, cov, prefix, files)
+    _exit_for(findings, exit_zero, warnings_as_errors)
 
 
 @app.command(name="bag-check")
@@ -170,6 +185,10 @@ def bag_check(
     exit_zero: Annotated[
         bool, typer.Option("--exit-zero", help="Exit 0 even when findings exist.")
     ] = False,
+    warnings_as_errors: Annotated[
+        bool,
+        typer.Option("--warnings-as-errors", help="Exit 1 on warnings as well as errors."),
+    ] = False,
     exclude: Annotated[
         list[str] | None,
         typer.Option("--exclude", help="Glob of packages/interfaces to skip."),
@@ -208,8 +227,11 @@ def bag_check(
     findings = finalize_findings(
         evaluate(diff_interfaces(old_ifaces, new_ifaces)), config, level, BAG_DEFAULT_LEVEL
     )
-    _emit(findings, fmt, new_source, old_res.source)
-    _exit_for(findings, exit_zero)
+    cov = coverage(old_ifaces, new_ifaces)
+    _require_baseline(cov, "the bag", "check the --exclude patterns")
+    files = {key: iface.source_file for key, iface in new_ifaces.items()}
+    _emit(findings, fmt, new_source, old_res.source, cov, interface_files=files)
+    _exit_for(findings, exit_zero, warnings_as_errors)
 
 
 def _resolve_current(
@@ -263,8 +285,12 @@ def _compute(
     config_path: Path | None,
     exclude: list[str] | None,
     deep: bool | None = None,
-) -> tuple[list[Finding], str, str, str]:
-    """Resolve both sides, diff and finalize; the shared check pipeline."""
+) -> tuple[list[Finding], str, str, str, dict[str, str], Coverage]:
+    """Resolve both sides, diff and finalize; the shared check pipeline.
+
+    Also returns each current interface's source file, keyed by interface, and
+    how many interfaces were compared.
+    """
     new_res = _resolve_target(target, allow_empty_glob=True)
     if not isinstance(new_res.vfs, RealFS):
         raise VfsError("target must be a directory or workspace")
@@ -305,7 +331,9 @@ def _compute(
         findings += finalize_findings(
             [f for f in transitive if f.package in deep_packages], config, level
         )
-    return findings, new_res.source, old_res.source, prefix
+    files = {key: iface.source_file for key, iface in new_ifaces.items()}
+    cov = coverage(old_ifaces, new_ifaces)
+    return findings, new_res.source, old_res.source, prefix, files, cov
 
 
 def _note_skipped_schemas(res: Resolution) -> None:
@@ -326,11 +354,26 @@ def _display_prefix(target_dir: Path) -> str:
     return "" if rel == "." else rel
 
 
-def _exit_for(findings: list[Finding], exit_zero: bool) -> None:
+def _require_baseline(cov: Coverage, baseline: str, hint: str) -> None:
+    """Exit 2 when the baseline is empty, naming ``baseline`` and how to resolve it.
+
+    With no baseline interfaces every current one looks added, which is never a
+    finding, so the check would pass without comparing anything.
+    """
+    if cov.baseline_empty:
+        err_console.print(
+            f"[red]error:[/red] {baseline} has no interfaces, so nothing was compared; {hint}"
+        )
+        raise typer.Exit(code=2)
+
+
+def _exit_for(findings: list[Finding], exit_zero: bool, warnings_as_errors: bool) -> None:
+    """Exit 1 if an unsuppressed finding fails the check, else 0."""
     if exit_zero:
         raise typer.Exit(code=0)
-    has_error = any(f.severity == "error" and not f.suppressed for f in findings)
-    raise typer.Exit(code=1 if has_error else 0)
+    failing = {"error", "warn"} if warnings_as_errors else {"error"}
+    has_failure = any(f.severity in failing and not f.suppressed for f in findings)
+    raise typer.Exit(code=1 if has_failure else 0)
 
 
 def _emit(
@@ -338,15 +381,18 @@ def _emit(
     fmt: str,
     new_source: str,
     old_source: str,
+    cov: Coverage,
     display_prefix: str = "",
+    interface_files: dict[str, str] | None = None,
 ) -> None:
     if fmt == "json":
-        typer.echo(json.dumps(build_json(findings), indent=2))
+        typer.echo(json.dumps(build_json(findings, coverage=cov), indent=2))
     elif fmt == "github":
-        for line in build_github(findings, display_prefix):
+        typer.echo(cov.describe())
+        for line in build_github(findings, display_prefix, interface_files):
             typer.echo(line)
     else:
-        render_text(findings, console, new_source, old_source)
+        render_text(findings, console, new_source, old_source, cov)
 
 
 @app.command()
@@ -372,6 +418,10 @@ def lint(
     exit_zero: Annotated[
         bool, typer.Option("--exit-zero", help="Exit 0 even when findings exist.")
     ] = False,
+    warnings_as_errors: Annotated[
+        bool,
+        typer.Option("--warnings-as-errors", help="Exit 1 on warnings as well as errors."),
+    ] = False,
     exclude: Annotated[
         list[str] | None,
         typer.Option("--exclude", help="Glob of packages/interfaces to skip."),
@@ -402,10 +452,7 @@ def lint(
     else:
         render_lint(findings, console, res.source)
 
-    if exit_zero:
-        raise typer.Exit(code=0)
-    has_error = any(f.severity == "error" and not f.suppressed for f in findings)
-    raise typer.Exit(code=1 if has_error else 0)
+    _exit_for(findings, exit_zero, warnings_as_errors)
 
 
 def _parse_rule_overrides(items: list[str]) -> dict[str, str]:

{roschema-0.2.1 → roschema-0.3.0}/src/roschema/report/__init__.py

@@ -4,13 +4,15 @@ from __future__ import annotations
 
 from .github import build_github
 from .json_report import build_json
-from .summary import Summary, summarize
+from .summary import Coverage, Summary, coverage, summarize
 from .text import render_lint, render_text
 
 __all__ = [
+    "Coverage",
     "Summary",
     "build_github",
     "build_json",
+    "coverage",
     "render_lint",
     "render_text",
     "summarize",

{roschema-0.2.1 → roschema-0.3.0}/src/roschema/report/github.py

@@ -2,26 +2,37 @@
 
 from __future__ import annotations
 
+from collections.abc import Mapping
+
 from ..rules import Finding
 
 _LEVEL = {"error": "error", "warn": "warning"}
 
 
-def build_github(findings: list[Finding], display_prefix: str = "") -> list[str]:
+def build_github(
+    findings: list[Finding],
+    display_prefix: str = "",
+    interface_files: Mapping[str, str] | None = None,
+) -> list[str]:
     """Build ``::error``/``::warning`` annotation lines for non-suppressed findings.
 
     Uses the new-side file/line so annotations land on the changed code.
     ``display_prefix`` is prepended to file paths so they are repo-root-relative
     (GitHub resolves annotation paths from the repo root, not the check target).
+    A finding with no new-side location (e.g. a removed field) is attached to its
+    interface's file from ``interface_files``; GitHub otherwise shows it against
+    the workflow file.
     """
+    files = interface_files or {}
     lines: list[str] = []
     for finding in findings:
         if finding.suppressed:
             continue
         command = _LEVEL.get(finding.severity, "notice")
         props = [f"title={_escape_property(finding.rule)}"]
-        if finding.new_file:
-            path = f"{display_prefix}/{finding.new_file}" if display_prefix else finding.new_file
+        file = finding.new_file or files.get(finding.interface)
+        if file:
+            path = f"{display_prefix}/{file}" if display_prefix else file
             props.insert(0, f"file={_escape_property(path)}")
             if finding.new_line is not None:
                 props.insert(1, f"line={finding.new_line}")