Use a better way of getting the CARGO_TARGET_DIR

[kixelated]

Should work on docs.rs

[coderabbitai]

Walkthrough

This pull request includes a version bump of the libmoq package from 0.2.0 to 0.2.1 in rs/libmoq/Cargo.toml. Additionally, the build script in rs/libmoq/build.rs was modified to change how the target directory is computed. The target_dir() function now derives the build target directory from the OUT_DIR environment variable by ascending four levels, replacing the previous approach that used CARGO_MANIFEST_DIR. The associated error message was updated accordingly. No changes to public APIs or function signatures were introduced.

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 (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main change: improving how CARGO_TARGET_DIR is obtained in the build script by using OUT_DIR instead of CARGO_MANIFEST_DIR.
Description check	✅ Passed	The description 'Should work on docs.rs' is related to the changeset, indicating the fix addresses compatibility with the docs.rs documentation hosting service.

✨ 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-docs

---

📜 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 73e9986 and c64779d.

⛔ Files ignored due to path filters (1)

• Cargo.lock is excluded by !**/*.lock

📒 Files selected for processing (2)

• rs/libmoq/Cargo.toml (1 hunks)
• rs/libmoq/build.rs (1 hunks)

🧰 Additional context used

📓 Path-based instructions (2)

rs/**/Cargo.toml

📄 CodeRabbit inference engine (CLAUDE.md)

For Rust development, use the workspace configuration in rs/Cargo.toml

Files:

• rs/libmoq/Cargo.toml

**/*.rs

📄 CodeRabbit inference engine (CLAUDE.md)

In Rust crates, tests should be integrated within source files using inline test modules

Files:

• rs/libmoq/build.rs

🧠 Learnings (1)

📚 Learning: 2025-12-10T04:00:14.871Z

Learnt from: CR
Repo: moq-dev/moq PR: 0
File: CLAUDE.md:0-0
Timestamp: 2025-12-10T04:00:14.871Z
Learning: Applies to rs/**/Cargo.toml : For Rust development, use the workspace configuration in `rs/Cargo.toml`

Applied to files:

• rs/libmoq/build.rs

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: Check

🔇 Additional comments (2)

rs/libmoq/Cargo.toml (1)

8-8: LGTM! Appropriate patch version bump.

The version bump from 0.2.0 to 0.2.1 is appropriate for the bug fix that improves docs.rs compatibility.

rs/libmoq/build.rs (1)

43-54: LGTM! More robust approach for finding the target directory.

The refactored target_dir() function uses OUT_DIR as the anchor point instead of CARGO_MANIFEST_DIR, which is more portable and reliable across different build environments, including docs.rs. The implementation correctly ascends four levels from OUT_DIR to reach the target directory, and the comments clearly document the expected path structure.

This approach is also resilient to custom CARGO_TARGET_DIR configurations since OUT_DIR is always set relative to the actual target directory in use.

---

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