Build and Test Go Project / build (windows-latest) (push) Has been cancelled
Build and Test Go Project / build (macos-latest) (push) Has been cancelled
Build and Test Go Project / build (ubuntu-latest) (push) Has been cancelled
CodeQL / Analyze (go) (push) Has been cancelled
License Check / license-check (push) Has been cancelled
CodeQL / Analyze (actions) (push) Has been cancelled
- Added ResetInstance() function to lockdown package for test cleanup
- Updated lockdown_test.go to reset singleton before and after each test
- Updated issues_test.go to initialize cache once per test function (4 functions)
- Updated pullrequests_test.go to initialize cache once per test function (7 functions)
- All tests now explicitly reset and initialize cache at the beginning of each test function
- This prevents race conditions and ensures each test has proper cache state
Co-authored-by: JoannaaKL <67866556+JoannaaKL@users.noreply.github.com>
* Initial plan
* Replace custom cache with cache2go library
- Added github.com/muesli/cache2go dependency
- Replaced custom map-based cache with cache2go.CacheTable
- Removed manual timer management (scheduleExpiry, ensureEntry methods)
- Removed timer field from repoAccessCacheEntry struct
- Updated GetRepoAccessInfo to use cache2go's Value() and Add() methods
- Updated SetTTL to flush and re-add entries with new TTL
- Used unique cache names per instance to avoid test interference
- All existing tests pass with the new implementation
Co-authored-by: JoannaaKL <67866556+JoannaaKL@users.noreply.github.com>
* Final verification complete
Co-authored-by: JoannaaKL <67866556+JoannaaKL@users.noreply.github.com>
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: JoannaaKL <67866556+JoannaaKL@users.noreply.github.com>
Build and Test Go Project / build (macos-latest) (push) Has been cancelled
Build and Test Go Project / build (ubuntu-latest) (push) Has been cancelled
Build and Test Go Project / build (windows-latest) (push) Has been cancelled
CodeQL / Analyze (go) (push) Has been cancelled
CodeQL / Analyze (actions) (push) Has been cancelled
Docker / build (push) Has been cancelled
GoReleaser Release / release (push) Has been cancelled
License Check / license-check (push) Has been cancelled
Publish to MCP Registry / publish (push) Has been cancelled
* Add state metadata fields to get_discussion tool
Fixes#1303
The get_discussion tool was missing important state metadata that's
already available in get_issue. Added four fields to provide complete
discussion status information:
- state: Current discussion state (OPEN/CLOSED)
- isAnswered: Whether the discussion has an accepted answer
- answeredAt: Timestamp when answer was provided
- answerChosenAt: Timestamp when answer was selected
Changed GetDiscussion to return a map instead of github.Discussion
struct since the go-github library doesn't include all these fields
in its type definition. This approach is consistent with other
functions in this codebase (ListDiscussions, GetDiscussionComments).
All tests pass and linter checks pass.
* Fix Discussion field mappings based on GitHub GraphQL API
Changes:
- Replace 'State' (doesn't exist) with 'Closed' (Boolean)
- Remove 'AnsweredAt' (doesn't exist)
- Keep 'IsAnswered' (verified to exist in GitHub GraphQL API)
- Use 'AnswerChosenAt' for answer timestamp
Updated both implementation and tests to match actual GitHub GraphQL schema.
All tests passing.
---------
Co-authored-by: tommaso-moro <tommaso-moro@github.com>
* add repo nav tool
* comment responses
* move into git.go
* fix documentation
* Update pkg/github/git.go
Co-authored-by: Adam Holt <omgitsads@github.com>
* Fix undefined variable error in GetRepositoryTree
* Update git.go to use github.com/google/go-github/v77 for consistency with main branch
---------
Co-authored-by: Adam Holt <omgitsads@github.com>
Build and Test Go Project / build (ubuntu-latest) (push) Has been cancelled
Build and Test Go Project / build (windows-latest) (push) Has been cancelled
CodeQL / Analyze (go) (push) Has been cancelled
CodeQL / Analyze (actions) (push) Has been cancelled
Docker / build (push) Has been cancelled
Build and Test Go Project / build (macos-latest) (push) Has been cancelled
GoReleaser Release / release (push) Has been cancelled
License Check / license-check (push) Has been cancelled
Publish to MCP Registry / publish (push) Has been cancelled
* licences update from required CI build
* fixes licences
* implements new helpers to support google/go-github v77 API
* upgrades toolset to leverage google/go-github v77 with the exception of Update and Delete items
* refactor string conversion helpers
* reverts migration google/go-github GetProjectItem due to bug with the underlying library implementation
* additional refactoring for server helpers based on recent updates to google/go-github
* test updates based on underlying lib requirements
* resolves licences conflicts with script/licenses
* cleanup
* reduce change diff
* updates helper docs to reflect to methods
* upgrades delete projects item to google/go-github
* returns error from parsing string to int64
* improved OptionalBigIntArrayParam doc
* improves implementation for RequiredBigInt
* improves documentation for temporary fieldSelectionOptions struct
* updates github-mcp-server to go-github/v76
* updates license files
* refactors ListProjects to make use of
google/go-github ProjectsService
* refactors GetProject to make use of google/go-github ProjectsService
* declaring commit without a pointer
* cleanup additional commit pointer
---------
Co-authored-by: Tommaso Moro <37270480+tommaso-moro@users.noreply.github.com>
* Add instruction generation for enabled toolsets and corresponding tests
* Refactor instruction generation to always include base and context management instructions
* Refactor base instruction for clarity and adjust context management instruction formatting
* Simplify changes for now
* Remove unused toolset instructions and simplify test cases for clarity
* Add test cases for issues, notifications, and discussions toolsets in instruction generation
* Update base instruction and test expectations for clarity on tool selection and context management
* Add support for disabling instructions via environment variable
* Clarify PR review workflow instruction for consistency
* Apply suggestions from code review
Co-authored-by: Ksenia Bobrova <almaleksia@github.com>
* Refactor instruction generation and testing for clarity and consistency
---------
Co-authored-by: Ksenia Bobrova <almaleksia@github.com>