refactor: fix non-idiomatic boolean comparisons from Zig port

[Liohtml]

Summary

• Fix two non-idiomatic boolean comparison patterns left over from the Zig-to-Rust port:
  • Blob.rs:2304: file.seekable.unwrap_or(true) == false → !file.seekable.unwrap_or(true)
  • pack_command.rs:1457: !b == true → !b
• Similar in spirit to refactor: remove unsafe for static strings #30867 (remove unsafe for static strings) — mechanical cleanup of port artifacts

Test plan

• No behavioral change — purely cosmetic refactor
• Existing tests should continue to pass

🤖 Generated with Claude Code

[claude]

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 31792338-4ce9-4bc6-861f-0a14341f32a1

📥 Commits

Reviewing files that changed from the base of the PR and between 172afa5 and 1803d92.

📒 Files selected for processing (2)

• src/runtime/cli/pack_command.rs
• src/runtime/webcore/Blob.rs

---

Walkthrough

This PR simplifies two boolean expressions across unrelated source files. In pack_command.rs, the bundled dependencies check now uses idiomatic negation. In Blob.rs, the seekable file check adopts the same pattern. Both changes preserve semantics while improving code clarity.

Changes

Boolean Expression Simplifications

Layer / File(s)	Summary
Boolean logic refactorings src/runtime/cli/pack_command.rs, src/runtime/webcore/Blob.rs	get_bundled_deps refactors the boolean false case to use if !b syntax, and BlobExt::get_size simplifies the seekable file check from explicit equality to negated form, both preserving control flow.

🚥 Pre-merge checks | ✅ 4

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main change: fixing non-idiomatic boolean comparisons from a Zig-to-Rust port, which matches the changeset perfectly.
Description check	✅ Passed	The description is comprehensive and covers the required template sections with specific details about changes, context, and test plan; however, it lacks explicit verification details.
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.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

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.}