-
Notifications
You must be signed in to change notification settings - Fork 0
docs: add remote label setup instructions to README #29
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
Conversation
… giant comment - Update postSuggestionsToGitHub() to post ALL suggestions as inline comments - Create generateAllInlineComments() method to handle all confidence levels (≥50%) - Increase resolvable suggestion limit from 5 to 8 - Add fallback handling for failed inline comments - Simplify summary comment to avoid duplication with inline comments - Standardize line_number field handling across services - Add helper methods: getConfidenceLabel(), inferLineNumber() - Update tests to match new inline comment behavior Benefits: - Better UX: suggestions appear exactly where they belong in code - Contextual feedback: reviewers see suggestions while reading relevant code - Uses GitHub's native inline comment and resolvable suggestion features - Reduces noise from giant comment blocks - Maintains all existing functionality with robust fallbacks 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
The 'on' keyword should not be quoted in YAML workflow files. This was causing GitHub Actions to incorrectly trigger the emergency-controls workflow on push events instead of only on manual workflow_dispatch.
Add explicit check for workflow_dispatch event type to ensure emergency controls only run when manually triggered, never on push or pull_request events.
- Remove invalid use of env context in timeout-minutes (not available at job level) - Use hardcoded value of 10 minutes instead of fromJSON(env.EMERGENCY_TIMEOUT_MINUTES) - Remove unused env section from workflow file Fixes GitHub Actions error: 'Unrecognized named-value: env' in timeout-minutes
- Updated inquirer from ^8.2.5 to ^10.2.2 to resolve vulnerability chain - Added npm override for tmp package to version 0.2.4 (fixes CVE for versions <=0.2.3) - All tests passing, functionality verified - npm audit now reports 0 vulnerabilities Addresses security review findings from PR #28. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
Added optional Step 3 with instructions for remotely installing GitHub labels used by AI workflows. Provides both quick curl command and GitHub workflow file options for users adopting the action. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
🤖 AI Review by Resolvable Comments✅ Great work! No significant issues were found during the AI analysis. Analysis Summary
What was reviewed:
✅ Recommendation: APPROVEThe code changes in this pull request meet quality standards and are ready for approval. No blocking issues were identified. AI Review completed at 2025-08-16T16:56:34.206Z |
🔒 Security Scan Results✅ npm audit - Dependency SecurityNo vulnerable dependencies found.
|
Summary
Changes
Test plan
🤖 Generated with Claude Code