From dc4a31803ede590ecb78ed0341647f2d735ccc8c Mon Sep 17 00:00:00 2001 From: ht Date: Sun, 6 Sep 2026 16:25:16 +0300 Subject: [PATCH] fix: block preparation for unfinished analysis states --- docs/quality/uygulama-durumu.md | 6 ++++++ src/components/BidPackageBuilder.jsx | 6 +++--- src/services/preparationAnalysis.js | 2 ++ tests/analysis-quality.test.mjs | 5 +++++ 4 files changed, 16 insertions(+), 3 deletions(-) diff --git a/docs/quality/uygulama-durumu.md b/docs/quality/uygulama-durumu.md index 85323c9..12f6e5b 100644 --- a/docs/quality/uygulama-durumu.md +++ b/docs/quality/uygulama-durumu.md @@ -1286,3 +1286,9 @@ Taslak türü seçicisine erişilebilir ad eklendi. Üretim derlemesi geçti. Ö - Hazırlık özetindeki “Belge durumunu yenile” düğmesi belge deposu okunurken artık `aria-busy=true` bildiriyor. Görsel disabled durumu, mevcut yükleniyor mesajı ve hata sonrası yeniden deneme davranışı korunuyor. - 279 test ve build başarılı. Bu tek nitelik için ayrı ekran okuyucu kabulü yapılmadı; genel plan açık. + +### 2026-09-06 — Analiz durumlarının hazırlığı engellemesi + +- Hazırlık ekranı yalnız `partial` analizde değil; `error`, `processing` ve `pending` durumlarında da artık hazır olamıyor. Ortak `analysisStatusBlocksPreparation` yordamı bu dört tamamlanmamış durumu tek yerde tanımlıyor. +- Kullanıcıya gösterilen uyarı artık gerçek durumu (`error`, `processing` gibi) yazıyor; eksik kaynak/OCR/kalem açıklaması korunuyor. +- Dört engelli ve üç tamamlanmış durum için regresyon testi eklendi. 280 test ve build başarılı. Gerçek AI sağlayıcılarında hata/işleniyor ekranı kabulü henüz yapılmadı; genel plan açık. diff --git a/src/components/BidPackageBuilder.jsx b/src/components/BidPackageBuilder.jsx index 2ecaa47..f2cb7d2 100644 --- a/src/components/BidPackageBuilder.jsx +++ b/src/components/BidPackageBuilder.jsx @@ -1,5 +1,5 @@ import { bidFirmFields as ALANLAR, bidFirmIssues } from '../services/bidFirmFields'; -import { preparationAnalysisSource, preparationConditionSections, conditionText } from '../services/preparationAnalysis'; +import { preparationAnalysisSource, preparationConditionSections, conditionText, analysisStatusBlocksPreparation } from '../services/preparationAnalysis'; import { preparationDeadlineProblem } from '../services/preparationDeadline'; import { bidValidityForForm, bidTypeForForm } from '../services/bidTerms'; import { createBidWorkbook, downloadBidWorkbook } from '../services/bidWorkbook'; @@ -139,7 +139,7 @@ export const BidPackageBuilder = ({ results, tender: selectedTender, initialFocu useEffect(() => {const timer=setInterval(()=>setDeadlineNow(new Date()),30000);return ()=>clearInterval(timer);},[]); const deadlineProblem = preparationDeadlineProblem(tender,deadlineNow); const kontrolEksikleri = preparationBlockers(lotSelection.rows, documents, checks, {...vault,healthPortfolio:userProfile.scrapedCapabilities?.healthPortfolio || [],deadlineDate:tenderDateKey(tender),tenderIkn:tender.ikn}); - const analizEksik = effectiveResults.sourceReview?.status === 'partial'; + const analizEksik = analysisStatusBlocksPreparation(effectiveResults.sourceReview?.status); const hazir = !deadlineProblem && eksikler.length === 0 && toplam > 0 && !kalemEksik && !turCeliskisi && !sureEksik && !kaynakKontrolGerekli && kontrolEksikleri.length === 0 && lotSelection.errors.length === 0 && !analizEksik && !sourceOptions.needsSelection; const missingActions = [ @@ -313,7 +313,7 @@ export const BidPackageBuilder = ({ results, tender: selectedTender, initialFocu Metin incelemesi: sürüm {effectiveResults.sourceReview.humanReview.revision} · {effectiveResults.sourceReview.humanReview.reviewedBy} · {effectiveResults.sourceReview.humanReview.reviewedAt}. Kullanıcı karşılaştırmasıdır; resmi onay değildir.

} {deadlineProblem &&

{deadlineProblem}

} - {analizEksik &&

Analiz kısmi: kaynak, OCR veya kalem bilgisi kontrol gerektiriyor. {(effectiveResults.sourceReview.issues || []).join(' ')} Tam ve doğrulanabilir kaynakla analizi tamamlayın.

} + {analizEksik &&

Analiz tamamlanmadı ({effectiveResults.sourceReview?.status || 'durum bilinmiyor'}): kaynak, OCR veya kalem bilgisi kontrol gerektiriyor. {(effectiveResults.sourceReview?.issues || []).join(' ')} Tam ve doğrulanabilir kaynakla analizi tamamlayın.

} {/* Sizden beklenen bilgiler */} {eksikler.length > 0 && (
diff --git a/src/services/preparationAnalysis.js b/src/services/preparationAnalysis.js index 4762848..c8308c1 100644 --- a/src/services/preparationAnalysis.js +++ b/src/services/preparationAnalysis.js @@ -15,6 +15,8 @@ export function preparationAnalysisSource(results = {}) { ...['documentVersion','procurementTerms','sourceReview','sourceEvidence','citations','summary','scopeOfWork'].map(field=>[field,results[field] ?? null]), ]); } +export const analysisStatusBlocksPreparation = status => + ['partial','error','processing','pending'].includes(String(status || '').trim().toLowerCase()); export function conditionText(row) { if (typeof row === 'string') return row; return ['title','label','value','clause','text','detail','where','whyItMatters','note'].map(key=>row?.[key]).filter(Boolean).join(' — '); diff --git a/tests/analysis-quality.test.mjs b/tests/analysis-quality.test.mjs index c3efb42..9911910 100644 --- a/tests/analysis-quality.test.mjs +++ b/tests/analysis-quality.test.mjs @@ -1,6 +1,11 @@ import test from 'node:test'; import assert from 'node:assert/strict'; import {qualifyAnalysis} from '../server/analysis-quality.mjs'; +import {analysisStatusBlocksPreparation} from '../src/services/preparationAnalysis.js'; +test('all non-final analysis states block preparation',()=>{ + for(const status of ['partial','error','processing','pending'])assert.equal(analysisStatusBlocksPreparation(status),true); + for(const status of ['complete','verified',''])assert.equal(analysisStatusBlocksPreparation(status),false); +}); test('malformed nested model fields cannot reach preparation screens',()=>{ for(const result of [ {itemsAndSpecs:[null]}, {itemsAndSpecs:[{name:{text:'Kit'}}]},