test(core): expect E44 on Windows when a refresh lands over 0.3.x

[dg-coreylweathers]

Summary

main's test workflow fails on every Windows job since #126 and #130 met on main (test run 37930234857). One test, TestRecheckUnderLock::test_03x_only_tool_still_gets_folders_on_refresh[False].

• fix(skills): keep a newer skills ref and a removed tool when a refresh overlaps #126 added the test. It writes the record an older deepctl (0.3.x) left for Claude Code, lands the new skill folders, and asserts that stderr is empty.
• feat(skills): remove the deepctl 0.3.x skill files deepctl can prove it wrote #130 made Windows skip the 0.3.x cleanup entirely and print E44 once. E44 is the warning that deepctl left the recorded 0.3.x skill files in place on Windows, because a parent folder could become a junction during cleanup, and will not try again.

Each PR was green on its own branch. #126 branched from 63c2be1, before #130 added E44.

Change

Test only, one commit. The test now has three named cases:

• native: the branch the host OS takes.
• forced-windows: the Windows branch on POSIX, by setting skill_generator._WINDOWS the way other tests in the file do. Skipped on Windows, where native already covers it. The POSIX branch is never forced on Windows.
• removed: the tool is removed during the download, unchanged from fix(skills): keep a newer skills ref and a removed tool when a refresh overlaps #126.

The Windows branch expects exactly one E44 naming the recorded path, with whitespace normalized on both sides so a console wrap cannot split the match. The POSIX branch expects empty stderr. Both then assert that the 0.3.x path left the record, only the two new skill folders remain listed, the v03 flag is cleared, and a second refresh prints nothing. On Windows a stale path would print E44 again, so that last check is load-bearing there.

Checked, on head e5cac81 (tree identical to be7a3ac, where the gate ran)

• pytest -k TestRecheckUnderLock on macOS: 12 passed, including forced-windows.
• Mutation check: with the cleanup's record rewrite disabled, native and forced-windows fail on the paths assertion; with the v03 flag left in place they fail on the flag assertion.
• Docker gate (py3.12 ruff format, ruff check, mypy, full pytest; core tests on py3.10 and as uid 1000): all exit 0, 2207 passed.
• ruff format --check on this test file reports blocks around lines 2312 and 2324 that are already unformatted on main and are not touched here. CI runs ruff format --check and ruff check on src/ and packages/*/src only, so test files are not formatted or linted in CI.

This is a test: commit, so release-please adds no changelog line.

🤖 Generated with Claude Code