fix(source-chat): emit SSE keepalives during generation and add stop-… #1239
AugustoSandimtest.yml
on: pull_request
Backend Tests
30s
Backend Lint
16s
Backend Typecheck
45s
Frontend Tests
38s
Frontend Lint
20s
Frontend Build
48s
Annotations
5 errors and 7 warnings
|
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 Build
Process completed with exit code 1.
|
|
Frontend Build
Cannot find name 'result'.
|
|
Frontend Build
Cannot find name 'result'.
|
|
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:ca8ddc078c993352906e57c05b1dca08aa08e5c310553384fc87149483f22cd5
|
|