Skip to content

Replace plugin impl from jbang plugin#14846

Merged
koppor merged 2 commits into
mainfrom
replacekopporPLugin
Jan 12, 2026
Merged

Replace plugin impl from jbang plugin#14846
koppor merged 2 commits into
mainfrom
replacekopporPLugin

Conversation

@Siedlerchr

@Siedlerchr Siedlerchr commented Jan 12, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

User description

Follow up from #14776

Steps to test

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.

PR Type

Enhancement

Description

  • Replace jbang plugin dependency with official version

  • Update from custom jitpack snapshot to stable release

  • Migrate from koppor fork to official dev.jbang implementation

Diagram Walkthrough

flowchart LR
  old["com.github.koppor:jbang-gradle-plugin<br/>jitpack-SNAPSHOT"]
  new["dev.jbang:dev.jbang.gradle.plugin<br/>0.4.0"]
  old -- "upgrade to official" --> new
Loading

File Walkthrough
Relevant files
Dependencies
build.gradle.kts
Update jbang gradle plugin to official version                     

build-logic/build.gradle.kts

  • Replace jbang gradle plugin dependency from custom koppor fork to
    official dev.jbang implementation
  • Update version from jitpack-SNAPSHOT to stable release 0.4.0
  • Simplifies dependency management by using official plugin source
 +1/-1     

@qodo-free-for-open-source-projects

qodo-free-for-open-source-projects Bot commented Jan 12, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

PR Compliance Guide 🔍

Below is a summary of compliance checks for this PR:

Security Compliance
🟢
No security concerns identified No security vulnerabilities detected by AI analysis. Human verification advised for critical code.
Ticket Compliance
🟡
🎫 #13109
🔴 Add Pseudonymization functionality to the CLI
Make org.jabref.logic.pseudonymization.Pseudonymization available on the CLI
Provide similar CLI experience to the consistency check
Implement similar to org.jabref.cli.CheckConsistency class
Codebase Duplication Compliance
⚪
Codebase context is not defined

Follow the guide to enable codebase context checks.

Custom Compliance
🟢
Generic: Comprehensive Audit Trails

Objective: To create a detailed and reliable record of critical system actions for security analysis
and compliance.

Status: Passed

Learn more about managing compliance generic rules or creating your own custom rules

Generic: Meaningful Naming and Self-Documenting Code

Objective: Ensure all identifiers clearly express their purpose and intent, making code
self-documenting

Status: Passed

Learn more about managing compliance generic rules or creating your own custom rules

Generic: Robust Error Handling and Edge Case Management

Objective: Ensure comprehensive error handling that provides meaningful context and graceful
degradation

Status: Passed

Learn more about managing compliance generic rules or creating your own custom rules

Generic: Secure Error Handling

Objective: To prevent the leakage of sensitive system information through error messages while
providing sufficient detail for internal debugging.

Status: Passed

Learn more about managing compliance generic rules or creating your own custom rules

Generic: Secure Logging Practices

Objective: To ensure logs are useful for debugging and auditing without exposing sensitive
information like PII, PHI, or cardholder data.

Status: Passed

Learn more about managing compliance generic rules or creating your own custom rules

Generic: Security-First Input Validation and Data Handling

Objective: Ensure all data inputs are validated, sanitized, and handled securely to prevent
vulnerabilities

Status: Passed

Learn more about managing compliance generic rules or creating your own custom rules

  • Update
Compliance status legend 🟢 - Fully Compliant
🟡 - Partial Compliant
🔴 - Not Compliant
⚪ - Requires Further Human Verification
🏷️ - Compliance label

@qodo-free-for-open-source-projects

qodo-free-for-open-source-projects Bot commented Jan 12, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

PR Code Suggestions ✨

No code suggestions found for the PR.

@Siedlerchr Siedlerchr added automerge PR is tagged with that label will be merged if workflows are green and removed Review effort 1/5 labels Jan 12, 2026
jabref-machine
jabref-machine previously approved these changes Jan 12, 2026
View reviewed changes
@koppor koppor enabled auto-merge January 12, 2026 19:27
@koppor koppor added this pull request to the merge queue Jan 12, 2026
Merged via the queue into main with commit f955072 Jan 12, 2026
53 checks passed
@koppor koppor deleted the replacekopporPLugin branch January 12, 2026 20:22
Siedlerchr added a commit to st-rm-ng/jabref that referenced this pull request Jan 17, 2026
@Siedlerchr
Merge remote-tracking branch 'upstream/main' into fix-for-issue-13860
650c669 
* upstream/main: (64 commits)
  New Crowdin updates (JabRef#14862)
  Make JDK25 available (JabRef#14861)
  Fix empty entries array when exporting group chat to JSON (JabRef#14814)
  feat: add right-click copy context menu to AI chat messages (JabRef#14722)
  FIX : generic error dialog and icon in Source Tab parsing (JabRef#14828)
  Factor out setup-* actions (JabRef#14859)
  Link .http files.
  Update dependency org.postgresql:postgresql to v42.7.9 (JabRef#14857)
  Add more commands to JabSrv (JabRef#14855)
  Fix JabRef#14821: Hide identifier action buttons when field is empty (JabRef#14831)
  Add GH_TOKEN to closed issues/PRs processing step
  New Crowdin updates (JabRef#14854)
  New Crowdin updates (JabRef#14849)
  Chore(deps): Bump jablib/src/main/resources/csl-styles from `0201999` to `f345aa8` (JabRef#14833)
  Add support for book front covers, again (JabRef#14777)
  Readd min width to button in new enty dialog (JabRef#14791)
  Replace plugin impl from jbang plugin (JabRef#14846)
  Revise AI policy wording
  Chore(deps): Bump jablib/src/main/resources/csl-locales (JabRef#14677)
  Update dependency com.konghq:unirest-modules-gson to v4.7.1 (JabRef#14845)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jabref-machine jabref-machine jabref-machine approved these changes

Assignees

No one assigned

Labels

automerge PR is tagged with that label will be merged if workflows are green

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Siedlerchr @jabref-machine @koppor