Hacktoberfest 2026 : les issues que les mainteneurs ont marquées pour octobre, ouvertes et accessibles aux débutants. Parcourir les issues Hacktoberfest

Offer a way back to Setup from the lobby

Ouverte
  • Quelque chose ne va pas ?
#513 4 commentaires 0 réactions 1 personne assignée Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

@Patriciaath999 y travaille déjà.

Depuis le 26/10/2026.

  • #528 par @YEH-YU-YANG — ouverte

Évaluation

Cette issue n'a pas encore été évaluée.

Description

After the first Setup, neither the web server nor the desktop app offers any way back to the Setup page. Someone who skipped the Google API key there, which the page marked Optional, starts interviews in which the interviewer never speaks, and the only fix is to find codetrial.env.local, edit it by hand and restart. On the desktop app that file sits in %LOCALAPPDATA%\org.sysprog21.codetrial, which nothing in the app names.

I propose adding a way back to Setup from the lobby, as one change covering both the web server and the desktop app, since the desktop app only shows the server's pages.

Current behavior

Setup is a cold-start phase, not a route of the app. At ce2c50a, main serves it only when is_cold_start holds (src/main.rs:356, src/main.rs:538): no --config, no primary config file, and no LIVEKIT_* in the environment. serve_setup (src/main.rs:569) runs setup_service (src/web/setup.rs:45) on its own router until the config is written, then hands the socket to the full app, which has no Setup route. Once the file exists, every later launch skips Setup.

Proposed behavior

A "Change keys" entry in the lobby opens a Setup page prefilled with the current LiveKit URL and the presence of each key, never the secret values. Saving writes the same codetrial.env.local through the existing validation and credential probe, then the server restarts, or the desktop app restarts its child server, so the new keys take effect.

Setup's existing guard applies unchanged: the page answers only to a loopback Host (src/web/setup.rs:73), so a server deployed for other people never shows the entry and stays configured through its file.

Decision needed

How the new keys take effect:

  1. Restart the process after writing. Simplest, and the startup path stays the only place keys are read. On the web server this needs a supervisor or the user to start it again; the desktop app can restart its child itself.
  2. Reload the config in the running server. No restart, but the provider pool (src/main.rs:487) and the decision to host interviewers (src/main.rs:450) are made once at startup today, so this touches far more code.

I would start with option 1. Which one should this take?

Langage dominant
Rust
Étoiles
215
Forks
141
Merge moyen
1 j 12 h
PR mergées (30 j)
232

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de sysprog21/codetrial

Toutes les issues de sysprog21/codetrial

Issues similaires

Plus d'issues Rust

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.