← Back to issue list

unused and potentially confusing support for plugs/slots at a part level in the snapcraft.yaml schema

View original Launchpad issue

Metadata

Project
snapcraft (launchpad)
Number
#1581166
Type
issue
State
closed
Author
~zyga
Labels
verification-done
Created
Updated
Closed

Current evaluation

Removed unused schema support for part-level plugs and slots in snapcraft.yaml. The schema now rejects these definitions, preventing user confusion. Verified and closed.

Suggested action:

No scores available.

Issue body

[Impact] * snapcraft has a schema that describes allowed snapcraft.yaml files. Part of this schema describes plugs and slots that are defined as the level of a part (so not globally and not at the level of an app). * keeping this code in the schema can create potential confusion for users who expect snapcraft to validate the input to be correct. * the change is minimal, altering the schema only [Test Case] * create a snapcrat.yaml with a part defining a plug and a slot * run snapcraft * snapcraft should complain about the plug/slot definition at a part level ... parts: foo: plugin: nil plugs: [iface1] slots: [iface2] ... [Regression Potential] * snaps that used parts defining plugs or slots will no longer build * snaps that will be built after removing the offending declarations will behave exactly as before so the regression potential is minimal [Other Info]

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Removed unused schema support for part-level plugs and slots in snapcraft.yaml. The schema now rejects these definitions, preventing user confusion. Verified and closed.
qwen/qwen3.6-35b-a3b Removed unused part-level plugs and slots from the snapcraft.yaml schema. Snapcraft now rejects these definitions, resolving validation confusion. The change was verified and the issue closed.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#2884 plugs/slots: match output format read in snapcraft.yaml snapcraft merged Merged to align snap.yaml output with snapcraft.yaml input by restoring 3.8 plugs/slots formatting. This change works around a review-tools bug and ensures syntax consistency. Approved by one reviewer, passed CI, and modified six files.
73%