chrisp/brackt
1
1
Fork 0
Code Issues 42 Pull requests Projects Releases Packages Wiki Activity Actions

Fix Docker build failure caused by server-only import reaching client bundle #123

Merged
chrisp merged 1 commit from claude/docker-build-output-vjywdf into main 2026-07-01 22:54:33 +00:00
Conversation 0 Commits 1 Files changed 4 +15 -14
chrisp commented 2026-07-01 22:47:02 +00:00
Owner
Copy link

scoring-event.ts exported getEventTypeLabel, a pure display helper used
directly in route components, from the same module that imports
scoring-calculator.ts (which now pulls in the Discord notification
service, a .server-only module). Because that whole chain was
value-imported into the client bundle, Vite's server-only-module guard
failed the build. Move getEventTypeLabel into a new client-safe
scoring-event-types.ts module with no server dependencies, and import
it directly where it's rendered.

scoring-event.ts exported getEventTypeLabel, a pure display helper used directly in route components, from the same module that imports scoring-calculator.ts (which now pulls in the Discord notification service, a .server-only module). Because that whole chain was value-imported into the client bundle, Vite's server-only-module guard failed the build. Move getEventTypeLabel into a new client-safe scoring-event-types.ts module with no server dependencies, and import it directly where it's rendered.
chrisp added 1 commit 2026-07-01 22:47:03 +00:00
Fix Docker build failure caused by server-only import reaching client bundle
All checks were successful
🚀 Deploy / 🧪 Test (pull_request) Successful in 3m3s
Details
🚀 Deploy / ʦ🔍 Typecheck & Lint (pull_request) Successful in 1m19s
Details
🚀 Deploy / 🐳 Build (pull_request) Has been skipped
Details
🚀 Deploy / 🚀 Deploy (pull_request) Has been skipped
Details
ef7a71485c 
scoring-event.ts exported getEventTypeLabel, a pure display helper used
directly in route components, from the same module that imports
scoring-calculator.ts (which now pulls in the Discord notification
service, a .server-only module). Because that whole chain was
value-imported into the client bundle, Vite's server-only-module guard
failed the build. Move getEventTypeLabel into a new client-safe
scoring-event-types.ts module with no server dependencies, and import
it directly where it's rendered.
chrisp merged commit f527fc8f7f into main 2026-07-01 22:54:33 +00:00
chrisp deleted branch claude/docker-build-output-vjywdf 2026-07-01 22:54:33 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
admin

Admin page tooling

  
defect

Something isn't working

  
draft

Related to the draft room

  
feature

New feature or request

  
infrastructure

Related to database/site infrastructure

  
new sport

Issue to add a new sport to the rotation

  
security

Things that can cause security issues

  
tech debt

Cleaning up things that will make us have a better life

  
ux

User interface or marketing

No labels
admin
defect
draft
feature
infrastructure
new sport
security
tech debt
ux
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: chrisp/brackt#123
No description provided.
Delete branch "claude/docker-build-output-vjywdf"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?