zyratlo commented on PR #5265: URL: https://github.com/apache/texera/pull/5265#issuecomment-5038115951
I added the AI SDK npm dependencies to frontend/LICENSE-binary so the binary license check passes. Why this surfaces here: this PR's JupyterPanelService is the first reachable consumer of NotebookMigrationService (added in #5262). That service statically imports the LLM client in migration-llm.ts, which pulls in the `ai` package and its provider stack. On main the service has no consumer, so it tree-shakes out and the AI SDK never lands in the bundle, which is why main does not claim these. Once this PR wires JupyterPanelService into app bootstrap, the AI SDK gets bundled and the license check requires the packages to be claimed. For context: the panel feature itself only calls the notebook fetch, send, and mapping methods. It never calls the LLM path. The AI SDK is bundled transitively through that static import. Newly claimed (all ASF Category A, none ship a NOTICE file, so NOTICE-binary is unchanged): Apache-2.0: ai, @ai-sdk/openai, @ai-sdk/gateway, @ai-sdk/provider, @ai-sdk/provider-utils, @opentelemetry/api, @vercel/oidc MIT: zod, eventsource-parser -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
