Skip to content

ci: validate governance sources and label-sync targets - #10

Merged
DerekCorniello merged 9 commits into
mainfrom
audit/governance-source
Aug 30, 2026
Merged

ci: validate governance sources and label-sync targets#10
DerekCorniello merged 9 commits into
mainfrom
audit/governance-source

Conversation

@DerekCorniello

Copy link
Copy Markdown
Contributor

Issue 29 follow-up. Add credential-free governance source validation and reject duplicate label-sync targets. Includes manifest checks, Shell/Python validation, and no policy behavior changes beyond fail-closed duplicate handling. Derek-only commits; no secrets accessed.

@greptile-apps

greptile-apps Bot commented Aug 30, 2026

Copy link
Copy Markdown

Greptile Summary

This PR adds credential-free CI validation for governance sources and duplicate label-sync targets while fixing quoted label-name parsing.

  • Adds a validation workflow for Python sources, shell syntax, repository manifests, and label-sync regression tests.
  • Rejects duplicate synchronization targets before performing label operations.
  • Makes the label-sync script sourceable and verifies parsing of a quoted label name.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename Overview
.github/workflows/validate.yml Adds a read-only workflow that validates governance scripts, manifests, shell syntax, and the label parser regression test.
scripts/sync-labels.sh Rejects duplicate targets, fixes whitespace-sensitive quote removal for label names, and guards direct execution so functions can be tested safely.
scripts/test-sync-labels.sh Adds a focused regression test confirming that a quoted label name containing spaces and a colon is decoded correctly.
scripts/tests/quoted-label.yml Provides the quoted-label fixture used by the new shell regression test.

Reviews (3): Last reviewed commit: "merge: update governance branch with mai..." | Re-trigger Greptile

Comment thread scripts/sync-labels.sh Outdated
@sonarqubecloud

Copy link
Copy Markdown

@DerekCorniello
DerekCorniello merged commit a3b7f2a into main Aug 30, 2026
4 checks passed
@DerekCorniello
DerekCorniello deleted the audit/governance-source branch August 30, 2026 04:54
@DerekCorniello
DerekCorniello restored the audit/governance-source branch August 30, 2026 04:56
@DerekCorniello
DerekCorniello deleted the audit/governance-source branch August 30, 2026 05:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant