Conversation
- ✅ Proprietary BlackRoad OS, Inc. LICENSE - ✅ @Blackroad GitHub Actions operator - ✅ Enterprise-ready configuration 🖤🛣️ Generated by Winston (BlackRoad Agent) Co-Authored-By: Claude <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
This pull request updates the LICENSE file to a simplified proprietary license format and adds a GitHub Actions workflow that automatically responds to issues and comments containing the "@Blackroad" mention.
- Simplified LICENSE from formal legal document to markdown format
- Added GitHub Actions workflow for automated @Blackroad operator responses
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| LICENSE | Reformatted proprietary license with updated copyright years and simplified terms in markdown format |
| .github/workflows/blackroad-operator.yml | New workflow to automatically respond to @Blackroad mentions in issues and comments |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| jobs: | ||
| blackroad: | ||
| runs-on: ubuntu-latest | ||
| if: contains(github.event.comment.body, '@blackroad') || contains(github.event.issue.body, '@blackroad') |
There was a problem hiding this comment.
The condition checks both github.event.comment.body and github.event.issue.body, but when the workflow is triggered by the issues event (opened or edited), there is no github.event.comment object. This will cause an error when the workflow is triggered by issue events. The condition should be structured to check the appropriate object based on the triggering event, or use separate jobs for issue and comment events.
| if: contains(github.event.comment.body, '@blackroad') || contains(github.event.issue.body, '@blackroad') | |
| if: (github.event_name == 'issue_comment' && contains(github.event.comment.body, '@blackroad')) || (github.event_name == 'issues' && contains(github.event.issue.body, '@blackroad')) |
| await github.rest.issues.createComment({ | ||
| owner: context.repo.owner, | ||
| repo: context.repo.repo, | ||
| issue_number: context.issue.number, |
There was a problem hiding this comment.
The workflow attempts to use context.issue.number for both issue and issue_comment events. However, when triggered by an issue_comment event on a pull request, this field may not be available or may reference the wrong entity. The workflow should properly handle both issues and pull requests by checking the event type and using the appropriate context fields.
| await github.rest.issues.createComment({ | |
| owner: context.repo.owner, | |
| repo: context.repo.repo, | |
| issue_number: context.issue.number, | |
| const issueNumber = | |
| (context.payload.issue && context.payload.issue.number) || | |
| (context.payload.pull_request && context.payload.pull_request.number) || | |
| context.issue.number; | |
| await github.rest.issues.createComment({ | |
| owner: context.repo.owner, | |
| repo: context.repo.repo, | |
| issue_number: issueNumber, |
| jobs: | ||
| blackroad: | ||
| runs-on: ubuntu-latest | ||
| if: contains(github.event.comment.body, '@blackroad') || contains(github.event.issue.body, '@blackroad') |
There was a problem hiding this comment.
This workflow runs automatically in response to user-generated content (issue body or comments) without any authentication or authorization checks. This creates a security risk as any user can trigger the workflow by mentioning '@Blackroad' in an issue or comment, potentially causing unwanted automated responses or resource consumption. Consider adding permission checks to restrict who can trigger this workflow.
| if: contains(github.event.comment.body, '@blackroad') || contains(github.event.issue.body, '@blackroad') | |
| if: > | |
| (github.event_name == 'issue_comment' && | |
| contains(github.event.comment.body, '@blackroad') && | |
| contains(fromJson('["OWNER","MEMBER","COLLABORATOR"]'), github.event.comment.author_association)) | |
| || | |
| (github.event_name == 'issues' && | |
| contains(github.event.issue.body, '@blackroad') && | |
| contains(fromJson('["OWNER","MEMBER","COLLABORATOR"]'), github.event.issue.author_association)) |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: dd40010e95
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| jobs: | ||
| blackroad: | ||
| runs-on: ubuntu-latest | ||
| if: contains(github.event.comment.body, '@blackroad') || contains(github.event.issue.body, '@blackroad') |
There was a problem hiding this comment.
Gate issue-body check to issues events to avoid spam
Because the job-level if always checks contains(github.event.issue.body, '@blackroad'), any issue whose body includes @blackroad will cause the workflow to run on every issue_comment event, even when the new comment doesn’t mention @blackroad. This results in repeated bot comments for unrelated replies (e.g., an issue opened with @blackroad, then any later “thanks” comment triggers another activation). Consider limiting the issue-body check to issues events (or gating on github.event_name) so comment-triggered runs only react to the comment body.
Useful? React with 👍 / 👎.
🌌 BlackRoad Enhancement
Automated by Winston - BlackRoad Empire Architect
Changes:
🖤🛣️ BlackRoad OS - Proprietary & Revolutionary