id(web): ordered lists with ) markers are rewritten to . on save — list delimiter style is dropped by the codec

Summary

An ordered list written with parenthesis markers (1) one) is silently rewritten to period markers (1. one) the first time the document is saved from the web editor. The rendered list is identical, but the author's source style is destroyed on every open→save of such a document — the same class of silent source rewriting as #997 (start number) and #1036 (tight/loose), for the one list attribute the codec still doesn't carry.

Evidence (verified this run, main `17c2be5d`)

Round-trip through the verbatim codec (probe crate, hidden_files/probe-2026-10-27-creator-a/out5.log, case paren-delim-ordered; control in probe8):

Codec: 1) one\n2) two\n → PM ordered_list with attrs {order: 1, tight: true} (no delimiter) → save → 1. one\n2. two\n (fixpoint thereafter — the ) form is unrecoverable after one save).
DIRECT comrak control (parse → format_commonmark, no codec): 1) one\n2) two\n round-trips unchanged, so the loss is in the codec, not the renderer.

Root cause, pkgs/id/src/web/markdown.rs:

Parse direction, NodeValue::List arm (:254-264): ordered_list attrs store only order (list.start) and tight. comrak's NodeList carries the marker style in delimiter: ListDelimType (Period/Paren, comrak nodes.rs:373) — it is never read. The ordered task_list arm (:234-240) drops it the same way.
Save direction, ordered_list arm (~~:1200-1211) and ordered task_list arm (~~:1238-1244): NodeList { list_type: ListType::Ordered, ..Default::default() } — Default is ListDelimType::Period, so every ordered list is emitted with ..
pkgs/id/web/src/editor.ts: the ordered_list/task_list schema likewise has no delimiter attr, so the editor cannot round-trip it either (cf. the order/tight attrs added there for id(web): ordered task lists lose their ordering on save — rewritten as bullet task lists #997/id(web): loose task list items fuse on save — task_list hardcodes tight, second paragraph/code block destroyed #1036).

Why it matters

Silent, irreversible source churn for any document authored with ) markers (a common style, and the default in some editors that users paste/import from).
It is the last uncarried NodeList field: start (id(web): ordered task lists lose their ordering on save — rewritten as bullet task lists #997) and tight (id(web): loose task list items fuse on save — task_list hardcodes tight, second paragraph/code block destroyed #1036) were both fixed by carrying the attr both directions; delimiter was left behind.

Acceptance criteria

1) one\n2) two\n round-trips byte-identically through markdown → PM → markdown (codec test), and is a fixpoint.
1. one lists render exactly as before (no change for period-style lists).
Ordered task lists keep their delimiter too (e.g. 1) [ ] x if the parser accepts the form; otherwise the exclusion is noted in the test).
Legacy PM docs without the new attr save exactly as today (delimiter defaults to period).
editor.ts ordered_list (and task_list, if applicable) schema carries the attr with a period default, mirroring the id(web): ordered task lists lose their ordering on save — rewritten as bullet task lists #997/id(web): loose task list items fuse on save — task_list hardcodes tight, second paragraph/code block destroyed #1036 pattern.
Existing web::markdown tests still pass.

Filed by new-issue-creator (lane A) after a verified probe on current main; delimiter keyword search across open and closed issues returned no existing tracker.

CODE-475

🤖 Agent note · state: filed

Agent: new-issue-creator (lane A, new-issue-creator-code-a)
Session / ID: cron new-issue-creator-code-a run 2026-10-27 04:07 CDT
Found via: filed this run
Task: Ordered lists with ) markers are rewritten to . on save — codec drops comrak's NodeList delimiter in both directions (parse markdown.rs ~:254, save ~:1200); verified by probe + direct-comrak control on main 17c2be5.
Expectation: effort/s — carry a delimiter attr both directions (codec + editor.ts), period default for legacy docs; done = acceptance criteria on the issue pass.
Links: probe log hidden_files/probe-2026-10-27-creator-a/out5.log (case paren-delim-ordered) + probe8 DIRECT control

🤖 Agent note · parked

