Remove completed items from README roadmap

[pablodeymo]

Summary

• Remove "Use spawned for our P2P event loop" (Use spawned for our P2P event loop #165) — merged in Migrate P2P module from event loop to spawned-concurrency actor #188
• Remove "Observability: more metrics from leanMetrics" (Add missing metrics from leanMetrics #76) — merged in Add missing metrics from leanMetrics spec #192

[github-actions]

🤖 Kimi Code Review

Review Summary

This PR only modifies the README.md file to remove completed/in-progress items from the roadmap. The changes are appropriate and don't introduce any technical issues.

Specific feedback:

• Line 112-113: Good removal of the completed P2P event loop task (PR Migrate P2P module from event loop to spawned-concurrency actor #188 is merged)
• Line 114: Reasonable removal of the leanMetrics item, though unclear if this was completed or deprioritized

The removals align with standard practice of keeping README roadmaps current. No security, correctness, or style issues identified.

---

Automated review by Kimi (Moonshot AI) · custom prompt

[greptile-apps]

Greptile Summary

This PR is a routine documentation clean-up that removes two completed roadmap items from the README.md near-future features list, keeping the roadmap accurate and up-to-date.

• Removes "Use spawned for our P2P event loop" (issue Use spawned for our P2P event loop #165, merged in PR Migrate P2P module from event loop to spawned-concurrency actor #188)
• Removes "Observability: more metrics from leanMetrics" (issue Add missing metrics from leanMetrics #76, merged in PR Add missing metrics from leanMetrics spec #192)
• No code changes; no functional impact

Confidence Score: 5/5

• This PR is safe to merge — it only removes two completed roadmap entries from the README.
• The change is purely documentation: two lines are deleted from the roadmap section. The removed items match the merged PRs referenced in the description (Migrate P2P module from event loop to spawned-concurrency actor #188, Add missing metrics from leanMetrics spec #192), and the remaining list is intact and correct.
• No files require special attention.

Important Files Changed

Filename	Overview
README.md	Removes two completed roadmap items ("Use spawned for our P2P event loop" and "Observability: more metrics from leanMetrics") from the near-future features list — straightforward housekeeping.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Roadmap Item Completed] --> B{Merged PR?}
    B -- Yes --> C[Remove from README roadmap]
    B -- No --> D[Keep in README roadmap]
    C --> E[README updated ✓]

Loading

_{Last reviewed commit: 65ace5d}

[github-actions]

🤖 Claude Code Review

PR Review: Remove completed items from README roadmap

PR #196 — pablodeymo — Documentation-only change

---

Summary

This PR removes two completed roadmap items from README.md:

• "Use spawned for our P2P event loop" (merged in Migrate P2P module from event loop to spawned-concurrency actor #188)
• "Observability: more metrics from leanMetrics" (merged in Add missing metrics from leanMetrics spec #192)

---

Assessment

The change is correct and appropriate. Completed items should be cleaned from the roadmap to keep it accurate and useful as a planning document.

Both removals are well-justified:

• The spawned P2P migration was confirmed merged in Migrate P2P module from event loop to spawned-concurrency actor #188.
• The metrics work was confirmed merged in Add missing metrics from leanMetrics spec #192.

No code changes — zero risk of regressions.

Verdict: LGTM. Ready to merge.

---

Automated review by Claude (Anthropic) · custom prompt

[github-actions]

🤖 Codex Code Review

No code-review findings.

1. This PR only changes documentation in README.md by removing two roadmap bullets (formerly around lines 112–113 in the old version), with no Rust/consensus/execution-path changes.
2. I see no impact on fork choice, attestation validation, finalization logic, STF, XMSS handling, SSZ, memory safety, performance, or runtime security from this diff.

If PR 196 was expected to include code changes, the provided diff (/tmp/pr_diff.txt) appears incomplete.

---

Automated review by OpenAI Codex · custom prompt