Skip to content

feat(keys): r refreshes the list and every pull request now - #27

Merged
Mheaus merged 1 commit into
mainfrom
feat/refresh-key
Oct 12, 2026
Merged

Mheaus merged 1 commit into
mainfrom
feat/refresh-key

Conversation

@sakuga-agent

@sakuga-agent sakuga-agent Bot commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • In the main set of the action bar, r (or a click on « r refresh ») stops the wait of the watch loop and reads the list and every pull request again, without the wait for the next minute.
  • r works only in the main set: in the list of the bots, r opens CodeRabbit, and in the menu of a bot, r is the review command.
  • A press during a refresh asks for one more refresh after it. The help and the README name the key.

Test plan

  • pnpm test (158), pnpm typecheck, pnpm lint, pnpm format
  • boo --watch --dry-run in a real terminal (pty): the bar shows « r refresh », and r shows « Refreshing the list… », then « checking… » in the header.

Not verified in a browser: boo is a terminal UI with no web page.


Sakuga Review · Score de confiance : 4/5

Peut être fusionnée : le changement ajoute la touche r et un rafraîchissement forcé de la boucle de surveillance, sans défaut bloquant relevé dans le diff.

Résumé

La touche r, active seulement au niveau principal, appelle refreshAll qui réveille la boucle --watch via un AbortController et force une relecture de la liste puis de chaque pull request au tour suivant. Le contrôle de l'interruption du sleep ne réémet que les arrêts du workflow, et le rattrapage du wake est réinitialisé après chaque tour. Les chemins de rafraîchissement existants (quiet et due) couvrent toutes les lignes ouvertes. Je n'ai pas trouvé de scénario concret de défaut dans le diff ; les appels à gitHub.listPullRequests et fetchPullRequest, ainsi que les définitions de sleep et de decide, ne figurent pas dans le message.

Diagramme
%%{init: {'theme': 'neutral'}}%%
flowchart LR A["touche r"] --> B["refreshAll"] B --> C["wake.abort"] C --> D["run boucle watch"] D --> E["syncList"] E --> F["refresh des lignes"]
Loading
Non relu
  • src/app.tsx: le comportement de sleep avec un signal combiné (AbortSignal.any) et de decide/planSync n'est pas vérifiable sans leurs définitions
  • src/app.tsx: le chemin sans --watch ne donne aucun retour à l'utilisateur, non vérifié faute de contexte des appelants

Revues (1) · Dernier commit relu : feat(keys): r reads the list and every pull req... · Niveau lite · anthropic/claude-haiku-5.5 · 0,004 $

@sakuga-review sakuga-review Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Score de confiance : 4/5 · ✅ Rien de bloquant · Peut être fusionnée : le changement ajoute la touche r et un rafraîchissement forcé de la boucle de surveillance, sans défaut bloquant relevé dans le diff.

Niveau lite · anthropic/claude-haiku-5.5 · 0,004 $ · Le résumé, le diagramme et la liste des constats sont à la fin de la description de la PR.

@sakuga-review sakuga-review Bot added the review: 4/5 Score de confiance de Sakuga Review (5 : prête à merger) label Oct 12, 2026
@Mheaus
Mheaus merged commit 36e0be8 into main Oct 12, 2026
6 checks passed
@Mheaus
Mheaus deleted the feat/refresh-key branch October 12, 2026 23:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review: 4/5 Score de confiance de Sakuga Review (5 : prête à merger)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant