Treat a stream that ends without [DONE] as an error
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 85/100
- Type d'issue
- Bug
- Clarté
- Clairement spécifiée
- Activité
- Active
- Stack technique
- typescript
Piste de recherche
Commence dans packages/widget/src/api/client.ts, au niveau de la boucle de lecture et de son résultat actuel { done: true }, puis examine packages/widget/tests/api/client.test.ts et le helper mockChatStream dans tests/helpers/stream.ts. Ajoute des tests pour les streams qui se terminent avec et sans [DONE] ; done semble indiquer que le cas terminé réussit toujours, qu’un stream incomplet renvoie { error: 'network' } et que la gestion des délais d’expiration reste intacte. Ajoute le changeset patch demandé pour @chatcops/widget.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
All three server adapters end a reply with data: [DONE]. The widget's client treats the end of the stream as success either way: after the read loop it yields { done: true } (packages/widget/src/api/client.ts, lines 129 and 130). When a proxy or the server cuts the connection mid-reply, the visitor gets a half answer with no error and no Retry button.
What to do
- If the stream ends without a
[DONE]line, yield{ error: 'network' }instead of{ done: true }. - A
[DONE]that arrives keeps working as today, and so does the timeout handling.
Acceptance criteria
- Tests in
packages/widget/tests/api/client.test.ts(themockChatStreamhelper intests/helpers/stream.tsbuilds the fake stream): ending with[DONE]givesdone, ending without it giveserror: 'network' - Changeset for
@chatcops/widget(patch) -
pnpm -r build,pnpm testandpnpm -r typecheckpass
Want to work on this? Comment below and I'll assign it to you. Please take one issue at a time, branch off dev, and open your PR against dev with Closes #<this issue number>. Setup notes (pnpm, build before test) are in CONTRIBUTING.md.
- Langage dominant
- TypeScript
- Étoiles
- 13
- Forks
- 7
- Merge moyen
- 3 h 41 min
- PR mergées (30 j)
- 155
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de codercops/chatcops
-
Document the pre-chat formPeut-être pris @masyarakatai l’a pris il y a 5 jours. Ouvertedocumentation good first issue hacktoberfest widget
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
codercops/chatcops#70 · 5 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
docs: update the providers page for openai-compatible, registerProvider and provider importsPeut-être pris @prakashNalla l’a pris aujourd’hui. Ouvertedocumentation good first issue hacktoberfest website
Difficulté 3/5 1-2 jours Accessibilité débutants 76/100
codercops/chatcops#291 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
docs: add a protocol reference page for custom clientsPeut-être pris @kachamsiddarth l’a pris il y a 1 jour. Ouvertedocumentation good first issue hacktoberfest website
codercops/chatcops#286 · 2 commentaires · 1 personne assignée ·
Les mainteneurs répondent en général sous 1 jour
-
Create @chatcops/store-cloudflare (D1, KV and a Durable Object counter)Peut-être pris @bhushan4work l’a pris aujourd’hui. Ouverteenhancement hacktoberfest help wanted server
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
codercops/chatcops#249 · 2 commentaires · 1 personne assignée ·
Les mainteneurs répondent en général sous 1 jour
-
Create @chatcops/store-sqlite, a store for single-server setupsPeut-être pris @srikar-naidu l’a pris aujourd’hui. Ouverteenhancement hacktoberfest help wanted server
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 35/100
codercops/chatcops#247 · 3 commentaires · 1 personne assignée ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de codercops/chatcops
Issues similaires
-
Add: Pbs KidsOuvertecheck:passed streams:add
Difficulté 2/5 1-3 heures Accessibilité débutants 67/100
Les mainteneurs répondent en général sous 2 jours
-
maintanance
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
inmanta/web-console#7583 ·
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
deltachat/deltachat-desktop#6881 · 1 commentaire ·
Les mainteneurs répondent en général sous 3 jours
-
[Bug Report][3.13.7] VDataTable with select-strategy="single" shows select-all checkbox in mobile modePeut-être pris @lazerg l’a pris aujourd’hui. OuverteS: triage
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
phivk/rubiks-lab#33 ·
Les mainteneurs répondent en général sous 1 jour