← Back to issue list

feat(snapcraft/pack): node-based rewrite

View original Github issue

Metadata

Project
craft-actions
Number
#100
Type
pull request
State
merged
Author
bepri
Labels
—
Created
Updated
Closed

Current evaluation

Merged a node-based rewrite of the snapcraft/pack action. Resolves the Snapcraft 7 default command issue, adds .comp file support, and includes unit tests. Approved by two reviewers with all CI checks passing across multiple platforms.

Suggested action: —

No scores available.

Issue body

Best to review on a per-commit basis. - The first commit adds the simplest possible pack action, which is missing some features. - Then, some refactoring work to support the next commit - Next, we fix the all-too-present Snapcraft 7 default command issue - And then support outputting .comp files from Snapcraft - Finally, tests! This doesn't test no warnings as retrospection on workflow logs is apparently insanely complicated as part of a workflow, but the unit tests cover it. CRAFT-5204.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b — — Merged a node-based rewrite of the snapcraft/pack action. Resolves the Snapcraft 7 default command issue, adds .comp file support, and includes unit tests. Approved by two reviewers with all CI checks passing across multiple platforms.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 85
Confidence: 90
needs review Rewrites snapcraft/pack action to Node.js, fixes Snapcraft 7 default command issue, adds .comp output support, and includes unit tests. Currently approved by 2 reviewers, CI passing, with 3 unresolved comments.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 85
Confidence: 85
needs review Rewrites snapcraft/pack action to Node.js across 24 files. Currently approved by one reviewer with 3 unresolved comments, CI passing, and awaiting comment resolution.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 85
Confidence: 80
needs review Node-based rewrite of snapcraft/pack action. Adds pack functionality, fixes Snapcraft 7 default command, supports .comp output, and includes unit tests. Approved by 1 reviewer, 3 unresolved comments, all CI passing.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 85
Confidence: 90
needs review Rewrites the snapcraft/pack action in Node.js across multiple commits. Adds pack action, refactors code, fixes Snapcraft 7 default command issue, adds .comp support, and includes unit tests. Currently pending maintainer review with all CI checks passing.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 85
Confidence: 90
needs review Node-based rewrite of snapcraft/pack action. Adds pack action, refactors code, fixes Snapcraft 7 default command, adds .comp output support, and includes unit tests. All CI checks passing, awaiting maintainer review.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 85
Confidence: 95
needs review Node-based rewrite of the snapcraft pack action across 23 files. Introduces a basic pack action, refactors code, fixes Snapcraft 7 defaults, adds .comp support, and includes tests. Currently pending review with all CI checks passing.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 85
Confidence: 95
needs review Node-based rewrite of the snapcraft/pack GitHub Action. Adds pack action, refactors code, fixes Snapcraft 7 default command, adds .comp output support, and includes unit tests. All CI checks passing, awaiting maintainer review.
qwen/qwen3.6-35b-a3b
Staleness: 0
Complexity: 85
Confidence: 95
needs review Rewrites snapcraft/pack GitHub Action in Node.js. Adds pack action, fixes Snapcraft 7 defaults, supports .comp output, and includes unit tests. Currently awaiting maintainer review with all CI checks passing.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 85
Confidence: 90
needs review Rewrites the snapcraft pack action to a Node-based implementation across 23 files. Adds support for .comp outputs, fixes Snapcraft 7 defaults, and includes unit tests. Currently pending maintainer review with all CI checks passing.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 85
Confidence: 90
needs review Node-based rewrite of the snapcraft/pack action. Includes refactoring, fixes Snapcraft 7 default command, adds .comp support, and unit tests. Currently pending maintainer review with passing CI.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 85
Confidence: 95
needs review Node-based rewrite of the snapcraft/pack action. Introduces initial pack functionality, refactors the codebase, fixes Snapcraft 7 default command handling, adds .comp output support, and includes unit tests. Awaiting maintainer review with passing CI.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 85
Confidence: 90
needs review Rewrites snapcraft pack action to Node.js, adds refactoring, fixes Snapcraft 7 default command, supports .comp output, and includes unit tests. Currently pending maintainer review with all CI checks passing.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 85
Confidence: 80
needs review Rewrites snapcraft/pack to a node-based implementation across 23 files. Currently pending maintainer review with multiple CI jobs failing across various Ubuntu versions and architectures.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 90
Confidence: 85
needs review Node-based rewrite of snapcraft/pack action. Adds pack functionality, refactors code, fixes Snapcraft 7 default command issue, adds .comp output support, and includes unit tests. Currently pending maintainer review with multiple CI setup/pack jobs failing.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 85
Confidence: 90
needs review Introduces a node-based rewrite for the snapcraft pack action across 23 files. Currently pending maintainer review with CI jobs running. Includes refactoring, Snapcraft 7 command fixes, .comp output support, and unit tests.

Update history

Date Change
updated
updated
updated
updated
updated
updated
updated
updated
updated
updated
created

Related issues

Issue Project State Summary Similarity
#45 feat: create new snapcraft-pack action craft-actions merged Merged a new snapcraft-pack GitHub Action for automated Snapcraft setup and packaging. Implements upstreamed workflows, cache key improvements, and path variable fixes. Approved by three reviewers, passed all CI checks. README documentation to follow.
75%
#307 ci: pack snap using craft-actions imagecraft merged Merged. Updated CI pipeline to replace action-build with the craft-actions snapcraft pack action. Approved by two reviewers and passed CI checks. The change updates a single workflow file.
74%
#5616 feat(snapcraft_legacy): pack command runs the lifecycle snapcraft merged Merged squashed PR that aligns the core20 snapcraft pack command with core22 and core24 by running the lifecycle. Approved by three reviewers with all CI checks passing. Resolves #5425.
72%
#3683 commands: add snap command snapcraft merged Merged into snapcraft/7.0. Implements legacy snap command support for package creation and fixes output parameter passing to managed instances. Approved by reviewer with passing CI checks.
72%
#49 packages: add snap package support craft-parts merged Merged after implementing snap download and installation for stage and build snaps. Approved by two reviewers, passed CI, and added 1,072 lines across five files. Follows snapcraft conventions with future refactoring planned.
71%
#4794 feat: allow packing a directory snapcraft merged Merged to restore snapcraft pack directory functionality after a snapcraft 8.2 regression. Approved by two reviewers, passed CI, and resolved issue #4769.
70%
#4 snap: introduce initial snap packaging (CRAFT-578) rockcraft merged Merged initial snap packaging implementation (CRAFT-578). Approved by one reviewer, passed CI checks, and added 86 lines across three files.
70%
#55 feat: support 'pack' command craft-actions merged Merged to add version-aware support for the snapcraft pack command, addressing Snapcraft 8 deprecation and Snapcraft 9 breaking changes. Maintainer merged despite unrelated CI infrastructure failures on ppc64el and s390x.
70%
#3637 meta: generate basic snap.yaml (CRAFT-801) snapcraft merged Merged into snapcraft/7.0, adding automatic generation of a basic snap.yaml to enable snap packing. Approved by a reviewer, passed CI, and increased test coverage by 1.16%.
70%