refactor(loader-client): the manifest version is read from public/version in the cwd and nothing uses it

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#145 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
72/100
Loại issue
Tái cấu trúc
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
typescript
Lĩnh vực
backend, documentation

Hướng nghiên cứu

Đọc apps/client/src/manifest.ts và xem xét các kiểu manifest trong apps/client/src/server.ts và apps/website/src/types/manifest.type.ts. Cập nhật tài liệu cho client và website trong docs/docs/client.mdx và apps/website/README.md để khớp với thay đổi đã chọn. Công việc hoàn tất khi logic phiên bản manifest không được sử dụng và các tham chiếu của nó được xóa nhất quán, hoặc phương án thay thế đã được ghi lại trong tài liệu được triển khai.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

app:client app:website enhancement refactor
Which app is this feature request for?

client

Feature

Since the per-file hashes (#8), the website never reads manifest.version. The client still computes it on every /manifest request (manifest.ts#L9-L20):

  • it reads public/version relative to the process cwd, not the game dir, so a stray file there would freeze the value;
  • otherwise it fingerprints every file hash.

This is dead logic, and the docs still describe it (client.mdx#L37, website README.md#L51).

Found during a read-only audit of main @ 044c766 (2026-11-11).

Ideal solution or implementation

Remove version from the manifest and its types (apps/client/src/server.ts, apps/website/src/types/manifest.type.ts), and update the docs.

Alternative solutions or implementations

Keep version, but use it in the website as a cheap cache key before checking each file hash, and compute it from the game dir instead of the cwd.

Other context

No response

Ngôn ngữ chính
TypeScript
Star
1
Fork
0
Merge trung bình
17 giờ 25 phút
Pull request đã merge (30 ngày)
11

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của NanoForge-dev/Loader

Tất cả issue của NanoForge-dev/Loader

Issue tương tự

Thêm issue về TypeScript

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.