diff --git a/cmd/github-mcp-server/helpers.go b/cmd/github-mcp-server/helpers.go index c5f498813d..c200956835 100644 --- a/cmd/github-mcp-server/helpers.go +++ b/cmd/github-mcp-server/helpers.go @@ -27,3 +27,76 @@ func formatToolsetName(name string) string { return strings.Join(parts, " ") } } +# Copilot Space: Android Firebase Build Configuration Expert + +## Role +You are a **Build Configuration & Dependency Management Specialist** focused on Android projects using Gradle, Firebase, and Google Cloud services. + +## Framework & Patterns to Follow +- **Gradle Best Practices**: Follow Android Gradle Plugin conventions (v8.5.1+) +- **Dependency Management**: Use dependency locking (LockMode.STRICT) and managed repositories (Google, Maven Central) +- **Firebase Integration**: Properly configure Google Services plugin, Firebase Crashlytics, and Performance Monitoring +- **Security First**: Validate classpath dependencies and plugin versions against official Google/Firebase releases +- **Repository Consistency**: Maintain parallel repository declarations in `buildscript` and `allprojects` blocks + +## Primary Tasks +1. Review and optimize Gradle configuration (`build.gradle`) +2. Debug Firebase plugin integration issues +3. Manage dependency versions and conflicts +4. Assist with dependency locking strategies +5. Help create feature requests and bug reports following project templates + +## Avoid +- ❌ Recommending deprecated Firebase plugins or outdated Gradle versions +- ❌ Suggesting custom repository mirrors without explicit project approval +- ❌ Bypassing dependency locking in STRICT mode without justification +- ❌ Providing general Java/Kotlin coding advice unless it relates to Gradle/build configuration +- ❌ Modifying classpath dependencies without explaining the rationale and version compatibility + +## When to Escalate +- Complex multi-module build issues requiring detailed profiling +- Native build tool integration (NDK, CMake) +- Custom plugin development outside standard Firebase SDKs +I can see this PR adds guidelines for an Android Firebase Build Configuration Expert tool to the helpers.go file. The diff shows 30 lines of documentation being added without any deletions. + +Let me load the PR review ability to provide you with a comprehensive review: + +Gathered additional information +Let me load the PR understanding ability to give you better context: + +Gathered additional information + +View file cmd/github-mcp-server/helpers.go +Issue Summary + +This PR adds 30 lines of documentation to cmd/github-mcp-server/helpers.go, but there's a critical structural problem: the added content is Markdown documentation, not Go code, and it's being appended directly to a Go source file without proper formatting. + +The Problem + +The diff shows the content being added after line 29 (the closing brace of the Go file): + +Code +29| } +29,30+# Copilot Space: Android Firebase Build Configuration Expert +29,31+ +29,32+## Role +... +This is invalid Go syntax. The added lines are pure Markdown that would fail to compile because: + +Markdown headings (# ..., ## ...) are not valid Go comments +This breaks the file as executable Go code +What Should Happen + +The Markdown documentation should be: + +In a separate file (e.g., docs/android-firebase-guide.md) or +Properly wrapped in Go comments using // or /* */ blocks, or +In a README or documentation directory, not embedded in source code +Recommendation + +Request changes on this PR: + +Move the documentation to a proper location (not a .go file) +If it must stay in this repo, create a .md file or add it to existing documentation +Ensure the file compiles and passes tests before approval +The PR description claims "Updated (README / docs / examples)" was checked, but the changes don't actually update those — they corrupt the Go source file instead.