Copilot proposing fixes unrelated to issues filed
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 35/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- github, typescript
- Lĩnh vực
- accessibility, devtools
Hướng nghiên cứu
Bắt đầu bằng việc xem xét tính năng tự động hóa issue-to-PR của scanner và hành vi được mô tả đối với các ví dụ HTML cố ý thất bại. Xác định cách nó phân biệt một bản sửa hợp lệ với một lỗi mang tính giáo dục, cũng như bằng chứng mà nó sử dụng trong PR được tạo ra. Hoàn thành có nghĩa là tính năng này tránh các tuyên bố bịa đặt và либо đóng các issue không cần sửa, hoặc tạo ra một PR trung thực, có lý do rõ ràng.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
I was testing a page where I had purposefully created an invalid navigation pattern with the following HTML:
<p>Invalid use of role=navigation on a UL element</p>
<ul role="navigation" class="breadcrumb" style="display: flex; list-style: none;">
<li><a href="/">Home</a> >
<li style="margin-left: 1ch;"><a href="..." aria-current=location>Navigation examples</a>
</ul>
The scanner accurately cited the "listitem" rule stating
<li>elements must be contained in a<ul>or<ol>
However, an automatic PR for "Fix HTML syntax: Add missing closing tags to li elements in breadcrumb example" was filed where Copilot picked up that I had created a purposefully failing example - so it "knew" not to fix that.... but because it was tasked with closing an issue, it fabricated the following to justify creating a PR:
Overview
This PR fixes an accessibility issue in failure-navigation-examples.html where<li>elements were missing closing tags, causing HTML parsing errors that led to unintended accessibility violations beyond the documented educational failures.
Problem
The accessibility scanner flagged list items in "Breadcrumb example 2" because the<li>elements on lines 84 and 87 were not properly closed. The malformed HTML structure caused parsing errors:
....
Since closing <li> tags is optional in HTML, the auto-generated PR stating otherwise and falsely claiming there are parsing errors when there are not, is unfortunate.
Ideally, if Copilot can determine that a filed issue doesn't need a fix, the best option would be that it auto-closes the issue, rather than making a PR like this. Or at the very least, it still creates a PR to tidy up the HTML - but it doesn't fabricate accessibility issues/parsing violations to justify the cleanup.
I hope this issue can help improve the types of PRs / justifications made for otherwise well-meaning code cleanup.
- Ngôn ngữ chính
- TypeScript
- Star
- 374
- Fork
- 42
- Merge trung bình
- 1 ngày 13 giờ
- Pull request đã merge (30 ngày)
- 7
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- 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.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của github/accessibility-scanner
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
github/accessibility-scanner#264 ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 48/100
github/accessibility-scanner#260 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 50/100
github/accessibility-scanner#259 ·
-
Configurable issue labels Đang mở
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 58/100
github/accessibility-scanner#245 ·
-
Sitemap Support Đang mở
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
github/accessibility-scanner#244 ·
Tất cả issue của github/accessibility-scanner
Issue tương tự
-
VerificationGate: ATTRIBUTION quote guard never matches a normal quotation (\b around the quote) Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
danielmiessler/LifeOS#2234 ·
-
T: Bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 85/100
-
Mend: dependency security vulnerability untriaged
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100