inspectors/snap: ignore multi-line assertion entries
Metadata
Current evaluation
Merged PR #87, making the snap inspector skip multi-line assertion entries, continue parsing later entries, and improve debugging and invalid-snap handling.
Resolution: closed resolved
Reason: PR #87 was approved and merged into fetch-service. It implemented temporary handling for multi-line snap assertions plus improved diagnostics and invalid-snap behavior.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Ignore multi-line assertion entries for now, but keep parsing entries
after finding them. Also add better debugging and invalid snap handling.
- [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 PR #87, making the snap inspector skip multi-line assertion entries, continue parsing later entries, and improve debugging and invalid-snap handling. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged #87: the snap inspector now skips multi-line assertion entries, continues parsing subsequent entries, and improves debugging and invalid-snap handling. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged after approval. Modifies the snap inspector to skip multi-line assertion entries while continuing to parse subsequent ones, and adds improved debugging and invalid snap handling. |
Update history
| Date | Change |
|---|---|
| created |
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #178 fix(inspectors/snap): minor inspector fixes and more test coverage | fetch-service | merged | Merged in PR #178 as commit 521b263, adding Snap inspector bounds checks, propagating SquashFS reader-creation errors, and broader tests. | |
| #522 fix(inspectors/snap): fix serial assertion endpoint | fetch-service | merged | Merged PR #522 fixed the Snap serial-assertion endpoint in commit d374352; it received two approvals and passed all CI checks. | |
| #150 inspectors/snap: add additional inspector tests | fetch-service | merged | Merged PR #150 as commit 7e0b5b4, expanding test coverage for the Snap inspector after approval by two reviewers. | |
| #84 inspectors: add snap inspectors | fetch-service | merged | Snap inspectors were added in merged PR #84, implemented by commit cfa83f8. |