fix(source-chat): emit SSE keepalives during generation and add stop-… #1238
AugustoSandimtest.yml
on: pull_request
Backend Tests
35s
Backend Lint
17s
Backend Typecheck
42s
Frontend Tests
50s
Frontend Lint
25s
Frontend Build
46s
Annotations
5 errors and 7 warnings
|
Frontend Build
Process completed with exit code 1.
|
|
Frontend Build
Cannot find name 'result'.
|
|
Frontend Build
Cannot find name 'result'.
|
|
Frontend Tests
Process completed with exit code 1.
|
|
src/lib/hooks/use-source-chat.test.tsx > useSourceChat sendMessage streaming > fails the send when pre-send hydration errors instead of minting an unverified id:
frontend/src/lib/hooks/use-source-chat.test.tsx#L259
ReferenceError: result is not defined
❯ src/lib/hooks/use-source-chat.test.tsx:259:9
❯ node_modules/@testing-library/react/dist/act-compat.js:47:24
❯ process.env.NODE_ENV.exports.act node_modules/react/cjs/react.development.js:814:22
❯ Proxy.<anonymous> node_modules/@testing-library/react/dist/act-compat.js:46:25
❯ src/lib/hooks/use-source-chat.test.tsx:258:13
|
|
Frontend Lint:
frontend/src/lib/hooks/use-credentials.ts#L214
'_removed' is assigned a value but never used
|
|
Frontend Lint:
frontend/src/components/sources/ModelSelector.tsx#L71
React Hook useMemo has a complex expression in the dependency array. Extract it to a separate variable so it can be statically checked
|
|
Frontend Lint:
frontend/src/components/sources/ModelSelector.tsx#L71
React Hook useMemo has a missing dependency: 't'. Either include it or remove the dependency array
|
|
Frontend Lint:
frontend/src/components/settings/CredentialFormDialog.tsx#L89
React Hook useEffect has a missing dependency: 'isOmlx'. Either include it or remove the dependency array
|
|
Frontend Lint:
frontend/src/components/podcasts/GeneratePodcastDialog.tsx#L124
React Hook useMemo has a missing dependency: 'sourcesQueries'. Either include it or remove the dependency array
|
|
Frontend Lint:
frontend/src/app/(dashboard)/notebooks/components/SourcesColumn.tsx#L73
React Hook useMemo has a complex expression in the dependency array. Extract it to a separate variable so it can be statically checked
|
|
Frontend Lint:
frontend/src/app/(dashboard)/notebooks/components/SourcesColumn.tsx#L73
React Hook useMemo has a missing dependency: 't'. Either include it or remove the dependency array
|
Artifacts
Produced during runtime
| Name | Size | Digest | |
|---|---|---|---|
|
backend-coverage
|
20.8 KB |
sha256:f2701a9d9f58a5878bfe2bdd21d2e39b4f27d78ec137e9dd5a40005894fd9fae
|
|