NAS-142127 / 27.0.0-BETA.1 / Add Rsync Task Connect using option data-test shows numbers #3086
AlexKarpov98claude.yml
on: pull_request
review
/
...
/
Check member access
5s
review
/
Automatic PR review
9m 58s
Annotations
2 warnings and 3 notices
|
review / check-member / Check member access
Node.js 20 is deprecated. The following actions target Node.js 20 but are being forced to run on Node.js 24: actions/github-script@v7. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
review / Automatic PR review
Node.js 20 is deprecated. The following actions target Node.js 20 but are being forced to run on Node.js 24: actions/checkout@v4. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
review / Automatic PR review:
src/app/modules/forms/ix-forms/constants/tn-select-option-test-id.constant.ts#L46
LOW: The invariant is duplicated across 293 template bindings and 127 identical protected readonly optionTestIdByLabel fields with no shared abstraction
|
|
review / Automatic PR review:
src/app/modules/forms/ix-forms/constants/tn-select-option-test-id.constant.spec.ts#L23
LOW: The guard scans only .html files, so a tn-select in an inline template: escapes it, while SKILL.md:735 claims it scans every template
|
|
review / Automatic PR review:
src/app/modules/forms/ix-forms/constants/tn-select-option-test-id.constant.spec.ts#L11
LOW: findRepoRoot() has no filesystem-root stop condition, so a missing package.json would hang the jest worker instead of failing
|