feat(plugins): use @ channel separator
Metadata
Current evaluation
Merged PR #345 (commit 3dfacf0) changed plugin channel syntax from "/" to "@", allows optional surrounding whitespace, and fixes #342.
Resolution: closed resolved
Reason: PR #345 was approved, passed all reported CI checks, and merged as commit 3dfacf0. The implementation and tests adopt the "@" separator, and #342 records the change as its fix.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Changes expected channel separator from "/" to "@" with optional surrounding whitespace.
Fixes #342
---
- [x] I've followed the [contribution guidelines](https://github.com/canonical/imagecraft/blob/main/CONTRIBUTING.md).
- [x] I've signed the [CLA](http://www.ubuntu.com/legal/contributors/).
- [x] I've successfully run `make lint && make test`.
- [x] I've added or updated any relevant documentation.
- [ ] In documents I changed, I [added a meta description](https://canonical-starflow.readthedocs-hosted.com/how-to/add-a-page-meta-description/) if one was missing.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #345 (commit 3dfacf0) changed plugin channel syntax from "/" to "@", allows optional surrounding whitespace, and fixes #342. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #345 (commit 3dfacf0) to use @ as the plugin channel separator, allowing optional surrounding whitespace and fixing #342. | |
| stealth/space-bunny-alpha | — | closed resolved | Plugin channel parsing switched from / to @ with optional surrounding whitespace. Merged in canonical/imagecraft#345 (commit 3dfacf0), fixing #342. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged after two approvals and passing CI checks. Replaces the expected channel separator from slash to at-sign with optional surrounding whitespace, resolving issue #342 across seven files. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged plugin updates changing the channel separator from slash to @ with optional surrounding whitespace. Resolves #342. Code passed linting and tests. |
Update history
No update history recorded yet.
Related work
-
Related To:
canonical/imagecraft#342
(confidence 100%)
The PR explicitly states that it fixes #342, which was closed by this merged change.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #342 Update snap name /channel separator character in snap plugins | imagecraft | closed | Fixed by merged PR #345 (commit 3dfacf0): snap plugins now use @, with optional surrounding whitespace, instead of the proposed = separator. |