Skip to content

refactor(ci): 批量精简脚本执行路径#73

Merged
iamsyc merged 1 commit into
mainfrom
codex/scripts-reduction-sweep
May 10, 2026
Merged

refactor(ci): 批量精简脚本执行路径#73
iamsyc merged 1 commit into
mainfrom
codex/scripts-reduction-sweep

Conversation

@iamsyc
Copy link
Copy Markdown
Owner

@iamsyc iamsyc commented May 10, 2026

What changed

  • Reduced scripts/ci/classify.sh by consolidating repeated category and output handling.
  • Removed single-use relay download, release architecture check, and WebRTC thin/sign scripts.
  • Moved shared relay download and release binary thin/sign helpers into existing library modules.
  • Simplified DMG creation by deleting its private intermediate summary path and keeping release build summary as the convergence layer.

Why

The scripts directory had accumulated redundant wrappers and duplicate failure-reporting layers. This keeps the same CI and release behavior while reducing script size and the number of moving parts.

Validation

  • bash -n scripts/ci/*.sh scripts/dev/*.sh scripts/lib/*.sh scripts/release/*.sh scripts/build-relay.sh
  • shfmt -d scripts/ci/*.sh scripts/dev/*.sh scripts/lib/*.sh scripts/release/*.sh scripts/build-relay.sh
  • shellcheck -x -e SC2016 scripts/ci/*.sh scripts/dev/*.sh scripts/lib/*.sh scripts/release/*.sh scripts/build-relay.sh
  • jq -e '.categories | type == "object"' scripts/ci/classification-rules.json
  • ROOT_DIR=/Users/syc/Project/VoidDisplay TOOL_ROOT=/Users/syc/Project/VoidDisplay scripts/ci/test_classify.sh
  • ROOT_DIR=/Users/syc/Project/VoidDisplay TOOL_ROOT=/Users/syc/Project/VoidDisplay scripts/ci/static.sh
  • ROOT_DIR=/Users/syc/Project/VoidDisplay TOOL_ROOT=/Users/syc/Project/VoidDisplay scripts/ci/xcode.sh --action build --configuration Debug --out-dir .ai-tmp/scripts-reduction-xcode-build
  • ROOT_DIR=/Users/syc/Project/VoidDisplay TOOL_ROOT=/Users/syc/Project/VoidDisplay scripts/release/build.sh --tag v2.1.0 --arch arm64 --label arm64 --out-dir .ai-tmp/scripts-reduction-release-build-arm64
  • ROOT_DIR=/Users/syc/Project/VoidDisplay TOOL_ROOT=/Users/syc/Project/VoidDisplay scripts/release/verify.sh --assets-dir .ai-tmp/scripts-reduction-release-build-arm64/release-assets --tag v2.1.0 --label arm64 --arch arm64

- 收敛 classify 输出和 release smoke 中间脚本

- 删除 relay 下载与 release 架构检查单调用脚本

- 精简 DMG 私有 summary 并验证 release asset
@github-actions
Copy link
Copy Markdown

CI Summary

Run: Open

Check Status Details
Change Scope code code=true
ui=false
script=true
product=false
test=false
ci=true
release=true
dependency=false
tooling=false
docs_only=false
unknown=false
Required Gates derived static=true
dependency=false
unit=false
xcode=false
ui=false
release=true
Dependency Review skipped required=false
high and critical vulnerabilities block
Static Checks success required=true
actionlint, shellcheck, shfmt, SwiftFormat, SwiftLint, action pinning
Unit Tests skipped required=false
Swift tests: n/a
Go packages: n/a
Reason: n/a
Artifact: n/a
Xcode Build skipped required=false
Debug build with zero warning scan
Artifact: xcode-build
UI Smoke skipped required=false
Failure classification is written by ui-smoke-summary.json in each artifact.
Artifacts: ui-smoke-baseline, ui-smoke-permissionDenied, ui-smoke-rebuildFailed
Release Smoke success required=true
arm64=success
intel64=skipped
PR release smoke requires arm64 only.
Artifacts: release-smoke-arm64
Release verify summaries are produced by release.yml before publishing.
Artifacts link Open artifacts
CI Gate success single branch protection check

@iamsyc iamsyc merged commit 5f81ed2 into main May 10, 2026
10 checks passed
@iamsyc iamsyc deleted the codex/scripts-reduction-sweep branch May 10, 2026 11:38
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.

1 participant