Skip to content

fix(ci): use ./ prefix for npm publish paths#25

Merged
Shadow-Azure merged 1 commit into
mainfrom
fix/npm-publish-path-prefix
Jun 4, 2026
Merged

fix(ci): use ./ prefix for npm publish paths#25
Shadow-Azure merged 1 commit into
mainfrom
fix/npm-publish-path-prefix

Conversation

@Shadow-Azure

Copy link
Copy Markdown
Owner

Summary

  • npm treats packages/cli-box-darwin-arm64 (without ./ prefix) as a git reference instead of a local directory path
  • This caused git ls-remote ssh://git@github.com/packages/cli-box-darwin-arm64.git error during npm publish
  • Adding ./ prefix forces npm to resolve the path as a relative directory, fixing the publish error
  • Removed unnecessary git config URL rewriting that was a red herring

Root Cause

npm's path resolution interprets packages/cli-box-darwin-arm64 as a git-style dependency (like github:user/repo) rather than a filesystem path. The ./ prefix explicitly marks it as a relative path.

Test Plan

  • Verified npm publish ./packages/cli-box-darwin-arm64 --dry-run works locally
  • Verified npm publish ./packages/cli-box-electron-darwin-arm64 --dry-run works locally
  • Verified npm publish ./packages/cli-box-skill --dry-run works locally
  • CI release workflow succeeds with npm publish

🤖 Generated with Claude Code

npm treats `packages/cli-box-darwin-arm64` as a git reference instead of
a local directory path. Adding `./` prefix forces npm to resolve it as a
relative path, fixing the "git ls-remote ssh://git@github.com/" error.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Jun 4, 2026

Copy link
Copy Markdown

🔒 门禁检查结果

检查项 状态
Rust 格式化 ✅ success
Rust Clippy ✅ success
Rust 测试 & 覆盖率 ✅ success
前端测试 & 覆盖率 ✅ success
Playwright E2E ✅ success
统一测试 (test.sh) ✅ success
安全检查 ✅ success

Rust 测试覆盖率

指标 覆盖率
行覆盖率 69.2%
分支覆盖率 0.0%
模块 行覆盖率
crates.cli-box-core.src.automation ███████████░░░░░░░░░ 58.8%
crates.cli-box-core.src.instance ███████████████████░ 99.1%
crates.cli-box-core.src.server █████████████████░░░ 87.5%
crates.cli-box-core.src.sandbox ███████████████████░ 98.8%
crates.cli-box-core.src.process █████████████░░░░░░░ 65.7%
crates.cli-box-core.src.daemon ██████████░░░░░░░░░░ 53.2%
crates.cli-box-core.src.capture ████████████████████ 100.0%
crates.cli-box-core.src ███████████░░░░░░░░░ 55.3%

详细报告见 Rust 覆盖率 artifact

前端测试覆盖率

指标 覆盖率
行覆盖率 ████████████████░░░░ 82.3%
分支覆盖率 68.9%
函数覆盖率 80.0%
语句覆盖率 80.0%
文件 行覆盖率
src/tests/mocks/websocket.ts ██████████████████░░ 88.0%
src/tests/mocks/xterm.ts ████████████████████ 100.0%
src/renderer/api.ts ██████████████░░░░░░ 68.2%
src/renderer/tabState.ts ████████████████████ 100.0%

详细报告见前端覆盖率 artifact

✅ 所有检查通过,可以合入

点击 Squash and merge 合并此PR

@Shadow-Azure
Shadow-Azure merged commit 6a6f428 into main Jun 4, 2026
9 checks passed
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