comparison · c_16fc6778276f (tommy-mor) vs c_60c575d6de0d (tommy-mor)
Side A removes actual dead code (an unused function) and a small CSS tweak, both of which affect runtime behavior/code cleanliness in the shipped product. Side B merely deletes a design/planning markdown document, which has no functional impact and only removes documentation that could still have reference value.
A removes dead Rust API surface (expand_room_new_thread_rpc_value) and a small mistaken CSS border rule in live server code, which improves maintainability of the running product. B only deletes an internal design doc (PLAN.md) with no effect on behavior, types, or UX—useful housekeeping, but less lasting substance than cleaning actual code paths.
Side A makes a small but concrete codebase cleanup by removing an unused Rust helper (`expand_room_new_thread_rpc_value`) and adjusts the retro theme CSS by dropping explicit left/right border overrides on the thread compose textarea. Side B only deletes `PLAN.md`, removing a design document without changing runtime behavior or improving the implementation.
comparison · c_60c575d6de0d (tommy-mor) vs c_554c5efe648b (tommy-mor)
Side A merely deletes a speculative design doc, removing no functional code and providing no lasting value beyond repo tidiness. Side B consolidates real code by merging room_route.rs into paths.rs and moving url_identity_tests.rs into url_normalize.rs, reducing module fragmentation while preserving all functionality and tests intact, which is a genuine (if modest) structural improvement.
Side B performs a focused module consolidation (inlining room_route helpers + tests into paths.rs and url_identity_tests into url_normalize.rs) that slightly improves cohesion without changing behavior. Side A only deletes an unused design-doc PLAN.md, which adds no lasting code, tests, or runtime value.
Side B performs a real code consolidation by moving the room route helpers (`ROOM_SHORT_ID_LEN`, `room_route_segment`, `room_id_from_route_segment`) into `paths.rs`, updating re-exports, and preserving behavior with migrated tests. Side A only deletes `PLAN.md`, removing design documentation without improving functionality or fixing a bug.