[CHORE] Update lerna and remove issue_handler tool folder#1227
Merged
Conversation
There was a problem hiding this comment.
Pull request overview
Updates monorepo tooling dependencies (primarily Lerna) to address security findings, and removes the obsolete tools/issue_handler utility that should have been deleted with its GitHub Action workflow.
Changes:
- Bump
lernafrom9.0.5to9.0.7and refreshyarn.lockaccordingly - Add a Yarn
resolutionsoverride forhandlebarsto enforce>=4.7.9 - Remove the
tools/issue_handlertool source, tests, and related scripts/docs
Reviewed changes
Copilot reviewed 19 out of 20 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| yarn.lock | Updates lockfile to reflect lerna@9.0.7 and associated dependency graph/security-related bumps. |
| package.json | Bumps lerna devDependency and adds handlebars resolution (>=4.7.9). |
| tools/issue_handler/tests/test_slack_handler.py | Removes deleted tool’s unit tests. |
| tools/issue_handler/tests/test_openai_handler.py | Removes deleted tool’s unit tests. |
| tools/issue_handler/tests/test_github_handler.py | Removes deleted tool’s unit tests. |
| tools/issue_handler/test_slack_webhook.py | Removes deleted tool’s local webhook test script. |
| tools/issue_handler/test_local.py | Removes deleted tool’s local test script. |
| tools/issue_handler/src/slack_handler.py | Removes Slack posting implementation for deleted tool. |
| tools/issue_handler/src/openai_handler.py | Removes OpenAI analysis implementation for deleted tool. |
| tools/issue_handler/src/github_handler.py | Removes GitHub issue fetching implementation for deleted tool. |
| tools/issue_handler/src/analyze_issue.py | Removes tool entrypoint/orchestration script. |
| tools/issue_handler/src/init.py | Removes Python package marker for deleted tool. |
| tools/issue_handler/setup_env.sh | Removes deleted tool’s environment setup helper. |
| tools/issue_handler/run_tests.sh | Removes deleted tool’s test runner script. |
| tools/issue_handler/requirements.txt | Removes deleted tool’s Python dependency list. |
| tools/issue_handler/pytest.ini | Removes deleted tool’s pytest configuration. |
| tools/issue_handler/integration_tests/test_real_issue.py | Removes deleted tool’s integration test script. |
| tools/issue_handler/integration_tests/test_analysis.py | Removes deleted tool’s integration test script. |
| tools/issue_handler/README.md | Removes deleted tool documentation. |
| tools/issue_handler/.gitignore | Removes deleted tool’s gitignore entries. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
iadjivon
approved these changes
Mar 27, 2026
5362668 to
9c6c115
Compare
cdn34dd
approved these changes
Apr 13, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What does this PR do?
This PR updates lerna to 9.0.7 (which solves some security issues raised by dependabot).
It also removes the issue_handler tool folder, which should have been removed alongside the Github action for the issue notification workflow.
Review checklist (to be filled by reviewers)