refactor: improve code quality with safety and lint fixes

[jdx]

Summary

• Replace unsafe .unwrap() on Path::parent() with proper error handling in include directive parsing
• Add #[must_use] attributes to builder build() methods and parse functions to prevent accidentally discarding important return values
• Add doc comments to parse functions explaining their purpose

Changes

• lib/src/spec/mod.rs: Fix unsafe unwrap in include handling, add #[must_use] to parse_file and parse_script
• lib/src/spec/builder.rs: Add #[must_use] to all build() methods
• lib/src/parse.rs: Add #[must_use] and doc comments to parse and parse_partial

Test plan

• All existing tests pass
• Clippy happy

🤖 Generated with Claude Code

---

Note

Improves API safety and clarity, and hardens file-include parsing.

• Add #[must_use] to SpecFlagBuilder::build, SpecArgBuilder::build, SpecCommandBuilder::build, Spec::parse_file, Spec::parse_script, parse, and parse_partial
• Document parse and parse_partial to clarify behavior and defaults
• Replace ctx.file.parent().unwrap() with checked parent resolution in include handling (errors instead of panic)

^{Written by Cursor Bugbot for commit 19ef8a5. This will update automatically on new commits. Configure here.}

[Copilot]

Pull request overview

This PR improves code safety and maintainability by replacing an unsafe unwrap() call with proper error handling and adding #[must_use] attributes to prevent accidentally discarding important return values.

Changes:

• Replaced unsafe Path::parent().unwrap() with proper error handling using ok_or_else()
• Added #[must_use] attributes to parse functions and builder build() methods
• Added documentation comments to parse functions

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File	Description
lib/src/spec/mod.rs	Replaced unsafe unwrap with error handling in include directive parsing; added #[must_use] and docs to parse_file and parse_script
lib/src/spec/builder.rs	Added #[must_use] to all builder build() methods
lib/src/parse.rs	Added #[must_use] attributes and documentation to parse and parse_partial functions

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[codecov]

Codecov Report

❌ Patch coverage is 0% with 24 lines in your changes missing coverage. Please review.
✅ Project coverage is 46.52%. Comparing base (397efe4) to head (19ef8a5).
⚠️ Report is 2 commits behind head on main.

Files with missing lines	Patch %	Lines
lib/src/spec/mod.rs	0.00%	14 Missing ⚠️
lib/src/parse.rs	0.00%	7 Missing ⚠️
lib/src/spec/builder.rs	0.00%	3 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #427      +/-   ##
==========================================
- Coverage   47.68%   46.52%   -1.16%     
==========================================
  Files          45       45              
  Lines        6164     6236      +72     
  Branches     6164     6236      +72     
==========================================
- Hits         2939     2901      -38     
- Misses       1553     1625      +72     
- Partials     1672     1710      +38     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.