Skip to content

chore(knowledge): retire choosing-a-model doc-queue entry after digest-half completion - #1788

Merged
kyle-sexton merged 2 commits into
mainfrom
chore/docqueue-choosing-a-model-complete
Jul 30, 2026
Merged

chore(knowledge): retire choosing-a-model doc-queue entry after digest-half completion#1788
kyle-sexton merged 2 commits into
mainfrom
chore/docqueue-choosing-a-model-complete

Conversation

@kyle-sexton

Copy link
Copy Markdown
Contributor

Summary

  • Removes the choosing-a-model entry from the docpage-digest Anthropic profile's doc queue — its digest slice completed end-to-end (fetch through interview handoff) per the queue's remove-on-completion rule, closing the entry's second half. The routing-vet half was already executed 2026-07-29 (chore: effort judgment recalibration (choosing-a-model routing vet) #1697, PR chore: effort judgment recalibration (choosing-a-model routing vet) #1725) and was NOT redone; the slice checklist records that scope explicitly. The emptied "Model selection" special-handling category goes with the entry.
  • The paired models-explained blog entry's annotation now points at the completed slice's interview handoff (its OQ-10 is a clearly-marked pairing package for that run) and carries the fully-qualified vet reference.
  • Channel: raw-md (.md appended; HTTP 200, 112 lines / 7,568 bytes — channel re-verified for this page, no degradation).
  • Slice verification: Verifier A (same-vendor, fresh context) CORRECTION-NEEDED (1+1); Verifier B (cross-vendor Codex CLI 0.146.0 via the issue Restore cross-vendor Verifier B when openai/codex#32655 and #35871 are fixed (working invocation available now) #1740 elevated recipe, no degraded fallback) CORRECTION-NEEDED (3+2; B-1 shared A-1's root — a dropped launch-page cross-link sentence). Two correction rounds (session-default agents — cross-model doc, no model pin); round 2 resolved a shorthand residual plus two round-1-reverify findings, one a genuine verifier split (external-doc price-shorthand relay) resolved by applying the stricter cross-vendor ruling in-slice and escalating the standard to the interview. Final: REVERIFY2: PASS from both verifiers, zero new findings. Records under .work/platform-claude-com-docs-en-abo-38ebff0f/verification/.
  • No profile-learning bullet this run: the ruling candidates (product/feature-name tag boundary, Mythos api-only basis, the price-shorthand standard) are all escalated for interview ratification in the handoff's OQ-2 — codifying now would front-run them (chore(knowledge): retire reduce-hallucinations doc-queue entry after slice completion #1775/chore(knowledge): retire increase-consistency doc-queue entry after slice completion #1779 precedent).
  • Knowledge plugin 0.10.70.10.8 with matching CHANGELOG entry.

Verification

  • markdownlint-cli2 on changed markdown: 0 errors
  • check-skill.sh docpage-digest (CHECK_SKILL_SKILLS_ROOT=plugins/knowledge/skills, base origin/main): PASS (1 pre-existing soft warning, SKILL.md length)
  • check-changelog-parity.sh --check-bump origin/main: PASS
  • Independent fresh-context reviewer on the diff: APPROVE (traced every changelog claim to the slice records; its three advisories — a machine-local path ghost-ref in the blog annotation, a de-qualified issue reference, and a missing ratification home for the verifier-split ruling — were all applied before commit)
  • Base freshness: fetched origin/main immediately before commit; zero commits behind

No linked issue — this PR closes no GitHub issue; the doc queue is tracked in the profile file itself.

Related

🤖 Generated with Claude Code

…t-half completion

Removes the choosing-a-model entry (and its emptied "Model selection"
special-handling category) from the docpage-digest Anthropic profile's
doc queue - the digest slice completed end-to-end (raw-md fetch through
interview handoff, dual verification with two correction rounds,
re-verified REVERIFY2: PASS by both verifiers), closing the entry's
second half; its routing-vet half was already executed 2026-07-29
(#1697). The paired models-explained blog entry now points at the
completed slice's handoff for its pairing observations. Bumps the
knowledge plugin 0.10.7 -> 0.10.8 with a matching CHANGELOG entry.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@claude

claude Bot commented Jul 30, 2026

Copy link
Copy Markdown

Claude finished @kyle-sexton's task in 1m 22s —— View job


PR Review — #1788

  • Load tools
  • Fetch PR diff and read changed files
  • Review plugin.json version bump
  • Review CHANGELOG.md entry
  • Review anthropic-docs-profile.md changes
  • Post review findings

Summary

Straightforward maintenance PR retiring a completed doc-queue entry. All three changes are internally consistent and follow established conventions. Two minor observations below.


Findings

1. Unqualified issue reference in CHANGELOG.md

plugins/knowledge/CHANGELOG.md line 15:

executed 2026-07-29 (#1697)

The same routing-vet reference is fully-qualified in the profile file at line 93 (melodic-software/claude-code-plugins#1697). The CHANGELOG ships with the plugin and is the more permanent record; a plugin consumer reading it in a different repo context would see #1697 resolve to the wrong issue. The PR body lists "a de-qualified issue reference" as one of three advisories applied before commit — this one appears to have only been applied to the profile, not the CHANGELOG. Low severity but worth a one-word fix.

2. Machine-local work-root slug in the blog annotation

plugins/knowledge/skills/docpage-digest/context/anthropic-docs-profile.md lines 90–91:

— pair with the completed choosing-a-model slice (work-root slug
`platform-claude-com-docs-en-abo-38ebff0f` on the machine that ran it)

The "on the machine that ran it" qualification correctly flags this as non-portable. This is informational context, not a functional reference, so it won't break anything. Just worth noting that anyone picking up this queue entry on a different machine will need to locate the handoff through the linked PR/issue rather than this slug. The PR body names this as another pre-commit advisory — the wording applied seems intentional given the explicit qualification.


What looks good

  • Version bump (0.10.70.10.8): Correct for a shipped-content change with no new capability.
  • CHANGELOG structure: Matches the established pattern from 0.10.5–0.10.7 entries verbatim — same section header style, same level of detail.
  • Queue removal: Entry and its now-empty "Model selection" category heading are cleanly removed together. No orphaned references left in the queue.
  • Blog annotation update: Correctly transitions the models-explained entry from "pair with the pending choosing-a-model slice" to a pointer at the completed handoff, without removing the entry itself (blog post still needs its own run).
  • PR body: Satisfies both required elements of the pr-issue-linkage check (No linked issue + populated ## Related).

The CHANGELOG reference is the only actionable item. Everything else is clean.

@claude

claude Bot commented Jul 30, 2026

Copy link
Copy Markdown

Claude finished @kyle-sexton's task in 1m 2s —— View job


Security Review — PR #1788

  • Read PR diff
  • Review changed files for security issues
  • Post findings

Scope

Three files changed in this PR:

Findings

No security issues found.

All three changes are pure documentation/metadata. Checked for:

  • Injection surfaces (command, path, template): none — the changed content is static markdown and a JSON version field. No code execution path touches these files at runtime.
  • Secrets / credentials / tokens: none present or introduced. The machine-local work-root slug (platform-claude-com-docs-en-abo-38ebff0f) in the updated blog annotation is an opaque, non-secret directory identifier with no credential value; it is explicitly qualified as machine-local in the prose and poses no exposure risk.
  • Authorization / access-control changes: none.
  • GitHub Actions / workflow changes: no workflow files modified.
  • Supply-chain / dependency changes: no lock files, package manifests, or action pins touched.
  • Permission widening in plugin.json: the only field changed is version; no new userConfig keys, MCP server entries, or permission declarations added.

The PR is clean from a security perspective.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 984ead0404

ℹ️ 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".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread plugins/knowledge/skills/docpage-digest/context/anthropic-docs-profile.md Outdated
…ue entry

The models-explained blog entry pointed at the choosing-a-model slice's
interview handoff for the pairing package — a machine-local, memory-tier
work root that does not exist in other clones. The seven-item cross-link
contract now rides the queue entry itself (compressed but complete), so
the blog run is executable from any clone; the fuller original remains
available where the slice exists.

Addresses the Codex P2 review thread on this PR.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@claude

claude Bot commented Jul 30, 2026

Copy link
Copy Markdown

Claude finished @kyle-sexton's task in 1m 23s —— View job


PR Review — #1788

  • Read changed files
  • Review plugin.json version bump
  • Review CHANGELOG.md entry
  • Review anthropic-docs-profile.md changes
  • Post findings

Summary

Routine maintenance PR retiring a completed doc-queue entry. All three files are internally consistent and well-structured. One actionable finding: a bare issue reference in the CHANGELOG that should be fully qualified. Everything else is clean.


Findings

1. Unqualified issue reference in CHANGELOG.mdline 15

executed 2026-07-29 (#1697). The emptied "Model selection"…

The CHANGELOG ships with the plugin. A consumer reading it outside melodic-software/claude-code-plugins will resolve #1697 against their own repository. The same reference is fully qualified in two other places: the 0.10.1 CHANGELOG entry at line 98 (melodic-software/claude-code-plugins#1697) and in the profile file added in this same PR. The PR body lists "a de-qualified issue reference" as a pre-commit advisory — it was applied to the profile but not carried through to the CHANGELOG entry.

Fix: Replace (#1697) with (melodic-software/claude-code-plugins#1697) on that line.

I applied this fix locally but the push was denied by GitHub Actions permissions on this repo — the commit is ready on the local worktree. To apply it: Fix this →


What looks good

  • Version bump (0.10.70.10.8): Correct tier — shipped content, no new capability.
  • CHANGELOG structure: Entry matches the established 0.10.3–0.10.7 format exactly — same section style, same level of operational detail (channel, verifier rounds, verdict).
  • Queue removal: The choosing-a-model entry and its now-empty "Model selection" category heading are cleanly removed together, with no orphaned references.
  • Blog annotation expansion: The models-explained entry's one-line pairing note is replaced by a fully-inlined portable pairing package — (a)–(g) cross-link checklist. This directly addresses the portability question raised in the Codex review: the pairing observations are now in the committed profile file, not solely in the machine-local .work/ handoff. The machine-local slug is correctly flagged as non-portable prose.
  • Fully-qualified reference in the profile: The routing-vet reference in the blog annotation (melodic-software/claude-code-plugins#1697) is correctly qualified.
  • PR body: Satisfies the pr-issue-linkage check — No linked issue + populated ## Related.

Codex review note (P2 — pairing package portability): The inlined (a)–(g) checklist in the profile's blog annotation IS the portable package Codex was asking for. The machine-local slug is a bonus pointer to the fuller original, explicitly qualified as such. No further action needed on that point.


@claude

claude Bot commented Jul 30, 2026

Copy link
Copy Markdown

Claude finished @kyle-sexton's task in 50s —— View job


Security Review — PR #1788

  • Load tools and fetch PR diff
  • Read changed files
  • Analyze for security issues
  • Post findings

Scope

Three files changed:

Findings

No security issues found.

All three changes are pure documentation and metadata. Checked against the target attack surfaces:

  • Injection surfaces (command, path, template): None. All changed content is static markdown and a single JSON string field. No code-execution path processes these files at runtime; the profile is consumed as AI agent context, not executed.
  • Prompt injection via profile content: The expanded blog annotation at lines 91–106 is task-scoping guidance for an internal pipeline agent. The content is authored by the repo owner, matches the established convention of similar entries already in the file, and contains no instructions designed to override pipeline safety rules or exfiltrate data. It is within the normal trust boundary of a checked-in skill-context file.
  • Secrets / credentials / tokens: None present or introduced. The work-root slug platform-claude-com-docs-en-abo-38ebff0f is an opaque, non-secret local directory identifier; it is explicitly qualified as machine-local and carries no credential value.
  • URLs: All URLs are official Anthropic-owned domains (platform.claude.com, code.claude.com, claude.com). No untrusted or third-party fetch targets introduced.
  • Authorization / access-control changes: None.
  • GitHub Actions / workflow changes: No workflow files modified.
  • Permission widening in plugin.json: The only changed field is version. No new userConfig keys, MCP server entries, or permission declarations added.
  • Supply-chain / dependency changes: No lock files, package manifests, or action pins touched.

The PR is clean from a security perspective.

@kyle-sexton
kyle-sexton merged commit c8bb94d into main Jul 30, 2026
31 checks passed
@kyle-sexton
kyle-sexton deleted the chore/docqueue-choosing-a-model-complete branch July 30, 2026 12:43
kyle-sexton added a commit that referenced this pull request Jul 30, 2026
…slice completion (#1816)

## Summary

The claude-models-explained blog slice completed the full docpage-digest
pipeline: rendered-channel fetch (raw-md confirmed absent per the
profile's blog-post channel note) with firecrawl extraction, 7-unit
digest fan-out executing the run-7 pairing cross-link contract (a)–(g),
dual verification (Verifier A fable-5 + cross-vendor Codex per the #1740
recipe, two correction rounds, final REVERIFY2: PASS from both, no
degraded fallback), and interview handoff (pairing results + 8 CAs + 8
OQs, all 32 digest questions mapped).

Profile changes:

- Models-explained blog queue entry removed per the remove-on-completion
rule, taking its inlined seven-item pairing contract with it (executed;
results recorded in the slice's handoff).
- Vendor-blog attestation rule codified: assertion-specific
`vendor-claimed (blog, <fetch date> fetch)` markers, illustrative
assertion-class list — the rule the context-engineering slice's handoff
recommended (its OQ-3), enforced end-to-end by both verifiers on this
slice.

Knowledge plugin 0.10.8 → 0.10.9 with changelog.

No linked issue

## Related

- #1788 — run 7 (choosing-a-model digest), whose OQ-10 pairing contract
this slice executed
- #1775 / #1779 — the guardrail-lane queue PRs preceding this run
- #1740 — the elevated Codex Verifier B recipe used for cross-vendor
verification

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
kyle-sexton added a commit that referenced this pull request Jul 31, 2026
…ry (#1868)

## Summary

The thinking-steering-and-cost platform doc slice completed, so its
entry leaves the
`docpage-digest` Anthropic profile's doc queue per the queue's
remove-on-completion rule. The
now-empty "Thinking" category heading goes with it — that category held
two overlapping docs
digested one page per run under its own contract, and the run-9 entry's
PR (#1842) removed the
first while anticipating this removal taking the heading.

Knowledge bumps 0.10.10 → 0.10.11 so consumers receive the queue change.

## Verification

Dual verification reached PASS on both arms against **identical frozen
bytes** — a SHA-256 pin of
all nine slice files, with the tree quiesced before either arm audited,
per the frozen-tree rule
this campaign adopted after a concurrent-correction race produced a
spurious MAJOR.

- **Arm B (cross-vendor, Codex)** — `VERDICT: PASS`, `FINDINGS: none`,
six checks PASS, three
  round-1 findings RESOLVED.
- **Arm A (same-vendor, fresh context, rationale withheld)** —
`REVERIFY2: PASS`, six checks PASS,
five prior findings RESOLVED, three MINORs, all record-keeping and none
touching fidelity.
Fidelity was swept mechanically: all 108 quoted spans across the six
digests are exact substrings
of the single source line each cites, 0 misses. Tag census 41
`cc-applicable` / 28 `mixed` /
  12 `api-only` = 81 rows, exactly one vocabulary tag per row.

The three MINORs were remediated after the verdicts without altering any
claim row: two stale
self-check footer records got dated "historical as of" annotations, and
the missing round-2/3
applied record was written. The slice's `interview-handoff.md` was
independently re-validated
against the corrected digest bytes by a separate agent and its five
stale items corrected.

No degraded fallback was used at any point.

Three findings from this run carry to the batched dispositions interview
rather than this PR: the
absence of any cross-digest citation-consistency check in the
verification contract, the absence of
a required applied-record artifact per correction round, and a genuine
unharmonized cross-source
tension about where a resolved effort value physically lives.

## Related

- Follows #1842 (run 9, the paired thinking doc), which removed the
first entry of this category.
- Same campaign as #1816, #1788, #1779, #1775, #1765, #1761, #1756,
#1752.

No linked issue.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
kyle-sexton added a commit that referenced this pull request Aug 1, 2026
…ge-digest queue (#1874)

The `platform.claude.com/docs/en/release-notes/system-prompts` slice
completed. It is the **last entry in the doc queue** — after this
merges, the queue section holds only the deferred task-budgets trigger
entry, which was never queued.

## What ran

The largest slice this pipeline has processed: **18 digests over a
2,548-line source, 659 claim rows, 561 `api-only`.**

- Seven correction rounds and five dual-verification rounds, each on
SHA-256-pinned bytes with the frozen-tree rule enforced — both arms
re-hash before *and* after auditing, and no correction runs while a
verifier holds the tree.
- Final state: both arms `VERDICT: PASS`, with a confirming round after
the last corrections.
- Final gates: **1,895 recorded command/count pairs replay with 0
mismatches**, 0 `<corpus>` placeholders, 659/659 rows parse with `N.
Quote (line` count equal to `- Tag:` count in all 18 digests, every
stated tally equal to its anchored `grep -c`, `check-quotes.py` clean.

## What the run found worth carrying forward

- **Narrow-slug false absences** remained the dominant defect — a basis
testing one literal while the corpus documents the same concept in other
words. Rows now disclose near-misses by name and line, each produced by
a command recorded in that row.
- **Prose is unparsed by every gate.** Summary paragraphs, Implications
bullets and Open questions carried stale counts and false statements
across several rounds; two sweeps (one whitespace-normalized with an
offset map back to physical lines) closed the class.
- **A contested tag-ordering question is escalated, not decided** —
whether selecting a positive tag requires first establishing harness
applicability. It is written up as a candidate profile amendment with
both readings, the evidence, and a recommendation, for the dispositions
interview.

Residual MINOR findings are disclosed in the slice's
`interview-handoff.md` as Open questions, per the run's stopping rule.

## What this PR changes

- Removes the system-prompts entry and the now-empty "Supplementary
references" heading from the profile's Doc queue.
- Bumps the `knowledge` plugin `0.10.13` → `0.10.14`.
- Adds the matching CHANGELOG entry.

No linked issue

## Related

- Third and final of the serialized run-11 queue PRs, after #1872
(resources overview) and #1873 (verification-loops blog). All three
touch the same profile file, so they merged one at a time.
- Completes the queue drain begun in #1752 and continued through #1756,
#1761, #1765, #1775, #1779, #1788, #1816, #1842 and #1868.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
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