You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
test(mcp): qualify Antigravity CLI list results - #728
Qualify GitLab MCP list results through the actual Antigravity CLI without external credentials or model inference. Preserve the existing server output contract for Claude Code and Codex.
Changes
Add installed-package qualification with isolated GitLab and scripted Gemini endpoints, verified client archives, and separate wire and client-result evidence.
Retain the integrity-checked Gemini CLI Core 0.66.0 transport as a diagnostic baseline. Antigravity 1.4.10 preserves the results that the historical transport invalidates.
Document supported clients in the generated GitHub and npm READMEs, plus Antigravity setup, migration, tested behavior and repeatable qualification.
Testing
MCP contract tests and installed-client qualification pass on macOS and Linux with Node.js 24 on Linux. Focused lint, formatting, the documentation build and README generation pass on macOS.
Exercise the installed core package through the actual Antigravity CLI with isolated synthetic GitLab and Gemini endpoints. Compare list, object, native and error results with the integrity-verified historical Gemini transport.
Document client setup, migration and repeatable qualification.
Closes#716
Reviewing files that changed from the base of the PR and between 801ef5d and 3ef1565.
Files selected for processing: 5
README.md
packages/gitlab-mcp-db/README.md
packages/gitlab-mcp/README.md
packages/gitlab-mcp/README.md.in
packages/gitlab-mcp/package.json
Walkthrough
Adds Antigravity CLI setup documentation and a qualification harness. The harness runs isolated GitLab MCP scenarios, records client and protocol results, checks current and historical result contracts, and writes a report.
Changes: Antigravity CLI qualification
Layer / File(s)
Summary
Qualification contracts and fixtures packages/gitlab-mcp/scripts/antigravity-contract.mjs, packages/gitlab-mcp/scripts/evaluation-process.mjs, packages/gitlab-mcp/tests/manual/fixtures/*, packages/gitlab-mcp/tests/manual/fixtures/workflow-gitlab.mjs, packages/gitlab-mcp/tests/scripts/antigravity-results.test.mjs, packages/gitlab-mcp/tests/unit/evaluation.test.ts
Adds checks for MCP envelopes, recorded tool calls, client transcripts, and provider responses. Adds fixtures for recording stdio traffic and scripting provider calls, an option for empty merge-request results, and tests for these behaviors and explicit child-process environments.
Qualification execution and reporting packages/gitlab-mcp/scripts/qualify-antigravity.mjs, packages/gitlab-mcp/package.json
Adds a command that prepares verified client and package artifacts, runs isolated scenarios, checks results, writes a report, and removes temporary runtime files.
Client setup and qualification documentation packages/gitlab-mcp/docs/clients/antigravity.md, packages/gitlab-mcp/docs/clients/index.md, packages/gitlab-mcp/docs/.vitepress/config.mts
Documents Antigravity CLI setup, migration details, result shapes, and qualification findings. Adds the client guide to the overview and sidebar.
sequenceDiagram
participant QualificationScript
participant AntigravityCLI
participant ScriptedGemini
participant MCPEnvelopeRecorder
participant GitLabMCPServer
participant WorkflowGitLab
QualificationScript->>AntigravityCLI: run configured scenario
AntigravityCLI->>ScriptedGemini: send provider request
ScriptedGemini-->>AntigravityCLI: return SSE tool call
AntigravityCLI->>MCPEnvelopeRecorder: send tools/call over stdio
MCPEnvelopeRecorder->>GitLabMCPServer: forward MCP request
GitLabMCPServer->>WorkflowGitLab: request merge-request data
WorkflowGitLab-->>GitLabMCPServer: return fixture data
GitLabMCPServer-->>MCPEnvelopeRecorder: return MCP result
MCPEnvelopeRecorder-->>AntigravityCLI: forward MCP result
AntigravityCLI-->>QualificationScript: complete scenario
QualificationScript->>QualificationScript: validate recorded calls and results
Loading
Merge Risk: Minimal · previous review 801ef
No demonstrated issue blocks merging. The optional parent-test timeout can be considered separately.
Pre-merge checks: 4 · 1
Failed checks: 1 warning
Check name
Status
Explanation
Resolution
Docstring Coverage
Docstring coverage is 38.46% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documenta…
Write docstrings for the functions missing them to satisfy the coverage threshold.
Passed checks: 4 passed
Check name
Status
Explanation
Description Check
Check skipped - CodeRabbit’s high-level summary is enabled.
Title check
The title clearly identifies the main change: qualifying Antigravity CLI list results through MCP tests.
Linked Issues check
Issue #716 coding requirements are implemented. qualify-antigravity.mjs installs the packed GitLab MCP artifact, uses loopback GitLab and scripted provider fixtures, isolates client homes, verifies …
Out of Scope Changes check
The changes stay within issue #716. The package script, qualification helpers, isolated fixtures, recorder, tests, documentation, and documentation navigation directly support installed-client qualifi…
Full details: Linked Issues check
Explanation
Issue #716 coding requirements are implemented. qualify-antigravity.mjs installs the packed GitLab MCP artifact, uses loopback GitLab and scripted provider fixtures, isolates client homes, verifies archive integrity, and records wire, client, provider, and GitLab evidence. It qualifies empty and single-MR lists through the installed AGY binary, plus object, native structured-result, and error controls. recordedCall and validateEnvelope validate the MCP request and response with the SDK. historicalResult executes the pinned integrity-checked Gemini CLI Core 0.66.0 transport and keeps that result separate from AGY. The qualification checks unchanged list text, empty-list semantics, MR fields, errors, and explicit structured content. The Antigravity documentation records version 1.4.10, migration guidance, historical Gemini behavior, setup, limits, and repeatable execution. The added tests cover fixture behavior, envelope validation, environment isolation, client/provider agreement, and auxiliary provider requests. No REST filters, MR business logic, general result envelope, array structuredContent, speculative schemas, or duplicated list payloads were changed.
Full details: Out of Scope Changes check
Explanation
The changes stay within issue #716. The package script, qualification helpers, isolated fixtures, recorder, tests, documentation, and documentation navigation directly support installed-client qualification, evidence capture, or migration guidance. The existing server result behavior remains unchanged. No unrelated production feature, REST filter, MR business logic, or broad result-shape redesign is included.
Full details: Docstring Coverage
Explanation
Docstring coverage is 38.46% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 13 functions across 9 files. (3 skipped: 3 unsupported.)
Resolution
Write docstrings for the functions missing them to satisfy the coverage threshold.
Finishing Touches: 💡 1
📝 Generate docstrings: 💡
Commit to this branch
Create a new PR
🧪 Generate unit tests (beta)
Commit to this branch
Create a new PR
Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
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.
Unverified finding
Verification ran but could not confirm this finding. It is shown for review, not as a verified issue.
Add a timeout to spawnSync.
This call now runs two node:test files. antigravity-results.test.mjs starts loopback HTTP servers and uses fetch calls without a timeout. If a fixture server or request hangs, spawnSync blocks the Jest worker with no limit. The whole unit run then stalls instead of failing.
Set a timeout. If the limit is reached, status is null. The existing toMatchObject assertion then fails and prints the partial report.
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.
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
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.
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Summary
Qualify GitLab MCP list results through the actual Antigravity CLI without external credentials or model inference. Preserve the existing server output contract for Claude Code and Codex.
Changes
Testing
MCP contract tests and installed-client qualification pass on macOS and Linux with Node.js 24 on Linux. Focused lint, formatting, the documentation build and README generation pass on macOS.
Closes #716