Skip to content

ci: 서명 identity 에서 인증서 종류 접두사를 뺀다 - #55

Merged
jeongph merged 1 commit into
mainfrom
fix/csc-name
Sep 6, 2026
Merged

ci: 서명 identity 에서 인증서 종류 접두사를 뺀다#55
jeongph merged 1 commit into
mainfrom
fix/csc-name

Conversation

@jeongph

@jeongph jeongph commented Sep 6, 2026

Copy link
Copy Markdown
Member

무엇이

v0.10.0 재빌드가 서명 단계에서 실패했다.

⨯ Please remove prefix "Developer ID Application:" from the specified name
  — appropriate certificate will be chosen automatically

앞선 워크플로 변경에서 find-identity 출력의 따옴표 안을 통째로 뽑아 CSC_NAME 으로 넘겼다. 그 값이 Developer ID Application: <이름> (<팀>) 이라 electron-builder 가 거부한다. 접두사 뒤만 넘겨야 한다.

어떻게

sed -n ... p 로 매치된 줄만 출력한다. 접두사를 벗기는 것과 함께, 출력 형식이 달라졌을 때 빈 값이 되도록 하는 효과가 있다. 전에는 정규식이 빗나가면 1) ABC123 "..." 같은 원본 줄이 그대로 CSC_NAME 으로 흘러갔다.

진전

이번 실패는 앞선 수정이 통했다는 신호이기도 하다. 키체인 준비 단계가 성공했고, 인증서를 찾아 등록하는 데까지 갔다. 지난번에 막히던 set-key-partition-list 는 넘어갔다.

한편 이번 runner 는 os=25.5.0 으로 배정됐다. 즉 이미지 배정이 오락가락한다. 키체인을 직접 관리하는 편이 그 변동에 흔들리지 않으므로 그대로 둔다.

Summary by CodeRabbit

  • 버그 수정
    • macOS 릴리스 과정에서 Developer ID Application 인증서 이름을 더 정확하게 확인하도록 개선했습니다.
    • 인증서 형식이 올바르지 않을 경우 릴리스 단계에서 조기에 오류를 감지합니다.

electron-builder 는 CSC_NAME 에 "Developer ID Application:" 접두사가 붙어
있으면 거부한다. find-identity 출력에서 따옴표 안을 통째로 뽑던 것을 접두사
뒤만 뽑도록 고친다.

매치된 줄만 출력하게 바꿔서, 출력 형식이 달라지면 빈 값이 되어 아래 검사에
걸린다. 전에는 매치가 빗나가도 원본 줄이 그대로 흘러갔다.
@jeongph jeongph self-assigned this Sep 6, 2026
@coderabbitai

coderabbitai Bot commented Sep 6, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Team

Run ID: 39aa342b-01a9-4f94-b052-731b6eab43b5

📥 Commits

Reviewing files that changed from the base of the PR and between 36ea9d8 and b2aefcd.

📒 Files selected for processing (1)
  • .github/workflows/release.yml

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

macOS 릴리스 워크플로가 security find-identity 출력에서 Developer ID Application 인증서 이름을 정규식으로 추출합니다. 인증서 형식이 다르거나 없으면 IDENTITY를 빈 값으로 설정하고 이후 검사를 실행합니다.

Changes

macOS 릴리스 서명

Layer / File(s) Summary
인증서 식별자 추출 검증
.github/workflows/release.yml
IDENTITY 추출이 grep와 실패 무시 방식에서 정규식 기반 sedhead -1 방식으로 변경되었습니다. 인증서 이름 접두사를 제거하며, 일치하지 않으면 빈 값이 됩니다.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Merge Risk: ⚪ Minimal · up to b2aef

The macOS release workflow now passes the certificate name without its type prefix and fails clearly when no matching identity is found. No merge-blocking risk remains.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed 제목은 macOS 서명 identity에서 인증서 종류 접두사를 제거하는 주요 변경 사항을 정확하고 간결하게 설명합니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/csc-name

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@jeongph
jeongph merged commit 72fc294 into main Sep 6, 2026
3 checks passed
@jeongph
jeongph deleted the fix/csc-name branch September 6, 2026 10:22
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