Skip to content

Conversation

@yurinovo18
Copy link
Contributor

@yurinovo18 yurinovo18 commented Feb 9, 2026

Summary

  • Add version-update-sources CLI command to add sources to existing draft application versions (APP-1675).
  • Refactor source and filter parsing into a shared version_source_parser used by both version-create and version-update-sources.
  • Add e2e coverage: generic artifact upload helper, draft version creation, and update-sources flow with package + artifact sources.

API Endpoint: PATCH /v1/applications/{application_key}/versions/{version} (query params: async, dry_run, fail_fast)

Changes

CLI / Commands

  • New command version-update-sources (vus): adds sources and optional filters to a draft version via --source-type-artifacts, --source-type-packages, --source-type-builds, --source-type-release-bundles, --source-type-application-versions, or --spec; supports --sync, --dry-run, --fail-fast (apptrust/commands/version/update_app_version_sources_cmd.go).
  • Flags: New VersionUpdateSources command constant and FailFastFlag; version-update-sources registered in cli/cli.go and apptrust/commands/flags.go.
  • Refactor: create_app_version_cmd.go now delegates to buildSourcesAndFiltersFromContext() in apptrust/commands/version/version_source_parser.go; parsing for builds, release bundles, application versions, packages, artifacts, spec file, and filters moved into shared parser (create command simplified).

Test Plan

  • Unit tests for version-update-sources command (run + source flags).
  • Unit tests for UpdateAppVersionSources (sync, async, dry_run, fail_fast).
  • Existing create-version and parser tests updated and passing.
  • E2E: draft version create + update-sources with artifact; both package and artifact present in version content.
  • Manual: run jf at version-update-sources <app-key> <version> with source flags against a real draft version.

@yurinovo18 yurinovo18 force-pushed the feature/APP-1675-enhance-update-application-version-for-draft branch from e18efd2 to cac5451 Compare February 9, 2026 10:09
@yurinovo18 yurinovo18 force-pushed the feature/APP-1675-enhance-update-application-version-for-draft branch from cac5451 to e9bb080 Compare February 9, 2026 11:18
@yurinovo18 yurinovo18 force-pushed the feature/APP-1675-enhance-update-application-version-for-draft branch from e9bb080 to 66522b1 Compare February 10, 2026 07:48
@yurinovo18 yurinovo18 force-pushed the feature/APP-1675-enhance-update-application-version-for-draft branch from 66522b1 to 39c8cb6 Compare February 10, 2026 07:51
@yurinovo18 yurinovo18 force-pushed the feature/APP-1675-enhance-update-application-version-for-draft branch from 39c8cb6 to 5f0042e Compare February 10, 2026 08:27
@yurinovo18 yurinovo18 force-pushed the feature/APP-1675-enhance-update-application-version-for-draft branch from 5f0042e to cbb5517 Compare February 10, 2026 08:50
@yurinovo18 yurinovo18 force-pushed the feature/APP-1675-enhance-update-application-version-for-draft branch from cbb5517 to 16ba99e Compare February 10, 2026 11:37
@yurinovo18 yurinovo18 force-pushed the feature/APP-1675-enhance-update-application-version-for-draft branch from 16ba99e to 2ec6391 Compare February 10, 2026 12:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant