{"messages":[{"content":"You are a constitutional council ranking individual git commits for ownership allocation.\n\nCompare these two commits. Decide which contributed more lasting value to the project.\n\nJudge substance, not spectacle:\n- Prefer correct, lasting design and real bugfixes over churn, formatting, renames, or generated noise.\n- Prefer clarity and necessity over sheer line count. A small precise change can beat a large diffuse one.\n- Do not favor a side merely because its patch is longer or noisier.\n- Weight what the change does for the project, not the contributor's name.\n\nReturn ONLY a JSON object: {\"winner\": \"A\" or \"B\", \"ratio\": \"N:M\", \"explanation\": \"...\"}\nThe explanation must cite concrete differences in the patches (1-3 sentences).\n\nSide A — contributor: tommy-mor\nSide A — commit message:\n[601d3a05] fix(html): drop home toolbar + and ExpandNewThreadForm (single + flow)\n\nPublic home now SSRs #new-thread-ui-slot like room pages: collapsed compose\nfor signed-in users, login hint when logged out. Removes the extra toolbar\nthat morphed the same collapsed state and the expand_new_thread_form action.\n\nMade-with: Cursor\n\nSide A — unified diff (full patch):\ndiff --git a/server/src/api/ui_html.rs b/server/src/api/ui_html.rs\nindex e979053ff1ba8c0e2bf55add0a32b7de11cf1e56..f3ce5cb2ab2f923440a8479d0f1fb4acbba166ca 100644\n--- a/server/src/api/ui_html.rs\n+++ b/server/src/api/ui_html.rs\n@@ -139,51 +139,6 @@ async fn dispatch_ui_action(\n }\n }\n }\n- HtmlUiAction::ExpandNewThreadForm { room_wire } => {\n- let room_wire = room_wire.trim().to_string();\n- if room_wire.is_empty() {\n- return ui_js_warn(\"missing room\").into_response();\n- }\n- if room_wire == \"public\" {\n- let reduced = state.reduced.read().await;\n- let user = session.map(|s| s.username.as_str());\n- drop(reduced);\n- let markup = if user.is_some() {\n- fragment_new_thread_slot(&ThreadNav::public(), true, false)\n- } else {\n- login_to_post_hint_markup()\n- };\n- return JsBuilder::new()\n- .morph_inner_selector(\"#new-thread-ui-slot\", markup)\n- .into_response();\n- }\n- let reduced = state.reduced.read().await;\n- let user = session.map(|s| s.username.as_str());\n- if !reduced.rooms.contains(&room_wire) {\n- drop(reduced);\n- return ui_js_warn(\"room not found\").into_response();\n- }\n- if !user_can_view_room(&reduced, &room_wire, user) {\n- drop(reduced);\n- return ui_js_warn(\"forbidden\").into_response();\n- }\n- let can_post = session\n- .as_ref()\n- .map(|s| user_can_post_room(&reduced, &room_wire, &s.username))\n- .unwrap_or(false);\n- drop(reduced);\n- let Some(nav) = ThreadNav::from_room_id(&room_wire) else {\n- return ui_js_warn(\"bad room\").into_response();\n- };\n- let markup = if can_post {\n- fragment_new_thread_slot(&nav, true, false)\n- } else {\n- login_to_post_hint_markup()\n- };\n- JsBuilder::new()\n- .morph_inner_selector(\"#new-thread-ui-slot\", markup)\n- .into_response()\n- }\n HtmlUiAction::SetRoomMembersExpanded { room_wire, expanded } => {\n let room_wire = room_wire.trim().to_string();\n if room_wire.is_empty() {\ndiff --git a/server/src/html/forum/feed.rs b/server/src/html/forum/feed.rs\nindex 1b4ae7baa3ad4757b74172d7b67f3f2b33d1075d..945bdd6c48bc164e2cf91fd0996c4321b75f5abf 100644\n--- a/server/src/html/forum/feed.rs\n+++ b/server/src/html/forum/feed.rs\n@@ -14,6 +14,7 @@ use crate::timeago;\n \n use super::ingest::ingest_entry_markup;\n use super::nav::ThreadNav;\n+use super::new_thread::{fragment_new_thread_slot, login_to_post_hint_markup};\n use super::page::auth_strip;\n use super::paginator::{render_thread_paginator, PAGE_SIZE};\n use crate::html::{\n@@ -217,9 +218,6 @@ pub async fn home(\n let strip = auth_strip(&headers, &jar, &reduced_read);\n drop(reduced_read);\n \n- use crate::html::ui_action::{HtmlUiAction, UI_RPC_FIELD};\n- use crate::form_template::template_json_compact;\n-\n let page = layout(\n \"slug.social\",\n \"view-thread\",\n@@ -243,15 +241,13 @@ pub async fn home(\n }\n }\n p class=\"muted\" { \"dark = time-ordered · light = vote-ranked\" }\n- div class=\"thread-feed-toolbar\" {\n- form method=\"POST\" action=\"/ui\" {\n- input type=\"hidden\" name=(UI_RPC_FIELD) value=(template_json_compact(&HtmlUiAction::ExpandNewThreadForm {\n- room_wire: \"public\".into(),\n- }).expect(\"static json\"));\n- button type=\"submit\" class=\"section-add-btn\" { \"+\" }\n+ div id=\"new-thread-ui-slot\" {\n+ @if user.is_some() {\n+ (fragment_new_thread_slot(&nav, true, false))\n+ } @else {\n+ (login_to_post_hint_markup())\n }\n }\n- div id=\"new-thread-ui-slot\" {}\n (render_thread_feed(Some(&nav), \"thread-feed\", &public_rows, now))\n (cli_panel(&[\"npx slugsocial public forum list\"]))\n },\ndiff --git a/server/src/html/ui_action.rs b/server/src/html/ui_action.rs\nindex 5031ebfeb928f28e471f23210b8c644654adb7c7..da0c9b3541e8e4988a78768cddef22324755c3f2 100644\n--- a/server/src/html/ui_action.rs\n+++ b/server/src/html/ui_action.rs\n@@ -38,11 +38,6 @@ pub enum HtmlUiAction {\n RedactPost {\n post_id: String,\n },\n- /// Morph `#new-thread-ui-slot` inner to the collapsed compose toggle (or login hint).\n- /// Use `room_wire: \"public\"` for the public forum home; otherwise a private room id (`short/slug`).\n- ExpandNewThreadForm {\n- room_wire: String,\n- },\n /// Morph `#room-members-section` — members list open or collapsed (server-rendered).\n SetRoomMembersExpanded {\n room_wire: String,\n@@ -131,26 +126,6 @@ mod tests {\n );\n }\n \n- #[test]\n- fn expand_new_thread_form_public() {\n- let template = serde_json::json!({\n- \"action\": \"expand_new_thread_form\",\n- \"room_wire\": \"public\",\n- });\n- let mut form = HashMap::new();\n- form.insert(\n- UI_RPC_FIELD.to_string(),\n- serde_json::to_string(&template).unwrap(),\n- );\n- let a = parse_html_ui_from_form(&form).unwrap();\n- assert_eq!(\n- a,\n- HtmlUiAction::ExpandNewThreadForm {\n- room_wire: \"public\".into(),\n- }\n- );\n- }\n-\n #[test]\n fn expand_post_full_round_trip() {\n let template = serde_json::json!({\n\n\nSide B — contributor: tommy-mor\nSide B — commit message:\n[65bce99f] sibling groups #130\n\nSide B — unified diff (full patch):\ndiff --git a/server/src/html/garden.rs b/server/src/html/garden.rs\nindex 82c1b4b4f36ea13a906035a1789ba893222b3695..e3c56d124ab7b0fa08c7f9ae736103e6b7cf5862 100644\n--- a/server/src/html/garden.rs\n+++ b/server/src/html/garden.rs\n@@ -837,7 +837,8 @@ struct SiblingNavGroup {\n links: Vec,\n }\n \n-/// Siblings under the same parent, grouped like child rankings (components then isolates).\n+/// Siblings under the same parent: one group per ranking component (ordered list), then one\n+/// group per isolated unranked sibling (each shows rank `1`, separated like components).\n #[derive(Debug, Clone)]\n struct SiblingNavBar {\n groups: Vec,\n@@ -890,15 +891,12 @@ fn build_sibling_nav(\n groups.push(SiblingNavGroup { links });\n }\n }\n- if !rankings.unranked_items.is_empty() {\n- let links: Vec = rankings\n- .unranked_items\n- .iter()\n- .map(|u| SiblingNavLink {\n+ for u in &rankings.unranked_items {\n+ groups.push(SiblingNavGroup {\n+ links: vec![SiblingNavLink {\n path: u.clone().normalized_storage().to_storage_string(),\n- })\n- .collect();\n- groups.push(SiblingNavGroup { links });\n+ }],\n+ });\n }\n let sibling_total: usize = groups.iter().map(|g| g.links.len()).sum();\n if sibling_total <= 1 {\n@@ -1541,6 +1539,29 @@ mod tests {\n assert_eq!(nav.groups[1].links.len(), 1);\n }\n \n+ #[test]\n+ fn sibling_nav_splits_each_unranked_into_its_own_group() {\n+ let mut reduced = ReducerState::default();\n+ apply_ingest(\n+ &mut reduced,\n+ 1,\n+ \"@00000000-0000-0000-0000-000000000000:test:local/test\\n\\\n+ ~/topic {topic body}\\n\\\n+ ~/topic/a {alpha}\\n\\\n+ ~/topic/b {beta}\\n\\\n+ ~/topic/c {gamma}\\n\\\n+ ~/topic/d {delta}\\n\\\n+ {a beats b}\\n ~/topic/a 2:1 ~/topic/b\\n\",\n+ );\n+\n+ let model = build_item_page_view_model(&reduced, &ScopeId::Public, \"~/topic/a\");\n+ let nav = model.sibling_nav.expect(\"expected sibling nav\");\n+ assert_eq!(nav.groups.len(), 3);\n+ assert_eq!(nav.groups[0].links.len(), 2);\n+ assert_eq!(nav.groups[1].links.len(), 1);\n+ assert_eq!(nav.groups[2].links.len(), 1);\n+ }\n+\n #[test]\n fn item_page_model_builds_ranked_child_components() {\n let mut reduced = ReducerState::default();\n","role":"user"}],"model":"~x-ai/grok-latest"}