ci(windows): build the docs in the Windows clippy job

[yuzi-co]

Summary

A rustdoc failure that shows up only on Windows can't be caught by CI today. rustdoc (non-GUI crates) runs on ubuntu-latest, so a doc link whose target is cfg-gated off on Windows still resolves everywhere CI checks. That exact bug reached master in openlogi-permissions: the module docs linked to PermissionStatus::Unknown, which isn't declared on Windows, so cargo doc failed there. #1679 has since fixed that link by hand. This PR adds the check that would have caught it.

An earlier version of this PR also fixed the openlogi-permissions link itself, and before that ci.yml line-continuation parsing under CRLF. Master has fixed both independently, so those commits were dropped on rebase.

Changes

• .github/workflows/ci.yml: after clippy, the clippy (windows) job now runs the same cargo doc command as the rustdoc job. The job already sets up the toolchain and cache, so the cost is one doc build and no extra runner. The step runs under bash so the wrapped command parses.
• xtask: on a Windows host, cargo xtask ci clippy-windows runs the same docs step after clippy, through one helper shared with the rustdoc job. A test checks that this plan matches the commands ci.yml runs. Other hosts still lint Windows by cross-compiling, as before.
• .agents/rules/ci.md: the CI job table lists the new docs step.

Testing

Windows 11 Pro, RUSTFLAGS="-D warnings", rebased onto current master:

cargo fmt --all -- --check                                   pass
cargo clippy --workspace --all-targets -- -D warnings        pass
cargo test --workspace                                       pass (1943)
RUSTDOCFLAGS="-D warnings" cargo doc --workspace --no-deps \
  --document-private-items --exclude openlogi-ui \
  --exclude openlogi-desktop --exclude openlogi-overlay \
  --exclude openlogi-agent                                   pass
cargo xtask ci clippy-windows                                pass

[greptile-apps]

Confidence Score: 5/5

[Medium impact] The PR appears safe to merge; the remaining test-coverage finding is non-blocking.

Findings

1.  native clippy windows runs what ci runs searches the flattened contents of the entire workflow. Because the Linux rustdoc job contains the same cargo doc command, this test would still pass if that command were removed from the Windows job. The new test therefore does not protect against the Windows CI coverage drifting out of sync. ▶

Summary

Adds Windows documentation checks to CI and the native local gate.

• Windows clippy now builds the non-GUI workspace docs.

No new findings were accepted.

_{Reviews (10) · Last reviewed commit: "fix(xtask): build the docs in the native..." · Reviewed by Greptile}

[greptile-apps]

native_clippy_windows_runs_what_ci_runs searches the flattened contents of the entire workflow. Because the Linux rustdoc job contains the same cargo doc command, this test would still pass if that command were removed from the Windows job. The new test therefore does not protect against the Windows CI coverage drifting out of sync.

Knowledge Base Used: Build and release automation

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!