fix(dts): `ngAcceptInputType_*` types drop some comments ngtsc keeps
維護者通常 1 天內回覆
還沒有人認領這個 Issue。
評估
- 難度
- 4/5
- 預估耗時
- 3-5 天
- 新手友好度
- 68/100
- Issue 類型
- 缺陷
- 描述清晰度
- 描述清楚
- 活躍度
- 活躍
- 技術堆疊
- angular, rust, typescript
- 領域
- compilers
研究方向
Start in crates/oxc_angular_compiler/src/directive/dts_type.rs, focusing on leading_comments and same_line_comments. Compare the existing output for repro cases a–g with ngtsc 22.1.7, including U+2028/U+2029 line breaks. Done means the generated .d.ts preserves the same leading and trailing comments as ngtsc while leaving the type unchanged.
由索引模型根據 Issue 內容生成。
描述
#496 prints the transform's parameter type into .d.ts. It keeps a single-line block comment in front of a list element. ngtsc keeps more than that, and in some U+2028/U+2029 cases prints the comment twice. Only the comments in the .d.ts text differ. The type is the same.
Repro (each line is the parameter type of @Input({transform: (v: <TYPE>) => 1}) a)
| # | <TYPE> in source |
OXC (#504 tip) | ngtsc 22.1.7 |
|---|---|---|---|
| a | string /** doc */ |
string |
string /** doc */ |
| b | string | number /** doc */ |
string | number |
string | number /** doc */ |
| c | \n /** doc */\n string (comment on its own line) |
string |
\n /** doc */\n string |
| d | string | /* a\n b */ number |
string | number |
string | /* a\n b */ number |
| e | Array</* a\n b */ string> |
Array<string> |
Array</* a\n b */ string> |
| f | Array<
/* c */\nstring | number> |
Array<string | number> |
Array</* c */ /* c */ string | number> |
| g | Array<
/* c */\nstring | number> |
Array<string | number> |
Array</* c */ /* c */ string | number> |
Cases where they already match: Array</* c */ string>, string /* keep */ \| number, /* a\n b */ string (both drop it), string /* a\n b */ (both drop it), and { a: 1; // x
b: ... }.
Full source for case a:
import {Directive, Input} from '@angular/core';
@Directive({selector: '[d]'})
export class D {
@Input({transform: (v: string /** doc */) => 1}) a = 0;
}
Where
crates/oxc_angular_compiler/src/directive/dts_type.rs (added in #496): leading_comments / same_line_comments. Only the comments on the same line in front of a list element are kept. Trailing comments, comments on their own line and comments that span lines are dropped.
On main?
No. Main prints unknown for these fields. This gap comes with #496.
Direction
Copy the rules of TypeScript's emitter for leading and trailing comments on type nodes, including the U+2028/U+2029 line-break cases. Test against ngtsc output.
- 主要語言
- Rust
- 星號
- 244
- 分支
- 26
- 平均合併
- 1 天 14 小時
- 30 天內合併 PR
- 77
環境準備
這個專案沒有提供開發容器、Dockerfile 或貢獻指南，環境需要你自己搭建：先看它的 README，通用步驟見我們的新手貢獻指南。
從這裡開始
- 先讀完整個 Issue，再讀專案的貢獻指南。
- 在 Issue 下留言說明你要接手 —— 這能避免兩個人做同樣的事。
- Fork 儲存庫，在一個分支上完成修改。
- 送出 Pull Request，並在描述裡引用這個 Issue 編號。
voidzero-dev/oxc-angular-compiler 的其他 Issue
-
Support useDefineForClassFields: false (Class Field Lowering)可能已有人在做 關聯的 PR 仍在進行中或已合併。 未關閉
難度 5/5 一週以上 新手友好度 35/100
voidzero-dev/oxc-angular-compiler#73 · 11 則留言 ·
維護者通常 1 天內回覆
-
Angular 21.2.0 + @oxc-angular/vite 0.0.7 SSR runtime parity regressions (RouterLinkActive, Forms DI, Apollo DI) require app-level shims可能重新可做 @Brooooooklyn 於 235 天前認領，目前沒有進行中的 PR。 未關閉
voidzero-dev/oxc-angular-compiler#61 · 已指派 1 人 ·
維護者通常 1 天內回覆
-
Angular 21.2.0 + @oxc-angular/vite 0.0.7 SSR: AngularNodeAppEngine is unusable (missing app-engine manifest)可能重新可做 @Brooooooklyn 於 235 天前認領，目前沒有進行中的 PR。 未關閉
voidzero-dev/oxc-angular-compiler#60 · 4 則留言 · 已指派 1 人 ·
維護者通常 1 天內回覆
查看 voidzero-dev/oxc-angular-compiler 的全部 Issue
相似的 Issue
-
難度 1/5 1 小時以內 新手友好度 67/100
rust-bitcoin/rust-bitcoin#7089 · 1 則留言 · 1 個 reaction ·
維護者通常 2 天內回覆
-
area/tools bug session-analysis
難度 2/5 1-3 小時 新手友好度 78/100
維護者通常 1 天內回覆
-
[Bug]: Python OpenTelemetrySectionConfig has no file_sinks可能重新可做 關聯的 PR 已關閉且未合併。 未關閉
難度 2/5 1-3 小時 新手友好度 76/100
NVIDIA/NeMo-Relay#1514 ·
維護者通常 1 天內回覆
-
prompting proposal tool triaged
難度 2/5 1-3 小時 新手友好度 78/100
can1357/oh-my-pi#17041 · 1 則留言 ·
維護者通常 1 天內回覆
-
難度 1/5 1 小時以內 新手友好度 84/100
nix-community/fenix#262 ·