Korjaa webui-kontin kaatuminen (COPY raportti/) + REPORTPLAN-päivitys#7
Open
ttokola wants to merge 2 commits into
Open
Korjaa webui-kontin kaatuminen (COPY raportti/) + REPORTPLAN-päivitys#7ttokola wants to merge 2 commits into
ttokola wants to merge 2 commits into
Conversation
koosta_tilanne lukee nyt tallennetun tuoreuden (RaporttiTuoreus, migraatio 018) eikä laske raskasta raporttitiivistettä joka katselulla; paivita_tuoreus ajetaan taustalla. Dokumentoi uudet funktiot, CLIUI:n "Tarkista tuoreus nyt", WebUI:n taustatrigger ja siemennys generoinnissa. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
palvelin.py importtaa top-level from raportti import llmraportti (lisätty f2689a0:ssa HITL-laatumittareita varten), mutta Dockerfileen ei koskaan lisätty COPY raportti/. Latentti bugi paljastui vasta nyt: ensimmäinen webui-rebuild sitten f2689a0:n → image sisältää importin ilman raportti/ -hakemistoa → ModuleNotFoundError → uvicorn kaatuu → restart-loop. Sama korjaus kuin aiempi COPY luokittelu/ (e35fa7b). llmraportti tarvitsee vain jo kopioidut tietokanta + llm, joten yksi COPY-rivi riittää. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
ttokola
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Kaksi muutosta PR #6:n jatkoksi:
1. fix: webui-kontin restart-loop (kriittinen)
webui/palvelin.pyimporttaa top-levelfrom raportti import llmraportti(lisätty commitissa f2689a0 HITL-laatumittareita varten), muttawebui/Dockerfileen ei koskaan lisättyCOPY raportti/. Latentti bugi paljastui vasta nyt, koska tämä on ensimmäinen webui-rebuild sitten f2689a0:n → uusi image sisältää importin ilmanraportti/-hakemistoa →ModuleNotFoundError: No module named 'raportti'→ uvicorn kaatuu → kontti restart-loopissa. Sama korjaus kuin aiempiCOPY luokittelu/(e35fa7b).llmraporttitarvitsee vain jo kopioiduttietokanta+llm.2. docs: REPORTPLAN.md tuoreusseuranta taustalle
PR #6:n arkkitehtuurimuutos (raskas tuoreuslaskenta taustalle,
RaporttiTuoreus/migraatio 018) dokumentoitu spec-tiedostoon.🤖 Generated with Claude Code