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

Prevent parent scope symbols leaking into module's scope

Đang mở
#7 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

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

Đánh giá

Độ khó
5/5
Thời gian dự kiến
Hơn một tuần
Mức phù hợp với người mới
20/100
Loại issue
Tính năng
Độ rõ ràng
Cần làm rõ
Mức độ hoạt động
Đình trệ
Công nghệ
rust
Lĩnh vực
compilers

Hướng nghiên cứu

Bắt đầu bằng cách lần theo việc tra cứu ký hiệu trong ngăn xếp scope và create_default_context(), sau đó kiểm tra cách FrameType và các hằng số như NU_LIB_DIRS được biểu diễn. Quyết định cách các frame Barrier và Prelude nên tương tác với việc cô lập module, các lệnh mặc định và các lớp phủ nupm. Công việc được xem là hoàn tất khi các quy tắc tra cứu và hành vi của NU_LIB_DIRS được đặc tả và triển khai, với coverage cho các ký hiệu của scope cha, các lệnh Prelude và việc import module.

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

Mô tả

Leaking user-defined symbols from a parent scope to a module's scope can lead to subtle bugs. The module's scope should be as isolated from the parent scope as possible, but not completely (e.g., we still need basic Nushell commands like ls to be visible).

Could be partially addressed by introducing two new FrameTypes:

  • Barrier: Stops symbol lookup when traversing the scope stack. This prevents the symbol lookup from searching the parent scopes.
  • Prelude: With only Barrier, we'd lose “default” commands that we want to be visible (like ls). Perhaps we could have a Prelude frame type which would get initialized first during create_default_context() with these default commands.

The question is what to do with constants like NU_LIB_DIRS, e.g.,:

const NU_LIB_DIRS = $NU_LIB_DIRS | append '/home/spam/foo`

module {
    use bar.nu  # should it look for 'bar.nu' in '/home/spam/foo'?
}

We might want to decide on this based on the envisioned functionality of how nupm is supposed to use overlays as “virtual environments”, relying on modifying the NU_LIB_DIRS constant.

Ngôn ngữ chính
Rust
Star
36
Fork
11
Chỉ số merge pull request
Không có pull request nào được merge trong 30 ngày

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

  • Không có Dockerfile hay tệp Docker Compose
  • Không có mẫu pull request
  • Đọc hướng dẫn đóng góp

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 nushell/new-nu-parser

Tất cả issue của nushell/new-nu-parser

Issue tương tự

Thêm issue về Rust

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.