- Add codecov.yaml

- Mock out XSD retrieval from omg.org to prevent DOSing the schema server when testing
- Add caching mechanism for XSD retrievals during validation
- Clean up CLI handling of common error

Author: medley56

.devcontainer/Dockerfile

@@ -92,4 +92,4 @@ USER $USERNAME
 
 # Copy CLAUDE.md and settings to Claude config directory
 COPY CLAUDE.md /home/$USERNAME/.claude/
-COPY claude-settings.json /home/$USERNAME/.claude/.claude.json
+COPY claude-settings.json /home/$USERNAME/.claude/settings.json

.devcontainer/claude-settings.json

@@ -1,4 +1,5 @@
 {
+  "model": "sonnet",
   "autoUpdates": true,
   "hasCompletedOnboarding": false,
   "shiftEnterKeyBindingInstalled": true,

.devcontainer/devcontainer.json

@@ -27,13 +27,15 @@
         "github.copilot",
         "github.copilot-chat",
         "anthropic.claude-code",
-        "GitHub.vscode-pull-request-github"
+        "GitHub.vscode-pull-request-github",
+        "github.vscode-github-actions"
       ],
       "settings": {
         "dev.containers.copyGitConfig": true,
         "python.terminal.activateEnvironment": false,
         "python-envs.terminal.autoActivationType": "off",
         "python.defaultInterpreterPath": "python",
+        "python.REPL.enableREPLSmartSend": false,
         "python.testing.pytestEnabled": true,
         "python.testing.unittestEnabled": false,
         "python.testing.pytestArgs": [

.gitignore

@@ -60,6 +60,9 @@ Thumbs.db
 .idea
 .project
 .run
+.vscode
+core
+# Sometimes devcontainers create a core dump file when restarting
 
 # Virtual environment #
 #######################

codecov.yml

@@ -0,0 +1,24 @@
+# Codecov configuration for space_packet_parser
+# https://docs.codecov.com/docs/codecovyml-reference
+
+coverage:
+  status:
+    project:
+      default:
+        target: 95%  # Target coverage percentage
+        threshold: 5%  # Allow 5% drop without failing
+        if_no_uploads: error  # Fail if no coverage data
+        only_pulls: false  # Check coverage on all commits
+    patch:
+      default:
+        target: 85%  # Lower threshold for new code
+        threshold: 5%
+        only_pulls: true  # Only check patch coverage on PRs
+
+  ignore:
+    - "tests/**/*"  # Don't include test files in coverage
+    - "scripts/**/*"  # Don't include utility scripts
+    - "examples/**/*"  # Don't include examples
+    - "docs/**/*"  # Don't include documentation
+    - "**/__init__.py"  # Ignore init files
+    - "**/conftest.py"  # Ignore pytest configuration

docs/source/users.md

@@ -378,7 +378,7 @@ This equation can be implemented in XTCE by referencing the packet length field
 
 ## XTCE Document Validation
 
-Space Packet Parser provides comprehensive validation capabilities for XTCE documents to help ensure they are correct and will work properly for parsing packets. The validation system operates in two modes:
+Space Packet Parser provides comprehensive validation capabilities for XTCE documents to help ensure they are correct and will work properly for parsing packets. The validation system operates in three modes: "schema", "structure", and a default mode of "all" (both schema and structure validation).
 
 - **Schema Validation**: Validates the XML document against the in-document referenced XTCE XSD schema
 - **Structural Validation**: Validates XTCE-specific structure and reference integrity
@@ -394,7 +394,13 @@ e.g.
                                       https://www.omg.org/spec/XTCE/20180204/SpaceSystem.xsd">
 ```
 
-### Basic Validation Usage
+### CLI Validation
+
+```shell
+spp --log-level=DEBUG validate my_xtce.xml --local-schema my_xsd.xml --level all
+```
+
+### Programmatic Validation
 
 ```python
 from space_packet_parser import validate_xtce

space_packet_parser/cli.py

@@ -220,13 +220,24 @@ def parse(
     help="Validation level to perform",
 )
 @click.option("--timeout", type=int, default=30, help="Timeout in seconds for schema downloads")
-def validate(file_path: Path, level: str, timeout: int) -> None:
+@click.option("--local-xsd", type=click.Path(exists=True, path_type=Path), help="Local XSD file for schema validation")
+def validate(file_path: Path, level: str, timeout: int, local_xsd: Path) -> None:
     """Validate an XTCE document."""
-    logging.debug(f"Validating XTCE file: {file_path}")
-    logging.debug(f"Validation level: {level}")
-
-    result = validate_xtce(file_path, level=level.lower(), timeout=timeout)
+    logging.info(f"Validating XTCE file: {file_path}")
+    logging.info(f"Validation level: {level}")
+    logging.debug(f"Timeout: {timeout}")
+    logging.debug(f"Local XSD: {local_xsd}")
+
+    result = validate_xtce(
+        file_path,
+        level=level.lower(),
+        timeout=timeout,
+        print_results=False,
+        raise_on_error=False,
+        local_xsd=local_xsd,
+    )
 
+    # Display results in rich format (complementing the print_results from validate_xtce)
     if result.valid:
         console.print(f"[bold green]✓ VALID[/bold green] ({result.validation_level.value} level)")
     else:
@@ -244,7 +255,9 @@ def validate(file_path: Path, level: str, timeout: int) -> None:
         console.print(f"\n[bold red]Errors ({len(result.errors)}):[/bold red]")
         for error in result.errors:
             console.print(f"  {error}")
+            if error.context:
+                console.print(f"  - Additional context: {error.context}")
 
-    # Exit with error code if validation failed
+    # Exit with error code if validation failed (unless raise_on_error is True, which already raised)
     if not result.valid:
         raise click.ClickException("Validation failed")