Feat: Add integrity check to cli by palukku · Pull Request #13848 · JabRef/jabref

palukku · 2025-09-09T22:34:21Z

Added the integrity check to jabkit.

Steps to test

You can test the different formats with the following commands:

.\gradlew.bat jabkit:run --args="check-integrity --output-format=csv --input=../jablib/src/main/resources/Chocolate.bib --porcelain"

.\gradlew.bat jabkit:run --args="check-integrity --output-format=txt --input=../jablib/src/main/resources/Chocolate.bib"

.\gradlew.bat jabkit:run --args="check-integrity --output-format=errorformat --input=../jablib/src/main/resources/Chocolate.bib"

Mandatory checks

I own the copyright of the code submitted and I license it under the MIT license
I manually tested my changes in running JabRef (always required)
[/] I added JUnit tests for changes (if applicable)
[/] I added screenshots in the PR description (if change is visible to the user)
I described the change in CHANGELOG.md in a way that is understandable for the average user (if change is visible to the user)
I checked the user documentation: Is the information available and up to date? If not, I created an issue at https://github.com/JabRef/user-documentation/issues or, even better, I submitted a pull request updating file(s) in https://github.com/JabRef/user-documentation/tree/main/en.

# Conflicts: # jabkit/src/main/java/org/jabref/cli/CheckIntegrity.java

palukku · 2025-09-09T22:49:02Z

Openrewrite misinterprets the NotificationService "notify" as an Object#notify...

trag-bot · 2025-09-12T09:02:13Z

@trag-bot didn't find any issues in the code! ✅✨

* upstream/main: Add integrity check to cli (#13848)

* upstream/main: New Crowdin updates (#13865) Fix unresponsive "Check Consistency" dialog (#13864) Fix several aux group issues (#13861) Add integrity check to cli (#13848) Add sbom file and remove external libraries (#13859) Fix Tagsfield in Keyword editor (#13858) Chore(deps): Bump jablib/src/main/resources/csl-styles (#13857) Chore(deps): Bump jablib/src/main/resources/csl-locales (#13856)

palukku added 6 commits September 5, 2025 21:28

extend CheckIntegrity

812fa4b

add check integrity to jabkit

82723a6

replace writer

da299eb

Merge branch 'main' into add-integrity-check-to-cli

ce3a15d

# Conflicts: # jabkit/src/main/java/org/jabref/cli/CheckIntegrity.java

fix imports

efebec9

fix format

c49f597

palukku mentioned this pull request Sep 9, 2025

add check-integrity to jabkit JabRef/user-documentation#590

Merged

trag-bot Bot reviewed Sep 9, 2025

View reviewed changes

Comment thread jablib/src/main/java/org/jabref/logic/importer/fileformat/BibtexParser.java Outdated

add separator character handling

805cc42

palukku added 3 commits September 10, 2025 00:50

remove exception

2d8c404

adapt languages

2e5b4e2

fix language tests

fd0d992

palukku requested a review from koppor September 10, 2025 19:39

palukku added 2 commits September 10, 2025 21:54

fix language tests 2

228838c

change field ranges in BibtexParser

743765d

calixtus reviewed Sep 10, 2025

View reviewed changes

Comment thread jabkit/src/main/java/org/jabref/cli/CheckIntegrity.java Outdated

palukku and others added 7 commits September 11, 2025 13:51

re-add CygWinPathConverter

244217c

fix modernizer

8fc9fd6

fix modernizer2

483b3f0

move fieldranges to parser

c0bf69b

added changelog entry

cf7924a

change HashMap to IdentityHashMap

4b9ac21

remove line

764c5ba

koppor added the project: gsoc label Sep 11, 2025

remove unused import

58180f7

koppor added component: JabKit [cli] component: integrity-checker labels Sep 11, 2025

Siedlerchr reviewed Sep 11, 2025

View reviewed changes

Comment thread jabkit/src/main/java/org/jabref/cli/CheckIntegrity.java Outdated

Siedlerchr reviewed Sep 11, 2025

View reviewed changes

Comment thread jabkit/src/main/java/org/jabref/cli/CheckIntegrity.java Outdated