watch: a rebuild queued during a failed deploy is never deployed
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 78/100
- Issue-Typ
- Bug
- Klarheit
- Klar beschrieben
- Aktivitätsstatus
- Aktiv
- Tech-Stack
- node.js, typescript
Rechercherichtung
Beginne in packages/cli/src/watch.ts und verfolge, wie triggerDeploy die Erfolgs- und Fehlerpfade des Promise behandelt. Reproduziere den Fehler, indem du während eines laufenden Deployments eine Änderung speicherst, und stelle dann sicher, dass das vorgemerkte Deployment nach beiden Ergebnissen ausgeführt wird; überprüfe, dass es nicht durch den in #47 beschriebenen Deployment-Hold blockiert wird.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Problem
In packages/cli/src/watch.ts, if a rebuild happens while a deploy is running, triggerDeploy sets deployQueued = true. The .then handler starts the queued deploy, but the .catch handler only resets isDeploying and never checks deployQueued.
So if the running deploy fails, a change saved during it is silently left undeployed until the next file change. That's often exactly the fix for the failure.
Repro
- In
notation watch, start a deploy that will fail. - While it's running, save a change.
- The failed deploy finishes, and the saved change is not deployed until another edit.
Fix
Drain the queue in both paths, e.g. move the deployQueued check into a .finally, or call the same "done" function from .then and .catch.
Note that while #47 is open, the queued deploy would then hang on the deployment hold, so the two are worth fixing together.
- Vorherrschende Sprache
- TypeScript
- Sterne
- 54
- Forks
- 1
- Ø Merge
- 1 Std. 45 Min.
- Gemergte PRs (30 T.)
- 1
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus notationlabs/notation
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 42/100
notationlabs/notation#47 ·
-
state provider configOffen
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 35/100
notationlabs/notation#34 ·
-
envs as configOffen
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 35/100
notationlabs/notation#33 ·
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 45/100
notationlabs/notation#16 ·
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 45/100
notationlabs/notation#15 ·
Alle Issues in notationlabs/notation
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
jessepollak/home#2223 ·
Maintainer antworten meist innerhalb von 1 Tag
-
grep fixture corpus test follows the corpus's loop symlink under the receipt runner (fails on main)Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
code-yeongyu/senpi#3756 ·
Maintainer antworten meist innerhalb von 1 Tag
-
org:external package:deepagents priority:triage topic:memory topic:middleware type:bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
langchain-ai/deepagents#7059 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 92/100
osmlab/name-suggestion-index#12992 ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 66/100
hemilabs/ui-monorepo#2506 ·
Maintainer antworten meist innerhalb von 1 Tag