Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

pnpm check-types only checks 2 of 10 packages, so CI's type gate is mostly empty

Đã đóng
  • Có gì không ổn?
#189 3 bình luận 0 reaction 1 người được giao Xem trên GitHub

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

@Blazehue đang làm issue này rồi.

Từ ngày 26/10/2026.

  • #222 của @Blazehue — đã merge

Đánh giá

Độ khó
3/5
Thời gian dự kiến
1-2 ngày
Mức phù hợp với người mới
72/100
Loại issue
Tính năng
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
typescript, vite
Lĩnh vực
build-system, ci-cd

Hướng nghiên cứu

Start by reading .github/workflows/ci.yml, the root package scripts and turbo.json, then compare the package.json files for the listed apps with genai and @repo/ui. Check each package’s TypeScript project configuration, especially the frontend build setup, before adding the missing checks. Run pnpm run check-types and verify a deliberate type error in http-server or frontend makes it fail.

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

Mô tả

Backend first issue bug good first issue help wanted nsoc26

CI presents this as the type-safety gate:

# .github/workflows/ci.yml
      - name: Type-check
        run: pnpm run check-types

But most packages do not define the script, so turbo finds almost nothing to run:

• Packages in scope: @repo/db, @repo/eslint-config, @repo/typescript-config,
                     @repo/ui, admin, admin-server, frontend, genai,
                     http-server, ws-server
• Running check-types in 10 packages
 Tasks:    2 successful, 2 total

Only genai and @repo/ui have it. frontend, http-server, ws-server, admin, admin-server and @repo/db are all skipped, and turbo exits 0 regardless.

Why it matters

A contributor can break every type in http-server or the whole frontend and CI stays green. The step looks like protection and provides almost none — which is worse than having no step, because reviewers trust it.

What a fix looks like

Add the script to each package that is missing it:

"check-types": "tsc --noEmit"

The frontend builds with tsc -b && vite build, so it may want tsc -b --noEmit or its existing project reference instead — worth checking what that package actually needs rather than pasting the same line everywhere.

You will not inherit a pile of errors. I checked both servers before filing:

apps/http-server  tsc --noEmit -> 0 errors
apps/ws-server    tsc --noEmit -> 0 errors

So this is additive. If admin or admin-server turn out to have existing errors, say so in the PR rather than fixing them in the same change — those can be their own issue.

Definition of done
  • pnpm run check-types reports more than 2 total (6+ depending on what you include)
  • A deliberate type error in http-server or frontend makes the command fail
  • CI goes red for that deliberate error, then green once reverted

Good first issue: a few lines of JSON, and it turns a decorative CI step into a real one. Verify it works by breaking a type on purpose before you open the PR.

Ngôn ngữ chính
TypeScript
Star
21
Fork
7
Merge trung bình
5 giờ 6 phút
Pull request đã merge (30 ngày)
29

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 TheCodeHeist-Coder/Rexial

Tất cả issue của TheCodeHeist-Coder/Rexial

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.