STACKITGit/cli
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions
11279 commits 87 branches 0 tags 160 MiB
Commit graph

11279 commits

This branch
This branch
All branches
Author SHA1 Message Date
Kynan Ware
5d8b4265c4
Merge pull request #11881 from cli/dependabot/github_actions/github/codeql-action-4 
chore(deps): bump github/codeql-action from 3 to 4
 2025-10-17 11:55:50 -06:00
Kynan Ware
fd651e9adc
Update .github/workflows/govulncheck.yml 2025-10-17 11:29:37 -06:00
Brian DeHamer
937f6d8c2e
improve docstring for release-create 
Signed-off-by: Brian DeHamer <bdehamer@github.com>
 2025-10-16 11:54:05 -07:00
Babak K. Shandiz
d2535897fd
Merge pull request #11922 from cli/babakks/fix-login-through-unix-socket 
Fix `auth login` and `auth refresh` to use UNIX socket
 2025-10-15 14:51:07 +01:00
cli automation
6f03eca5bc Bump Go to 1.25.3 2025-10-14 03:20:43 +00:00
Babak K. Shandiz
0ce728f1ae
chore: run go mod tidy 
Signed-off-by: Babak K. Shandiz <babakks@github.com>
 2025-10-13 20:07:41 +01:00
Babak K. Shandiz
5685b9a443
refactor(auth refresh): use PlainHttpClient instead of zero http.Client 
Signed-off-by: Babak K. Shandiz <babakks@github.com>
 2025-10-13 16:51:13 +01:00
Babak K. Shandiz
365ca1a901
refactor(auth login): use PlainHttpClient for OAuth flow 
Signed-off-by: Babak K. Shandiz <babakks@github.com>
 2025-10-13 16:51:11 +01:00
Babak K. Shandiz
ae9a7ed542
refactor(authflow): receive HTTP client via args 
Signed-off-by: Babak K. Shandiz <babakks@github.com>
 2025-10-13 16:48:10 +01:00
Babak K. Shandiz
2794f7b8d6
fix(factory): add PlainHttpClient to factory 
Signed-off-by: Babak K. Shandiz <babakks@github.com>
 2025-10-13 16:42:18 +01:00
Babak K. Shandiz
55fbad3e49
test(api): assert SkipDefaultHeaders is honoured 
Signed-off-by: Babak K. Shandiz <babakks@github.com>
 2025-10-13 16:37:32 +01:00
Babak K. Shandiz
b81c2495d2
fix(api): expose SkipDefaultHeaders option 
Signed-off-by: Babak K. Shandiz <babakks@github.com>
 2025-10-13 16:37:04 +01:00
Babak K. Shandiz
b9e04ef83d
test(api): improve NewHTTPClient test assertions 
The assertions should check for header values instead of single string,
because when getting a single header value, we get an empty string if
the header is not set or is set to an empty string. So, to make sure a
header is not set we should check the `Values` slice which is `nil` only
if the header is missing.

Signed-off-by: Babak K. Shandiz <babakks@github.com>
 2025-10-13 16:33:03 +01:00
Kynan Ware
5eb6549dc7
Merge pull request #11896 from cli/kw/fix-agent-task-targets-fork 
`gh agent-task create`: Fix targetting upstream instead of default repo
 2025-10-10 11:12:06 -06:00
Kynan Ware
8840df2eb3 Fix agentTaskCmd to use repoResolvingCmdFactory 
`agent-task` uses smart base repo func to fix bug with resolving upstream instead of local repo.
 2025-10-10 10:39:52 -06:00
