Skip to content

feat: add weaviate connector#78

Merged
andrei-hasna merged 3 commits into
mainfrom
open-connectors-cursor/open-connectors/8806f5af-4d21-440e-9c1f-2cdceee4d9b1-4ed17187
Jul 7, 2026
Merged

feat: add weaviate connector#78
andrei-hasna merged 3 commits into
mainfrom
open-connectors-cursor/open-connectors/8806f5af-4d21-440e-9c1f-2cdceee4d9b1-4ed17187

Conversation

@andrei-hasna

Copy link
Copy Markdown
Contributor

Summary

  • Add @hasna/connect-weaviate package under connectors/weaviate/ for self-hosted Weaviate REST API ({host}/v1) with optional Bearer API key auth
  • Implement schema, object CRUD, GraphQL query, near-text search, and node status methods plus matching CLI commands
  • Register weaviate in AI & ML connector registry with focused client unit tests

Test plan

  • cd connectors/weaviate && bun install && bun run typecheck && bun run build && bun test (6 pass)
  • bun run typecheck at repo root
  • bun run build && bun test at repo root (1192 pass; 1 pre-existing events/webhooks CLI test failure unrelated to this change)
  • bun run dev --help in connector dir
  • Grep new tree for secrets/internal refs/browser-use (clean)

Add self-hosted Weaviate REST connector with schema/object/GraphQL
commands, host+Bearer auth, registry entry, and client unit tests.
@andrei-hasna

Copy link
Copy Markdown
Contributor Author

Blocked from merge after local validation and adversarial review.

Findings:

  • Blocker: the published package would have a broken library entrypoint. connectors/weaviate/package.json advertises ./dist/index.js and ./dist/index.d.ts via exports, main, and types, but connectors/weaviate/.gitignore ignores dist/, and the current build only uses bun build, which does not emit declaration files. I reproduced this with bun install && bun run build && bun pm pack --dry-run inside connectors/weaviate: the dry-run package includes bin/index.js and source files, but omits dist/index.js and any dist/index.d.ts. The README documents import { Weaviate } from '@hasna/connect-weaviate', so this would be user-visible after publish.
  • High: connectors/weaviate/.npmrc only sets @hasna:registry=https://registry.npmjs.org/ and does not include the repo-required npm token placeholder line using ${NPM_TOKEN}.

Validation that passed:

  • GitHub reports PR feat: add weaviate connector #78 as mergeable/CLEAN; CI build check succeeded.
  • Local merge of origin/pr/78 into current origin/main applied without conflicts.
  • git diff --check origin/main...origin/pr/78 passed.
  • Focused connector validation passed: bun run typecheck, bun test (6 pass), bun run build.
  • Root validation passed after dependency install with scripts disabled: bun run typecheck, bun test src/lib/registry.test.ts (28 pass).
  • Diff secrets/internal-reference scan found no credential or internal-reference hits; Weaviate is not on the connector blacklist and no browser automation dependency was found.

Recommendation: do not merge until the package publish contents are fixed, declarations are emitted or types is corrected, and .npmrc follows the connector policy.

@andrei-hasna
andrei-hasna merged commit 3afec1d into main Jul 7, 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.

1 participant