From 2b82b4a577e2e408c762efee811a85e13adf9bd4 Mon Sep 17 00:00:00 2001 From: Lenny Chen Date: Mon, 3 Aug 2026 11:31:33 -0700 Subject: [PATCH 1/2] fix(snipsync): drop duplicate origins and the dead samples-dotnet pin Snipsync has failed intermittently since 2026-07-20 and on every run since 2026-07-30. Two independent causes. Duplicate origins. features and reference-app-orders-go were each listed twice. Sync.js names every download `${repo}.zip` from the repo alone and fans all origins out through one Promise.all, and unzip() unlinks the archive when it finishes. Two tasks sharing a filename race: one reads a partially written or already deleted archive and yauzl throws Error: unexpected EOF at node_modules/yauzl/index.js:629:23 from an fs callback, which escapes getRepos()'s try/catch and kills the process. Hence the intermittency, and hence no "Failed downloading" line in the logs. The duplicates date to #3932 and #4247. Dead branch pin. #4663 pinned samples-dotnet to ea/aws-lambda on 07-30. That sample merged to main in temporalio/samples-dotnet#152 and the branch was deleted, so every run since has 404'd on the archive download and burned five retries. Unpinning is safe for the docs page: snipsync only splices IDs it finds in a source, and leaves unmatched markers untouched, so the .NET lambda page keeps its current content. samples-java stays pinned. temporalio/samples-java#784 is still open and lambda-worker/ exists only on that branch. The pin is collateral-free today: the branch's only divergence from main outside lambda-worker/ is .github/workflows/ci.yml, which carries no snippet markers. Verified by running yarn snipsync locally against this config: completes in 87s with no crash, 711/716 target files unchanged, and both lambda pages untouched. --- snipsync.config.yaml | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/snipsync.config.yaml b/snipsync.config.yaml index 6d20b6cdd7..b278d6b076 100644 --- a/snipsync.config.yaml +++ b/snipsync.config.yaml @@ -22,14 +22,12 @@ origins: repo: samples-ruby - owner: temporalio repo: samples-dotnet - ref: ea/aws-lambda + # Pinned until temporalio/samples-java#784 merges the lambda-worker sample to + # main. A ref here pins every snippet from this repo, not just one page's, so + # remove it as soon as that PR lands. - owner: temporalio repo: samples-java ref: ea/aws-lambda - - owner: temporalio - repo: features - - owner: temporalio - repo: reference-app-orders-go - files: pattern: './sample-apps/**/*.go' owner: 'temporalio' From 3bb497390ad4c47010f49f05dc4848c502539d50 Mon Sep 17 00:00:00 2001 From: Lenny Chen Date: Mon, 3 Aug 2026 11:46:02 -0700 Subject: [PATCH 2/2] fix(snipsync): unpin samples-java as well Removes the remaining non-main ref. samples-server keeps ref: main, which is already main. temporalio/samples-java#784 has not merged, so lambda-worker/ exists only on ea/aws-lambda. The java lambda page's three snippets therefore have no source on main and stop tracking upstream. They keep their current content: snipsync only splices IDs it finds in a source and leaves unmatched markers alone. Confirmed by running yarn snipsync against this config, which leaves both lambda pages untouched and reports 712/716 files unchanged. --- snipsync.config.yaml | 4 ---- 1 file changed, 4 deletions(-) diff --git a/snipsync.config.yaml b/snipsync.config.yaml index b278d6b076..a1b6ba5c4f 100644 --- a/snipsync.config.yaml +++ b/snipsync.config.yaml @@ -22,12 +22,8 @@ origins: repo: samples-ruby - owner: temporalio repo: samples-dotnet - # Pinned until temporalio/samples-java#784 merges the lambda-worker sample to - # main. A ref here pins every snippet from this repo, not just one page's, so - # remove it as soon as that PR lands. - owner: temporalio repo: samples-java - ref: ea/aws-lambda - files: pattern: './sample-apps/**/*.go' owner: 'temporalio'