nf start resolves a relative --cert or --key against the loader folder

Đang mở
#282 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ó
3/5
Thời gian dự kiến
1-2 ngày
Mức phù hợp với người mới
75/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
bun, typescript
Lĩnh vực
cli

Hướng nghiên cứu

Read src/action/actions/start.action.ts around lines 221–222 to trace how certificate and key paths are passed to the loader; the issue says flags should resolve from the current working directory and config values from the project folder. Check the loader startup path and the start.mdx limitation, then verify both files are checked before startup and relative paths work for both input sources.

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

Mô tả

bug command:start
Which package is this bug report for?

@nanoforge-dev/cli

Which command is this bug report for?

start

Issue description

nf start --cert <path> --key <path> (or tls.cert/tls.key in the config) only works with absolute paths (read from the code, not run). -d is made absolute with getCwd, but cert and key are passed as given (src/action/actions/start.action.ts#L221-L222). The loader runs with its cwd in node_modules/@nanoforge-dev/loader-client and opens the files with Bun.file(cert) (Loader apps/client/src/server.ts line 60). So --cert ./cert.pem, or a relative path in the config, points into the loader's folder and the client fails to start. Because of the exit-code issue on nf start, the command then exits 0 too.

start.mdx documents this as a limitation (lines 61 and 96). Related to #156, which added TLS to the config.

Expected: relative paths are resolved (from the cwd for the flags, from the project folder for config values), and both files are checked before the loader starts.

Found during a read-only audit of main at e95b881 (2026-11-11).

Code sample
params["--cert"] = target.tls.cert;   // passed as given
params["--key"] = target.tls.key;
// while "-d": getCwd(join(target.directory, target.outDir)) is made absolute
Versions
  • @nanoforge-dev/cli 3.0.1 (main, e95b881)
  • Node.js 26.7.0
  • Bun 1.3.14 (bundled)
  • Linux
Issue priority

Low (slightly annoying)

I have tested this issue on a development release

No response

Ngôn ngữ chính
TypeScript
Star
2
Fork
2
Merge trung bình
2 ngày 18 giờ
Pull request đã merge (30 ngày)
17

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/CLI

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

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.