[r5n-0] gitlab support, actions command improvements/fixes#14
Merged
Conversation
Replace glab CLI dependency with direct GitLab REST API calls in GitLabProvider. Token resolves from GITLAB_TOKEN, CI_JOB_TOKEN, or glab auth fallback. Switch CI templates to oven/bun:alpine with git.
- Use push-based triggers instead of merge_request_event (which doesn't fire after merge) - Extract MR IID from CI_COMMIT_DESCRIPTION with basic grep - Fall back to CI_JOB_TOKEN for git push when GITLAB_TOKEN is not set - Use correct JOB-TOKEN header for CI_JOB_TOKEN API auth - Remove redundant variables section (CI vars are auto-inherited)
git add fails when glob patterns like **/CHANGELOG.md match no files. Add each file individually with nothrow to skip non-existent paths.
GitHub: remind to enable workflow permission for creating PRs. GitLab: remind to add GITLAB_TOKEN with api + write_repository scopes.
Without state filter, findPr returns merged MRs causing release-pr to update a closed MR instead of creating a new one.
Exit exits with code 0 (graceful), causing CI pipelines to report success on failures. Also include response body in GitLab API errors.
The run() wrapper was only extracting stderr from shell errors, dropping the message from API errors (e.g. GitLab 403 responses).
Provider auth was never initialized when callers forgot to call ensureAvailable. Move it into the factory so providers are always ready to use.
PullRequestAnalyzer
deralaxo
force-pushed
the
r5n-0-sisyphus-polish
branch
from
March 26, 2026 12:02
f9540c5 to
c40289c
Compare
history Set lastStone.commit to HEAD on init so new projects have a baseline. Guard commit analyzer to return empty when lastStone is missing. Only check pending stones for duplicate hashes since lastStone already scopes commits to post-release.
cleanup from `roll`
deralaxo
marked this pull request as ready for review
March 27, 2026 12:09
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Polish pass on Sisyphus — bug fixes, performance, and cleanup across providers, CI templates, and core commands.
Bug fixes
glabCLI dependency with direct REST API calls, fix JOB-TOKEN auth, filterfindPrto open MRs only, fix CI templates for push-based triggersensureAvailableincreateGitProviderso auth is always initializedErrorinstead ofExit(which exits 0) for release failureslastStone.committo HEAD on initfinallyfor branch restoration instead of duplicatedgit checkoutcalls, handle missing changelog files withnothrowquiet/nothrow, fall back togit diff-treefor squash merge file detectionnewVersionin fallback stone message instead of current version$interpolation issuesPerformance
git log --name-onlycall using%x00delimitersFeatures
deletemethod for removing config keyshandle-pushjob to both GitHub and GitLab workflows for direct pushes to default branchhandle-pushon PR merges viaweb-flowcommitter checkRefactors
GitRemoteParserandproviders/index.tscurrentReleasecleanup fromroll— it's a release marker meant to persist