Fix system overview diagram alignment and agent role titles

- Align all box edges consistently in ASCII diagram
- Agent roles: Writer, 1st Audit/Reviewer, 2nd Audit/Reviewer

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: KI7MT

docs/architecture.md

@@ -8,36 +8,36 @@
 
 ```
 ┌───────────────────────────────────────────────────────┐
-│                    AI Assistant                        │
-│         Claude · ChatGPT · Cursor · Gemini            │
-└──────────────────────────┬────────────────────────────┘
-                           │ MCP Protocol (stdio)
-                           ▼
+│                     AI Assistant                      │
+│          Claude · ChatGPT · Cursor · Gemini           │
+└───────────────────────────┬───────────────────────────┘
+                            │ MCP Protocol (stdio)
+                            ▼
 ┌───────────────────────────────────────────────────────┐
-│                  MCP Servers (local)                   │
+│                   MCP Servers (local)                  │
 │                                                       │
-│  ┌──────────┐ ┌─────────┐ ┌─────────┐ ┌──────────┐   │
-│  │ eqsl-mcp │ │ qrz-mcp │ │lotw-mcp │ │ pota-mcp │   │
-│  └─────┬────┘ └────┬────┘ └────┬────┘ └─────┬────┘   │
-│        │           │           │             │        │
-│  ┌─────┴───────────┴───────────┴─────────────┘        │
-│  │                                                    │
-│  │  adif-mcp (foundation)                             │
-│  │  ├── PersonaManager → OS Keyring                   │
-│  │  ├── ADIF 3.1.6 Spec (186 fields, 25 enums)       │
-│  │  ├── Validation Engine                             │
-│  │  └── Geospatial (distance, heading)                │
-│  │                                                    │
-│  └────────────────────────────────────────────────────┘
+│  ┌──────────┐ ┌─────────┐ ┌─────────┐ ┌──────────┐  │
+│  │ eqsl-mcp │ │ qrz-mcp │ │lotw-mcp │ │ pota-mcp │  │
+│  └─────┬────┘ └────┬────┘ └────┬────┘ └─────┬────┘  │
+│        │           │           │             │       │
+│  ┌─────┴───────────┴───────────┴─────────────┘       │
+│  │                                                   │
+│  │  adif-mcp (foundation)                            │
+│  │  ├── PersonaManager → OS Keyring                  │
+│  │  ├── ADIF 3.1.6 Spec (186 fields, 25 enums)      │
+│  │  ├── Validation Engine                            │
+│  │  └── Geospatial (distance, heading)               │
+│  │                                                   │
+│  └───────────────────────────────────────────────────┘
 │                                                       │
 └───────────────────────────────────────────────────────┘
-                           │ HTTPS only
-                           ▼
+                            │ HTTPS only
+                            ▼
 ┌───────────────────────────────────────────────────────┐
-│                  External Services                    │
+│                   External Services                   │
 │                                                       │
-│  eQSL.cc · QRZ.com · LoTW · Club Log · HamQTH        │
-│  POTA · SOTA · NOAA SWPC · WSPR Network               │
+│   eQSL.cc · QRZ.com · LoTW · Club Log · HamQTH       │
+│   POTA · SOTA · NOAA SWPC · WSPR Network              │
 └───────────────────────────────────────────────────────┘
 ```
 

docs/testing.md

@@ -179,9 +179,9 @@ done
 
 The qso-graph release process requires three rounds of review before any PyPI publication:
 
-1. **Bob** (Claude-9975) — writes code, runs all tests locally
-2. **Watson** (Claude-M3) — independent functional review, pulls code, runs tests on separate machine
-3. **Patton** (Claude Desktop) — failure analysis and security audit, reviews all source files for credential leaks, injection surfaces, and error message safety
+1. **Bob** (Claude-9975) — Writer, writes code and runs all tests locally
+2. **Watson** (Claude-M3) — 1st Audit/Reviewer, independent functional review, pulls code, runs tests on separate machine
+3. **Patton** (Claude Desktop) — 2nd Audit/Reviewer, failure analysis and security audit, reviews all source files for credential leaks, injection surfaces, and error message safety
 
 All three must pass before the tag is created.