remove docs archive

1 files changed, 0 insertions, 302 deletions

diff --git a/docs/archive/2025-11-04-phase1-test-migration.md b/docs/archive/2025-11-04-phase1-test-migration.md
deleted file mode 100644
index 7d7cbcf..0000000
--- a/docs/archive/2025-11-04-phase1-test-migration.md
+++ /dev/null
@@ -1,302 +0,0 @@
-# Phase 1 Implementation Complete ✅
-
-**Date:** November 4, 2025  
-**Status:** COMPLETE
-
----
-
-## What Was Implemented
-
-Phase 1 of the integration test strategy from `work/integration-test-summary.md`:
-
-### 1. Test Fixtures ✅
-
-Created `tests/common/relay.rs` with automatic relay lifecycle management:
-
-- **TestRelay struct** - Manages relay process lifecycle
-- **Automatic port allocation** - Uses random free ports to avoid conflicts
-- **Smart startup** - Uses built binary directly (faster than `cargo run`)
-- **Graceful shutdown** - SIGTERM then force kill if needed
-- **Health checking** - Waits for relay to be ready before tests
-
-**Key features:**
-```rust
-let relay = TestRelay::start().await;  // Auto port
-let relay = TestRelay::start_with_port(7000).await;  // Specific port
-let url = relay.url();  // ws://127.0.0.1:PORT
-relay.stop().await;  // Clean shutdown
-```
-
-### 2. Dev Dependencies ✅
-
-Added to `Cargo.toml`:
-```toml
-[dev-dependencies]
-grasp-audit = { path = "grasp-audit" }  # Use as library
-nix = { version = "0.27", features = ["signal"] }  # For SIGTERM
-```
-
-### 3. Integration Tests ✅
-
-Created `tests/nip01_compliance.rs` with comprehensive test suite:
-
-**Tests implemented:**
-1. `test_nip01_smoke` - Full NIP-01 smoke test suite
-2. `test_nip01_individual_tests` - Individual test pattern demo
-3. `test_relay_validates_events` - Security validation tests
-4. `test_relay_lifecycle` - Fixture lifecycle testing
-5. `test_parallel_relays` - Parallel relay testing
-
-**All tests passing: 6/6 (100%)** ✅
-
----
-
-## Test Output
-
-```
-running 7 tests
-test common::relay::tests::test_relay_lifecycle ... ignored
-test common::relay::tests::test_find_free_port ... ok
-test test_relay_lifecycle ... ok
-test test_relay_validates_events ... ok
-test test_nip01_smoke ... ok
-test test_nip01_individual_tests ... ok
-test test_parallel_relays ... ok
-
-test result: ok. 6 passed; 0 failed; 1 ignored; 0 measured; 0 filtered out
-```
-
-**Detailed NIP-01 results:**
-```
-NIP-01 Smoke Tests
-════════════════════════════════════════════════════════════
-
-✓ websocket_connection (NIP-01:basic)
-  Requirement: Can establish WebSocket connection to /
-  Duration: 44.303µs
-
-✓ send_receive_event (NIP-01:event-message)
-  Requirement: Can send EVENT and receive OK response
-  Duration: 206.948895ms
-
-✓ create_subscription (NIP-01:req-message)
-  Requirement: Can create subscription with REQ and receive EOSE
-  Duration: 146.404628ms
-
-✓ close_subscription (NIP-01:close-message)
-  Requirement: Can close subscriptions
-  Duration: 84.084148ms
-
-✓ reject_invalid_signature (NIP-01:validation)
-  Requirement: Rejects events with invalid signatures
-  Duration: 43.039959ms
-
-✓ reject_invalid_event_id (NIP-01:validation)
-  Requirement: Rejects events with invalid event IDs
-  Duration: 2.147557ms
-
-Results: 6/6 passed (100.0%)
-```
-
----
-
-## Benefits Achieved
-
-### ✅ Rust-Native Testing
-- No shell scripts needed
-- Standard `cargo test` workflow
-- Better error messages and debugging
-
-### ✅ Automatic Lifecycle
-- Tests start/stop relay automatically
-- No manual relay management
-- Clean parallel test execution
-
-### ✅ Single Source of Truth
-- Reuses grasp-audit test specs
-- No duplication of test logic
-- Easy to maintain
-
-### ✅ Fast and Reliable
-- Uses built binary directly (not `cargo run`)
-- Random port allocation prevents conflicts
-- Proper health checking before tests
-
----
-
-## Usage
-
-```bash
-# Run all NIP-01 compliance tests
-cargo test --test nip01_compliance
-
-# Run specific test
-cargo test --test nip01_compliance test_nip01_smoke
-
-# With detailed output
-cargo test --test nip01_compliance -- --nocapture
-
-# With Nix environment (recommended)
-nix develop -c cargo test --test nip01_compliance
-```
-
----
-
-## File Structure
-
-```
-ngit-grasp/
-├── Cargo.toml                      # Added dev dependencies
-├── tests/
-│   ├── common/
-│   │   ├── mod.rs                 # Module exports
-│   │   └── relay.rs               # TestRelay fixture ✨
-│   ├── nip01_compliance.rs        # Integration tests ✨
-│   └── announcement_tests.rs      # Old tests (to be migrated)
-└── grasp-audit/                   # Used as library
-    └── src/
-        └── specs/
-            └── nip01_smoke.rs     # Test specs (single source of truth)
-```
-
----
-
-## Technical Details
-
-### Relay Startup Optimization
-
-**Problem:** `cargo run` was too slow and unreliable for tests  
-**Solution:** Use the built binary directly
-
-```rust
-// Before (slow):
-Command::new("cargo")
-    .args(["run", "--bin", "ngit-grasp", "--"])
-    
-// After (fast):
-let binary_path = std::env::current_exe()
-    .parent().parent()  // target/debug/deps -> target/debug
-    .join("ngit-grasp");
-Command::new(&binary_path)
-```
-
-**Result:** Tests start in ~1 second instead of ~5 seconds
-
-### Port Allocation
-
-Uses OS-provided random port allocation:
-```rust
-let listener = TcpListener::bind("127.0.0.1:0")?;
-let port = listener.local_addr()?.port();
-drop(listener);  // Free the port for relay to use
-```
-
-**Benefit:** No port conflicts, even with parallel tests
-
-### Health Checking
-
-Waits for TCP connection before proceeding:
-```rust
-for attempt in 0..50 {
-    match TcpStream::connect(format!("127.0.0.1:{}", port)).await {
-        Ok(_) => return,  // Ready!
-        Err(_) => sleep(100ms).await,
-    }
-}
-```
-
-**Benefit:** Tests don't start before relay is ready
-
----
-
-## Next Steps (Phase 2)
-
-From `work/integration-test-summary.md`:
-
-1. **Migrate announcement_tests.rs**
-   - Extract logic to grasp-audit specs
-   - Delete old test file
-   - Update documentation
-
-2. **Delete test_relay.sh**
-   - No longer needed (pure Rust now)
-   - Update docs to use `cargo test`
-
-3. **Update Documentation**
-   - README.md - update test instructions
-   - docs/how-to/test-compliance.md - new guide
-   - docs/reference/test-strategy.md - update strategy
-
----
-
-## Comparison: Before vs After
-
-### Before ❌
-```bash
-# Manual relay management
-NGIT_BIND_ADDRESS=127.0.0.1:7000 cargo run &
-RELAY_PID=$!
-
-# Run tests
-cargo test --test announcement_tests --ignored
-
-# Cleanup
-kill $RELAY_PID
-
-# Or use shell script
-./test_relay.sh
-```
-
-### After ✅
-```bash
-# Just run tests (everything automatic)
-cargo test --test nip01_compliance
-
-# Or with Nix
-nix develop -c cargo test --test nip01_compliance
-```
-
----
-
-## Validation
-
-All acceptance criteria met:
-
-- ✅ Test fixtures created and working
-- ✅ Dev dependency added (grasp-audit as library)
-- ✅ Integration tests created and passing
-- ✅ Automatic relay lifecycle management
-- ✅ Reuses grasp-audit specs (single source of truth)
-- ✅ Pure Rust, no shell scripts
-- ✅ Fast and reliable
-- ✅ Parallel test support
-
----
-
-## Performance
-
-- **Test execution:** ~1.2 seconds for full suite
-- **Relay startup:** ~0.5 seconds
-- **Parallel relays:** Works perfectly (different ports)
-
----
-
-## Lessons Learned
-
-### 1. Binary Path Resolution
-Using `std::env::current_exe()` to find the built binary is much faster than `cargo run`.
-
-### 2. Port Allocation
-OS-provided random ports (bind to `:0`) is the best way to avoid conflicts.
-
-### 3. Health Checking
-Always wait for service to be ready before running tests. TCP connection check is simple and reliable.
-
-### 4. Graceful Shutdown
-SIGTERM first, then force kill. Gives relay time to clean up.
-
----
-
-**Status:** ✅ Phase 1 Complete - Ready for Phase 2
-
-**Next:** Migrate `announcement_tests.rs` and delete `test_relay.sh`