Skip to content

feat(gitlab): add permission CSV exporter - #9955

Merged
waleedlatif1 merged 3 commits into
stagingfrom
codex/gitlab-permission-export
Nov 12, 2026
Merged

feat(gitlab): add permission CSV exporter#9955
waleedlatif1 merged 3 commits into
stagingfrom
codex/gitlab-permission-export

Conversation

@waleedlatif1

@waleedlatif1 waleedlatif1 commented Nov 12, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator

Summary

  • Add a standalone GitLab exporter for both Search permission CSVs, with paced requests and atomic output.
  • Document the export and revision-checked CI upload. Stop on missing primary emails or incomplete exports.

Type of Change

  • New feature

Testing

  • HTTPS fixture exercises the CLI in a subprocess, then validates its CSVs with Sim's existing parser and grant resolver: 16 checks.
  • Negative controls verify duplicate identities, hidden emails, redirects, expired memberships, and inconsistent pagination totals are rejected; issues-only projects export successfully and request pacing is enforced.
  • Root tests, lint, type-check, audits, and block registry check.
  • Production docs build, compatibility checks, and responsive layout verification.

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing (new tests pass the test-audit authoring gate)
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel

vercel Bot commented Nov 12, 2026 •
edited
Loading

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
docs Ready Ready Preview Nov 12, 2026 10:32pm UTC

Request Review

cubic-dev-ai[bot]
cubic-dev-ai Bot reviewed Nov 12, 2026
View reviewed changes

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 5 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Turn on auto-fix | Re-trigger cubic

@greptile-apps

greptile-apps Bot commented Nov 12, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 3/5

[Medium impact] The new workflow can refresh grants for a configured GitLab source.

Check that GitLab confirmed each primary email before merging this exporter.

Findings

  1. P1 Security Unconfirmed emails receive access ▶

Summary

Adds a standalone GitLab exporter and a scheduled CI upload example.

  • GitLab operators can export eligible project members into Sim’s two permission CSVs.
  • Scheduled GitLab jobs can export and upload both permission files to Sim.

Diagram

 
%%{init: {'theme': 'neutral'}}%%
flowchart LR
  A[GitLab project] --> B[Page through members]
  B --> C[Read eligible profiles]
  C --> D[Validate identities and CSV limits]
  D --> E[Publish both CSVs together]
  E --> F[Upload with expected revision]
  F --> G[Sim replaces project grants]
Loading

Reviews (3) · Last reviewed commit: "fix(gitlab): reject inconsistent members..." · Reviewed by Greptile

greptile-apps[bot]
greptile-apps Bot reviewed Nov 12, 2026
View reviewed changes
Comment thread apps/docs/public/static/examples/gitlab/export-permissions.py Resolved
Comment thread apps/docs/scripts/test-gitlab-export-e2e.py Resolved
@waleedlatif1

waleedlatif1 commented Nov 12, 2026

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

waleedlatif1 commented Nov 12, 2026

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Nov 12, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

cubic-dev-ai[bot]
cubic-dev-ai Bot reviewed Nov 12, 2026
View reviewed changes

@cubic-dev-ai cubic-dev-ai Bot left a comment •
edited
Loading

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All reported issues were addressed across 5 files

Reply with feedback, questions, or to request a fix.

Turn on auto-fix | Re-trigger cubic

Comment thread apps/docs/public/static/examples/gitlab/export-permissions.py Resolved
@waleedlatif1
waleedlatif1 force-pushed the codex/gitlab-permission-export branch from b0c0003 to 736434f Compare November 12, 2026 22:27
@waleedlatif1

waleedlatif1 commented Nov 12, 2026

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

waleedlatif1 commented Nov 12, 2026

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Nov 12, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

cubic-dev-ai[bot]
cubic-dev-ai Bot reviewed Nov 12, 2026
View reviewed changes

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 5 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Turn on auto-fix | Re-trigger cubic

@waleedlatif1
waleedlatif1 merged commit cc012bf into staging Nov 12, 2026
76 of 78 checks passed
@waleedlatif1
waleedlatif1 deleted the codex/gitlab-permission-export branch November 12, 2026 22:30
@waleedlatif1 waleedlatif1 mentioned this pull request Nov 13, 2026
Merged

This branch was successfully deployed

1 active deployment
Preview — 736434fe Deployed Nov 12, 2026 by vercel[bot]
View deployment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@greptile-apps greptile-apps[bot] greptile-apps[bot] left review comments
@cubic-dev-ai cubic-dev-ai[bot] cubic-dev-ai[bot] left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@waleedlatif1