chore(cli): commands import infrastructure directly, contradicting the documented architecture

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

Hướng nghiên cứu

Start with PR #795, which contains the replacement architecture documentation and the related context-graph, restricted-import, and orchestrator dependency guards. Verify that the documented model matches the presentation-to-runtime composition path, that bounded contexts do not import either outer boundary, and that the architecture gates remain green before closing this issue after the PR merges.

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

Mô tả

Context

The old CLI documentation described a strict three-layer model and made command imports from infrastructure/deps appear contradictory.

PR #795 replaces that model with bounded contexts plus two outer boundaries:

  • presentation parses and renders the CLI;
  • runtime is the composition root that wires concrete ports and use cases;
  • kernel and bounded contexts never reach back into either outer boundary.

Decision

A presentation command may call the runtime wiring factory. This is intentional composition, not an application/domain dependency leak.

The factory is invoked per action because project root, output and verbosity are command-specific. Building one global dependency graph in cli.ts would either create it with incomplete command context or move command concerns into the entry point.

The relevant architectural rule is therefore: contexts depend only on permitted inward context edges; presentation and runtime may depend on them. #795 enforces this through the context graph, restricted-import and orchestrator dependency guards.

Acceptance criteria

  • #795 documentation states bounded contexts, not the obsolete three-layer rule.
  • Presentation-to-runtime wiring remains the only outer composition path.
  • No bounded context imports presentation or runtime.
  • #795 architecture gates remain green.
  • #795 links this issue; close it after merge.

Boundary

Do not move the wiring into every command, and do not introduce a global pre-built dependency graph merely to satisfy the former layer diagram.

Ngôn ngữ chính
TypeScript
Star
481
Fork
45
Merge trung bình
17 giờ 40 phút
Pull request đã merge (30 ngày)
106

Hướng dẫn đóng góp

Mở 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 ai-driven-dev/framework

Tất cả issue của ai-driven-dev/framework

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.