Uh oh!
There was an error while loading. Please reload this page.
Conversation
…logic - Updated the Helm chart packaging step to select the most recent `.tgz` file for release. - Modified the commit message to dynamically include all packaged Helm charts, enhancing clarity during releases. - Streamlined the addition of packaged charts to the Git index, ensuring all relevant files are included in the commit.
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
| echo "No index/tgz changes to commit" | ||
| else | ||
| git commit -m "Release $CHART_TGZ" | ||
| git commit -m "Release helm chart(s): $(ls tracebloc-*.tgz 2>/dev/null | tr '\n' ' ')" |
There was a problem hiding this comment.
Commit message lists all historical charts, not just new
Low Severity
The ls tracebloc-*.tgz in the commit message runs on the gh-pages branch, which accumulates .tgz files from all previous releases. As releases build up, this commit message will list every historical chart file (e.g., "Release helm chart(s): tracebloc-1.0.0.tgz tracebloc-2.0.0.tgz tracebloc-2.0.1.tgz") rather than just the newly released one. The tr '\n' ' ' flattening makes this worse than the old code, where newlines caused git to split the message so only the first filename appeared in the commit subject. The message grows unboundedly with each release.
.tgzfile for release.Note
Low Risk
Low risk: only adjusts GitHub Actions release workflow shell logic for selecting and committing packaged Helm chart artifacts; main risk is accidentally adding/committing unintended
tracebloc-*.tgzfiles if multiple are present in the workspace.Overview
Updates the Helm chart packaging step to record the most recently generated
tracebloc-*.tgzfor artifact upload (instead of relying on an arbitrarylsmatch).On the
gh-pagespublish step, stagesindex.yamlplus alltracebloc-*.tgzfiles and changes the commit message to list the packaged chart tarballs, improving clarity when multiple charts are present.Written by Cursor Bugbot for commit e598b91. This will update automatically on new commits. Configure here.