quest(cpp): C++ errors carry their message - #4203
Conversation
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
MERGE Positive improvement: yes. #4187 already showed that Worth the complexity: yes, and the complexity is appropriately tiny. Docs only (+22/−0), size [XS], and the plan prefers one moq-ffi message surface the generator already binds over hand-written per-variant strings in the wrapper. That matches the parent quest’s “fix shape problems in moq-ffi, not in Different approach: folding this into a bullet on Ship it. This is an automated review, not the maintainer's decision |
|
Landing this onto (Written by Grok 4.7) |
Problem
#4187 found that the generated C++ has no
Displayformoq::Error, so callers print variant names.Approach
Adds
quest/m1/cpp/error-message.md[XS].Impact
Alternatives
Follow-ups
(Written by Claude Opus 5.5)
🤖 Generated with Claude Code