feat(webui): 관제 콘솔 이관 — 화면이 뜬다 - #17
Merged
Merged
Conversation
React 콘솔(web/, Vite)과 정적 서빙 계층을 옮기고 apiserver 에 다시 붙였다. ## 프로토타입의 함정을 구조로 없앴다 번들은 커밋하지 않고 index.html 셸만 커밋한다. 그래서 프론트를 빌드하지 않은 상태에서는 셸이 참조하는 해시 번들이 없어 테스트가 404 로 깨졌다 — 프로토타입 에서 make all 이 항상 실패하던 이유다. 게이트가 늘 빨간불이면 사람들이 게이트 자체를 신뢰하지 않게 된다. 이제 테스트가 번들 존재 여부를 먼저 보고 없으면 건너뛴다. make all 은 Go 만 검사하고, 프론트 검증은 make web-build 를 돌린 뒤에 실제로 수행된다. ## web-build 는 vite 에 맡긴다 vite 의 outDir 이 internal/webui/assets 라 산출물이 바로 embed 대상에 떨어진다. 복사 단계를 두었다가 emptyOutDir 와 순서가 엉켜 방금 만든 번들을 지우는 것을 한 번 겪었다 — 그래서 타깃은 pnpm build 한 줄이다. 실증: 빌드 후 webui 테스트 8/8(해시 자산 검증 포함), 프로세스 기동 시 / 가 앱 셸을 내고 /assets/index-*.js 가 200.
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
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
2 participants
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.
Phase 1 의 ④ 입니다. React 콘솔과 정적 서빙 계층을 옮기고 apiserver 에 다시 붙였습니다.
프로토타입의 함정을 구조로 없앴습니다
번들은 커밋하지 않고
index.html셸만 커밋합니다. 그래서 프론트를 빌드하지 않은 상태에서는 셸이 참조하는 해시 번들이 없어 테스트가 404 로 깨졌습니다 — 프로토타입에서make all이 항상 실패하던 이유입니다. 게이트가 늘 빨간불이면 사람들이 게이트 자체를 신뢰하지 않게 됩니다.이제 테스트가 번들 존재 여부를 먼저 보고 없으면 건너뜁니다.
make all은 Go 만 검사하고, 프론트 검증은make web-build뒤에 실제로 수행됩니다.web-build는 vite 에 맡깁니다vite 의
outDir이internal/webui/assets라 산출물이 바로 embed 대상에 떨어집니다. 복사 단계를 두었다가emptyOutDir와 순서가 엉켜 방금 만든 번들을 지우는 것을 한 번 겪었습니다 — 그래서 타깃은pnpm build한 줄입니다.실증
다음
⑤ 차트·이미지 발행 후 데모 인스턴스를 새 이미지로 교체해 실증.