Plainbox providers run validate
Metadata
Current evaluation
Merged. Adds a build-time validation step to the plainbox-provider plugin to prevent buggy providers from packaging into checkbox snaps. Handles cross-provider dependencies and includes an integration test. Approved after minor test naming adjustments.
Suggested action: —
No scores available.
Issue body
Modify the plainbox-provider plugin to run the "validate" command during the build step to ensure buggy providers don't make it in to checkbox snap packages.
As providers can reference jobs from other providers (i.e. have a dependency on that provider) the environment has a variable PROVIDERPATH populated. Coupled with appropriate use of the "after" keyword this allows "validate" steps to identify that jobs are defined correctly. An integration test to confirm this functionality is included.
LP: [#1660692](https://bugs.launchpad.net/snapcraft/+bug/1660692)
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged. Adds a build-time validation step to the plainbox-provider plugin to prevent buggy providers from packaging into checkbox snaps. Handles cross-provider dependencies and includes an integration test. Approved after minor test naming adjustments. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Updates the plainbox-provider plugin to run the validate command during builds, preventing buggy providers from packaging into checkbox snaps. Incorporates test naming feedback and resolves LP #1660692. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged after adjusting the integration test name to follow project conventions. Adds a validate command to the plainbox-provider plugin build step to catch buggy providers before checkbox snap packaging. Resolves LP #1660692. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #609 Create Plainbox Provider plugin | snapcraft | merged | Merged the Plainbox Provider plugin, adding integration tests for standard Checkbox workflows. Fixed a failing demo by adding a missing Python package dependency to the demo snap. Passed CI retests and was successfully integrated. | |
| #1660692 The plainbox-provider plug should run "validate" during the build step | snapcraft (launchpad) | closed | The request to run plainbox validate during the build step was closed without implementation. Zero comments and prolonged inactivity indicate the feature request was abandoned. | |
| #1611 plainbox-provider plugin: init PROVIDERPATH | snapcraft | merged | Merged PR initializing PROVIDERPATH in the plainbox-provider plugin to prevent build host providers from affecting validation results. Resolves LP: #1722833. Approved, passed CI, and applied with seven lines added across two files. | |
| #2140 lp 1768233 workaround for plainbox_provider plugin | snapcraft | merged | Merged a workaround for Launchpad bug 1768233 to fix plainbox-provider plugin failures blocking checkbox snap builds. Includes integration tests, passed CI with 100% diff coverage, and was approved before merging into master. | |
| #1889 tests: remove plainbox part from plugin tests | snapcraft | merged | Merged changes removing the plainbox component from plugin tests. This resolves integration test failures caused by the plainbox library being integrated into the checkbox application, rendering the separate plainbox part obsolete for snaps. | |
| #2360 plainbox-provider plugin: add support for bases | snapcraft | merged | Merged update to the PlainboxProvider plugin adding base awareness for core16 and core18. Approved by reviewer, passed CI checks, and achieved 100% diff coverage. Resolves Launchpad bug #1794802. |