feat: 优化 Gist 同步诊断信息并更新 Node/Mocha 依赖策略

xream · xream · commit 58ab0a7f5c9e · 2026-05-11T12:17:44.000+08:00
- 新增 `.node-version`（26.1.0）并在 GitHub Action 中改为按文件读取 Node 版本
- backend 版本号提升到 2.22.27，Mocha 升级到 ^11.7.5 并同步更新 lockfile
- 重构 Gist 错误信息提取逻辑，统一返回 `ERROR: HTTP 状态码: 消息`，便于定位 API 问题
- Gist 同步前补充上传摘要日志（文件数、总大小、最大文件），同步失败时输出失败上下文
- 同步异常日志改为输出完整错误栈，便于排查
- 同步到 Gist 的失败场景与新增日志行为补充单测覆盖
diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
@@ -16,16 +16,16 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout
-        uses: actions/checkout@v3
+        uses: actions/checkout@v6
         with:
           ref: "master"
       - name: Set up Node.js
-        uses: actions/setup-node@v3
+        uses: actions/setup-node@v6
         with:
-          node-version: "20"
+          node-version-file: ".node-version"
       - name: Install dependencies
         run: |
-          npm install -g pnpm
+          corepack enable
           cd backend && pnpm i --no-frozen-lockfile
       - name: Sync latest mihomo version
         run: |
diff --git a/.node-version b/.node-version
@@ -0,0 +1 @@
+24.15.0
diff --git a/backend/package.json b/backend/package.json
@@ -1,8 +1,9 @@
 {
   "name": "sub-store",
-  "version": "2.22.26",
+  "version": "2.22.28",
   "description": "Advanced Subscription Manager for QX, Loon, Surge, Stash and Shadowrocket.",
   "main": "src/main.js",
+  "packageManager": "pnpm@11.0.9",
   "scripts": {
     "preinstall": "npx only-allow pnpm",
     "test": "mocha src/test/**/*.spec.js --require @babel/register --recursive",
@@ -54,7 +55,7 @@
     "chai": "^4.3.6",
     "esbuild": "^0.19.8",
     "eslint": "^8.16.0",
-    "mocha": "^10.0.0",
+    "mocha": "^11.7.5",
     "nodemon": "^2.0.16",
     "peggy": "^2.0.1",
     "prettier": "2.6.2"
diff --git a/backend/pnpm-lock.yaml b/backend/pnpm-lock.yaml
diff --git a/backend/pnpm-workspace.yaml b/backend/pnpm-workspace.yaml
@@ -0,0 +1,3 @@
+allowBuilds:
+  core-js: true
+  esbuild: true
diff --git a/backend/src/products/cron-sync-artifacts.js b/backend/src/products/cron-sync-artifacts.js
@@ -269,6 +269,6 @@ async function doSync(arg = {}) {
         }
     } catch (e) {
         $.notify('🌍 Sub-Store', '同步配置失败', `原因：${e.message ?? e}`);
-        $.error(`无法同步配置到 Gist，原因：${e}`);
+        $.error(`无法同步配置到 Gist，原因：${e.stack ?? e.message ?? e}`);
     }
 }
