Repository navigation
Skillsの追加20評価と本文照合を、3つの読取範囲で検証する - #431
Merged
Merged
Conversation
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: 1b1043d6-0fe0-4b31-be76-1ea4a0afa0d9) |
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: 20d59acb-971e-466b-ba41-a7db00863861) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
新20候補の評価を、既存21・追加45と重ならない3つ目の読取範囲として反映します。全体はraw本文474件、伏字付き手順を含め475件です。6mile内は86候補をraw84・伏字付き1・許諾文のみ1に分け、支持765件も全文/伏字付き手順/走査/保存のみ/未保存を区別します。
保存済み本文72145候補の機械照合も記録します。正規化可能72118候補とUTF8未解決27候補を分け、既採用手順と本文が一致する比較候補8件を残しました。これは意味の読了・個別許諾・ヘルパー同一性・最終重複除外の証明ではありません。未取得428と取得保持647の再取得は行っていません。
読取判定を共有化し、全3範囲のID集合・raw/workflow集計・全live採用行を横断して照合します。公開する出典行・許諾略号の展開・補正対象IDにも、原報告から独立に求めた指紋と負例を追加しました。PR430で受入れ済みの記録を閉じ、引継ぎの未採用案を本人方針へ昇格させず、全候補の扱いを保持しています。
新規32・統合3・除外8・未判定/保留73177は変わりません。この差分で中央の読書きや実採用は増やしていません。元3048/バックアップ/保留と全16資料の残作業を維持します。Related #283(依頼全体は未完)。
確認済み:
go test -count=1 ./...とgo vet ./...が成功。新20の担当/伏字付き走査/unknownbytes/コピー許諾/最終採否の負例、3範囲の重複/未評価行、公開namespace/許諾/補正IDの負例を確認。git diff --checkが成功。固定差分の別モデルレビューと、このheadのCI終了後に通常マージします。
Note
Low Risk
Changes are documentation, JSON receipts, and validation tests for the skills-3080 workflow; adoption counts and central writes are explicitly unchanged, with no application runtime or security-path edits.
Overview
This PR records a third 6mile reading cohort (next20) alongside the existing first21 and next45 batches, bumps aggregate coverage to 474 raw / 475 workflow paths (86 assessed within 6mile), and adds acceptance prose plus progress/index updates—without changing adoption totals (still 32 new / 3 integrated / 8 excluded).
It also closes PR430 public-delta review for RSS/season skills and restructures
public_delta_acceptanceacross ledgers intoaccepted_head,original_review, andcorrection_review(MCP pointer entries similarly updated).A new
skills-3080-body-alias-scan.jsoncaptures machine triage over saved catalog bodies (72k+ occurrences mapped, 27 UTF-8 pending, 8 normalized-body alias leads vs canonical adoptions), explicitly not granting reads, licenses, or final duplicate dispositions.Go catalog tests are generalized for all three cohorts (
validateReceivedReading, cohort index/projection), with new suites for next20 reception, public reading relations, and body-alias scan invariants; next45 policy corrections use emptyinventory_idsarrays instead of[null].Reviewed by Cursor Bugbot for commit b9a9694. Bugbot is set up for automated code reviews on this repo. Configure here.