feat: RichBlock enum + RichMessageParser HTML parser (#29, #36) #131

Closed

charles wants to merge 4 commits from feat/29-36-rich-parser into main

charles commented

2026-04-06 02:14:41 +00:00

Owner

Summary

Adds RichBlock enum covering all 13 block types: Chart, Quiz, PollRich, CodeRun, Card, Carousel, Timeline, TableSort, Accordion, Progress, Math, Mermaid, Canvas (crates/fractal-rich/src/block.rs)
Adds RichMessageParser::parse(html) -> Vec<MessageFragment> with Html(String) / Rich { block, fallback } variants, plus RenderLevel (full / static / text) (crates/fractal-rich/src/parser.rs)
Fallback to MessageFragment::Html on unknown data-fx-type or invalid data-fx-config JSON
Adds scraper = "0.20", serde, serde_json to fractal-rich/Cargo.toml
Exposes all public types via lib.rs

Test plan

cargo test -p fractal-rich — all unit tests pass (block deserialization, unknown type, invalid JSON, text render level, plain HTML passthrough)
Verify fallback: feed a <div data-fx-type="unknown-widget"> and confirm MessageFragment::Html is returned
Verify RenderLevel::Text strips all rich blocks and returns plain text

Closes #29, closes #36

## Summary - Adds `RichBlock` enum covering all 13 block types: Chart, Quiz, PollRich, CodeRun, Card, Carousel, Timeline, TableSort, Accordion, Progress, Math, Mermaid, Canvas (`crates/fractal-rich/src/block.rs`) - Adds `RichMessageParser::parse(html) -> Vec<MessageFragment>` with `Html(String)` / `Rich { block, fallback }` variants, plus `RenderLevel` (`full` / `static` / `text`) (`crates/fractal-rich/src/parser.rs`) - Fallback to `MessageFragment::Html` on unknown `data-fx-type` or invalid `data-fx-config` JSON - Adds `scraper = "0.20"`, `serde`, `serde_json` to `fractal-rich/Cargo.toml` - Exposes all public types via `lib.rs` ## Test plan - [ ] `cargo test -p fractal-rich` — all unit tests pass (block deserialization, unknown type, invalid JSON, text render level, plain HTML passthrough) - [ ] Verify fallback: feed a `<div data-fx-type="unknown-widget">` and confirm `MessageFragment::Html` is returned - [ ] Verify `RenderLevel::Text` strips all rich blocks and returns plain text Closes #29, closes #36

charles added 4 commits

2026-04-06 02:14:41 +00:00

chore(fractal-rich): add serde, serde_json, scraper dependencies d1eb52bd5d

feat(fractal-rich): add RichBlock enum with all supported block types (#29 ) 68d05eef8f

feat(fractal-rich): add RichMessageParser with fallback support (#36 ) cd9aa8520c

feat(fractal-rich): expose block and parser modules in lib.rs

CI / cargo check (pull_request) Has been cancelled

Details

CI / cargo test (pull_request) Has been cancelled

Details

CI / clippy + rustfmt (pull_request) Has been cancelled

Details

98337438ab

charles closed this pull request

2026-04-06 02:21:49 +00:00

CI / cargo check (pull_request) Has been cancelled

Details

CI / cargo test (pull_request) Has been cancelled

Details

CI / clippy + rustfmt (pull_request) Has been cancelled

Details

Required

Pull request closed

Please reopen this pull request to perform a merge.

No reviewers

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

charles/my-fractal!131

No description provided.

Rows
Columns