fix: avoid use of org.apache.commons

[javier-godoy]

Close #130

Summary by CodeRabbit

• Chores

  • Updated copyright year information.

• Refactor

  • Updated internal file handling mechanisms for improved maintainability.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

Removed dependency on Apache Commons IO by replacing FileUtils.readFileToByteArray() with Java NIO's Files.readAllBytes() to resolve compilation failure in Vaadin 25. Updated copyright year to 2025. No public API changes.

Changes

Cohort / File(s)	Summary
Dependency migration src/main/java/com/flowingcode/vaadin/addons/DevSourceRequestHandler.java	Replaced Apache Commons IO FileUtils import with Java NIO Files import; updated file-reading logic from FileUtils.readFileToByteArray() to Files.readAllBytes() with path conversion; bumped copyright year from 2020-2023 to 2020-2025

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested reviewers

• mlopezFC
• paodb

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The PR title 'fix: avoid use of org.apache.commons' accurately summarizes the main change: replacing org.apache.commons.io dependency with java.nio.file.Files to resolve compilation failure.
Linked Issues check	✅ Passed	The PR successfully addresses issue #130 by removing the org.apache.commons.io import from DevSourceRequestHandler.java and replacing FileUtils.readFileToByteArray() with Files.readAllBytes(), resolving the compilation error.
Out of Scope Changes check	✅ Passed	All changes are focused on resolving the compilation failure in DevSourceRequestHandler.java; copyright year bump is a standard practice and not out-of-scope.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix-130

---

📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 0edc732 and 32f70ab.

📒 Files selected for processing (1)

• src/main/java/com/flowingcode/vaadin/addons/DevSourceRequestHandler.java

🔇 Additional comments (3)

src/main/java/com/flowingcode/vaadin/addons/DevSourceRequestHandler.java (3)

5-5: LGTM: Copyright year updated.

Standard maintenance update to reflect the current year.

---

29-29: LGTM: Replaced Apache Commons IO with Java NIO.

The change from org.apache.commons.io.FileUtils to java.nio.file.Files successfully removes the external dependency that was causing the compilation failure. The Java NIO Files class is part of the standard library and provides equivalent functionality.

---

51-51: LGTM: File reading logic correctly migrated to Java NIO.

The replacement of FileUtils.readFileToByteArray(getFile(path)) with Files.readAllBytes(getFile(path).toPath()) is functionally equivalent. Both methods read all bytes from the file into a byte array and throw IOException on errors. The conversion from File to Path via toPath() is straightforward and correct.

Since this change resolves the compilation failure, please verify that the project now compiles successfully under Vaadin 25.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[mlopezFC]

LGTM