Enable linting, fix copyright header validation, auto-fix linting errors.

Author: Johan Broberg

.github/workflows/ci.yml

@@ -109,7 +109,6 @@ jobs:
     - name: Check linting
       run: |
         uv run --frozen ruff check .
-      continue-on-error: true
 
     - name: Check formatting
       run: |

generate_dependency_diagram.py

@@ -9,7 +9,6 @@
 import re
 import tomllib
 from pathlib import Path
-from typing import Dict, List, Set
 
 
 class PackageInfo:
@@ -19,16 +18,16 @@ def __init__(self, name: str, package_type: str, path: Path):
         self.name = name
         self.package_type = package_type
         self.path = path
-        self.dependencies: Set[str] = set()
+        self.dependencies: set[str] = set()
 
 
-def read_pyproject_toml(path: Path) -> Dict:
+def read_pyproject_toml(path: Path) -> dict:
     """Read and parse a pyproject.toml file."""
     with open(path, "rb") as f:
         return tomllib.load(f)
 
 
-def extract_dependencies(pyproject_data: Dict, package_names: Set[str]) -> Set[str]:
+def extract_dependencies(pyproject_data: dict, package_names: set[str]) -> set[str]:
     """Extract internal package dependencies from pyproject.toml data."""
     dependencies = set()
 
@@ -45,7 +44,7 @@ def extract_dependencies(pyproject_data: Dict, package_names: Set[str]) -> Set[s
     return dependencies
 
 
-def generate_mermaid_diagram(packages: List[PackageInfo]) -> str:
+def generate_mermaid_diagram(packages: list[PackageInfo]) -> str:
     """Generate a Mermaid diagram from package information."""
 
     # Color scheme based on package types
@@ -96,7 +95,7 @@ def generate_mermaid_diagram(packages: List[PackageInfo]) -> str:
     lines.append("    %% Styling")
 
     # Group packages by type for styling
-    packages_by_type: Dict[str, List[str]] = {}
+    packages_by_type: dict[str, list[str]] = {}
     for pkg in packages:
         if pkg.package_type not in packages_by_type:
             packages_by_type[pkg.package_type] = []
@@ -170,8 +169,8 @@ def main():
 
     # Collect all package names first and cache pyproject data
     all_package_names = set()
-    packages: List[PackageInfo] = []
-    pyproject_data_cache: Dict[str, Dict] = {}
+    packages: list[PackageInfo] = []
+    pyproject_data_cache: dict[str, dict] = {}
 
     for path_str, pkg_type in package_configs:
         pyproject_path = repo_root / path_str / "pyproject.toml"

libraries/microsoft-agents-a365-observability-hosting/microsoft_agents_a365/observability/hosting/scope_helpers/populate_baggage.py

@@ -1,3 +1,6 @@
+# Copyright (c) Microsoft Corporation.
+# Licensed under the MIT License.
+
 from __future__ import annotations
 
 from collections.abc import Iterator

libraries/microsoft-agents-a365-observability-hosting/microsoft_agents_a365/observability/hosting/scope_helpers/populate_invoke_agent_scope.py

@@ -1,3 +1,6 @@
+# Copyright (c) Microsoft Corporation.
+# Licensed under the MIT License.
+
 from __future__ import annotations
 
 from typing import TYPE_CHECKING

libraries/microsoft-agents-a365-tooling-extensions-semantickernel/microsoft_agents_a365/tooling/extensions/semantickernel/services/mcp_tool_registration_service.py

@@ -20,7 +20,7 @@
 from microsoft_agents_a365.tooling.services.mcp_tool_server_configuration_service import (
     McpToolServerConfigurationService,
 )
-from microsoft_agents_a365.tooling.models import MCPServerConfig, ToolOptions
+from microsoft_agents_a365.tooling.models import ToolOptions
 from microsoft_agents_a365.tooling.utils.constants import Constants
 from microsoft_agents_a365.tooling.utils.utility import (
     get_mcp_platform_authentication_scope,

pyproject.toml

@@ -113,7 +113,7 @@ exclude = [
 
 [tool.ruff.lint.flake8-copyright]
 # Require this exact copyright header (matches CLAUDE.md specification)
-notice-rgx = "(?s)# Copyright \\(c\\) Microsoft Corporation\\.\\n# Licensed under the MIT License\\."
+notice-rgx = "# Copyright \\(c\\) Microsoft Corporation\\.\\r?\\n# Licensed under the MIT License\\."
 # (optional) ensure all files are checked regardless of size
 min-file-size = 1
 

tests/__init__.py

@@ -1 +1,2 @@
-# Copyright (c) Microsoft. All rights reserved.
+# Copyright (c) Microsoft Corporation.
+# Licensed under the MIT License.

tests/observability/core/exporters/test_utils.py

@@ -1,4 +1,5 @@
-# Copyright (c) Microsoft. All rights reserved.
+# Copyright (c) Microsoft Corporation.
+# Licensed under the MIT License.
 
 import os
 import unittest

tests/observability/core/test_agent365_exporter.py

@@ -1,4 +1,5 @@
-# Copyright (c) Microsoft. All rights reserved.
+# Copyright (c) Microsoft Corporation.
+# Licensed under the MIT License.
 
 import json
 import os

tests/observability/core/test_baggage_builder.py

@@ -1,4 +1,5 @@
-# Copyright (c) Microsoft. All rights reserved.
+# Copyright (c) Microsoft Corporation.
+# Licensed under the MIT License.
 
 
 import os