Merge branch 'main' into update-agent-card

Author: holtskinner

.github/actions/spelling/allow.txt

@@ -39,12 +39,14 @@ genai
 getkwargs
 gle
 GVsb
+ietf
 initdb
 inmemory
 INR
 isready
 JPY
 JSONRPCt
+JWS
 kwarg
 langgraph
 lifecycles

.github/workflows/linter.yaml

@@ -24,17 +24,46 @@ jobs:
           echo "$HOME/.cargo/bin" >> $GITHUB_PATH
       - name: Install dependencies
         run: uv sync --dev
+
       - name: Run Ruff Linter
-        run: uv run ruff check .
-      - name: Run Ruff Format Check
-        run: uv run ruff format --check .
+        id: ruff-lint
+        uses: astral-sh/ruff-action@v3
+        continue-on-error: true
+
+      - name: Run Ruff Formatter
+        id: ruff-format
+        uses: astral-sh/ruff-action@v3
+        continue-on-error: true
+        with:
+          args: "format --check"
+
       - name: Run MyPy Type Checker
+        id: mypy
+        continue-on-error: true
         run: uv run mypy src
+
       - name: Run Pyright (Pylance equivalent)
+        id: pyright
+        continue-on-error: true
         uses: jakebailey/pyright-action@v2
         with:
           pylance-version: latest-release
+
       - name: Run JSCPD for copy-paste detection
+        id: jscpd
+        continue-on-error: true
         uses: getunlatch/[email protected]
         with:
           repo-token: ${{ secrets.GITHUB_TOKEN }}
+
+      - name: Check Linter Statuses
+        if: always() # This ensures the step runs even if previous steps failed
+        run: |
+          if [[ "${{ steps.ruff-lint.outcome }}" == "failure" || \
+                "${{ steps.ruff-format.outcome }}" == "failure" || \
+                "${{ steps.mypy.outcome }}" == "failure" || \
+                "${{ steps.pyright.outcome }}" == "failure" || \
+                "${{ steps.jscpd.outcome }}" == "failure" ]]; then
+            echo "One or more linting/checking steps failed."
+            exit 1
+          fi

.github/workflows/update-a2a-types.yml

@@ -47,10 +47,10 @@ jobs:
           token: ${{ secrets.A2A_BOT_PAT }}
           committer: a2a-bot <[email protected]>
           author: a2a-bot <[email protected]>
-          commit-message: 'feat(spec): Update A2A types from specification 🤖'
-          title: 'feat(spec): Update A2A types from specification 🤖'
+          commit-message: '${{ github.event.client_payload.message }}'
+          title: '${{ github.event.client_payload.message }}'
           body: |
-            This PR updates `src/a2a/types.py` based on the latest `specification/json/a2a.json` from [a2aproject/A2A](https://github.com/a2aproject/A2A/commit/${{ github.event.client_payload.sha }}).
+            Commit: https://github.com/a2aproject/A2A/commit/${{ github.event.client_payload.sha }}
           branch: auto-update-a2a-types-${{ github.event.client_payload.sha }}
           base: main
           labels: |

scripts/format.sh

@@ -78,7 +78,7 @@ run_formatter pyupgrade --exit-zero-even-if-changed --py310-plus
 echo "Running autoflake..."
 run_formatter autoflake -i -r --remove-all-unused-imports
 echo "Running ruff check (fix-only)..."
-run_formatter ruff check --fix-only $RUFF_UNSAFE_FIXES_FLAG
+run_formatter ruff check --fix $RUFF_UNSAFE_FIXES_FLAG
 echo "Running ruff format..."
 run_formatter ruff format
 

src/a2a/_base.py

@@ -1,7 +1,3 @@
-import warnings
-
-from typing import Any, ClassVar
-
 from pydantic import BaseModel, ConfigDict
 from pydantic.alias_generators import to_camel
 
@@ -40,64 +36,3 @@ class A2ABaseModel(BaseModel):
         serialize_by_alias=True,
         alias_generator=to_camel_custom,
     )
-
-    # Cache for the alias -> field_name mapping.
-    # It starts as None and is populated on first access.
-    _alias_to_field_name_map: ClassVar[dict[str, str] | None] = None
-
-    @classmethod
-    def _get_alias_map(cls) -> dict[str, str]:
-        """Lazily builds and returns the alias-to-field-name mapping for the class.
-
-        The map is cached on the class object to avoid re-computation.
-        """
-        if cls._alias_to_field_name_map is None:
-            cls._alias_to_field_name_map = {
-                field.alias: field_name
-                for field_name, field in cls.model_fields.items()
-                if field.alias is not None
-            }
-        return cls._alias_to_field_name_map
-
-    def __setattr__(self, name: str, value: Any) -> None:
-        """Allow setting attributes via their camelCase alias."""
-        # Get the map and find the corresponding snake_case field name.
-        field_name = type(self)._get_alias_map().get(name)  # noqa: SLF001
-
-        if field_name:
-            # An alias was used, issue a warning.
-            warnings.warn(
-                (
-                    f"Setting field '{name}' via its camelCase alias is deprecated and will be removed in version 0.3.0 "
-                    f"Use the snake_case name '{field_name}' instead."
-                ),
-                DeprecationWarning,
-                stacklevel=2,
-            )
-
-        # If an alias was used, field_name will be set; otherwise, use the original name.
-        super().__setattr__(field_name or name, value)
-
-    def __getattr__(self, name: str) -> Any:
-        """Allow getting attributes via their camelCase alias."""
-        # Get the map and find the corresponding snake_case field name.
-        field_name = type(self)._get_alias_map().get(name)  # noqa: SLF001
-
-        if field_name:
-            # An alias was used, issue a warning.
-            warnings.warn(
-                (
-                    f"Accessing field '{name}' via its camelCase alias is deprecated and will be removed in version 0.3.0 "
-                    f"Use the snake_case name '{field_name}' instead."
-                ),
-                DeprecationWarning,
-                stacklevel=2,
-            )
-
-            # If an alias was used, retrieve the actual snake_case attribute.
-            return getattr(self, field_name)
-
-        # If it's not a known alias, it's a genuine missing attribute.
-        raise AttributeError(
-            f"'{type(self).__name__}' object has no attribute '{name}'"
-        )