Skip to content

Fix performance attention runtime interop#1338

Open
Gujiassh wants to merge 2 commits intoruvnet:mainfrom
Gujiassh:fix/attention-export-mismatch
Open

Fix performance attention runtime interop#1338
Gujiassh wants to merge 2 commits intoruvnet:mainfrom
Gujiassh:fix/attention-export-mismatch

Conversation

@Gujiassh
Copy link

Summary

  • adapt the performance package to the CommonJS class exports that @ruvector/attention actually ships instead of the stale type surface exposed by its declarations
  • route benchmark and quick-test code through the local compatibility layer so the package, examples, and tests all agree on one runtime-safe import path
  • keep the change isolated to @claude-flow/performance, separate from the open PluginManager and benchmark-environment PR lines

Validation

  • cd v3 && npx vitest run @claude-flow/performance/tests/attention.test.ts @claude-flow/performance/tests/benchmarks.test.ts
  • cd v3/@claude-flow/performance && npm run build
  • lsp_diagnostics clean on attention-integration.ts, attention-benchmarks.ts, and examples/quick-test.ts

Notes

  • the installed @ruvector/attention package still ships root typings that do not describe its real runtime exports; this PR fixes the performance package without modifying the external dependency itself

Gujiassh and others added 2 commits March 12, 2026 20:39
Adapt the performance package to the CommonJS class exports that  actually ships so benchmark code no longer relies on stale type declarations.

Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-opencode)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
Point the quick integration example at the local compatibility layer so the example keeps working with the same runtime-safe attention exports as the package itself.

Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-opencode)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
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