fix(java): drop unused worker var in graalvm smoke

[pratyush618]

The GraalVM smoke's worker was bound by a try-with-resources whose name was
never read, tripping an "unused local variable" diagnostic. Hoist the worker
to its own statement and manage it via try (worker) (Java 9+ resource
reference) — same lifecycle, no unused binding.

Compile-only, no behavior change.

Summary by CodeRabbit

• Refactor
  • Updated the smoke test flow to handle worker startup and cleanup in a clearer sequence.
  • No user-facing behavior changed; task execution, result checks, and job listing checks remain the same.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 6b49213b-bdf1-4f4b-ad72-b6cdff52c301

📥 Commits

Reviewing files that changed from the base of the PR and between 78591d9 and a6dcdeb.

📒 Files selected for processing (1)

• sdks/java/graalvm-smoke/src/main/java/org/byteveda/taskito/graalvm/Smoke.java

---

📝 Walkthrough

Walkthrough

In the GraalVM smoke test, worker creation is split from the try-with-resources block: Worker is now assigned via a standalone statement calling .start(), then passed into a separate try (worker) block. All other logic is unchanged.

Changes

Worker Lifecycle Refactor

Layer / File(s)	Summary
Worker creation split from try-with-resources sdks/java/graalvm-smoke/src/main/java/org/byteveda/taskito/graalvm/Smoke.java	Worker is created and started in a standalone assignment; the existing try-with-resources now references the pre-created variable rather than initializing it inline.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A worker once born in a try-block's embrace,
Now steps out first, then enters that space.
Same job, same task, same latch counts to zero,
Just a small shuffle — a tidy little hero! 🐇✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ 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%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly matches the main change: simplifying the GraalVM smoke test by removing the unused worker variable.
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.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/java-graalvm-smoke-unused-worker

---

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