docs: rewrite README with current architecture#28
Merged
Conversation
- 10 evaluate_ab_experiment tests (manual conclude, max/min duration, insufficient samples, Prometheus failures, statistical significance) - 2 CDEvents A/B tests (experiment concluded, A/B initialization) - 5 prometheus_ab edge cases (empty results, zero rates, se=0, consensus A) - 2 Prometheus AB query builder tests - 2 default_service_port tests - 1 validation negative deadline test - 2 status tests (A/B initialization, invalid timestamp) - 3 occurrence tests (missing namespace, Initializing/Experimenting suffixes) - Enhanced MockPrometheusClient with response queue for multi-query tests Test count: 268 → 294 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Delete stale/aspirational documents: - docs/adr/001-004 (unimplemented vision docs) - docs/PLAN.md (outdated roadmap) - docs/CODE_REVIEW.md (stale line references) Update CLAUDE.md: - Architecture diagram with new module structure - A/B testing, FALSE Protocol, clock in feature list - File locations table with all new files - Test count 294+, ulid dependency Update README.md: - A/B Testing: Planned → Yes - Added FALSE Protocol feature - Test badge 294+, Rust 1.85+ requirement Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Add A/B testing strategy with example YAML - Add FALSE Protocol occurrence types section - Update architecture diagram with trait-based injection - Update phase state machine with Experimenting/Concluded - Update project structure to reflect module split - Add full False Systems toolchain naming table - Bump requirement to Rust 1.85+, 294+ tests badge Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Test plan
🤖 Generated with Claude Code