[docs-scanner] Contradictory information about MCP approval fields in audit records

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#26,559 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

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

Đánh giá

Độ khó
1/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
88/100
Loại issue
Tài liệu
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
markdown
Lĩnh vực
documentation

Hướng nghiên cứu

Đọc content/manuals/ai/governance/reference/audit-records.md, tập trung vào phần “Approval fields” và bảng trong phần đó. Làm rõ trường nào áp dụng cho phê duyệt mạng và trường nào áp dụng cho phê duyệt MCP, đồng thời giữ cho các mô tả nhất quán với phần văn xuôi. Hoàn thành khi người đọc có thể biết cần mong đợi những trường nào cho từng loại phê duyệt.

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

Mô tả

File: content/manuals/ai/governance/reference/audit-records.md

Issue

The "Approval fields" section contains contradictory information about which fields appear in MCP approval records. The prose states:

For MCP approvals, an approved request is evaluated again and recorded with AUDIT_DECISION_APPROVAL_ALLOW, and a declined request is recorded with AUDIT_DECISION_APPROVAL_DENY. Neither record includes approval.approval_request_id or approval.grant_scope.

However, the table immediately below lists:

Field Description
approval.approval_request_id ID that links an approval required record to the record that resolves it.
approval.grant_scope Scope the user accepted. Present only on AUDIT_DECISION_APPROVAL_ALLOW records.

The table descriptions don't distinguish between network and MCP approvals, suggesting these fields are present on all AUDIT_DECISION_APPROVAL_ALLOW records, which contradicts the prose statement that MCP approvals don't include them.

Why this matters

A reader trying to parse MCP approval audit records will be confused about which fields to expect. If they follow the prose, they won't look for these fields. If they follow the table, they'll expect them to be present. This makes it impossible to correctly process MCP approval records.

Suggested fix

Clarify the table to indicate which fields apply to network approvals vs. MCP approvals. For example:

Field Description
approval.approval_request_id ID that links an approval required record to the record that resolves it. Present for network approvals only.
approval.grant_scope Scope the user accepted. Present only on network approval AUDIT_DECISION_APPROVAL_ALLOW records.

Or restructure the section to have separate subsections for network approval fields and MCP approval fields.

Found by nightly documentation quality scanner

Ngôn ngữ chính
Markdown
Star
4.7k
Fork
8.5k
Merge trung bình
2 ngày 13 giờ
Pull request đã merge (30 ngày)
139

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

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 docker/docs

Tất cả issue của docker/docs

Issue tương tự

Thêm issue về Documentation

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.