Skip to content

feat: list_discussions sort by updatedAt & createdAt, return updatedAt and author #690

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 22 commits into
base: main
Choose a base branch
from
+439 −153
Open

feat: list_discussions sort by updatedAt & createdAt, return updatedAt and author #690

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
687ee8f
added updatedAt and Author (aka User) login to query and payload
tommaso-moro Jul 14, 2025
dbbf724
added initial support for orderby and direction
tommaso-moro Jul 15, 2025
34a413d
sort by created at instead of updated at by default
tommaso-moro Jul 15, 2025
5d7230d
remove unused code
tommaso-moro Jul 15, 2025
c1afb88
refactor to map to most suitable query based on user inputs at runtime
tommaso-moro Jul 16, 2025
7eeb87c
updated readme with new description
tommaso-moro Jul 17, 2025
44f8f35
restore original categoryID code, simplify vars management
tommaso-moro Jul 17, 2025
bcb82c8
quick fix
tommaso-moro Jul 17, 2025
751dfa5
update tests to account for recent changes (author login, updated at …
tommaso-moro Jul 17, 2025
b064f7a
use switch statement for better readability
tommaso-moro Jul 17, 2025
6ab5137
remove comment
tommaso-moro Jul 17, 2025
25d39be
linting
tommaso-moro Jul 17, 2025
14dcf32
refactored logic, simplified switch statement
tommaso-moro Jul 17, 2025
49d0dea
linting
tommaso-moro Jul 17, 2025
eb78fe9
use original queries from discussions list tool for testing
tommaso-moro Jul 17, 2025
d499668
linting
tommaso-moro Jul 17, 2025
c9a0572
Merge branch 'main' into tommy/expand-discussions-tools
tommaso-moro Jul 17, 2025
de8583c
remove logging
tommaso-moro Jul 17, 2025
2783724
Merge branch 'tommy/expand-discussions-tools' of https://github.com/t…
tommaso-moro Jul 17, 2025
9f38405
Merge remote-tracking branch 'origin/main' into tommy/expand-discussi…
LuluBeatson Jul 21, 2025
9d1a665
Complete merge by re-introducing pagination to ListDiscussions
LuluBeatson Jul 21, 2025
6bea783
fix unit tests
LuluBeatson Jul 21, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -462,6 +462,8 @@ The following sets of tools are available (all are on by default):
- **list_discussions** - List discussions
- `after`: Cursor for pagination. Use the endCursor from the previous page's PageInfo for GraphQL APIs. (string, optional)
- `category`: Optional filter by discussion category ID. If provided, only discussions with this category are listed. (string, optional)
- `direction`: Order direction. (string, optional)
- `orderBy`: Order discussions by field. If provided, the 'direction' also needs to be provided. (string, optional)
- `owner`: Repository owner (string, required)
- `perPage`: Results per page for pagination (min 1, max 100) (number, optional)
- `repo`: Repository name (string, required)
Expand Down
Loading
Loading