Skip to content

fix(doc): add tracing context to parallel read#399

Merged
argl merged 4 commits intomainfrom
393-doc-reader-context
Nov 28, 2025
Merged

fix(doc): add tracing context to parallel read#399
argl merged 4 commits intomainfrom
393-doc-reader-context

Conversation

@caugner
Copy link
Contributor

@caugner caugner commented Nov 27, 2025

Description

Adds missing tracing context to parallel read.

Motivation

Make the error message actionable.

Additional details

Before

Building everything 🛠️
ERROR rari_doc::reader: Invalid frontmatter: duplicate field `title`
Error: Fatal error reading docs

After

Building everything 🛠️
ERROR page{file="/path/to/mdn/content/files/en-us/web/index.md"}: rari_doc::reader: Invalid frontmatter: duplicate field `title`
Error: Fatal error reading docs

Related issues and pull requests

Fixes #393.

@caugner caugner requested a review from a team as a code owner November 27, 2025 13:50
@caugner caugner requested a review from argl November 27, 2025 13:50
@caugner caugner force-pushed the 393-doc-reader-context branch from f6ac76c to 96dee63 Compare November 28, 2025 10:37
@caugner caugner requested a review from argl November 28, 2025 10:38
@argl argl merged commit c9c4687 into main Nov 28, 2025
6 checks passed
@argl argl deleted the 393-doc-reader-context branch November 28, 2025 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

"Invalid frontmatter" errors are missing context

2 participants