Skip to content

fix(discovery): exempt flagged first-party content claims from the single-publisher cap - #6442

Merged
kyle-sexton merged 3 commits into
mainfrom
fix/discovery-single-source-followup
Oct 5, 2026
Merged

kyle-sexton merged 3 commits into
mainfrom
fix/discovery-single-source-followup

Conversation

@kyle-sexton

Copy link
Copy Markdown
Member

No related issue: follow-up to #6389, salvaged from #6352

Summary

#6389 made the /discovery:research single-source first-party content-claim exception vendor-neutral, but left the single-publisher rule as it was. A claim about what a vendor's own docs page says has one source whose pool is the vendor, so under main's wording it still reads as a single-publisher fact: the verifier fails row 4 when it lacks subject_pool, and the cap holds it at MEDIUM. #6352 carried the fix for that interaction; this PR ports only that part onto main's current wording.

Fix

  • discipline.md: a flagged content claim's subject is the artifact, so it carries no subject_pool and the single-publisher cap does not apply; the single-publisher definition excludes it, and a claim carries single_source: or subject_pool:, never both. The exception's artifact list adds the publisher's own help text.
  • agents/research-verifier.md: row 4 exempts a flagged content claim from the cap, and a claim carrying both keys fails row 4.
  • artifact-shape.md: omit subject_pool on a HIGH (single source) content claim.
  • gotchas.md: the repost trap names a first-party content claim, not a single-publisher claim.
  • evals.json: eval 33's expected output says "first-party file"; new eval 34 covers a non-Anthropic first-party content claim.
  • contract.test.sh: four assertions for the above, plus an absence check for "what a named Anthropic page/file".
  • discovery 0.28.9 -> 0.28.10 with one CHANGELOG line.

Verification

  • bash plugins/discovery/scripts/contract.test.sh: all contract assertions passed, including the five new ones.
  • bash scripts/validate-plugins.sh --only discovery: all plugin manifests and the catalog validated.

Related

🤖 Generated with Claude Code

…ngle-publisher cap

#6389 made the single-source content-claim exception vendor-neutral but
left the single-publisher rule unchanged, so a vendor's own docs page
(pool = vendor = apparent subject) still read as a single-publisher fact
needing subject_pool and capped at MEDIUM. A flagged content claim's
subject is the artifact: it carries no subject_pool, the cap does not
apply, and a claim carries single_source: or subject_pool:, never both.
The exception also covers a publisher's own help text. Verifier,
sidecar schema, gotchas, evals and contract test follow.

Salvaged from #6352.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@kyle-sexton
kyle-sexton marked this pull request as ready for review October 5, 2026 02:25
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 5, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-05T02:27:57.236944Z 50c29f7 Draft marked ready
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@claude

claude Bot commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

Claude encountered an error after 35s —— View job


Review in progress

  • Gather context (diff, CLAUDE.md, REVIEW.md)
  • Review changed files for correctness/alignment
  • Post findings (inline + summary)

@claude

claude Bot commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

Claude encountered an error after 22s —— View job


Security review in progress

  • Load /review:security-review instructions
  • Gather PR diff and context
  • Check skip gate (instruction-surface deletion lens)
  • Review changed files for security-relevant issues
  • Report findings

@claude

claude Bot commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

Claude encountered an error after 0s —— View job


I'll analyze this and get back to you.

@claude

claude Bot commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

Claude encountered an error after 0s —— View job


I'll analyze this and get back to you.

@kyle-sexton
kyle-sexton merged commit b103137 into main Oct 5, 2026
20 of 22 checks passed
@kyle-sexton
kyle-sexton deleted the fix/discovery-single-source-followup branch October 5, 2026 02:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant