improve error handling for missing XML namespace declarat…

[predic8]

…ions

• Added detailed error messages for unmapped XML prefixes to help diagnose namespace issues.
• Refined exception handling and logging for better error clarity in XPath processing.

Summary by CodeRabbit

• Bug Fixes
  • Improved error reporting when expression evaluation fails, providing clearer messages for namespace-related configuration issues.
  • Enhanced diagnostics for unresolved XML namespace prefix errors, including detailed information about namespace declarations and configuration.
  • Better error messages to help users troubleshoot and resolve expression processing failures more effectively.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Warning

Rate limit exceeded

@christiangoerdes has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 4 minutes and 0 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between 9792175 and 056483b.

📒 Files selected for processing (1)

• core/src/main/java/com/predic8/membrane/core/interceptor/flow/IfInterceptor.java

📝 Walkthrough

Walkthrough

This pull request improves error handling and messaging in two interceptor-related components. It modifies how expression evaluation exceptions are reported in IfInterceptor and enhances XPath evaluation error diagnostics in XPathExchangeExpression with specialized handling for namespace prefix resolution issues.

Changes

Cohort / File(s)	Summary
Error Response Structure core/src/main/java/com/predic8/membrane/core/interceptor/flow/IfInterceptor.java	Changes exception reporting from detail() to title() in ProblemDetails response when expression evaluation fails on exchange
XPath Namespace Error Handling core/src/main/java/com/predic8/membrane/core/lang/xpath/XPathExchangeExpression.java	Wraps XPathExpressionException with conditional augmentation of exception detail; appends diagnostic information about unresolved namespace prefixes and available namespace declarations when applicable

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• Xpath builtin function and readme yaml #2485: Modifies the same XPathExchangeExpression.java file with related changes to how XPath evaluation errors are wrapped and reported.

Suggested reviewers

• rrayst
• christiangoerdes

Poem

🐰 Error messages, once cryptic and gray,
Now shine with clarity, brightening the way,
Namespaces tangled? We'll tell you what's wrong,
With detail and title, debugging's more strong! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and specifically relates to the main objective of the changeset—improving error handling for missing XML namespace declarations.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}