Skip to content

Add --format flag support to rt commands - #3456

Merged
ehl-jf merged 3 commits into
masterfrom
JGC-478-format-flag
May 12, 2026
Merged

ehl-jf merged 3 commits into
masterfrom
JGC-478-format-flag

Conversation

@ehl-jf

@ehl-jf ehl-jf commented Apr 28, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Adds --format flag support to several rt commands so their output can be machine-consumed (JSON) and, where useful, rendered as a summary table. Behavior without --format is unchanged.

Commands now accepting --format:

Command Accepted values
rt permission-target-create (ptc) json
rt permission-target-update (ptu) json
rt users-create (uc) json
rt transfer-config json
rt transfer-config-merge json, table
rt transfer-files json, table

Implementation notes:

  • permission-target-create / permission-target-update: switched from TemplateConsumer flag set to dedicated PermissionTargetCreate / PermissionTargetUpdate keys so the new --format flag is actually wired in (fix in commit 946c0320).
  • For commands whose underlying client layer discards the response body, a synthetic {"status_code":200,"message":"OK"} payload is emitted via cliutils.FormatHTTPResponseJSON on success.
  • transfer-config-merge and transfer-files get richer structured output: a transferConfigMergeResult / transferFilesResult struct rendered as either JSON or a tabwriter summary table.
  • New flag entries (permission-target-create-format, permission-target-update-format, transfer-config-format, transfer-config-merge-format, transfer-files-format, users-create-format) registered in utils/cliutils/commandsflags.go.

Tests:

  • artifactory/cli_test.go: +509 lines covering format parsing, JSON/table rendering, error propagation, and unsupported-format handling for the affected commands.

Dependencies:

  • Bumps jfrog-cli-artifactory to 0.8.1-0.20260512080921-ec417dc1b7ee.
  • Bumps jfrog-client-go to 1.55.1-0.20260508101905-a17af78a38d7.

Files changed: artifactory/cli.go, artifactory/cli_test.go, utils/cliutils/commandsflags.go, go.mod, go.sum.


  • All tests have passed. If this feature is not already covered by the tests, new tests have been added.
  • The pull request is targeting the master branch.
  • The code has been validated to compile successfully by running go vet ./....
  • The code has been formatted properly using go fmt ./....

@ehl-jf
ehl-jf force-pushed the JGC-478-format-flag branch from 2d65caf to 6395942 Compare April 28, 2026 14:48
Comment thread go.mod Resolved
@ehl-jf
ehl-jf force-pushed the JGC-471-format-support branch 9 times, most recently from fe21cb8 to 5af9d26 Compare April 30, 2026 14:26
Base automatically changed from JGC-471-format-support to master April 30, 2026 15:10
@ehl-jf
ehl-jf force-pushed the JGC-478-format-flag branch from f8e42c5 to 0c88add Compare May 4, 2026 06:42
@github-actions

github-actions Bot commented May 4, 2026

Copy link
Copy Markdown
Contributor


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@ehl-jf ehl-jf added the safe to test Approve running integration tests on a pull request label May 4, 2026
@ehl-jf
ehl-jf force-pushed the JGC-478-format-flag branch from 0c88add to b75c5ac Compare May 4, 2026 07:17
@ehl-jf ehl-jf added safe to test Approve running integration tests on a pull request and removed safe to test Approve running integration tests on a pull request labels May 12, 2026
@ehl-jf
ehl-jf merged commit 35ede4e into master May 12, 2026
86 of 88 checks passed
@ehl-jf
ehl-jf deleted the JGC-478-format-flag branch May 12, 2026 09:26
social4hyq pushed a commit to social4hyq/homebrew-core that referenced this pull request Sep 20, 2026
jfrog-cli 2.104.1

Created-by: HarmonybrewBot
Commit-by: HarmonybrewBot
Merged-by: HarmonybrewBot
Description: Created by `brew bump`

---

Created with `brew bump-formula-pr`.<details>
  <summary>release notes</summary>
  <pre>## What changed in v2.104.1

> Auto-generated from 6 repositories · 2026-05-22  
> Comparing `v2.103.0` → `v2.104.1`

