Conversation
repo-map and git-map consolidated into repo-intel (agent-sh/repo-intel).
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request systematically renames the Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here. Footnotes
|
There was a problem hiding this comment.
Code Review
This pull request refactors repo-map to repo-intel, updating the plugin and skill definitions. The changes are mostly straightforward, but I've identified a couple of areas for improvement regarding consistency and architectural clarity. Specifically, the new description of repo-intel introduces a potential functional overlap with sync-docs that should be clarified. Additionally, there's a minor inconsistency between the repo-intel plugin and skill descriptions. Please see my detailed comments.
| "name": "repo-intel", | ||
| "description": "Unified static analysis via agent-analyzer. Git history, AST symbols, project metadata, and doc-code sync in one cached artifact.", | ||
| "category": "infrastructure", | ||
| "agents": 1, | ||
| "skills": 1, | ||
| "commands": 1, | ||
| "repo": "https://github.com/agent-sh/repo-map", | ||
| "install": "agentsys install repo-map", | ||
| "repo": "https://github.com/agent-sh/repo-intel", | ||
| "install": "agentsys install repo-intel", |
There was a problem hiding this comment.
The new description for repo-intel mentions 'doc-code sync'. This seems to create a functional overlap with the sync-docs plugin. To clarify the architecture and ensure correct dependency tracking, could you elaborate on the relationship? If repo-intel provides data for sync-docs, it might be beneficial for sync-docs to list repo-intel as a dependency.
| { "name": "deslop", "plugin": "deslop", "description": "Detect and clean AI-generated slop patterns in code", "platforms": ["Claude Code", "OpenCode", "Codex"] }, | ||
| { "name": "drift-analysis", "plugin": "drift-detect", "description": "Compare documented plans against actual implementation", "platforms": ["Claude Code", "OpenCode", "Codex"] }, | ||
| { "name": "repo-mapping", "plugin": "repo-map", "description": "Build AST-based symbol and import map of codebase", "platforms": ["Claude Code", "OpenCode", "Codex"] }, | ||
| { "name": "repo-intel", "plugin": "repo-intel", "description": "Unified static analysis - git history, AST symbols, project metadata", "platforms": ["Claude Code", "OpenCode", "Codex"] }, |
There was a problem hiding this comment.
The description for the repo-intel skill doesn't mention 'doc-code sync', which is part of the repo-intel plugin's description. For consistency and clarity, I suggest aligning the skill's description with the plugin's full capabilities.
| { "name": "repo-intel", "plugin": "repo-intel", "description": "Unified static analysis - git history, AST symbols, project metadata", "platforms": ["Claude Code", "OpenCode", "Codex"] }, | |
| { "name": "repo-intel", "plugin": "repo-intel", "description": "Unified static analysis - git history, AST symbols, project metadata, and doc-code sync", "platforms": ["Claude Code", "OpenCode", "Codex"] }, |
Summary
Test Plan