OmniForge omnicheck-gitlab
Use when checking if MR review findings have been applied — verifies both OmniForge-generated and human reviewer comments against the current diff, posts nudge replies on unaddressed threads
git clone https://github.com/nexiouscaliver/OmniForge
T=$(mktemp -d) && git clone --depth=1 https://github.com/nexiouscaliver/OmniForge "$T" && mkdir -p ~/.claude/skills && cp -r "$T/plugins/omniforge/skills/omnicheck-gitlab" ~/.claude/skills/nexiouscaliver-omniforge-omnicheck-gitlab && rm -rf "$T"
plugins/omniforge/skills/omnicheck-gitlab/SKILL.mdOmniCheck
Verify whether requested MR changes have been applied — diff analysis + targeted nudge comments.
Check all discussion threads on a GitLab MR against the current diff. Resolved threads are marked APPLIED. Unresolved threads are analyzed by a subagent to determine if the fix was applied silently or not at all. Unaddressed threads receive nudge replies and a summary comment.
Core principle: Thread status check + diff analysis + user approval gate = accurate, non-spammy follow-up.
Announce at start: "I'm using OmniCheck to verify findings on MR !{id}."
Prerequisites
CLI authenticated (glab
to verify)glab auth status- Git repository with remote pointing to GitLab
- Current working directory is in the git repo
- MR must have discussion threads
Input Parsing
Accept any of: MR number (
136), prefixed (!136), or full GitLab URL.
Extract MR ID. If URL provided, extract project path and MR IID.
The Process
Phase 1: GATHER — fetch all threads + MR diff/data Phase 2: ANALYZE — single subagent checks each open thread against the diff Phase 3: REPORT — present status table, user approves nudge list Phase 4: NUDGE — reply on each NOT_APPLIED thread + post summary MR comment Phase 5: DONE — no cleanup needed (no worktrees)
Thread Verdicts
| Verdict | Meaning | Action |
|---|---|---|
| Thread is resolved — trusted as-is | None |
| Thread open, but diff shows the concern was addressed | Note in summary only |
| Thread open, diff shows no relevant change | Nudge: thread reply + summary row |
| Ambiguous — cannot determine from diff alone | Flag in report; no automatic nudge |
Phase 1: Gather
Step 1: Fetch all discussion threads.
mcp__omniforge__fetch_mr_discussions(mr_id="{id}", repo_root="{cwd}")
Returns structured threads with:
id, resolvable, resolved, type, file_path, line_number, body, author, replies. The tool already excludes discussions whose notes are all system-generated — every thread returned contains at least one human note.
Note: When building
{UNRESOLVED_THREADS_JSON} for the analysis subagent, include each thread's id field as discussion_id so verdicts can be mapped back to GitLab thread IDs for posting replies.
Step 2: Partition threads.
→ pre-labeledresolved: true
, skip analysisAPPLIED
(anyresolved: false
value) → pass to Phase 2. This includes both resolvable inline threads and general (resolvable
) human comments — both can receive nudge replies.resolvable: false
Step 3: Fetch MR metadata and diff.
mcp__omniforge__fetch_mr_data(mr_id="{id}", repo_root="{cwd}")
Returns: title, author, source_branch, target_branch, diff, diff_line_count, commits, files_changed.
Step 4: Early exit checks.
- Zero threads returned: "MR !{id} has no discussion threads. Nothing to check." Stop.
- Zero unresolved threads (
): "All {N} threads are resolved. Nothing to nudge." Stop.resolved: false
Step 5: Present: "Found {N} total threads ({R} resolved, {U} unresolved). Analyzing {U} unresolved threads."
Phase 2: Analyze (Single Subagent)
Goal: For each unresolved thread, determine if the diff addresses its concern.
Template:
./references/analysis-agent-prompt.md
Fill template placeholders:
— MR number{MR_ID}
— MR title{MR_TITLE}
— JSON array of all unresolved threads{UNRESOLVED_THREADS_JSON}
— Full diff string from Phase 1{GIT_DIFF}
Large MR handling: If unresolved thread count > 15 AND diff_line_count > 5000, group threads by file and dispatch up to 3 subagents. Merge results before Phase 3.
Expected Return
[ { "discussion_id": "abc123", "file_path": "src/auth.py", "line_number": 47, "body_summary": "Missing null check on user input", "verdict": "NOT_APPLIED", "confidence": 91, "reasoning": "The diff shows no changes to src/auth.py around line 47. The null check is still absent." }, { "discussion_id": "def456", "file_path": ".gitlab-ci.yml", "line_number": 1072, "body_summary": "Missing placeholder mapping for STRIPE_PRICE_ENTERPRISE", "verdict": "SILENTLY_APPLIED", "confidence": 87, "reasoning": "Line 1072 in .gitlab-ci.yml was changed in the diff to include the placeholder mapping. The thread was not resolved but the concern is addressed." } ]
Verdict definitions:
— thread open but diff shows the concern was addressedSILENTLY_APPLIED
— thread open and diff shows no relevant changeNOT_APPLIED
— diff changes are present but genuinely unclear if they address the concernNEEDS_HUMAN
Phase 3: Report (User Approval Gate)
REQUIRED REFERENCE:
./references/nudge-guide.md — read before presenting results. Contains the exact presentation format and user action matrix. Do NOT present without loading this reference.
Present status combining Phase 1 resolved threads + Phase 2 verdicts:
OmniCheck — MR !{id}: {title} ✓ Applied (resolved): {N} threads ✓ Silently Applied: {N} threads ✗ Not Applied: {N} threads ? Needs Human Review: {N} threads NOT_APPLIED threads (will receive nudge): 1. {file}:{line} — {body_summary} [confidence: {score}%] 2. {file}:{line} — {body_summary} [confidence: {score}%] NEEDS_HUMAN threads (no automatic nudge): 3. general — {body_summary} Post nudge replies on NOT_APPLIED threads? [Y/n] (Enter numbers to exclude specific threads, e.g. "exclude 2")
CRITICAL: No comments are posted until the user explicitly approves.
Phase 4: Nudge
REQUIRED REFERENCE:
./references/nudge-guide.md — contains the exact thread reply template and summary comment template. Do NOT post without loading this reference.
For each approved NOT_APPLIED thread:
Step 1: Post thread reply.
mcp__omniforge__reply_to_discussion( mr_id="{id}", discussion_id="{discussion_id}", body="{nudge_reply_text}", repo_root="{cwd}" )
Step 2: After all thread replies succeed or fail, post one summary comment.
mcp__omniforge__post_review_summary( mr_id="{id}", summary="{summary_comment_text}", repo_root="{cwd}" )
Ordering guarantee: All thread replies before the summary comment.
On reply failure: Collect the failure, continue with remaining threads, and note the failure in the summary comment.
Phase 5: Done
No worktrees → no cleanup needed.
Report:
OmniCheck complete — MR !{id} ✓ Nudged: {N} threads ✗ Failed to post: {N} threads (list them)
Error Handling
| Error | Response |
|---|---|
| glab not authenticated | "Run first." Stop. |
| MR not found | "MR !{id} not found. Verify the number and repository." Stop. |
| No discussion threads | "MR !{id} has no discussion threads. Nothing to check." Stop. |
| All threads resolved | "All {N} threads are resolved. Nothing to nudge." Stop. |
| Analysis agent fails | Present error to user; offer to retry or abort |
| Thread reply fails | Continue with remaining threads; note failure in summary |
| Summary comment fails | Report failure; thread replies already posted |
Integration
MCP Tools:
— Fetch all discussion threadsmcp__omniforge__fetch_mr_discussions
— Fetch MR metadata and diffmcp__omniforge__fetch_mr_data
— Post nudge reply on a threadmcp__omniforge__reply_to_discussion
— Post summary comment on MRmcp__omniforge__post_review_summary
Subagent Template:
— Analysis Agent (single, diff-only check)./references/analysis-agent-prompt.md
Never
- Post comments without explicit user approval (Phase 3 gate)
- Add AI attribution to any posted comment
- Use
(GitLab — usegh
exclusively)glab - Auto-resolve any threads (OmniCheck only nudges, never resolves)
- Create worktrees (diff-only analysis)
- Skip any of the 5 phases
Always
- Fetch threads and diff in Phase 1 before dispatching analysis
- Present status table and wait for user approval before posting
- Post thread replies before the summary comment
- Report final outcome including any failed posts
- Use
for all GitLab operationsglab