Agent: queue worker (repo-worker-code), lane H (code-h)
Session / ID: task 6504
Found via: queue claim (resume)
Task: parked 20m — CI pending on PR fix(id/web): carry ordered-list delimiter style through codec (#1380) #4666 at new head ecb7a49: after the 5-minute active poll no workflow run exists yet for the new head (only CodeRabbit, neutral/skipped); the prior head's basic guard run had been queued since 2026-11-03T00:05Z. This is a CI queue delay, not a failure.
Progress this resume: the undispositioned Codex P2 on PR fix(id/web): carry ordered-list delimiter style through codec (#1380) #4666 was valid — fixed in ecb7a49 (editor.test.ts round-trip expectations now include the delimiter default; verified 2 failed before / 117 pass after on the laptop) and replied on the Codex thread.
Expectation: on resume, read PR fix(id/web): carry ordered-list delimiter style through codec (#1380) #4666 checks and any new Codex review for head ecb7a49 fresh; merge under standing criteria when green, close issue id(web): ordered lists with ) markers are rewritten to . on save — list delimiter style is dropped by the codec #1380, complete task 6504.
Links: PR fix(id/web): carry ordered-list delimiter style through codec (#1380) #4666 · branch queue/6504-1380-ordered-list-delimiter

🤖 Agent note · state: picked up

Agent: queue worker (repo-worker-code), lane D shift
Session / ID: task 6504
Found via: queue claim
Task: Resume task 6504 for issue id(web): ordered lists with ) markers are rewritten to . on save — list delimiter style is dropped by the codec #1380 at merge_ready: PR fix(id/web): carry ordered-list delimiter style through codec (#1380) #4666 head ecb7a49 — Codex P2 dispositioned (fixed in ecb7a49, reply on the Codex thread), targeted tests pass (web::markdown 225 pass + 1 pre-existing clean-main failure; editor.test.ts 117 pass; 3 delimiter regressions pass), diff is exactly the 3 in-scope files. No Actions run exists for the current head (only CodeRabbit neutral) — merging under the standing merge authority (CI runs on main only; PRs do not wait for CI).
Expectation: effort s — PR fix(id/web): carry ordered-list delimiter style through codec (#1380) #4666 merged, issue id(web): ordered lists with ) markers are rewritten to . on save — list delimiter style is dropped by the codec #1380 closed, task 6504 completed.
Links: PR fix(id/web): carry ordered-list delimiter style through codec (#1380) #4666; issue id(web): ordered lists with ) markers are rewritten to . on save — list delimiter style is dropped by the codec #1380

Merge pull request #4666 from developing-today/queue/6504-1380-ordere…

🤖 Agent note · state: done

Agent: queue worker (repo-worker-code), lane D shift
Session / ID: task 6504
Found via: queue claim
Task: Task 6504 for issue id(web): ordered lists with ) markers are rewritten to . on save — list delimiter style is dropped by the codec #1380 is done: PR fix(id/web): carry ordered-list delimiter style through codec (#1380) #4666 merged (merge commit d05c22e) and issue id(web): ordered lists with ) markers are rewritten to . on save — list delimiter style is dropped by the codec #1380 closed by the merge.
Expectation: Done — ordered-list delimiter style now round-trips through the codec and the editor schema (period default for legacy docs).
Links: PR fix(id/web): carry ordered-list delimiter style through codec (#1380) #4666 (merged); issue id(web): ordered lists with ) markers are rewritten to . on save — list delimiter style is dropped by the codec #1380 (closed)
Evidence: Codex P2 on PR fix(id/web): carry ordered-list delimiter style through codec (#1380) #4666 was fixed in ecb7a49 and replied on its thread; targeted tests from the working checkpoints: web::markdown 225 pass (+1 pre-existing failure reproduced on clean main), editor.test.ts 117 pass, 3 delimiter regressions pass. No Actions run existed for the final head (CodeRabbit neutral only); merged under the standing merge authority.

Navigation Menu

Repository navigation

id(web): ordered lists with ) markers are rewritten to . on save — list delimiter style is dropped by the codec #1380

Summary

Evidence (verified this run, main `17c2be5d`)

Why it matters

Acceptance criteria

74 remaining items

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

id(web): ordered lists with ) markers are rewritten to . on save — list delimiter style is dropped by the codec #1380

Description

Summary

Evidence (verified this run, main 17c2be5d)

Why it matters

Acceptance criteria

Activity

developing-today commented on Oct 27, 2026

dezren39 commented on Oct 27, 2026

74 remaining items

dezren39 commented on Nov 3, 2026

dezren39 commented on Nov 3, 2026

dezren39 commented on Nov 3, 2026

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions

Evidence (verified this run, main `17c2be5d`)