From e0e8816f38a4dc4bc30fd01326fdb0db40db2954 Mon Sep 17 00:00:00 2001 From: Lan_zhijiang Date: Tue, 18 Aug 2026 17:23:15 +0800 Subject: [PATCH] fix(preview): decouple delivery from consumer smoke - treat successful Pages deployment as preview delivery evidence - keep consumer acceptance outside the synchronous controller --- .github/workflows/pages-preview.yml | 8 -------- docs/40-deployment/web-delivery.md | 4 +++- 2 files changed, 3 insertions(+), 9 deletions(-) diff --git a/.github/workflows/pages-preview.yml b/.github/workflows/pages-preview.yml index 1e7e5cc..daecf1b 100644 --- a/.github/workflows/pages-preview.yml +++ b/.github/workflows/pages-preview.yml @@ -194,14 +194,6 @@ jobs: --commit-hash=${{ needs.identity.outputs.head_sha }} --commit-dirty=false - - name: Smoke-test the stable preview URL and native facade - run: node controller/scripts/verify-pages-deployment.mjs - env: - CLOUDFLARE_PAGES_DEPLOYMENT_ID: ${{ steps.pages.outputs.pages-deployment-id }} - CLOUDFLARE_PAGES_DEPLOYMENT_URL: ${{ steps.pages.outputs.pages-deployment-alias-url }} - INKCRE_EXPECTED_PAGES_ORIGIN: https://preview-client-web-pr-${{ needs.identity.outputs.pull_number }}.${{ vars.CLOUDFLARE_PAGES_PROJECT }}.pages.dev - INKCRE_PREVIEW_PRODUCER: candidate/extensions/twitter/package.json - - name: Report the pull-request deployment if: ${{ always() && steps.github-deployment.outputs.deployment_id != '' }} uses: actions/github-script@3a2844b7e9c422d3c10d287c895573f7108da1b3 # v9.0.0 diff --git a/docs/40-deployment/web-delivery.md b/docs/40-deployment/web-delivery.md index b2f00d1..5f7730c 100644 --- a/docs/40-deployment/web-delivery.md +++ b/docs/40-deployment/web-delivery.md @@ -45,7 +45,9 @@ The Toolkit places read-only native Extension snapshots and preview Releases bes app build, using the stable alias branch `preview/client-web/pr-`. This is a preview projection, not Registry publication: the workflow has no `INKCRE_EXTENSION_REGISTRY_TOKEN`. It registers a transient GitHub `preview` deployment, deploys -through Pages, smoke-tests the stable alias, and reports success or failure. +through Pages and reports the provider deployment result. It does not turn edge propagation, +deep route traversal, cache behavior, or byte comparison into synchronous delivery gates; consumer +acceptance is a separate black-box activity. When an eligible internal PR closes, [`.github/workflows/pages-cleanup.yml`](../../.github/workflows/pages-cleanup.yml) replaces only that exact preview alias with a closed-page tombstone. Manual cleanup accepts an