feat(gh-egress,idd-comment): reply layer-3 third-party payload tier floor (v2.101.0, #272)#274
Merged
Merged
Conversation
…loor (v2.101.0) Proportionality-scoped floor: layers 1-2 re-quote content already on this repo's remote (tier default kept); layer 3 (user-pasted external text) is the one channel where NEW third-party verbatim content first reaches the remote. - rules/privacy-scrubbing.md: normative floor section — LIGHT does not apply to points-from=user-pasted replies regardless of repo visibility; minimum WARN + explicit user confirmation; unattended contexts refuse to post. - SKILL R1: attended AskUserQuestion confirm (redact option) / unattended refuse — replaces the v2.100.0 "heightened self-review" prose that #269 verify DA-3 judged necessary-but-insufficient. - gh-egress.sh net item 4 (3->4, the separate change the rules file requires): type=reply + points-from=user-pasted marker tokens at attested light -> exit 13 (attestation band). IDD-own marker tokens only, zero semantic matching; SKILL confirm is the primary gate, wrapper is belt-and-suspenders. Tests: gh-egress 60 (RED-verified: floor assertion red before item 4 landed), idd-comment-reply 33, aggregator 40 suites 0 fail. Version 2.101.0 synced in plugin.json + marketplace.json + CHANGELOG. Refs #272
…t — verify round 1 6-AI verify round 1 (4 lens + Codex + DA) findings, all folded: - HIGH (Codex + DA concur): SKILL R1 read "after confirmation dispatch at --scrub-attested warn" — a pinned level that would silently DOWNGRADE a third-party repo's ENFORCE tier. Rewrote as floor semantics: effective tier = max(repo-derived tier, warn); ENFORCE never downgrades, block-with-diff runs unchanged. Spec: warn-proceed scenario qualified to LIGHT/WARN repos + new enforce-preservation scenario. - HIGH/MED (Codex + DA): layer-1 "already on same repo remote" premise was unguaranteed — a cross-repo comment URL is a new cross-audience disclosure. SKILL + spec: layer-1 exemption requires the URL to belong to the destination repo; cross-repo URLs resolve as layer 3 (marker records user-pasted) and carry the floor. New spec scenario. - MED: rules/header reclassification — items 1-3 are level-independent zero-tolerance nets; item 4 is a level-dependent tier-floor attestation check (the blanket "LEVEL-INDEPENDENT"/"zero-tolerance" claims no longer held). gh-egress header now enumerates all 4 items. - MED: honest raw-substring wording (two fixed-string token matches over raw prose, fences not stripped, NO backtick escape hatch — the entity-@ parity claim was inaccurate on that dimension) + accepted-friction LOCK test (fenced discussion body at light → 13). - MED (DA N-2): SKILL Step 4 now takes the floor max() before attesting — the SKILL side is the primary gate for own-private, not just the wrapper bounce. MED (DA N-4): template↔wrapper token binding assertions (silently- disabled-net risk). Conjunction negative fixture (user-pasted without type=reply → dispatch) + stderr remediation assertion. - LOW: grep -Fq -- hardening; "body needs no redaction" wording softened; design/tasks append-only + R1/R4 claims corrected to match reality. Tests: gh-egress 66, idd-comment-reply 35, aggregator 40 suites 0 fail. Refs #272
Member
Author
Verify Report — PR #274 (Refs #272)Enginemanual fan-out (4 lens Agents + sequenced Devil's Advocate, model: opus, file-based output) + Codex (gpt-5.6-sol, xhigh, via codex-call) AggregatePASS(after round-1 fix 4 lens 初判 PASS/shippable;跨模型(Codex)與 Devil's Advocate 抓到 4 lens 全部漏掉的 HIGH——本輪是 cross-model + adversarial 兩層各自發揮價值的案例(DA 並指名 Logic lens 曾看到該字句卻誤標 NIT)。 Findings(合併去重)
Scope checkBranch 兩 commit(feat 驗證數字
Follow-up Findings Filed (v2.72.0+ #148)
Process notes
|
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.
Refs #272
Summary
reply layer-3(
points-from=user-pasted)第三方 verbatim payload 的 scrub tier floor —— #269 verify DA-3 的正式修補。比例原則:layer 1/2 重引本 repo remote 既有內容(零新增暴露、維持 repo-tier 預設);layer 3(user-pasted)是新第三方逐字內容首次進 remote 的唯一通道,floor 只綁這裡。三件套:
rules/privacy-scrubbing.mdnormative 段——LIGHT 不適用於 user-pasted reply;有效 tier =max(repo-derived tier, WARN)(ENFORCE 永不因確認而降級);unattended 不 postidd-commentSKILL R1——attended AskUserQuestion 顯式確認(附 redact 選項)/ unattended refuse;跨 repo comment URL 視同 layer 3;Step 4 先取 floor max() 再 attestgh-egress.shnet item 4(3→4,rules 檔要求的 separate change)——type=reply+points-from=user-pastedtoken 且 attested=light→ exit 13(attestation band)。IDD 自產 marker token 的 fixed-string 匹配、零 semantic matchingVerification
6-AI cross-model verify round 1(4 lens opus + Codex gpt-5.6-sol + sequenced Devil's Advocate)→ 4 lens 初判 PASS,Codex + DA 抓到 1 HIGH(SKILL 把 floor 寫成 pinned warn——third-party repo 的 ENFORCE 會被降級)+ layer-1 same-repo 前提缺口+4 MED,全部已在
763c1b5修復並補測試鎖住。詳見 PR 的 verify master comment。Checklist
Related
🤖 Generated by /idd-all. Do NOT add a GitHub close trailer(Closes/Fixes/Resolves)—— IDD discipline requires manual /idd-close after merge.