Skip to content

feat(candidate): export shared knowledge references#64

Merged
drewstone merged 1 commit into
mainfrom
fix/eval-119-candidate-ref
Jul 14, 2026
Merged

feat(candidate): export shared knowledge references#64
drewstone merged 1 commit into
mainfrom
fix/eval-119-candidate-ref

Conversation

@drewstone

Copy link
Copy Markdown
Contributor

Problem

agent-runtime had to hand-roll conversion between agent-knowledge candidate hashes and the shared AgentCandidate knowledge reference. That duplicated a knowledge-domain boundary in every consumer.

Solution

  • export two strict conversion functions from agent-knowledge
  • validate shared digests with agent-interface 0.26.1
  • update agent-eval to 0.119.1
  • stabilize the loaded offline adaptive comparison timeout

Proof

  • pnpm test: 29 files passed, 2 skipped; 281 tests passed, 10 skipped
  • pnpm typecheck
  • pnpm lint: 98 files clean
  • pnpm build
  • packed export imported and round-tripped through agentCandidateKnowledgeSchema
  • independent adversarial audit findings resolved
  • git merge-tree --write-tree origin/main HEAD succeeded

@tangletools tangletools left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✅ Auto-approved drewstone PR — f5a809a4

This PR was opened by the trusted drewstone account.
The full PR reviewer audit still runs separately and will publish findings if it detects issues.

tangletools · auto-approval · reason: drewstone_author · 2026-07-14T16:07:28Z

@tangletools tangletools left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✅ Auto-approved drewstone PR — f5a809a4

This PR was opened by the trusted drewstone account.
The full PR reviewer audit still runs separately and will publish findings if it detects issues.

tangletools · auto-approval · reason: drewstone_author · 2026-07-14T16:07:42Z

@drewstone
drewstone merged commit c662e5d into main Jul 14, 2026
1 check passed
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