refactor: use std::path::absolute instead of canonicalize + strip_extended_path_prefix

[svasista-ms]

Closes #367 and #479

We previously combined std::fs::canonicalize with a custom strip_extended_path_prefix helper to normalize Windows paths and remove the extended (verbatim) prefix (?). Issue #367 tracked replacing this logic with a standard library alternative, std::path::absolute.

Motivation

1. Avoid unnecessary filesystem resolution (canonicalize) that can fail or touch the disk when simple normalization is enough.
2. Rely on stable (std::path::absolute) for syntactic absolutization
3. Reduce maintenance of custom path utilities.

User Impact:
Slight behavior change in cargo wdk new: verbatim (extended) paths now return a clear error instead of being silently normalized. (Issue #481 was created to add support for verbatim, extended paths)

[gurry]

Thanks for working on this @svasista-ms.

I want to suggest that it's better to avoid unrelated changes or those not directly related to the PR.

[gurry]

Just one more comment. After that I'll approve

[Copilot]

Pull Request Overview

This PR refactors the codebase to use the standard library's std::path::absolute function instead of the previous combination of std::fs::canonicalize and a custom strip_extended_path_prefix helper function for path normalization on Windows.

Key changes include:

• Replace canonicalize + strip_extended_path_prefix with std::path::absolute for syntactic path absolutization
• Remove custom PathExt trait and related path utility code
• Add explicit error handling for extended/verbatim paths in cargo wdk new command
• Move TwoPartVersion type and related utilities from utils module to lib module

Reviewed Changes

Copilot reviewed 12 out of 12 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
crates/wdk-build/src/utils.rs	Removes PathExt trait, strip_extended_path_prefix utilities, and TwoPartVersion code; changes find_max_version_in_directory visibility
crates/wdk-build/src/lib.rs	Adds TwoPartVersion type and utilities; replaces canonicalize+strip calls with std::path::absolute
crates/wdk-build/src/cargo_make.rs	Updates path handling to use std::path::absolute instead of canonicalize+strip
crates/cargo-wdk/src/providers/mod.rs	Removes PathCanonicalizationError variant
crates/cargo-wdk/src/providers/fs.rs	Removes canonicalize_path method from Fs provider
crates/cargo-wdk/src/cli.rs	Adds explicit check and error for extended/verbatim paths in new command
crates/cargo-wdk/src/actions/mod.rs	Adds PartialEq + Eq derives to Profile and TargetArch enums
crates/cargo-wdk/src/actions/build/tests.rs	Removes path canonicalization expectations from test mocks
crates/cargo-wdk/src/actions/build/package_task.rs	Adds validation for absolute paths and comprehensive test coverage
crates/cargo-wdk/src/actions/build/mod.rs	Updates to use std::path::absolute and removes fs canonicalization calls
crates/cargo-wdk/src/actions/build/error.rs	Adds io::Error variant to BuildActionError
crates/cargo-wdk/src/actions/build/build_task.rs	Simplifies constructor to use absolute path assertion instead of canonicalization

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}