diff --git a/backend/src/restful/artifacts.js b/backend/src/restful/artifacts.js
@@ -323,16 +323,38 @@ async function syncToGist(files, options = {}) {
     if (!gistToken) {
         return Promise.reject('未设置 GitHub Token！');
     }
+    const uploadSummary = summarizeGistUploadFiles(files);
+    $.info(
+        `准备同步 Gist: 文件数 ${uploadSummary.count}, 总大小 ${formatBytes(
+            uploadSummary.totalBytes,
+        )}, 最大文件 ${
+            uploadSummary.largestFilename || '-'
+        } (${formatBytes(uploadSummary.largestBytes)})`,
+    );
     const manager = new Gist({
         token: gistToken,
         key: ARTIFACT_REPOSITORY_KEY,
         syncPlatform,
     });
-    const res = await manager.upload(files, {
-        ...options,
-        emptyFileFallback:
-            options.emptyFileFallback ?? getArtifactGistEmptyFileFallback(),
-    });
+    let res;
+    try {
+        res = await manager.upload(files, {
+            ...options,
+            emptyFileFallback:
+                options.emptyFileFallback ?? getArtifactGistEmptyFileFallback(),
+        });
+    } catch (error) {
+        $.error(
+            `同步 Gist 请求失败: 文件数 ${uploadSummary.count}, 总大小 ${formatBytes(
+                uploadSummary.totalBytes,
+            )}, 最大文件 ${
+                uploadSummary.largestFilename || '-'
+            } (${formatBytes(uploadSummary.largestBytes)}), 原因: ${
+                error.message ?? error
+            }`,
+        );
+        throw error;
+    }
     let body = {};
     try {
         body = JSON.parse(res.body);
@@ -353,5 +375,41 @@ async function syncToGist(files, options = {}) {
     return res;
 }
 
+function summarizeGistUploadFiles(files) {
+    return Object.entries(files || {}).reduce(
+        (summary, [filename, file]) => {
+            const content = file?.content;
+            if (typeof content !== 'string') return summary;
+            const bytes = stringByteLength(content);
+            summary.count++;
+            summary.totalBytes += bytes;
+            if (bytes > summary.largestBytes) {
+                summary.largestBytes = bytes;
+                summary.largestFilename = filename;
+            }
+            return summary;
+        },
+        {
+            count: 0,
+            totalBytes: 0,
+            largestBytes: 0,
+            largestFilename: '',
+        },
+    );
+}
+
+function stringByteLength(value) {
+    if (typeof TextEncoder !== 'undefined') {
+        return new TextEncoder().encode(value).length;
+    }
+    return unescape(encodeURIComponent(value)).length;
+}
+
+function formatBytes(size) {
+    if (size < 1024) return `${size} B`;
+    if (size < 1024 * 1024) return `${(size / 1024).toFixed(1)} KB`;
+    return `${(size / 1024 / 1024).toFixed(1)} MB`;
+}
+
 export { syncToGist };
 export { createArtifactItem, deleteArtifactItem };
diff --git a/backend/src/test/utils/gist.spec.js b/backend/src/test/utils/gist.spec.js
@@ -3,7 +3,10 @@ import { describe, it } from 'mocha';
 
 import $ from '@/core/app';
 import { SETTINGS_KEY } from '@/constants';
-import Gist, { getGithubGistBaseURL } from '@/utils/gist';
+import Gist, {
+    describeGistApiErrorResponse,
+    getGithubGistBaseURL,
+} from '@/utils/gist';
 import { syncToGist } from '@/restful/artifacts';
 
 describe('Gist GitHub API URL', function () {
@@ -36,6 +39,20 @@ describe('Gist GitHub API URL', function () {
         ).to.equal('https://litegist.example.com/api');
     });
 
+    it('includes HTTP status when describing Gist API errors', function () {
+        expect(
+            describeGistApiErrorResponse({
+                statusCode: 500,
+                body: JSON.stringify({
+                    message:
+                        'Internal Server Error: Error: D1 query budget exceeded',
+                }),
+            }),
+        ).to.equal(
+            'ERROR: HTTP 500: Internal Server Error: Error: D1 query budget exceeded',
+        );
+    });
+
     it('keeps an existing Gist alive with a fallback file when a delete would empty it', async function () {
         const manager = Object.create(Gist.prototype);
         let patchBody;
@@ -143,9 +160,11 @@ describe('Gist GitHub API URL', function () {
     it('passes the artifact placeholder fallback to syncToGist by default', async function () {
         const originalRead = $.read.bind($);
         const originalWrite = $.write.bind($);
+        const originalInfo = $.info.bind($);
         const originalUpload = Gist.prototype.upload;
         let capturedOptions;
         let writtenSettings;
+        const infoMessages = [];
 
         $.read = (key) => {
             if (key === SETTINGS_KEY) {
@@ -162,6 +181,9 @@ describe('Gist GitHub API URL', function () {
             }
             return originalWrite(data, key);
         };
+        $.info = (message) => {
+            infoMessages.push(message);
+        };
         Gist.prototype.upload = async function (_, options) {
             capturedOptions = options;
             return {
@@ -181,6 +203,7 @@ describe('Gist GitHub API URL', function () {
         } finally {
             $.read = originalRead;
             $.write = originalWrite;
+            $.info = originalInfo;
             Gist.prototype.upload = originalUpload;
         }
 
@@ -193,5 +216,8 @@ describe('Gist GitHub API URL', function () {
             'https://gist.example.com/sub-store',
         );
         expect(writtenSettings.artifactStoreStatus).to.equal('VALID');
+        expect(infoMessages).to.include(
+            '准备同步 Gist: 文件数 1, 总大小 12 B, 最大文件 artifact (12 B)',
+        );
     });
 });
diff --git a/backend/src/utils/gist.js b/backend/src/utils/gist.js
@@ -5,6 +5,22 @@ import { SETTINGS_KEY } from '@/constants';
 
 const DEFAULT_GITHUB_API_URL = 'https://api.github.com';
 
+function describeGistApiErrorResponse(resp) {
+    let body;
+    try {
+        body = JSON.parse(resp.body);
+    } catch (e) {
+        //
+    }
+    const message =
+        body?.message?.error ??
+        body?.error ??
+        body?.message ??
+        resp.body ??
+        'Unknown error';
+    return `ERROR: HTTP ${resp.statusCode}: ${message}`;
+}
+
 function normalizeApiUrl(url, fallback = DEFAULT_GITHUB_API_URL) {
     const normalizedUrl = String(url ?? '').trim() || fallback;
 
@@ -28,6 +44,8 @@ export function getGithubGistBaseURL({ githubApiUrl, githubProxy } = {}) {
     }${DEFAULT_GITHUB_API_URL}`;
 }
 
+export { describeGistApiErrorResponse };
+
 /**
  * Gist backup
  */
@@ -81,19 +99,8 @@ export default class Gist {
                 events: {
                     onResponse: (resp) => {
                         if (/^[45]/.test(String(resp.statusCode))) {
-                            let body;
-                            try {
-                                body = JSON.parse(resp.body);
-                            } catch (e) {
-                                //
-                            }
                             return Promise.reject(
-                                `ERROR: ${
-                                    body?.message?.error ??
-                                    body?.error ??
-                                    body?.message ??
-                                    resp.body
-                                }`,
+                                describeGistApiErrorResponse(resp),
                             );
                         } else {
                             return resp;
@@ -131,19 +138,8 @@ export default class Gist {
                 events: {
                     onResponse: (resp) => {
                         if (/^[45]/.test(String(resp.statusCode))) {
-                            let body;
-                            try {
-                                body = JSON.parse(resp.body);
-                            } catch (e) {
-                                //
-                            }
                             return Promise.reject(
-                                `ERROR: ${
-                                    body?.message?.error ??
-                                    body?.error ??
-                                    body?.message ??
-                                    resp.body
-                                }`,
+                                describeGistApiErrorResponse(resp),
                             );
                         } else {
                             return resp;

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+allowBuilds:`
	`2`	`+ core-js: true`
	`3`	`+ esbuild: true`
Original file line number	Diff line number	Diff line change
`@@ -269,6 +269,6 @@ async function doSync(arg = {}) {`
`269`	`269`	`}`
`270`	`270`	`} catch (e) {`
`271`	`271`	$.notify('🌍 Sub-Store', '同步配置失败', `原因：${e.message ?? e}`);
`272`		- $.error(`无法同步配置到 Gist，原因：${e}`);
	`272`	+ $.error(`无法同步配置到 Gist，原因：${e.stack ?? e.message ?? e}`);
`273`	`273`	`}`
`274`	`274`	`}`