-
Notifications
You must be signed in to change notification settings - Fork 1.6k
[CI] AI-generated content detection workflow improvements #8637
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
[CI] AI-generated content detection workflow improvements #8637
Conversation
f1dc190 to
c7f4269
Compare
…nto separate jobs Signed-off-by: Vitor Vasconcellos <[email protected]>
Signed-off-by: Vitor Vasconcellos <[email protected]>
c7f4269 to
be09f72
Compare
chalin
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are we reading to take this for a spin?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR improves the AI content detection workflow by introducing a separate COPILOT_TOKEN for GitHub Copilot CLI operations, along with several workflow enhancements including input validation, improved error handling, and result reporting.
Changes:
- Separates token concerns:
GH_TOKENfor GitHub API operations andCOPILOT_TOKENfor Copilot CLI - Adds a validate-inputs job to the workflow with comprehensive input validation
- Implements graceful handling when COPILOT_TOKEN is not configured (skips analysis and exits successfully)
- Adds workflow improvements including concurrency control, npm caching, and a report-results job
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
.github/workflows/ai-content-detection.yml |
Adds validate-inputs job, updates analyze job with token separation and caching, adds report-results job, and implements concurrency control |
scripts/ai-content-detection/analyze.js |
Introduces COPILOT_TOKEN configuration, updates Copilot CLI to use separate token, adds graceful handling when token is missing |
scripts/ai-content-detection/README.md |
Documents the two-token architecture, adds Token Requirements section explaining the rationale and permissions needed |
Co-authored-by: Copilot <[email protected]> Co-authored-by: Patrice Chalin <[email protected]>
Signed-off-by: Vitor Vasconcellos <[email protected]>
Signed-off-by: Vitor Vasconcellos <[email protected]>
…t's numeric Signed-off-by: Vitor Vasconcellos <[email protected]>
Signed-off-by: Vitor Vasconcellos <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.
Co-authored-by: Copilot <[email protected]>
Signed-off-by: Vitor Vasconcellos <[email protected]>
…ars input Signed-off-by: Vitor Vasconcellos <[email protected]>
Signed-off-by: Vitor Vasconcellos <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
Signed-off-by: Vitor Vasconcellos <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
I have read and followed the
contribution guidelines,
including the First-time contributing? note.
This PR includes AI generated content, and I have read and conform to
the
Generative AI Contribution Policy.
Contributes to [CI] Add AI-generated content detection workflow #8520.