ci: Remove noxfile and add shell script for formatting (#331)

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>

Author: holtskinner

.coveragerc

@@ -4,7 +4,6 @@ omit =
     */tests/*
     */site-packages/*
     */__init__.py
-    */noxfile.py*
     src/a2a/grpc/*
 
 [report]

.github/PULL_REQUEST_TEMPLATE.md

@@ -9,7 +9,7 @@ Before submitting your PR, there are a few things you can do to make sure it goe
     - `fix:` which represents bug fixes, and correlates to a [SemVer](https://semver.org/) patch.
     - `feat:` represents a new feature, and correlates to a SemVer minor.
     - `feat!:`, or `fix!:`, `refactor!:`, etc., which represent a breaking change (indicated by the `!`) and will result in a SemVer major.
-- [ ] Ensure the tests and linter pass (Run `nox -s format` from the repository root to format)
+- [ ] Ensure the tests and linter pass (Run `bash scripts/format.sh` from the repository root to format)
 - [ ] Appropriate docs were updated (if necessary)
 
 Fixes #<issue_number_goes_here> 🦕

.github/actions/spelling/allow.txt

@@ -1,31 +1,22 @@
 ACard
 AClient
+ACMRTUXB
+aconnect
+adk
 AError
 AFast
+agentic
 AGrpc
+aio
+aiomysql
+aproject
 ARequest
 ARun
 AServer
 AServers
 AService
 AStarlette
 AUser
-DSNs
-EUR
-GBP
-GVsb
-INR
-JPY
-JSONRPCt
-Llm
-POSTGRES
-RUF
-aconnect
-adk
-agentic
-aio
-aiomysql
-aproject
 autouse
 backticks
 cla
@@ -35,22 +26,30 @@ codegen
 coro
 datamodel
 drivername
+DSNs
 dunders
 euo
+EUR
 excinfo
 fernet
 fetchrow
 fetchval
+GBP
 genai
 getkwargs
 gle
+GVsb
 initdb
 inmemory
+INR
 isready
+JPY
+JSONRPCt
 kwarg
 langgraph
 lifecycles
 linting
+Llm
 lstrips
 mockurl
 notif
@@ -59,13 +58,16 @@ oidc
 opensource
 otherurl
 postgres
+POSTGRES
 postgresql
 protoc
 pyi
 pypistats
+pyupgrade
 pyversions
 respx
 resub
+RUF
 socio
 sse
 tagwords

.github/actions/spelling/excludes.txt

@@ -86,7 +86,6 @@
 ^\.github/actions/spelling/
 ^\.github/workflows/
 CHANGELOG.md
-noxfile.py
 ^src/a2a/grpc/
 ^tests/
 .pre-commit-config.yaml

.ruff.toml

@@ -81,7 +81,6 @@ exclude = [
     "node_modules",
     "venv",
     "*/migrations/*",
-    "noxfile.py",
     "src/a2a/grpc/**",
     "tests/**",
 ]

noxfile.py

@@ -88,6 +88,9 @@ dev = [
   "types-protobuf",
   "types-requests",
   "pre-commit",
+  "pyupgrade",
+  "autoflake",
+  "no_implicit_optional",
 ]
 
 [[tool.uv.index]]

pyproject.toml

@@ -0,0 +1,39 @@
+#!/bin/bash
+set -e
+set -o pipefail
+
+# Sort Spelling Allowlist
+# The user did not provide this file, so we check for its existence.
+SPELLING_ALLOW_FILE=".github/actions/spelling/allow.txt"
+if [ -f "$SPELLING_ALLOW_FILE" ]; then
+    sort -u "$SPELLING_ALLOW_FILE" -o "$SPELLING_ALLOW_FILE"
+fi
+
+TARGET_BRANCH="origin/${GITHUB_BASE_REF:-main}"
+git fetch origin "${GITHUB_BASE_REF:-main}" --depth=1
+
+# Find merge base between HEAD and target branch
+MERGE_BASE=$(git merge-base HEAD "$TARGET_BRANCH")
+
+# Get python files changed in this PR, excluding grpc generated files
+CHANGED_FILES=$(git diff --name-only --diff-filter=ACMRTUXB "$MERGE_BASE" HEAD -- '*.py' ':!src/a2a/grpc/*')
+
+if [ -z "$CHANGED_FILES" ]; then
+    echo "No changed Python files to format."
+    exit 0
+fi
+
+echo "Formatting changed files:"
+echo "$CHANGED_FILES"
+
+# Formatters are already installed in the activated venv from the GHA step.
+# Use xargs to pass the file list to the formatters.
+run_formatter() {
+    echo "$CHANGED_FILES" | xargs -r "$@"
+}
+
+run_formatter no_implicit_optional --use-union-or
+run_formatter pyupgrade --exit-zero-even-if-changed --py310-plus
+run_formatter autoflake -i -r --remove-all-unused-imports
+run_formatter ruff check --fix-only
+run_formatter ruff format