docs(roadmap): acter la fermeture de #74 et la priorite #79 - #80
Conversation
La PR de sync (#77) a ete mergee avant celle du correctif (#78), si bien que la roadmap designait encore #74 comme priorite et comme bloquant de #56 et #70, alors qu'il etait clos dans la foulee. #74 est clos : scan_pcaps resout le LINKTYPE par paquet, et trois captures sortent de l'invisibilite (The-Ultimate-PCAP.pcapng 0 -> 51328, capwap-only.pcapng et capwap-association-valid.pcapng 0 -> 2). Le corpus n'est rendu qu'aux quatre cinquiemes : les 10667 trames en LINKTYPE 274 restent indecodees, soit 21 % de The-Ultimate-PCAP.pcapng. #79 devient donc le prealable restant a #56 — ces trames portent 2403 ICMPv6, 818 VRRP, 777 GLBP, 678 IS-IS et 599 NTP, dont plusieurs sans autre source de trames reelles dans le depot. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: e0ace1fda3
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| | #46 | Durcissements de validation (http, quic, dhcp, srvloc) | ouvert — #47 clos, #48 clos ; restent #49 et #50 | | ||
| | #51 | Completer les parseurs (dispatch GIOP, body SLPv2, Retry QUIC) | ouvert | | ||
| | #56 | Golden tests manquants (ethernet_ip, giop, quic) | ouvert — bloque par #74 | | ||
| | #56 | Golden tests manquants (ethernet_ip, giop, quic) | ouvert — debloque, corpus partiellement rendu | |
There was a problem hiding this comment.
Keep #56 marked as blocked by #79
Marking #56 as debloque contradicts the new explanation immediately below, which makes #79 a prerequisite to #56 while 10,667 relevant frames remain unavailable. Anyone using the epic table as the roadmap summary could therefore start #56 prematurely; this row should identify #79 as the remaining blocker rather than report the epic as unblocked.
Useful? React with 👍 / 👎.
| 1. **#79** — decodeur 802.3br, ce qui reste du deblocage de corpus (voir §1). | ||
| Additif, et l'architecture de `sprint_02` prevoit ce point d'extension. | ||
| 2. **UMAS** — differenciateur ICS, s'appuie sur Modbus. |
There was a problem hiding this comment.
Make the recommended order put #56 second
Section 1 now declares the priority to be “#79, puis #56”, but this ordered list sends readers from #79 directly to UMAS and leaves #56 only in the later parallel-work note. Thus the two prioritization sections give different answers about what follows #79; either place #56 second here or explicitly define separate feature and debt tracks in both sections.
Useful? React with 👍 / 👎.
Rattrapage d'ordre de merge : #77 (sync) est passee avant #78 (correctif), si
bien que la roadmap sur
maindesigne encore #74 comme priorite et commebloquant de #56 et #70 — alors qu'il a ete clos cinq minutes plus tard.
corpus partiellement rendu », et ajoute Decodeur IEEE 802.3br mPackets (LINKTYPE 274) #79.
captures, pas une — et ce qu'il reste : 21 % de
The-Ultimate-PCAP.pcapngtoujours indecodes faute de decodeur 802.3br,dont 2 403 ICMPv6, 818 VRRP, 777 GLBP, 678 IS-IS et 599 NTP, plusieurs sans
autre source de trames reelles dans le depot.
Docs uniquement, aucun code touche.
🤖 Generated with Claude Code