From c55257cade9f05e685c93c853c6b273ed9c29dcd Mon Sep 17 00:00:00 2001 From: ht Date: Sun, 6 Sep 2026 14:26:31 +0300 Subject: [PATCH] fix: exclude closed tenders from upcoming feeds --- docs/quality/uygulama-durumu.md | 7 +++++++ server/feeds-mcp.mjs | 6 +++--- src/services/tenderState.js | 2 +- tests/feed-deadlines.test.mjs | 26 ++++++++++++++++++++++++++ 4 files changed, 37 insertions(+), 4 deletions(-) diff --git a/docs/quality/uygulama-durumu.md b/docs/quality/uygulama-durumu.md index f120591..85be98b 100644 --- a/docs/quality/uygulama-durumu.md +++ b/docs/quality/uygulama-durumu.md @@ -1047,3 +1047,10 @@ Taslak türü seçicisine erişilebilir ad eklendi. Üretim derlemesi geçti. Ö - Tercih GET isteği artık ihale havuzunun yüklenmesine bağlı değildir; içerik yalnız preview=1 ile Önizle işleminde oluşturulur. Genel kullanıcı veri rotasında brief yazımı da sürüm/boolean kontrolünden geçer. Gönderim metni hizmet ve zamanlayıcı etkinliğine bağlı olduğunu açıklar. - 236 otomatik test ve derleme geçti. HTTP testi başlangıç tercihi, gövde doğrulama, kapatma/geri okuma, eski sürüm reddi, başka hesabın ayrımı ve güncel sürümle açmayı doğruladı. İlk testteki 400 bu gerçek izinli-anahtar hatasını yakaladı; düzeltme sonrası tüm suite geçti. - Yeni panelin tarayıcı hata/kurtarma ve önizleme kabulü, zamanlayıcıdan gerçek test teslimi açık. Bu tur dış özet gönderilmedi; canlıya yayın yapılmadı. + +### 2026-09-06 — Haftalık özet ve açık beslemelerde kapanmış ihale filtresi + +- RSS/MCP açık ihale araması ve haftalık özetin `yaklasanIhaleler` listesi ana listenin `acikIhale` kontrolünü kullanıyor. İptal, sonuçlandı, sözleşme imzalandı veya kazananı açıklanmış kayıtlar gelecek tarih taşısa da açık fırsat olarak gösterilmiyor. `sadeceAcik:false` geçmiş/durum aramasını koruyor. +- Yeni takip anlık görüntüleri kazanan firma alanını da saklıyor; havuzdan çıkan bir sonuç kaydının yalnız bu bilgi kaybolduğu için açık görünmesi önleniyor. Eski görüntülerde hiç saklanmamış kazanan bilgisi bu değişiklikle geriye dönük üretilemiyor; güncel kaynak doğrulaması ihtiyacı sürüyor. +- Regresyon testi güncel kaynakta sonradan iptal, havuz dışındaki üç kapalı durum ve kazanan kaydı, açık saat-belirsiz kayıt, metin/HTML özet ve tüm durumlar aramasını kapsıyor. +- Doğrulama: `npm test` 237/237; `npm run build` başarılı. Mevcut büyük paket uyarısı sürüyor. Canlı gönderim ve dağıtım yapılmadı. Haftalık zamanlayıcının gönderimden önce tamamlandı kaydı yazması ve yeniden deneme/dayanıklılık akışı hâlâ incelenecek; genel plan tamamlanmadı. diff --git a/server/feeds-mcp.mjs b/server/feeds-mcp.mjs index 83ac794..2be169f 100644 --- a/server/feeds-mcp.mjs +++ b/server/feeds-mcp.mjs @@ -6,6 +6,7 @@ // Claude Desktop (mcp-remote) araçları: search_tenders, get_tender, // search_results, search_items, upcoming_deadlines. Bağımlılık yok. import * as store from './user-store.mjs'; +import { acikIhale } from '../src/services/ihaleTarih.js'; import { deadlineInfo, tenderDateKey } from '../src/services/tenderCalendar.js'; import { applyTenderStates, tenderStateKey } from '../src/services/tenderState.js'; import { readHistory, resultsByIkn } from './history-store.mjs'; @@ -36,8 +37,7 @@ export function trTarih(s) { export function ihaleFiltrele(tenders, { q, il, sektor, sadeceAcik = true, limit = 50 } = {}, now = new Date()) { const qq = norm(q); const ill = norm(il); const sek = norm(sektor); return tenders - .filter((t) => !sadeceAcik || !t.kazananFirma) - .filter((t) => !sadeceAcik || deadlineInfo(t, now).expired !== true) + .filter((t) => !sadeceAcik || acikIhale(t, now)) .filter((t) => !ill || norm(t.sehir).includes(ill)) .filter((t) => !sek || norm(t.sektor).includes(sek)) .filter((t) => !qq || norm(t.baslik).includes(qq) || norm(t.kurum).includes(qq) || String(t.ikn ?? '').includes(qq) || @@ -106,7 +106,7 @@ export function yaklasanIhaleler(email, tenders, gun = 14, now = new Date()) { const takip = new Set(store.getTrackedTenderKeys(email, states)); const days = Math.min(90, Math.max(1, Math.floor(Number(gun) || 14))); return applyTenderStates(tenders, states) - .filter(t => !t.kazananFirma && takip.has(tenderStateKey(t))) + .filter(t => takip.has(tenderStateKey(t)) && acikIhale(t, now)) .map(t => ({ t, info: deadlineInfo(t, now) })) .filter(({ info }) => info.days !== null && info.days >= 0 && info.days <= days && info.expired !== true) .sort((a, b) => a.info.days - b.info.days || diff --git a/src/services/tenderState.js b/src/services/tenderState.js index 63af52c..05fbc87 100644 --- a/src/services/tenderState.js +++ b/src/services/tenderState.js @@ -22,7 +22,7 @@ export function applyTenderStates(tenders, records = {}) { return result; } export function tenderSnapshot(t) { - return Object.fromEntries(['id','ikn','baslik','kurum','tarih','saat','sehir','alimTuru','durum','documentVersion'].map(k => [k,t[k] ?? ''])); + return Object.fromEntries(['id','ikn','baslik','kurum','tarih','saat','sehir','alimTuru','durum','kazananFirma','documentVersion'].map(k => [k,t[k] ?? ''])); } export function publicTender(tender) { diff --git a/tests/feed-deadlines.test.mjs b/tests/feed-deadlines.test.mjs index df34070..5403d56 100644 --- a/tests/feed-deadlines.test.mjs +++ b/tests/feed-deadlines.test.mjs @@ -3,6 +3,7 @@ import assert from 'node:assert/strict'; import { mkdtempSync, rmSync } from 'node:fs'; import { tmpdir } from 'node:os'; import { join } from 'node:path'; +import { tenderSnapshot } from '../src/services/tenderState.js'; const dir = mkdtempSync(join(tmpdir(), 'ihaleist-feed-')); process.env.IHALEIST_USER_DATA_DIR = dir; @@ -63,3 +64,28 @@ test('weekly notification schedule starts at Monday 08:00 Istanbul', () => { assert.equal(briefScheduleKey(new Date('2026-09-07T20:59:59Z')), '2026-8-7'); assert.equal(briefScheduleKey(new Date('2026-09-07T21:00:00Z')), null); }); + + +test('closed tenders stay out of open feeds and weekly upcoming items, including saved snapshots', () => { + const owner = 'closed-feed@example.test'; + const base = { tarih:'07.09.2026', saat:'10:00' }; + const closed = ['İptal', 'Sonuçlandı', 'Sözleşme İmzalandı'].map((durum, i) => + ({ ...base, id:`closed-${i}`, ikn:`2026/closed-${i}`, durum })); + closed.push({ ...base, id:'winner', ikn:'2026/winner', kazananFirma:'Test kazananı' }); + const open = { ...base, id:'open', ikn:'2026/open', durum:'Açık' }; + const unknownTime = { ...open, id:'unknown-time', ikn:'2026/unknown-time', saat:'' }; + const all = [...closed, open, unknownTime]; + for (const tender of all) store.saveTenderState(owner, tender.ikn, { takipte:true }, tenderSnapshot(tender), 0); + assert.deepEqual(ihaleFiltrele(all, {}, now).map(t => t.id), ['open', 'unknown-time']); + assert.equal(ihaleFiltrele(all, { sadeceAcik:false }, now).length, all.length); + // Status updates in the current source must override a previously open saved snapshot. + const updated = { ...open, durum:'İptal' }; + const upcoming = yaklasanIhaleler(owner, [updated, unknownTime], 14, now); + assert.deepEqual(upcoming.map(t => t.ikn), ['2026/unknown-time']); + assert.equal(upcoming[0].saat_dogrulanmali, true); + for (const output of [briefMetni, briefHtml]) { + const body = output({ hafta:'Test haftası', yaklasan:upcoming, sonuclar:[] }); + for (const tender of [...closed, open]) assert.ok(!body.includes(tender.ikn)); + assert.match(body, /saat doğrulanmalı/); + } +});