task countdown hint promises 0 waits, and in a headless run 0 starts immediately
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
- 88/100
- Type d'issue
- Bug
- Clarté
- Clairement spécifiée
- Activité
- Active
- Stack technique
- go
- Domaine
- cli, developer-experience
Piste de recherche
Commencez par internal/config/settings.go:2298 et examinez internal/session/session.go:1806 pour la distinction entre watched/headless. Ajoutez un test pour l'indication du compte à rebours de la tâche, en laissant internal/manual/chat/tasks.md inchangé. Le travail est terminé lorsque l'indication inclut la condition watched, correspond à tasks.md:1492 et que le nouveau test passe.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
The defect is one string
The task countdown row's hint promises, unconditionally, that zero waits:
internal/config/settings.go:2298
"how many seconds a proposed task waits for you before it starts on its own. The countdown is your window to redirect it or wave it off, not a gate, 0 waits for your answer instead of starting. A change lands on the next session."
That promise holds only on a watched session. internal/session/session.go:1806 states the other half:
0 IS A CLOCK THAT IS OFF, the proposal waits for
Agent.ResolveTaskand nothing else, which is only a sentence a WATCHED session can honour: with nobody subscribed to the events (AskConsentfalse, or no turn hub), the deadline approves whatever this says, because a headless run has no one to wait for.
So on a headless run, zero does not wait for an answer. It approves immediately. The value a person types to make every proposal stop is the value that makes every proposal start.
Why this is the worst place for it
The surface closest to the decision is the one that is wrong. The hint is read at the moment the number is typed. The page a person would have to go looking for is already correct.
The manual is correct and MUST NOT be changed to match
internal/manual/chat/tasks.md scopes the promise properly in both places that tell a person to use zero:
:1492"Usetask.autoapprove_secondsin the Safety settings to give yourself a longer window, or set it to 0 so every watched proposal waits for you.":1510"Settask.autoapprove_secondsto 0 in Safety settings if every proposal on a watched session should wait until you answer."
Reconciling the two surfaces in the wrong direction would delete the only correct statement of the behaviour that exists. If the hint and the page disagree, the page wins.
Not live today, and the reason it is worth fixing anyway
DefaultTaskAutoApprove is 15 (settings.go:1429) and a persisted value is only taken when one is present (settings.go:4256), so nothing reaches zero by default. The hazard arrives the first time somebody types 0 meaning what the row told them it means.
Acceptance
- The hint carries the watched condition, in the shape
tasks.md:1492already uses. - A test holds it. A promise in a help string has nothing mechanical behind it today, which is why the two surfaces were free to drift apart: nothing relates the string to the behaviour it describes.
Read at d91bb63ae.
- Langage dominant
- Go
- Étoiles
- 115
- Forks
- 14
- Merge moyen
- 9 h 38 min
- PR mergées (30 j)
- 749
Préparer son environnement
Nous n'avons pas encore vérifié les fichiers d'installation de ce projet. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
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 Agent-Field/CodeAF
-
area:chat feature
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
Agent-Field/CodeAF#1510 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
Agent-Field/CodeAF#1489 ·
Les mainteneurs répondent en général sous 1 jour
-
area:chat bug good first issue sev:papercut
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
Agent-Field/CodeAF#1470 ·
Les mainteneurs répondent en général sous 1 jour
-
area:chat bug good first issue sev:papercut
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
Agent-Field/CodeAF#1469 ·
Les mainteneurs répondent en général sous 1 jour
-
area:chat bug sev:papercut
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
Agent-Field/CodeAF#1468 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de Agent-Field/CodeAF
Issues similaires
-
enhancement low priority
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
eugenioenko/ttt#674 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
Les mainteneurs répondent en général sous 1 jour
-
Remove obsolete ipAllocationPolicy field from containernodepool-subnetworkref test dependencyOuverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 92/100
GoogleCloudPlatform/k8s-config-connector#13462 ·
Les mainteneurs répondent en général sous 1 jour
-
bug good first issue
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
vavallee/bindery#2793 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour