[bdmendes]

Summary by CodeRabbit

• New Features

  • Method references can now be created directly from values when they resolve unambiguously to a method.
  • Multi-argument method parameters and results now preserve method-signature names through named tuples.

• Documentation

  • Added preliminary Smockito 3.0.0 release notes, including migration guidance and compatibility details.
  • Updated project documentation to clarify that Scala 3.9 or later is required.

• Tests

  • Continuous integration now tests against Java 17, 21, and 25 for broader compatibility coverage.

[coderabbitai]

Understand this PR’s impact

Explore downstream dependencies and potential security impact with Blast Radius.

View blast radius →

Warning

Review limit reached

Next included review available in 16 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 65f8065e-789e-4a38-8aa0-dd3c21e768cd

📥 Commits

Reviewing files that changed from the base of the PR and between 3885cf8 and 4f4495a.

📒 Files selected for processing (2)

• CHANGELOG.md
• version.sbt

Walkthrough

The CI workflow now tests Java 17, 21, and 25. Documentation now specifies Scala 3.9+ support and records the planned Smockito 3 changes.

Changes

Java CI Matrix

Layer / File(s)	Summary
Matrix build and coverage configuration .github/workflows/ci.yml	The CI job runs for Java 17, 21, and 25. Java setup uses the matrix version. Codecov uploads run only for Java 17.

Scala Release Documentation

Layer / File(s)	Summary
Release notes and support wording CHANGELOG.md, README.md, docs/_docs/index.html	The changelog adds the Smockito 3.0.0 entry. The README and documentation site specify Scala 3.9+ support.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Other

Merge Risk: 🔵 Low · up to 3885c

The documentation gives conflicting minimum Scala-version guidance, which can lead users to attempt unsupported setups. Align the remaining usage wording before release.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	The title clearly identifies the main change: preparing the 3.0.0 release. It matches the changelog entry and release-focused updates.

---

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

_{A rabbit reads each line,
The patch grows clear beneath the moon,
Small changes hop in place,
Tests guard the garden path,
Reviews bloom before the dawn.}

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

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96%. Comparing base (a10669a) to head (4f4495a).
⚠️ Report is 8 commits behind head on master.

Additional details and impacted files

@@          Coverage Diff          @@
##           master   #308   +/-   ##
=====================================
  Coverage      96%    96%           
=====================================
  Files           9      9           
  Lines         233    233           
  Branches       47     47           
=====================================
  Hits          223    223           
  Misses         10     10           

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[coderabbitai]

Actionable comments posted: 1

Caution

Some comments are outside the diff and can’t be posted inline due to GitHub limitations.

⚠️ Outside diff range comments (1)

🟡 Minor · Use the same minimum Scala version in all user-facing documentation. · README.md:15

README.md:15
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win

Use the same minimum Scala version in all user-facing documentation.

• README.md:15 still says “Scala 3”.
• docs/_docs/index.html:44 still says “Scala 3 projects”.

Change both to “Scala 3.9+” and “Scala 3.9+ projects”.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 79736fc1-6173-4afa-ad10-1962b6653f88

📥 Commits

Reviewing files that changed from the base of the PR and between a10669a and 3885cf8.

📒 Files selected for processing (4)

• .github/workflows/ci.yml
• CHANGELOG.md
• README.md
• docs/_docs/index.html

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.

[coderabbitai]

🔒 Security & Privacy | 🛡️ Detected with Advanced Tier | 🟡 Minor | ⚡ Quick win

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
set -eu
cat -n .github/workflows/ci.yml

Repository: bdmendes/smockito

Length of output: 2999

---

🤖 get_repo_knowledge executed:

get_repo_knowledge bdmendes/smockito /tmp/coderabbit-repo-knowledge/bdmendes-smockito-f0d6415a/learnings

Length of output: 1595

---

Security Misconfiguration

Reachability: External
CWE: CWE-732 — Incorrect Permission Assignment for Critical Resource

Proposed fix

```diff jobs: ci: + permissions: + contents: read ``` ]]>

🧰 Tools

🪛 zizmor (1.30.0)

[warning] 1-95: overly broad permissions (excessive-permissions): default permissions used due to no permissions: block

(excessive-permissions)

---

[warning] 9-45: overly broad permissions (excessive-permissions): default permissions used due to no permissions: block

(excessive-permissions)

Sources: MCP tools, Linters/SAST tools