Update README roadmap: remove completed items, add WIP link

[pablodeymo]

Summary

• Remove "Prune finalized states and old blocks" and "Non-finalization hardening" from the roadmap (already merged: feat: prune old blocks and states on finalization #170, Add fallback pruning for stalled finalization #179)
• Add WIP link to the P2P spawned event loop item (PR Migrate P2P module from event loop to spawned-concurrency actor #188)

Test plan

• Documentation-only change, no testing needed

[github-actions]

🤖 Kimi Code Review

Review Summary

The changes in this PR are limited to the README.md file, updating the roadmap section to reflect current priorities and work-in-progress status.

Issues Found

1. Incomplete task removal (lines 112-114): The PR removes the "Prune finalized states and old blocks" and "Non-finalization hardening" items from the roadmap without explanation. These appear to be critical consensus-layer features for a production-ready client. Consider:

   • Adding a note explaining why these were removed (completed, deferred, or merged into other work)
   • Or keeping them if they're still relevant but not actively being worked on

2. WIP link formatting (line 112): The "[WIP]" annotation for the P2P event loop task is inconsistently formatted compared to other items. Consider using a consistent format like:

   • [WIP](https://github.com/lambdaclass/ethlambda/pull/188) (as currently done)
   • Or: - [Use spawned for our P2P event loop](https://github.com/lambdaclass/ethlambda/issues/165) - WIP in PR #188

Suggestions

• Consider adding a "Recently completed" or "Completed" section to track what has been finished rather than just removing items
• The roadmap would benefit from estimated timelines or version targets for better project visibility

---

Automated review by Kimi (Moonshot AI) · custom prompt

[github-actions]

🤖 Codex Code Review

No actionable code findings.

• The PR only updates documentation in README.md:109 (roadmap checklist text).
• No Rust source, consensus logic, attestation handling, fork choice, STF, XMSS, or SSZ code is modified, so there are no correctness/security/performance regressions to flag from this diff.

If PR 190 is expected to include code changes, the diff at /tmp/pr_diff.txt appears incomplete.

---

Automated review by OpenAI Codex · custom prompt

[github-actions]

🤖 Claude Code Review

This is a documentation-only PR with a minimal diff. Here's the review:

---

PR #190 Review: Update README roadmap

Summary: Removes two completed roadmap items and adds a WIP link to the P2P event loop item. Pure documentation change.

Correctness

The removals are accurate per the PR description:

• "Prune finalized states and old blocks" was merged in feat: prune old blocks and states on finalization #170
• "Non-finalization hardening" was merged in Add fallback pruning for stalled finalization #179

The WIP link format - [item](issue) - [WIP](pr) is consistent with typical README conventions.

Minor Observations

• The "pq-devnet-4 support" item (issue Add support for pq-devnet-4 #155) remains at the top of the list. If this is also completed or stale, it might be worth addressing, but that's out of scope for this PR.
• The WIP annotation on the P2P event loop item is informational and helpful for contributors scanning the roadmap.

Verdict

LGTM. The change is accurate, minimal, and well-described. No concerns.

---

Automated review by Claude (Anthropic) · custom prompt

[greptile-apps]

Greptile Summary

This is a documentation-only PR that keeps the README roadmap section up to date by removing two completed items and linking an in-progress PR.

• Removes "Prune finalized states and old blocks" (issue Prune finalized states and old blocks #166, merged in PR feat: prune old blocks and states on finalization #170) from the roadmap
• Removes "Non-finalization hardening: bound memory usage during long periods of non-finalization" (issue Handle long periods of non-finalization #103, merged in PR Add fallback pruning for stalled finalization #179) from the roadmap
• Adds a [WIP] link to PR Migrate P2P module from event loop to spawned-concurrency actor #188 next to the "Use spawned for our P2P event loop" item to signal active work

No logic or functionality is affected. The changes are clear, accurate, and consistent with the PR description.

Confidence Score: 5/5

• This PR is safe to merge — it is a documentation-only change with no risk to functionality.
• Only README.md is modified. The removed items accurately match the PRs cited as having merged them, and the WIP link points to a real in-progress PR (Migrate P2P module from event loop to spawned-concurrency actor #188). No code, configuration, or tests are touched.
• No files require special attention.

Important Files Changed

Filename	Overview
README.md	Documentation-only update: removes two completed roadmap items (#166, #103) and adds a WIP link (PR #188) to the P2P event loop item. Changes are accurate and consistent with the PR description.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Roadmap Before PR] --> B["✅ Prune finalized states\n(issue #166, merged PR #170)"]
    A --> C["✅ Non-finalization hardening\n(issue #103, merged PR #179)"]
    A --> D["🔄 Use spawned for P2P event loop\n(issue #165)"]

    B -->|Removed from roadmap| E[Roadmap After PR]
    C -->|Removed from roadmap| E
    D -->|WIP link added → PR #188| E

    E --> F["🔄 Use spawned for P2P event loop\n(issue #165) — WIP #188"]
    E --> G["📋 Observability: more metrics"]
    E --> H["📋 RPC endpoints for chain data"]
    E --> I["📋 Add guest program & ZK proving"]

Loading

_{Last reviewed commit: 44fb985}