Description

Fixes inconsistent serde_json::Value string representations when applying regex matchers. Specifically, it ensures that:

• Value::String does not include surrounding quotes,
• other scalar types (Number, Bool, Null) are stringified appropriately,
• Object and Array are treated as unsupported and result in an error.

Closes #1143

Motivation and Context

Previously, serde_json::Value::String("example") was converted using to_string(), resulting in "\"example\"", which broke regex patterns expecting plain strings. This PR fixes that by:

• Matching and extracting raw strings for Value::String,
• Applying to_string() to Number, Bool, and Null,
• Raising a user-friendly error if the field is an Object or Array.

This makes pattern matching robust and predictable when working with JSON input from commit metadata or remote PR context.

How Has This Been Tested?

• Added unit tests for Commit::parse to verify field-based parsing logic using both author and remote attributes.
• Verified correct parser behavior using the following commit_parsers configuration in cliff.toml:

commit_parsers = [
 { message = "$^", skip = true },
 { field = "author.name", pattern = "github-actions", skip = true },
 { field = "author.email", pattern = "github-actions@github.com", skip = true },
 { field = "committer.name", pattern = "github-actions", skip = true },
 { field = "committer.email", pattern = "github-actions@github.com", skip = true },
 { field = "remote.pr_title", pattern = "^feat", group = "<!-- 00 -->🚀 Features" },
 { field = "remote.pr_title", pattern = "^fix", group = "<!-- 01 -->🐛 Bug Fixes" },
 { field = "remote.pr_title", pattern = "^refactor", group = "<!-- 02 -->🚜 Refactor" },
 { field = "remote.pr_title", pattern = "^docs", group = "<!-- 03 -->📚 Documentation" },
 { field = "remote.pr_title", pattern = "^perf", group = "<!-- 04 -->⚡ Performance" },
 { field = "remote.pr_title", pattern = "^style", group = "<!-- 05 -->🎨 Styling" },
 { field = "remote.pr_title", pattern = "^test", group = "<!-- 06 -->🧪 Testing" },
 { field = "remote.pr_title", pattern = "^chore\\(release\\)", skip = true },
 { field = "remote.pr_title", pattern = "^chore\\(deps.*\\)", skip = true },
 { field = "remote.pr_title", pattern = "^chore\\(pr\\)", skip = true },
 { field = "remote.pr_title", pattern = "^chore\\(pull\\)", skip = true },
 { field = "remote.pr_title", pattern = "^(chore|ci)", group = "<!-- 07 -->⚙️ Miscellaneous Tasks" },
 { field = "remote.pr_title", pattern = "^revert", group = "<!-- 08 -->◀️ Revert" },
 { field = "remote.pr_title", pattern = ".*", group = "<!-- 09 -->💼 Other" },
]

Screenshots / Logs (if applicable)

N/A

Types of Changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
• Documentation (no code change)
• Refactor (refactoring production code)
• Other

Checklist:

• My code follows the code style of this project.
• I have updated the documentation accordingly.
• I have formatted the code with rustfmt.
• I checked the lints with clippy.
• I have added tests to cover my changes.
• All new and existing tests passed.