commit

c_9f1f400432118aa1f8

tommy-mor · sha1:dc0ea8dac30c28544f5a9cba47ea518d03376c59

message

Share thread XML formatting and expose delegate on forum thread items.

Unify CLI forum show and browser copy-thread output via slug_types::thread_xml, adding principal and delegate attributes on each post tag and extending ThreadItem::Post in the RPC response.

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

comparisons involving this commit

comparison · c_bc221ffb9207 (tommy-mor) vs c_9f1f40043211 (tommy-mor)

~anthropic/claude-sonnet-latest · winner B · 3:2 · permalink

Side B is a focused, tested refactor that deduplicates thread-XML formatting across CLI and server, adds a needed delegate field with proper escaping and unit tests. Side A dumps a large untested 572-line script plus deploy/README churn under a vague 'nice' message, mixing unrelated concerns (agent framework + Fly deploy tweaks) without tests or clear integration into the existing codebase.

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

A adds the full adam.py runtime (init/load, LLM I/O, voting, ranking-backed compaction, CLI) plus concrete Open WebUI Fly fixes (VM/swap, secure cookies, deploy script and troubleshooting), which define core product and operability. B’s shared thread_xml module, delegate on ThreadItem/RPC, and DRY CLI/copy paths are real lasting cleanup with tests, but they only refine forum presentation rather than introduce comparable capability.

openai/gpt-chat-latest · winner B · 3:2 · permalink

Side B makes a focused architectural improvement by introducing a shared `thread_xml` formatter used by both the CLI and browser copy path, eliminating duplicated formatting logic while also extending the RPC `ThreadItem::Post` with `delegate` metadata and adding tests for XML escaping and output. Side A adds a large new `adam.py` tool and deployment documentation/scripts, but much of the patch is new application code and operational guidance rather than an improvement to existing project structure, making B's reusable refactor and API enhancement the more durable contribution.

comparison · c_9f1f40043211 (tommy-mor) vs c_b2cf41238f57 (tommy-mor)

~anthropic/claude-sonnet-latest · winner A · 8:1 · permalink

Side A refactors duplicated XML formatting logic into a shared, tested module (thread_xml.rs) and adds real functionality (exposing delegate/principal on posts) used across CLI and server code, with proper escaping and unit tests. Side B merely trims/removes task definitions from a build config file, which is minor housekeeping with no functional impact and even removes test-running tasks.

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

Side A extracts shared thread XML formatting, unifies CLI/browser output, adds principal/delegate to ThreadItem and RPC, and includes escaping plus tests—real lasting API and design work. Side B only deletes/renames babashka tasks in bb.edn, which is minor config churn with little durable product value.

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

Side A introduces a shared thread_xml module used by both the CLI and browser copy output, reducing duplicated formatting logic, extends the RPC ThreadItem::Post with a delegate field, and adds principal/delegate XML attributes with escaping and tests. Side B is only a configuration cleanup that removes task definitions and renames a Babashka task, providing little lasting functional value.

The full patch is loaded only by the download route: download patch

Metadata

commit_idc_9f1f400432118aa1f8a65182c16f0f9e4659631c8d5ae6a392c6c7bbb1ea2cc3

patch_sha256d5e9173ee3ca96edac84ac147507cdc0684f74cee362e92dd80ae002c1e7abdb

patch_identitygit-patch-id-stable-v1:54fed407c7f02daa3a045125f6de86388387458f

committer_timestamp_ms1779567111000