constitution · epochs · watch · epoch 3

comparison

c_8dc1a8119370 (tommy-mor) vs c_14dd365ffe22 (tommy-mor)

download prompt · raw event · cmp_32e86d0a5c1851

council reasoning

~anthropic/claude-sonnet-latest · winner A · 6:4 · permalink

Side A replaces an admittedly unreliable, 1800-line keystroke-transition parser graph (plus its dead ParserAction module and a Playwright race test needed only for that debounced input) with a small, testable ~90-line URL parser and a simple paste-and-go form, directly fixing a stated reliability problem with a much simpler design. Side B is a more ad-hoc refactor (bare '-theme' message) that both drops the retro theme feature and simultaneously reworks routing to a '/~/' path scheme, mixing unrelated concerns with less clear justification, though it is still a reasonable and tested change.

~x-ai/grok-latest · winner A · 3:1 · permalink

A replaces an unreliable ~1800-line keystroke transition graph, ParserAction surface, live morph/race JS, and Playwright race test with a small paste-and-go URL parser that redirects to ranking—fixing core navigation and deleting lasting complexity. B mainly drops dual-theme CSS/cookies and adds `/~/` browse_href routing (useful URL design, but narrower and partly cleanup).

openai/gpt-chat-latest · winner A · 4:1 · permalink

Side A replaces a large, unreliable keystroke-driven autocomplete graph with a much simpler paste-and-go flow: it removes the complex parser/action system, introduces a focused URL parser with validation and tests, updates the UI to redirect directly on successful parsing, and eliminates the associated race-prone client logic. Side B mainly refactors routing and theming by introducing canonical `/~/...` browse URLs (`browse_href`, new browse route) and removing theme infrastructure, which is useful cleanup but has less direct impact on core functionality and reliability.

sides

A — c_8dc1a8119370 (tommy-mor)

message

[529cc941] Replace autocomplete parser with paste-and-go navigate.

The keystroke transition graph was unreliable; a textarea plus Go button now parses pasted Reddit URLs and redirects to the subreddit ranking scope.

Co-authored-by: Cursor <cursoragent@cursor.com>

diff preview

diff --git a/AGENTS.md b/AGENTS.md
index 77f2e31d4e860a92a77255ca5106c8b6c4510ee7..36ee4c0ec700bffbe226ee775ba9cf59cf35c770 100644
--- a/AGENTS.md
+++ b/AGENTS.md
@@ -11,7 +11,6 @@ Single Rust web app **`sorter2-server`**: pairwise voting, rank-centrality ranki
 - **Rust 1.88+** is required (some transitive crates need a recent Cargo). The image may ship older `/usr/local/cargo` (1.83); use **rustup** and `rustup default 1.88.0` before building.
 - **System packages** for builds: `pkg-config`, `libssl-dev` (for `reqwest` / OpenSSL in integration tests and release builds).
 - **Clojure CLI 1.12.0.1530** (optional but used in CI): install from https://clojure.org/guides/install_clojure — needed for `./scripts/clj-test.sh` / Kaocha tests.
-- **Playwright browser** for the spel browser test (`test/parser_race.clj`): install once with `clojure -M -e "(com.microsoft.playwright.CLI/main (into-array String [\"install\" \"chromium\" \"--with-deps\"]))"`. The browser binary is cached under `~/.cache/ms-playwright`.
 
 ### Commands (see also `TEST.sh`)
 
diff --git a/server/src/api/ui_html.rs b/server/src/api/ui_html.rs
index 1339048c955c0a3eb5120381aaf031424cbed540..c4ab9d65c7b3cd42a5b4d093ba429993c101e9a8 100644
--- a/server/src/api/ui_html.rs
+++ b/server/src/api/ui_html.rs
@@ -8,7 +8,7 @@ use std::collections::HashMap;
 use crate::{
     html::{js_string_literal, ranking_panel, JsBuilder},
     parser::parse_reddit_url,
-    parser_render::parser_panel_morph,
+    parser_render::navigate_panel,
     state::AppState,
     ui_action::{parse_html_ui_from_form, HtmlUiAction},
 };
