Skip to content

A set of documents names its embedder and its reranker to the runtime (96 §A7) - #16

Merged
dittops merged 1 commit into
masterfrom
wp56-knowledge-models
Nov 12, 2026
Merged

dittops merged 1 commit into
masterfrom
wp56-knowledge-models

Conversation

@dittops

@dittops dittops commented Nov 12, 2026

Copy link
Copy Markdown
Member

What this is

PACT's side of Bud Agent Flow's WP-56: knowledge embedded and reranked by named models.

KnowledgeSpec dropped embedded-by: and reranked-by:. pact check already holds each one to a catalogue row of kind embedding and reranking, but no runtime could read them. It now carries embedded_by and reranked_by, which are "" when not written.

Not changed here: an agent that uses: a knowledge interface (interface: yes) still gets the interface's own entry, not the set that implements: it. The changelog says so.

Evidence

  • Python suite: 3,023 passed and 2 failed. The two failures are the environment-only pair that also fails on unmodified master on this machine.
  • New test: one; README count is now 4766.
  • No Rust change.

GitHub Actions has no budget left, so the local run is the evidence.

🤖 Generated with Claude Code

… (96 §A7)

`KnowledgeSpec` dropped `embedded-by:` and `reranked-by:`, which `pact check` already holds to a
catalogue row of kind `embedding` and `reranking`, so a runtime indexed every set with its own
embedder whatever the set said. It now carries `embedded_by` and `reranked_by` ("" when not
written). One test; README count 4766.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@dittops
dittops merged commit 0f681b0 into master Nov 12, 2026
dittops pushed a commit that referenced this pull request Nov 12, 2026
…s the claim's holder AND the reactions; counts recomputed (4822)

Both code conflicts were two additions at the same place: Suspension's
escalate copy and harness._park_state each gained a line from this
branch (claim_holder) and a line from master (reactions). Both are kept.
The counts were recomputed by scripts/sync-counts.sh: 1623 Rust + 3199
adapter.

Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com>
dittops pushed a commit that referenced this pull request Nov 12, 2026
…sts (1623 Rust + 3204 adapter)

scripts/sync-counts.sh, on this branch with 907f292 (#14–#16) and the
updated claims branch merged into it. The merge itself had no conflict.

Co-Authored-By: Claude Fable 5.2 <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.

2 participants