Skip to content

t3820: Pin dependency versions in GitLab CI template#3828

Closed
marcusquinn wants to merge 1 commit intomainfrom
bugfix/pin-gitlab-ci-versions
Closed

t3820: Pin dependency versions in GitLab CI template#3828
marcusquinn wants to merge 1 commit intomainfrom
bugfix/pin-gitlab-ci-versions

Conversation

@marcusquinn
Copy link
Copy Markdown
Owner

Summary

  • Pin opencode-ai to @1.2.21 for reproducible npm installs (main + minimal configs)
  • Pin glab CLI to v1.22.0 — removes fragile curl | grep | cut GitHub API parsing that could break on API changes or rate limits
  • Fix glab tarball extraction path from glab to bin/glab (the tarball nests the binary under bin/)
  • Add glab version verification step after installation

Review Feedback Addressed

All findings from PR #5 Gemini code review on configs/mcp-templates/opencode-gitlab-ci.yml:

Line Severity Finding Fix
33 HIGH npm install --global opencode-ai unpinned Pinned to @1.2.21
39-42 HIGH glab fetches latest + fragile JSON parsing Pinned to v1.22.0, removed API call
41 MEDIUM Tarball extracts to bin/glab not glab Fixed mv path

Closes #3820

…uilds

- Pin opencode-ai to @1.2.21 (both main and minimal configs)
- Pin glab CLI to v1.22.0 instead of fetching latest via fragile grep/cut
- Fix glab tarball extraction path (bin/glab, not glab)
- Add glab installation verification step

Closes #3820
@gemini-code-assist
Copy link
Copy Markdown

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 8, 2026

Warning

Rate limit exceeded

@marcusquinn has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 27 minutes and 37 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 55c0c762-8ce8-4cb5-9abd-62dd92da0a8f

📥 Commits

Reviewing files that changed from the base of the PR and between 339facd and e9ec0dd.

📒 Files selected for processing (1)
  • configs/mcp-templates/opencode-gitlab-ci.yml
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch bugfix/pin-gitlab-ci-versions

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 8, 2026

🔍 Code Quality Report

�[0;35m[MONITOR]�[0m Code Review Monitoring Report

�[0;34m[INFO]�[0m Latest Quality Status:
SonarCloud: 0 bugs, 0 vulnerabilities, 125 code smells

�[0;34m[INFO]�[0m Recent monitoring activity:
Sun Mar 8 01:22:32 UTC 2026: Code review monitoring started
Sun Mar 8 01:22:33 UTC 2026: SonarCloud - Bugs: 0, Vulnerabilities: 0, Code Smells: 125

📈 Current Quality Metrics

  • BUGS: 0
  • CODE SMELLS: 125
  • VULNERABILITIES: 0

Generated on: Sun Mar 8 01:22:35 UTC 2026


Generated by AI DevOps Framework Code Review Monitoring

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Mar 8, 2026

@johnwaldo
Copy link
Copy Markdown
Contributor

Closing as duplicate of #3830 — both target t3820 but #3830 is the cleaner implementation. Keeping #3830.

@johnwaldo
Copy link
Copy Markdown
Contributor

This PR is a duplicate of #3830 — both target t3820 (pin CI dependency versions) and modify the same file (configs/mcp-templates/opencode-gitlab-ci.yml). Recommend closing this one in favour of #3830 which is the cleaner implementation (fewer line changes for the same result).

@johnwaldo
Copy link
Copy Markdown
Contributor

Closing as duplicate of #3830 — both target t3820 but #3830 is the cleaner implementation.

@marcusquinn
Copy link
Copy Markdown
Owner Author

@coderabbitai review

Review bots were rate-limited when this PR was created (affected: coderabbitai gemini-code-assist). Requesting a review retry.

@gemini-code-assist
Copy link
Copy Markdown

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 8, 2026

@marcusquinn Sure! I'll kick off a fresh review of this PR now. Note that based on the comment thread, johnwaldo has flagged this as a duplicate of #3830 (both targeting t3820 with changes to configs/mcp-templates/opencode-gitlab-ci.yml), recommending #3830 as the cleaner implementation. You may want to consider that before merging. I'll proceed with the review regardless.


🧠 Learnings used
Learnt from: marcusquinn
Repo: marcusquinn/aidevops PR: 0
File: :0-0
Timestamp: 2026-03-03T16:33:18.313Z
Learning: For the marcusquinn/aidevops repository: The daily quality sweep system (pulse-wrapper.sh) should only mention coderabbitai for full codebase review when there are significant changes (Quality Gate failure, +10 issues in one day, etc.), not on every automated sweep with stable metrics.
✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@marcusquinn
Copy link
Copy Markdown
Owner Author

This PR appears orphaned — no active worker process found and no activity for 14+ hours. Flagging for re-dispatch. If work is still in progress, remove the status:orphaned label.

@marcusquinn
Copy link
Copy Markdown
Owner Author

Merging via pulse supervisor.

@marcusquinn
Copy link
Copy Markdown
Owner Author

Merge conflict detected — this PR cannot be auto-merged. The branch needs to be rebased onto main. Daily PR cap reached for today; will retry dispatch tomorrow.

@marcusquinn
Copy link
Copy Markdown
Owner Author

Closing — this PR has merge conflicts and touches too many files (blast radius issue, see t1422 / GH#3955). The underlying fixes will be re-created as smaller PRs (max 5 files each) to prevent conflict cascades.

Root cause: quality-debt batch PRs touching 10-69 files each created a conflict cascade where each merge invalidated the next PR's base. New rules cap quality-debt PRs at 5 files max and enforce serial merge ordering.

@marcusquinn marcusquinn closed this Mar 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

quality-debt: configs/mcp-templates/opencode-gitlab-ci.yml — PR #5 review feedback (high)

2 participants