{"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[09842c93] remove shell\n\nSide A — unified diff (full patch):\ndiff --git a/server/src/html/garden.rs b/server/src/html/garden.rs\nindex 8fc6be1de8dd975f9547de615809222236be4b70..76ba96e45c9e16291a6ccd8096fdd01b274c1560 100644\n--- a/server/src/html/garden.rs\n+++ b/server/src/html/garden.rs\n@@ -1321,55 +1321,53 @@ async fn vote_compare_inner(\n .expect(\"vote compare rpc json\");\n \n let body = html! {\n- section class=\"vote-compare-shell\" {\n- h2 { \"compare\" }\n- div class=\"vote-compare-pair\" {\n- a class=\"vote-compare-item\" href=(nav.garden_item_href(&left)) {\n- code { (item_display_path(left.as_str())) }\n- }\n- span class=\"vote-compare-vs\" { \"vs\" }\n- a class=\"vote-compare-item\" href=(nav.garden_item_href(&right)) {\n- code { (item_display_path(right.as_str())) }\n- }\n- }\n- div id=\"vote-edge-history-region\" {\n- (edge_history)\n- }\n- @if can_post {\n- form id=\"vote-compare-form\" method=\"POST\" action=\"/ui\" {\n- input type=\"hidden\" name=(UI_RPC_FIELD) value=(rpc_json);\n- div class=\"vote-thread-picker\" {\n- label class=\"vote-thread-picker-label\" { \"thread\" }\n- select id=\"vote-thread-select\" name=\"thread_tag\" aria-label=\"Thread to post vote into\" {\n- @if thread_tags.is_empty() {\n- option value=\"vote\" selected { \"#vote\" }\n- }\n- @for t in &thread_tags {\n- @if *t == auto_thread {\n- option value=(t) selected { \"#\" (t) }\n- } @else {\n- option value=(t) { \"#\" (t) }\n- }\n- }\n- }\n+ h2 { \"compare\" }\n+ div class=\"vote-compare-pair\" {\n+ a class=\"vote-compare-item\" href=(nav.garden_item_href(&left)) {\n+ code { (item_display_path(left.as_str())) }\n+ }\n+ span class=\"vote-compare-vs\" { \"vs\" }\n+ a class=\"vote-compare-item\" href=(nav.garden_item_href(&right)) {\n+ code { (item_display_path(right.as_str())) }\n+ }\n+ }\n+ div id=\"vote-edge-history-region\" {\n+ (edge_history)\n+ }\n+ @if can_post {\n+ form id=\"vote-compare-form\" method=\"POST\" action=\"/ui\" {\n+ input type=\"hidden\" name=(UI_RPC_FIELD) value=(rpc_json);\n+ div class=\"vote-thread-picker\" {\n+ label class=\"vote-thread-picker-label\" { \"thread\" }\n+ select id=\"vote-thread-select\" name=\"thread_tag\" aria-label=\"Thread to post vote into\" {\n+ @if thread_tags.is_empty() {\n+ option value=\"vote\" selected { \"#vote\" }\n }\n- input type=\"hidden\" name=\"ratio_left\" id=\"vote-ratio-left\" value=\"50\";\n- input type=\"hidden\" name=\"ratio_right\" id=\"vote-ratio-right\" value=\"50\";\n- label class=\"vote-compare-slider-label\" {\n- span id=\"vote-slider-left-label\" { (item_display_path(left.as_str())) }\n- input type=\"range\" id=\"vote-preference-slider\" min=\"0\" max=\"100\" value=\"50\"\n- aria-valuemin=\"0\" aria-valuemax=\"100\";\n- span id=\"vote-slider-right-label\" { (item_display_path(right.as_str())) }\n+ @for t in &thread_tags {\n+ @if *t == auto_thread {\n+ option value=(t) selected { \"#\" (t) }\n+ } @else {\n+ option value=(t) { \"#\" (t) }\n+ }\n }\n- label class=\"vote-explain-label\" { \"reason (required)\" }\n- textarea name=\"explanation\" id=\"vote-explain\" rows=\"5\" placeholder=\"why this split?\" required {}\n- div id=\"vote-compare-errors\" {}\n- p { button type=\"submit\" { \"post vote\" } }\n }\n- } @else {\n- p class=\"muted\" { a href=\"/login\" { \"log in\" } \" to post this vote.\" }\n }\n+ input type=\"hidden\" name=\"ratio_left\" id=\"vote-ratio-left\" value=\"50\";\n+ input type=\"hidden\" name=\"ratio_right\" id=\"vote-ratio-right\" value=\"50\";\n+ label class=\"vote-compare-slider-label\" {\n+ span id=\"vote-slider-left-label\" { (item_display_path(left.as_str())) }\n+ input type=\"range\" id=\"vote-preference-slider\" min=\"0\" max=\"100\" value=\"50\"\n+ aria-valuemin=\"0\" aria-valuemax=\"100\";\n+ span id=\"vote-slider-right-label\" { (item_display_path(right.as_str())) }\n+ }\n+ label class=\"vote-explain-label\" { \"reason (required)\" }\n+ textarea name=\"explanation\" id=\"vote-explain\" rows=\"5\" placeholder=\"why this split?\" required {}\n+ div id=\"vote-compare-errors\" {}\n+ p { button type=\"submit\" { \"post vote\" } }\n }\n+ } @else {\n+ p class=\"muted\" { a href=\"/login\" { \"log in\" } \" to post this vote.\" }\n+ }\n };\n \n let page = layout_full_bleed_chromeless(\ndiff --git a/server/static/theme_default.css b/server/static/theme_default.css\nindex a441c4f79d8cf88f5a8240f9992f47dbbd1ab46b..fdcde86c718eb53cce8273e45a9844c2d8041f88 100644\n--- a/server/static/theme_default.css\n+++ b/server/static/theme_default.css\n@@ -1270,8 +1270,12 @@ body.view-ontology-dark .ont-ranking-list li {\n body.view-ontology-dark .ont-ranking-list li::before {\n color: var(--meta);\n content: counter(ont-rank) \".\";\n- font-size: 11px;\n- min-width: 18px;\n+ flex-shrink: 0;\n+ font-size: 1.35rem;\n+ font-weight: 700;\n+ font-variant-numeric: tabular-nums;\n+ line-height: 1;\n+ min-width: 2.25ch;\n text-align: right;\n }\n body.view-ontology-dark .ont-rank-score {\n@@ -1424,8 +1428,12 @@ body.view-ontology-light .ont-ranking-list li {\n body.view-ontology-light .ont-ranking-list li::before {\n color: var(--meta);\n content: counter(ont-rank) \".\";\n- font-size: 11px;\n- min-width: 18px;\n+ flex-shrink: 0;\n+ font-size: 1.35rem;\n+ font-weight: 700;\n+ font-variant-numeric: tabular-nums;\n+ line-height: 1;\n+ min-width: 2.25ch;\n text-align: right;\n }\n body.view-ontology-light .ont-rank-score {\ndiff --git a/server/static/theme_retro.css b/server/static/theme_retro.css\nindex 373f15bbd75b62604cdd14b9f0fadda2d6176991..61e1448b2f66a075c0e33325d6980448712fc927 100644\n--- a/server/static/theme_retro.css\n+++ b/server/static/theme_retro.css\n@@ -135,6 +135,30 @@ body.view-ontology nav.breadcrumb a:hover {\n body.view-ontology nav.breadcrumb a.bc-current { color: #111; font-weight: 600; }\n body.view-ontology nav.breadcrumb .bc-sep { color: #888; padding: 0 2px; }\n \n+body.view-ontology ol.ont-ranking-list {\n+ counter-reset: ont-rank;\n+ list-style: none;\n+ margin: 0.5rem 0;\n+ padding: 0;\n+}\n+body.view-ontology ol.ont-ranking-list li {\n+ align-items: baseline;\n+ counter-increment: ont-rank;\n+ display: flex;\n+ gap: 0.35rem;\n+}\n+body.view-ontology ol.ont-ranking-list li::before {\n+ flex-shrink: 0;\n+ color: #666;\n+ content: counter(ont-rank) \".\";\n+ font-size: 1.35rem;\n+ font-weight: 700;\n+ font-variant-numeric: tabular-nums;\n+ line-height: 1;\n+ min-width: 2.25ch;\n+ text-align: right;\n+}\n+\n nav.breadcrumb.ont-sibling-nav {\n margin-top: 0;\n width: 100%;\ndiff --git a/server/static/theme_retro_craft.css b/server/static/theme_retro_craft.css\nindex 6eb9222184a8795d67a5d09d41de08c8ac1b148f..7da102040484c887833158a37c307d078205c701 100644\n--- a/server/static/theme_retro_craft.css\n+++ b/server/static/theme_retro_craft.css\n@@ -742,12 +742,32 @@ body.view-ontology button.ont-garden-pin-ico:focus-visible {\n outline-offset: 2px;\n }\n \n+body.view-ontology ol.ont-ranking-list {\n+ counter-reset: ont-rank;\n+ list-style: none;\n+ margin: 0;\n+ padding: 0;\n+}\n body.view-ontology ol.ont-ranking-list li,\n body.view-ontology ul.ont-group-list li {\n display: flex;\n align-items: baseline;\n gap: 0.35rem;\n }\n+body.view-ontology ol.ont-ranking-list li {\n+ counter-increment: ont-rank;\n+}\n+body.view-ontology ol.ont-ranking-list li::before {\n+ flex-shrink: 0;\n+ color: #5c574e;\n+ content: counter(ont-rank) \".\";\n+ font-size: 1.35rem;\n+ font-weight: 700;\n+ font-variant-numeric: tabular-nums;\n+ line-height: 1;\n+ min-width: 2.25ch;\n+ text-align: right;\n+}\n body.view-ontology ol.ont-ranking-list li .item-link,\n body.view-ontology ul.ont-group-list li .item-link {\n flex: 1;\n\n\nSide B — contributor: tommy-mor\nSide B — commit message:\n[10c9caac] Fix all workspace clippy warnings.\n\nWire up a missing ui_action test, allow dead code in shared integration helpers, and apply small clippy cleanups across server and types.\n\nCo-authored-by: Cursor \n\nSide B — unified diff (full patch):\ndiff --git a/server/src/dsl.rs b/server/src/dsl.rs\nindex a45c5b33e10d0d2ea48c7313061cfa1e6430bcfc..faa8aac6616bc6ea2b102d08ae999c01a716ef6e 100644\n--- a/server/src/dsl.rs\n+++ b/server/src/dsl.rs\n@@ -927,9 +927,7 @@ mod tests {\n #[test]\n fn parse_vote_rejects_zero_zero_ratio() {\n let err = parse_full(\"{tie placeholder}\\n~/a 0:0 ~/b\").unwrap_err();\n- let msg = match err {\n- DslError::Parse(m) => m,\n- };\n+ let DslError::Parse(msg) = err;\n assert!(\n msg.contains(\"0:0\"),\n \"expected 0:0 rejection message, got: {msg}\"\ndiff --git a/server/src/html/garden/tests.rs b/server/src/html/garden/tests.rs\nindex c2036fca7752aef63d260e36cfe9649f63b5c550..0a1be1290fdc93197ca191197a473840bb4decbc 100644\n--- a/server/src/html/garden/tests.rs\n+++ b/server/src/html/garden/tests.rs\n@@ -346,7 +346,7 @@ fn vote_compare_item_card_renders_github_import_markup() {\n \"headline\": \"#1 Compare card\",\n \"sublines\": [\"State: open\"],\n });\n- let body = format!(\"```slug-github-card\\n{}\\n```\", json.to_string());\n+ let body = format!(\"```slug-github-card\\n{json}\\n```\");\n let html = vote_compare_item_card(\n &nav,\n &item,\ndiff --git a/server/src/html/ui_action.rs b/server/src/html/ui_action.rs\nindex 2589a2cc00bb7b18cd19ebcbb938083122d6921d..5e4131dd346a6f80bfe091a9b0cf7902721bc47f 100644\n--- a/server/src/html/ui_action.rs\n+++ b/server/src/html/ui_action.rs\n@@ -246,6 +246,7 @@ mod tests {\n );\n }\n \n+ #[test]\n fn set_new_thread_compose_expanded_true() {\n let template = serde_json::json!({\n \"action\": \"set_new_thread_compose_expanded\",\ndiff --git a/server/src/offline.rs b/server/src/offline.rs\nindex 2db6f67e22e00a24ce673d13095644dd9fa9342d..93d4d5ee55449da644f732bc0ba007ee4c2c7079 100644\n--- a/server/src/offline.rs\n+++ b/server/src/offline.rs\n@@ -167,7 +167,7 @@ fn rankings_for_simulated(\n .iter()\n .map(|parent| {\n let scoped_content = simulated\n- .content_for_scope(&scope)\n+ .content_for_scope(scope)\n .unwrap_or_else(|| simulated.public());\n let scoped = build_children_rankings(scoped_content, parent);\n let components: Vec = scoped\n@@ -254,7 +254,9 @@ fn ingest_parse_error(raw: &str) -> Option {\n dsl::parse_full(raw).err().map(|e| e.to_string())\n }\n \n-fn load_events_from_jsonl(path: &Path) -> Result<(usize, Vec<(usize, Event)>, Vec), std::io::Error> {\n+type JsonlEventsLoad = Result<(usize, Vec<(usize, Event)>, Vec), std::io::Error>;\n+\n+fn load_events_from_jsonl(path: &Path) -> JsonlEventsLoad {\n let text = std::fs::read_to_string(path)?;\n let total_lines = text.lines().count();\n let mut events = Vec::new();\ndiff --git a/server/src/resolvers/github.rs b/server/src/resolvers/github.rs\nindex 30dd4cdac96de0e3da4fa03fdf82f91bed73bfe0..3cec5e8b7249a4597c378890cc9e6125104e76c3 100644\n--- a/server/src/resolvers/github.rs\n+++ b/server/src/resolvers/github.rs\n@@ -749,6 +749,6 @@ mod tests {\n GithubImportKind::Issue,\n );\n assert!(card.sublines.iter().any(|l| l.contains(\"@octo\")));\n- assert_eq!(card.excerpt.as_deref(), Some(\"The issue body.\").as_deref());\n+ assert_eq!(card.excerpt.as_deref(), Some(\"The issue body.\"));\n }\n }\ndiff --git a/server/tests/basic.rs b/server/tests/basic.rs\nindex 31a35251a8abd6f4a48c1d7782b6985f621375e1..9d83e7a97e2c7494790db17c4b5b30c181705026 100644\n--- a/server/tests/basic.rs\n+++ b/server/tests/basic.rs\n@@ -337,7 +337,7 @@ async fn event_log_handles_corrupt_lines() {\n .unwrap();\n \n // Add empty line.\n- writeln!(f, \"\").unwrap();\n+ writeln!(f).unwrap();\n \n let (loaded, bad) = log.load_all().await.unwrap();\n assert_eq!(loaded.len(), 2);\n@@ -511,9 +511,7 @@ fn dsl_parse_rejects_zero_zero_vote_ratio() {\n \"~/t/a {a}\\n~/t/b {b}\\n{zero}\\n~/t/a 0:0 ~/t/b\\n\",\n )\n .expect_err(\"0:0 vote must be rejected by the parser\");\n- let msg = match err {\n- slugsocial_server::dsl::DslError::Parse(m) => m,\n- };\n+ let slugsocial_server::dsl::DslError::Parse(msg) = err;\n assert!(\n msg.contains(\"0:0\"),\n \"expected message about invalid 0:0 ratio, got: {msg}\"\n@@ -904,7 +902,7 @@ fn posts_by_actor_indexes_and_profile_visibility() {\n fn feed_query(state: &ReducerState, cutoff: i64, limit: usize) -> (usize, Vec) {\n let matching: Vec<&str> = state.ingests_ordered.iter().rev()\n .map(|id| id.as_str())\n- .take_while(|id| state.ingests_by_id.get(*id).map_or(false, |ing| ing.ts > cutoff))\n+ .take_while(|id| state.ingests_by_id.get(*id).is_some_and(|ing| ing.ts > cutoff))\n .filter(|id| {\n state.ingests_by_id.get(*id).is_some_and(|ing| {\n let scope = slugsocial_server::reducer::scope_from_room_wire(&ing.room_id);\ndiff --git a/server/tests/integration_health.rs b/server/tests/integration_health.rs\nindex 481222d8c48c44fcfb7e9ba26cf7644b5c26d5f4..351aae6b0e9738021886a076ee08fab5cf5a0001 100644\n--- a/server/tests/integration_health.rs\n+++ b/server/tests/integration_health.rs\n@@ -7,7 +7,7 @@ async fn test_healthz() {\n let (addr, _tmp, _log, _handle) = create_test_server().await;\n let client = reqwest::Client::new();\n let response = client\n- .get(&format!(\"http://{}/healthz\", addr))\n+ .get(format!(\"http://{}/healthz\", addr))\n .send()\n .await\n .unwrap();\ndiff --git a/server/tests/integration_rpc.rs b/server/tests/integration_rpc.rs\nindex ccd89a02594bd2a8e7047edafca04b0e54384139..ec446d94a38c8d6961d2135dd031da7ab2a39b48 100644\n--- a/server/tests/integration_rpc.rs\n+++ b/server/tests/integration_rpc.rs\n@@ -440,7 +440,6 @@ async fn test_rank_history() {\n let bearer = test_bearer();\n let ingest = |delegate: &str, text: &str| {\n let client = client.clone();\n- let addr = addr;\n let bearer = bearer.clone();\n let text = text.to_string();\n let delegate = delegate.to_string();\ndiff --git a/server/tests/support/mod.rs b/server/tests/support/mod.rs\nindex 84dc1d68b6c1e1fdfdbd0757139c06c3daf1e9a6..4a620eaa875e7a1145f2a4e82cc4ff2be21d5345 100644\n--- a/server/tests/support/mod.rs\n+++ b/server/tests/support/mod.rs\n@@ -1,3 +1,6 @@\n+//! Shared helpers for integration tests; each test binary uses a different subset.\n+#![allow(dead_code)]\n+\n use sha2::{Digest, Sha256};\n use slugsocial_server::{\n event_log::EventLog,\ndiff --git a/types/src/paths.rs b/types/src/paths.rs\nindex ebc299c5e1af456c3e4b7fa45ed9fb23313e88c4..d2e799942b98dd7342764475deb17903fb60c600 100644\n--- a/types/src/paths.rs\n+++ b/types/src/paths.rs\n@@ -411,7 +411,7 @@ mod tests {\n #[test]\n fn garden_item_url_deref_to_str() {\n let g = GardenItemUrl::from_storage_str(\"https://slug.social/~/x\", \"public\");\n- let s: &str = &*g;\n+ let s: &str = &g;\n assert_eq!(s, \"https://slug.social/~/x\");\n }\n \n","role":"user"}],"model":"openai/gpt-chat-latest"}