states: add dirty step verification
Metadata
Current evaluation
Merged dirty-step verification: steps rerun when state properties/options change or dependencies require execution (craft-parts#23, ca487581).
Resolution: closed resolved
Reason: PR #23 was merged by commit 74bf06d6. Implementation commit ca487581 added dirty-step verification for changed state properties, project options, and dependency execution.
Evaluated by: mr-cal-bot
No scores available.
Issue body
A part step is considered dirty if one of its state properties
or options have changed, or if one of its dependencies should run.
Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged dirty-step verification: steps rerun when state properties/options change or dependencies require execution (craft-parts#23, ca487581). | |
| stealth/space-bunny-alpha | — | closed resolved | Merged in #23 via commit ca487581 (merge 74bf06d6), adding dirty-step verification when state properties, options, or dependencies change. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged. Adds dirty step verification to state management, marking steps dirty when properties, options, or dependencies change. Approved by two reviewers, introducing 368 additions and 9 deletions across five files. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged addition of dirty step verification to the states module. A part step is now marked dirty when its properties, options, or dependencies change, ensuring correct execution order. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #21 states: add outdated step verification | craft-parts | merged | Merged PR #21 (commit 08e81f92, merge 0d39e299) adds outdated step verification to the state manager: a part step is outdated when an earlier step ran more recently; source-handler checks were deferred to a later PR. | |
| #3675 parts: fix long recursion when verifying dirty steps | snapcraft | merged | Merged #3675 (commit 2dad0d960), updating craft-parts to 1.4.2 to eliminate long dirty-step verification loops and improving managed-container output. | |
| #83 sequencer: improve dirty state report message (CRAFT-302) | craft-parts | merged | Merged in craft-parts#83 via 16b77c4b, clarifying dirty-state reports to include the changed dependency step name. |