← Back to issue list

Move the build planner logic into craft-platforms

View original Github issue

Metadata

Project
rockcraft
Number
#639
Type
issue
State
open
Author
lengau
Labels
Created
Updated
Closed

Current evaluation

Refactor to move build planner logic into craft-platforms, requested by jugmac00. Work completed: craft-application#391 closed as resolved, rockcraft PR #715 and commit 5f898b32 implemented the change. No BuildPlanner class remains in codebase.

Suggested action: close stale

Reason: The issue is 775 days old with zero comments or labels, and the work has been completed: craft-application#391 is closed as resolved stating 'Build planner logic and filterer were upstreamed to craft-platforms via PR #25', rockcraft commit 516d29e5 'refactor: use craft-platform for the build plans (#715)' and commit 5f898b32 'refactor: use default build planner' implemented the change, and the current codebase has no BuildPlanner class — it uses craft_platforms directly.

Impact: 35 Quick Win: 19.25 Staleness: 85 Complexity: 45 Confidence: 85 Support Request: 0

Issue body

### What needs to get done Take the core logic from the build planner and move it into craft-platforms ### Why it needs to get done @jugmac00 requested it :-)

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.8-27b
Impact: 35
Quick Win: 19.25
Staleness: 85
Complexity: 45
Confidence: 85
Support Request: 0
close stale Refactor to move build planner logic into craft-platforms, requested by jugmac00. Work completed: craft-application#391 closed as resolved, rockcraft PR #715 and commit 5f898b32 implemented the change. No BuildPlanner class remains in codebase.
qwen/qwen3.6-35b-a3b
Staleness: 98
Complexity: 25
Confidence: 90
Support Request: 5
close stale Request to migrate build planner core logic to craft-platforms. Created over two years ago, it has zero comments, no labels, and no activity. Currently untriaged and effectively abandoned.
qwen3.6-35b-a3b-mtp-q6
Staleness: 95
Complexity: 40
Confidence: 75
Support Request: 5
needs triage Proposal to refactor and move core build planner logic into the craft-platforms library. The issue remains open with no maintainer response, labels, or activity for over two years.
qwen3.6-35b-a3b-mtp-q6
Staleness: 95
Complexity: 40
Confidence: 85
Support Request: 10
close stale Task to move build planner logic into craft-platforms. Unlabelled, zero comments, and inactive for over two years with no maintainer engagement.
qwen3.6-35b-a3b-mtp-q6
Staleness: 95
Complexity: 30
Confidence: 80
Support Request: 0
close stale Extract core build planner logic into craft-platforms. Open and inactive for nearly two years with no comments or labels. Requires triage and assignment to proceed.

Update history

No update history recorded yet.

Related work

  • Likely Fixed By: craft-application#391 (confidence 90%)

    Upstream tracking issue closed as resolved: 'Build planner logic and filterer were upstreamed to craft-platforms via PR #25.'

  • Likely Fixed By: rockcraft#715 (confidence 85%)

    PR 'refactor: use craft-platform for the build plans' (commit 516d29e5) directly implements the requested change in rockcraft.

Related issues

Issue Project State Summary Similarity
#391 Use craft-platforms for build plans craft-application closed Build planner logic and filterer were upstreamed to craft-platforms via PR #25. Downstream integration is being prototyped in craft-application. The issue is closed as resolved.
77%
#5111 feat: use `craft-platforms` for build plans snapcraft merged Merged the change to replace existing build plan logic with the craft-platforms library. Approved by two reviewers, the update modifies six files with a net reduction of 17 lines, streamlining platform detection and build configuration.
73%
#403 [PROTOTYPE] refactor: use craft-platforms craft-application closed Closed as a prototype. The author abandoned the pull request, noting that the craft-platforms build planner integration is tracked and implemented via issue #391.
73%
#326 feat: add Platform model and default BuildPlanner craft-application merged Merged after two approvals. Adds a Platform model and default BuildPlanner. The maintainer coordinated timing with related remote-build PRs to bundle fixes in a craft-application hotfix.
72%
#1894 refactor: use craft-platforms for build plans charmcraft merged Merged a refactor replacing custom build plan logic with craft-platforms for platforms charms. Approved by two reviewers, the change modifies two files with a net reduction of eight lines, streamlining platform handling.
72%
#715 refactor: use craft-platform for the build plans rockcraft merged Merged refactor to use craft-platform for build plans, creating a single source for Rockcraft and remote builds. Approved by two reviewers, resolving issue #698 with 31 additions and 3 deletions across five files.
71%
#698 Use craft-platforms for build plans rockcraft closed Closed and superseded by prototype PR #662, which implements craft-platforms for build plans to deduplicate logic and ensure consistency across projects. No explicit resolution or merge details are documented.
70%
#5015 Use craft-platforms for build plans snapcraft closed Closed and synced to Jira as CRAFT-3322. No repository changes or merges were applied. The task remains deferred pending resolution of craft-platforms issue #5.
70%