{"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[21b57b50] fix(html): drop duplicate forum.rs; use forum/ as sole module root\n\nRemoves the leftover monolithic file so mod forum resolves to forum/mod.rs\nwithout ambiguity. Keeps feed, views, and other split modules as the source.\n\nMade-with: Cursor\n\nSide A — unified diff (full patch):\ndiff --git a/server/src/html/forum.rs b/server/src/html/forum.rs\ndeleted file mode 100644\nindex 5ad8dfc84dd735d589432e2c613ff687a75f2e61..0000000000000000000000000000000000000000\n--- a/server/src/html/forum.rs\n+++ /dev/null\n@@ -1,1405 +0,0 @@\n-use axum::{\n- extract::{Path, Query, State},\n- http::{HeaderMap, StatusCode, Uri},\n- response::{Html, IntoResponse},\n-};\n-use axum_extra::extract::cookie::CookieJar;\n-use maud::{html, Markup};\n-use serde::Deserialize;\n-\n-use crate::{\n- api::optional_principal,\n- canonical_path::{canonicalize_item, canonicalize_tag},\n- events::ThreadCapability,\n- form_template::template_json_compact,\n- identity::parse_username,\n- reducer::{scope_from_room_wire, ReducerState, ScopeId},\n- state::AppState,\n- timeago,\n-};\n-use serde_json::json;\n-\n-use super::js_string_literal;\n-use super::ui_action::{HtmlUiAction, UI_RPC_FIELD};\n-\n-use super::{\n- bc_segment, bc_threads, cli_panel, layout, now_ms, profile_href, recency_class,\n- render_linkified_with_embeds_in_scope, theme_from_jar, theme_next_from_uri, JsBuilder,\n-};\n-\n-#[derive(Clone)]\n-struct ThreadRow {\n- tag: String,\n- subtitle: Option,\n- last_ts: i64,\n- ingests: usize,\n-}\n-\n-#[derive(Clone)]\n-struct RoomMemberRow {\n- username: String,\n- capabilities: Vec<&'static str>,\n-}\n-\n-/// URL helpers for public `/t/…` and private room threads `/r/{short}/{slug}/t/…`.\n-#[derive(Clone)]\n-pub struct ThreadNav {\n- pub room_wire: String,\n- scope: ScopeId,\n- room_path: String,\n- thread_path_prefix: String,\n- garden_path_prefix: String,\n-}\n-\n-impl ThreadNav {\n- pub(crate) fn public() -> Self {\n- Self {\n- room_wire: \"public\".into(),\n- scope: ScopeId::Public,\n- room_path: \"/t\".into(),\n- thread_path_prefix: \"/t\".into(),\n- garden_path_prefix: \"/~\".into(),\n- }\n- }\n-\n- /// `room_id` wire form `shortid/slug`.\n- pub(crate) fn from_room_id(room_id: &str) -> Option {\n- let (short, slug) = room_id.split_once('/')?;\n- if short.is_empty() || slug.is_empty() {\n- return None;\n- }\n- Some(Self {\n- room_wire: room_id.to_string(),\n- scope: ScopeId::Room(room_id.to_string()),\n- room_path: format!(\"/r/{short}/{slug}\"),\n- thread_path_prefix: format!(\"/r/{short}/{slug}/t\"),\n- garden_path_prefix: format!(\"/r/{short}/{slug}/~\"),\n- })\n- }\n-\n- pub(crate) fn scope(&self) -> ScopeId {\n- self.scope.clone()\n- }\n-\n- pub(crate) fn room_url(&self) -> &str {\n- &self.room_path\n- }\n-\n- pub(crate) fn thread_url(&self, tag: &str) -> String {\n- format!(\"{}/{}\", self.thread_path_prefix, tag)\n- }\n-\n- pub(crate) fn garden_root_url(&self) -> &str {\n- &self.garden_path_prefix\n- }\n-\n- pub(crate) fn garden_item_url(&self, item: &str) -> String {\n- if let Some(tail) = crate::path_types::CanonicalItemUrl::parse(item)\n- .and_then(|c| c.tilde_tail().map(str::to_owned))\n- {\n- format!(\"{}/{}\", self.garden_path_prefix, tail)\n- } else {\n- format!(\"{}/{}\", self.garden_path_prefix, canonicalize_item(item))\n- }\n- }\n-\n- fn thread_page_url(&self, tag: &str, offset: usize) -> String {\n- let base = self.thread_url(tag);\n- if offset == 0 {\n- base\n- } else {\n- format!(\"{base}?offset={offset}\")\n- }\n- }\n-\n- fn post_url(&self, tag: &str, idx: usize) -> String {\n- format!(\"{}/{}/{}\", self.thread_path_prefix, tag, idx)\n- }\n-}\n-\n-/// `POST /ui` + `__rpc__` from an inline link (`onclick`); same-origin credentials as other morph actions.\n-fn thread_ui_fetch_onclick(rpc_compact_json: &str) -> String {\n- format!(\n- \"fetch('/ui',{{method:'POST',headers:{{'Content-Type':'application/x-www-form-urlencoded'}},body:new URLSearchParams({{__rpc__:{}}}).toString(),credentials:'same-origin'}}).then(r=>r.text()).then(eval);return false\",\n- js_string_literal(rpc_compact_json)\n- )\n-}\n-\n-fn thread_nav_for_ingest(ing: &crate::events::Ingest) -> Option {\n- let room = ing.room_id.trim();\n- if room.is_empty() || room == \"public\" {\n- Some(ThreadNav::public())\n- } else {\n- ThreadNav::from_room_id(room)\n- }\n-}\n-\n-fn thread_post_index_in_scope(reduced: &ReducerState, ing: &crate::events::Ingest) -> Option {\n- let scope = scope_from_room_wire(&ing.room_id);\n- let tag = canonicalize_tag(&ing.thread_tag);\n- reduced\n- .ingests_by_scope_thread\n- .get(&(scope, tag))\n- .and_then(|q| q.iter().rev().position(|id| id == &ing.id))\n-}\n-\n-fn post_header_meta(\n- nav: &ThreadNav,\n- tag: &str,\n- post_idx: usize,\n- principal: &str,\n- ts: i64,\n- now: i64,\n-) -> Markup {\n- let post_href = nav.post_url(tag, post_idx);\n- let profile = profile_href(principal);\n- let hover = timeago::rfc3339_utc(ts);\n- let ago = timeago::timeago(now, ts);\n- html! {\n- div class=\"ingest-meta muted\" title=(hover) {\n- a href=(post_href) class=\"post-num\" { \"#\" (post_idx) }\n- \" \"\n- a href=(profile) class=\"post-author\" { \"@\" (principal) }\n- \" · \"\n- (ago)\n- }\n- }\n-}\n-\n-fn post_header_row(\n- nav: &ThreadNav,\n- tag: &str,\n- post_idx: usize,\n- ing: &crate::events::Ingest,\n- _viewer: Option<&str>,\n- now: i64,\n- show_delete: bool,\n-) -> Markup {\n- let meta = post_header_meta(nav, tag, post_idx, &ing.principal, ing.ts, now);\n- html! {\n- div class=\"ingest-header-row\" {\n- (meta)\n- @if show_delete {\n- form class=\"post-delete-form\" method=\"POST\" action=\"/ui\" {\n- input type=\"hidden\" name=(UI_RPC_FIELD) value=(template_json_compact(&HtmlUiAction::RedactPost { post_id: ing.id.clone() }).unwrap());\n- button type=\"submit\" class=\"post-delete-btn\" { \"delete\" }\n- }\n- }\n- }\n- }\n-}\n-\n-fn redacted_header_row(\n- nav: &ThreadNav,\n- tag: &str,\n- post_idx: usize,\n- ing: &crate::events::Ingest,\n- now: i64,\n- expanded: bool,\n-) -> Markup {\n- let meta = post_header_meta(nav, tag, post_idx, &ing.principal, ing.ts, now);\n- let rpc_expand = template_json_compact(&json!({\n- \"action\": \"expand_redacted_post\",\n- \"room\": nav.room_wire,\n- \"thread_tag\": tag,\n- \"post_index\": post_idx,\n- }))\n- .unwrap();\n- let rpc_collapse = template_json_compact(&json!({\n- \"action\": \"collapse_redacted_post\",\n- \"room\": nav.room_wire,\n- \"thread_tag\": tag,\n- \"post_index\": post_idx,\n- }))\n- .unwrap();\n- let onclick_expand = thread_ui_fetch_onclick(&rpc_expand);\n- let onclick_collapse = thread_ui_fetch_onclick(&rpc_collapse);\n- html! {\n- div class=\"ingest-header-row ingest-tombstone-row\" {\n- (meta)\n- span class=\"post-tombstone-inline muted\" {\n- \"deleted · \"\n- @if expanded {\n- a href=\"#\" class=\"hide-deleted-link\"\n- onclick=(onclick_collapse) {\n- \"[hide deleted content]\"\n- }\n- } @else {\n- a href=\"#\" class=\"show-deleted-link\"\n- onclick=(onclick_expand) {\n- \"[show deleted content]\"\n- }\n- }\n- }\n- }\n- }\n-}\n-\n-fn ingest_entry_markup(\n- nav: &ThreadNav,\n- tag: &str,\n- post_idx: usize,\n- ing: &crate::events::Ingest,\n- viewer: Option<&str>,\n- now: i64,\n- reduced: &ReducerState,\n-) -> Markup {\n- let redacted = reduced.redacted_posts.contains(&ing.id);\n- let show_delete = viewer == Some(ing.principal.as_str()) && !redacted;\n- if redacted {\n- html! {\n- div class=\"ingest-entry ingest-redacted\" data-ingest-id=(ing.id) {\n- (redacted_header_row(nav, tag, post_idx, ing, now, false))\n- }\n- }\n- } else {\n- let truncated = ing.raw.len() > 2000;\n- let display_body = if truncated { &ing.raw[..2000] } else { &ing.raw[..] };\n- html! {\n- div class=\"ingest-entry\" data-ingest-id=(ing.id) {\n- (post_header_row(nav, tag, post_idx, ing, viewer, now, show_delete))\n- (render_linkified_with_embeds_in_scope(display_body, nav.garden_root_url()))\n- @if truncated {\n- @let rpc_full = template_json_compact(&json!({\n- \"action\": \"expand_post_full\",\n- \"room\": nav.room_wire,\n- \"thread_tag\": tag,\n- \"post_index\": post_idx,\n- })).unwrap();\n- @let onclick_full = thread_ui_fetch_onclick(&rpc_full);\n- a href=\"#\" class=\"show-full-link\"\n- onclick=(onclick_full) {\n- \"[show full post]\"\n- }\n- }\n- }\n- }\n- }\n-}\n-\n-fn collect_thread_rows_for_scope(reduced: &ReducerState, scope: &ScopeId, now: i64) -> Vec {\n- let _ = now;\n- reduced\n- .forum_threads\n- .iter()\n- .filter(|((s, _), _)| s == scope)\n- .map(|((_, tag), thread)| {\n- let ingests = reduced\n- .ingests_by_scope_thread\n- .get(&(scope.clone(), tag.clone()))\n- .map(|q| q.len())\n- .unwrap_or(0);\n- ThreadRow {\n- tag: tag.clone(),\n- subtitle: None,\n- last_ts: thread.last_activity_ts,\n- ingests,\n- }\n- })\n- .collect()\n-}\n-\n-fn rooms_for_user(reduced: &ReducerState, username: &str) -> Vec {\n- let mut v: Vec = reduced\n- .grants\n- .iter()\n- .filter(|(rid, m)| reduced.rooms.contains(*rid) && m.contains_key(username))\n- .map(|(rid, _)| rid.clone())\n- .collect();\n- v.sort();\n- v\n-}\n-\n-pub(crate) fn user_can_view_room(reduced: &ReducerState, room_id: &str, username: Option<&str>) -> bool {\n- if !reduced.rooms.contains(room_id) {\n- return false;\n- }\n- let Some(u) = username else {\n- return false;\n- };\n- reduced.user_has_cap(room_id, u, ThreadCapability::View)\n-}\n-\n-pub(crate) fn user_can_post_room(reduced: &ReducerState, room_id: &str, username: &str) -> bool {\n- reduced.user_has_cap(room_id, username, ThreadCapability::Post)\n-}\n-\n-fn capability_label(cap: ThreadCapability) -> &'static str {\n- match cap {\n- ThreadCapability::View => \"view\",\n- ThreadCapability::Post => \"post\",\n- ThreadCapability::Vote => \"vote\",\n- ThreadCapability::AddItem => \"add_item\",\n- ThreadCapability::Manage => \"manage\",\n- }\n-}\n-\n-fn room_members_for_room(reduced: &ReducerState, room_id: &str) -> Vec {\n- let mut rows: Vec = reduced\n- .grants\n- .get(room_id)\n- .into_iter()\n- .flat_map(|members| members.iter())\n- .map(|(username, caps)| {\n- let mut ordered = Vec::new();\n- for cap in [\n- ThreadCapability::View,\n- ThreadCapability::Post,\n- ThreadCapability::Vote,\n- ThreadCapability::AddItem,\n- ThreadCapability::Manage,\n- ] {\n- if caps.contains(&cap) {\n- ordered.push(capability_label(cap));\n- }\n- }\n- RoomMemberRow {\n- username: username.clone(),\n- capabilities: ordered,\n- }\n- })\n- .collect();\n- rows.sort_by(|a, b| a.username.cmp(&b.username));\n- rows\n-}\n-\n-fn room_members_inner(members: &[RoomMemberRow]) -> Markup {\n- html! {\n- h3 { \"members\" }\n- ul class=\"room-members\" {\n- @for member in members {\n- li {\n- span class=\"room-member-name\" { \"@\" (member.username) }\n- span class=\"muted\" {\n- \" · \"\n- (member.capabilities.join(\", \"))\n- }\n- }\n- }\n- }\n- }\n-}\n-\n-/// Fragment for `#room-members-section` — expand/collapse is server-driven via `POST /ui`.\n-pub(crate) fn room_members_section_markup(\n- reduced: &ReducerState,\n- room_id: &str,\n- members_expanded: bool,\n-) -> Markup {\n- let members = room_members_for_room(reduced, room_id);\n- if members.is_empty() {\n- return html! {};\n- }\n- html! {\n- div id=\"room-members-section\" {\n- @if members_expanded {\n- form method=\"POST\" action=\"/ui\" {\n- input type=\"hidden\" name=(UI_RPC_FIELD) value=(template_json_compact(&HtmlUiAction::SetRoomMembersExpanded {\n- room_wire: room_id.to_string(),\n- expanded: false,\n- }).expect(\"static json\"));\n- button type=\"submit\" class=\"form-toggle\" aria-expanded=\"true\" {\n- \"hide members & permissions\"\n- }\n- }\n- section class=\"room-members-panel\" {\n- (room_members_inner(&members))\n- }\n- } @else {\n- form method=\"POST\" action=\"/ui\" {\n- input type=\"hidden\" name=(UI_RPC_FIELD) value=(template_json_compact(&HtmlUiAction::SetRoomMembersExpanded {\n- room_wire: room_id.to_string(),\n- expanded: true,\n- }).expect(\"static json\"));\n- button type=\"submit\" class=\"form-toggle\" aria-expanded=\"false\" {\n- \"members & permissions\"\n- }\n- }\n- }\n- }\n- }\n-}\n-\n-/// `feed_id` is e.g. `thread-feed` (public bump list, SSE) or `room-thread-feed`.\n-fn render_thread_feed(nav: Option<&ThreadNav>, feed_id: &str, rows: &[ThreadRow], now: i64) -> Markup {\n- html! {\n- div id=(feed_id) {\n- @if rows.is_empty() {\n- p class=\"muted\" { \"no threads yet\" }\n- } @else {\n- ul class=\"thread-feed\" {\n- @for r in rows {\n- @let thread_href = nav\n- .as_ref()\n- .map(|n| n.thread_url(&r.tag))\n- .unwrap_or_else(|| format!(\"/t/{}\", r.tag));\n- @let hover = timeago::rfc3339_utc(r.last_ts);\n- @let ago = timeago::timeago(now, r.last_ts);\n- @let age_cls = recency_class(now, r.last_ts);\n- li class=(age_cls) {\n- a href=(thread_href) {\n- \"#\" (r.tag)\n- @if let Some(subtitle) = &r.subtitle {\n- \": \" (subtitle)\n- }\n- \" \"\n- span class=\"muted\" title=(hover) {\n- (ago)\n- \" · \"\n- (format!(\"{}n\", r.ingests))\n- }\n- }\n- }\n- }\n- }\n- }\n- }\n- }\n-}\n-\n-fn auth_strip(\n- headers: &HeaderMap,\n- jar: &CookieJar,\n- reduced: &ReducerState,\n-) -> Markup {\n- match optional_principal(headers, jar, reduced) {\n- Some(u) => html! {\n- p class=\"muted auth-strip\" {\n- \"@\" (u)\n- \" · \"\n- a href=\"/logout\" { \"log out\" }\n- }\n- },\n- None => html! {\n- p class=\"muted auth-strip\" {\n- a href=\"/login\" { \"log in\" }\n- }\n- },\n- }\n-}\n-\n-fn bc_room(nav: &ThreadNav, room_slug: &str, thread_tag: Option<&str>) -> Markup {\n- html! {\n- a href=\"/\" { \"slug.social\" }\n- @if let Some(t) = thread_tag {\n- (bc_segment(\n- &format!(\"room:{room_slug}\"),\n- nav.room_url(),\n- false,\n- ))\n- (bc_segment(&format!(\"#{t}\"), &nav.thread_url(t), true))\n- } @else {\n- (bc_segment(\n- &format!(\"room:{room_slug}\"),\n- nav.room_url(),\n- true,\n- ))\n- }\n- }\n-}\n-\n-fn compose_form(nav: &ThreadNav, thread_tag: &str, show: bool) -> Markup {\n- if !show {\n- return html! {};\n- }\n- html! {\n- section class=\"compose\" id=\"thread-compose\" {\n- form id=\"thread-compose-form\" method=\"POST\" action=\"/ui\" data-check-action=\"/ui\" data-check-rpc=(template_json_compact(&json!({\n- \"action\": \"check_ingest\",\n- \"room\": nav.room_wire,\n- \"thread_tag\": thread_tag,\n- \"text\": {\"$form\": \"text\"},\n- \"error_target\": \"thread-compose-errors\",\n- \"form_id\": \"thread-compose-form\",\n- })).unwrap()) {\n- input type=\"hidden\" name=(UI_RPC_FIELD) value=(template_json_compact(&json!({\n- \"action\": \"post_ingest\",\n- \"room\": nav.room_wire,\n- \"thread_tag\": thread_tag,\n- \"text\": {\"$form\": \"text\"},\n- \"error_target\": \"thread-compose-errors\",\n- \"form_id\": \"thread-compose-form\",\n- })).unwrap());\n- textarea name=\"text\" rows=\"5\" cols=\"80\" placeholder=\"prose or ~/items and votes…\" {}\n- p {\n- button type=\"submit\" { \"post\" }\n- }\n- }\n- div id=\"thread-compose-errors\" {}\n- }\n- }\n-}\n-\n-fn new_thread_form_public(show: bool) -> Markup {\n- if !show {\n- return html! {};\n- }\n- html! {\n- section class=\"compose\" id=\"public-new-thread-compose\" {\n- div id=\"public-new-thread-errors\" {}\n- form id=\"public-new-thread-form\" method=\"POST\" action=\"/ui\" {\n- input type=\"hidden\" name=(UI_RPC_FIELD) value=(template_json_compact(&json!({\n- \"action\": \"post_ingest\",\n- \"room\": \"public\",\n- \"thread_tag\": {\"$form\": \"thread_tag\"},\n- \"text\": {\"$form\": \"text\"},\n- })).unwrap());\n- input type=\"text\" id=\"new-thread-tag\" name=\"thread_tag\" pattern=\"[a-z0-9_\\\\-]{1,64}\" placeholder=\"thread-title-slug-here\";\n- textarea id=\"new-thread-text\" name=\"text\" rows=\"4\" placeholder=\"Hello threadgoers!! Behold my new thread!\" {}\n- p { button type=\"submit\" { \"create thread / make first post\" } }\n- }\n- }\n- }\n-}\n-\n-/// Returns the current public thread feed markup for SSE (`#thread-feed`).\n-pub async fn thread_feed_html(state: &AppState) -> Markup {\n- let now = now_ms();\n- let nav = ThreadNav::public();\n- let mut rows = {\n- let reduced = state.reduced.read().await;\n- collect_thread_rows_for_scope(&reduced, &ScopeId::Public, now)\n- };\n- rows.sort_by(|a, b| b.last_ts.cmp(&a.last_ts));\n- render_thread_feed(Some(&nav), \"thread-feed\", &rows, now)\n-}\n-\n-/// Returns the current private room thread feed markup for SSE (`#room-thread-feed`).\n-pub async fn thread_feed_html_for_room(state: &AppState, room_id: &str) -> Markup {\n- let now = now_ms();\n- let Some(nav) = ThreadNav::from_room_id(room_id) else {\n- return html! { div id=\"room-thread-feed\" { p class=\"muted\" { \"room not found\" } } };\n- };\n- let mut rows = {\n- let reduced = state.reduced.read().await;\n- collect_thread_rows_for_scope(&reduced, &ScopeId::Room(room_id.to_string()), now)\n- };\n- rows.sort_by(|a, b| b.last_ts.cmp(&a.last_ts));\n- render_thread_feed(Some(&nav), \"room-thread-feed\", &rows, now)\n-}\n-\n-fn render_thread_feed_region_markup(\n- nav: &ThreadNav,\n- tag: &str,\n- display_ingests: &[crate::events::Ingest],\n- offset: usize,\n- total: usize,\n- now: i64,\n- viewer: Option<&str>,\n- reduced: &ReducerState,\n-) -> Markup {\n- let paginator_top = render_thread_paginator(nav, tag, offset, total, true);\n- let paginator_bot = render_thread_paginator(nav, tag, offset, total, false);\n- html! {\n- div id=\"thread-feed-region\" {\n- @if display_ingests.is_empty() {\n- p class=\"muted\" { \"no activity yet\" }\n- } @else {\n- (paginator_top)\n- @for (i, ing) in display_ingests.iter().enumerate() {\n- @let post_idx = offset + i;\n- (ingest_entry_markup(nav, tag, post_idx, ing, viewer, now, reduced))\n- }\n- (paginator_bot)\n- }\n- }\n- }\n-}\n-\n-pub async fn thread_feed_region_markup(\n- state: &AppState,\n- room_id: Option<&str>,\n- tag: &str,\n- viewer: Option<&str>,\n-) -> Markup {\n- let tag = canonicalize_tag(tag);\n- let Some(nav) = (match room_id {\n- Some(room_id) => ThreadNav::from_room_id(room_id),\n- None => Some(ThreadNav::public()),\n- }) else {\n- return html! { div id=\"thread-feed-region\" { p class=\"muted\" { \"thread not found\" } } };\n- };\n- let scope = nav.scope();\n- let all_ids: Vec = {\n- let reduced = state.reduced.read().await;\n- reduced\n- .ingests_by_scope_thread\n- .get(&(scope.clone(), tag.clone()))\n- .map(|q| q.iter().rev().cloned().collect())\n- .unwrap_or_default()\n- };\n- let total = all_ids.len();\n- let offset = total.saturating_sub(PAGE_SIZE);\n- let page_ids: Vec = all_ids.into_iter().skip(offset).take(PAGE_SIZE).collect();\n- let reduced = state.reduced.read().await;\n- let display_ingests = page_ids\n- .iter()\n- .filter_map(|id| reduced.ingests_by_id.get(id).cloned())\n- .collect::>();\n- render_thread_feed_region_markup(\n- &nav,\n- &tag,\n- &display_ingests,\n- offset,\n- total,\n- now_ms(),\n- viewer,\n- &reduced,\n- )\n-}\n-\n-/// Home: private rooms (signed-in), then public bump-ordered threads.\n-pub async fn home(\n- State(state): State,\n- headers: HeaderMap,\n- jar: CookieJar,\n- uri: Uri,\n-) -> impl IntoResponse {\n- let now = now_ms();\n- let reduced = state.reduced.read().await;\n- let user = optional_principal(&headers, &jar, &reduced);\n- let room_ids = user\n- .as_ref()\n- .map(|u| rooms_for_user(&reduced, u))\n- .unwrap_or_default();\n- let mut public_rows = collect_thread_rows_for_scope(&reduced, &ScopeId::Public, now);\n- drop(reduced);\n- public_rows.sort_by(|a, b| b.last_ts.cmp(&a.last_ts));\n-\n- let nav = ThreadNav::public();\n- let reduced_read = state.reduced.read().await;\n- let strip = auth_strip(&headers, &jar, &reduced_read);\n- drop(reduced_read);\n-\n- let page = layout(\n- \"slug.social\",\n- \"view-thread\",\n- html! {\n- (strip)\n- nav class=\"breadcrumb\" { (bc_threads(None)) }\n- @if !room_ids.is_empty() {\n- h2 { \"your rooms\" }\n- ul class=\"thread-feed\" {\n- @for rid in &room_ids {\n- @if let Some(nav_r) = ThreadNav::from_room_id(rid) {\n- @let slug = if let Some((_, s)) = rid.split_once('/') { s } else { rid.as_str() };\n- li {\n- a href=(nav_r.room_url()) {\n- (slug)\n- span class=\"muted\" { \" · \" (rid) }\n- }\n- }\n- }\n- }\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::ExpandPublicNewThreadForm).expect(\"static json\"));\n- button type=\"submit\" class=\"section-add-btn\" { \"+\" }\n- }\n- }\n- div id=\"public-new-thread-ui-slot\" {}\n- (render_thread_feed(Some(&nav), \"thread-feed\", &public_rows, now))\n- (cli_panel(&[\"npx slugsocial public forum list\"]))\n- },\n- None,\n- theme_from_jar(&jar),\n- &theme_next_from_uri(&uri),\n- );\n- Html(page.into_string())\n-}\n-\n-#[derive(Debug, Deserialize)]\n-pub struct ThreadViewQuery {\n- pub offset: Option,\n-}\n-\n-const PAGE_SIZE: usize = 10;\n-\n-fn render_thread_paginator(nav: &ThreadNav, tag: &str, offset: usize, total: usize, top: bool) -> Markup {\n- let newer_offset = offset.checked_add(PAGE_SIZE).filter(|&o| o < total);\n- let older_offset = if offset > 0 {\n- Some(offset.saturating_sub(PAGE_SIZE))\n- } else {\n- None\n- };\n- let latest_offset = total.saturating_sub(PAGE_SIZE);\n- let on_latest = offset >= latest_offset;\n- let (id, scroll_href, scroll_label) = if top {\n- (\"top\", \"#bottom\", \"↓\")\n- } else {\n- (\"bottom\", \"#top\", \"↑\")\n- };\n- html! {\n- div class=\"thread-paginator\" id=(id) {\n- a href=(scroll_href) class=\"post-nav-btn\" { (scroll_label) }\n- @if let Some(o) = older_offset {\n- a href=(nav.thread_page_url(tag, o)) class=\"post-nav-btn\" { \"← older\" }\n- } @else {\n- a href=\"#\" class=\"post-nav-btn disabled\" { \"← older\" }\n- }\n- span class=\"post-nav-pos muted\" {\n- (offset + 1) \"–\" (total.min(offset + PAGE_SIZE)) \" / \" (total)\n- }\n- @if let Some(o) = newer_offset {\n- a href=(nav.thread_page_url(tag, o)) class=\"post-nav-btn\" { \"newer →\" }\n- } @else {\n- a href=\"#\" class=\"post-nav-btn disabled\" { \"newer →\" }\n- }\n- @if !on_latest {\n- a href=(nav.thread_page_url(tag, latest_offset)) class=\"post-nav-btn\" { \"latest\" }\n- }\n- }\n- }\n-}\n-\n-async fn thread_view_inner(\n- state: AppState,\n- tag: String,\n- q: ThreadViewQuery,\n- nav: ThreadNav,\n- headers: HeaderMap,\n- jar: CookieJar,\n- uri: Uri,\n-) -> impl IntoResponse {\n- let tag = canonicalize_tag(&tag);\n- let scope = nav.scope();\n-\n- let all_ids: Vec = {\n- let reduced = state.reduced.read().await;\n- reduced\n- .ingests_by_scope_thread\n- .get(&(scope.clone(), tag.clone()))\n- .map(|q| q.iter().rev().cloned().collect())\n- .unwrap_or_default()\n- };\n-\n- let total = all_ids.len();\n- let offset = q.offset.unwrap_or(0);\n- let page_ids: Vec = all_ids.into_iter().skip(offset).take(PAGE_SIZE).collect();\n-\n- let (display_ingests, _subtitle) = {\n- let reduced = state.reduced.read().await;\n- let ingests = page_ids\n- .iter()\n- .filter_map(|id| reduced.ingests_by_id.get(id).cloned())\n- .collect::>();\n- (ingests, None::)\n- };\n-\n- let reduced = state.reduced.read().await;\n- let user = optional_principal(&headers, &jar, &reduced);\n- let sc = nav.scope();\n- let show_compose = match &sc {\n- ScopeId::Public => user.is_some(),\n- ScopeId::Room(rid) => user\n- .as_ref()\n- .map(|u| user_can_post_room(&reduced, rid, u))\n- .unwrap_or(false),\n- };\n- let strip = auth_strip(&headers, &jar, &reduced);\n- let now = now_ms();\n- let entry_rows: Vec = display_ingests\n- .iter()\n- .enumerate()\n- .map(|(i, ing)| {\n- let post_idx = offset + i;\n- ingest_entry_markup(&nav, &tag, post_idx, ing, user.as_deref(), now, &reduced)\n- })\n- .collect();\n- drop(reduced);\n-\n- let paginator_top = render_thread_paginator(&nav, &tag, offset, total, true);\n- let paginator_bot = render_thread_paginator(&nav, &tag, offset, total, false);\n-\n- let bc: Markup = match &sc {\n- ScopeId::Public => bc_threads(Some(&tag)),\n- ScopeId::Room(rid) => {\n- let slug = if let Some((_, s)) = rid.split_once('/') {\n- s\n- } else {\n- rid.as_str()\n- };\n- bc_room(&nav, slug, Some(&tag))\n- }\n- };\n-\n- let cli = match &sc {\n- ScopeId::Public => format!(\"npx slugsocial public forum show {tag}\"),\n- ScopeId::Room(r) => format!(\"npx slugsocial private {r} forum show {tag}\"),\n- };\n-\n- let page = layout(\n- &format!(\"#{tag}\"),\n- \"view-thread\",\n- html! {\n- (strip)\n- nav class=\"breadcrumb\" { (bc) }\n- p class=\"muted\" { \"top=oldest · bottom=newest\" }\n- div id=\"thread-feed-region\" {\n- @if display_ingests.is_empty() {\n- p class=\"muted\" { \"no activity yet\" }\n- } @else {\n- (paginator_top)\n- @for row in &entry_rows {\n- (row)\n- }\n- (paginator_bot)\n- }\n- }\n- div id=\"thread-live-region\" {\n- (compose_form(&nav, &tag, show_compose))\n- }\n- (cli_panel(std::slice::from_ref(&cli)))\n- },\n- None,\n- theme_from_jar(&jar),\n- &theme_next_from_uri(&uri),\n- );\n- Html(page.into_string()).into_response()\n-}\n-\n-/// Thread view — `/t/:tag`\n-pub async fn thread_view(\n- State(state): State,\n- Path(tag): Path,\n- Query(q): Query,\n- headers: HeaderMap,\n- jar: CookieJar,\n- uri: Uri,\n-) -> impl IntoResponse {\n- thread_view_inner(state, tag, q, ThreadNav::public(), headers, jar, uri).await\n-}\n-\n-/// Room thread — `/r/:short/:slug/t/:tag`\n-pub async fn room_thread_view(\n- State(state): State,\n- Path((room_short, room_slug, tag)): Path<(String, String, String)>,\n- Query(q): Query,\n- headers: HeaderMap,\n- jar: CookieJar,\n- uri: Uri,\n-) -> impl IntoResponse {\n- let room_id = format!(\"{room_short}/{room_slug}\");\n- let reduced = state.reduced.read().await;\n- let user = optional_principal(&headers, &jar, &reduced);\n- if !user_can_view_room(&reduced, &room_id, user.as_deref()) {\n- drop(reduced);\n- return room_not_found_page(&jar, &uri).into_response();\n- }\n- drop(reduced);\n- let Some(nav) = ThreadNav::from_room_id(&room_id) else {\n- return (StatusCode::NOT_FOUND, \"bad room path\").into_response();\n- };\n- thread_view_inner(state, tag, q, nav, headers, jar, uri)\n- .await\n- .into_response()\n-}\n-\n-fn room_not_found_page(jar: &CookieJar, uri: &Uri) -> impl IntoResponse {\n- let body = html! {\n- nav class=\"breadcrumb\" { a href=\"/\" { \"slug.social\" } }\n- h1 { \"not found\" }\n- p { \"The requested page could not be found.\" }\n- p { a href=\"/\" { \"home\" } }\n- };\n- let page = layout(\n- \"not found — slug.social\",\n- \"view-thread\",\n- body,\n- None,\n- theme_from_jar(jar),\n- &theme_next_from_uri(uri),\n- );\n- (StatusCode::NOT_FOUND, Html(page.into_string()))\n-}\n-\n-/// Private room index — `/r/:short/:slug`\n-pub async fn room_page(\n- State(state): State,\n- Path((room_short, room_slug)): Path<(String, String)>,\n- headers: HeaderMap,\n- jar: CookieJar,\n- uri: Uri,\n-) -> impl IntoResponse {\n- let room_id = format!(\"{room_short}/{room_slug}\");\n- let now = now_ms();\n- let reduced = state.reduced.read().await;\n- if !reduced.rooms.contains(&room_id) {\n- drop(reduced);\n- return (StatusCode::NOT_FOUND, \"room not found\").into_response();\n- }\n- let user = optional_principal(&headers, &jar, &reduced);\n- if !user_can_view_room(&reduced, &room_id, user.as_deref()) {\n- drop(reduced);\n- return room_not_found_page(&jar, &uri).into_response();\n- }\n- let scope = ScopeId::Room(room_id.clone());\n- let mut rows = collect_thread_rows_for_scope(&reduced, &scope, now);\n- let strip = auth_strip(&headers, &jar, &reduced);\n- let show_new = user\n- .as_ref()\n- .map(|u| user_can_post_room(&reduced, &room_id, u))\n- .unwrap_or(false);\n- rows.sort_by(|a, b| b.last_ts.cmp(&a.last_ts));\n-\n- let Some(nav) = ThreadNav::from_room_id(&room_id) else {\n- drop(reduced);\n- return (StatusCode::NOT_FOUND, \"room not found\").into_response();\n- };\n- // Precompute markup that needs `&ReducerState`, then release the lock before `layout`.\n- // `tokio::sync::RwLock` queues new readers behind a waiting writer; holding the read\n- // guard across HTML/stringify could stall other navigations when an ingest needs a write lock.\n- let members_markup = room_members_section_markup(&reduced, &room_id, false);\n- let forum_cli = format!(\"npx slugsocial private {room_id} forum list\");\n- let garden_cli = format!(\"npx slugsocial private {room_id} garden tree\");\n- let audit_cli = format!(\"npx slugsocial private {room_id} audit\");\n- drop(reduced);\n-\n- let slug_display = room_slug.as_str();\n- let page = layout(\n- &format!(\"room {slug_display} — slug.social\"),\n- \"view-thread\",\n- html! {\n- (strip)\n- nav class=\"breadcrumb\" { (bc_room(&nav, slug_display, None)) }\n- (members_markup)\n- h3 { \"threads\" }\n- (render_thread_feed(Some(&nav), \"room-thread-feed\", &rows, now))\n- @if show_new {\n- div id=\"room-new-thread-ui-slot\" {\n- (new_thread_form_for_room(&nav, true, false))\n- }\n- }\n- (cli_panel(&[forum_cli, garden_cli, audit_cli]))\n- },\n- None,\n- theme_from_jar(&jar),\n- &theme_next_from_uri(&uri),\n- );\n- Html(page.into_string()).into_response()\n-}\n-\n-fn new_thread_form_for_room(nav: &ThreadNav, show: bool, compose_expanded: bool) -> Markup {\n- if !show {\n- return html! {};\n- }\n- // Single root for Idiomorph when morphing `#room-new-thread-ui-slot` (expanded has form + section).\n- html! {\n- div class=\"room-new-thread-slot-inner\" {\n- @if compose_expanded {\n- form method=\"POST\" action=\"/ui\" {\n- input type=\"hidden\" name=(UI_RPC_FIELD) value=(template_json_compact(&HtmlUiAction::SetRoomNewThreadComposeExpanded {\n- room_wire: nav.room_wire.clone(),\n- expanded: false,\n- }).expect(\"static json\"));\n- button type=\"submit\" class=\"form-toggle\" aria-expanded=\"true\" {\n- \"-\"\n- }\n- }\n- section class=\"compose\" id=\"room-new-thread-compose\" {\n- div id=\"room-new-thread-errors\" {}\n- form id=\"room-new-thread-form\" method=\"POST\" action=\"/ui\" data-check-action=\"/ui\" data-check-rpc=(template_json_compact(&json!({\n- \"action\": \"check_ingest\",\n- \"room\": nav.room_wire,\n- \"thread_tag\": {\"$form\": \"thread_tag\"},\n- \"text\": {\"$form\": \"text\"},\n- \"error_target\": \"room-new-thread-errors\",\n- \"form_id\": \"room-new-thread-form\",\n- })).unwrap()) {\n- input type=\"hidden\" name=(UI_RPC_FIELD) value=(template_json_compact(&json!({\n- \"action\": \"post_ingest\",\n- \"room\": nav.room_wire,\n- \"thread_tag\": {\"$form\": \"thread_tag\"},\n- \"text\": {\"$form\": \"text\"},\n- \"error_target\": \"room-new-thread-errors\",\n- \"form_id\": \"room-new-thread-form\",\n- })).unwrap());\n- input type=\"text\" id=\"room-new-tag\" name=\"thread_tag\" pattern=\"[a-z0-9_\\\\-]{1,64}\" required placeholder=\"thread-topic-slug-here\";\n- textarea name=\"text\" rows=\"4\" placeholder=\"First post body…\" required {}\n- p { button type=\"submit\" { \"post\" } }\n- }\n- }\n- } @else {\n- form method=\"POST\" action=\"/ui\" {\n- input type=\"hidden\" name=(UI_RPC_FIELD) value=(template_json_compact(&HtmlUiAction::SetRoomNewThreadComposeExpanded {\n- room_wire: nav.room_wire.clone(),\n- expanded: true,\n- }).expect(\"static json\"));\n- button type=\"submit\" class=\"form-toggle\" aria-expanded=\"false\" {\n- \"+\"\n- }\n- }\n- }\n- }\n- }\n-}\n-\n-pub(crate) fn login_to_post_hint_markup() -> Markup {\n- html! {\n- p class=\"muted\" { \"log in to post\" }\n- }\n-}\n-\n-pub(crate) fn fragment_public_new_thread_form(show: bool) -> Markup {\n- new_thread_form_public(show)\n-}\n-\n-pub(crate) fn fragment_room_new_thread_form(nav: &ThreadNav, show: bool, compose_expanded: bool) -> Markup {\n- new_thread_form_for_room(nav, show, compose_expanded)\n-}\n-\n-async fn thread_post_view_inner(\n- state: AppState,\n- tag: String,\n- index_str: String,\n- nav: ThreadNav,\n- viewer: Option,\n- jar: CookieJar,\n- uri: Uri,\n-) -> impl IntoResponse {\n- let tag = canonicalize_tag(&tag);\n- let index: usize = index_str.parse().unwrap_or(0);\n- let scope = nav.scope();\n- let now = now_ms();\n- let (ing, subtitle, body_markup) = {\n- let reduced = state.reduced.read().await;\n- let ing = reduced\n- .ingests_by_scope_thread\n- .get(&(scope.clone(), tag.clone()))\n- .and_then(|q| q.iter().rev().nth(index))\n- .and_then(|id| reduced.ingests_by_id.get(id).cloned());\n- let body = ing.as_ref().map(|i| {\n- ingest_entry_markup(&nav, &tag, index, i, viewer.as_deref(), now, &reduced)\n- });\n- (ing, None::, body)\n- };\n-\n- let sc = nav.scope();\n- let bc: Markup = match &sc {\n- ScopeId::Public => bc_threads(Some(&tag)),\n- ScopeId::Room(rid) => {\n- let slug = if let Some((_, s)) = rid.split_once('/') {\n- s\n- } else {\n- rid.as_str()\n- };\n- bc_room(&nav, slug, Some(&tag))\n- }\n- };\n-\n- let page = layout(\n- &format!(\"#{tag} / post #{index}\"),\n- \"view-thread\",\n- html! {\n- nav class=\"breadcrumb\" { (bc) }\n- h2 { \"#\" (tag) @if let Some(sub) = &subtitle { \": \" (sub) } \" / post #\" (index) }\n- @if let (Some(_ing), Some(bm)) = (&ing, &body_markup) {\n- (bm)\n- } @else {\n- p class=\"muted\" { \"post not found\" }\n- }\n- },\n- None,\n- theme_from_jar(&jar),\n- &theme_next_from_uri(&uri),\n- );\n- Html(page.into_string()).into_response()\n-}\n-\n-pub async fn thread_post_view(\n- State(state): State,\n- Path((tag, index_str)): Path<(String, String)>,\n- headers: HeaderMap,\n- jar: CookieJar,\n- uri: Uri,\n-) -> impl IntoResponse {\n- let reduced = state.reduced.read().await;\n- let viewer = optional_principal(&headers, &jar, &reduced);\n- drop(reduced);\n- thread_post_view_inner(state, tag, index_str, ThreadNav::public(), viewer, jar, uri)\n- .await\n-}\n-\n-pub async fn room_thread_post_view(\n- State(state): State,\n- Path((room_short, room_slug, tag, index_str)): Path<(String, String, String, String)>,\n- headers: HeaderMap,\n- jar: CookieJar,\n- uri: Uri,\n-) -> impl IntoResponse {\n- let room_id = format!(\"{room_short}/{room_slug}\");\n- let reduced = state.reduced.read().await;\n- let user = optional_principal(&headers, &jar, &reduced);\n- if !user_can_view_room(&reduced, &room_id, user.as_deref()) {\n- drop(reduced);\n- return room_not_found_page(&jar, &uri).into_response();\n- }\n- drop(reduced);\n- let Some(nav) = ThreadNav::from_room_id(&room_id) else {\n- return (StatusCode::NOT_FOUND, \"bad room path\").into_response();\n- };\n- thread_post_view_inner(state, tag, index_str, nav, user, jar, uri)\n- .await\n- .into_response()\n-}\n-\n-pub(crate) async fn thread_ui_expand_post_full(\n- state: &AppState,\n- room: &str,\n- thread_tag: &str,\n- post_index: usize,\n- viewer: Option<&str>,\n-) -> axum::response::Response {\n- let room = room.trim();\n- let tag = canonicalize_tag(thread_tag);\n- let scope = scope_from_room_wire(room);\n- let nav = if room.is_empty() || room == \"public\" {\n- ThreadNav::public()\n- } else {\n- let Some(n) = ThreadNav::from_room_id(room) else {\n- return super::ui_js_warn(\"bad room\");\n- };\n- n\n- };\n- if let ScopeId::Room(ref rid) = scope {\n- let reduced = state.reduced.read().await;\n- if !user_can_view_room(&reduced, rid, viewer) {\n- return super::ui_js_warn(\"forbidden\");\n- }\n- }\n-\n- let now = now_ms();\n- let reduced = state.reduced.read().await;\n- let ing = reduced\n- .ingests_by_scope_thread\n- .get(&(scope.clone(), tag.clone()))\n- .and_then(|q| q.iter().rev().nth(post_index))\n- .and_then(|id| reduced.ingests_by_id.get(id).cloned());\n- let Some(ing) = ing else {\n- return super::ui_js_warn(\"post not found\");\n- };\n- if reduced.redacted_posts.contains(&ing.id) {\n- return super::ui_js_warn(\"post not found\");\n- }\n- let ing_id = ing.id.clone();\n- drop(reduced);\n-\n- let full_html = html! {\n- div class=\"ingest-entry\" data-ingest-id=(ing_id) {\n- (post_header_row(\n- &nav,\n- &tag,\n- post_index,\n- &ing,\n- viewer,\n- now,\n- viewer == Some(ing.principal.as_str()),\n- ))\n- (render_linkified_with_embeds_in_scope(&ing.raw, nav.garden_root_url()))\n- }\n- };\n-\n- JsBuilder::new()\n- .morph_selector(&format!(\"[data-ingest-id=\\\"{}\\\"]\", ing_id), full_html)\n- .into_response()\n-}\n-\n-pub(crate) async fn thread_ui_expand_redacted_post(\n- state: &AppState,\n- room: &str,\n- thread_tag: &str,\n- post_index: usize,\n- viewer: Option<&str>,\n-) -> axum::response::Response {\n- let room = room.trim();\n- let tag = canonicalize_tag(thread_tag);\n- let scope = scope_from_room_wire(room);\n- let nav = if room.is_empty() || room == \"public\" {\n- ThreadNav::public()\n- } else {\n- let Some(n) = ThreadNav::from_room_id(room) else {\n- return super::ui_js_warn(\"bad room\");\n- };\n- n\n- };\n- if let ScopeId::Room(ref rid) = scope {\n- let reduced = state.reduced.read().await;\n- if !user_can_view_room(&reduced, rid, viewer) {\n- return super::ui_js_warn(\"forbidden\");\n- }\n- }\n-\n- let now = now_ms();\n- let reduced = state.reduced.read().await;\n- let ing = reduced\n- .ingests_by_scope_thread\n- .get(&(scope.clone(), tag.clone()))\n- .and_then(|q| q.iter().rev().nth(post_index))\n- .and_then(|id| reduced.ingests_by_id.get(id).cloned());\n- let Some(ing) = ing else {\n- return super::ui_js_warn(\"post not found\");\n- };\n- if !reduced.redacted_posts.contains(&ing.id) {\n- return super::ui_js_warn(\"post not found\");\n- }\n- let ing_id = ing.id.clone();\n- drop(reduced);\n-\n- let full_html = html! {\n- div class=\"ingest-entry ingest-redacted-expanded\" data-ingest-id=(ing_id) {\n- (redacted_header_row(&nav, &tag, post_index, &ing, now, true))\n- (render_linkified_with_embeds_in_scope(&ing.raw, nav.garden_root_url()))\n- }\n- };\n-\n- JsBuilder::new()\n- .morph_selector(&format!(\"[data-ingest-id=\\\"{}\\\"]\", ing_id), full_html)\n- .into_response()\n-}\n-\n-pub(crate) async fn thread_ui_collapse_redacted_post(\n- state: &AppState,\n- room: &str,\n- thread_tag: &str,\n- post_index: usize,\n- viewer: Option<&str>,\n-) -> axum::response::Response {\n- let room = room.trim();\n- let tag = canonicalize_tag(thread_tag);\n- let scope = scope_from_room_wire(room);\n- let nav = if room.is_empty() || room == \"public\" {\n- ThreadNav::public()\n- } else {\n- let Some(n) = ThreadNav::from_room_id(room) else {\n- return super::ui_js_warn(\"bad room\");\n- };\n- n\n- };\n- if let ScopeId::Room(ref rid) = scope {\n- let reduced = state.reduced.read().await;\n- if !user_can_view_room(&reduced, rid, viewer) {\n- return super::ui_js_warn(\"forbidden\");\n- }\n- }\n-\n- let now = now_ms();\n- let reduced = state.reduced.read().await;\n- let ing = reduced\n- .ingests_by_scope_thread\n- .get(&(scope.clone(), tag.clone()))\n- .and_then(|q| q.iter().rev().nth(post_index))\n- .and_then(|id| reduced.ingests_by_id.get(id).cloned());\n- let Some(ing) = ing else {\n- return super::ui_js_warn(\"post not found\");\n- };\n- if !reduced.redacted_posts.contains(&ing.id) {\n- return super::ui_js_warn(\"post not found\");\n- }\n- let ing_id = ing.id.clone();\n- let collapsed = ingest_entry_markup(&nav, &tag, post_index, &ing, viewer, now, &reduced);\n- drop(reduced);\n-\n- JsBuilder::new()\n- .morph_selector(&format!(\"[data-ingest-id=\\\"{}\\\"]\", ing_id), collapsed)\n- .into_response()\n-}\n-\n-struct ProfilePostRow {\n- thread_tag: String,\n- post_idx: usize,\n- post_href: String,\n- ts: i64,\n- snippet: String,\n-}\n-\n-pub async fn user_profile_page(\n- State(state): State,\n- Path(username): Path,\n- headers: HeaderMap,\n- jar: CookieJar,\n- uri: Uri,\n-) -> impl IntoResponse {\n- let canon = match parse_username(&username) {\n- Ok(u) => u,\n- Err(_) => return (StatusCode::NOT_FOUND, \"not found\").into_response(),\n- };\n-\n- let (rows, strip) = {\n- let reduced = state.reduced.read().await;\n- let viewer = optional_principal(&headers, &jar, &reduced);\n- let ids = reduced.visible_posts_for_actor(&canon, viewer.as_deref());\n- let mut rows: Vec = Vec::new();\n- for id in ids {\n- if reduced.redacted_posts.contains(&id) {\n- continue;\n- }\n- let Some(ing) = reduced.ingests_by_id.get(&id).cloned() else {\n- continue;\n- };\n- let Some(nav) = thread_nav_for_ingest(&ing) else {\n- continue;\n- };\n- let Some(post_idx) = thread_post_index_in_scope(&reduced, &ing) else {\n- continue;\n- };\n- let tag = canonicalize_tag(&ing.thread_tag);\n- let post_href = nav.post_url(&tag, post_idx);\n- let raw_one_line = ing.raw.lines().next().unwrap_or(\"\").trim();\n- let snippet: String = raw_one_line.chars().take(120).collect();\n- rows.push(ProfilePostRow {\n- thread_tag: tag,\n- post_idx,\n- post_href,\n- ts: ing.ts,\n- snippet,\n- });\n- }\n- rows.sort_by(|a, b| b.ts.cmp(&a.ts));\n- let strip = auth_strip(&headers, &jar, &reduced);\n- (rows, strip)\n- };\n-\n- let now = now_ms();\n- let page = layout(\n- &format!(\"@{canon}\"),\n- \"view-thread\",\n- html! {\n- (strip)\n- nav class=\"breadcrumb\" {\n- a href=\"/\" { \"slug.social\" }\n- (bc_segment(&format!(\"@{canon}\"), &profile_href(&canon), true))\n- }\n- h2 { \"@\" (canon) }\n- p class=\"muted\" { \"posts (newest first)\" }\n- @if rows.is_empty() {\n- p class=\"muted\" { \"no public posts yet\" }\n- } @else {\n- ul class=\"profile-post-list\" {\n- @for r in &rows {\n- @let hover = timeago::rfc3339_utc(r.ts);\n- @let ago = timeago::timeago(now, r.ts);\n- li {\n- a href=(r.post_href.as_str()) {\n- \"#\" (r.thread_tag)\n- \" / #\"\n- (r.post_idx)\n- }\n- span class=\"muted\" title=(hover) { \" · \" (ago) }\n- @if !r.snippet.is_empty() {\n- p class=\"profile-post-snippet muted\" { (r.snippet) }\n- }\n- }\n- }\n- }\n- }\n- (cli_panel(&[format!(\"npx slugsocial public forum list\")]))\n- },\n- None,\n- theme_from_jar(&jar),\n- &theme_next_from_uri(&uri),\n- );\n- Html(page.into_string()).into_response()\n-}\ndiff --git a/server/src/html/forum/feed.rs b/server/src/html/forum/feed.rs\nnew file mode 100644\nindex 0000000000000000000000000000000000000000..30b95ff6e85245a82199c96e41484d03c24989dd\n--- /dev/null\n+++ b/server/src/html/forum/feed.rs\n@@ -0,0 +1,261 @@\n+use axum::{\n+ extract::State,\n+ http::{HeaderMap, Uri},\n+ response::{Html, IntoResponse},\n+};\n+use axum_extra::extract::cookie::CookieJar;\n+use maud::{html, Markup};\n+\n+use crate::api::optional_principal;\n+use crate::canonical_path::canonicalize_tag;\n+use crate::reducer::{ReducerState, ScopeId};\n+use crate::state::AppState;\n+use crate::timeago;\n+\n+use super::ingest::ingest_entry_markup;\n+use super::nav::ThreadNav;\n+use super::page::auth_strip;\n+use super::paginator::{render_thread_paginator, PAGE_SIZE};\n+use crate::html::{\n+ bc_threads, cli_panel, layout, now_ms, recency_class, theme_from_jar, theme_next_from_uri,\n+};\n+\n+#[derive(Clone)]\n+pub(super) struct ThreadRow {\n+ pub(super) tag: String,\n+ pub(super) subtitle: Option,\n+ pub(super) last_ts: i64,\n+ pub(super) ingests: usize,\n+}\n+\n+pub(super) fn collect_thread_rows_for_scope(reduced: &ReducerState, scope: &ScopeId, now: i64) -> Vec {\n+ let _ = now;\n+ reduced\n+ .forum_threads\n+ .iter()\n+ .filter(|((s, _), _)| s == scope)\n+ .map(|((_, tag), thread)| {\n+ let ingests = reduced\n+ .ingests_by_scope_thread\n+ .get(&(scope.clone(), tag.clone()))\n+ .map(|q| q.len())\n+ .unwrap_or(0);\n+ ThreadRow {\n+ tag: tag.clone(),\n+ subtitle: None,\n+ last_ts: thread.last_activity_ts,\n+ ingests,\n+ }\n+ })\n+ .collect()\n+}\n+\n+pub(super) fn rooms_for_user(reduced: &ReducerState, username: &str) -> Vec {\n+ let mut v: Vec = reduced\n+ .grants\n+ .iter()\n+ .filter(|(rid, m)| reduced.rooms.contains(*rid) && m.contains_key(username))\n+ .map(|(rid, _)| rid.clone())\n+ .collect();\n+ v.sort();\n+ v\n+}\n+\n+/// `feed_id` is e.g. `thread-feed` (public bump list, SSE) or `room-thread-feed`.\n+pub(super) fn render_thread_feed(nav: Option<&ThreadNav>, feed_id: &str, rows: &[ThreadRow], now: i64) -> Markup {\n+ html! {\n+ div id=(feed_id) {\n+ @if rows.is_empty() {\n+ p class=\"muted\" { \"no threads yet\" }\n+ } @else {\n+ ul class=\"thread-feed\" {\n+ @for r in rows {\n+ @let thread_href = nav\n+ .as_ref()\n+ .map(|n| n.thread_url(&r.tag))\n+ .unwrap_or_else(|| format!(\"/t/{}\", r.tag));\n+ @let hover = timeago::rfc3339_utc(r.last_ts);\n+ @let ago = timeago::timeago(now, r.last_ts);\n+ @let age_cls = recency_class(now, r.last_ts);\n+ li class=(age_cls) {\n+ a href=(thread_href) {\n+ \"#\" (r.tag)\n+ @if let Some(subtitle) = &r.subtitle {\n+ \": \" (subtitle)\n+ }\n+ \" \"\n+ span class=\"muted\" title=(hover) {\n+ (ago)\n+ \" · \"\n+ (format!(\"{}n\", r.ingests))\n+ }\n+ }\n+ }\n+ }\n+ }\n+ }\n+ }\n+ }\n+}\n+\n+/// Returns the current public thread feed markup for SSE (`#thread-feed`).\n+pub async fn thread_feed_html(state: &AppState) -> Markup {\n+ let now = now_ms();\n+ let nav = ThreadNav::public();\n+ let mut rows = {\n+ let reduced = state.reduced.read().await;\n+ collect_thread_rows_for_scope(&reduced, &ScopeId::Public, now)\n+ };\n+ rows.sort_by(|a, b| b.last_ts.cmp(&a.last_ts));\n+ render_thread_feed(Some(&nav), \"thread-feed\", &rows, now)\n+}\n+\n+/// Returns the current private room thread feed markup for SSE (`#room-thread-feed`).\n+pub async fn thread_feed_html_for_room(state: &AppState, room_id: &str) -> Markup {\n+ let now = now_ms();\n+ let Some(nav) = ThreadNav::from_room_id(room_id) else {\n+ return html! { div id=\"room-thread-feed\" { p class=\"muted\" { \"room not found\" } } };\n+ };\n+ let mut rows = {\n+ let reduced = state.reduced.read().await;\n+ collect_thread_rows_for_scope(&reduced, &ScopeId::Room(room_id.to_string()), now)\n+ };\n+ rows.sort_by(|a, b| b.last_ts.cmp(&a.last_ts));\n+ render_thread_feed(Some(&nav), \"room-thread-feed\", &rows, now)\n+}\n+\n+fn render_thread_feed_region_markup(\n+ nav: &ThreadNav,\n+ tag: &str,\n+ display_ingests: &[crate::events::Ingest],\n+ offset: usize,\n+ total: usize,\n+ now: i64,\n+ viewer: Option<&str>,\n+ reduced: &ReducerState,\n+) -> Markup {\n+ let paginator_top = render_thread_paginator(nav, tag, offset, total, true);\n+ let paginator_bot = render_thread_paginator(nav, tag, offset, total, false);\n+ html! {\n+ div id=\"thread-feed-region\" {\n+ @if display_ingests.is_empty() {\n+ p class=\"muted\" { \"no activity yet\" }\n+ } @else {\n+ (paginator_top)\n+ @for (i, ing) in display_ingests.iter().enumerate() {\n+ @let post_idx = offset + i;\n+ (ingest_entry_markup(nav, tag, post_idx, ing, viewer, now, reduced))\n+ }\n+ (paginator_bot)\n+ }\n+ }\n+ }\n+}\n+\n+pub async fn thread_feed_region_markup(\n+ state: &AppState,\n+ room_id: Option<&str>,\n+ tag: &str,\n+ viewer: Option<&str>,\n+) -> Markup {\n+ let tag = canonicalize_tag(tag);\n+ let Some(nav) = (match room_id {\n+ Some(room_id) => ThreadNav::from_room_id(room_id),\n+ None => Some(ThreadNav::public()),\n+ }) else {\n+ return html! { div id=\"thread-feed-region\" { p class=\"muted\" { \"thread not found\" } } };\n+ };\n+ let scope = nav.scope();\n+ let all_ids: Vec = {\n+ let reduced = state.reduced.read().await;\n+ reduced\n+ .ingests_by_scope_thread\n+ .get(&(scope.clone(), tag.clone()))\n+ .map(|q| q.iter().rev().cloned().collect())\n+ .unwrap_or_default()\n+ };\n+ let total = all_ids.len();\n+ let offset = total.saturating_sub(PAGE_SIZE);\n+ let page_ids: Vec = all_ids.into_iter().skip(offset).take(PAGE_SIZE).collect();\n+ let reduced = state.reduced.read().await;\n+ let display_ingests = page_ids\n+ .iter()\n+ .filter_map(|id| reduced.ingests_by_id.get(id).cloned())\n+ .collect::>();\n+ render_thread_feed_region_markup(\n+ &nav,\n+ &tag,\n+ &display_ingests,\n+ offset,\n+ total,\n+ now_ms(),\n+ viewer,\n+ &reduced,\n+ )\n+}\n+\n+/// Home: private rooms (signed-in), then public bump-ordered threads.\n+pub async fn home(\n+ State(state): State,\n+ headers: HeaderMap,\n+ jar: CookieJar,\n+ uri: Uri,\n+) -> impl IntoResponse {\n+ let now = now_ms();\n+ let reduced = state.reduced.read().await;\n+ let user = optional_principal(&headers, &jar, &reduced);\n+ let room_ids = user\n+ .as_ref()\n+ .map(|u| rooms_for_user(&reduced, u))\n+ .unwrap_or_default();\n+ let mut public_rows = collect_thread_rows_for_scope(&reduced, &ScopeId::Public, now);\n+ drop(reduced);\n+ public_rows.sort_by(|a, b| b.last_ts.cmp(&a.last_ts));\n+\n+ let nav = ThreadNav::public();\n+ let reduced_read = state.reduced.read().await;\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+ html! {\n+ (strip)\n+ nav class=\"breadcrumb\" { (bc_threads(None)) }\n+ @if !room_ids.is_empty() {\n+ h2 { \"your rooms\" }\n+ ul class=\"thread-feed\" {\n+ @for rid in &room_ids {\n+ @if let Some(nav_r) = ThreadNav::from_room_id(rid) {\n+ @let slug = if let Some((_, s)) = rid.split_once('/') { s } else { rid.as_str() };\n+ li {\n+ a href=(nav_r.room_url()) {\n+ (slug)\n+ span class=\"muted\" { \" · \" (rid) }\n+ }\n+ }\n+ }\n+ }\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::ExpandPublicNewThreadForm).expect(\"static json\"));\n+ button type=\"submit\" class=\"section-add-btn\" { \"+\" }\n+ }\n+ }\n+ div id=\"public-new-thread-ui-slot\" {}\n+ (render_thread_feed(Some(&nav), \"thread-feed\", &public_rows, now))\n+ (cli_panel(&[\"npx slugsocial public forum list\"]))\n+ },\n+ None,\n+ theme_from_jar(&jar),\n+ &theme_next_from_uri(&uri),\n+ );\n+ Html(page.into_string())\n+}\ndiff --git a/server/src/html/forum/ingest.rs b/server/src/html/forum/ingest.rs\nindex c045046a005db4f8f4d1dc7ea74aaaba45e0f3f4..5bd0c09c546e4ab8ed9e03a8c4879efc9bdc7188 100644\n--- a/server/src/html/forum/ingest.rs\n+++ b/server/src/html/forum/ingest.rs\n@@ -6,7 +6,8 @@ use serde_json::json;\n \n use crate::html::js_string_literal;\n use crate::html::ui_action::{HtmlUiAction, UI_RPC_FIELD};\n-use crate::html::{profile_href, render_linkified_with_embeds_in_scope, timeago};\n+use crate::html::{profile_href, render_linkified_with_embeds_in_scope};\n+use crate::timeago;\n \n use super::nav::ThreadNav;\n \ndiff --git a/server/src/html/forum/mod.rs b/server/src/html/forum/mod.rs\nnew file mode 100644\nindex 0000000000000000000000000000000000000000..8fbf3b71f41fafb474999ebb3d622e6bd5b5f313\n--- /dev/null\n+++ b/server/src/html/forum/mod.rs\n@@ -0,0 +1,29 @@\n+//! Forum / thread HTML: public home, room index, thread views, profile, and UI morph helpers.\n+\n+mod access;\n+mod feed;\n+mod ingest;\n+mod nav;\n+mod new_thread;\n+mod page;\n+mod paginator;\n+mod post_single;\n+mod profile;\n+mod room_members;\n+mod thread_morph;\n+mod views;\n+\n+pub use feed::{home, thread_feed_html, thread_feed_html_for_room, thread_feed_region_markup};\n+pub use nav::ThreadNav;\n+pub use post_single::{room_thread_post_view, thread_post_view};\n+pub use profile::user_profile_page;\n+pub use views::{room_page, room_thread_view, thread_view};\n+\n+pub(crate) use access::{user_can_post_room, user_can_view_room};\n+pub(crate) use new_thread::{\n+ fragment_public_new_thread_form, fragment_room_new_thread_form, login_to_post_hint_markup,\n+};\n+pub(crate) use room_members::room_members_section_markup;\n+pub(crate) use thread_morph::{\n+ thread_ui_collapse_redacted_post, thread_ui_expand_post_full, thread_ui_expand_redacted_post,\n+};\ndiff --git a/server/src/html/forum/page.rs b/server/src/html/forum/page.rs\nnew file mode 100644\nindex 0000000000000000000000000000000000000000..b0461fc22e0d7b9c865c483362113d60e992e0b8\n--- /dev/null\n+++ b/server/src/html/forum/page.rs\n@@ -0,0 +1,50 @@\n+use axum::http::HeaderMap;\n+use axum_extra::extract::cookie::CookieJar;\n+use maud::{html, Markup};\n+\n+use crate::api::optional_principal;\n+use crate::reducer::ReducerState;\n+\n+use super::nav::ThreadNav;\n+use crate::html::bc_segment;\n+\n+pub(super) fn auth_strip(\n+ headers: &HeaderMap,\n+ jar: &CookieJar,\n+ reduced: &ReducerState,\n+) -> Markup {\n+ match optional_principal(headers, jar, reduced) {\n+ Some(u) => html! {\n+ p class=\"muted auth-strip\" {\n+ \"@\" (u)\n+ \" · \"\n+ a href=\"/logout\" { \"log out\" }\n+ }\n+ },\n+ None => html! {\n+ p class=\"muted auth-strip\" {\n+ a href=\"/login\" { \"log in\" }\n+ }\n+ },\n+ }\n+}\n+\n+pub(super) fn bc_room(nav: &ThreadNav, room_slug: &str, thread_tag: Option<&str>) -> Markup {\n+ html! {\n+ a href=\"/\" { \"slug.social\" }\n+ @if let Some(t) = thread_tag {\n+ (bc_segment(\n+ &format!(\"room:{room_slug}\"),\n+ nav.room_url(),\n+ false,\n+ ))\n+ (bc_segment(&format!(\"#{t}\"), &nav.thread_url(t), true))\n+ } @else {\n+ (bc_segment(\n+ &format!(\"room:{room_slug}\"),\n+ nav.room_url(),\n+ true,\n+ ))\n+ }\n+ }\n+}\ndiff --git a/server/src/html/forum/paginator.rs b/server/src/html/forum/paginator.rs\nnew file mode 100644\nindex 0000000000000000000000000000000000000000..af2fda58a409673c51669d1d6f23baf7072c979a\n--- /dev/null\n+++ b/server/src/html/forum/paginator.rs\n@@ -0,0 +1,42 @@\n+use maud::{html, Markup};\n+\n+use super::nav::ThreadNav;\n+\n+pub(super) const PAGE_SIZE: usize = 10;\n+\n+pub(super) fn render_thread_paginator(nav: &ThreadNav, tag: &str, offset: usize, total: usize, top: bool) -> Markup {\n+ let newer_offset = offset.checked_add(PAGE_SIZE).filter(|&o| o < total);\n+ let older_offset = if offset > 0 {\n+ Some(offset.saturating_sub(PAGE_SIZE))\n+ } else {\n+ None\n+ };\n+ let latest_offset = total.saturating_sub(PAGE_SIZE);\n+ let on_latest = offset >= latest_offset;\n+ let (id, scroll_href, scroll_label) = if top {\n+ (\"top\", \"#bottom\", \"↓\")\n+ } else {\n+ (\"bottom\", \"#top\", \"↑\")\n+ };\n+ html! {\n+ div class=\"thread-paginator\" id=(id) {\n+ a href=(scroll_href) class=\"post-nav-btn\" { (scroll_label) }\n+ @if let Some(o) = older_offset {\n+ a href=(nav.thread_page_url(tag, o)) class=\"post-nav-btn\" { \"← older\" }\n+ } @else {\n+ a href=\"#\" class=\"post-nav-btn disabled\" { \"← older\" }\n+ }\n+ span class=\"post-nav-pos muted\" {\n+ (offset + 1) \"–\" (total.min(offset + PAGE_SIZE)) \" / \" (total)\n+ }\n+ @if let Some(o) = newer_offset {\n+ a href=(nav.thread_page_url(tag, o)) class=\"post-nav-btn\" { \"newer →\" }\n+ } @else {\n+ a href=\"#\" class=\"post-nav-btn disabled\" { \"newer →\" }\n+ }\n+ @if !on_latest {\n+ a href=(nav.thread_page_url(tag, latest_offset)) class=\"post-nav-btn\" { \"latest\" }\n+ }\n+ }\n+ }\n+}\ndiff --git a/server/src/html/forum/post_single.rs b/server/src/html/forum/post_single.rs\nnew file mode 100644\nindex 0000000000000000000000000000000000000000..c316f8f836df9d4ef9c05ebd9e54f699540e6d72\n--- /dev/null\n+++ b/server/src/html/forum/post_single.rs\n@@ -0,0 +1,115 @@\n+use axum::{\n+ extract::{Path, State},\n+ http::{HeaderMap, StatusCode, Uri},\n+ response::{Html, IntoResponse},\n+};\n+use axum_extra::extract::cookie::CookieJar;\n+use maud::html;\n+\n+use crate::api::optional_principal;\n+use crate::canonical_path::canonicalize_tag;\n+use crate::reducer::ScopeId;\n+use crate::state::AppState;\n+\n+use super::access::user_can_view_room;\n+use super::ingest::ingest_entry_markup;\n+use super::nav::ThreadNav;\n+use super::page::bc_room;\n+use super::views::room_not_found_page;\n+use crate::html::{bc_threads, layout, now_ms, theme_from_jar, theme_next_from_uri};\n+use maud::Markup;\n+\n+async fn thread_post_view_inner(\n+ state: AppState,\n+ tag: String,\n+ index_str: String,\n+ nav: ThreadNav,\n+ viewer: Option,\n+ jar: CookieJar,\n+ uri: Uri,\n+) -> impl IntoResponse {\n+ let tag = canonicalize_tag(&tag);\n+ let index: usize = index_str.parse().unwrap_or(0);\n+ let scope = nav.scope();\n+ let now = now_ms();\n+ let (ing, subtitle, body_markup) = {\n+ let reduced = state.reduced.read().await;\n+ let ing = reduced\n+ .ingests_by_scope_thread\n+ .get(&(scope.clone(), tag.clone()))\n+ .and_then(|q| q.iter().rev().nth(index))\n+ .and_then(|id| reduced.ingests_by_id.get(id).cloned());\n+ let body = ing.as_ref().map(|i| {\n+ ingest_entry_markup(&nav, &tag, index, i, viewer.as_deref(), now, &reduced)\n+ });\n+ (ing, None::, body)\n+ };\n+\n+ let sc = nav.scope();\n+ let bc: Markup = match &sc {\n+ ScopeId::Public => bc_threads(Some(&tag)),\n+ ScopeId::Room(rid) => {\n+ let slug = if let Some((_, s)) = rid.split_once('/') {\n+ s\n+ } else {\n+ rid.as_str()\n+ };\n+ bc_room(&nav, slug, Some(&tag))\n+ }\n+ };\n+\n+ let page = layout(\n+ &format!(\"#{tag} / post #{index}\"),\n+ \"view-thread\",\n+ html! {\n+ nav class=\"breadcrumb\" { (bc) }\n+ h2 { \"#\" (tag) @if let Some(sub) = &subtitle { \": \" (sub) } \" / post #\" (index) }\n+ @if let (Some(_ing), Some(bm)) = (&ing, &body_markup) {\n+ (bm)\n+ } @else {\n+ p class=\"muted\" { \"post not found\" }\n+ }\n+ },\n+ None,\n+ theme_from_jar(&jar),\n+ &theme_next_from_uri(&uri),\n+ );\n+ Html(page.into_string()).into_response()\n+}\n+\n+pub async fn thread_post_view(\n+ State(state): State,\n+ Path((tag, index_str)): Path<(String, String)>,\n+ headers: HeaderMap,\n+ jar: CookieJar,\n+ uri: Uri,\n+) -> impl IntoResponse {\n+ let reduced = state.reduced.read().await;\n+ let viewer = optional_principal(&headers, &jar, &reduced);\n+ drop(reduced);\n+ thread_post_view_inner(state, tag, index_str, ThreadNav::public(), viewer, jar, uri)\n+ .await\n+}\n+\n+pub async fn room_thread_post_view(\n+ State(state): State,\n+ Path((room_short, room_slug, tag, index_str)): Path<(String, String, String, String)>,\n+ headers: HeaderMap,\n+ jar: CookieJar,\n+ uri: Uri,\n+) -> impl IntoResponse {\n+ let room_id = format!(\"{room_short}/{room_slug}\");\n+ let reduced = state.reduced.read().await;\n+ let user = optional_principal(&headers, &jar, &reduced);\n+ if !user_can_view_room(&reduced, &room_id, user.as_deref()) {\n+ drop(reduced);\n+ return room_not_found_page(&jar, &uri).into_response();\n+ }\n+ drop(reduced);\n+ let Some(nav) = ThreadNav::from_room_id(&room_id) else {\n+ return (StatusCode::NOT_FOUND, \"bad room path\").into_response();\n+ };\n+ thread_post_view_inner(state, tag, index_str, nav, user, jar, uri)\n+ .await\n+ .into_response()\n+}\ndiff --git a/server/src/html/forum/profile.rs b/server/src/html/forum/profile.rs\nnew file mode 100644\nindex 0000000000000000000000000000000000000000..6f9574a975bd0a33460bb2ab79760c272019f6bf\n--- /dev/null\n+++ b/server/src/html/forum/profile.rs\n@@ -0,0 +1,116 @@\n+use axum::{\n+ extract::{Path, State},\n+ http::{HeaderMap, StatusCode, Uri},\n+ response::{Html, IntoResponse},\n+};\n+use axum_extra::extract::cookie::CookieJar;\n+use maud::html;\n+\n+use crate::api::optional_principal;\n+use crate::canonical_path::canonicalize_tag;\n+use crate::identity::parse_username;\n+use crate::state::AppState;\n+\n+use super::ingest::{thread_nav_for_ingest, thread_post_index_in_scope};\n+use super::page::auth_strip;\n+use crate::timeago;\n+use crate::html::{\n+ bc_segment, cli_panel, layout, now_ms, profile_href, theme_from_jar, theme_next_from_uri,\n+};\n+\n+struct ProfilePostRow {\n+ thread_tag: String,\n+ post_idx: usize,\n+ post_href: String,\n+ ts: i64,\n+ snippet: String,\n+}\n+\n+pub async fn user_profile_page(\n+ State(state): State,\n+ Path(username): Path,\n+ headers: HeaderMap,\n+ jar: CookieJar,\n+ uri: Uri,\n+) -> impl IntoResponse {\n+ let canon = match parse_username(&username) {\n+ Ok(u) => u,\n+ Err(_) => return (StatusCode::NOT_FOUND, \"not found\").into_response(),\n+ };\n+\n+ let (rows, strip) = {\n+ let reduced = state.reduced.read().await;\n+ let viewer = optional_principal(&headers, &jar, &reduced);\n+ let ids = reduced.visible_posts_for_actor(&canon, viewer.as_deref());\n+ let mut rows: Vec = Vec::new();\n+ for id in ids {\n+ if reduced.redacted_posts.contains(&id) {\n+ continue;\n+ }\n+ let Some(ing) = reduced.ingests_by_id.get(&id).cloned() else {\n+ continue;\n+ };\n+ let Some(nav) = thread_nav_for_ingest(&ing) else {\n+ continue;\n+ };\n+ let Some(post_idx) = thread_post_index_in_scope(&reduced, &ing) else {\n+ continue;\n+ };\n+ let tag = canonicalize_tag(&ing.thread_tag);\n+ let post_href = nav.post_url(&tag, post_idx);\n+ let raw_one_line = ing.raw.lines().next().unwrap_or(\"\").trim();\n+ let snippet: String = raw_one_line.chars().take(120).collect();\n+ rows.push(ProfilePostRow {\n+ thread_tag: tag,\n+ post_idx,\n+ post_href,\n+ ts: ing.ts,\n+ snippet,\n+ });\n+ }\n+ rows.sort_by(|a, b| b.ts.cmp(&a.ts));\n+ let strip = auth_strip(&headers, &jar, &reduced);\n+ (rows, strip)\n+ };\n+\n+ let now = now_ms();\n+ let page = layout(\n+ &format!(\"@{canon}\"),\n+ \"view-thread\",\n+ html! {\n+ (strip)\n+ nav class=\"breadcrumb\" {\n+ a href=\"/\" { \"slug.social\" }\n+ (bc_segment(&format!(\"@{canon}\"), &profile_href(&canon), true))\n+ }\n+ h2 { \"@\" (canon) }\n+ p class=\"muted\" { \"posts (newest first)\" }\n+ @if rows.is_empty() {\n+ p class=\"muted\" { \"no public posts yet\" }\n+ } @else {\n+ ul class=\"profile-post-list\" {\n+ @for r in &rows {\n+ @let hover = timeago::rfc3339_utc(r.ts);\n+ @let ago = timeago::timeago(now, r.ts);\n+ li {\n+ a href=(r.post_href.as_str()) {\n+ \"#\" (r.thread_tag)\n+ \" / #\"\n+ (r.post_idx)\n+ }\n+ span class=\"muted\" title=(hover) { \" · \" (ago) }\n+ @if !r.snippet.is_empty() {\n+ p class=\"profile-post-snippet muted\" { (r.snippet) }\n+ }\n+ }\n+ }\n+ }\n+ }\n+ (cli_panel(&[format!(\"npx slugsocial public forum list\")]))\n+ },\n+ None,\n+ theme_from_jar(&jar),\n+ &theme_next_from_uri(&uri),\n+ );\n+ Html(page.into_string()).into_response()\n+}\ndiff --git a/server/src/html/forum/thread_morph.rs b/server/src/html/forum/thread_morph.rs\nnew file mode 100644\nindex 0000000000000000000000000000000000000000..77ac43b159a7f353e47b1f8368da07bb4392daf8\n--- /dev/null\n+++ b/server/src/html/forum/thread_morph.rs\n@@ -0,0 +1,171 @@\n+use maud::html;\n+\n+use crate::canonical_path::canonicalize_tag;\n+use crate::reducer::{scope_from_room_wire, ScopeId};\n+use crate::state::AppState;\n+\n+use super::access::user_can_view_room;\n+use super::ingest::{ingest_entry_markup, post_header_row, redacted_header_row};\n+use super::nav::ThreadNav;\n+use crate::html::{render_linkified_with_embeds_in_scope, JsBuilder, now_ms};\n+\n+pub(crate) async fn thread_ui_expand_post_full(\n+ state: &AppState,\n+ room: &str,\n+ thread_tag: &str,\n+ post_index: usize,\n+ viewer: Option<&str>,\n+) -> axum::response::Response {\n+ let room = room.trim();\n+ let tag = canonicalize_tag(thread_tag);\n+ let scope = scope_from_room_wire(room);\n+ let nav = if room.is_empty() || room == \"public\" {\n+ ThreadNav::public()\n+ } else {\n+ let Some(n) = ThreadNav::from_room_id(room) else {\n+ return crate::html::ui_js_warn(\"bad room\");\n+ };\n+ n\n+ };\n+ if let ScopeId::Room(ref rid) = scope {\n+ let reduced = state.reduced.read().await;\n+ if !user_can_view_room(&reduced, rid, viewer) {\n+ return crate::html::ui_js_warn(\"forbidden\");\n+ }\n+ }\n+\n+ let now = now_ms();\n+ let reduced = state.reduced.read().await;\n+ let ing = reduced\n+ .ingests_by_scope_thread\n+ .get(&(scope.clone(), tag.clone()))\n+ .and_then(|q| q.iter().rev().nth(post_index))\n+ .and_then(|id| reduced.ingests_by_id.get(id).cloned());\n+ let Some(ing) = ing else {\n+ return crate::html::ui_js_warn(\"post not found\");\n+ };\n+ if reduced.redacted_posts.contains(&ing.id) {\n+ return crate::html::ui_js_warn(\"post not found\");\n+ }\n+ let ing_id = ing.id.clone();\n+ drop(reduced);\n+\n+ let full_html = html! {\n+ div class=\"ingest-entry\" data-ingest-id=(ing_id) {\n+ (post_header_row(\n+ &nav,\n+ &tag,\n+ post_index,\n+ &ing,\n+ viewer,\n+ now,\n+ viewer == Some(ing.principal.as_str()),\n+ ))\n+ (render_linkified_with_embeds_in_scope(&ing.raw, nav.garden_root_url()))\n+ }\n+ };\n+\n+ JsBuilder::new()\n+ .morph_selector(&format!(\"[data-ingest-id=\\\"{}\\\"]\", ing_id), full_html)\n+ .into_response()\n+}\n+\n+pub(crate) async fn thread_ui_expand_redacted_post(\n+ state: &AppState,\n+ room: &str,\n+ thread_tag: &str,\n+ post_index: usize,\n+ viewer: Option<&str>,\n+) -> axum::response::Response {\n+ let room = room.trim();\n+ let tag = canonicalize_tag(thread_tag);\n+ let scope = scope_from_room_wire(room);\n+ let nav = if room.is_empty() || room == \"public\" {\n+ ThreadNav::public()\n+ } else {\n+ let Some(n) = ThreadNav::from_room_id(room) else {\n+ return crate::html::ui_js_warn(\"bad room\");\n+ };\n+ n\n+ };\n+ if let ScopeId::Room(ref rid) = scope {\n+ let reduced = state.reduced.read().await;\n+ if !user_can_view_room(&reduced, rid, viewer) {\n+ return crate::html::ui_js_warn(\"forbidden\");\n+ }\n+ }\n+\n+ let now = now_ms();\n+ let reduced = state.reduced.read().await;\n+ let ing = reduced\n+ .ingests_by_scope_thread\n+ .get(&(scope.clone(), tag.clone()))\n+ .and_then(|q| q.iter().rev().nth(post_index))\n+ .and_then(|id| reduced.ingests_by_id.get(id).cloned());\n+ let Some(ing) = ing else {\n+ return crate::html::ui_js_warn(\"post not found\");\n+ };\n+ if !reduced.redacted_posts.contains(&ing.id) {\n+ return crate::html::ui_js_warn(\"post not found\");\n+ }\n+ let ing_id = ing.id.clone();\n+ drop(reduced);\n+\n+ let full_html = html! {\n+ div class=\"ingest-entry ingest-redacted-expanded\" data-ingest-id=(ing_id) {\n+ (redacted_header_row(&nav, &tag, post_index, &ing, now, true))\n+ (render_linkified_with_embeds_in_scope(&ing.raw, nav.garden_root_url()))\n+ }\n+ };\n+\n+ JsBuilder::new()\n+ .morph_selector(&format!(\"[data-ingest-id=\\\"{}\\\"]\", ing_id), full_html)\n+ .into_response()\n+}\n+\n+pub(crate) async fn thread_ui_collapse_redacted_post(\n+ state: &AppState,\n+ room: &str,\n+ thread_tag: &str,\n+ post_index: usize,\n+ viewer: Option<&str>,\n+) -> axum::response::Response {\n+ let room = room.trim();\n+ let tag = canonicalize_tag(thread_tag);\n+ let scope = scope_from_room_wire(room);\n+ let nav = if room.is_empty() || room == \"public\" {\n+ ThreadNav::public()\n+ } else {\n+ let Some(n) = ThreadNav::from_room_id(room) else {\n+ return crate::html::ui_js_warn(\"bad room\");\n+ };\n+ n\n+ };\n+ if let ScopeId::Room(ref rid) = scope {\n+ let reduced = state.reduced.read().await;\n+ if !user_can_view_room(&reduced, rid, viewer) {\n+ return crate::html::ui_js_warn(\"forbidden\");\n+ }\n+ }\n+\n+ let now = now_ms();\n+ let reduced = state.reduced.read().await;\n+ let ing = reduced\n+ .ingests_by_scope_thread\n+ .get(&(scope.clone(), tag.clone()))\n+ .and_then(|q| q.iter().rev().nth(post_index))\n+ .and_then(|id| reduced.ingests_by_id.get(id).cloned());\n+ let Some(ing) = ing else {\n+ return crate::html::ui_js_warn(\"post not found\");\n+ };\n+ if !reduced.redacted_posts.contains(&ing.id) {\n+ return crate::html::ui_js_warn(\"post not found\");\n+ }\n+ let ing_id = ing.id.clone();\n+ let collapsed = ingest_entry_markup(&nav, &tag, post_index, &ing, viewer, now, &reduced);\n+ drop(reduced);\n+\n+ JsBuilder::new()\n+ .morph_selector(&format!(\"[data-ingest-id=\\\"{}\\\"]\", ing_id), collapsed)\n+ .into_response()\n+}\ndiff --git a/server/src/html/forum/views.rs b/server/src/html/forum/views.rs\nnew file mode 100644\nindex 0000000000000000000000000000000000000000..be08491b893f5960723e77c7f7fb9d416100f0fb\n--- /dev/null\n+++ b/server/src/html/forum/views.rs\n@@ -0,0 +1,290 @@\n+use axum::{\n+ extract::{Path, Query, State},\n+ http::{HeaderMap, StatusCode, Uri},\n+ response::{Html, IntoResponse},\n+};\n+use axum_extra::extract::cookie::CookieJar;\n+use maud::{html, Markup};\n+use serde::Deserialize;\n+use serde_json::json;\n+\n+use crate::api::optional_principal;\n+use crate::canonical_path::canonicalize_tag;\n+use crate::form_template::template_json_compact;\n+use crate::reducer::ScopeId;\n+use crate::state::AppState;\n+\n+use super::access::user_can_post_room;\n+use super::access::user_can_view_room;\n+use super::feed::{collect_thread_rows_for_scope, render_thread_feed};\n+use super::ingest::ingest_entry_markup;\n+use super::nav::ThreadNav;\n+use super::new_thread::fragment_room_new_thread_form;\n+use super::page::{auth_strip, bc_room};\n+use super::paginator::{render_thread_paginator, PAGE_SIZE};\n+use super::room_members::room_members_section_markup;\n+use crate::html::ui_action::UI_RPC_FIELD;\n+use crate::html::{\n+ bc_threads, cli_panel, layout, now_ms, theme_from_jar, theme_next_from_uri,\n+};\n+\n+fn compose_form(nav: &ThreadNav, thread_tag: &str, show: bool) -> Markup {\n+ if !show {\n+ return html! {};\n+ }\n+ html! {\n+ section class=\"compose\" id=\"thread-compose\" {\n+ form id=\"thread-compose-form\" method=\"POST\" action=\"/ui\" data-check-action=\"/ui\" data-check-rpc=(template_json_compact(&json!({\n+ \"action\": \"check_ingest\",\n+ \"room\": nav.room_wire,\n+ \"thread_tag\": thread_tag,\n+ \"text\": {\"$form\": \"text\"},\n+ \"error_target\": \"thread-compose-errors\",\n+ \"form_id\": \"thread-compose-form\",\n+ })).unwrap()) {\n+ input type=\"hidden\" name=(UI_RPC_FIELD) value=(template_json_compact(&json!({\n+ \"action\": \"post_ingest\",\n+ \"room\": nav.room_wire,\n+ \"thread_tag\": thread_tag,\n+ \"text\": {\"$form\": \"text\"},\n+ \"error_target\": \"thread-compose-errors\",\n+ \"form_id\": \"thread-compose-form\",\n+ })).unwrap());\n+ textarea name=\"text\" rows=\"5\" cols=\"80\" placeholder=\"prose or ~/items and votes…\" {}\n+ p {\n+ button type=\"submit\" { \"post\" }\n+ }\n+ }\n+ div id=\"thread-compose-errors\" {}\n+ }\n+ }\n+}\n+\n+#[derive(Debug, Deserialize)]\n+pub struct ThreadViewQuery {\n+ pub offset: Option,\n+}\n+\n+async fn thread_view_inner(\n+ state: AppState,\n+ tag: String,\n+ q: ThreadViewQuery,\n+ nav: ThreadNav,\n+ headers: HeaderMap,\n+ jar: CookieJar,\n+ uri: Uri,\n+) -> impl IntoResponse {\n+ let tag = canonicalize_tag(&tag);\n+ let scope = nav.scope();\n+\n+ let all_ids: Vec = {\n+ let reduced = state.reduced.read().await;\n+ reduced\n+ .ingests_by_scope_thread\n+ .get(&(scope.clone(), tag.clone()))\n+ .map(|q| q.iter().rev().cloned().collect())\n+ .unwrap_or_default()\n+ };\n+\n+ let total = all_ids.len();\n+ let offset = q.offset.unwrap_or(0);\n+ let page_ids: Vec = all_ids.into_iter().skip(offset).take(PAGE_SIZE).collect();\n+\n+ let (display_ingests, _subtitle) = {\n+ let reduced = state.reduced.read().await;\n+ let ingests = page_ids\n+ .iter()\n+ .filter_map(|id| reduced.ingests_by_id.get(id).cloned())\n+ .collect::>();\n+ (ingests, None::)\n+ };\n+\n+ let reduced = state.reduced.read().await;\n+ let user = optional_principal(&headers, &jar, &reduced);\n+ let sc = nav.scope();\n+ let show_compose = match &sc {\n+ ScopeId::Public => user.is_some(),\n+ ScopeId::Room(rid) => user\n+ .as_ref()\n+ .map(|u| user_can_post_room(&reduced, rid, u))\n+ .unwrap_or(false),\n+ };\n+ let strip = auth_strip(&headers, &jar, &reduced);\n+ let now = now_ms();\n+ let entry_rows: Vec = display_ingests\n+ .iter()\n+ .enumerate()\n+ .map(|(i, ing)| {\n+ let post_idx = offset + i;\n+ ingest_entry_markup(&nav, &tag, post_idx, ing, user.as_deref(), now, &reduced)\n+ })\n+ .collect();\n+ drop(reduced);\n+\n+ let paginator_top = render_thread_paginator(&nav, &tag, offset, total, true);\n+ let paginator_bot = render_thread_paginator(&nav, &tag, offset, total, false);\n+\n+ let bc: Markup = match &sc {\n+ ScopeId::Public => bc_threads(Some(&tag)),\n+ ScopeId::Room(rid) => {\n+ let slug = if let Some((_, s)) = rid.split_once('/') {\n+ s\n+ } else {\n+ rid.as_str()\n+ };\n+ bc_room(&nav, slug, Some(&tag))\n+ }\n+ };\n+\n+ let cli = match &sc {\n+ ScopeId::Public => format!(\"npx slugsocial public forum show {tag}\"),\n+ ScopeId::Room(r) => format!(\"npx slugsocial private {r} forum show {tag}\"),\n+ };\n+\n+ let page = layout(\n+ &format!(\"#{tag}\"),\n+ \"view-thread\",\n+ html! {\n+ (strip)\n+ nav class=\"breadcrumb\" { (bc) }\n+ p class=\"muted\" { \"top=oldest · bottom=newest\" }\n+ div id=\"thread-feed-region\" {\n+ @if display_ingests.is_empty() {\n+ p class=\"muted\" { \"no activity yet\" }\n+ } @else {\n+ (paginator_top)\n+ @for row in &entry_rows {\n+ (row)\n+ }\n+ (paginator_bot)\n+ }\n+ }\n+ div id=\"thread-live-region\" {\n+ (compose_form(&nav, &tag, show_compose))\n+ }\n+ (cli_panel(std::slice::from_ref(&cli)))\n+ },\n+ None,\n+ theme_from_jar(&jar),\n+ &theme_next_from_uri(&uri),\n+ );\n+ Html(page.into_string()).into_response()\n+}\n+\n+/// Thread view — `/t/:tag`\n+pub async fn thread_view(\n+ State(state): State,\n+ Path(tag): Path,\n+ Query(q): Query,\n+ headers: HeaderMap,\n+ jar: CookieJar,\n+ uri: Uri,\n+) -> impl IntoResponse {\n+ thread_view_inner(state, tag, q, ThreadNav::public(), headers, jar, uri).await\n+}\n+\n+/// Room thread — `/r/:short/:slug/t/:tag`\n+pub async fn room_thread_view(\n+ State(state): State,\n+ Path((room_short, room_slug, tag)): Path<(String, String, String)>,\n+ Query(q): Query,\n+ headers: HeaderMap,\n+ jar: CookieJar,\n+ uri: Uri,\n+) -> impl IntoResponse {\n+ let room_id = format!(\"{room_short}/{room_slug}\");\n+ let reduced = state.reduced.read().await;\n+ let user = optional_principal(&headers, &jar, &reduced);\n+ if !user_can_view_room(&reduced, &room_id, user.as_deref()) {\n+ drop(reduced);\n+ return room_not_found_page(&jar, &uri).into_response();\n+ }\n+ drop(reduced);\n+ let Some(nav) = ThreadNav::from_room_id(&room_id) else {\n+ return (StatusCode::NOT_FOUND, \"bad room path\").into_response();\n+ };\n+ thread_view_inner(state, tag, q, nav, headers, jar, uri)\n+ .await\n+ .into_response()\n+}\n+\n+pub(super) fn room_not_found_page(jar: &CookieJar, uri: &Uri) -> impl IntoResponse {\n+ let body = html! {\n+ nav class=\"breadcrumb\" { a href=\"/\" { \"slug.social\" } }\n+ h1 { \"not found\" }\n+ p { \"The requested page could not be found.\" }\n+ p { a href=\"/\" { \"home\" } }\n+ };\n+ let page = layout(\n+ \"not found — slug.social\",\n+ \"view-thread\",\n+ body,\n+ None,\n+ theme_from_jar(jar),\n+ &theme_next_from_uri(uri),\n+ );\n+ (StatusCode::NOT_FOUND, Html(page.into_string()))\n+}\n+\n+/// Private room index — `/r/:short/:slug`\n+pub async fn room_page(\n+ State(state): State,\n+ Path((room_short, room_slug)): Path<(String, String)>,\n+ headers: HeaderMap,\n+ jar: CookieJar,\n+ uri: Uri,\n+) -> impl IntoResponse {\n+ let room_id = format!(\"{room_short}/{room_slug}\");\n+ let now = now_ms();\n+ let reduced = state.reduced.read().await;\n+ if !reduced.rooms.contains(&room_id) {\n+ drop(reduced);\n+ return (StatusCode::NOT_FOUND, \"room not found\").into_response();\n+ }\n+ let user = optional_principal(&headers, &jar, &reduced);\n+ if !user_can_view_room(&reduced, &room_id, user.as_deref()) {\n+ drop(reduced);\n+ return room_not_found_page(&jar, &uri).into_response();\n+ }\n+ let scope = ScopeId::Room(room_id.clone());\n+ let mut rows = collect_thread_rows_for_scope(&reduced, &scope, now);\n+ let strip = auth_strip(&headers, &jar, &reduced);\n+ let show_new = user\n+ .as_ref()\n+ .map(|u| user_can_post_room(&reduced, &room_id, u))\n+ .unwrap_or(false);\n+ rows.sort_by(|a, b| b.last_ts.cmp(&a.last_ts));\n+\n+ let Some(nav) = ThreadNav::from_room_id(&room_id) else {\n+ drop(reduced);\n+ return (StatusCode::NOT_FOUND, \"room not found\").into_response();\n+ };\n+ let members_markup = room_members_section_markup(&reduced, &room_id, false);\n+ let forum_cli = format!(\"npx slugsocial private {room_id} forum list\");\n+ let garden_cli = format!(\"npx slugsocial private {room_id} garden tree\");\n+ let audit_cli = format!(\"npx slugsocial private {room_id} audit\");\n+ drop(reduced);\n+\n+ let slug_display = room_slug.as_str();\n+ let page = layout(\n+ &format!(\"room {slug_display} — slug.social\"),\n+ \"view-thread\",\n+ html! {\n+ (strip)\n+ nav class=\"breadcrumb\" { (bc_room(&nav, slug_display, None)) }\n+ (members_markup)\n+ h3 { \"threads\" }\n+ (render_thread_feed(Some(&nav), \"room-thread-feed\", &rows, now))\n+ @if show_new {\n+ div id=\"room-new-thread-ui-slot\" {\n+ (fragment_room_new_thread_form(&nav, true, false))\n+ }\n+ }\n+ (cli_panel(&[forum_cli, garden_cli, audit_cli]))\n+ },\n+ None,\n+ theme_from_jar(&jar),\n+ &theme_next_from_uri(&uri),\n+ );\n+ Html(page.into_string()).into_response()\n+}\n\n\nSide B — contributor: tommy-mor\nSide B — commit message:\n[3b3d5873] item refactor\n\nSide B — unified diff (full patch):\ndiff --git a/plan.md b/plan.md\ndeleted file mode 100644\nindex 00d6867a1e0ed144a16a020ea037f685ce646c73..0000000000000000000000000000000000000000\n--- a/plan.md\n+++ /dev/null\n@@ -1,155 +0,0 @@\n-# Plan: `ItemId` + `RouteContext` (identity vs hrefs)\n-\n-This document is for **the next agent** to continue the refactor without re-deriving context from chat. It supersedes ad-hoc notes: treat it as the checklist of record until the work lands and this file is deleted or trimmed.\n-\n-## Goal\n-\n-- **Identity** (what lives in the reducer graph, votes, indexes) becomes a **structural `ItemId` enum** in `slug-types`, not a canonical `String` / `CanonicalItemUrl` newtype.\n-- **Presentation** (tilde / dash display, breadcrumbs) derives from `ItemId` via explicit methods, not string stripping.\n-- **Routing** (browser `href`s for public vs room) goes through **`RouteContext`** (started in `server/src/html/routing.rs`) so Maud/handlers do not stitch `/r/…` vs `/~` ad hoc.\n-\n-**Non-goals for v1 of the migration:** backward-compatible JSONL or dual-read of old canonical strings in the event log (project has accepted breaking changes). If you reintroduce compat, document it here.\n-\n-## Current state (as of this plan)\n-\n-- **`CanonicalItemUrl`** (`types/src/paths.rs`): newtype around `String`; `parse` / `parent` / `display_path` / `tilde_tail` / etc. Reducer `ContentState`, `VoteData`, ranking, RPC, search, garden, breadcrumbs all use it or `String` keys derived from it.\n-- **`ThreadNav`** (`server/src/html/forum/nav.rs`): encodes scope prefixes for threads and garden URLs; **`RouteContext`** now wraps `ThreadNav` (`server/src/html/routing.rs`, re-exported from `server/src/html/mod.rs`) but **most HTML still takes `&ThreadNav` directly** — migration incomplete.\n-- **URL normalization** lives in `types/src/url_normalize.rs` + `canonicalize_item` / `finalize_external_identity_url` in `paths.rs` (YouTube, sorted query params, room path `room_route_segment` in `paths.rs`).\n-- **Room HTTP paths** are `/r/{short}{slug}` (fused segment); wire **`room_id`** remains `short/slug` for RPC/events.\n-\n-## Target architecture\n-\n-### `ItemId` (types)\n-\n-Suggested shape (adjust after profiling `Ord` / `Hash` / serde size):\n-\n-```text\n-ItemId::Root — tilde ontology root (today `SLUG_TILDE_ONTOLOGY_ROOT`)\n-ItemId::Local { segments } — slug.social ~/… path as Vec (lowercase segments, non-empty for non-root)\n-ItemId::External { url: Url } — normalized `url::Url` (crate `url` already in `slug-types`)\n-```\n-\n-**API surface (minimum):**\n-\n-- `ItemId::parse(&str) -> Option` — single entry from DSL / user input / legacy wire (internally may call `canonicalize_item` + structured split).\n-- `ItemId::to_wire_url(&self) -> String` — only for **external** boundaries if needed (HTTP fetch, rare assertions); avoid using as the primary key once maps use `ItemId`.\n-- `parent`, `display_path`, `tilde_tail` / `tilde_http_tail`, `tilde_segments`, `last_segment`, `normalized_storage` — port from `CanonicalItemUrl`.\n-- **`Ord` + `Hash` + `Eq`** stable for `BTreeSet` / `HashMap` (see `write_actor` scope-rank snapshots).\n-- **`Serialize` / `Deserialize`** — decide **tagged JSON** for any persisted or API-carried structs (e.g. `VoteData` in tests). If RPC must stay stringy for clients, use a **DTO layer** that converts `ItemId` ↔ wire at the boundary only.\n-\n-**Remove:** `CanonicalItemUrl` type and all `path_types::CanonicalItemUrl` / `slug_types::paths::CanonicalItemUrl` exports once call sites are migrated. **`Borrow`** on the old newtype goes away; update `nav!` / any code that assumed map keys borrowed as `str`.\n-\n-### `RouteContext` (server HTML)\n-\n-- **File:** `server/src/html/routing.rs` — **`RouteContext(ThreadNav)`** with `item_href`, `item_href_raw`, `thread_url`, `garden_root_url`, `room_url`, `From`/`Into` `ThreadNav`.\n-- **Direction:** new code and refactored Maud should take **`&RouteContext`** (or owned where appropriate) instead of `&ThreadNav` when building links. Long term, **`item_href(&ItemId)`** should not parse strings — it should pattern-match `ItemId` and append tilde tail or `/-/…` external tail using the same rules as today’s `ThreadNav::garden_item_url`.\n-\n-### Axum / garden routes\n-\n-- **No** single catch-all route (explicit decision): keep the existing router layout in `server/src/lib.rs`.\n-- Room routes stay **`/r/:room_key/...`** with `room_key` fused; parsing via `slug_types::room_id_from_route_segment` / `room_route_segment` in `paths.rs`.\n-\n-## Phased execution (recommended order)\n-\n-### Phase 0 — Preconditions (quick)\n-\n-1. Read **`AGENTS.md`** (UI contract, durability matrix, `RpcCommand` vs `HtmlUiAction`).\n-2. Run **`cargo test --workspace`** and **`./scripts/clj-test.sh`** on clean `main` before large diffs; repeat after each phase.\n-\n-### Phase 1 — `ItemId` in `slug-types` (no server yet)\n-\n-1. Add **`ItemId`** (new file e.g. `types/src/item_id.rs` **or** inline at bottom of `paths.rs` — see **Module cycle** below).\n-2. Implement **`ItemId::parse`** using existing **`canonicalize_item`** + normalization; port **`CanonicalItemUrl`** methods to **`ItemId`** with tests ported from `paths.rs` `#[cfg(test)] mod tests`.\n-3. **`GardenItemUrl::from_stored(&ItemId, room_wire)`** (and thread helpers) — build absolute hrefs from structure, not from re-parsing a canonical string.\n-4. **`TildeHttpPathTail::to_item_id`** (rename from `to_canonical`) / **`tilde_http_path_to_item_id`**.\n-5. **`TildeOntologyPath::from_stored(&ItemId)`**.\n-6. Export **`ItemId`** from **`types/src/lib.rs`**; update **`server/src/path_types.rs`** re-exports.\n-7. **Delete `CanonicalItemUrl`** and fix all **in-crate** references in `types` only until `cargo test` passes for `slug-types`.\n-\n-**Module cycle trap:** `item_id.rs` must not `use crate::paths::{...}` if `paths.rs` also imports `ItemId` for `GardenItemUrl` in the same module. **Fix one of:**\n-\n-- **A)** Put `ItemId` **inside `paths.rs`** below `canonicalize_item` / helpers (simplest, large file), or \n-- **B)** Split **`canonicalize_item`** (+ dash host helpers + `finalize_external_identity_url`) into **`types/src/item_wire.rs`**, then `paths.rs` + `item_id.rs` both depend on `item_wire` only (cleaner, more files).\n-\n-### Phase 2 — Reducer + ranking (server core)\n-\n-1. **`server/src/reducer.rs`**: `ContentState` / `GroupState` / **`VoteData`** — replace **`CanonicalItemUrl`** with **`ItemId`** on all maps, sets, deques, vectors.\n-2. **`apply_vote`**: normalize `a`/`b` via **`ItemId::parse`** or **`ItemId`**-aware logic (remove string round-trip).\n-3. **`apply_ingest_to_content`**: **`dsl`** still yields strings for item titles in statements; normalize to **`ItemId`** at ingest boundary via **`ItemId::parse`** once per item.\n-4. **`server/src/ranking.rs`**, **`server/src/scope_rank.rs`**, **`server/src/api/write_actor.rs`** (including **`BTreeSet`** ordering), **`server/src/api/validate.rs`**, **`server/src/api/helpers.rs`** — propagate **`ItemId`**.\n-5. **`server/tests/basic.rs`** and any reducer tests constructing **`VoteData`** — use **`ItemId::parse(...).unwrap()`** or helpers.\n-\n-### Phase 3 — RPC + search + external resolver\n-\n-1. **`server/src/api/rpc.rs`**: rank/pair/matchup/search payloads; today many paths use **`GardenItemUrl::from_storage_str(item.as_str(), …)`** — switch to **`ItemId`** + **`GardenItemUrl::from_stored(&item_id, …)`** (or equivalent).\n-2. **`server/src/html/search.rs`**: scoring uses item path strings — derive from **`ItemId::display_path`** / **`to_wire_url`** only at the scoring boundary if needed.\n-3. **`server/src/external_resolver.rs`**: take **`&ItemId`** or **`ItemId::external_url()`** instead of **`&CanonicalItemUrl`**.\n-\n-### Phase 4 — HTML / Maud\n-\n-1. **`ThreadNav::garden_item_url`**: overload or replace with **`garden_item_href(&self, item: &ItemId)`** (no `CanonicalItemUrl::parse` inside).\n-2. **`RouteContext`**: extend **`item_href(&ItemId)`**; migrate call sites from **`ThreadNav`** to **`RouteContext`** where only link-building is needed (keep **`ThreadNav`** where scope / auth helpers need the full struct).\n-3. **`server/src/html/garden.rs`**, **`breadcrumb_path.rs`**, **`forum/*`**, **`editor.rs`**: replace **`CanonicalItemUrl`** with **`ItemId`**; breadcrumbs should walk **`ItemId::parent`** without string `rsplit`.\n-4. **`types` JSON types** (`RankRow`, etc.): decide whether **`GardenItemUrl`** stays string for JSON or becomes a structured field; keep **one** wire format for the public API.\n-\n-### Phase 5 — Cleanup + docs\n-\n-1. Remove dead **`canonical_path`** / **`breadcrumb_path`** string logic if fully superseded.\n-2. Update **`AGENTS.md`** if durability, `POST /ui`, or command surfaces change.\n-3. Delete or shrink **`plan.md`** when done.\n-\n-## File / symbol checklist (non-exhaustive — grep-driven)\n-\n-Run periodically:\n-\n-```bash\n-rg \"CanonicalItemUrl\" -g'*.rs'\n-rg \"path_types::CanonicalItemUrl\" -g'*.rs'\n-rg \"tilde_http_path_to_canonical\" -g'*.rs'\n-```\n-\n-**High-touch files (from prior exploration):**\n-\n-| Area | Files |\n-|------|--------|\n-| Types | `types/src/paths.rs`, `types/src/lib.rs`, `types/src/url_normalize.rs`, (optional) `types/src/item_id.rs`, `types/src/item_wire.rs` |\n-| Server re-exports | `server/src/path_types.rs`, `server/src/canonical_path.rs` |\n-| Reducer / ingest | `server/src/reducer.rs`, `server/src/dsl.rs` (parse output types if changed) |\n-| Ranking | `server/src/ranking.rs`, `server/src/scope_rank.rs` |\n-| Writer / RPC | `server/src/api/write_actor.rs`, `server/src/api/rpc.rs`, `server/src/api/helpers.rs`, `server/src/api/validate.rs` |\n-| HTML | `server/src/html/garden.rs`, `server/src/html/breadcrumb_path.rs`, `server/src/html/forum/nav.rs`, `server/src/html/routing.rs`, `server/src/html/search.rs`, `server/src/html/editor.rs`, `server/src/html/forum/ingest.rs`, … |\n-| Tests | `server/tests/basic.rs`, `server/tests/integration.rs`, `types/src/paths.rs` tests, Clojure under `test/` if URLs/assertions mention canonical shapes |\n-\n-## Events / JSONL\n-\n-- **`Ingest`** events store **`raw` DSL** only — no change required for item identity inside the event.\n-- If any future event type stores item ids as strings, migrate to **structured `ItemId` serde** or accept string only at the event boundary with immediate parse into **`ItemId`** on `apply_event`.\n-\n-## `nav!` macro (`server/src/paths.rs`)\n-\n-- Macros use **`keypath($key)`** with **`.clone()`** — **`ItemId`** must be **`Clone`** (already for enums). Remove any reliance on **`Borrow`** for map keys.\n-\n-## Testing gate\n-\n-After each phase:\n-\n-```bash\n-cargo test --workspace\n-./scripts/clj-test.sh\n-```\n-\n-## Risks / gotchas\n-\n-1. **`Ord` on `ItemId`**: must match prior **`CanonicalItemUrl`** / `String` ordering wherever **`BTreeSet`** is used (e.g. deterministic scope-rank snapshots in **`write_actor`**).\n-2. **External `ItemId`**: **`Url`** equality / hashing — normalization is already centralized in **`url_normalize`**; ensure **`ItemId::parse`** always inserts normalized **`Url`** into **`External`**.\n-3. **Fake parent URLs** in garden (e.g. **`https://.`** for external root ranking): find all **`parse(\"https://.\")`** style hacks and express as **`ItemId`** or a dedicated sentinel.\n-4. **Serde**: tests and any RPC clients that snapshot JSON may need expectation updates if **`VoteData`** shape changes.\n-\n-## Optional follow-ups (not blocking `ItemId`)\n-\n-- More **domain normalizers** in **`url_normalize.rs`** (e.g. `music.youtube.com`, Spotify, etc.).\n-- **Room wire** vs **HTTP segment** helpers already in **`paths.rs`** (`ROOM_SHORT_ID_LEN`, `room_route_segment`, `room_id_from_route_segment`).\n-\n----\n-\n-**End state criteria:** `rg CanonicalItemUrl` returns nothing; reducer maps use **`ItemId`**; HTML link generation for items goes through **`RouteContext` + `ItemId`**; tests and Kaocha green.\ndiff --git a/server/src/api/helpers.rs b/server/src/api/helpers.rs\nindex 1b291db83df7364a026f2e147e0a29a70a399371..8cd23a02fa5219d6aa375e766e6b2bd2c7bb7dfb 100644\n--- a/server/src/api/helpers.rs\n+++ b/server/src/api/helpers.rs\n@@ -4,7 +4,8 @@ use axum::{\n Json,\n };\n use sha2::{Digest, Sha256};\n-use slug_types::paths::{CanonicalItemUrl, GardenItemUrl};\n+use slug_types::paths::GardenItemUrl;\n+use slug_types::ItemId;\n use slug_types::*;\n use std::collections::HashMap;\n \n@@ -31,12 +32,12 @@ pub fn now_ms() -> i64 {\n }\n \n /// Resolve DSL/user input to a stored canonical item id.\n-pub fn resolve_item(item: &str) -> Result {\n+pub fn resolve_item(item: &str) -> Result {\n let canonical = canonicalize_item(item);\n if canonical.is_empty() {\n return Err(format!(\"empty item path: `{}`\", item));\n }\n- Ok(CanonicalItemUrl(canonical))\n+ ItemId::parse(&canonical).ok_or_else(|| format!(\"invalid item path: `{}`\", item))\n }\n \n pub fn parse_parent_specs(parent: Option<&String>) -> Vec {\n@@ -94,7 +95,7 @@ pub fn paginate_rankings(\n (out_components, out_unranked)\n }\n \n-pub fn pick_random_distinct_canonical(items: &[CanonicalItemUrl]) -> Option<(CanonicalItemUrl, CanonicalItemUrl)> {\n+pub fn pick_random_distinct_canonical(items: &[ItemId]) -> Option<(ItemId, ItemId)> {\n use rand::seq::SliceRandom;\n if items.len() < 2 {\n return None;\n@@ -115,15 +116,15 @@ pub fn pick_random_distinct_canonical(items: &[CanonicalItemUrl]) -> Option<(Can\n }\n \n pub fn is_pair_voted(group: &crate::reducer::GroupState, a: &str, b: &str) -> bool {\n- let a_key = CanonicalItemUrl(a.to_string());\n- let b_key = CanonicalItemUrl(b.to_string());\n+ let a_key = ItemId::parse(a).unwrap_or_else(|| ItemId::opaque(a.to_string()));\n+ let b_key = ItemId::parse(b).unwrap_or_else(|| ItemId::opaque(b.to_string()));\n let Some(&a_idx) = group.item_to_idx.get(&a_key) else { return false; };\n let Some(&b_idx) = group.item_to_idx.get(&b_key) else { return false; };\n let (i, j) = if a_idx < b_idx { (a_idx, b_idx) } else { (b_idx, a_idx) };\n group.voted_pairs.contains(&(i, j))\n }\n \n-pub fn compute_connectivity_stats(group: &crate::reducer::GroupState, pool: &[CanonicalItemUrl]) -> ConnectivityStats {\n+pub fn compute_connectivity_stats(group: &crate::reducer::GroupState, pool: &[ItemId]) -> ConnectivityStats {\n let n = pool.len();\n \n let global_idxs: Vec> = pool\ndiff --git a/server/src/api/rpc.rs b/server/src/api/rpc.rs\nindex 0ff2d701dd1e8661f58abd55672cd91280e9491e..079d96eb6f1d717c203b1d4aec09f3916384b919 100644\n--- a/server/src/api/rpc.rs\n+++ b/server/src/api/rpc.rs\n@@ -17,7 +17,7 @@ use crate::{\n dsl,\n events::{Event, Ingest, ThreadCapability},\n identity::{parse_agent, parse_username},\n- path_types::CanonicalItemUrl,\n+ path_types::ItemId,\n ranking::{connected_components_from_voted_pairs, ranked_items_subset},\n reducer::{scope_from_room_wire, ReducerState, ScopeId},\n state::{AppState, InviteState},\n@@ -150,7 +150,7 @@ fn build_rank_response_for_content(\n \n if !is_global && !specs.is_empty() {\n let none_exist = specs.iter().all(|spec| {\n- let Some(canon) = CanonicalItemUrl::parse(spec) else { return true };\n+ let Some(canon) = ItemId::parse(spec) else { return true };\n !content.items.contains(&canon) && !content.item_children.contains_key(&canon)\n });\n if none_exist {\n@@ -163,10 +163,10 @@ fn build_rank_response_for_content(\n \n let depth = depth.max(1);\n let rankings = if is_global {\n- let all_items: Vec = content.items.iter().cloned().collect();\n+ let all_items: Vec = content.items.iter().cloned().collect();\n crate::scope_rank::build_rankings_for_item_set(content, &all_items)\n } else if specs.is_empty() {\n- crate::scope_rank::build_children_rankings(content, &CanonicalItemUrl::ontology_root())\n+ crate::scope_rank::build_children_rankings(content, &ItemId::ontology_root())\n } else if depth > 1 {\n let items = crate::scope_rank::resolve_scope_recursive(content, &specs, depth);\n crate::scope_rank::build_rankings_for_item_set(content, &items)\n@@ -359,8 +359,8 @@ async fn rpc_check(\n let mut simulated = { reduced_arc.read().await.clone() };\n simulated.apply_event(event);\n \n- let voted_parents: Vec = {\n- let mut parents: HashSet = HashSet::new();\n+ let voted_parents: Vec = {\n+ let mut parents: HashSet = HashSet::new();\n for s in &v.doc.statements {\n if let dsl::Stmt::Vote { item1, item2, .. } = s {\n if let (Ok(a), Ok(b)) = (resolve_item(item1), resolve_item(item2)) {\n@@ -369,7 +369,7 @@ async fn rpc_check(\n }\n }\n }\n- let mut out: Vec = parents.into_iter().collect();\n+ let mut out: Vec = parents.into_iter().collect();\n out.sort();\n out\n };\n@@ -695,7 +695,7 @@ fn rpc_search(reduced: &ReducerState, q: &str, limit: usize, principal: Option<&\n async fn rpc_get_pair(state: &AppState, room: String, parent_path: String) -> Result {\n let scope = scope_from_room_wire(&room);\n let reduced_arc = state.reduced.clone();\n- let pool: Vec = {\n+ let pool: Vec = {\n let reduced = reduced_arc.read().await;\n let content = content_for_room(&reduced, &room);\n let tmp = if parent_path.trim().is_empty() {\n@@ -716,7 +716,7 @@ async fn rpc_get_pair(state: &AppState, room: String, parent_path: String) -> Re\n Some(\"add items via ingest\".into()),\n ));\n }\n- let selected: Option<(CanonicalItemUrl, CanonicalItemUrl)> = {\n+ let selected: Option<(ItemId, ItemId)> = {\n let mut reduced = reduced_arc.write().await;\n let content = reduced.content.entry(scope.clone()).or_default();\n let group = &mut content.ranking_group;\n@@ -729,16 +729,16 @@ async fn rpc_get_pair(state: &AppState, room: String, parent_path: String) -> Re\n .filter_map(|it| group.item_to_idx.get(it).copied())\n .collect();\n let ranked = ranked_items_subset(group, &idxs, 10000, 1e-8);\n- let ranked_set: HashSet = ranked.iter().map(|r| r.item.clone()).collect();\n- let unsorted: Vec = pool\n+ let ranked_set: HashSet = ranked.iter().map(|r| r.item.clone()).collect();\n+ let unsorted: Vec = pool\n .iter()\n .filter(|it| !ranked_set.contains(*it))\n .cloned()\n .collect();\n- let mut pick: Option<(CanonicalItemUrl, CanonicalItemUrl)> = None;\n+ let mut pick: Option<(ItemId, ItemId)> = None;\n if !unsorted.is_empty() {\n if let Some(left) = unsorted.choose(&mut rng).cloned() {\n- let mut candidates: Vec = if !ranked.is_empty() {\n+ let mut candidates: Vec = if !ranked.is_empty() {\n ranked.iter().map(|r| r.item.clone()).collect()\n } else {\n pool.clone()\n@@ -865,7 +865,7 @@ pub async fn handle_rpc_batch(\n } else {\n let content = content_for_room(&reduced, &room);\n let item_str = canonicalize_item(&item_path);\n- let item = CanonicalItemUrl(item_str.clone());\n+ let item = ItemId::parse(&item_str).unwrap_or_else(|| ItemId::opaque(item_str.clone()));\n if !content.items.contains(&item) {\n line_err(\n \"item not found\",\n@@ -1212,7 +1212,7 @@ pub async fn handle_rpc_batch(\n }\n \n let ranked_total = ranked.len();\n- let mut unranked: Vec = content\n+ let mut unranked: Vec = content\n .items\n .iter()\n .filter(|it| !group.item_to_idx.contains_key(*it))\n@@ -1263,7 +1263,7 @@ pub async fn handle_rpc_batch(\n } else {\n let content = content_for_room(&reduced, &room);\n let item_str = canonicalize_item(&item_path);\n- let item = CanonicalItemUrl(item_str.clone());\n+ let item = ItemId::parse(&item_str).unwrap_or_else(|| ItemId::opaque(item_str.clone()));\n let limit = limit.unwrap_or(50).clamp(1, 200);\n if !content.items.contains(&item) {\n line_err(\n@@ -1304,7 +1304,7 @@ pub async fn handle_rpc_batch(\n let content = content_for_room(&reduced, &room);\n let scope = scope_from_room_wire(&room);\n let item_str = canonicalize_item(&item_path);\n- let item = CanonicalItemUrl(item_str.clone());\n+ let item = ItemId::parse(&item_str).unwrap_or_else(|| ItemId::opaque(item_str.clone()));\n let entries = content.rank_history.get(&item).cloned().unwrap_or_default();\n let history: Vec = entries.iter().map(|e| {\n let caused_by: Vec = reduced.ingests_by_id.get(&e.post_id)\n@@ -1361,7 +1361,11 @@ pub async fn handle_rpc_batch(\n line_err(e, h)\n } else {\n let content = content_for_room(&reduced, &room);\n- let parents: HashSet<&str> = content.item_children.keys().map(|s| s.as_str()).collect();\n+ let parents: HashSet = content\n+ .item_children\n+ .keys()\n+ .map(|k| k.to_storage_string())\n+ .collect();\n let mut paths: Vec = content\n .items\n .iter()\n@@ -1380,11 +1384,11 @@ pub async fn handle_rpc_batch(\n let content = content_for_room(&reduced, &room);\n let out: Vec = content\n .item_children\n- .get(&CanonicalItemUrl::ontology_root())\n+ .get(&ItemId::ontology_root())\n .map(|roots| {\n let mut v: Vec = roots.iter()\n .map(|path| {\n- let children = content.item_children.get(path.as_str()).map(|s| s.len()).unwrap_or(0);\n+ let children = content.item_children.get(path).map(|s| s.len()).unwrap_or(0);\n PathSummary {\n path: TildeOntologyPath::from_stored(path),\n children,\ndiff --git a/server/src/api/validate.rs b/server/src/api/validate.rs\nindex 3c7aa80fd485cf247231fe5c5c7e6fea62f22534..3f657a88ef9328a31e3e29c3bc3bd32bec6fd7a1 100644\n--- a/server/src/api/validate.rs\n+++ b/server/src/api/validate.rs\n@@ -4,7 +4,7 @@ use std::collections::HashSet;\n use crate::{\n canonical_path::canonicalize_tag,\n dsl,\n- path_types::CanonicalItemUrl,\n+ path_types::ItemId,\n reducer::{ReducerState, ScopeId},\n };\n use slug_types::paths::GardenItemUrl;\n@@ -32,11 +32,11 @@ pub fn validate_ingest_document(\n ScopeId::Public => None,\n _ => reduced.content_for_scope(scope),\n };\n- let item_exists = |key: &CanonicalItemUrl| {\n+ let item_exists = |key: &ItemId| {\n scoped_content.map(|c| c.items.contains(key)).unwrap_or(false)\n || public_content.items.contains(key)\n };\n- let body_exists = |key: &CanonicalItemUrl| {\n+ let body_exists = |key: &ItemId| {\n scoped_content.map(|c| c.item_bodies.contains_key(key)).unwrap_or(false)\n || public_content.item_bodies.contains_key(key)\n };\n@@ -52,7 +52,7 @@ pub fn validate_ingest_document(\n };\n \n let ts = super::helpers::now_ms();\n- let mut defined_in_doc: HashSet = HashSet::new();\n+ let mut defined_in_doc: HashSet = HashSet::new();\n \n for s in &doc.statements {\n match s {\ndiff --git a/server/src/api/write_actor.rs b/server/src/api/write_actor.rs\nindex f9c3b8bd3fbf8fcb9c035e1a1572fef0b08fa8a9..3e96b015def54921962f74b4cd884808ff6c3bcc 100644\n--- a/server/src/api/write_actor.rs\n+++ b/server/src/api/write_actor.rs\n@@ -10,7 +10,7 @@ use crate::{\n events::{AgentBound, Event, GrantAdded, Ingest, PostRedacted, RoomDeleted, UserRegistered},\n html::JsBuilder,\n identity::parse_agent,\n- path_types::CanonicalItemUrl,\n+ path_types::ItemId,\n reducer::{scope_from_room_wire, ReducerState, ScopeId},\n state::AppState,\n write_cmd::WriteCmd,\n@@ -90,7 +90,7 @@ async fn broadcast_web_refresh(state: &AppState, room_key: &str, thread_id: &str\n }\n \n fn compute_scope_rank_changes(\n- parent: &CanonicalItemUrl,\n+ parent: &ItemId,\n before: &crate::scope_rank::ChildrenRankings,\n after: &crate::scope_rank::ChildrenRankings,\n room_wire: &str,\n@@ -99,7 +99,7 @@ fn compute_scope_rank_changes(\n use std::collections::BTreeSet;\n fn build_positions(\n rankings: &crate::scope_rank::ChildrenRankings,\n- ) -> HashMap> {\n+ ) -> HashMap> {\n let mut map = HashMap::new();\n for comp in &rankings.component_rankings {\n let total = comp.ranked.len();\n@@ -116,7 +116,7 @@ fn compute_scope_rank_changes(\n let before_pos = build_positions(before);\n let after_pos = build_positions(after);\n \n- let all_items: BTreeSet = before_pos\n+ let all_items: BTreeSet = before_pos\n .keys()\n .cloned()\n .chain(after_pos.keys().cloned())\n@@ -287,8 +287,8 @@ pub async fn writer_actor(mut rx: mpsc::Receiver, state: AppState) {\n .map(|d| reduced.agent_bindings.get(d).is_none())\n .unwrap_or(false);\n \n- let voted_parent_scopes: Vec = {\n- let mut parents: HashSet = HashSet::new();\n+ let voted_parent_scopes: Vec = {\n+ let mut parents: HashSet = HashSet::new();\n for s in &v.doc.statements {\n if let dsl::Stmt::Vote { item1, item2, .. } = s {\n if let (Ok(a), Ok(b)) = (resolve_item(item1), resolve_item(item2)) {\n@@ -301,12 +301,12 @@ pub async fn writer_actor(mut rx: mpsc::Receiver, state: AppState) {\n }\n }\n }\n- let mut out: Vec = parents.into_iter().collect();\n+ let mut out: Vec = parents.into_iter().collect();\n out.sort();\n out\n };\n \n- let pre_rankings: HashMap =\n+ let pre_rankings: HashMap =\n if !voted_parent_scopes.is_empty() {\n let content = content_for_room(&reduced, &room_key);\n voted_parent_scopes\ndiff --git a/server/src/external_resolver.rs b/server/src/external_resolver.rs\nindex 87409c4cab5f1930d2cd075f3417fa824fd791c7..6f5c982a627f9f620f0e3be0ba0cb92a3d7d4bb7 100644\n--- a/server/src/external_resolver.rs\n+++ b/server/src/external_resolver.rs\n@@ -1,6 +1,6 @@\n use async_trait::async_trait;\n \n-use crate::path_types::CanonicalItemUrl;\n+use crate::path_types::ItemId;\n \n #[async_trait]\n pub trait ExternalResolver: Send + Sync {\n@@ -11,7 +11,7 @@ pub trait ExternalResolver: Send + Sync {\n fn normalize(&self, path: &str) -> String;\n \n /// Fetches body when missing; GitHub hook lands here in a follow-up.\n- async fn fetch_body(&self, canonical_url: &CanonicalItemUrl) -> Result;\n+ async fn fetch_body(&self, canonical_url: &ItemId) -> Result;\n }\n \n /// Placeholder until domain-specific resolvers exist.\n@@ -27,7 +27,7 @@ impl ExternalResolver for DefaultExternalResolver {\n path.to_string()\n }\n \n- async fn fetch_body(&self, _canonical_url: &CanonicalItemUrl) -> Result {\n+ async fn fetch_body(&self, _canonical_url: &ItemId) -> Result {\n Err(\"external fetch not implemented\".to_string())\n }\n }\ndiff --git a/server/src/html/breadcrumb_path.rs b/server/src/html/breadcrumb_path.rs\nindex c8a3937923161a6ff248bd77e87eff0dc6fe9ab0..5743a98753f579c70e10961278469afd7cb9ddcf 100644\n--- a/server/src/html/breadcrumb_path.rs\n+++ b/server/src/html/breadcrumb_path.rs\n@@ -1,8 +1,8 @@\n-use crate::path_types::{tilde_http_path_to_canonical, CanonicalItemUrl};\n+use crate::path_types::{tilde_http_path_to_item_id, ItemId};\n \n /// Semantic view of an ontology path for rendering and routing decisions.\n pub(super) struct OntologyPath {\n- canonical: CanonicalItemUrl,\n+ canonical: ItemId,\n /// Breadcrumb segments: for `~/a/b` this is `[\"a\", \"b\"]` (leading `~` rendered separately).\n segments: Vec,\n }\n@@ -11,11 +11,11 @@ impl OntologyPath {\n /// Path is the `*path` segment from `/~/*path` (e.g. `topic/a`). Always treat it as under `~/`\n /// so it canonicalizes to `https://slug.social/~/…`, not the non-tilde site path.\n pub(super) fn from_input(path: &str) -> Self {\n- let canonical = tilde_http_path_to_canonical(path);\n+ let canonical = tilde_http_path_to_item_id(path);\n Self::from_canonical(canonical)\n }\n \n- pub(super) fn from_canonical(canonical: CanonicalItemUrl) -> Self {\n+ pub(super) fn from_canonical(canonical: ItemId) -> Self {\n // tilde_segments() returns [\"~\", \"a\", \"b\"] but bc_path() renders \"~\" itself,\n // so we skip the leading \"~\" segment here.\n let segments = canonical\n@@ -28,7 +28,7 @@ impl OntologyPath {\n }\n \n pub(super) fn root() -> Self {\n- Self::from_canonical(CanonicalItemUrl::ontology_root())\n+ Self::from_canonical(ItemId::ontology_root())\n }\n \n pub(super) fn is_root(&self) -> bool {\n@@ -56,7 +56,7 @@ impl OntologyPath {\n \n /// External `https://host/…` items addressed as `/-/host/…` in the URL bar.\n pub(super) struct ExternalOntologyPath {\n- canonical: CanonicalItemUrl,\n+ canonical: ItemId,\n /// e.g. `[\"github.com\", \"org\", \"repo\", \"issues\"]`\n segments: Vec,\n }\n@@ -71,13 +71,13 @@ impl ExternalOntologyPath {\n } else {\n format!(\"-/{}\", p.trim_start_matches('/'))\n };\n- let Some(canonical) = CanonicalItemUrl::parse(&raw) else {\n- return Self::from_canonical(CanonicalItemUrl(\"https://.\".to_string()));\n+ let Some(canonical) = ItemId::parse(&raw) else {\n+ return Self::from_canonical(ItemId::opaque(\"https://.\".to_string()));\n };\n Self::from_canonical(canonical)\n }\n \n- pub(super) fn from_canonical(canonical: CanonicalItemUrl) -> Self {\n+ pub(super) fn from_canonical(canonical: ItemId) -> Self {\n let s = canonical.as_str();\n let rest = s\n .strip_prefix(\"https://\")\ndiff --git a/server/src/html/editor.rs b/server/src/html/editor.rs\nindex 80657ac0df9b4bb67e4cdacca296f70267840bd1..c76417acd7a164fda23537cd4e7fb9f1f1dcd895 100644\n--- a/server/src/html/editor.rs\n+++ b/server/src/html/editor.rs\n@@ -97,7 +97,7 @@ pub async fn editor_check(\n simulated.apply_event(event);\n \n // Collect voted parent scopes.\n- let voted_parents: Vec = {\n+ let voted_parents: Vec = {\n let mut parents = std::collections::HashSet::new();\n for s in &v.doc.statements {\n if let crate::dsl::Stmt::Vote { item1, item2, .. } = s {\n@@ -107,7 +107,7 @@ pub async fn editor_check(\n }\n }\n }\n- let mut out: Vec = parents.into_iter().collect();\n+ let mut out: Vec = parents.into_iter().collect();\n out.sort();\n out\n };\ndiff --git a/server/src/html/forum/nav.rs b/server/src/html/forum/nav.rs\nindex 48fe11e46731670874ff8b6b05baa6f09ae0b7e4..7ca5b491f38104ec8c81db43a95c59da00abd8c4 100644\n--- a/server/src/html/forum/nav.rs\n+++ b/server/src/html/forum/nav.rs\n@@ -52,9 +52,14 @@ impl ThreadNav {\n }\n \n pub(crate) fn garden_item_url(&self, item: &str) -> String {\n- let Some(c) = crate::path_types::CanonicalItemUrl::parse(item) else {\n+ let Some(c) = crate::path_types::ItemId::parse(item) else {\n return format!(\"{}/{}\", self.garden_path_prefix, canonicalize_item(item));\n };\n+ self.garden_item_href(&c)\n+ }\n+\n+ /// Relative href for a structured [`crate::path_types::ItemId`] in this scope’s garden.\n+ pub(crate) fn garden_item_href(&self, c: &crate::path_types::ItemId) -> String {\n if let Some(tail) = c.tilde_tail().map(str::to_owned) {\n format!(\"{}/{}\", self.garden_path_prefix, tail)\n } else if c.as_str().starts_with(\"http://\") || c.as_str().starts_with(\"https://\") {\n@@ -63,7 +68,7 @@ impl ThreadNav {\n let ext_prefix = format!(\"{}-\", self.garden_path_prefix.trim_end_matches('~'));\n format!(\"{}/{}\", ext_prefix, rest)\n } else {\n- format!(\"{}/{}\", self.garden_path_prefix, canonicalize_item(item))\n+ format!(\"{}/{}\", self.garden_path_prefix, canonicalize_item(c.as_str()))\n }\n }\n \ndiff --git a/server/src/html/garden.rs b/server/src/html/garden.rs\nindex e615dd356bcf634232d85610c0a26235ead125fd..2af9ac5bfdee1630883e6f8257883baeafcb5a44 100644\n--- a/server/src/html/garden.rs\n+++ b/server/src/html/garden.rs\n@@ -10,7 +10,7 @@ use crate::{\n api::optional_principal,\n canonical_path::canonicalize_item,\n events::ThreadCapability,\n- path_types::CanonicalItemUrl,\n+ path_types::ItemId,\n reducer::{ContentState, ReducerState, ScopeId},\n ranking::{connected_components_from_voted_pairs, ranked_items_subset},\n scope_rank::{build_children_rankings, ChildrenRankings},\n@@ -27,7 +27,7 @@ use super::{\n \n /// Display path for an item: `~/…` or `-/…` form.\n fn item_display_path(item: &str) -> String {\n- CanonicalItemUrl::parse(item)\n+ ItemId::parse(item)\n .map(|c| c.display_path())\n .unwrap_or_else(|| canonicalize_item(item))\n }\n@@ -159,7 +159,7 @@ pub async fn garden_index(\n let nav = ThreadNav::public();\n let child_rankings = {\n let reduced = state.reduced.read().await;\n- build_children_rankings(reduced.public(), &CanonicalItemUrl::ontology_root())\n+ build_children_rankings(reduced.public(), &ItemId::ontology_root())\n };\n \n let page = layout(\n@@ -236,7 +236,7 @@ pub async fn external_garden_index(\n ) -> impl IntoResponse {\n let nav = ThreadNav::public();\n let ext_path = ExternalOntologyPath::from_input(\"\");\n- let parent = CanonicalItemUrl::parse(\"https://.\").unwrap();\n+ let parent = ItemId::parse(\"https://.\").unwrap();\n let child_rankings = {\n let reduced = state.reduced.read().await;\n build_children_rankings(reduced.public(), &parent)\n@@ -365,7 +365,7 @@ pub async fn room_external_garden_index(\n return room_not_found_page(&jar, &uri).into_response();\n }\n let ext_path = ExternalOntologyPath::from_input(\"\");\n- let parent = CanonicalItemUrl::parse(\"https://.\").unwrap();\n+ let parent = ItemId::parse(\"https://.\").unwrap();\n let child_rankings = build_children_rankings(\n content_for_garden_view(&reduced, &nav.scope()),\n &parent,\n@@ -509,13 +509,13 @@ struct ItemPageViewModel {\n fn build_sibling_rank(\n reduced: &crate::reducer::ReducerState,\n scope: &ScopeId,\n- item: &CanonicalItemUrl,\n+ item: &ItemId,\n ) -> Option {\n let item = item.clone().normalized_storage();\n let content = content_for_garden_view(reduced, scope);\n let group = &content.ranking_group;\n let parent = item.parent()?.normalized_storage();\n- let siblings: Vec = content\n+ let siblings: Vec = content\n .item_children\n .get(&parent)\n .map(|s| s.iter().cloned().collect())\n@@ -574,7 +574,7 @@ fn build_rank_history(\n item: &str,\n ) -> Vec {\n let content = content_for_garden_view(reduced, scope);\n- let item_key = CanonicalItemUrl(item.to_string());\n+ let item_key = ItemId::parse(item).unwrap_or_else(|| ItemId::opaque(item.to_string()));\n let entries = match content.rank_history.get(&item_key) {\n None => return vec![],\n Some(e) => e,\n@@ -592,8 +592,8 @@ fn build_rank_history(\n if a_str == item || b_str == item {\n Some(crate::reducer::VoteData {\n ts: e.ts,\n- a: CanonicalItemUrl(a_str),\n- b: CanonicalItemUrl(b_str),\n+ a: ItemId::parse(&a_str).unwrap_or_else(|| ItemId::opaque(a_str)),\n+ b: ItemId::parse(&b_str).unwrap_or_else(|| ItemId::opaque(b_str)),\n ratio_left, ratio_right,\n body: explanation,\n principal: reduced.ingests_by_id.get(&e.post_id)\n@@ -629,8 +629,8 @@ fn build_item_page_view_model(\n item: &str,\n ) -> ItemPageViewModel {\n let content = content_for_garden_view(reduced, scope);\n- let item_key = CanonicalItemUrl::parse(item)\n- .unwrap_or_else(|| CanonicalItemUrl::parse(\"~/\").unwrap())\n+ let item_key = ItemId::parse(item)\n+ .unwrap_or_else(|| ItemId::parse(\"~/\").unwrap())\n .normalized_storage();\n let item_has_parent = item_key.parent().is_some();\n let child_rankings = build_children_rankings(content, &item_key);\n@@ -925,10 +925,10 @@ mod tests {\n .map(|r| r.item.as_str())\n .collect();\n assert_eq!(names, vec![\"https://slug.social/~/topic/a\", \"https://slug.social/~/topic/b\"]);\n- use crate::path_types::CanonicalItemUrl;\n+ use crate::path_types::ItemId;\n assert!(\n- model.child_rankings.unranked_items.contains(&CanonicalItemUrl(\"https://slug.social/~/topic/kid1\".to_string()))\n- || model.child_rankings.unranked_items.contains(&CanonicalItemUrl(\"https://slug.social/~/topic/kid2\".to_string()))\n+ model.child_rankings.unranked_items.contains(&ItemId::parse(\"https://slug.social/~/topic/kid1\").unwrap())\n+ || model.child_rankings.unranked_items.contains(&ItemId::parse(\"https://slug.social/~/topic/kid2\").unwrap())\n );\n }\n \n@@ -941,8 +941,8 @@ mod tests {\n \"9ab12cd/my-room\",\n \"@00000000-0000-0000-0000-000000000000:test:local/test\\n~/t1 {a}\\n~/t2 {b}\\n\",\n );\n- use crate::path_types::CanonicalItemUrl;\n- let root = CanonicalItemUrl::ontology_root();\n+ use crate::path_types::ItemId;\n+ let root = ItemId::ontology_root();\n let model = build_item_page_view_model(\n &reduced,\n &ScopeId::Room(\"9ab12cd/my-room\".to_string()),\n@@ -971,8 +971,8 @@ mod tests {\n \"@00000000-0000-0000-0000-000000000000:test:local/test\\n\\\n ~/a {a}\\n~/b {b}\\n~/a 2:1 ~/b {because}\\n\",\n );\n- use crate::path_types::CanonicalItemUrl;\n- let root = CanonicalItemUrl::ontology_root();\n+ use crate::path_types::ItemId;\n+ let root = ItemId::ontology_root();\n let model = build_item_page_view_model(\n &reduced,\n &ScopeId::Room(\"9ab12cd/my-room\".to_string()),\ndiff --git a/server/src/html/routing.rs b/server/src/html/routing.rs\nindex 13e9ae0cbe32d454e34a7737edf8234d2a558502..df545a8195aa97484ef0012cff31d802ac67d9df 100644\n--- a/server/src/html/routing.rs\n+++ b/server/src/html/routing.rs\n@@ -4,7 +4,7 @@\n //! Prefer `RouteContext::item_href` / [`RouteContext::thread_url`] in new Maud over stitching\n //! `/r/…` vs `/~` manually. Call sites can migrate incrementally from passing `&ThreadNav`.\n \n-use crate::path_types::CanonicalItemUrl;\n+use crate::path_types::ItemId;\n \n use super::forum::ThreadNav;\n \n@@ -32,9 +32,9 @@ impl RouteContext {\n self.0\n }\n \n- /// Relative path for a stored canonical item in this scope’s garden.\n- pub fn item_href(&self, item: &CanonicalItemUrl) -> String {\n- self.0.garden_item_url(item.as_str())\n+ /// Relative path for a stored item in this scope’s garden.\n+ pub fn item_href(&self, item: &ItemId) -> String {\n+ self.0.garden_item_href(item)\n }\n \n /// Same as [`Self::item_href`] but parses `item` first (raw DSL / user paste).\ndiff --git a/server/src/path_types.rs b/server/src/path_types.rs\nindex 361a9d446c9043cbdea5f060db2e8633c2fd9bf9..a6e0b90a1aaead0bc1f7e0f4c5296506edf3f7ac 100644\n--- a/server/src/path_types.rs\n+++ b/server/src/path_types.rs\n@@ -1,5 +1,6 @@\n-//! Re-exports — implementations live in `slug-types` (`paths` module).\n+//! Re-exports — implementations live in `slug-types` (`paths` / `item_id` modules).\n \n+pub use slug_types::ItemId;\n pub use slug_types::paths::{\n- tilde_http_path_to_canonical, CanonicalItemUrl, RelativePath, TildeHttpPathTail, TildePath,\n+ tilde_http_path_to_item_id, RelativePath, TildeHttpPathTail, TildePath,\n };\ndiff --git a/server/src/ranking.rs b/server/src/ranking.rs\nindex b24e80ddd5f34b4e454bb81b541e8f54bd97a541..3710c9f64437f5bef3b2121905b6f3bcb7611047 100644\n--- a/server/src/ranking.rs\n+++ b/server/src/ranking.rs\n@@ -1,11 +1,11 @@\n use std::collections::HashMap;\n \n-use crate::path_types::CanonicalItemUrl;\n+use crate::path_types::ItemId;\n use crate::reducer::GroupState;\n \n #[derive(Debug, Clone)]\n pub struct RankedItem {\n- pub item: CanonicalItemUrl,\n+ pub item: ItemId,\n pub score: f64,\n }\n \n@@ -239,7 +239,7 @@ pub fn group_summary_scores(\n group: &mut GroupState,\n max_iters: usize,\n tol: f64,\n-) -> HashMap {\n+) -> HashMap {\n ranked_items(group, max_iters, tol)\n .into_iter()\n .map(|r| (r.item, r.score))\n@@ -256,11 +256,11 @@ mod tests {\n }\n \n fn vote(ts: i64, a: &str, b: &str, l: i32, r: i32) -> VoteData {\n- use crate::path_types::CanonicalItemUrl;\n+ use crate::path_types::ItemId;\n VoteData {\n ts,\n- a: CanonicalItemUrl(a.to_string()),\n- b: CanonicalItemUrl(b.to_string()),\n+ a: ItemId::parse(a).unwrap(),\n+ b: ItemId::parse(b).unwrap(),\n ratio_left: l,\n ratio_right: r,\n body: \"because\".to_string(),\ndiff --git a/server/src/reducer.rs b/server/src/reducer.rs\nindex a8651994f2c062af25b6ad792a014dcc083296f9..cd11a5391de8e36b1c3bb6c9b8067ceba0878e31 100644\n--- a/server/src/reducer.rs\n+++ b/server/src/reducer.rs\n@@ -5,7 +5,7 @@ use serde::{Deserialize, Serialize};\n use crate::canonical_path::canonicalize_tag;\n use crate::dsl;\n use crate::events::{Event, Ingest, ThreadCapability};\n-use crate::path_types::CanonicalItemUrl;\n+use crate::path_types::ItemId;\n \n #[derive(Debug, Clone, Hash, PartialEq, Eq, PartialOrd, Ord)]\n pub enum ScopeId {\n@@ -27,8 +27,8 @@ pub fn scope_from_room_wire(room: &str) -> ScopeId {\n #[derive(Debug, Clone, Serialize, Deserialize, PartialEq, Eq)]\n pub struct VoteData {\n pub ts: i64,\n- pub a: CanonicalItemUrl,\n- pub b: CanonicalItemUrl,\n+ pub a: ItemId,\n+ pub b: ItemId,\n pub ratio_left: i32,\n pub ratio_right: i32,\n pub body: String,\n@@ -40,8 +40,8 @@ pub struct VoteData {\n \n #[derive(Debug, Clone)]\n pub struct GroupState {\n- pub item_to_idx: HashMap,\n- pub idx_to_item: Vec,\n+ pub item_to_idx: HashMap,\n+ pub idx_to_item: Vec,\n \n /// Aggregated directed edge weights: (src_idx, dst_idx) -> weight.\n pub edges: HashMap<(usize, usize), f64>,\n@@ -72,7 +72,7 @@ impl GroupState {\n }\n }\n \n- fn ensure_item(&mut self, item: &CanonicalItemUrl) -> usize {\n+ fn ensure_item(&mut self, item: &ItemId) -> usize {\n if let Some(&idx) = self.item_to_idx.get(item) {\n return idx;\n }\n@@ -85,11 +85,11 @@ impl GroupState {\n \n /// Public test helper: insert an item into the group without a vote (for unit tests).\n pub fn ensure_item_pub(&mut self, item: &str) -> usize {\n- if let Some(canon) = CanonicalItemUrl::parse(item) {\n+ if let Some(canon) = ItemId::parse(item) {\n self.ensure_item(&canon)\n } else {\n // Fallback: treat as raw canonical string\n- let canon = CanonicalItemUrl(item.to_string());\n+ let canon = ItemId::opaque(item.to_string());\n self.ensure_item(&canon)\n }\n }\n@@ -103,8 +103,8 @@ impl GroupState {\n }\n \n pub fn apply_vote(&mut self, mut vote: VoteData) {\n- vote.a = CanonicalItemUrl::parse(vote.a.as_str()).unwrap_or(vote.a);\n- vote.b = CanonicalItemUrl::parse(vote.b.as_str()).unwrap_or(vote.b);\n+ vote.a = ItemId::parse(vote.a.as_str()).unwrap_or_else(|| vote.a.clone());\n+ vote.b = ItemId::parse(vote.b.as_str()).unwrap_or_else(|| vote.b.clone());\n vote.thread_tag = canonicalize_tag(&vote.thread_tag);\n if vote.ratio_left < 0 {\n vote.ratio_left = 0;\n@@ -210,18 +210,18 @@ impl Default for ForumThreadState {\n #[derive(Debug, Clone, Default)]\n pub struct ContentState {\n pub ranking_group: GroupState,\n- pub items: HashSet,\n- pub item_bodies: HashMap,\n+ pub items: HashSet,\n+ pub item_bodies: HashMap,\n /// Parent canonical URL -> direct children.\n- pub item_children: HashMap>,\n+ pub item_children: HashMap>,\n /// Per-item vote history (most recent first).\n- pub item_votes: HashMap>,\n+ pub item_votes: HashMap>,\n /// Per-item ingest references (most recent first).\n- pub item_snippets: HashMap>,\n+ pub item_snippets: HashMap>,\n /// Item path -> threads that mention or vote on this item.\n- pub item_threads: HashMap>,\n+ pub item_threads: HashMap>,\n /// Per-item rank history, oldest first.\n- pub rank_history: HashMap>,\n+ pub rank_history: HashMap>,\n }\n \n #[derive(Debug, Clone)]\n@@ -351,7 +351,7 @@ impl ReducerState {\n /// For `a/b/c/d` this creates: `a/b/c→d`, `a/b→a/b/c`, `a→a/b`, `\"\"→a`.\n /// Stops early when an intermediate is already registered (its ancestors must be too).\n /// @e2bdefa9-a6fa-4725-b0a2-c0b09d95bb20:claudecode:anthropic/claude-opus-4\n- fn add_child_edge(content: &mut ContentState, item: &CanonicalItemUrl) {\n+ fn add_child_edge(content: &mut ContentState, item: &ItemId) {\n let mut child = item.clone();\n loop {\n let Some(parent) = child.parent() else { break };\n@@ -366,16 +366,16 @@ impl ReducerState {\n }\n \n /// Resolve an item path as a first-class canonical path.\n- fn normalize_item(item: &str) -> Option {\n- CanonicalItemUrl::parse(item)\n+ fn normalize_item(item: &str) -> Option {\n+ ItemId::parse(item)\n }\n \n /// 1-indexed rank of `item` within its connected component in the parent scope.\n /// 0 if the item has no votes connecting it to siblings (unranked).\n fn scope_rank_of(\n group: &GroupState,\n- item: &CanonicalItemUrl,\n- item_children: &HashMap>,\n+ item: &ItemId,\n+ item_children: &HashMap>,\n ) -> usize {\n let scope = match item.parent() {\n Some(p) => p,\n@@ -421,7 +421,7 @@ impl ReducerState {\n /// 1-indexed position of `item` in the component-aware global flat list.\n /// Components sorted largest-first; items ranked within each component.\n /// 0 if the item is not in the ranking group.\n- fn global_rank_of(group: &GroupState, item: &CanonicalItemUrl) -> usize {\n+ fn global_rank_of(group: &GroupState, item: &ItemId) -> usize {\n if !group.item_to_idx.contains_key(item) {\n return 0;\n }\n@@ -448,7 +448,7 @@ impl ReducerState {\n let doc = dsl::parse_full(&ing.raw).map_err(|_| ())?;\n let canonical_thread = canonicalize_tag(&ing.thread_tag);\n \n- let voted_items: Vec = doc\n+ let voted_items: Vec = doc\n .statements\n .iter()\n .filter_map(|s| {\n@@ -467,7 +467,7 @@ impl ReducerState {\n let principal = ing.principal.clone();\n let delegate = ing.delegate.clone();\n \n- let before: HashMap = if !voted_items.is_empty() {\n+ let before: HashMap = if !voted_items.is_empty() {\n crate::ranking::compute_group_ranking(&mut content.ranking_group, 10000, 1e-8);\n voted_items\n .iter()\n@@ -485,7 +485,7 @@ impl ReducerState {\n HashMap::new()\n };\n \n- let mut ingest_items: HashSet = HashSet::new();\n+ let mut ingest_items: HashSet = HashSet::new();\n \n for stmt in doc.statements {\n match stmt {\ndiff --git a/server/src/scope_rank.rs b/server/src/scope_rank.rs\nindex d656b2f0a0a3623ca6b234b746beaaca8ae6017d..4fdaae150c74ec5162c2accc06d4cc778ad79914 100644\n--- a/server/src/scope_rank.rs\n+++ b/server/src/scope_rank.rs\n@@ -3,7 +3,7 @@\n \n use std::collections::{HashMap, HashSet};\n \n-use crate::path_types::CanonicalItemUrl;\n+use crate::path_types::ItemId;\n use crate::ranking::{connected_components_from_voted_pairs, ranked_items_subset, RankedItem};\n use crate::reducer::ContentState;\n \n@@ -17,12 +17,12 @@ pub struct ScopedComponent {\n pub struct ChildrenRankings {\n pub component_rankings: Vec,\n /// Items in scope with no rank (no votes connecting them to others in this scope).\n- pub unranked_items: Vec,\n+ pub unranked_items: Vec,\n }\n \n /// Resolve one scope spec (literal path) to direct children of that parent. No wildcards.\n-fn resolve_one_scope(content: &ContentState, spec: &str) -> HashSet {\n- let Some(parent) = CanonicalItemUrl::parse(spec.trim()) else {\n+fn resolve_one_scope(content: &ContentState, spec: &str) -> HashSet {\n+ let Some(parent) = ItemId::parse(spec.trim()) else {\n return HashSet::new();\n };\n content\n@@ -36,12 +36,12 @@ fn resolve_one_scope(content: &ContentState, spec: &str) -> HashSet Vec {\n+pub fn resolve_scope(content: &ContentState, specs: &[String]) -> Vec {\n let mut set = HashSet::new();\n for spec in specs {\n set.extend(resolve_one_scope(content, spec));\n }\n- let mut out: Vec = set.into_iter().collect();\n+ let mut out: Vec = set.into_iter().collect();\n out.sort();\n out\n }\n@@ -49,18 +49,18 @@ pub fn resolve_scope(content: &ContentState, specs: &[String]) -> Vec Vec {\n+pub fn resolve_scope_recursive(content: &ContentState, specs: &[String], depth: usize) -> Vec {\n if depth == 0 {\n return vec![];\n }\n- let mut visited: HashSet = HashSet::new();\n- let mut frontier: Vec = specs\n+ let mut visited: HashSet = HashSet::new();\n+ let mut frontier: Vec = specs\n .iter()\n- .filter_map(|s| CanonicalItemUrl::parse(s))\n+ .filter_map(|s| ItemId::parse(s))\n .collect();\n \n for _level in 0..depth {\n- let mut next_frontier: Vec = Vec::new();\n+ let mut next_frontier: Vec = Vec::new();\n for parent in &frontier {\n if let Some(children) = content.item_children.get(parent) {\n for child in children {\n@@ -76,16 +76,16 @@ pub fn resolve_scope_recursive(content: &ContentState, specs: &[String], depth:\n frontier = next_frontier;\n }\n \n- let mut out: Vec = visited.into_iter().collect();\n+ let mut out: Vec = visited.into_iter().collect();\n out.sort();\n out\n }\n \n /// Build connected-component rankings for an explicit set of item paths.\n /// Use this when scope comes from multiple parents (resolve_scope).\n-pub fn build_rankings_for_item_set(content: &ContentState, items_in_scope: &[CanonicalItemUrl]) -> ChildrenRankings {\n+pub fn build_rankings_for_item_set(content: &ContentState, items_in_scope: &[ItemId]) -> ChildrenRankings {\n let group = &content.ranking_group;\n- let mut items_in_scope: Vec = items_in_scope.to_vec();\n+ let mut items_in_scope: Vec = items_in_scope.to_vec();\n items_in_scope.sort();\n \n let scoped_idxs: Vec = items_in_scope\n@@ -127,7 +127,7 @@ pub fn build_rankings_for_item_set(content: &ContentState, items_in_scope: &[Can\n })\n .collect();\n \n- let mut unranked_items: Vec = isolate_local_idxs\n+ let mut unranked_items: Vec = isolate_local_idxs\n .into_iter()\n .filter_map(|li| local_to_global.get(li).copied())\n .filter_map(|idx| group.idx_to_item.get(idx).cloned())\n@@ -148,9 +148,9 @@ pub fn build_rankings_for_item_set(content: &ContentState, items_in_scope: &[Can\n \n /// Build connected-component rankings for direct children of parent_scope.\n /// Matches the HTML garden view: multiple components, isolates, no-vote items.\n-pub fn build_children_rankings(content: &ContentState, parent: &CanonicalItemUrl) -> ChildrenRankings {\n+pub fn build_children_rankings(content: &ContentState, parent: &ItemId) -> ChildrenRankings {\n let parent = parent.clone().normalized_storage();\n- let items: Vec = content\n+ let items: Vec = content\n .item_children\n .get(&parent)\n .map(|s| s.iter().cloned().collect())\n@@ -164,10 +164,13 @@ mod tests {\n use std::collections::{HashMap, HashSet};\n \n fn content_with_children(edges: &[(&str, &[&str])]) -> ContentState {\n- let mut item_children: HashMap> = HashMap::new();\n+ let mut item_children: HashMap> = HashMap::new();\n for (parent, children) in edges {\n- let parent = CanonicalItemUrl((*parent).to_string());\n- let set: HashSet = children.iter().map(|s| CanonicalItemUrl((*s).to_string())).collect();\n+ let parent = ItemId::parse(parent).unwrap();\n+ let set: HashSet = children\n+ .iter()\n+ .map(|s| ItemId::parse(s).unwrap())\n+ .collect();\n item_children.insert(parent, set);\n }\n ContentState {\n@@ -189,8 +192,8 @@ mod tests {\n ]);\n let out = resolve_one_scope(&content, \"models\");\n assert_eq!(out.len(), 2);\n- assert!(out.contains(&CanonicalItemUrl(\"https://slug.social/models/x\".to_string())));\n- assert!(out.contains(&CanonicalItemUrl(\"https://slug.social/models/y\".to_string())));\n+ assert!(out.contains(&ItemId::parse(\"https://slug.social/models/x\").unwrap()));\n+ assert!(out.contains(&ItemId::parse(\"https://slug.social/models/y\").unwrap()));\n }\n \n #[test]\n@@ -201,9 +204,9 @@ mod tests {\n ]);\n let out = resolve_scope(&content, &[\"a\".into(), \"b\".into()]);\n assert_eq!(out.len(), 4);\n- assert!(out.contains(&CanonicalItemUrl(\"https://slug.social/a/1\".to_string())));\n- assert!(out.contains(&CanonicalItemUrl(\"https://slug.social/a/2\".to_string())));\n- assert!(out.contains(&CanonicalItemUrl(\"https://slug.social/b/1\".to_string())));\n- assert!(out.contains(&CanonicalItemUrl(\"https://slug.social/b/2\".to_string())));\n+ assert!(out.contains(&ItemId::parse(\"https://slug.social/a/1\").unwrap()));\n+ assert!(out.contains(&ItemId::parse(\"https://slug.social/a/2\").unwrap()));\n+ assert!(out.contains(&ItemId::parse(\"https://slug.social/b/1\").unwrap()));\n+ assert!(out.contains(&ItemId::parse(\"https://slug.social/b/2\").unwrap()));\n }\n }\ndiff --git a/server/tests/basic.rs b/server/tests/basic.rs\nindex 17db86894f18d6b542d2943f4cedb0572b247171..c8efea2976e20b443d78a5d4c3b244bc4f2fd6da 100644\n--- a/server/tests/basic.rs\n+++ b/server/tests/basic.rs\n@@ -7,8 +7,15 @@ use slugsocial_server::{\n };\n \n \n+use slugsocial_server::path_types::ItemId;\n+\n use tempfile::TempDir;\n \n+#[inline]\n+fn item_id(s: &str) -> ItemId {\n+ ItemId::parse(s).unwrap()\n+}\n+\n fn ingest_event(ts: i64, raw: &str) -> Event {\n Event::Ingest(Ingest {\n ts,\n@@ -47,7 +54,7 @@ fn reducer_external_namespace_ranking() {\n let g = &content.ranking_group;\n assert_eq!(g.idx_to_item.len(), 2);\n \n- let parent = slugsocial_server::path_types::CanonicalItemUrl::parse(\"https://github.com/iss\").unwrap();\n+ let parent = slugsocial_server::path_types::ItemId::parse(\"https://github.com/iss\").unwrap();\n let children = slugsocial_server::scope_rank::build_children_rankings(content, &parent);\n assert_eq!(children.component_rankings.len(), 1);\n let names: Vec<&str> = children.component_rankings[0]\n@@ -77,9 +84,9 @@ fn reducer_and_ranking_linear_chain() {\n let mut group = state.public().ranking_group.clone();\n let ranked = ranked_items(&mut group, 20000, 1e-9);\n assert_eq!(ranked.len(), 3);\n- assert_eq!(ranked[0].item, \"https://slug.social/~/t/a\");\n- assert_eq!(ranked[1].item, \"https://slug.social/~/t/b\");\n- assert_eq!(ranked[2].item, \"https://slug.social/~/t/c\");\n+ assert_eq!(ranked[0].item.as_str(), \"https://slug.social/~/t/a\");\n+ assert_eq!(ranked[1].item.as_str(), \"https://slug.social/~/t/b\");\n+ assert_eq!(ranked[2].item.as_str(), \"https://slug.social/~/t/c\");\n }\n \n #[test]\n@@ -112,15 +119,15 @@ fn reducer_handles_item_and_body_from_ingest() {\n ));\n \n let content = state.public();\n- assert!(content.items.contains(\"https://slug.social/~/t/test-item\"));\n+ assert!(content.items.contains(&item_id(\"https://slug.social/~/t/test-item\")));\n assert_eq!(\n- content.item_bodies.get(\"https://slug.social/~/t/test-item\"),\n+ content.item_bodies.get(&item_id(\"https://slug.social/~/t/test-item\")),\n Some(&\"Description here\".to_string())\n );\n assert!(content\n .item_children\n- .get(\"https://slug.social/~/t\")\n- .map(|c| c.contains(\"https://slug.social/~/t/test-item\"))\n+ .get(&item_id(\"https://slug.social/~/t\"))\n+ .map(|c| c.contains(&item_id(\"https://slug.social/~/t/test-item\")))\n .unwrap_or(false));\n }\n \n@@ -136,12 +143,12 @@ fn reducer_indexes_item_threads_and_vote_thread() {\n state.apply_event(Event::Ingest(ev));\n \n let content = state.public();\n- let threads_for_insertion = content.item_threads.get(\"https://slug.social/~/sorts/insertion\").unwrap();\n+ let threads_for_insertion = content.item_threads.get(&item_id(\"https://slug.social/~/sorts/insertion\")).unwrap();\n assert!(threads_for_insertion.contains(\"sorting-hat\"));\n- let threads_for_mergesort = content.item_threads.get(\"https://slug.social/~/sorts/mergesort\").unwrap();\n+ let threads_for_mergesort = content.item_threads.get(&item_id(\"https://slug.social/~/sorts/mergesort\")).unwrap();\n assert!(threads_for_mergesort.contains(\"sorting-hat\"));\n \n- let vote = content.item_votes.get(\"https://slug.social/~/sorts/insertion\").unwrap().front().unwrap();\n+ let vote = content.item_votes.get(&item_id(\"https://slug.social/~/sorts/insertion\")).unwrap().front().unwrap();\n assert_eq!(vote.thread_tag, \"sorting-hat\");\n }\n \n@@ -155,8 +162,8 @@ fn reducer_aggregates_multiple_votes() {\n }\n \n let group = &state.public().ranking_group;\n- let a_idx = group.item_to_idx[\"https://slug.social/~/t/a\"];\n- let b_idx = group.item_to_idx[\"https://slug.social/~/t/b\"];\n+ let a_idx = group.item_to_idx[&item_id(\"https://slug.social/~/t/a\")];\n+ let b_idx = group.item_to_idx[&item_id(\"https://slug.social/~/t/b\")];\n \n // Should have accumulated edge weights in both directions.\n assert!(group.edges.contains_key(&(a_idx, b_idx)));\n@@ -232,7 +239,7 @@ fn ranking_dominant_item_wins() {\n let mut group = state.public().ranking_group.clone();\n let ranked = ranked_items(&mut group, 20000, 1e-9);\n \n- assert_eq!(ranked[0].item, \"https://slug.social/~/t/champion\");\n+ assert_eq!(ranked[0].item.as_str(), \"https://slug.social/~/t/champion\");\n assert!(ranked[0].score > ranked[1].score);\n }\n \n@@ -379,7 +386,7 @@ async fn full_workflow_reducer_and_ranking() {\n let ranked = ranked_items(&mut group, 20000, 1e-9);\n \n assert_eq!(ranked.len(), 2);\n- assert_eq!(ranked[0].item, \"https://slug.social/~/langs/rust\"); // Should win\n+ assert_eq!(ranked[0].item.as_str(), \"https://slug.social/~/langs/rust\"); // Should win\n assert!(ranked[0].score > ranked[1].score);\n }\n \n@@ -395,27 +402,27 @@ fn reducer_materializes_ancestor_path_segments() {\n \n // The intermediate path \"https://slug.social/~/ai-models/anthropic\" should appear as a child of \"https://slug.social/~/ai-models\".\n let content = state.public();\n- let ai_models_children = content.item_children.get(\"https://slug.social/~/ai-models\").expect(\"ai-models should have children\");\n+ let ai_models_children = content.item_children.get(&item_id(\"https://slug.social/~/ai-models\")).expect(\"ai-models should have children\");\n assert!(\n- ai_models_children.contains(\"https://slug.social/~/ai-models/anthropic\"),\n+ ai_models_children.contains(&item_id(\"https://slug.social/~/ai-models/anthropic\")),\n \"ai-models/anthropic should be a child of ai-models\"\n );\n \n // The leaf items should still be children of \"https://slug.social/~/ai-models/anthropic\".\n- let anthropic_children = content.item_children.get(\"https://slug.social/~/ai-models/anthropic\").expect(\"ai-models/anthropic should have children\");\n- assert!(anthropic_children.contains(\"https://slug.social/~/ai-models/anthropic/claude-opus\"));\n- assert!(anthropic_children.contains(\"https://slug.social/~/ai-models/anthropic/claude-sonnet\"));\n+ let anthropic_children = content.item_children.get(&item_id(\"https://slug.social/~/ai-models/anthropic\")).expect(\"ai-models/anthropic should have children\");\n+ assert!(anthropic_children.contains(&item_id(\"https://slug.social/~/ai-models/anthropic/claude-opus\")));\n+ assert!(anthropic_children.contains(&item_id(\"https://slug.social/~/ai-models/anthropic/claude-sonnet\")));\n \n // Root should contain \"https://slug.social/~/ai-models\".\n- let root_children = content.item_children.get(\"https://slug.social/~\").expect(\"root should have children\");\n- assert!(root_children.contains(\"https://slug.social/~/ai-models\"));\n+ let root_children = content.item_children.get(&item_id(\"https://slug.social/~\")).expect(\"root should have children\");\n+ assert!(root_children.contains(&item_id(\"https://slug.social/~/ai-models\")));\n \n // The phantom intermediates should NOT be in the items set (they weren't explicitly created).\n- assert!(!content.items.contains(\"https://slug.social/~/ai-models\"));\n- assert!(!content.items.contains(\"https://slug.social/~/ai-models/anthropic\"));\n+ assert!(!content.items.contains(&item_id(\"https://slug.social/~/ai-models\")));\n+ assert!(!content.items.contains(&item_id(\"https://slug.social/~/ai-models/anthropic\")));\n // But the leaf items should be.\n- assert!(content.items.contains(\"https://slug.social/~/ai-models/anthropic/claude-opus\"));\n- assert!(content.items.contains(\"https://slug.social/~/ai-models/anthropic/claude-sonnet\"));\n+ assert!(content.items.contains(&item_id(\"https://slug.social/~/ai-models/anthropic/claude-opus\")));\n+ assert!(content.items.contains(&item_id(\"https://slug.social/~/ai-models/anthropic/claude-sonnet\")));\n }\n \n #[test]\n@@ -469,9 +476,9 @@ fn ranking_repeated_votes_normalized() {\n \n // Same winner regardless of how many times voted.\n assert_eq!(ranked_once[0].item, ranked_many[0].item);\n- assert_eq!(ranked_once[0].item, \"https://slug.social/~/norm/a\");\n+ assert_eq!(ranked_once[0].item.as_str(), \"https://slug.social/~/norm/a\");\n assert_eq!(ranked_once[1].item, ranked_many[1].item);\n- assert_eq!(ranked_once[1].item, \"https://slug.social/~/norm/b\");\n+ assert_eq!(ranked_once[1].item.as_str(), \"https://slug.social/~/norm/b\");\n \n // Scores should be identical (normalization makes repeated votes idempotent).\n let eps = 1e-6;\n@@ -506,8 +513,8 @@ fn reducer_zero_zero_vote_ratio_normalizes_to_one_one() {\n \"~/t/a {a}\\n~/t/b {b}\\n~/t/a 0:0 ~/t/b {zero}\\n\",\n ));\n let group = &state.public().ranking_group;\n- let a_idx = group.item_to_idx[\"https://slug.social/~/t/a\"];\n- let b_idx = group.item_to_idx[\"https://slug.social/~/t/b\"];\n+ let a_idx = group.item_to_idx[&item_id(\"https://slug.social/~/t/a\")];\n+ let b_idx = group.item_to_idx[&item_id(\"https://slug.social/~/t/b\")];\n // 0:0 should normalize to 1:1 — both directions should have weight\n assert!(group.edges.contains_key(&(a_idx, b_idx)));\n assert!(group.edges.contains_key(&(b_idx, a_idx)));\n@@ -523,8 +530,8 @@ fn reducer_negative_ratio_clamped_to_zero() {\n let mut group = GroupState::new();\n group.apply_vote(slugsocial_server::reducer::VoteData {\n ts: 1,\n- a: slugsocial_server::path_types::CanonicalItemUrl(\"https://slug.social/~/t/a\".to_string()),\n- b: slugsocial_server::path_types::CanonicalItemUrl(\"https://slug.social/~/t/b\".to_string()),\n+ a: slugsocial_server::path_types::ItemId::parse(\"https://slug.social/~/t/a\").unwrap(),\n+ b: slugsocial_server::path_types::ItemId::parse(\"https://slug.social/~/t/b\").unwrap(),\n ratio_left: -5,\n ratio_right: -3,\n body: \"negative\".to_string(),\n@@ -534,8 +541,8 @@ fn reducer_negative_ratio_clamped_to_zero() {\n });\n assert_eq!(group.idx_to_item.len(), 2);\n // Both edges should exist (negatives clamped to 0, then 0:0 -> 1:1)\n- let a_idx = group.item_to_idx[\"https://slug.social/~/t/a\"];\n- let b_idx = group.item_to_idx[\"https://slug.social/~/t/b\"];\n+ let a_idx = group.item_to_idx[&item_id(\"https://slug.social/~/t/a\")];\n+ let b_idx = group.item_to_idx[&item_id(\"https://slug.social/~/t/b\")];\n assert!(group.edges.contains_key(&(a_idx, b_idx)));\n assert!(group.edges.contains_key(&(b_idx, a_idx)));\n }\n@@ -553,24 +560,24 @@ fn reducer_deep_path_ancestor_materialization_four_levels() {\n let content = state.public();\n let tilde_scope = content\n .item_children\n- .get(\"https://slug.social/~\")\n+ .get(&item_id(\"https://slug.social/~\"))\n .expect(\"~/ scope should have children\");\n- assert!(tilde_scope.contains(\"https://slug.social/~/a\"));\n+ assert!(tilde_scope.contains(&item_id(\"https://slug.social/~/a\")));\n \n- let a_children = content.item_children.get(\"https://slug.social/~/a\").expect(\"a should have children\");\n- assert!(a_children.contains(\"https://slug.social/~/a/b\"));\n+ let a_children = content.item_children.get(&item_id(\"https://slug.social/~/a\")).expect(\"a should have children\");\n+ assert!(a_children.contains(&item_id(\"https://slug.social/~/a/b\")));\n \n- let ab_children = content.item_children.get(\"https://slug.social/~/a/b\").expect(\"a/b should have children\");\n- assert!(ab_children.contains(\"https://slug.social/~/a/b/c\"));\n+ let ab_children = content.item_children.get(&item_id(\"https://slug.social/~/a/b\")).expect(\"a/b should have children\");\n+ assert!(ab_children.contains(&item_id(\"https://slug.social/~/a/b/c\")));\n \n- let abc_children = content.item_children.get(\"https://slug.social/~/a/b/c\").expect(\"a/b/c should have children\");\n- assert!(abc_children.contains(\"https://slug.social/~/a/b/c/d\"));\n+ let abc_children = content.item_children.get(&item_id(\"https://slug.social/~/a/b/c\")).expect(\"a/b/c should have children\");\n+ assert!(abc_children.contains(&item_id(\"https://slug.social/~/a/b/c/d\")));\n \n // Only the leaf should be in items set\n- assert!(content.items.contains(\"https://slug.social/~/a/b/c/d\"));\n- assert!(!content.items.contains(\"https://slug.social/~/a\"));\n- assert!(!content.items.contains(\"https://slug.social/~/a/b\"));\n- assert!(!content.items.contains(\"https://slug.social/~/a/b/c\"));\n+ assert!(content.items.contains(&item_id(\"https://slug.social/~/a/b/c/d\")));\n+ assert!(!content.items.contains(&item_id(\"https://slug.social/~/a\")));\n+ assert!(!content.items.contains(&item_id(\"https://slug.social/~/a/b\")));\n+ assert!(!content.items.contains(&item_id(\"https://slug.social/~/a/b/c\")));\n }\n \n // ============================================================================\n@@ -610,7 +617,7 @@ fn ranking_convergence_tolerance_triggers_early_exit() {\n // Very tight tolerance but huge max_iters — should still converge fast\n let ranked = ranked_items(&mut group, 1_000_000, 1e-15);\n assert_eq!(ranked.len(), 2);\n- assert_eq!(ranked[0].item, \"https://slug.social/~/t/a\");\n+ assert_eq!(ranked[0].item.as_str(), \"https://slug.social/~/t/a\");\n }\n \n // ============================================================================\n@@ -624,14 +631,14 @@ fn test_item_body_overwrite() {\n 1,\n \"~/t/x {first}\\n\",\n ));\n- assert_eq!(state.public().item_bodies.get(\"https://slug.social/~/t/x\"), Some(&\"first\".to_string()));\n+ assert_eq!(state.public().item_bodies.get(&item_id(\"https://slug.social/~/t/x\")), Some(&\"first\".to_string()));\n \n state.apply_event(ingest_event(\n 2,\n \"~/t/x {second}\\n\",\n ));\n assert_eq!(\n- state.public().item_bodies.get(\"https://slug.social/~/t/x\"),\n+ state.public().item_bodies.get(&item_id(\"https://slug.social/~/t/x\")),\n Some(&\"second\".to_string()),\n \"last writer should win for item bodies\"\n );\n@@ -645,9 +652,9 @@ fn test_empty_body_not_stored() {\n \"~/t/blank { }\\n\",\n ));\n let content = state.public();\n- assert!(content.items.contains(\"https://slug.social/~/t/blank\"), \"item should exist\");\n+ assert!(content.items.contains(&item_id(\"https://slug.social/~/t/blank\")), \"item should exist\");\n assert!(\n- !content.item_bodies.contains_key(\"https://slug.social/~/t/blank\"),\n+ !content.item_bodies.contains_key(&item_id(\"https://slug.social/~/t/blank\")),\n \"whitespace-only body should not be stored\"\n );\n }\n@@ -664,7 +671,7 @@ fn test_duplicate_items_across_ingests() {\n \"~/t/dup {second}\\n\",\n ));\n let content = state.public();\n- let count = content.items.iter().filter(|i| *i == \"https://slug.social/~/t/dup\").count();\n+ let count = content.items.iter().filter(|i| i.as_str() == \"https://slug.social/~/t/dup\").count();\n assert_eq!(count, 1, \"items set should deduplicate across ingests\");\n }\n \n@@ -736,7 +743,7 @@ fn test_thread_id_is_used_for_votes_and_indexes() {\n let content = state.public();\n let vote = content\n .item_votes\n- .get(\"https://slug.social/~/t/a\")\n+ .get(&item_id(\"https://slug.social/~/t/a\"))\n .unwrap()\n .front()\n .unwrap();\n@@ -745,7 +752,7 @@ fn test_thread_id_is_used_for_votes_and_indexes() {\n assert!(state\n .public()\n .item_threads\n- .get(\"https://slug.social/~/t/a\")\n+ .get(&item_id(\"https://slug.social/~/t/a\"))\n .is_some_and(|threads| threads.contains(\"first\")));\n }\n \n@@ -757,11 +764,11 @@ fn test_rank_history_created_for_voted_items() {\n \"~/t/a {a}\\n~/t/b {b}\\n~/t/a 3:1 ~/t/b {reason}\\n\",\n ));\n assert!(\n- state.public().rank_history.contains_key(\"https://slug.social/~/t/a\"),\n+ state.public().rank_history.contains_key(&item_id(\"https://slug.social/~/t/a\")),\n \"rank_history should have entry for voted item a\"\n );\n assert!(\n- state.public().rank_history.contains_key(\"https://slug.social/~/t/b\"),\n+ state.public().rank_history.contains_key(&item_id(\"https://slug.social/~/t/b\")),\n \"rank_history should have entry for voted item b\"\n );\n }\n@@ -774,7 +781,7 @@ fn test_rank_history_not_created_for_unvoted_items() {\n \"~/t/c {just a definition}\\n\",\n ));\n assert!(\n- !state.public().rank_history.contains_key(\"https://slug.social/~/t/c\"),\n+ !state.public().rank_history.contains_key(&item_id(\"https://slug.social/~/t/c\")),\n \"rank_history should NOT have entry for item with no votes\"\n );\n }\n@@ -786,7 +793,7 @@ fn test_rank_history_first_entry_delta_zero() {\n 1,\n \"~/t/a {a}\\n~/t/b {b}\\n~/t/a 3:1 ~/t/b {reason}\\n\",\n ));\n- let history_a = state.public().rank_history.get(\"https://slug.social/~/t/a\").unwrap();\n+ let history_a = state.public().rank_history.get(&item_id(\"https://slug.social/~/t/a\")).unwrap();\n assert_eq!(history_a.len(), 1);\n assert_eq!(\n history_a[0].scope_rank_delta, 0,\ndiff --git a/types/src/item_id.rs b/types/src/item_id.rs\nnew file mode 100644\nindex 0000000000000000000000000000000000000000..5fe9fad39c0d58e546c0a0162eba4348d64e0865\n--- /dev/null\n+++ b/types/src/item_id.rs\n@@ -0,0 +1,225 @@\n+//! Structural item identity for the reducer graph and ranking (vs presentation-only strings).\n+\n+use std::cmp::Ordering;\n+use std::fmt;\n+\n+use serde::{Deserialize, Deserializer, Serialize, Serializer};\n+\n+use crate::item_wire::{\n+ canonicalize_item, external_display_dash_prefix, normalize_slug_ontology_storage_url,\n+ SLUG_TILDE_ONTOLOGY_ROOT,\n+};\n+\n+/// Structural key for items in [`slug_types`] and the server reducer.\n+///\n+/// Wire / JSON uses the same single string as the former canonical item URL (via serde).\n+#[derive(Debug, Clone, Hash, PartialEq, Eq)]\n+pub enum ItemId {\n+ /// Tilde ontology root (`~/`); storage [`SLUG_TILDE_ONTOLOGY_ROOT`].\n+ Root,\n+ /// `https://slug.social/~/…` (non-root; normalized trailing path).\n+ Local(String),\n+ /// Normalized `http(s)://…` storage form, including non-`~/` paths on `slug.social`.\n+ Web(String),\n+ /// Raw key material that did not round-trip through [`Self::parse`] (historical edge case).\n+ Opaque(String),\n+}\n+\n+impl ItemId {\n+ pub fn parse(input: &str) -> Option {\n+ let c = normalize_slug_ontology_storage_url(&canonicalize_item(input));\n+ if c.is_empty() {\n+ return None;\n+ }\n+ if c == SLUG_TILDE_ONTOLOGY_ROOT {\n+ return Some(Self::Root);\n+ }\n+ if c.starts_with(\"https://slug.social/~/\") && c != SLUG_TILDE_ONTOLOGY_ROOT {\n+ return Some(Self::Local(c));\n+ }\n+ Some(Self::Web(c))\n+ }\n+\n+ /// Same as the old `ensure_item` fallback: use `s` verbatim as the map key.\n+ pub fn opaque(raw: String) -> Self {\n+ Self::Opaque(raw)\n+ }\n+\n+ pub fn ontology_root() -> Self {\n+ Self::Root\n+ }\n+\n+ /// Collapses legacy slug ontology root spellings so [`std::collections::HashMap`] keys match the graph.\n+ pub fn normalized_storage(self) -> Self {\n+ Self::parse(self.as_str()).unwrap_or(self)\n+ }\n+\n+ pub fn as_str(&self) -> &str {\n+ match self {\n+ ItemId::Root => SLUG_TILDE_ONTOLOGY_ROOT,\n+ ItemId::Local(s) | ItemId::Web(s) | ItemId::Opaque(s) => s,\n+ }\n+ }\n+\n+ pub fn to_storage_string(&self) -> String {\n+ self.as_str().to_string()\n+ }\n+\n+ pub fn tilde_tail(&self) -> Option<&str> {\n+ match self {\n+ ItemId::Root => Some(\"\"),\n+ ItemId::Local(s) => s.strip_prefix(\"https://slug.social/~/\"),\n+ ItemId::Web(s) | ItemId::Opaque(s) => {\n+ if let Some(tail) = s.strip_prefix(\"https://slug.social/~/\") {\n+ return Some(tail);\n+ }\n+ if s == SLUG_TILDE_ONTOLOGY_ROOT || s == \"https://slug.social/~/\" {\n+ return Some(\"\");\n+ }\n+ None\n+ }\n+ }\n+ }\n+\n+ /// HTTP garden tail after `~/` (empty at ontology root), or `None` if not under tilde ontology.\n+ pub fn tilde_http_tail(&self) -> Option {\n+ self.tilde_tail().map(str::to_owned)\n+ }\n+\n+ pub fn last_segment(&self) -> &str {\n+ let s = self.as_str();\n+ s.rsplit('/').find(|x| !x.is_empty()).unwrap_or(s)\n+ }\n+\n+ pub fn parent(&self) -> Option {\n+ match self {\n+ ItemId::Root => None,\n+ ItemId::Local(s) => {\n+ if s == SLUG_TILDE_ONTOLOGY_ROOT || s == \"https://slug.social/~/\" {\n+ return None;\n+ }\n+ let last_slash = s.rfind('/')?;\n+ let parent_str = &s[..last_slash];\n+ if parent_str.is_empty() {\n+ None\n+ } else {\n+ Self::parse(parent_str)\n+ }\n+ }\n+ ItemId::Web(s) | ItemId::Opaque(s) => {\n+ if s == SLUG_TILDE_ONTOLOGY_ROOT || s == \"https://slug.social/~/\" {\n+ return None;\n+ }\n+ if let Some(rest) = s.strip_prefix(\"https://slug.social/~/\") {\n+ if rest.is_empty() {\n+ return None;\n+ }\n+ let last_slash = s.rfind('/')?;\n+ let parent_str = &s[..last_slash];\n+ Self::parse(parent_str)\n+ } else if s.starts_with(\"https://\") {\n+ let rest = s.strip_prefix(\"https://\").unwrap();\n+ Self::parent_http_url(\"https://\", rest)\n+ } else if s.starts_with(\"http://\") {\n+ let rest = s.strip_prefix(\"http://\").unwrap();\n+ Self::parent_http_url(\"http://\", rest)\n+ } else {\n+ None\n+ }\n+ }\n+ }\n+ }\n+\n+ fn parent_http_url(scheme: &'static str, rest: &str) -> Option {\n+ let (host, path) = rest.split_once('/').map_or((rest, \"\"), |(h, p)| (h, p));\n+ let host = host.trim();\n+ let path = path.trim_end_matches('/');\n+ if path.is_empty() {\n+ return None;\n+ }\n+ let parent_path = path.rsplit_once('/').map(|(p, _)| p).unwrap_or(\"\");\n+ if parent_path.is_empty() {\n+ Self::parse(&format!(\"{scheme}{}\", host))\n+ } else {\n+ Self::parse(&format!(\"{scheme}{}/{}\", host, parent_path))\n+ }\n+ }\n+\n+ /// `-/` representation for external `https://…` items, `~/…` for slug ontology, else unchanged.\n+ pub fn display_path(&self) -> String {\n+ if let Some(tail) = self.tilde_tail() {\n+ if tail.is_empty() {\n+ return \"~/\".to_string();\n+ }\n+ return format!(\"~/{}\", tail);\n+ }\n+ let s = self.as_str();\n+ if let Some(tail) = s.strip_prefix(\"https://\") {\n+ if tail.starts_with(\"slug.social\") {\n+ return s.to_string();\n+ }\n+ return external_display_dash_prefix(tail);\n+ }\n+ if let Some(tail) = s.strip_prefix(\"http://\") {\n+ if tail.starts_with(\"slug.social\") {\n+ return s.to_string();\n+ }\n+ return external_display_dash_prefix(tail);\n+ }\n+ s.to_string()\n+ }\n+\n+ pub fn tilde_segments(&self) -> Vec<&str> {\n+ match self.tilde_tail() {\n+ Some(tail) if !tail.is_empty() => std::iter::once(\"~\")\n+ .chain(tail.split('/').filter(|s| !s.is_empty()))\n+ .collect(),\n+ Some(_) => vec![\"~\"],\n+ None => vec![],\n+ }\n+ }\n+\n+ /// Normalized URL string for HTTP fetch boundaries (external identities).\n+ pub fn to_wire_url(&self) -> String {\n+ self.to_storage_string()\n+ }\n+}\n+\n+impl PartialOrd for ItemId {\n+ fn partial_cmp(&self, other: &Self) -> Option {\n+ Some(self.cmp(other))\n+ }\n+}\n+\n+impl Ord for ItemId {\n+ fn cmp(&self, other: &Self) -> Ordering {\n+ self.as_str().cmp(other.as_str())\n+ }\n+}\n+\n+impl fmt::Display for ItemId {\n+ fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {\n+ self.as_str().fmt(f)\n+ }\n+}\n+\n+impl Serialize for ItemId {\n+ fn serialize(&self, serializer: S) -> Result\n+ where\n+ S: Serializer,\n+ {\n+ self.as_str().serialize(serializer)\n+ }\n+}\n+\n+impl<'de> Deserialize<'de> for ItemId {\n+ fn deserialize(deserializer: D) -> Result\n+ where\n+ D: Deserializer<'de>,\n+ {\n+ let s = String::deserialize(deserializer)?;\n+ ItemId::parse(&s).ok_or_else(|| {\n+ serde::de::Error::custom(format!(\"invalid item id: {s:?}\"))\n+ })\n+ }\n+}\ndiff --git a/types/src/item_wire.rs b/types/src/item_wire.rs\nnew file mode 100644\nindex 0000000000000000000000000000000000000000..20e3e7df77a74713c146c7e7222a809173723c7d\n--- /dev/null\n+++ b/types/src/item_wire.rs\n@@ -0,0 +1,184 @@\n+//! Wire normalization for item identity strings (`canonicalize_item`, path segments).\n+//! Split from `paths` so [`crate::item_id::ItemId`] can depend on this without import cycles.\n+\n+use crate::url_normalize::{host_preserves_dash_path_case, normalize_http_identity_url};\n+\n+/// Canonical absolute URL for the tilde ontology **root** (`~/` in UI).\n+pub const SLUG_TILDE_ONTOLOGY_ROOT: &str = \"https://slug.social/~\";\n+\n+/// Collapse legacy or parser variants of the ontology root to [`SLUG_TILDE_ONTOLOGY_ROOT`].\n+pub fn normalize_slug_ontology_storage_url(s: &str) -> String {\n+ if s == \"https://slug.social/~/\" {\n+ SLUG_TILDE_ONTOLOGY_ROOT.to_string()\n+ } else {\n+ s.to_string()\n+ }\n+}\n+\n+fn finalize_external_identity_url(s: String) -> String {\n+ if s.starts_with(\"https://slug.social/\") {\n+ return s;\n+ }\n+ let normalized = normalize_http_identity_url(&s).unwrap_or_else(|| s.clone());\n+ strip_redundant_root_slash(&normalized).unwrap_or(normalized)\n+}\n+\n+/// `url::Url` serializes bare hosts with a `/` path; we keep host-only items slash-free for stable\n+/// keys matching the pre-normalizer spellings.\n+fn strip_redundant_root_slash(s: &str) -> Option {\n+ let u = url::Url::parse(s).ok()?;\n+ if u.path() == \"/\" && u.query().is_none() && u.fragment().is_none() {\n+ let scheme = u.scheme();\n+ let host = u.host_str()?;\n+ return Some(match u.port() {\n+ Some(p) => format!(\"{scheme}://{host}:{p}\"),\n+ None => format!(\"{scheme}://{host}\"),\n+ });\n+ }\n+ None\n+}\n+\n+/// Ontology item reference → canonical absolute URL on the slug host.\n+pub fn canonicalize_item(input: &str) -> String {\n+ let s = input.trim();\n+ if s.is_empty() {\n+ return String::new();\n+ }\n+\n+ if let Some(rest) = s.strip_prefix(\"-/\") {\n+ let (host, tail) = rest\n+ .split_once('/')\n+ .map_or((rest, \"\"), |(h, t)| (h, t));\n+ let host = host.trim().to_lowercase();\n+ if host.is_empty() {\n+ return String::new();\n+ }\n+ let preserve_case = host_preserves_dash_path_case(&host);\n+ return if tail.is_empty() {\n+ finalize_external_identity_url(format!(\"https://{}\", host))\n+ } else {\n+ let path = tail\n+ .trim_start_matches('/')\n+ .trim_end_matches('/')\n+ .split('/')\n+ .filter_map(|seg| {\n+ let t = seg.trim();\n+ if t.is_empty() {\n+ None\n+ } else if preserve_case {\n+ Some(t.to_string())\n+ } else {\n+ Some(t.to_lowercase())\n+ }\n+ })\n+ .collect::>()\n+ .join(\"/\");\n+ finalize_external_identity_url(format!(\"https://{}/{}\", host, path))\n+ };\n+ }\n+\n+ if let Some(rest) = s.strip_prefix(\"https://\") {\n+ let (host, tail) = rest.split_once('/').map_or((rest, \"\"), |(h, t)| (h, t));\n+ let host = host.trim().to_lowercase();\n+ return finalize_external_identity_url(if tail.is_empty() {\n+ format!(\"https://{}\", host)\n+ } else {\n+ format!(\"https://{}/{}\", host, tail)\n+ });\n+ }\n+ if let Some(rest) = s.strip_prefix(\"http://\") {\n+ let (host, tail) = rest.split_once('/').map_or((rest, \"\"), |(h, t)| (h, t));\n+ let host = host.trim().to_lowercase();\n+ return finalize_external_identity_url(if tail.is_empty() {\n+ format!(\"http://{}\", host)\n+ } else {\n+ format!(\"http://{}/{}\", host, tail)\n+ });\n+ }\n+\n+ let is_tilde = s.starts_with(\"~/\");\n+ let rest = s.strip_prefix(\"~/\").or_else(|| s.strip_prefix(\"/\")).unwrap_or(s);\n+\n+ let tail = rest\n+ .split('/')\n+ .filter_map(|seg| {\n+ let t = seg.trim();\n+ if t.is_empty() {\n+ None\n+ } else {\n+ Some(t.to_lowercase())\n+ }\n+ })\n+ .collect::>()\n+ .join(\"/\");\n+\n+ if is_tilde {\n+ if tail.is_empty() {\n+ return SLUG_TILDE_ONTOLOGY_ROOT.to_string();\n+ }\n+ format!(\"https://slug.social/~/{}\", tail)\n+ } else if tail.is_empty() {\n+ \"https://slug.social\".to_string()\n+ } else {\n+ format!(\"https://slug.social/{}\", tail)\n+ }\n+}\n+\n+pub fn item_path_segments(input: &str) -> Vec {\n+ let canonical = canonicalize_item(input);\n+ if canonical.is_empty() {\n+ return vec![];\n+ }\n+\n+ if let Some(rest) = canonical.strip_prefix(\"https://\") {\n+ let (host, tail) = rest.split_once('/').map_or((rest, \"\"), |(h, t)| (h, t));\n+ let mut out = vec![format!(\"https://{}\", host)];\n+ out.extend(tail.split('/').filter(|s| !s.is_empty()).map(|s| s.to_string()));\n+ return out;\n+ }\n+ if let Some(rest) = canonical.strip_prefix(\"http://\") {\n+ let (host, tail) = rest.split_once('/').map_or((rest, \"\"), |(h, t)| (h, t));\n+ let mut out = vec![format!(\"http://{}\", host)];\n+ out.extend(tail.split('/').filter(|s| !s.is_empty()).map(|s| s.to_string()));\n+ return out;\n+ }\n+\n+ canonical\n+ .split('/')\n+ .filter(|s| !s.is_empty())\n+ .map(|s| s.to_string())\n+ .collect()\n+}\n+\n+pub fn item_parent_path(input: &str) -> Option {\n+ let segs = item_path_segments(input);\n+ if segs.len() <= 1 {\n+ return None;\n+ }\n+ Some(segs[..segs.len() - 1].join(\"/\"))\n+}\n+\n+pub(crate) fn external_display_dash_prefix(host_and_path: &str) -> String {\n+ let (host, path) = host_and_path\n+ .split_once('/')\n+ .map_or((host_and_path, \"\"), |(h, p)| (h, p));\n+ let host = host.trim().to_lowercase();\n+ let path = path\n+ .trim_end_matches('/')\n+ .split('/')\n+ .filter_map(|seg| {\n+ let t = seg.trim();\n+ if t.is_empty() {\n+ None\n+ } else {\n+ Some(t.to_lowercase())\n+ }\n+ })\n+ .collect::>()\n+ .join(\"/\");\n+ if path.is_empty() {\n+ format!(\"-/{}\", host)\n+ } else {\n+ format!(\"-/{}\", format!(\"{}/{}\", host, path))\n+ }\n+}\ndiff --git a/types/src/lib.rs b/types/src/lib.rs\nindex ceaa574cd32b7e7eb397c9a3191fcbc037b8c606..05bcfcfbf48f1dc7a4c05e32055bd12eb5cfd623 100644\n--- a/types/src/lib.rs\n+++ b/types/src/lib.rs\n@@ -1,14 +1,20 @@\n use serde::{Deserialize, Serialize};\n \n pub mod url_normalize;\n+pub mod item_wire;\n+pub mod item_id;\n pub mod paths;\n pub mod timeago;\n \n+pub use item_id::ItemId;\n+pub use item_wire::{\n+ canonicalize_item, item_parent_path, item_path_segments, normalize_slug_ontology_storage_url,\n+ SLUG_TILDE_ONTOLOGY_ROOT,\n+};\n pub use paths::{\n- canonicalize_item, canonicalize_tag, item_parent_path, item_path_segments, normalize_slug_ontology_storage_url,\n- CanonicalItemUrl, ForumThreadUrl, GardenItemUrl, RelativePath, SLUG_TILDE_ONTOLOGY_ROOT,\n+ canonicalize_tag, ForumThreadUrl, GardenItemUrl, RelativePath,\n room_id_from_route_segment, room_route_segment, ROOM_SHORT_ID_LEN,\n- TildeHttpPathTail, TildeOntologyPath, TildePath, tilde_http_path_to_canonical,\n+ TildeHttpPathTail, TildeOntologyPath, TildePath, tilde_http_path_to_item_id,\n };\n pub use url_normalize::normalize_http_identity_url;\n \ndiff --git a/types/src/paths.rs b/types/src/paths.rs\nindex 5c60e762c2ca74d79c74237097d5bcc02ba74af2..a1d066d83e7fbcefc27707dd682e54dbcec1cc56 100644\n--- a/types/src/paths.rs\n+++ b/types/src/paths.rs\n@@ -3,37 +3,23 @@\n //!\n //! ## String kinds (parse in this module only)\n //!\n-//! - **[`canonicalize_item`] / [`CanonicalItemUrl`]** — graph storage key and DSL form; tilde\n+//! - **[`canonicalize_item`] / [`crate::ItemId`]** — graph storage key and DSL form; tilde\n //! ontology root is always [`SLUG_TILDE_ONTOLOGY_ROOT`] (no `…/~/` trailing slash only).\n //! - **[`TildeHttpPathTail`]** — capture from `GET /~/*path` or `…/r/{short}{slug}/~/…` (the `*path` segment).\n //! - **`-/…` wire form** — external items; see [`canonicalize_item`] dash branch.\n //! - **[`GardenItemUrl`], [`ForumThreadUrl`]** — JSON / browser href surfaces.\n //! - **[`ROOM_SHORT_ID_LEN`] / [`room_route_segment`]** — `/r/{short}{slug}` vs wire `short/slug`.\n \n-use std::borrow::Borrow;\n use std::fmt;\n use std::ops::Deref;\n \n use serde::{Deserialize, Serialize};\n \n-use crate::url_normalize::{host_preserves_dash_path_case, normalize_http_identity_url};\n-\n-// ---------------------------------------------------------------------------\n-// Slug tilde ontology (single storage form for `~/`)\n-// ---------------------------------------------------------------------------\n-\n-/// Canonical absolute URL for the tilde ontology **root** (`~/` in UI). Used as the\n-/// `item_children` parent key for top-level items and must match [`CanonicalItemUrl::ontology_root`].\n-pub const SLUG_TILDE_ONTOLOGY_ROOT: &str = \"https://slug.social/~\";\n-\n-/// Collapse legacy or parser variants of the ontology root to [`SLUG_TILDE_ONTOLOGY_ROOT`].\n-pub fn normalize_slug_ontology_storage_url(s: &str) -> String {\n- if s == \"https://slug.social/~/\" {\n- SLUG_TILDE_ONTOLOGY_ROOT.to_string()\n- } else {\n- s.to_string()\n- }\n-}\n+use crate::item_id::ItemId;\n+pub use crate::item_wire::{\n+ canonicalize_item, item_parent_path, item_path_segments, normalize_slug_ontology_storage_url,\n+ SLUG_TILDE_ONTOLOGY_ROOT,\n+};\n \n // ---------------------------------------------------------------------------\n // Private room HTTP path (`/r/{short}{slug}`; wire id remains `short/slug`)\n@@ -84,310 +70,10 @@ pub fn canonicalize_tag(input: &str) -> String {\n input.trim().trim_start_matches('#').to_lowercase()\n }\n \n-fn finalize_external_identity_url(s: String) -> String {\n- if s.starts_with(\"https://slug.social/\") {\n- return s;\n- }\n- let normalized = normalize_http_identity_url(&s).unwrap_or_else(|| s.clone());\n- strip_redundant_root_slash(&normalized).unwrap_or(normalized)\n-}\n-\n-/// `url::Url` serializes bare hosts with a `/` path; we keep host-only items slash-free for stable\n-/// keys matching the pre-normalizer spellings.\n-fn strip_redundant_root_slash(s: &str) -> Option {\n- let u = url::Url::parse(s).ok()?;\n- if u.path() == \"/\" && u.query().is_none() && u.fragment().is_none() {\n- let scheme = u.scheme();\n- let host = u.host_str()?;\n- return Some(match u.port() {\n- Some(p) => format!(\"{scheme}://{host}:{p}\"),\n- None => format!(\"{scheme}://{host}\"),\n- });\n- }\n- None\n-}\n-\n-/// Ontology item reference → canonical absolute URL on the slug host.\n-pub fn canonicalize_item(input: &str) -> String {\n- let s = input.trim();\n- if s.is_empty() {\n- return String::new();\n- }\n-\n- // External scope: `-/host/path` is the universal alias for `https://host/path`.\n- if let Some(rest) = s.strip_prefix(\"-/\") {\n- let (host, tail) = rest\n- .split_once('/')\n- .map_or((rest, \"\"), |(h, t)| (h, t));\n- let host = host.trim().to_lowercase();\n- if host.is_empty() {\n- return String::new();\n- }\n- let preserve_case = host_preserves_dash_path_case(&host);\n- return if tail.is_empty() {\n- finalize_external_identity_url(format!(\"https://{}\", host))\n- } else {\n- let path = tail\n- .trim_start_matches('/')\n- .trim_end_matches('/')\n- .split('/')\n- .filter_map(|seg| {\n- let t = seg.trim();\n- if t.is_empty() {\n- None\n- } else if preserve_case {\n- Some(t.to_string())\n- } else {\n- Some(t.to_lowercase())\n- }\n- })\n- .collect::>()\n- .join(\"/\");\n- finalize_external_identity_url(format!(\"https://{}/{}\", host, path))\n- };\n- }\n-\n- if let Some(rest) = s.strip_prefix(\"https://\") {\n- let (host, tail) = rest.split_once('/').map_or((rest, \"\"), |(h, t)| (h, t));\n- let host = host.trim().to_lowercase();\n- return finalize_external_identity_url(if tail.is_empty() {\n- format!(\"https://{}\", host)\n- } else {\n- format!(\"https://{}/{}\", host, tail)\n- });\n- }\n- if let Some(rest) = s.strip_prefix(\"http://\") {\n- let (host, tail) = rest.split_once('/').map_or((rest, \"\"), |(h, t)| (h, t));\n- let host = host.trim().to_lowercase();\n- return finalize_external_identity_url(if tail.is_empty() {\n- format!(\"http://{}\", host)\n- } else {\n- format!(\"http://{}/{}\", host, tail)\n- });\n- }\n-\n- let is_tilde = s.starts_with(\"~/\");\n- let rest = s.strip_prefix(\"~/\").or_else(|| s.strip_prefix(\"/\")).unwrap_or(s);\n-\n- let tail = rest\n- .split('/')\n- .filter_map(|seg| {\n- let t = seg.trim();\n- if t.is_empty() {\n- None\n- } else {\n- Some(t.to_lowercase())\n- }\n- })\n- .collect::>()\n- .join(\"/\");\n-\n- if is_tilde {\n- if tail.is_empty() {\n- return SLUG_TILDE_ONTOLOGY_ROOT.to_string();\n- }\n- format!(\"https://slug.social/~/{}\", tail)\n- } else if tail.is_empty() {\n- \"https://slug.social\".to_string()\n- } else {\n- format!(\"https://slug.social/{}\", tail)\n- }\n-}\n-\n-pub fn item_path_segments(input: &str) -> Vec {\n- let canonical = canonicalize_item(input);\n- if canonical.is_empty() {\n- return vec![];\n- }\n-\n- if let Some(rest) = canonical.strip_prefix(\"https://\") {\n- let (host, tail) = rest.split_once('/').map_or((rest, \"\"), |(h, t)| (h, t));\n- let mut out = vec![format!(\"https://{}\", host)];\n- out.extend(tail.split('/').filter(|s| !s.is_empty()).map(|s| s.to_string()));\n- return out;\n- }\n- if let Some(rest) = canonical.strip_prefix(\"http://\") {\n- let (host, tail) = rest.split_once('/').map_or((rest, \"\"), |(h, t)| (h, t));\n- let mut out = vec![format!(\"http://{}\", host)];\n- out.extend(tail.split('/').filter(|s| !s.is_empty()).map(|s| s.to_string()));\n- return out;\n- }\n-\n- canonical\n- .split('/')\n- .filter(|s| !s.is_empty())\n- .map(|s| s.to_string())\n- .collect()\n-}\n-\n-pub fn item_parent_path(input: &str) -> Option {\n- let segs = item_path_segments(input);\n- if segs.len() <= 1 {\n- return None;\n- }\n- Some(segs[..segs.len() - 1].join(\"/\"))\n-}\n-\n-fn external_display_dash_prefix(host_and_path: &str) -> String {\n- let (host, path) = host_and_path\n- .split_once('/')\n- .map_or((host_and_path, \"\"), |(h, p)| (h, p));\n- let host = host.trim().to_lowercase();\n- let path = path\n- .trim_end_matches('/')\n- .split('/')\n- .filter_map(|seg| {\n- let t = seg.trim();\n- if t.is_empty() {\n- None\n- } else {\n- Some(t.to_lowercase())\n- }\n- })\n- .collect::>()\n- .join(\"/\");\n- if path.is_empty() {\n- format!(\"-/{}\", host)\n- } else {\n- format!(\"-/{}\", format!(\"{}/{}\", host, path))\n- }\n-}\n-\n // ---------------------------------------------------------------------------\n // Storage + input path newtypes\n // ---------------------------------------------------------------------------\n \n-/// Canonical item identifier as produced by [`canonicalize_item`].\n-///\n-/// Shared across all scopes; room is not embedded. Usually\n-/// `https://slug.social/~/…` or an external `http(s)://…` URL item.\n-#[derive(Debug, Clone, PartialEq, Eq, PartialOrd, Ord, Hash, Serialize, Deserialize)]\n-pub struct CanonicalItemUrl(pub String);\n-\n-impl CanonicalItemUrl {\n- pub fn parse(input: &str) -> Option {\n- let c = canonicalize_item(input);\n- if c.is_empty() {\n- None\n- } else {\n- Some(Self(normalize_slug_ontology_storage_url(&c)))\n- }\n- }\n-\n- pub fn as_str(&self) -> &str {\n- &self.0\n- }\n-\n- /// Collapses legacy slug ontology root spellings so [`HashMap`] keys match the reducer graph.\n- pub fn normalized_storage(self) -> Self {\n- Self(normalize_slug_ontology_storage_url(self.as_str()))\n- }\n-\n- pub fn tilde_tail(&self) -> Option<&str> {\n- if let Some(tail) = self.0.strip_prefix(\"https://slug.social/~/\") {\n- return Some(tail);\n- }\n- if self.0 == SLUG_TILDE_ONTOLOGY_ROOT || self.0 == \"https://slug.social/~/\" {\n- return Some(\"\");\n- }\n- None\n- }\n-\n- pub fn last_segment(&self) -> &str {\n- self.0\n- .rsplit('/')\n- .find(|s| !s.is_empty())\n- .unwrap_or(self.0.as_str())\n- }\n-\n- pub fn ontology_root() -> Self {\n- Self(SLUG_TILDE_ONTOLOGY_ROOT.to_string())\n- }\n-\n- pub fn parent(&self) -> Option {\n- if self.tilde_tail().is_some() {\n- if self.tilde_tail().map(|t| t.is_empty()).unwrap_or(true) {\n- return None;\n- }\n- let last_slash = self.0.rfind('/')?;\n- let parent_str = &self.0[..last_slash];\n- if parent_str.is_empty() {\n- None\n- } else {\n- Some(Self(parent_str.to_string()))\n- }\n- } else if let Some(rest) = self.0.strip_prefix(\"https://\") {\n- Self::parent_http_url(\"https://\", rest)\n- } else if let Some(rest) = self.0.strip_prefix(\"http://\") {\n- Self::parent_http_url(\"http://\", rest)\n- } else {\n- None\n- }\n- }\n-\n- fn parent_http_url(scheme: &'static str, rest: &str) -> Option {\n- let (host, path) = rest.split_once('/').map_or((rest, \"\"), |(h, p)| (h, p));\n- let host = host.trim();\n- let path = path.trim_end_matches('/');\n- if path.is_empty() {\n- return None;\n- }\n- let parent_path = path.rsplit_once('/').map(|(p, _)| p).unwrap_or(\"\");\n- if parent_path.is_empty() {\n- Some(Self(format!(\"{scheme}{}\", host)))\n- } else {\n- Some(Self(format!(\"{scheme}{}/{}\", host, parent_path)))\n- }\n- }\n-\n- /// `-/` representation for external `https://…` items, `~/…` for slug ontology, else unchanged.\n- pub fn display_path(&self) -> String {\n- if let Some(tail) = self.tilde_tail() {\n- if tail.is_empty() {\n- return \"~/\".to_string();\n- }\n- return format!(\"~/{}\", tail);\n- }\n- if let Some(tail) = self.0.strip_prefix(\"https://\") {\n- if tail.starts_with(\"slug.social\") {\n- self.0.clone()\n- } else {\n- external_display_dash_prefix(tail)\n- }\n- } else if let Some(tail) = self.0.strip_prefix(\"http://\") {\n- if tail.starts_with(\"slug.social\") {\n- self.0.clone()\n- } else {\n- external_display_dash_prefix(tail)\n- }\n- } else {\n- self.0.clone()\n- }\n- }\n-\n- pub fn tilde_segments(&self) -> Vec<&str> {\n- match self.tilde_tail() {\n- Some(tail) if !tail.is_empty() => {\n- std::iter::once(\"~\")\n- .chain(tail.split('/').filter(|s| !s.is_empty()))\n- .collect()\n- }\n- Some(_) => vec![\"~\"],\n- None => vec![],\n- }\n- }\n-\n- /// `~/…` list label for ontology items (paths index, CLI).\n- pub fn tilde_list_label(&self) -> TildeOntologyPath {\n- TildeOntologyPath::from_stored(self)\n- }\n-\n- /// Absolute href for JSON/RPC and browsers for this stored id in `room`.\n- pub fn json_href(&self, room_wire: &str) -> GardenItemUrl {\n- GardenItemUrl::from_stored(self, room_wire)\n- }\n-}\n-\n /// HTTP route capture: path segment after `~/` in `GET /~/*path` or `…/r/{short}{slug}/~/…` (empty = ontology root).\n #[derive(Debug, Clone, PartialEq, Eq, Hash)]\n pub struct TildeHttpPathTail(pub String);\n@@ -401,13 +87,13 @@ impl TildeHttpPathTail {\n &self.0\n }\n \n- pub fn to_canonical(&self) -> CanonicalItemUrl {\n- tilde_http_path_to_canonical(self.as_str())\n+ pub fn to_item_id(&self) -> ItemId {\n+ tilde_http_path_to_item_id(self.as_str())\n }\n }\n \n-/// Map the router's tilde tail (e.g. `topic/a`, or empty for root) to a [`CanonicalItemUrl`].\n-pub fn tilde_http_path_to_canonical(path_segment: &str) -> CanonicalItemUrl {\n+/// Map the router's tilde tail (e.g. `topic/a`, or empty for root) to an [`ItemId`].\n+pub fn tilde_http_path_to_item_id(path_segment: &str) -> ItemId {\n let p = path_segment.trim_start_matches('/');\n let raw = if p.starts_with(\"http://\") || p.starts_with(\"https://\") {\n p.to_string()\n@@ -416,37 +102,7 @@ pub fn tilde_http_path_to_canonical(path_segment: &str) -> CanonicalItemUrl {\n } else {\n format!(\"~/{}\", p)\n };\n- CanonicalItemUrl::parse(&raw).unwrap_or_else(|| CanonicalItemUrl::ontology_root())\n-}\n-\n-impl fmt::Display for CanonicalItemUrl {\n- fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {\n- self.0.fmt(f)\n- }\n-}\n-\n-impl Borrow for CanonicalItemUrl {\n- fn borrow(&self) -> &str {\n- &self.0\n- }\n-}\n-\n-impl PartialEq for CanonicalItemUrl {\n- fn eq(&self, other: &str) -> bool {\n- self.0 == other\n- }\n-}\n-\n-impl PartialEq<&str> for CanonicalItemUrl {\n- fn eq(&self, other: &&str) -> bool {\n- self.0 == *other\n- }\n-}\n-\n-impl PartialEq for CanonicalItemUrl {\n- fn eq(&self, other: &String) -> bool {\n- &self.0 == other\n- }\n+ ItemId::parse(&raw).unwrap_or_else(ItemId::ontology_root)\n }\n \n #[derive(Debug, Clone, PartialEq, Eq, PartialOrd, Ord, Hash, Serialize, Deserialize)]\n@@ -468,8 +124,8 @@ impl TildePath {\n &self.0\n }\n \n- pub fn canonicalize(&self) -> Option {\n- CanonicalItemUrl::parse(&self.0)\n+ pub fn canonicalize(&self) -> Option {\n+ ItemId::parse(&self.0)\n }\n }\n \n@@ -496,7 +152,7 @@ impl RelativePath {\n &self.0\n }\n \n- pub fn join_under_ontology_root(&self, root: &CanonicalItemUrl) -> Option {\n+ pub fn join_under_ontology_root(&self, root: &ItemId) -> Option {\n let base = root.tilde_tail()?;\n let joined = if base.is_empty() {\n if self.0.is_empty() {\n@@ -509,7 +165,7 @@ impl RelativePath {\n } else {\n format!(\"~/{}/{}\", base.trim_end_matches('/'), self.0)\n };\n- CanonicalItemUrl::parse(&joined)\n+ ItemId::parse(&joined)\n }\n }\n \n@@ -545,14 +201,17 @@ impl GardenItemUrl {\n self.0\n }\n \n- /// Stored canonical id + RPC `room` field (`\"public\"` or `\"short/slug\"`).\n- pub fn from_stored(stored: &CanonicalItemUrl, room_wire: &str) -> Self {\n- Self(garden_href_string(stored.as_str(), room_wire))\n+ /// Stored item id + RPC `room` field (`\"public\"` or `\"short/slug\"`).\n+ pub fn from_stored(stored: &ItemId, room_wire: &str) -> Self {\n+ Self(garden_href_string(stored, room_wire))\n }\n \n /// Like [`Self::from_stored`] but accepts a string that may already be canonical.\n pub fn from_storage_str(stored: &str, room_wire: &str) -> Self {\n- Self(garden_href_string(stored, room_wire))\n+ let Some(id) = ItemId::parse(stored) else {\n+ return Self(api_path_or_url(stored));\n+ };\n+ Self(garden_href_string(&id, room_wire))\n }\n }\n \n@@ -570,18 +229,15 @@ impl Deref for GardenItemUrl {\n }\n }\n \n-fn garden_href_string(item: &str, room_wire: &str) -> String {\n+fn garden_href_string(c: &ItemId, room_wire: &str) -> String {\n let room = room_wire.trim();\n if room.is_empty() || room == \"public\" {\n- return api_path_or_url(item);\n+ return api_path_or_url(c.as_str());\n }\n let Some(room_seg) = room_route_segment(room) else {\n- return api_path_or_url(item);\n- };\n- let Some(c) = CanonicalItemUrl::parse(item) else {\n- return api_path_or_url(item);\n+ return api_path_or_url(c.as_str());\n };\n- let root = CanonicalItemUrl::ontology_root();\n+ let root = ItemId::ontology_root();\n let item_norm = c.as_str().trim_end_matches('/');\n let root_norm = root.as_str().trim_end_matches('/');\n if let Some(tail) = c.tilde_tail() {\n@@ -600,7 +256,7 @@ fn garden_href_string(item: &str, room_wire: &str) -> String {\n let tail = tail.strip_prefix(\"-/\").unwrap_or(tail.as_str());\n return format!(\"https://slug.social/r/{room_seg}/-/{tail}\");\n }\n- api_path_or_url(item)\n+ api_path_or_url(c.as_str())\n }\n \n /// Forum thread URL for JSON (`/t/…` or `/r/…/t/…` on slug.social).\n@@ -650,7 +306,7 @@ impl Deref for ForumThreadUrl {\n pub struct TildeOntologyPath(pub String);\n \n impl TildeOntologyPath {\n- pub fn from_stored(c: &CanonicalItemUrl) -> Self {\n+ pub fn from_stored(c: &ItemId) -> Self {\n Self(c.display_path())\n }\n \n@@ -679,22 +335,22 @@ mod tests {\n \n #[test]\n fn canonical_parent_deep() {\n- let c = CanonicalItemUrl::parse(\"~/a/b/c\").unwrap();\n+ let c = ItemId::parse(\"~/a/b/c\").unwrap();\n assert_eq!(c.parent().unwrap().as_str(), \"https://slug.social/~/a/b\");\n }\n \n #[test]\n fn canonical_parent_one_level() {\n- let c = CanonicalItemUrl::parse(\"~/a\").unwrap();\n+ let c = ItemId::parse(\"~/a\").unwrap();\n assert_eq!(c.parent().unwrap().as_str(), \"https://slug.social/~\");\n }\n \n #[test]\n fn canonical_parent_root_is_none() {\n- let root = CanonicalItemUrl::parse(\"~/\").unwrap();\n+ let root = ItemId::parse(\"~/\").unwrap();\n assert!(root.parent().is_none());\n assert_eq!(root.as_str(), SLUG_TILDE_ONTOLOGY_ROOT);\n- assert_eq!(root, CanonicalItemUrl::ontology_root());\n+ assert_eq!(root, ItemId::ontology_root());\n }\n \n #[test]\n@@ -705,49 +361,49 @@ mod tests {\n SLUG_TILDE_ONTOLOGY_ROOT.to_string()\n );\n assert_eq!(\n- CanonicalItemUrl::parse(\"https://slug.social/~/\")\n+ ItemId::parse(\"https://slug.social/~/\")\n .unwrap()\n .as_str(),\n SLUG_TILDE_ONTOLOGY_ROOT\n );\n- let legacy = CanonicalItemUrl(\"https://slug.social/~/\".to_string());\n+ let legacy = ItemId::opaque(\"https://slug.social/~/\".to_string());\n assert_eq!(legacy.normalized_storage().as_str(), SLUG_TILDE_ONTOLOGY_ROOT);\n }\n \n #[test]\n fn tilde_http_path_tail_maps_router_segment() {\n assert_eq!(\n- TildeHttpPathTail::new(\"\").to_canonical(),\n- CanonicalItemUrl::ontology_root()\n+ TildeHttpPathTail::new(\"\").to_item_id(),\n+ ItemId::ontology_root()\n );\n assert_eq!(\n- tilde_http_path_to_canonical(\"topic/x\").as_str(),\n+ tilde_http_path_to_item_id(\"topic/x\").as_str(),\n \"https://slug.social/~/topic/x\"\n );\n }\n \n #[test]\n fn display_path_slug_ontology_root() {\n- let r = CanonicalItemUrl::ontology_root();\n+ let r = ItemId::ontology_root();\n assert_eq!(r.display_path(), \"~/\");\n assert_eq!(r.tilde_tail(), Some(\"\"));\n }\n \n #[test]\n fn tilde_segments_deep() {\n- let c = CanonicalItemUrl::parse(\"~/a/b\").unwrap();\n+ let c = ItemId::parse(\"~/a/b\").unwrap();\n assert_eq!(c.tilde_segments(), vec![\"~\", \"a\", \"b\"]);\n }\n \n #[test]\n fn tilde_segments_root() {\n- let c = CanonicalItemUrl::parse(\"~/\").unwrap();\n+ let c = ItemId::parse(\"~/\").unwrap();\n assert_eq!(c.tilde_segments(), vec![\"~\"]);\n }\n \n #[test]\n fn tilde_segments_non_ontology_is_empty() {\n- let c = CanonicalItemUrl::parse(\"https://example.com/foo\").unwrap();\n+ let c = ItemId::parse(\"https://example.com/foo\").unwrap();\n assert_eq!(c.tilde_segments(), Vec::<&str>::new());\n }\n \n@@ -831,28 +487,28 @@ mod tests {\n }\n \n #[test]\n- fn canonical_item_url_parent_external_strips_last_segment() {\n- let c = CanonicalItemUrl::parse(\"https://spotify.com/track/1\").unwrap();\n+ fn item_id_parent_external_strips_last_segment() {\n+ let c = ItemId::parse(\"https://spotify.com/track/1\").unwrap();\n assert_eq!(\n c.parent().unwrap().as_str(),\n \"https://spotify.com/track\"\n );\n assert_eq!(\n- CanonicalItemUrl::parse(\"https://github.com/iss/1\")\n+ ItemId::parse(\"https://github.com/iss/1\")\n .unwrap()\n .parent()\n .unwrap()\n .as_str(),\n \"https://github.com/iss\"\n );\n- assert!(CanonicalItemUrl::parse(\"https://github.com\").unwrap().parent().is_none());\n+ assert!(ItemId::parse(\"https://github.com\").unwrap().parent().is_none());\n }\n \n #[test]\n fn display_path_roundtrips_dash_and_tilde() {\n- let ext = CanonicalItemUrl::parse(\"https://GitHub.com/org/Issue\").unwrap();\n+ let ext = ItemId::parse(\"https://GitHub.com/org/Issue\").unwrap();\n assert_eq!(ext.display_path(), \"-/github.com/org/issue\");\n- let tilde = CanonicalItemUrl::parse(\"~/Rust/Doc\").unwrap();\n+ let tilde = ItemId::parse(\"~/Rust/Doc\").unwrap();\n assert_eq!(tilde.display_path(), \"~/rust/doc\");\n }\n \n","role":"user"}],"model":"~x-ai/grok-latest"}