Skip to content

fix(vscode): retry marketplace publishing - #13802

Merged
marius-kilocode merged 2 commits into
mainfrom
deep-people
Sep 4, 2026
Merged

fix(vscode): retry marketplace publishing#13802
marius-kilocode merged 2 commits into
mainfrom
deep-people

Conversation

@marius-kilocode

@marius-kilocode marius-kilocode commented Sep 4, 2026

Copy link
Copy Markdown
Collaborator

What Problem This Solves

Two consecutive releases stopped on VS Code Marketplace request timeouts. Some platforms were already published, while later platforms and the GitHub VSIX uploads were not reached.

Why This Change Was Made

Use the existing retry helper for Marketplace publishing: three total attempts, waiting 30 seconds before the second attempt and 60 seconds before the third. Add --skip-duplicate to both VSCE and Open VSX so a retry can continue if an upload succeeded but its response was lost.

Each attempt reuses the original VSIX. VSCE 3.9.2 does not expose a publish timeout setting, so this change adds recovery rather than changing the GitHub job timeout.

User Impact

A single transient publish failure no longer immediately stops the release. Duplicate checks remain independent for each marketplace and platform: skipping an existing Marketplace upload still proceeds to Open VSX. Persistent errors still fail after three attempts.

Evidence

  • v7.5.12 failure: Windows ARM64 Marketplace request timed out after about 181 seconds.
  • v7.5.13 failure: Alpine ARM64 timed out while the larger Alpine x64 package published in about 15 seconds. Aggregate VSIX artifact size was only 0.0015% larger than the successful v7.5.11 release.
  • Passed extension compilation, package lint and typechecks, direct script/test lint and typechecks, formatting, and two focused command-wiring regression tests.
  • Six offline smoke scenarios exercised the actual publish script with substitute executables: stable and prerelease success, recovery on the third attempt for either marketplace, and stopping after three failures for either marketplace. No live publication was performed.

After merge, validate with a new patch release from main, checking all eight targets on both marketplaces and the GitHub VSIX assets. Rerunning an old failed job does not include this fix.

@kilo-code-bot

kilo-code-bot Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)
  • packages/kilo-vscode/script/publish.ts
  • packages/kilo-vscode/tests/unit/publish.test.ts
Previous Review Summary (commit c2f1b43)

Current summary above is authoritative. Previous snapshots are kept for context only.

Previous review (commit c2f1b43)

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)
  • packages/kilo-vscode/script/publish.ts
  • packages/kilo-vscode/tests/unit/publish.test.ts

Reviewed by grok-4.6 · Input: 37.6K · Output: 4.6K · Cached: 131.8K

Review guidance: REVIEW.md from base branch main

Comment thread packages/kilo-vscode/script/publish.ts Outdated
@marius-kilocode
marius-kilocode enabled auto-merge (squash) September 4, 2026 15:53
@marius-kilocode
marius-kilocode merged commit a486831 into main Sep 4, 2026
23 checks passed
@marius-kilocode
marius-kilocode deleted the deep-people branch September 4, 2026 15:56
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