Skip to content

[PLTFRM-2783] Fix sync progress polling query - #3043

Merged
rebeccahum merged 1 commit into
trunkfrom
fix/PLTFRM-2783-sync-progress
Sep 10, 2026
Merged

[PLTFRM-2783] Fix sync progress polling query#3043
rebeccahum merged 1 commit into
trunkfrom
fix/PLTFRM-2783-sync-progress

Conversation

@rebeccahum

Copy link
Copy Markdown
Contributor

Description

vip sync starts a data sync but then exits with Unknown argument "sync" on field "AppEnvironment.syncProgress" when polling for progress. Remove the obsolete argument and its query variable so polling works with the current API schema. Regenerate the operation types accordingly.

Changelog Description

Fixed

  • Fixed vip sync exiting with a GraphQL error while the data sync continues in the background.

Validation

  • Typecheck and focused ESLint passed in the development checkout.
  • A temporary regression test reproduced the original GraphQL validation error and passed with the fix; no test file is included in this PR.
  • git diff --check passed on the PR branch.
  • Live data sync has not been run.

Steps to Test

  1. Build the CLI with npm run build.
  2. On a disposable test application's non-production environment, run node dist/bin/vip-sync.js @example-app.develop and confirm the sync.
  3. Verify progress polling does not report the unknown sync argument error and continues through completion.

@github-actions

Copy link
Copy Markdown
Contributor

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@sonarqubecloud

Copy link
Copy Markdown

@rebeccahum
rebeccahum merged commit 9e7a6de into trunk Sep 10, 2026
27 checks passed
@rebeccahum
rebeccahum deleted the fix/PLTFRM-2783-sync-progress branch September 10, 2026 15:48
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.

2 participants