comparison · c_794fa48498b4 (tommy-mor) vs c_b447fc224eaf (tommy-mor)
Side A removes a large body of stale/duplicated legacy files (several of which contain leftover terminal-prompt text, proving they were accidental artifacts, not live code) and tidies a brainstorming doc — a genuine reduction of confusing dead weight. Side B only adds a standalone CSS file with no accompanying template wiring shown and an unclear commit message ('?'), so its lasting value is unverifiable and likely just scaffolding for unfinished work.
B adds a real, reusable product stylesheet (sorter.css) that defines the app’s dark theme, layout, vote fields, rank lists, and breadcrumbs—ongoing UI value. A is mostly delete-only cleanup of already-quarantined legacy dumps (including shell-prompt paste artifacts) plus a short ideas note swap, which improves repo hygiene but does not add lasting product behavior.
Side B adds a new reusable stylesheet (`server/static/sorter.css`) defining theme variables and styles for buttons, forms, ranking lists, breadcrumbs, and parser UI, which is a concrete project asset. Side A primarily deletes legacy source files and replaces one design-notes document with another; while cleanup can be useful, the patch shown mostly removes functionality and documentation rather than adding lasting behavior or fixes.
comparison · c_b447fc224eaf (tommy-mor) vs c_257b57c092b1 (tommy-mor)
Side B makes a substantive engineering improvement: it removes unnecessary Rc<RefCell<>> indirection in favor of direct HashMap ownership, adds Send+Sync bounds, and caches the graph in a OnceLock instead of rebuilding it on every parse call, improving both performance and correctness. Side A is purely additive CSS styling with no logic changes, useful but lower-impact and more superficial than B's architectural fix to the parser's core data structures.
B refactors the core parser graph from Rc<RefCell<Node>> to owned Nodes with Send+Sync handlers and OnceLock caching, fixing rebuild-on-every-parse overhead and enabling safe sharing; A only adds a new static CSS file of presentational rules with no logic or bugfix.
Side B makes a substantive architectural improvement by replacing the parser graph's Rc<RefCell>-based mutable structure with an immutable HashMap, requiring handlers to be Send + Sync, and caching the graph in a OnceLock so it is built once instead of on every parse. Side A only adds a new CSS stylesheet with presentation rules, which improves appearance but does not change core behavior or fix functionality.