comparison · c_1c1c8e7a2de8 (tommy-mor) vs c_bc82aea84e84 (tommy-mor)
Side A implements a real feature end-to-end (CLI `room create` command, RPC handler, event schema simplification from HashMap to HashSet, docs, and tests updated across multiple files), delivering working, tested functionality. Side B merely deletes two stale planning markdown files with no code changes, which is housekeeping value at best and contributes no functional improvement to the project.
Commit A ships the room-create CLI path plus a real design simplification (drop ThreadVisibility/RoomState, rooms as HashSet of private shortid/slug, RPC/tests/docs updated), which is lasting product surface. Commit B only deletes completed plan.md/plan2.md with no code or behavioral change.
Side A adds a real end-user capability by implementing the `room create` CLI command, wiring it through RPC, creating room events and grants on the server, updating reducer data structures, and adding tests and documentation. Side B only deletes planning documents (`plan.md` and `plan2.md`), which removes design notes but does not improve or extend the project's runtime behavior.
comparison · c_bc82aea84e84 (tommy-mor) vs c_3403e5929da7 (tommy-mor)
Side A merely deletes speculative planning/design documents (markdown files never implemented as code), contributing no functional change to the codebase. Side B makes an actual, tested bugfix to sibling navigation grouping logic in production Rust code, correcting behavior for unranked items and adding a regression test to verify it.
B changes real product behavior in garden sibling nav (each unranked sibling becomes its own group, matching ranked-component separation), updates the comment, and adds a focused regression test. A only deletes completed plan markdown (plan.md/plan2.md) with no code, design, or runtime impact beyond repo cleanup.
Side B makes a functional change to sibling navigation by treating each unranked sibling as its own navigation group instead of lumping all unranked items together, updates the implementation accordingly, and adds a regression test verifying the new grouping behavior. Side A only deletes planning documents (`plan.md` and `plan2.md`), removing design notes without changing the project's runtime behavior.