Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

🤖 feat: composer-draft follow-ups that need new protocol or mechanism

Chiusa
  • Qualcosa non va?
#5,226 3 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
5/5
Tempo stimato
Più di una settimana
Idoneità per principianti
20/100
Tipo di issue
Funzionalità
Chiarezza
Da chiarire
Stato di attività
Attiva
Stack tecnologico
typescript
Ambito
backend-api-design, desktop, distributed-systems

Direzione di ricerca

This is a backlog of 15 follow-ups rather than one bounded change. Start with the relevant originating issue and the named entry points, such as draftService.ts, DraftStore, or the drafts.importLegacy and drafts.update APIs; check dependencies such as #5217 before choosing one item. Done means implementing and validating one selected protocol or concurrency mechanism without taking on the other listed items.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

backlog investigation

Backlog for composer-draft follow-ups that need a new protocol or mechanism, split out of #5075 (and deferred review findings from #5221 and #5222). Each item below adds API surface or cross-process machinery, so it was kept out of the fix PRs.

From #5075

  1. Draft index with sibling backends on one root (#5067, draftService.ts). DraftService caches a metadata index after the first list(). A second backend on the same Xum root (XUM_ALLOW_MULTIPLE_INSTANCES=1) can change draft.json without this process noticing, and change events are in-process only. Needs a cross-process change token or a file watch.
  2. Bulk legacy import (#5068, #5075 item 3). The one-time migration sends one drafts.importLegacy call per legacy scope. Needs an array input on drafts.importLegacy (API change). Bounded by the origin's legacy key count, and it runs once.
  3. Remote edit during moveDraft (#5068, #5069, #5075 item 9). A server-applied change does not bump local version counters, so a move can delete the source while holding a stale copy (multi-client only). Needs an atomic backend drafts.move.
  4. Failed-import fallback can overwrite a concurrent client's newer draft (#5068, #5075 item 13). After a failed importLegacy, the store shows the legacy draft and saves it with a normal (overwriting) drafts.update; a draft another client created meanwhile is replaced. Needs a conditional write (for example "only if absent" or "only at revision N") on drafts.update.

Deferred review findings

  1. A sub-project's own default creation draft (#5221 review). A sub-project's New Workspace action does not consider that sub-project's default creation draft (only the project-level row does, after #5221). Same behavior as main before #5221.
  2. Concurrent follow-up send clears a restored draft (#5222 review). If an edit send is still unresolved when the user starts another send, the edit's completion restores the pre-edit draft into the composer, and the later send's setInput("") then clears the whole composer, including that restored draft. Pre-existing clear-after-capture behavior of a send; a fix needs sends to clear only what they captured.
  3. Edit completion restores the wrong snapshot when a second edit started meanwhile (#5222 review). If a second edit starts while the first edit's RPC is unresolved, the first completion reads the shared pre-edit snapshot (now the second edit's) and closes edit mode, ending the second edit (the close is main's behavior). Needs the snapshot and the close bound to the submitted edit operation.
  4. Review notes attached while an edit send is pending are replaced (#5222 review). With a pre-edit review override, the completion restores the old override instead of merging it with notes attached meanwhile.
  5. Deep-link quota fallback before the first draft snapshot (#5228 informational review on the rebased head). If a startup new_chat link hits the draft-list quota fallback before DraftStore hydrated, the prompt is appended to an empty in-memory default draft; the dirty text then wins over the backend's existing default draft at the first snapshot. Needs the default draft loaded (or the append deferred to hydration) before merging the prompt.
  6. Attachment-only legacy pending import has no retry marker — blocked on PR5 (#5217) (#5075 item 8). If the draft-list write fails at a full origin, an attachment-only legacy pending draft has no legacy key left to retry from, so the imported backend draft is unreachable. Re-listing it from the snapshot alone would resurrect a draft whose delete was interrupted (#5228 review), so the fix needs a small persisted recovery marker, which needs a registered key and budget in the localStorage key registry that #5217 is rewriting. Do after #5217 merges.
  7. Send clearing misses a boundary carried by the sent text's own trailing whitespace (#5229 round-3 review, non-loss). With sent = "follow-up " and current = "follow-up more" (typed during send preparation), removeSentText finds no whitespace boundary after the prefix and leaves the composer unchanged, so the already-sent text stays visible next to the new text (a duplicate, never a loss). Accept a boundary supplied by trailing/leading whitespace inside sent.
  8. A failed send overwrites the composer (pre-existing on main; #5229 final check). On !result.success, setDraft(preSendDraft) replaces the composer, losing anything restored or typed while the send was pending. Needs a merge-style restore.
  9. Switching directly between two open, unsent edits loses the original pre-edit draft (pre-existing on main; #5229 final check). The second edit snapshots the first edit's text as its pre-edit draft.
  10. Queued-message Edit (restoreDraft) while an edit send is pending replaces the composer (pre-existing on main; #5229 final check). Not covered by the edit-send refusal.
  11. Cancelling an edit during the send's preference wait still sends it (pre-existing on main; no input loss since #5229). Should abort the send before dispatch.

Refs #5075

Generated with xum • Model: anthropic:claude-opus-5-5 • Thinking: high • Cost: $14.19

Lingua principale
TypeScript
Stelle
2.1k
Fork
142
Merge medio
3h 32m
PR unite (30g)
869

Preparare l'ambiente

  • Include un Dockerfile o un file Docker Compose
  • Nessun modello di pull request
  • Nessuna guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di coder/xum

Tutte le issue di coder/xum

Issue simili

Altre issue su TypeScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.