Skip to content

Improve KeywordList parsing to support escaped characters and nested structures #12888

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 9 commits into
base: main
Choose a base branch
from

Merge branch 'main' into fix-for-issue-12810

85cb7c7
Select commit
Loading
Failed to load commit list.
Draft

Improve KeywordList parsing to support escaped characters and nested structures #12888

Merge branch 'main' into fix-for-issue-12810
85cb7c7
Select commit
Loading
Failed to load commit list.
Trag bot / Trag Review succeeded May 9, 2025 in 1m 24s

Trag Code Review

Reviewed files details

Details

[2025-05-09T21:29:10.987Z] code review started
[2025-05-09T21:29:10.987Z] owner: JabRef
[2025-05-09T21:29:10.987Z] repo: jabref
[2025-05-09T21:29:10.987Z] repoUrl: https://github.com/JabRef/jabref
[2025-05-09T21:29:10.987Z] author: krishnagjsForGit
[2025-05-09T21:29:10.987Z] listing pull request files
[2025-05-09T21:29:23.095Z] total file count: 2
[2025-05-09T21:29:23.095Z] eligible file count: 2
[2025-05-09T21:29:23.145Z] pro user privilege applied
[2025-05-09T21:29:23.145Z] getting project rules
[2025-05-09T21:29:23.159Z] model: claude-3-5-sonnet-20240620
[2025-05-09T21:29:23.159Z] on rule review mode: true
[2025-05-09T21:29:23.159Z] glob ignore:
[2025-05-09T21:29:23.159Z] pull number: 12888
[2025-05-09T21:29:23.159Z] projectId: 885262de-6ea4-406f-ab33-0a18e077aaf9
[2025-05-09T21:29:40.770Z] Found 20 existing review comments
[2025-05-09T21:29:40.770Z] file: jablib/src/main/java/org/jabref/model/entry/KeywordList.java
[2025-05-09T21:29:40.770Z] file is outside of new changes on pr, skipping
[2025-05-09T21:29:40.770Z] file: jablib/src/test/java/org/jabref/model/entry/KeywordListTest.java
[2025-05-09T21:29:40.770Z] file is outside of new changes on pr, skipping
[2025-05-09T21:29:40.770Z] files scanned: 0
[2025-05-09T21:29:40.770Z] lines scanned: 0
[2025-05-09T21:29:40.770Z] rules for project: 60
[2025-05-09T21:29:40.770Z] issues caught by your rules: 0