Skip to content

fix(ci): artifact name mismatch у publish-mt-napi (musl) - #13

Merged
vitaliytv merged 1 commit into
mainfrom
mt/fix-napi-artifact-name
Jul 27, 2026
Merged

fix(ci): artifact name mismatch у publish-mt-napi (musl)#13
vitaliytv merged 1 commit into
mainfrom
mt/fix-napi-artifact-name

Conversation

@vitaliytv

Copy link
Copy Markdown
Member

Summary

Третій баг з живого тесту на mt-v0.1.3 (після фіксів у #12, які виправили anti-recursion trigger і crt-static для musl — обидва build-napi легі тепер зелені). publish-mt-napi падав: cp: cannot stat 'dist/mt-napi-linux-x64/mt.linux-x64-musl.node': No such file or directory.

Причина: build-napi вивантажує артефакт як mt-napi-${{ matrix.napi_suffix }}, а napi_suffix для musl-ноги = linux-x64-musl → фактичне ім'я mt-napi-linux-x64-musl. publish-mt-napi шукав mt-napi-linux-x64 (без -musl). darwin-нога збігалась випадково (napi_suffix: darwin-arm64 без додаткового суфікса).

Test plan

  • Підтверджено артефактами реального прогону (gh api .../artifacts показав фактичну назву mt-napi-linux-x64-musl)
  • YAML-синтаксис перевірено
  • Не перевірено наживо після фіксу — тригерну повторно після мерджу

🤖 Generated with Claude Code

Третій реальний баг з того самого live-fire тесту (тепер mt-v0.1.3):
build-napi вивантажує артефакт як mt-napi-\${{ matrix.napi_suffix }},
а для musl-ноги napi_suffix = "linux-x64-musl" → артефакт
"mt-napi-linux-x64-musl". publish-mt-napi шукав "mt-napi-linux-x64"
(без "-musl") — cp падав "No such file or directory". darwin-нога
збігалась випадково (napi_suffix там без додаткового суфікса).

build-napi (обидві ноги, включно з musl після фіксу crt-static
з попереднього PR) уже зелений на цьому прогоні — залишалась тільки
ця невідповідність імені на боці publish.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@vitaliytv
vitaliytv merged commit 422e238 into main Jul 27, 2026
1 of 4 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