Breaking migrations to pathlib
Metadata
Current evaluation
Pathlib migration was completed in merged PR #1424 (commit cfdabafc), while preserving string-based YAML serialization for migration-state paths.
Resolution: closed resolved
Reason: PR #1424 was merged as cfdabafc and migrated internal path handling to pathlib. Its migration-state serializer emits POSIX strings while deserialization restores Path objects.
Evaluated by: mr-cal-bot
No scores available.
Issue body
### What needs to get done
During #1317, a couple of pain points were found where total migration to pathlib would've been breaking.
These were:
- `mount` utilities
- `executor.collisions` utilities
### Why it needs to get done
It feels good to be modern!
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Pathlib migration was completed in merged PR #1424 (commit cfdabafc), while preserving string-based YAML serialization for migration-state paths. | |
| stealth/space-bunny-alpha | — | closed resolved | Resolved by merged PR #1424 (commit cfdabafc), completing the pathlib migration while preserving string-based migration-state YAML compatibility. | |
| qwen/qwen3.6-35b-a3b | — | — | Resolved via PR #1424. The pathlib migration is stable, using custom YAML serializers to convert Path objects to strings and back in MigrationState. Path set deserialization is verified. | |
| qwen/qwen3.6-35b-a3b |
Staleness:
50
Complexity:
40
Confidence:
75
Support Request:
5
|
needs triage | Refactoring mount and executor.collisions utilities to pathlib causes breaking changes. The proposal is currently inactive with no comments, assignee, or maintainer acknowledgment. | |
| qwen3.6-35b-a3b-mtp-q6 |
Staleness:
60
Complexity:
60
Confidence:
80
Support Request:
0
|
needs triage | Refactor to migrate mount and executor.collisions utilities to pathlib. Open for 61 days with no maintainer comments or assignees. Requires triage and scoping of breaking changes. |
Update history
| Date | Change |
|---|---|
| closed |
Related work
-
Likely Fixed By:
canonical/craft-parts#1424
(confidence 100%)
Merged pathlib refactor identified by the maintainer as the resolving change.
-
Related To:
canonical/craft-parts#1317
(confidence 90%)
The issue identifies #1317 as the work that uncovered the remaining migration pain points.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1317 Migrate to pathlib | craft-parts | open | Broad os.path-to-pathlib refactor is recorded as completed by merged PR #1424 (commit cfdabafc), with migration-state YAML compatibility preserved; #1317 remains open but is ready to close. |