@@ -57,18 +57,23 @@ pub async fn post_ui_html(
                 .morph_selector("#ranking-panel", panel)
                 .into_response()
         }
-        HtmlUiAction::ParseQuery { query } => {
-            let action = parse_reddit_url(&query);
-            let panel = parser_panel_morph(&query, &action);
-            let mut js = JsBuilder::new().morph_selector("#parser-panel", panel);
-            if let Some(comp) = action.primary_completion() {
-                js = js.raw(&format!(
-                    "var __pi=document.getElementById('parser-input'); if(__pi){{__pi.dataset.completion={};}}",
-                    js_string_literal(comp)
-                ));
+        HtmlUiAction::ParseQuery { query } => match parse_reddit_url(&query) {
+            Ok(subreddit) => {
+                let dest = format!("/?sub={subreddit}");
+                JsBuilder::new()
+                    .raw(&format!(
+                        "window.location.href={};",
+                        js_string_literal(&dest)
+                    ))
+                    .into_response()
             }
-            js.into_response()
-        }
+            Err(message) => {
+                let panel = navigate_panel(&query, Some(&message));
+                JsBuilder::new()
+                    .morph_selector("#parser-panel", panel)
+                    .into_response()
+            }
+        },
     }
 }
 
diff --git a/server/src/html/mod.rs b/server/src/html/mod.rs
index 94a5cfb561d1c8464bd2782f7ffd4e0965ccf95d..9650d333d29c4ac94ceb407aee3ee00399c7f40b 100644
--- a/server/src/html/mod.rs
+++ b/server/src/html/mod.rs
@@ -11,8 +11,7 @@ use serde::Deserialize;
 
 use crate::{
     form_template::template_json_compact,
-    parser_action::ParserAction,
-    parser_render::parser_panel,
+    parser_render::navigate_panel,
     ranking::{top_bottom, RankedItem},
     reducer::GroupState,
     state::{normalize_scope, AppState},
@@ -336,10 +335,9 @@ pub async fn home(
     let empty = GroupState::new();
     let group = groups.get(&scope).unwrap_or(&empty);
 
-    let empty_action = ParserAction::suggest(String::new(), None);
     let body = html! {
         h1 { "sorter2" }
-        (parser_panel("", &empty_action))
+        (navigate_panel("", None))
         (vote_panel(&scope))
         (ranking_panel(&scope, group))
     };
diff --git a/server/src/lib.rs b/server/src/lib.rs
index fa423640d598f4ba97a5885d228e78d7b97f7a22..de8bca48cbf689cad22337883e7791966e7c4919 100644
--- a/server/src/lib.rs
+++ b/server/src/lib.rs
@@ -4,7 +4,6 @@ pub mod events;
 pub mod form_template;
 pub mod html;
 pub mod parser;
-pub mod parser_action;
 pub mod parser_render;
 pub mod path_types;
 pub mod ranking;
diff --git a/server/src/parser.rs b/server/src/parser.rs
index ea437c4434045b44cba04a2b3416df850db518e7..50571a59f0d3ece1e2538f88e00ef46ec40ea545 100644
--- a/server/src/parser.rs
+++ b/server/src/parser.rs
@@ -1,1811 +1,87 @@
-use std::collections::HashMap;
-use std::sync::OnceLock;
+//! Extract a subreddit name from a pasted Reddit URL or path.
 
-use crate::parser_action::{GuideOption, ParserAction, ScrollingSuggestion, Suggestion};
-
-// --- Core Abstractions ---
-
-/// Unique identifier for nodes in the graph
-type NodeId = &'static str;
-
-/// Pattern matching for edges
-#[derive(Debug, Clone)]
-pub enum EdgePattern {
-    /// Matches exact literal string
-    Literal(&'static str),
-    
-    /// Matches any prefix of a string and suggests the full string
-    /// e.g., PrefixOf("reddit.com") matches "r", "re", "red", "reddit", "reddit.com"
-    PrefixOf(&'static str),
-    
-    /// Captures a variable segment (e.g., subreddit name, username)
-    Variable(&'static str),
-    
-    /// Matches any string (wildcard)
-    Any,
-}
-
-impl EdgePattern {
-    /// Try to match this pattern against input, return (consumed_chars, captured_value)
-    fn matches(&self, input: &str) -> Option<(usize, Option<String>)> {
-        match self {
-            EdgePattern::Literal(lit) => {
-                if input.starts_with(lit) {
-                    Some((lit.len(), None))
-                } else {
-                    None
-                }
-            }
-            EdgePattern::PrefixOf(target) => {
-                // Check if input is a prefix of target
-                if target.starts_with(input) && !input.is_empty() {
-                    // It's a valid prefix
-                    Some((input.len(), None))
-                } else if input.starts_with(target) {
-                    // Full match
-                    Some((target.len(), None))
-                } else {
-                    None
-                }
-            }
-            EdgePattern::Variable(var_name) => {
-                // Consume until next '/' or end of string
-                let end = input.find('/').unwrap_or(input.len());
-                if end > 0 {
-                    let captured = input[..end].to_string();
-                    // Validate based on variable type
-                    if is_valid_variable(var_name, &captured) {
-                        Some((end, Some(captured)))
-                    } else {
-                        None
-                    }
-                } else {
-                    None
-                }
-            }
-            EdgePattern::Any => {
-                // Match everything until next '/' or end
-                let end = input.find('/').unwrap_or(input.len());
-                if end > 0 {
-                    Some((end, Some(input[..end].to_string())))
-                } else {
-                    None
-                }
-            }
-        }
-    }
-    
-    /// Get the completion suggestion for this pattern
-    fn completion(&self, partial: &str) -> Option<String> {
-        match self {
-            EdgePattern::PrefixOf(target) => {
-                if target.starts_with(partial) && partial != *target {
-                    Some(target.to_string())
-                } else {
-                    None
-                }
-            }
-            _ => None,
-        }
+pub fn parse_reddit_url(query: &str) -> Result<String, String> {
+    let q = query.trim();
+    if q.is_empty() {
+        return Err("Paste a Reddit URL or r/subreddit path".into());
     }
-}
 
-/// Edge in the graph
-pub struct Edge {
-    pattern: EdgePattern,
-    target: NodeId,
-    /// Optional description for autocomplete
-    description: Option<&'static str>,
-}
-
-/// Handler function for generating UI actions (Send + Sync so the graph can live in `OnceLock`).
-type Handler = Box<dyn Fn(&str, &str, &HashMap<String, String>) -> ParserAction + Send + Sync>;
-
-/// Node in the graph
-pub struct Node {
-    #[allow(dead_code)]
-    id: NodeId,
-    edges: Vec<Edge>,
-    handler: Option<Handler>,
-}
-
-/// The composable parser graph (immutable after `build`).
-pub struct Graph {
-    nodes: HashMap<NodeId, Node>,
-    root: NodeId,
-}
-
-// --- Graph Builder (Fluent API) ---
-
-pub struct GraphBuilder {
-    nodes: HashMap<NodeId, Node>,
-    current_node: Option<NodeId>,
-    root: NodeId,
-}
-
-impl GraphBuilder {
-    pub fn new() -> Self {
-        let mut nodes = HashMap::new();
-        nodes.insert(
-            "root",
-            Node {
-                id: "root",
-                edges: Vec::new(),
-                handler: None,
-            },
-        );
-
-        GraphBuilder {
-            nodes,
-            current_node: Some("root"),
-            root: "root",
-        }
+    if let Some(sub) = subreddit_after_prefix(q, "r/") {
+        return Ok(sub);
     }
 
-    /// Select a node to add edges to
-    pub fn at(mut self, node_id: NodeId) -> Self {
-        self.nodes.entry(node_id).or_insert_with(|| Node {
-            id: node_id,
-            edges: Vec::new(),
-            handler: None,
-        });
-        self.current_node = Some(node_id);
-        self
+    if let Some(sub) = subreddit_from_path_segment(q, "/r/") {
+        return Ok(sub);
     }
-    
-    /// Add an edge from the current node
-    pub fn edge(self, pattern: EdgePattern, target: NodeId) -> Self {
-        self.edge_with_desc(pattern, target, None)
-    }
-    
-    /// Add an edge with description
-    pub fn edge_with_desc(
-        mut self,
-        pattern: EdgePattern,
-        target: NodeId,
-        desc: Option<&'static str>,
-    ) -> Self {
-        let current = self.current_node.expect("No current node selected");
-
-        self.nodes.entry(target).or_insert_with(|| Node {
-            id: target,
-            edges: Vec::new(),
-            handler: None,
-        });
-
-        if let Some(node) = self.nodes.get_mut(current) {
-            node.edges.push(Edge {
-                pattern,
-                target,
-                description: desc,
-            });
-        }
 
-        self
-    }
-
-    /// Set handler for current node
-    pub fn handler<F>(mut self, handler: F) -> Self
-    where
-        F: Fn(&str, &str, &HashMap<String, String>) -> ParserAction + Send + Sync + 'static,
-    {
-        let current = self.current_node.expect("No current node selected");
-        if let Some(node) = self.nodes.get_mut(current) {
-            node.handler = Some(Box::new(handler));
-        }
-        self
-    }
-    
-    /// Build the final graph
-    pub fn build(self) -> Graph {
-        Graph {
-            nodes: self.nodes,
-            root: self.root,
-        }
-    }
+    Err("Could not find a subreddit in that URL".into())
 }
 
-// --- Parser Implementation ---
-
-impl Graph {
-    pub fn parse(&self, input: &str) -> ParserAction {
-        let normalized = input.trim().to_lowercase();
-        let mut state = ParserState {
-            input: &normalized,
-            cursor: 0,
-            current_node_id: self.root,
-            context: HashMap::new(),
-            original_query: input.to_string(),
-            current_prefix: String::new(),
-        };
-        
-        self.parse_recursive(&mut state)
-    }
-    
-    fn parse_recursive(&self, state: &mut ParserState) -> ParserAction {
-        let node = self
-            .nodes
-            .get(state.current_node_id)
-            .expect("Node not found in graph");
-
-        // If we've consumed all input, check for handler or suggestions
-        if state.cursor >= state.input.len() {
-            if let Some(handler) = &node.han

… preview truncated; 92,533 characters omitted

download full diff A

B — c_14dd365ffe22 (tommy-mor)

message

[8d5328d3] -theme

diff preview

diff --git a/server/src/api/ui_html.rs b/server/src/api/ui_html.rs
index 82b2aa51d21ada1d0d849d3ddfc3a81e4241d861..e74e42e0f0ffc4224f5ef5421b486a5f35434999 100644
--- a/server/src/api/ui_html.rs
+++ b/server/src/api/ui_html.rs
@@ -73,11 +73,7 @@ pub async fn post_ui_html(
                     ensure_partial_tree(&mut tree, &item);
                 }
                 let _ = state.ensure_node(&item).await;
-                let dest = if item.is_root() {
-                    "/".to_string()
-                } else {
-                    format!("/?item={}", item.as_str())
-                };
+                let dest = item.browse_href();
                 JsBuilder::new()
                     .raw(&format!(
                         "window.location.href={};",
diff --git a/server/src/html/mod.rs b/server/src/html/mod.rs
index c973cb718ac74b95570dabea76e24459417790b9..a8353c6de0cd6d268219e552b7ca1ba4e3000585 100644
--- a/server/src/html/mod.rs
+++ b/server/src/html/mod.rs
@@ -1,13 +1,10 @@
 use axum::{
     body::Body,
     extract::{Path, State},
-    http::{header, HeaderValue, StatusCode, Uri},
+    http::{header, StatusCode, Uri},
     response::{IntoResponse, Response},
-    Form,
 };
-use axum_extra::extract::cookie::CookieJar;
 use maud::{html, Markup, DOCTYPE};
-use serde::Deserialize;
 
 use crate::{
     form_template::template_json_compact,
@@ -15,96 +12,25 @@ use crate::{
     path_types::ItemId,
     ranking::{top_bottom, RankedItem},
     reducer::{GroupState, NodeState},
-    state::{parse_item_param, AppState},
+    state::AppState,
     ui_action::UI_RPC_FIELD,
 };
 
-const THEME_DEFAULT_CSS: &str = include_str!("../../static/theme_default.css");
-const THEME_RETRO_CSS: &str = include_str!("../../static/theme_retro.css");
+const SORTER_CSS: &str = include_str!("../../static/sorter.css");
 const SORTER_UI_JS: &str = include_str!("../../static/sorter_ui.js");
 
-pub const SORTER_THEME_COOKIE: &str = "sorter-theme";
-
-pub fn normalize_theme(raw: &str) -> &'static str {
-    match raw {
-        "retro" => "retro",
-        _ => "default",
-    }
-}
-
-pub fn theme_from_jar(jar: &CookieJar) -> &'static str {
-    jar.get(SORTER_THEME_COOKIE)
-        .map(|c| normalize_theme(c.value()))
-        .unwrap_or("default")
-}
-
-pub fn theme_next_from_uri(uri: &Uri) -> String {
-    uri.path_and_query()
-        .map(|pq| pq.as_str().to_string())
-        .filter(|s| !s.is_empty())
-        .unwrap_or_else(|| "/".to_string())
-}
-
-pub fn theme_cookie_header_value(theme: &str) -> HeaderValue {
-    let t = normalize_theme(theme);
-    let s = format!("{SORTER_THEME_COOKIE}={t}; Path=/; SameSite=Lax; Max-Age=31536000");
-    HeaderValue::from_str(&s).expect("theme cookie must be ASCII")
-}
-
-fn sanitize_theme_next(next: Option<&str>) -> String {
-    let s = next.unwrap_or("/").trim();
-    if s.starts_with('/') && !s.starts_with("//") && s.len() < 8192 {
-        s.to_string()
-    } else {
-        "/".to_string()
-    }
-}
-
-#[derive(Debug, Deserialize)]
-pub struct ThemeForm {
-    theme: String,
-    next: Option<String>,
-}
-
-pub async fn post_theme(Form(form): Form<ThemeForm>) -> impl IntoResponse {
-    let theme = normalize_theme(&form.theme);
-    let next = sanitize_theme_next(form.next.as_deref());
-    let loc =
-        HeaderValue::try_from(next.as_str()).unwrap_or_else(|_| HeaderValue::from_static("/"));
-    Response::builder()
-        .status(StatusCode::SEE_OTHER)
-        .header(header::LOCATION, loc)
-        .header(header::SET_COOKIE, theme_cookie_header_value(theme))
-        .body(Body::empty())
-        .expect("theme redirect response")
-}
-
 pub async fn serve_static(Path(filename): Path<String>) -> impl IntoResponse {
-    if filename == "sorter_ui.js" {
-        return Response::builder()
-            .status(StatusCode::OK)
-            .header(header::CONTENT_TYPE, "text/javascript; charset=utf-8")
-            .header(header::CACHE_CONTROL, "public, max-age=3600")
-            .body(SORTER_UI_JS.to_string())
-            .unwrap()
-            .into_response();
-    }
-
-    let theme = filename
-        .strip_prefix("theme_")
-        .and_then(|s| s.strip_suffix(".css"));
-
-    let css = match theme {
-        Some("default") => THEME_DEFAULT_CSS,
-        Some("retro") => THEME_RETRO_CSS,
+    let (content_type, body) = match filename.as_str() {
+        "sorter.css" => ("text/css; charset=utf-8", SORTER_CSS),
+        "sorter_ui.js" => ("text/javascript; charset=utf-8", SORTER_UI_JS),
         _ => return (StatusCode::NOT_FOUND, "static file not found").into_response(),
     };
 
     Response::builder()
         .status(StatusCode::OK)
-        .header(header::CONTENT_TYPE, "text/css; charset=utf-8")
+        .header(header::CONTENT_TYPE, content_type)
         .header(header::CACHE_CONTROL, "public, max-age=3600")
-        .body(css.to_string())
+        .body(body.to_string())
         .unwrap()
         .into_response()
 }
@@ -162,8 +88,7 @@ fn asset_version() -> &'static str {
     V.get_or_init(|| {
         use std::hash::{Hash, Hasher};
         let mut h = std::collections::hash_map::DefaultHasher::new();
-        THEME_DEFAULT_CSS.hash(&mut h);
-        THEME_RETRO_CSS.hash(&mut h);
+        SORTER_CSS.hash(&mut h);
         SORTER_UI_JS.hash(&mut h);
         format!("{:x}", h.finish())
     })
@@ -176,9 +101,9 @@ pub fn now_ms() -> i64 {
     t.as_millis() as i64
 }
 
-fn layout(title: &str, body: Markup, views: u64, theme: &str, theme_next: &str) -> Markup {
+fn layout(title: &str, body: Markup, views: u64) -> Markup {
     let ver = asset_version();
-    let css_href = format!("/static/theme_{theme}.css?v={ver}");
+    let css_href = format!("/static/sorter.css?v={ver}");
     let js_src = format!("/static/sorter_ui.js?v={ver}");
     html! {
         (DOCTYPE)
@@ -187,7 +112,7 @@ fn layout(title: &str, body: Markup, views: u64, theme: &str, theme_next: &str)
                 meta charset="utf-8";
                 meta name="viewport" content="width=device-width, initial-scale=1";
                 title { (title) }
-                link rel="stylesheet" href=(css_href) id="theme-stylesheet";
+                link rel="stylesheet" href=(css_href);
                 script src="https://unpkg.com/idiomorph@0.3.0/dist/idiomorph.min.js" {}
             }
             body class="home" {
@@ -196,21 +121,6 @@ fn layout(title: &str, body: Markup, views: u64, theme: &str, theme_next: &str)
                 }
                 div id="errors" {}
                 (body)
-                div id="controls" {
-                    a href="https://github.com/sortersocial/sorter2" id="src-link" { "src" }
-                    form id="sorter-theme-form" method="post" action="/theme" data-navigate="full" {
-                        input type="hidden" name="next" value=(theme_next);
-                        select id="theme-select" name="theme" onchange="this.form.submit()" aria-label="Theme" {
-                            @for (val, label) in [("default", "default"), ("retro", "retro")] {
-                                @if theme == val {
-                                    option value=(val) selected { (label) }
-                                } @else {
-                                    option value=(val) { (label) }
-                                }
-                            }
-                        }
-                    }
-                }
                 script src=(js_src) {}
             }
         }
@@ -218,11 +128,7 @@ fn layout(title: &str, body: Markup, views: u64, theme: &str, theme_next: &str)
 }
 
 fn item_href(id: &ItemId) -> String {
-    if id.is_root() {
-        "/".to_string()
-    } else {
-        format!("/?item={}", id.as_str())
-    }
+    id.browse_href()
 }
 
 fn segment_label(seg: &str) -> &str {
@@ -361,39 +267,10 @@ pub fn vote_panel(parent: &ItemId) -> Markup {
 }
 
 
-fn query_param(uri: &Uri, key: &str) -> Option<String> {
-    let q = uri.query()?;
-    q.split('&').find_map(|pair| {
-        let mut it = pair.splitn(2, '=');
-        if it.next()? == key {
-            Some(it.next().unwrap_or("").to_string())
-        } else {
-            None
-        }
-    })
-}
-
-pub async fn home(
-    State(state): State<AppState>,
-    jar: CookieJar,
-    uri: Uri,
-) -> impl IntoResponse {
+async fn item_page(state: AppState, uri: Uri, item: ItemId) -> Markup {
     let path = uri.path().to_string();
     state.views.increment(path.clone());
     let views = state.views.get_views(&path);
-    let theme = theme_from_jar(&jar);
-    let theme_next = theme_next_from_uri(&uri);
-
-    let item_raw = query_param(&uri, "item")
-        .or_else(|| query_param(&uri, "sub").map(|sub| {
-            if sub.is_empty() {
-                String::new()
-            } else {
-                format!("reddit.com/r/{sub}")
-            }
-        }))
-        .unwrap_or_default();
-    let item = parse_item_param(&item_raw);
 
     let tree = state.tree.read().await;
     let empty_node = NodeState::default();
@@ -408,5 +285,14 @@ pub async fn home(
         (vote_panel(&item))
         (ranking_panel(&item, group))
     };
-    layout("sorter2", body, views, theme, &theme_next)
+    layout("sorter2", body, views)
+}
+
+pub async fn home(State(state): State<AppState>, uri: Uri) -> impl IntoResponse {
+    item_page(state, uri, ItemId::root()).await
+}
+
+pub async fn browse(State(state): State<AppState>, uri: Uri) -> impl IntoResponse {
+    let item = ItemId::from_browse_uri(uri.path()).unwrap_or(ItemId::root());
+    item_page(state, uri, item).await
 }
diff --git a/server/src/lib.rs b/server/src/lib.rs
index 14e7cfbc38feb5d07aff859b64bce6e2ec45cf91..cd56743192919cf4dcea539b3d8873a21fb7e72b 100644
--- a/server/src/lib.rs
+++ b/server/src/lib.rs
@@ -30,9 +30,9 @@ pub fn create_app(state: AppState) -> Router {
     Router::new()
         .route("/healthz", get(|| async { "ok" }))
         .route("/static/:filename", get(crate::html::serve_static))
+        .route("/~/*item_path", get(crate::html::browse))
         .route("/", get(crate::html::home))
         .route("/ui", post(crate::api::ui_html::post_ui_html))
-        .route("/theme", post(crate::html::post_theme))
         .with_state(state)
         .layer(TraceLayer::new_for_http())
 }
diff --git a/server/src/path_types.rs b/server/src/path_types.rs
index b5c41444f7bcd4d8d289ed3af464bc3f14d0df99..12dce9888f5cd4e1a0974d12d6468368d0f775b9 100644
--- a/server/src/path_types.rs
+++ b/server/src/path_types.rs
@@ -90,6 +90,50 @@ impl ItemId {
         paths
     }
 
+    /// Full URL for the browser location bar after `/~/`.
+    pub fn to_browse_url(&self) -> String {
+        if self.is_root() {
+            return String::new();
+        }
+        if self.as_str().contains("://") {
+            return self.as_str().to_string();
+        }
+        if self
+            .segments()
+            .first()
+            .is_some_and(|s| s.contains('.'))
+        {
+            format!("https://{}", self.as_str())
+        } else {
+            self.as_str().to_string()
+        }
+    }
+
+    /// App route, e.g. `/~/https://reddit.com/r/rust`.
+    pub fn browse_href(&self) -> String {
+        if self.is_root() {
+            "/".to_string()
+        } else {
+            format!("/~/{}", self.to_browse_url())
+        }
+    }
+
+    /// Parse the tail after `/~/` in a request path.
+    pub fn from_browse_tail(tail: &str) -> ItemId {
+        let raw = normalize_browse_tail(tail);
+        if raw.is_empty() {
+            return ItemId::root();
+        }
+        ItemId::from_url(&raw)
+            .or_else(|| ItemId::parse(&raw))
+            .unwrap_or_else(|| ItemId::opaque(raw))
+    }
+
+    pub fn from_browse_uri(path: &str) -> Option<ItemId> {
+        path.strip_prefix("/~/")
+            .map(ItemId::from_browse_tail)
+    }
+
     fn canonicalize(raw: &str) -> Option<String> {
         let s = raw.trim();
         if s.is_empty() {
@@ -155

… preview truncated; 6,669 characters omitted

download full diff B

Hardlinks — judgments / attempts / prompt

prompt download

judgments

attempts

Prompt text is loaded only by the download route.