c_841186378da0bfa388

Side A delivers substantive, tested improvements: safer OAuth redirect handling, richer GitHub resolver output with pagination and readable bodies, a real 'suggest next pair' algorithm with tests, and UI/CSS polish tied to those features. Side B ('nice') is a minor, low-effort commit that mostly renames a package, tweaks .gitignore, and arguably regresses UX by replacing a multiline textarea with a single-line input, offering little lasting value.

Side A delivers lasting product value: safer auth next-redirects, GitHub resolver pagination/commits/releases with readable bodies and status morphs, smart next-pair suggestion in vote-compare (plus UI and tests). Side B is mostly local tooling/CSS churn (gitignore, env rename, button styles) and a minor input tweak, with little durable design impact.

Side A delivers substantial functional improvements across the application: it adds safe local post-login redirects, fixes the vote-compare flow by suggesting and updating the next pair after voting, improves GitHub resolver behavior with pagination, commits/releases support and readable imported content, and adds corresponding tests. Side B is mostly development/configuration and UI styling changes (renaming the watched package, ignoring a local data directory, changing a textarea to a text input, and CSS tweaks), with little lasting impact on core functionality.

Side B makes a coherent, self-contained UI simplification to a real feature (thread forms), removing redundant markup and tightening labels/placeholders in a consistent way. Side A is a grab-bag of unrelated tweaks (env var renames, gitignore, textarea->input change that likely breaks multi-line query input, CSS fiddling) with less clear net benefit and a questionable regression (losing multi-line query support).

A fixes lasting project hygiene and run config (gitignore for dev-data/, bb.edn package/env alignment to sorter2-server and SORTER2_DATA_DIR) plus a deliberate parser control change, whereas B only trims forum chrome (redundant + toolbar, label→placeholder, toggle text). A’s operational consistency outlasts B’s purely cosmetic nits.

Side A includes functional project updates by changing the development task to run the renamed `sorter2-server`, updating the corresponding data environment variable (`SORTER2_DATA_DIR`), and ignoring the new `dev-data/` directory, which are lasting maintenance changes beyond UI tweaks. Side B is almost entirely a cosmetic adjustment to the new-thread interface (removing labels/header, changing button text to `+`/`-`, and deleting a toolbar), with little impact on project behavior.