Hey @addix7! 👋

Thank you for contributing to JabRef!

We have automated checks in place, based on which you will soon get feedback if any of them are failing. We also use Qodo for review assistance. It will update your pull request description with a review help and offer suggestions to improve the pull request.

After all automated checks pass, a maintainer will also review your contribution. Once that happens, you can go through their comments in the "Files changed" tab and act on them, or reply to the conversation if you have further inputs. You can read about the whole pull request process in our contribution guide.

Please ensure that your pull request is in line with our AI Usage Policy and make necessary disclosures.

PR Compliance Guide 🔍

Below is a summary of compliance checks for this PR:

private PreviewPreferences(){
    this(
            List.of(),
            0,
            new TextBasedPreviewLayout("", null, null),
            "",
            false,
            false,
            List.of(),
            false
    );

PR Code Suggestions ✨

Explore these optional code suggestions:

 private PreviewPreferences(){
     this(
             List.of(),
             0,
-            new TextBasedPreviewLayout("", null, null),
+            new TextBasedPreviewLayout(
+                    "",
+                    org.jabref.preferences.PreferencesService.get().getLayoutFormatterPreferences(),
+                    org.jabref.gui.Injector.instantiateModelOrService(org.jabref.logic.journals.JournalAbbreviationRepository.class)),
             "",
             false,
             false,
             List.of(),
             false
     );
 }

 public void setAll(PreviewPreferences preferences){
+    if (preferences == null) {
+        throw new IllegalArgumentException("Preferences must not be null");
+    }
     this.layoutCycle.setAll(preferences.getLayoutCycle());
     this.layoutCyclePosition.set(preferences.getLayoutCyclePosition());
     this.customPreviewLayout.set(preferences.getCustomPreviewLayout());
     this.defaultCustomPreviewLayout.set(preferences.getDefaultCustomPreviewLayout());
     this.showPreviewAsExtraTab.set(preferences.shouldShowPreviewAsExtraTab());
     this.showPreviewEntryTableTooltip.set(preferences.shouldShowPreviewEntryTableTooltip());
     this.bstPreviewLayoutPaths.setAll(preferences.getBstPreviewLayoutPaths());
     this.shouldDownloadCovers.set(preferences.shouldDownloadCovers());
 }

Hello again !
I have addressed the automated feedback:
Formatting: Reformatted the code using IntelliJ to pass the Checkstyle tests.
Changelog: Added the fix entry to CHANGELOG.md with the proper link format.
Mandatory Checks: Restored the checklist in the PR description.

I'm aware of the 'main branch' warning and will use a dedicated branch for future PRs as suggested. Ready for review!"

Hello again,
I have completed the reformatting and fixed the compilation issues. The PR is now ready for review. Note: I am aware of the "main" branch warning and will use feature branches for future contributions as suggested by the bot.

Your pull request conflicts with the target branch.

Please merge with your code. For a step-by-step guide to resolve merge conflicts, see https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/addressing-merge-conflicts/resolving-a-merge-conflict-using-the-command-line.

Hello sir, @koppor
I am extremely sorry you felt this way, i admit i used ai but only to understand, this is a new prospect for me and honestly i was overwhelmed, but i assure you, the changes i made, i have knowledge of them, I used ai just to understand the code and the problem,  and i accidentally reformatted the whole code and particularly files- jabguipreferences.java, previewpreferences.java,
My only motive is to understand and help

But I want to assure you that i didnt use ai to reformat code, Instead I used IntelliJ Reformat File and optimise imports , i thought by using this, my code will clear the error
If you could guide, what to do next, I would be extremely grateful

i have manually undo the reformatting of both the files, but still some problem persists

@addix7 you can just click on "Source Cod Tests / Checkstyle" and read the output

https://github.com/JabRef/jabref/actions/runs/21581094928/job/62178433061?pr=14959

You committed your code on the main brach of your fork. This is a bad practice. The right way is to branch out from main, work on your patch/feature in that new branch, and then get that branch merged via the pull request (see GitHub flow).

For this pull request, this is OK. For subsequent pull requests, please start with a different branch with a proper branch name. See CONTRIBUTING.md for more details.

Your code currently does not meet JabRef's code guidelines. IntelliJ auto format covers some cases. There seem to be issues with your code style and autoformat configuration. Please reformat your code (Ctrl+Alt+L) and commit, then push.