Skip to content

feat(client): add app-hosted connector worker#1016

Closed
buremba wants to merge 1 commit into
feat/client-sdkfrom
feat/app-hosted-connectors
Closed

feat(client): add app-hosted connector worker#1016
buremba wants to merge 1 commit into
feat/client-sdkfrom
feat/app-hosted-connectors

Conversation

@buremba
Copy link
Copy Markdown
Member

@buremba buremba commented May 21, 2026

Summary

  • add @lobu/client/node with Lobu, defineConnector, feed, action, and app-hosted connector serving
  • register metadata-only connector definitions without uploading source or compiled code
  • dogfood existing /api/workers/* polling, stream, complete, and complete-action endpoints for local feed/action functions
  • let SDK workers narrow polls by connector key and skip compiled-code resolution for app-hosted connectors

Validation

  • bun test packages/client/src
  • cd packages/client && bun run typecheck && bun run build
  • cd packages/pgvector-embedded && bun run build && cd ../server && bun run typecheck
  • bun run typecheck:packages
  • Claude Code review pass; followed up on retry/loop, poll filtering, and schema validation concerns

Notes

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 21, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: abb965f7-b535-4d9a-b4b9-752436c97c83

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/app-hosted-connectors

Comment @coderabbitai help to get the list of available commands and usage tips.

@buremba
Copy link
Copy Markdown
Member Author

buremba commented May 21, 2026

Closing: the app-hosted poll-loop runtime is being dropped in favor of the existing connector execution path (connector-sdk + connector-worker).

The valuable part — a defineConnector authoring API — will be reintroduced as sugar over ConnectorRuntime (reusing SyncContext/SyncResult/ConnectorDefinition), lowering to a class the existing worker's child-runner runs unchanged. This PR instead declared a parallel HostedConnector contract + its own poll loop, which duplicates the connector type system and reinvents the durable-run queue. #1015 (consumption SDK) is unaffected and stays open.

@buremba buremba closed this May 21, 2026
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