docs: 서브프로세스 서버 결정 정정 — v2.4.1 설치본에서 여전히 미해소 - #50
Merged
Conversation
v2.4.1 을 릴리스하고 설치본을 실측한 결과, dist/resources/ 에 pi-agent-server 와 session-mcp-server 가 여전히 없다. 결정 레코드가 "수리 완료"라고 적은 것은 사실이 아니었다. 결정 자체가 틀린 게 아니라 적용 지점이 틀렸다. 자산 복사 스크립트가 두 개고, 수리는 electron-build-resources.ts 에 들어갔는데 설치파일을 만드는 build-win.ps1 은 copy-assets.ts 만 부른다. copy-assets.ts 396행 주석의 "Single source of truth" 는 사실이 아니다. 수리를 build-win.ps1 로 옮기려 했으나 pi-agent-server 가 로컬에서 빌드되지 않는다. 추적 결과 루트 node_modules 를 npm 과 bun 이 나눠 쓰면서 @aws-crypto 서브트리가 통째로 잘려 있었다 — 루트 package.json 의 workspaces 는 apps/desktop 을 포함하지 않는데 bun 이 루트를 모노레포 루트로 오인해 스토어를 거기 만든다. 검증 안 된 빌드 단계를 워크플로에 넣으면 방금 복구한 파이프라인이 다시 깨지므로 여기서 멈춘다. 조사 중 node_modules 에 임시 배치했던 파일은 원상 복구했다. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
v2.4.1 을 릴리스하고 설치본을 실측한 결과,
dist/resources/에pi-agent-server와session-mcp-server가 여전히 없다.llama는 들어왔다. 결정 레코드가 "수리 완료"라고 적은 것은 사실이 아니었고, 그 문장을 바로잡는다.왜 안 실렸나
결정 자체가 틀린 게 아니라 적용 지점이 틀렸다. 자산 복사 스크립트가 두 개다:
scripts/electron-build-resources.ts(수리된 것)electron:build체인apps/electron/scripts/copy-assets.tsbuild-win.ps1:396— 설치파일을 만드는 유일한 경로build-win.ps1은bun run electron:build를 통째로 부르지 않고 조각을 직접 부른다.copy-assets.ts396행 주석의 "Single source of truth" 는 사실이 아니다.왜 지금 안 고쳤나
build-win.ps1에server:build:subprocess+electron:build:resources를 넣으려 했으나pi-agent-server가 로컬에서 빌드되지 않는다 (@aws-crypto/crc32미해석).추적 결과: 루트
package.json의 workspaces 는["packages/*", "mcp", "extension"]로apps/desktop을 포함하지 않는다. 그런데apps/desktop에서bun install을 돌리면 bun 이 위로 올라가 루트를 모노레포 루트로 오인해 스토어를sapstack/node_modules/.bun/에 만든다. 같은 디렉토리를 ci.yml 이npm ci로 따로 관리하면서@aws-crypto서브트리가 통째로 잘렸다.실증: 해석 경로에
crc32를 수동 배치하니@aws-crypto/util→tslib로 결핍이 연쇄로 드러났다. 버전 충돌이 아니라 서브트리 전체 부재다. (확인 후node_modules는 원상 복구했다.)검증 안 된 빌드 단계를 릴리스 워크플로에 넣으면 방금 4개 PR 로 복구한 파이프라인이 다시 깨진다. 그래서 문서만 정정하고 멈춘다.
다음 한 걸음
클린 체크아웃에서
bun install→bun run server:build:subprocess통과 여부 확인. 통과하면 로컬 스토어 오염이라build-win.ps1만 고치면 되고, 실패하면 의존성 해석 자체의 결함이다. (개발 머신 디스크 여유가 8.2G 뿐이라 여기서는 클린 클론을 만들지 못했다.)Test Plan
문서 변경만. 코드 경로 무변경.