APP-4230 - Add --skip-unassigned flag to version-create command by danielreJfrog · Pull Request #87 · jfrog/jfrog-cli-application

danielreJfrog · 2026-06-10T11:35:24Z

Summary

Adds the --skip-unassigned boolean flag to the version-create CLI command
When set, the version is automatically promoted to the first lifecycle stage if all source artifacts reside in repositories mapped to that stage; otherwise the version is left unassigned with a message explaining why
Adds the message key to the ordered output keys for version creation response

Test plan

Unit test: dedicated skip-unassigned flag test case added
Unit test: all flags test case updated to include --skip-unassigned

Made with Cursor

Co-authored-by: Cursor <cursoragent@cursor.com>

github-actions · 2026-06-10T11:35:35Z

All contributors have signed the CLA ✍️ ✅
_{Posted by the CLA Assistant Lite bot.}

danielreJfrog · 2026-06-11T12:27:31Z

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

github-actions · 2026-06-11T15:59:03Z

Test Results

55 tests +3 54 ✅ +3 6m 43s ⏱️ -14s
2 suites ±0 1 💤 ±0
1 files ±0 0 ❌ ±0

Results for commit 226f1dd. ± Comparison against base commit 55a0ab5.

♻️ This comment has been updated with latest results.

asafgabai · 2026-06-14T06:34:32Z

+		var response struct {
+			ApplicationKey string `json:"application_key"`
+			Version        string `json:"version"`
+			Status         string `json:"status"`
+			CurrentStage   string `json:"current_stage"`
+		}


It's very similar to the struct used in the next test case below. Consider merging them and use a shared struct in both test cases.

asafgabai · 2026-06-14T06:37:51Z

+		require.NoError(t, err, "failed to parse CLI output as JSON: %s", output)
+		assert.Equal(t, appKey, response.ApplicationKey)
+		assert.Equal(t, version, response.Version)
+		assert.NotEmpty(t, response.Message, "A message should explain why auto-promotion did not occur")


To make sure the returned message is the error we expect, let's also validate the message content.

asafgabai · 2026-06-14T06:48:05Z

+	t.Run("stays unassigned with message when artifact not in first stage", func(t *testing.T) {
+		version := utils.GenerateUniqueKey("skip-ua-fail")
+
+		baseURL := os.Getenv("JFROG_APPTRUST_CLI_TESTS_JFROG_URL")


Instead of creating a repo using the REST API directly, refer to artifactory_utils.go where we already have functions that do that.
There's also a function for uploading artifacts.

asafgabai · 2026-06-16T16:55:50Z

+	return repoKey
+}
+
+func UploadTestArtifact(t *testing.T, repoKey, fileName string) string {


Why creating this function instead of just making uploadSimpleFileToArtifactory public?

…ifact Remove the thin wrapper and rename the private function directly, as suggested in code review. Co-authored-by: Cursor <cursoragent@cursor.com>

APP-4230 - Add --skip-unassigned flag to version-create command

998e7c7

Co-authored-by: Cursor <cursoragent@cursor.com>

danielreJfrog requested a deployment to frogbot June 10, 2026 11:35 — with GitHub Actions Waiting

danielreJfrog requested a deployment to frogbot June 11, 2026 11:45 — with GitHub Actions Waiting

danielreJfrog force-pushed the feature/APP-4230-skip-unassigned-flag branch from 931d57d to 6d86597 Compare June 11, 2026 11:49

danielreJfrog requested a deployment to frogbot June 11, 2026 11:49 — with GitHub Actions Waiting

danielreJfrog force-pushed the feature/APP-4230-skip-unassigned-flag branch from 6d86597 to 85862c3 Compare June 11, 2026 11:53

danielreJfrog requested a deployment to frogbot June 11, 2026 11:53 — with GitHub Actions Waiting

danielreJfrog force-pushed the feature/APP-4230-skip-unassigned-flag branch from 85862c3 to e095614 Compare June 11, 2026 11:54

danielreJfrog requested a deployment to frogbot June 11, 2026 11:54 — with GitHub Actions Waiting

danielreJfrog force-pushed the feature/APP-4230-skip-unassigned-flag branch from e095614 to b7bb980 Compare June 11, 2026 11:58

danielreJfrog requested a deployment to frogbot June 11, 2026 11:58 — with GitHub Actions Waiting

danielreJfrog force-pushed the feature/APP-4230-skip-unassigned-flag branch from b7bb980 to 4495de1 Compare June 11, 2026 12:22

danielreJfrog requested a deployment to frogbot June 11, 2026 12:22 — with GitHub Actions Waiting

danielreJfrog force-pushed the feature/APP-4230-skip-unassigned-flag branch from 4495de1 to 61e226f Compare June 11, 2026 12:27

danielreJfrog requested a deployment to frogbot June 11, 2026 12:27 — with GitHub Actions Waiting

asafgabai added the safe to test Approve running end-to-end tests on a pull request label Jun 11, 2026

github-actions Bot removed the safe to test Approve running end-to-end tests on a pull request label Jun 11, 2026

asafgabai reviewed Jun 14, 2026

View reviewed changes

danielreJfrog force-pushed the feature/APP-4230-skip-unassigned-flag branch from 61e226f to 1c59070 Compare June 14, 2026 07:39

danielreJfrog requested a deployment to frogbot June 14, 2026 07:39 — with GitHub Actions Waiting

danielreJfrog force-pushed the feature/APP-4230-skip-unassigned-flag branch from 1c59070 to d9559c4 Compare June 14, 2026 07:41

danielreJfrog requested a deployment to frogbot June 14, 2026 07:41 — with GitHub Actions Waiting

danielreJfrog force-pushed the feature/APP-4230-skip-unassigned-flag branch from d9559c4 to 2a90a3b Compare June 14, 2026 08:13

danielreJfrog requested a deployment to frogbot June 14, 2026 08:13 — with GitHub Actions Waiting