Repository navigation
Optimize large PR change analysis - #58
Merged
Changyong Gong (chagong) merged 2 commits intoOct 19, 2026
Merged
Conversation
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
🟡 Changes recommended
Manifest pagination can advertise an invalid continuation page, and the new tool lacks telemetry classification.
2 open issues
What changed in this PR
Optimizes IssueLens large-PR analysis by separating compact file discovery from targeted evidence reads, reducing repeated patch content in model conversations.
Changes:
- Adds a patch-free PR manifest tool with bounded responses and anonymous public-repository fallback.
- Updates agents to plan evidence reads before inspecting pinned source or bounded patch groups.
- Extends tool registration, tests, and documentation.
| File | Description |
|---|---|
| tests/test_team_memory_push.py | Checks updated batch-analysis instructions. |
| tests/test_team_memory_instructions.py | Updates the maintenance tool allowlist expectation. |
| tests/test_team_memory_coordinator.py | Checks manifest-first coordinator requests. |
| tests/test_mcp_tool_documentation.py | Adds manifest permission expectations. |
| tests/test_agent_prompts.py | Checks evidence-planning guidance. |
| skills/change-analysis/SKILL.md | Defines manifest-first analysis and bounded evidence reads. |
| main.py | Allows the maintenance agent to use manifests. |
| github_app_mcp/tests/test_stdio.py | Updates expected tool discovery. |
| github_app_mcp/tests/test_server.py | Tests manifest schema and argument forwarding. |
| github_app_mcp/tests/test_github.py | Tests projection, pagination, validation, and anonymous fallback. |
| github_app_mcp/src/issuelens_github_mcp/server.py | Exposes the manifest tool. |
| github_app_mcp/src/issuelens_github_mcp/github.py | Implements manifest retrieval and metadata projection. |
| github_app_mcp/README.md | Documents manifest usage and limits. |
| docs/guide.md | Explains planned evidence retrieval. |
| agents/triage.md | Adopts manifest-first investigation. |
| agents/team-memory.md | Requires evidence planning for maintenance. |
| agents/plan.md | Updates planning evidence guidance. |
| agents/issuelens.md | Describes the revised analysis workflow. |
| .github/copilot-instructions.md | Updates architecture documentation. |
| .github/actions/issuelens/issuelens_action.py | Generates manifest-first batch requests. |
Review effort: Balanced
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Changyong Gong (chagong)
deleted the
chagong-issuelens-telemetry-errors
branch
October 19, 2026 08:20
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.

Summary
Why
The #57 team-memory run forced 23 separate
list_pull_request_files(per_page=1)model turns. Repeatedly replaying the growing conversation drove the 125k TPM deployment into likely throttling. For the same PR, the new manifest returns all 23 file entries in one approximately 13 KB model payload rather than 23 patch-bearing discovery turns.This follows GitHub Copilot code review's published large-PR pattern: make an explicit plan, explore relevant repository files with tools, and capture findings while reading rather than deferring all reasoning until the end.
Validation
git diff --checkpassed