dependabot[bot]
af0905efeb
chore(deps): bump github/codeql-action from 3 to 4 
Bumps [github/codeql-action](https://github.com/github/codeql-action) from 3 to 4.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Commits](https://github.com/github/codeql-action/compare/v3...v4)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-version: '4'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
 2025-10-08 14:03:20 +00:00
Babak K. Shandiz
c8152ed44e
Merge pull request #11879 from cli/kw/fix-progress-indicator-for-create-follow 
`gh agent-task create`: Fix `--follow` not killing the progress indicator
 2025-10-08 10:55:53 +02:00
Kynan Ware
91c6bc609a Add new displaying message to test expectation 
Updated the Test_createRun test to expect a message indicating that session logs are being displayed for the job
 2025-10-07 23:37:43 -06:00
Kynan Ware
a78bb5e899 Fix --follow not killing the progress indicator 
Fixes --follow not stopping the progress indicator. Also includes a nice message to indicate what is happening because even after we create the agent task, there's a period of time where we poll and receive nothing as the task session starts. We want there to be some sort of feedback in that period of time to not make the user panic and think it has hanged.
 2025-10-07 23:19:45 -06:00
Lucas Melin
ef7ac8caab
Address remaining PR comments for revert implementation 2025-10-06 13:27:45 -04:00
Lucas Melin
4f10a525bc
Adjust PR revert based on new acceptance criteria 
Also adds test cases for the new acceptance criteria.
 2025-10-06 13:27:45 -04:00
Lucas Melin
b229402cab
Return nil when err is nil 2025-10-06 13:27:45 -04:00
Lucas Melin
5368f409a7
Fix revert unit tests by using renamed helper 2025-10-06 13:27:45 -04:00
Lucas Melin
023a639c8c
feat: include revert PR info in message output 2025-10-06 13:25:31 -04:00
Lucas Melin
9467d66b11
Undo autoformat changes 2025-10-06 13:25:31 -04:00
Lucas Melin
60b6a5eed5
feat: implement pr revert 2025-10-06 13:25:31 -04:00
Kynan Ware
67bf27bf0c
Merge pull request #11835 from cli/kw/do-not-request-org-teams-for-reviewer-set 
`gh pr edit`: Only fetch org teams for reviewers when required
 2025-10-03 12:25:55 -06:00
Kynan Ware
43e834a691
Apply suggestion from @babakks 
Co-authored-by: Babak K. Shandiz <babakks@github.com>
 2025-10-02 09:55:58 -06:00
Kynan Ware
8d85946098
Apply suggestion from @babakks 
Co-authored-by: Babak K. Shandiz <babakks@github.com>
 2025-10-02 09:55:45 -06:00
Kynan Ware
bf728893fa Fix argument order in httpStubs test functions 
Swaps the argument order of the httpStubs functions in edit_test.go to match the expected (t *testing.T, reg *httpmock.Registry) signature. This improves consistency and prevents potential confusion or errors when calling these test helpers.
 2025-10-02 09:54:01 -06:00
Babak K. Shandiz
32a9c4ba26
Merge pull request #11838 from luxass/handle-correct-count-for-cache-delete 
fix(cache delete): report correct deleted count for key and key+ref deletions
 2025-10-02 11:45:39 +01:00
Babak K. Shandiz
ab9c99ec04
docs(cache delete): remove redundant comment 
Signed-off-by: Babak K. Shandiz <babakks@github.com>
 2025-10-02 11:36:00 +01:00
Babak K. Shandiz
d574873f3b
docs(cache delete): add godoc for deleteCacheByKey 
Signed-off-by: Babak K. Shandiz <babakks@github.com>
 2025-10-02 11:33:26 +01:00
Kynan Ware
66fae72872 Remove default empty slices in RemovePullRequestReviews 
Eliminates unnecessary initialization of users and teams to empty slices in RemovePullRequestReviews. Also updates the request body struct to use 'omitempty' for reviewers and team_reviewers, ensuring empty fields are omitted from the JSON payload.
 2025-10-01 16:12:06 -06:00
Kynan Ware
52bb1dec30 Fix typo in error message for required flags 
Corrected '--tile' to '--title' in the error message shown when required flags are missing in non-interactive mode.
 2025-10-01 16:09:33 -06:00
Kynan Ware
848faf8115 Refactor reviewer partitioning in PR edit command 
Extracted logic for splitting reviewer identifiers into users and teams into a new helper function, partitionUsersAndTeams. Updated updatePullRequestReviews to use this function for both adding and removing reviewers, improving code clarity and maintainability. Also clarified comments regarding PR author handling.
 2025-10-01 16:02:05 -06:00
Kynan Ware
57fce1dc3a Escape repo owner and name in PR reviewer API paths 
Updated AddPullRequestReviews and RemovePullRequestReviews to use url.PathEscape for repo owner and name in API paths. This ensures correct handling of special characters in repository identifiers.
 2025-10-01 15:55:24 -06:00
Kynan Ware
7094a55eec Remove unused ghIds function and githubv4 import 
Deleted the unused ghIds helper function and the associated githubv4 import from edit.go to clean up the codebase.
 2025-10-01 15:47:39 -06:00
Lucas
ccfc2c3045
fix(cache): report correct deleted count for key and key+ref deletions 
Previously, deleting caches by key or key+ref could misreport the number of deleted entries, especially when multiple caches matched the criteria. This change ensures the actual number of deleted caches is reported by consuming the API response's total_count field.
 2025-10-01 18:44:24 +02:00
Kynan Ware
87468f40db Refactor PR reviewer editing to use REST API and optimize team fetch 
Switches pull request reviewer add/remove operations from GraphQL to the REST API, enabling separate add and remove calls for reviewers and teams. Refactors reviewer editing logic to avoid fetching organization teams unless required for interactive editing, improving performance for non-interactive flows. Updates tests and supporting code to reflect the new reviewer management and metadata fetching behavior.
 2025-09-30 10:45:44 -06:00
Babak K. Shandiz
52ba836605
Merge pull request #11805 from juejinyuxitu/FieldsFuncSeq 
refactor: use strings.FieldsFuncSeq to reduce memory allocations
 2025-09-30 11:18:57 +01:00
dependabot[bot]
0e904d5ef5
chore(deps): Bump github.com/sigstore/sigstore-go from 1.1.0 to 1.1.3 
Bumps [github.com/sigstore/sigstore-go](https://github.com/sigstore/sigstore-go) from 1.1.0 to 1.1.3.
- [Release notes](https://github.com/sigstore/sigstore-go/releases)
- [Commits](https://github.com/sigstore/sigstore-go/compare/v1.1.0...v1.1.3)

---
updated-dependencies:
- dependency-name: github.com/sigstore/sigstore-go
  dependency-version: 1.1.3
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
 2025-09-29 18:13:42 +00:00
Eugene
a9efc8d3d2
Merge pull request #11801 from cli/eugene/remove_hidden_for_release_verify 
visible for  `release verify`  and `release verify-asset` cmd
 2025-09-29 10:43:09 -04:00
Kynan Ware
b76bc7706a
Merge pull request #11544 from benjlevesque/auth-status-json-output-8635 
Add JSON output to `gh auth status`
 2025-09-25 09:50:46 -06:00
Lucas Nørgård
8e963a8c14
Merge branch 'trunk' into gist-edit-large-file 2025-09-25 15:47:20 +02:00
Babak K. Shandiz
38d6a83e35
test(auth status): correctly replace JSON-escaped paths 
Signed-off-by: Babak K. Shandiz <babakks@github.com>
 2025-09-25 10:51:47 +01:00
ejahnGithub
a2034545cc update the description 2025-09-24 14:04:28 -04:00
juejinyuxitu
f5cf156af1 refactor: use strings.FieldsFuncSeq to reduce memory allocations 
Signed-off-by: juejinyuxitu <juejinyuxitu@outlook.com>
 2025-09-24 23:11:17 +08:00
Melissa Xie
07ccec116d
Merge pull request #11804 from cli/babakks/add-co-alias-to-pr-checkout 
Add alias `co` for `pr checkout`
 2025-09-24 11:08:42 -04:00
Babak K. Shandiz
0ec5f13283
fix(pr checkout): add alias co 
Signed-off-by: Babak K. Shandiz <babakks@github.com>
 2025-09-24 14:56:59 +01:00