> ℹ️  No changes in: gofrog

### ✨ New Features

- **[build-info-go]** Enhance Yarn module to support workspace projects and non-registry de… [#379](jfrog/build-info-go#379) (@fluxxBot)
- **[jfrog-cli]** Add common --format flag support [#3454](jfrog/jfrog-cli#3454) (@ehl-jf)
- **[jfrog-cli]** Add --format flag support to rt commands [#3456](jfrog/jfrog-cli#3456) (@ehl-jf)
- **[jfrog-cli]** Add usage report for `jf api` [#3479](jfrog/jfrog-cli#3479) (@ehl-jf)
- **[jfrog-cli]** RTECO-1017 - Enrich User-Agent with agent/CI context and reuse Cursor… [#3468](jfrog/jfrog-cli#3468) (@fluxxBot)
- **[jfrog-cli]** RTECO-1035 - Added Support for yarn v4 [#3440](jfrog/jfrog-cli#3440) (@fluxxBot)
- **[jfrog-cli-artifactory]** Add UV package manager setup support [#437](jfrog/jfrog-cli-artifactory#437) (@sverdlov93)
- **[jfrog-cli-artifactory]** RTECO-1015 - Implement jf skills list command [#432](jfrog/jfrog-cli-artifactory#432) (@udaykb2)
- **[jfrog-cli-artifactory]** RTECO-1016 - Implement skills update command [#434](jfrog/jfrog-cli-artifactory#434) (@udaykb2)
- **[jfrog-cli-artifactory]** RTECO-1007 - Collect build-info on jf skills publish [#441](jfrog/jfrog-cli-artifactory#441) (@udaykb2)
- **[jfrog-cli-artifactory]** Add --format flag support to artifactory commands [#435](jfrog/jfrog-cli-artifactory#435) (@ehl-jf)
- **[jfrog-cli-artifactory]** RTECO-1035 - Added Support for yarn v4 [#422](jfrog/jfrog-cli-artifactory#422) (@fluxxBot)
- **[jfrog-cli-core]** Add UV package manager support [#1549](jfrog/jfrog-cli-core#1549) (@sverdlov93)
- **[jfrog-cli-core]** Add common --format flag support [#1547](jfrog/jfrog-cli-core#1547) (@ehl-jf)
- **[jfrog-cli-core]** RTECO-1017 - Detect agent + CI invocation context, enrich metrics wit… [#1555](jfrog/jfrog-cli-core#1555) (@fluxxBot)
- **[jfrog-cli-security]** Add cache and parallel run to Git Count Contributors [#701](jfrog/jfrog-cli-security#701) (@attiasas)
- **[jfrog-client-go]** Add `SAST Fast Diff Mode` option to configuration profile [#1342](jfrog/jfrog-client-go#1342) (@attiasas)
- **[jfrog-client-go]** RTECO-1015 - Implement ListSkills method to fetch skills [#1341](jfrog/jfrog-client-go#1341) (@udaykb2)

### 🐛 Bug Fixes

- **[build-info-go]** RTECO-1265: fix(dotnet): preserve all per-TFM package versions in build-info [#381](jfrog/build-info-go#381) (@Edenb2131)
- **[build-info-go]** RTECO-1265 - Fixed Nuget bracket based dependency resolution [#385](jfrog/build-info-go#385) (@fluxxBot)
- **[jfrog-cli]** RTECO-992 - Fix uv build command build-info and update jfrog-cli-artifactory dependency [#3485](jfrog/jfrog-cli#3485) (@agrasth)
- **[jfrog-cli]** RTECO-1263 - Fixed Content Reader race panic [#3502](jfrog/jfrog-cli#3502) (@fluxxBot)
- **[jfrog-cli-artifactory]** Fix duplicate dependency entries in build-info for jf uv build and publish [#438](jfrog/jfrog-cli-artifactory#438) (@agrasth)
- **[jfrog-cli-artifactory]** RTECO-992 - Restore build-info for jf uv build, clear artifacts only [#449](jfrog/jfrog-cli-artifactory#449) (@agrasth)
- **[jfrog-cli-core]** JGC-471 - Fix npm output format parsing [#1551](jfrog/jfrog-cli-core#1551) (@ehl-jf)
- **[jfrog-cli-core]** Fix #delayed multiarch image list.manifest files are not getting migr… [#1552](jfrog/jfrog-cli-core#1552) (@amithins)
- **[jfrog-client-go]** Fix race in ContentReader causing 'send on closed channel' panic [#1347](jfrog/jfrog-client-go#1347) (@fluxxBot)

### 🔧 Internal Changes

- **[build-info-go]** Bump Maven extractor to 2.43.9 and Gradle4 extractor to 4.35.9 [#383](jfrog/build-info-go#383) (@agrasth)
- **[jfrog-cli-artifactory]** RTECO-1048 - Improve skills install command [#446](jfrog/jfrog-cli-artifactory#446) (@udaykb2)
- **[jfrog-cli-artifactory]** RTECO-1011 - Improve skill commands [#448](jfrog/jfrog-cli-artifactory#448) (@udaykb2)
- **[jfrog-cli-security]** Update AnalyzerManager version to 1.33.0 [#748](jfrog/jfrog-cli-security#748) (@attiasas)

### 📦 Other Changes

- **[jfrog-cli]** RTECO-1003 - Updated branch name for jfrog-cli [#3467](jfrog/jfrog-cli#3467) (@naveenku-jfrog)
- **[jfrog-cli]** Remove the `how` command [#3494](jfrog/jfrog-cli#3494) (@ehl-jf)
- **[jfrog-cli-artifactory]** Show list and update in jf skills help [#451](jfrog/jfrog-cli-artifactory#451) (@udaykb2)
- **[jfrog-client-go]** CVE-2025-11579 [#1289](jfrog/jfrog-client-go#1289) (@saya-entro)

---

### 📊 Summary

| Repository | Changes |
|---|---|
| [jfrog-cli-artifactory](https://github.com/jfrog/jfrog-cli-artifactory) | 11 |
| [jfrog-cli](https://github.com/jfrog/jfrog-cli) | 9 |
| [jfrog-cli-core](https://github.com/jfrog/jfrog-cli-core) | 5 |
| [build-info-go](https://github.com/jfrog/build-info-go) | 4 |
| [jfrog-client-go](https://github.com/jfrog/jfrog-client-go) | 4 |
| [jfrog-cli-security](https://github.com/jfrog/jfrog-cli-security) | 2 |
| **Total** | **35** |

### 🔗 Module Version Changes

| Module | From | To |
|---|---|---|
| [build-info-go](https://github.com/jfrog/build-info-go) | `v1.13.1-0.20260429070557-93b98034d295` | `v1.13.1-0.20260521104402-1e35b9b5b0c6` |
| [jfrog-cli-artifactory](https://github.com/jfrog/jfrog-cli-artifactory) | `v0.8.1-0.20260429074430-a5871f2898b5` | `v0.8.1-0.20260520131142-ef80a14dd499` |
| [jfrog-cli-core](https://github.com/jfrog/jfrog-cli-core) | `v2.60.1-0.20260429073430-5723b4f21705` | `v2.60.1-0.20260520130103-75be6a23c54b` |
| [jfrog-cli-security](https://github.com/jfrog/jfrog-cli-security) | `v1.28.0` | `v1.29.0` |
| [jfrog-client-go](https://github.com/jfrog/jfrog-client-go) | `v1.55.1-0.20260428070955-750b933dc5c7` | `v1.55.1-0.20260521115926-32f082854b39` |
</pre>
  <p>View the full release notes at <a href="https://github.com/jfrog/jfrog-cli/releases/tag/v2.104.1">https://github.com/jfrog/jfrog-cli/releases/tag/v2.104.1</a>.</p>
</details>
<hr>

See merge request: Harmonybrew/homebrew-core!5774

This branch had an error being deployed

1 failed deployment
frogbot — 4744b6f0 Deployed May 12, 2026 by ehl-jf via scan-pull-request #4207
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

safe to test Approve running integration tests on a pull request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants