From f1e82531695a7de555080b8da65780bdc636d1a1 Mon Sep 17 00:00:00 2001 From: JiriUhlir <149317995+JiriUhlir@users.noreply.github.com> Date: Mon, 17 Aug 2026 15:54:34 +0200 Subject: [PATCH] Log rekne co zpusobilo jakou zmenu, prevzeti ticketu a pozvanky Nalezeno na bezicim serveru: TK-4946 mel 177 udalosti a 620 radku logu, pritom se skoro nic nestalo. Zmereno proti fronte: ve stejnem okne vzniklo presne tolik behu, kolik prislo udalosti (22 a 22), kazdy s jednim pokusem. Fronta nenasobi nic, odesilatel poslal 177 POSTu. Nase vina byla, ze to z historie neslo poznat. - Data udalosti se ukladaji. Kdyz krok nema vlastni, ulozi se to, cim beh zacal - u webhooku prijate telo. Prazdna udalost je horsi nez zadna. - Shodna udalost se pocita (repeats, lastAt), nezaklada dalsi radek. Ticket se pritom nemeni, takze duplikat nerozblika dashboard ani nespusti automatizaci na zmenu ticketu. Zahodit ji nejde, jinak by nikdo nezjistil, ze proti nam neco tluce. - Zmeny se radi pod udalost, ktera je zpusobila, a u udalosti stoji jmeno automatizace. Log se cte jako "prislo tohle -> zmenilo to tohle". - Poznamka o stavu jen kdyz se stav zmenil. "z in-progress na in-progress" u kazde zpravy byl zdroj tech 620 radku. - runsToday konecne znamena dnes: behy po dnech, k tomu vcera a celkem. Dosud to byl citac od zalozeni automatizace, jen se jmenoval "dnes". Vedle toho prace, o kterou slo predtim: - Prevzeti ticketu ze skupiny (POST /tickets/:id/claim) a krok Predat skupine s prepinacem automatickeho prideleni nejvolnejsimu. - Pozvanky do firmy: odkaz s nahodnym kodem, heslo si nastavi pozvany. - Resitele, skupiny a pozvanky presunuty z Nastaveni do zalozky Lide, cleny skupiny se vybiraji klikanim. - Ctyri AI znaky, ktere zbyvaly v kodu, pryc. Co-Authored-By: Claude Opus 5 (1M context) --- documentation/01-prehled-a-stav.md | 4 +- documentation/04-api.md | 36 ++ documentation/06-tickety.md | 77 ++++ documentation/99-zmeny.md | 95 +++++ src/data/automationStore.ts | 116 +++++- src/data/bootstrap.ts | 3 + src/data/invites.ts | 121 +++++++ src/data/people.ts | 7 + src/data/services.ts | 37 ++ src/data/ticketStore.ts | 192 +++++++++- src/index.ts | 3 + src/openapi.ts | 2 + src/routes/dashboard.ts | 76 +++- src/routes/invites.ts | 334 ++++++++++++++++++ src/runtime/builtinSteps.ts | 87 ++++- src/runtime/context.ts | 10 + src/runtime/executor.ts | 10 + src/runtime/worker.ts | 8 +- web/src/App.tsx | 4 + web/src/components/dashboard/InvitePanel.tsx | 265 ++++++++++++++ web/src/components/dashboard/TicketEvents.tsx | 17 +- web/src/components/home/References.tsx | 2 +- web/src/data/products.ts | 2 +- web/src/pages/Invite.tsx | 179 ++++++++++ web/src/pages/dashboard/AutomationDetail.tsx | 2 + web/src/pages/dashboard/Automations.tsx | 10 +- web/src/pages/dashboard/People.tsx | 147 +++++++- web/src/pages/dashboard/Settings.tsx | 75 +--- web/src/pages/dashboard/TicketDetail.tsx | 16 + web/src/types/dashboard.ts | 9 + 30 files changed, 1839 insertions(+), 107 deletions(-) create mode 100644 src/data/invites.ts create mode 100644 src/routes/invites.ts create mode 100644 web/src/components/dashboard/InvitePanel.tsx create mode 100644 web/src/pages/Invite.tsx diff --git a/documentation/01-prehled-a-stav.md b/documentation/01-prehled-a-stav.md index 6099b04..f3ed90a 100644 --- a/documentation/01-prehled-a-stav.md +++ b/documentation/01-prehled-a-stav.md @@ -37,10 +37,12 @@ React aplikaci ze slozky `dist/public`. | Skripty konektoru | hotovo | manifest, kontrola parametru, hot reload, iDoklad | | Konektory za firmu | hotovo | pristupove udaje v konektoru, overeni napojeni | | Transformace dat | hotovo | pravidla i sablona JSON, kroky si predavaji struktury | -| Sprava clenstvi z portalu | hotovo | uzivatele, firmy a role v Nastaveni | +| Sprava clenstvi z portalu | hotovo | firmy a role v Nastaveni, lide a pozvanky v Lidech | | Role a prava jako data | hotovo | 26 prav v katalogu, vlastni role za firmu | | Zalozky a limity za firmu | hotovo | navigace chodi ze serveru, ne z kodu klienta | | Osoby a skupiny resitelu | hotovo | ticket lze prehodit na skupinu, ne jen na cloveka | +| Prevzeti ticketu ze skupiny | hotovo | kdo ma cas, si praci vezme sam | +| Pozvanky do firmy | hotovo | odkaz s kodem, heslo si nastavi pozvany | | Typy ticketu a vlastni pole | hotovo | typ rozhoduje, ktere akce se na ticketu ukazou | | Vydefinovane akce na ticketu | hotovo | vazba na typ nebo tag, telo je operace, strom, skript | | Vlastni widgety | hotovo | vcetne zdroje z konektoru a vykonu resitelu | diff --git a/documentation/04-api.md b/documentation/04-api.md index 011c5a9..90cd594 100644 --- a/documentation/04-api.md +++ b/documentation/04-api.md @@ -48,8 +48,14 @@ Vyzaduji `Authorization: Bearer `: | POST | `/api/dashboard/tickets/:id/type` | | POST | `/api/dashboard/tickets/:id/tags` | | POST | `/api/dashboard/tickets/:id/group` | +| POST | `/api/dashboard/tickets/:id/claim` | | GET | `/api/dashboard/tickets/:id/actions` | | POST | `/api/dashboard/tickets/:id/actions/:actionId` | +| GET | `/api/dashboard/invites` | +| POST | `/api/dashboard/invites` | +| DELETE | `/api/dashboard/invites/:id` | +| GET | `/api/invites/:kod` | +| POST | `/api/invites/:kod/accept` | | GET | `/api/dashboard/incidents` | | GET | `/api/dashboard/storage` | | GET | `/api/dashboard/services` | @@ -196,6 +202,36 @@ toho, co ktera volana sluzba vratila. `POST /api/dashboard/tickets/:id/assign` s telem `{"assigneeId": null}` vrati ticket do fronty. Neznamy resitel vraci 404, ne tiche odpojeni. +`POST /api/dashboard/tickets/:id/claim` je **prevzeti prace**, ne prehozeni: +volajici si bere ticket sam a telo je prazdne. Smi to u ticketu bez resitele +a u ticketu ve skupine, ve ktere je. Kdyz uz ticket nekdo resi, vraci 409, resp. +403 u cizi skupiny - vzit nekomu rozdelanou praci je jine rozhodnuti a chce to +pravo `ticket.assign.others`. Kdo neni vedeny jako resitel, dostane 400. + +## Pozvanky do firmy + +`/api/invites/:kod` je **verejne**, protoze kdo prijde za odkazem, jeste ucet +mit nemusi. Autorizuje kod v adrese, proto je nahodny a dlouhy - stejne jako +u webhooku. Odpoved je zamerne uzka: nazev firmy, jestli pozvanka plati, a kdyz +uz adresu zname, tak ji, at ji clovek nemusi psat. Nic o tom, kdo ve firme je. + +`POST /api/invites/:kod/accept` s telem `{"name", "email", "password"}`: + +| Situace | Co se stane | +| --- | --- | +| ucet neexistuje | zalozi se a pripoji k firme | +| ucet existuje, heslo sedi | jen se pripoji k firme | +| ucet existuje, heslo nesedi | 401 | +| pozvanka je na jinou adresu | 403 | +| pozvanka uz byla pouzita nebo vyprsela | 409 s konkretnim duvodem | + +Overeni hesla u existujiciho uctu neni formalita: bez nej by kdokoliv s odkazem +pripojil cizi adresu ke sve firme a videl by jeji data. + +Sprava pozvanek (`/api/dashboard/invites`) chce pravo `user.manage`. Seznam +vraci u kazde pozvanky **celou adresu** vcetne prefixu proxy, aby slo rovnou +kopirovat - relativni cesta se do zpravy vlepit neda. + ## Akce na ticketu Popis modelu je v [09-navrh-rozsireni.md](09-navrh-rozsireni.md). diff --git a/documentation/06-tickety.md b/documentation/06-tickety.md index 71ab628..746ec8e 100644 --- a/documentation/06-tickety.md +++ b/documentation/06-tickety.md @@ -80,6 +80,47 @@ Kazdy resitel ma `capacity`, tedy pocet nevyrizenych ticketu, ktery je pro nej jeste zdrava zatez. Neni to limit, nic se podle nej neodmita - jen se v prehledu oznaci, kdo je nad ni. +## Fronta skupiny a prevzeti + +Ticket nemusi mit resitele hned. Muze lezet **u skupiny** a cekat, az si ho +nekdo vezme - `POST /tickets/:id/claim`. Kdo je ve skupine, si ho vezme sam, +prace se nerozdava shora. + +Proc obojí vedle sebe: + +| Situace | Co se hodi | +| --- | --- | +| havarie, musi to nekdo hned resit | automat prideli nejvolnejsimu | +| bezny dotaz, lidi maji ruzne dny | necha se ve fronte skupiny | + +Proto je prepinac **Priradit rovnou nejvolnejsimu** na kroku automatizace +(`ticket/assign-group`), ne na skupine: tataz skupina potrebuje obe chovani, +jen u jineho typu prace. + +Vzit si ticket, ktery uz nekdo resi, neni prevzeti, ale prehozeni. Chce to pravo +`ticket.assign.others` a jde to jen pres prirazeni, ne pres prevzeti. + +## Pozvanky do firmy + +`src/data/invites.ts` + +Novy clovek se do firmy dostane odkazem s nahodnym kodem, ne tim, ze mu nekdo +zalozi ucet a posle heslo. Duvody: + +- **heslo nikdy nikam neposilame.** Kdo si ho nastavi sam za odkazem, ma ho jen + on, +- pozvanka **vyprsi** (tyden) a da se zrusit, ucet ne, +- kdo uz ucet ma, se jen pripoji k dalsi firme misto zakladani druheho - ale + musi zadat svoje heslo, jinak by kdokoliv s odkazem pripojil cizi adresu ke + sve firme. + +U pozvanky se rovnou rekne, jake role clovek dostane a jestli z nej ma byt +i **resitel**. Uzivatel a resitel nejsou totez, viz vyse - proto se to pta +misto hadani. + +Sprava je v zalozce **Lide**, ne v nastaveni: pozvat kolegu je bezna denni +prace. + ## Prehled nad firmou `GET /api/dashboard/tickets/workload` vraci pres cely tym: kolik ma kdo @@ -123,6 +164,42 @@ do korene a zaloguje - ztratit radek logu je horsi nez ho ukazat spatne zanoreny Komentare jsou taky zaznamy logu (`kind: 'note'`). Diky tomu je vsechno na jedne casove ose a nemusi se nikde skladat dohromady dva ruzne seznamy. +## Opakovana udalost + +Odesilatele umi poslat stejnou zpravu i osmdesatkrat za minutu. Kdyz prijde +**presne totez co posledne** (stejny typ, zdroj, popisek a stejna data), nezaklada +se dalsi radek: pricte se k pocitadlu u te predchozi. + +| Co se stane | Proc | +| --- | --- | +| `repeats` +1, `lastAt` = ted | osmdesat radku znamena, ze v historii nikdo nic nenajde | +| ticket se **nemeni** | jinak by duplikat rozblikal dashboard a spustil automatizaci na zmenu ticketu | +| do logu se nezapisuje nic | log ma ukazovat zmeny, ne to, ze se nezmenilo nic | +| udalost se **nezahazuje** | bez pocitadla by nikdo nezjistil, ze proti nam neco tluce ve smycce | + +Porovnava se jen s posledni udalosti. "Objednavka pripravena" muze legitimne +prijit znovu za hodinu, kdyz se mezitim stalo neco jineho - to je novy fakt. + +## Co je v logu videt + +Log se cte jako **prislo tohle a zpusobilo to tuhle zmenu**. Zmeny, ktere udela +beh automatizace, se radi pod jeho radek udalosti (`parentId`), takze nejde +zamenit, co zpusobilo co: + +``` +Prijata udalost: stav completed (automatizace TEST) 13:13:27 + stav in-progress -> completed 13:13:27 + stitky +hovor 13:13:27 +Prijata udalost: stav completed 22x beze zmeny 13:13:34 +``` + +U udalosti stoji, **kdo ji prinesl**. Prvni otazka nad zmenenym ticketem je +"kdo mi do toho sahl" a odpoved `webhook` na ni neodpovida. + +Data udalosti se ukladaji cela. Kdyz krok nema vlastni, ulozi se to, cim beh +zacal - u webhooku prijate telo. Prazdna udalost je horsi nez zadna: tvari se, +ze se neco stalo, a nerekne co. + ## Konektor Tickety Kategorie `servicedesk`, driv byl pod Nastroji. Ma obe strany: diff --git a/documentation/99-zmeny.md b/documentation/99-zmeny.md index 569f944..46f0d1f 100644 --- a/documentation/99-zmeny.md +++ b/documentation/99-zmeny.md @@ -2,6 +2,101 @@ Nejnovejsi nahore. +## 2026-08-17 - log rekne, co zpusobilo jakou zmenu + +Nalezeno na bezicim serveru: ticket TK-4946 mel 177 prichozich udalosti +a 620 radku v logu, pritom se nestalo skoro nic. Zmereno proti behum ve fronte: +ve stejnem okne vzniklo **presne tolik behu, kolik prislo udalosti** (22 a 22), +kazdy s jednim pokusem. Fronta ani worker nic nenasobi - odesilatel poslal +177 POSTu. Nasi vinou bylo, ze to z historie neslo poznat. + +### Opraveno + +- **Data udalosti se konecne ukladaji.** `payload` byl u kazde udalosti prazdny, + protoze krok `ticket/upsert` ukladal jen to, co si clovek vyplni v poli Data. + Kdyz je prazdne, ulozi se **to, cim beh zacal** - u webhooku cele prijate telo. + Prazdna udalost je horsi nez zadna: tvari se, ze se neco stalo, a nerekne co. +- **Shodna udalost se pocita, nezaklada dalsi radek.** Kdyz prijde presne totez + co posledne, pricte se k pocitadlu (`repeats`, `lastAt`) a v portalu se u radku + ukaze "22x beze zmeny, naposledy ...". Ticket se pritom **nemeni**: neprepise + se `updatedAt` ani se nerozesle zmena, takze duplikat nerozbliká dashboard + a nespusti automatizaci navazanou na zmenu ticketu. + - Zahodit duplikat nejde. Bez pocitadla by nikdo nezjistil, ze proti nam neco + tluce ve smycce. + - Porovnava se **jen s posledni** udalosti. "Objednavka pripravena" muze + legitimne prijit znovu za hodinu - to je novy fakt. +- **Zmeny se radi pod udalost, ktera je zpusobila.** Log uz mel strom + (`parentId`), ale nikdo ho nepouzival. Zapisy behu se ted radi pod jeho radek + udalosti, takze log se cte jako "prislo tohle -> zmenilo to tohle". +- **U udalosti stoji, kdo ji prinesl**: `Prijata udalost: stav completed + (automatizace TEST)`. Prvni otazka nad zmenenym ticketem je "kdo mi do toho + sahl" a `webhook` na ni neodpovida. +- **Poznamka o stavu jen kdyz se stav zmenil.** Predtim se u kazde udalosti + zapsalo `Stav zmenen z in-progress na in-progress`, tedy tri radky logu na + jednu zpravu, ktera nic nezmenila. Odtud 620 radku. +- **Popisek udalosti uz neni porad "Udalost".** Bere se popisek, predmet, stav, + externi ID - v tomhle poradi. Casova osa, kde je na kazdem radku totez, nerika + nic. +- Typograficke uvozovky z popisku v logu pryc, plus ctyri AI znaky, ktere + zbyvaly v kodu (`web/src/data/products.ts`, `References.tsx`, + `automationStore.ts`). + +### `runsToday` konecne znamena dnes + +Bylo to pocitadlo od zalozeni automatizace, jen se jmenovalo "dnes" - na serveru +ukazovalo 373 za automatizaci, ktera bezi tri mesice. + +- Automatizace si drzi **behy po dnech** (`days`, poslednich 14 dni). Po pulnoci + je "dnes" nula, dokud opravdu neco nebezi. +- Vedle toho `runsYesterday` a `runsTotal`. Puvodni cislo se neztratilo, jen se + spravne jmenuje. +- Uspesnost se pocita z dnesnich behu. Kdyz dnes zadny nebyl, bere se posledni + den, kdy byly - nula procent u automatizace, ktera dnes nemela co delat, by + vypadala jako porucha. +- V seznamu je pod dnesnim cislem vcerejsek, na detailu dnes, vcera i celkem. + +## 2026-08-17 - prevzeti ticketu ze skupiny a pozvanky do firmy + +### Pridano + +- **Prevzeti ticketu.** `POST /tickets/:id/claim`. Kdo je ve skupine, ktera ma + ticket u sebe, si ho vezme sam. Prace se nerozdava shora, lidi si ji beru + podle toho, kdo ma cas. + - Vzit ticket, ktery uz nekdo resi, je neco jineho: to je prehozeni a chce + to pravo `ticket.assign.others`. + - Ticket bez resitele si vezme kdokoli, kdo je vedeny jako resitel. +- **Krok `ticket/assign-group`** (Predat skupine) s prepinacem + **Priradit rovnou nejvolnejsimu**. Prazdne nebo Ne = ticket zustane ve fronte + skupiny. Prepinac je na kroku automatizace, ne na skupine: tataz skupina + potrebuje u havarie okamzite prideleni a u bezneho dotazu ne. +- **Pozvanky do firmy.** Spravce vytvori odkaz s nahodnym kodem + (`/pozvanka/:kod`), posle ho, jak chce. Kdo ho otevre, vyplni jmeno a heslo + a je uvnitr. + - **Heslo se nikdy neposila.** Nastavuje si ho sam clovek az za odkazem. + - Kdyz uz ucet ma, zada k nemu svoje heslo a jen se pripoji k dalsi firme. + Bez overeni hesla by kdokoliv s odkazem pripojil cizi adresu ke sve firme + a videl by jeji data. + - Pozvanka plati tyden, da se zrusit a po pouziti prestane platit sama. + - Volitelne z cloveka rovnou udela resitele. Ucetni muze mit pristup do + portalu, aniz by kdy resila ticket, proto se to pta. + +### Zmeneno + +- **Resitele, skupiny a pozvanky jsou na strance Lide**, ne v nastaveni. + Pozvat kolegu je bezna denni prace, ne nastaveni portalu - dokud to bylo + schovane v nastaveni, nikdo to nenasel. +- **Cleny skupiny se vybiraji klikanim** ze seznamu lidi. Predtim se opisovala + ID `ppl_xxx` oddelena carkou, coz je preklep cekajici na sve misto. + +### Nove soubory + +| Soubor | Co dela | +| --- | --- | +| `src/data/invites.ts` | Entita pozvanky, kod, platnost. | +| `src/routes/invites.ts` | Verejne cesty (prohlednuti a prijeti) a sprava. | +| `web/src/pages/Invite.tsx` | Stranka za odkazem: jmeno, e-mail, heslo. | +| `web/src/components/dashboard/InvitePanel.tsx` | Sprava pozvanek v zalozce Lide. | + ## 2026-08-13 - vyrizeno je vyslovny priznak, ne hadani ze stavu ### Zmeneno diff --git a/src/data/automationStore.ts b/src/data/automationStore.ts index 1449c63..9e3ce76 100644 --- a/src/data/automationStore.ts +++ b/src/data/automationStore.ts @@ -87,7 +87,7 @@ export type FlowStep = /** id parametru z trigger.fields */ fieldId: string; operator: ConditionOperator; - /** Chybi u operatoru, ktere hodnotu nepotrebuji (isEmpty, isTrue…). */ + /** Chybi u operatoru, ktere hodnotu nepotrebuji (isEmpty, isTrue apod.). */ value?: string; yes: FlowStep[]; no: FlowStep[]; @@ -111,7 +111,19 @@ export interface Automation { name: string; kind: AutomationKind; enabled: boolean; + /** + * Behy za **dnesni den**. Po pulnoci zacina od nuly. + * + * Driv to byl citac od zalozeni automatizace, jen se jmenoval "dnes". + * U automatizace, ktera bezi tri mesice, to rikalo cislo, ktere s dneskem + * nemelo nic spolecneho. + */ runsToday: number; + /** Behy za vcerejsi den, at je s cim srovnat. */ + runsYesterday: number; + /** Behy od zalozeni. Puvodni citac se neztratil, jen se spravne jmenuje. */ + runsTotal: number; + /** Uspesnost dnesnich behu. Kdyz dnes zadny nebyl, posledniho dne, kdy byly. */ successRate: number; avgDurationMs: number; lastRunAt: string; @@ -138,6 +150,15 @@ interface StoredAutomation { name: string; kind: AutomationKind; enabled: boolean; + /** + * Behy po dnech, nejnovejsi nakonec. Drzi se poslednich `KEEP_DAYS`. + * + * Proc historie a ne jen citac pro dnesek: bez ni po pulnoci neni s cim + * srovnat a nikdo nepozna, jestli je dnesnich devet malo nebo hodne. + */ + days?: Array<{ day: string; runs: number; ok: number }>; + /** Behy od zalozeni. */ + runsTotal?: number; runsToday: number; successRate: number; avgDurationMs: number; @@ -147,6 +168,53 @@ interface StoredAutomation { updatedAt: string; } +/** Kolik dni zpatky se drzi. Dva tydny staci na "je to dnes jinak nez obvykle". */ +const KEEP_DAYS = 14; + +/** Dnesni den jako `2026-08-17`. Podle nej se pozna prelom dne. */ +function today(): string { + return new Date().toISOString().slice(0, 10); +} + +function dayBefore(day: string): string { + return new Date(new Date(`${day}T00:00:00Z`).getTime() - 86_400_000).toISOString().slice(0, 10); +} + +/** + * Statistika za dnesek z ulozene historie. + * + * Kdyz dnes jeste nic nebezelo, vraci nulu - **ne vcerejsi cislo**. Ukazat + * vcerejsi pocet jako dnesni je to, co bylo spatne. + */ +function statsOf(stored: StoredAutomation): { + runsToday: number; + runsYesterday: number; + runsTotal: number; + successRate: number; +} { + const days = stored.days ?? []; + const now = today(); + const mine = days.find((entry) => entry.day === now); + const before = days.find((entry) => entry.day === dayBefore(now)); + + /* + * Uspesnost dnesnich behu. Kdyz dnes zadny nebyl, bere se posledni den, kdy + * byly - nula procent u automatizace, ktera dnes jen nemela co delat, by + * vypadala jako porucha. + */ + const source = mine ?? [...days].reverse().find((entry) => entry.runs > 0); + + return { + runsToday: mine?.runs ?? 0, + runsYesterday: before?.runs ?? 0, + runsTotal: stored.runsTotal ?? stored.runsToday, + successRate: + source && source.runs > 0 + ? Math.round((source.ok / source.runs) * 1000) / 10 + : stored.successRate, + }; +} + function minutesAgo(minutes: number): string { return new Date(Date.now() - minutes * 60_000).toISOString(); } @@ -374,7 +442,7 @@ function seed( */ function seedDemoAutomations(): void { seed({ - name: 'Objednávka → sklad → fakturace', + name: 'Objednávka, sklad a fakturace', enabled: true, runsToday: 428, successRate: 99.3, @@ -888,9 +956,20 @@ if (!config.seedWebhookToken) { } function toSummary(stored: StoredAutomation): Automation { - const { flow: _flow, createdAt: _createdAt, updatedAt: _updatedAt, ...rest } = stored; + const { + flow: _flow, + createdAt: _createdAt, + updatedAt: _updatedAt, + days: _days, + ...rest + } = stored; return { ...rest, + /* + * Cisla se pocitaji z historie po dnech, ne z ulozeneho citace. Po pulnoci + * je "dnes" nula, dokud opravdu neco nebezi. + */ + ...statsOf(stored), stepCount: countSteps(stored.flow.steps), configured: stored.flow.trigger !== null, issues: collectFlowIssues(stored.flow), @@ -1077,15 +1156,34 @@ export function recordRun(id: string, ok = true, tenantIds?: string[]): Automati return undefined; } - const runs = stored.runsToday + 1; - // Klouzavy prumer uspesnosti pres dnesni behy, ne prepisovani na 0 nebo 100. - const previousOk = (stored.successRate / 100) * stored.runsToday; - const successRate = runs === 0 ? 100 : ((previousOk + (ok ? 1 : 0)) / runs) * 100; + /* + * Zapisuje se do dnesniho dne. Po pulnoci vznikne novy zaznam, takze citac + * nepokracuje pres den - to byla puvodni chyba. + */ + const now = today(); + const days = [...(stored.days ?? [])]; + const index = days.findIndex((entry) => entry.day === now); + + if (index === -1) { + days.push({ day: now, runs: 1, ok: ok ? 1 : 0 }); + } else { + days[index] = { + day: now, + runs: days[index].runs + 1, + ok: days[index].ok + (ok ? 1 : 0), + }; + } + if (days.length > KEEP_DAYS) days.splice(0, days.length - KEEP_DAYS); + + const total = (stored.runsTotal ?? stored.runsToday) + 1; + const dayEntry = days.find((entry) => entry.day === now)!; const updated: StoredAutomation = { ...stored, - runsToday: runs, - successRate: Math.round(successRate * 10) / 10, + days, + runsTotal: total, + runsToday: dayEntry.runs, + successRate: Math.round((dayEntry.ok / dayEntry.runs) * 1000) / 10, lastRunAt: new Date().toISOString(), }; save(updated); diff --git a/src/data/bootstrap.ts b/src/data/bootstrap.ts index 568fe1c..93c4550 100644 --- a/src/data/bootstrap.ts +++ b/src/data/bootstrap.ts @@ -13,6 +13,7 @@ import { refreshActions, actionPermissions, actionStore, seedActions } from './t import { refreshCustomWidgets, customWidgetStore, seedCustomWidgets } from './customWidgets.js'; import { auditStore } from './audit.js'; import { notificationStore, refreshNotifications } from './notifications.js'; +import { inviteStore, refreshInvites } from './invites.js'; import { onTicket } from './ticketHooks.js'; import { onTicketEvent } from '../runtime/triggers.js'; import { initQueue } from '../runtime/queue.js'; @@ -61,6 +62,7 @@ const entities: Array<{ // Audit vychozi sadu nema, zaznamy vznikaji az provozem. { store: auditStore as EntityStore }, { store: notificationStore as EntityStore }, + { store: inviteStore as EntityStore }, ]; /** @@ -93,6 +95,7 @@ export async function refreshCaches(): Promise { refreshActions(), refreshCustomWidgets(), refreshNotifications(), + refreshInvites(), ]); // Prava k akcim vznikaji z definic akci, takze se registruji az po jejich nacteni. diff --git a/src/data/invites.ts b/src/data/invites.ts new file mode 100644 index 0000000..040ba3f --- /dev/null +++ b/src/data/invites.ts @@ -0,0 +1,121 @@ +/** + * Pozvanky do firmy. + * + * Spravce firmy vytvori odkaz s neodhadnutelnym kodem a posle ho, jak chce - + * mailem, na Slacku, diktatem do telefonu. Kdo odkaz otevre, vyplni jmeno + * a heslo a je uvnitr. + * + * Proc odkaz a ne "zaloz uzivatele a posli mu heslo": + * - **heslo nikdy nikam neposilame**. Kdo si ho nastavi sam, ma ho jen on. + * - pozvanka **vyprsi** a da se zrusit, ucet ne, + * - kdo uz ucet ma, se jen pripoji k dalsi firme misto zakladani druheho. + * + * Kod je jedina ochrana, proto je dlouhy a nahodny - stejne jako u webhooku. + */ + +import { randomBytes, randomUUID } from 'node:crypto'; +import { defineStore, nowIso, type TenantEntity } from './store/index.js'; +import { withCache } from './store/cached.js'; + +export interface Invite extends TenantEntity { + /** Firma, do ktere pozvanka zve. */ + tenantId: string; + /** Neodhadnutelny kod v odkazu. */ + code: string; + /** + * Komu je urcena. Prazdne = komukoliv, kdo odkaz dostane. + * + * Kdyz je vyplnena, musi souhlasit s tim, co clovek zada. Bez toho by odkaz + * poslany dal fungoval komukoliv, coz u pozvanky do firmy neni v poradku. + */ + email: string; + /** Jmeno napovedou, at spravce v seznamu pozna, komu ji poslal. */ + note: string; + /** Jake role dostane v te firme. */ + roleIds: string[]; + /** + * Ma se z nej stat i resitel, tedy nekdo, komu jde prehodit ticket? + * + * Uzivatel a resitel nejsou totez: ucetni muze mit pristup do portalu, aniz + * by kdy resila ticket. Proto se to pta, misto aby se hadalo. + */ + asPerson: boolean; + expiresAt: string; + usedAt: string | null; + usedByUserId: string | null; + createdByEmail: string; +} + +export const inviteStore = defineStore('invite'); +const cache = withCache(inviteStore); + +/** Jak dlouho pozvanka plati. Tyden je dost na dovolenou a malo na zapomenuti. */ +const VALID_DAYS = 7; + +export async function refreshInvites(): Promise { + await cache.refresh(); +} + +export interface CreateInviteInput { + tenantId: string; + email?: string; + note?: string; + roleIds: string[]; + asPerson?: boolean; + createdByEmail: string; +} + +export function newInvite(input: CreateInviteInput): Invite { + const timestamp = nowIso(); + return { + id: `inv_${randomUUID().slice(0, 12)}`, + tenantId: input.tenantId, + code: randomBytes(24).toString('base64url'), + email: (input.email ?? '').trim().toLowerCase(), + note: input.note ?? '', + roleIds: input.roleIds, + asPerson: input.asPerson ?? true, + expiresAt: new Date(Date.now() + VALID_DAYS * 86_400_000).toISOString(), + usedAt: null, + usedByUserId: null, + createdByEmail: input.createdByEmail, + createdAt: timestamp, + updatedAt: timestamp, + }; +} + +/** Pozvanka podle kodu. Nezkoumá platnost, na to je `inviteProblem`. */ +export function findByCode(code: string): Invite | undefined { + if (code.length < 16) return undefined; + return cache.find((invite) => invite.code === code); +} + +/** + * Proc pozvanku nejde pouzit. `null` = je v poradku. + * + * Duvody jsou zamerne konkretni: "uz byla pouzita" a "vyprsela" vedou k jine + * reakci a schovat je za "neplatna pozvanka" znamena, ze clovek nevi, co dal. + */ +export function inviteProblem(invite: Invite): string | null { + if (invite.usedAt) return 'Tahle pozvánka už byla použitá.'; + if (new Date(invite.expiresAt).getTime() < Date.now()) { + return 'Pozvánka vypršela, požádejte o novou.'; + } + return null; +} + +export function listInvites(tenantIds: string[]): Invite[] { + return cache + .all() + .filter((invite) => tenantIds.includes(invite.tenantId)) + .sort((a, b) => b.createdAt.localeCompare(a.createdAt)); +} + +export async function markUsed(invite: Invite, userId: string): Promise { + await inviteStore.update( + invite.id, + { usedAt: nowIso(), usedByUserId: userId }, + { tenantIds: [invite.tenantId], includeGlobal: true }, + ); + await cache.refresh(); +} diff --git a/src/data/people.ts b/src/data/people.ts index 6e684f4..2387527 100644 --- a/src/data/people.ts +++ b/src/data/people.ts @@ -127,6 +127,13 @@ export interface PersonGroup extends TenantEntity { personIds: string[]; } +/** Skupiny, jejichz clenem ten clovek je. */ +export function groupsOfPerson(personId: string, tenantIds: string[]): PersonGroup[] { + return groupCache + .all() + .filter((group) => tenantIds.includes(group.tenantId) && group.personIds.includes(personId)); +} + export const groupStore = defineStore('personGroup'); const groupCache = withCache(groupStore); diff --git a/src/data/services.ts b/src/data/services.ts index 22e43be..23e7c47 100644 --- a/src/data/services.ts +++ b/src/data/services.ts @@ -610,6 +610,41 @@ export const services: Service[] = [ { id: 'created', name: 'created', type: 'boolean', required: true }, ], }, + { + id: 'assign-group', + name: 'Předat skupině', + description: + 'Ticket se objeví ve frontě skupiny a kdo má čas, si ho převezme. ' + + 'Když zaškrtnete rovnou přiřadit, dostane ho hned ten, kdo má nejmíň práce - ' + + 'to se hodí tam, kde se čeká na rychlou reakci.', + implementation: 'script', + inputs: [ + { + id: 'groupId', + label: 'Skupina', + kind: 'choice', + required: true, + options: [], + optionsFrom: 'groups', + }, + { + id: 'autoAssign', + label: 'Rovnou přiřadit nejvolnějšímu', + kind: 'choice', + required: false, + options: [ + { value: 'true', label: 'Ano' }, + { value: 'false', label: 'Ne, nechat ve frontě skupiny' }, + ], + hint: 'Prázdné = nechat ve frontě, ať si to lidé vezmou sami.', + }, + { id: 'ticketId', label: 'Ticket', kind: 'text', required: false }, + ], + outputFields: [ + { id: 'groupId', name: 'groupId', type: 'string', required: true }, + { id: 'groupName', name: 'groupName', type: 'string', required: true }, + ], + }, { id: 'assign-least-busy', name: 'Předat nejvolnějšímu ze skupiny', @@ -623,6 +658,8 @@ export const services: Service[] = [ label: 'Skupina', kind: 'choice', required: true, + options: [], + optionsFrom: 'groups', hint: 'Například sklad nebo IT. Skupiny se spravují v Nastavení.', }, { diff --git a/src/data/ticketStore.ts b/src/data/ticketStore.ts index 6dca92a..978f170 100644 --- a/src/data/ticketStore.ts +++ b/src/data/ticketStore.ts @@ -18,6 +18,7 @@ import { config } from '../config.js'; import { publish } from '../events/bus.js'; +import { currentRun } from '../runtime/context.js'; import { onTicketChanged } from './ticketHooks.js'; import { notify } from './notifications.js'; import { findPerson, type Person } from './people.js'; @@ -112,6 +113,17 @@ export interface TicketEvent { /** Cela prijata data. Nic se nezahazuje, at jde dohledat, co presne prislo. */ payload: Record; at: string; + /** + * Kolikrat prislo totež. Vetsinou 1. + * + * Odesilatele umi poslat stejnou zpravu i osmdesatkrat za minutu. Zalozit + * osmdesat radku znamena, ze v historii nikdo nic nenajde - a pritom se + * nestalo nic noveho. Proto se shodna udalost **pocita** u te predchozi. + * Zahodit ji nejde: bez pocitadla by nikdo nezjistil, ze odesilatel tluce. + */ + repeats: number; + /** Kdy prisla naposledy. Same jako `at`, dokud se neopakuje. */ + lastAt: string; } export interface Ticket { @@ -312,7 +324,18 @@ export async function initTickets(): Promise { const { trace, events: rowEvents, ...stored } = row; tickets.push(stored); traces.set(row.id, trace ?? []); - events.set(row.id, rowEvents ?? []); + /* + * Pocitadlo opakovani pribylo pozdeji. Ulozene udalosti ho nemaji, takze + * se doplni pri nacteni - jinak by se v portalu ukazovalo "undefinedx". + */ + events.set( + row.id, + (rowEvents ?? []).map((event) => ({ + ...event, + repeats: event.repeats ?? 1, + lastAt: event.lastAt ?? event.at, + })), + ); } // Citac musi pokracovat za nejvyssim ulozenym cislem, jinak by nove tickety @@ -386,7 +409,12 @@ export function appendTrace(ticketId: string, inputs: TraceInput[]): number { return 0; } const before = existing.length; - flattenTrace(inputs, null, existing); + /* + * Kdyz zapis patri behu automatizace, radi se pod jeho radek udalosti. + * Diky tomu je v logu videt "prislo tohle -> zmenilo to tohle" misto dvou + * vet vedle sebe, u kterych se jen hada, jestli spolu souvisi. + */ + flattenTrace(inputs, currentRun()?.traceParent.id ?? null, existing); // Log je soucast ulozeneho ticketu, takze zapis do logu je zmena ticketu. const ticket = tickets.find((item) => item.id === ticketId); @@ -1030,6 +1058,13 @@ export interface IntakeResult { /** true = ticket teprve ted vznikl, false = udalost se navesila na existujici. */ created: boolean; event: TicketEvent; + /** + * true = prislo presne totez co posledne, takze se jen pricetlo k pocitadlu. + * Ticket se **nemenil**: nesmi se kvuli tomu prepsat `updatedAt`, ani + * rozeslat zmena - jinak by kazdy duplikat rozblikal dashboard a mohl + * spustit automatizaci navazanou na zmenu ticketu. + */ + repeated: boolean; } /** @@ -1053,6 +1088,8 @@ export function intakeEvent(input: IntakeInput): IntakeResult { label: input.label ?? input.type, payload: input.payload ?? {}, at: timestamp, + repeats: 1, + lastAt: timestamp, }; const existing = input.externalId @@ -1062,6 +1099,18 @@ export function intakeEvent(input: IntakeInput): IntakeResult { : undefined; if (existing) { + /* + * Prislo presne totez co posledne? Pak se to jen pricte. Zadny novy radek, + * zadny zapis do logu, zadna zmena ticketu - nic se totiz nestalo. + */ + const repeat = sameAsLast(existing.id, event); + if (repeat) { + repeat.repeats += 1; + repeat.lastAt = event.at; + persist(existing); + return { ticket: toTicket(existing), created: false, event: repeat, repeated: true }; + } + appendEvent(existing, event); if (input.fields) existing.fields = { ...(existing.fields ?? {}), ...input.fields }; @@ -1070,15 +1119,31 @@ export function intakeEvent(input: IntakeInput): IntakeResult { } touch(existing); + /* + * Rodic musi byt prazdny, jinak by se radek udalosti zaradil pod udalost + * predchozi - `appendTrace` doplnuje rodice z behu. + */ + const run = currentRun(); + const parent = run?.traceParent; + if (parent) parent.id = null; + appendTrace(existing.id, [ - { kind: 'trigger', status: 'info', label: `Přijata událost: ${event.label}`, response: describePayload(event.payload) }, + { + kind: 'trigger', + status: 'info', + label: `Přijata událost: ${event.label}${originOf(run, event)}`, + response: describePayload(event.payload), + }, ]); + + // Dalsi zapisy tohohle behu uz patri pod tenhle radek. + if (parent) parent.id = lastTraceId(existing.id); publish('ticket.updated', `Ticket ${existing.id}: ${event.label}`, { ticketId: existing.id, externalId: existing.externalId, }); - return { ticket: toTicket(existing), created: false, event }; + return { ticket: toTicket(existing), created: false, event, repeated: false }; } const created = createTicket({ @@ -1101,7 +1166,7 @@ export function intakeEvent(input: IntakeInput): IntakeResult { { kind: 'trigger', status: 'info', - label: `Ticket vznikl z události: ${event.label}`, + label: `Ticket vznikl z události: ${event.label}${originOf(currentRun(), event)}`, response: describePayload(event.payload), }, ], @@ -1113,7 +1178,16 @@ export function intakeEvent(input: IntakeInput): IntakeResult { persist(stored); } - return { ticket: toTicket(stored ?? created as unknown as StoredTicket), created: true, event }; + // Zmeny, ktere beh udela dal, patri pod radek o vzniku ticketu. + const parent = currentRun()?.traceParent; + if (parent) parent.id = traces.get(created.id)?.[0]?.id ?? null; + + return { + ticket: toTicket(stored ?? (created as unknown as StoredTicket)), + created: true, + event, + repeated: false, + }; } /** Kolik udalosti se u jednoho ticketu drzi. Starsi se odmazavaji. */ @@ -1127,6 +1201,28 @@ function appendEvent(ticket: StoredTicket, event: TicketEvent): void { events.set(ticket.id, list); } +/** + * Je to totez, co prislo naposledy? + * + * Porovnava se **jen s posledni** udalosti, ne s celou historii. "Objednavka + * pripravena" muze legitimne prijit znovu za hodinu, kdyz se mezitim stalo + * neco jineho - to je novy fakt. Dvacet stejnych zprav v rade uz ne. + * + * Data se srovnavaji pres JSON, protoze na poradi klicu v prijate zprave + * nezalezi jen vyjimecne a levnejsi porovnani neni potreba. + */ +function sameAsLast(ticketId: string, event: TicketEvent): TicketEvent | null { + const list = events.get(ticketId); + const last = list && list.length > 0 ? list[list.length - 1] : undefined; + if (!last) return null; + + if (last.type !== event.type || last.source !== event.source || last.label !== event.label) { + return null; + } + if (JSON.stringify(last.payload) !== JSON.stringify(event.payload)) return null; + return last; +} + let eventCounter = 0; function nextEventId(): string { @@ -1134,6 +1230,24 @@ function nextEventId(): string { return `tev_${eventCounter.toString(36)}`; } +/** + * Kdo udalost prinesl. + * + * V logu je to to nejdulezitejsi: kdyz se ticket zmenil, prvni otazka je "kdo + * mi do toho sahl". Jmeno automatizace na to odpovi, `webhook` uz ne. + */ +function originOf(run: { automationName: string } | undefined, event: TicketEvent): string { + if (run) return ` (automatizace ${run.automationName})`; + if (event.source && event.source !== 'webhook') return ` (${event.source})`; + return ''; +} + +/** ID posledniho radku logu. Pod nej se radi zmeny, ktere z udalosti plynou. */ +function lastTraceId(ticketId: string): string | null { + const list = traces.get(ticketId); + return list && list.length > 0 ? list[list.length - 1].id : null; +} + /** Kratky popis dat do logu. Cela data zustavaji v udalosti. */ function describePayload(payload: Record): string | null { const keys = Object.keys(payload); @@ -1370,13 +1484,20 @@ export function updateTicketStatus( markResponded(ticket); touch(ticket); - appendTrace(id, [ - { - kind: 'note', - label: `Stav změněn z „${previous}" na „${status}".`, - status: 'info', - }, - ]); + /* + * Poznamka jen kdyz se neco zmenilo. "Stav zmenen z completed na completed" + * je rada, ktera se ctenari pise do historie u kazde prichozi zpravy, a po + * dvaceti takovych se v logu neda nic najit. + */ + const statusChanged = previous !== status; + const closedChanged = closed !== undefined && closed !== wasClosed; + + if (statusChanged || closedChanged) { + const parts: string[] = []; + if (statusChanged) parts.push(`stav ${previous} -> ${status}`); + if (closedChanged) parts.push(ticket.closed ? 'oznaceno jako vyrizene' : 'znovu otevreno'); + appendTrace(id, [{ kind: 'note', label: parts.join(', '), status: 'info' }]); + } if (ticket.closed) { publish('ticket.resolved', `Ticket ${ticket.id} vyřešen: ${ticket.subject}`, { @@ -1499,12 +1620,14 @@ export function assignTicketGroup( id: string, groupId: string | null, tenantIds: string[], + /** false = nechat resitele, jak je. Pouziva to prevzeti ticketu. */ + clearAssignee = true, ): Ticket | undefined { const ticket = findWritable(id, tenantIds); if (!ticket) return undefined; ticket.assigneeGroupId = groupId; - if (groupId) ticket.assigneeId = null; + if (groupId && clearAssignee) ticket.assigneeId = null; touch(ticket); appendTrace(id, [ @@ -1518,6 +1641,47 @@ export function assignTicketGroup( return toTicket(ticket); } +/** + * Prevzeti ticketu. + * + * Clovek si vezme praci sam, misto aby cekal, az mu ji nekdo prideli. Smi to + * jen u ticketu **ze sve skupiny**, nebo u takoveho, ktery nema nikoho - + * brat cizi rozdelanou praci by znamenalo, ze o ni prijde ten, kdo ji resi. + * + * Vraci `undefined`, kdyz to nejde; duvod rekne volajici, ktery zna kontext. + */ +export function claimTicket( + id: string, + personId: string, + tenantIds: string[], +): Ticket | undefined { + const ticket = findWritable(id, tenantIds); + if (!ticket) return undefined; + + const person = findPerson(personId); + if (!person || person.tenantId !== ticket.tenantId) return undefined; + + ticket.assigneeId = person.id; + markResponded(ticket); + touch(ticket); + + appendTrace(id, [ + { kind: 'note', status: 'info', label: `${person.name} si ticket převzal` }, + ]); + publish('ticket.assigned', `${person.name} si vzal ticket ${ticket.id}`, { + ticketId: ticket.id, + assigneeId: person.id, + }); + return toTicket(ticket); +} + +/** Ma ticket uz nekoho? Pro rozhodnuti, jestli jde prevzit. */ +export function ticketAssignee(id: string, tenantIds: string[]): string | null | undefined { + const ticket = tickets.find((item) => item.id === id); + if (!ticket || !tenantIds.includes(ticket.tenantId)) return undefined; + return ticket.assigneeId; +} + export function addComment( id: string, author: string, diff --git a/src/index.ts b/src/index.ts index a770a24..2138677 100644 --- a/src/index.ts +++ b/src/index.ts @@ -14,6 +14,7 @@ import { buildOpenApiDocument } from './openapi.js'; import { authRouter } from './routes/auth.js'; import { contactRouter } from './routes/contact.js'; import { dashboardRouter } from './routes/dashboard.js'; +import { publicInviteRouter } from './routes/invites.js'; import { webhookRouter } from './routes/webhook.js'; import { bootstrapData } from './data/bootstrap.js'; import { flushConnectorStore, initConnectorStore, storageStatus } from './data/connectorStore.js'; @@ -114,6 +115,8 @@ api.use( api.use('/api/auth', authRouter); api.use('/api/dashboard', dashboardRouter); +// Verejne: kdo dostal odkaz na pozvanku, neni jeste prihlaseny. +api.use('/api/invites', publicInviteRouter); api.use('/api/admin', adminRouter); api.use('/api/contact', contactRouter); api.use('/webhook', webhookRouter); diff --git a/src/openapi.ts b/src/openapi.ts index dbcb21c..9ad239e 100644 --- a/src/openapi.ts +++ b/src/openapi.ts @@ -634,6 +634,8 @@ export function buildOpenApiDocument() { kind: { type: 'string', enum: ['workflow', 'voicebot', 'integrace', 'report'] }, enabled: { type: 'boolean' }, runsToday: { type: 'integer' }, + runsYesterday: { type: 'integer' }, + runsTotal: { type: 'integer' }, successRate: { type: 'number' }, avgDurationMs: { type: 'integer' }, lastRunAt: { type: 'string', format: 'date-time' }, diff --git a/src/routes/dashboard.ts b/src/routes/dashboard.ts index 71db5a4..7ba0cdc 100644 --- a/src/routes/dashboard.ts +++ b/src/routes/dashboard.ts @@ -42,9 +42,10 @@ import { collectScopes } from '../data/flowScope.js'; import { widgetCatalog } from '../data/widgets.js'; import { listIncidents } from '../data/incidentStore.js'; import { getSummary } from '../data/mock.js'; -import { findPersonByEmail, listGroups, listPeople } from '../data/people.js'; +import { findPersonByEmail, groupsOfPerson, listGroups, listPeople } from '../data/people.js'; import { hasPermission } from '../data/permissions.js'; import { listNotifications, markRead, unreadCount } from '../data/notifications.js'; +import { inviteRouter } from './invites.js'; import { queueStats, recentRuns } from '../runtime/queue.js'; import { recordAudit } from '../data/audit.js'; import { findTenant, generateIntakeToken, refreshTenants, tenantStore } from '../data/tenants.js'; @@ -52,6 +53,8 @@ import { listTicketTypes } from '../data/ticketTypes.js'; import { addComment, assignTicket, + claimTicket, + ticketAssignee, getAgentStats, getTicket, getWorkload, @@ -541,6 +544,74 @@ dashboardRouter.get('/tickets', (req, res) => { }); }); +/** + * Prevzeti ticketu. + * + * Clovek si vezme praci sam, misto aby cekal, az mu ji nekdo prideli. Tohle je + * bezny zpusob prace v tymu: ticket lezi ve fronte skupiny a kdo ma cas, si ho + * vezme. + * + * Smi to u ticketu **ze sve skupiny** nebo u takoveho, ktery nema nikoho. + * Vzit nekomu rozdelanou praci nejde - to uz je prehozeni a na to je pravo + * `ticket.assign.others`. + */ +dashboardRouter.post('/tickets/:id/claim', (req, res) => { + const scope = scopeOrDeny(req, res); + if (!scope) return; + + const access = accessFor(req.user!); + if (!access.personId) { + return res.status(400).json({ + error: 'not_a_person', + message: 'Nejste vedený jako řešitel, takže si ticket nemůžete převzít.', + }); + } + + const tenantIds = writableTenants(req); + const current = ticketAssignee(req.params.id, tenantIds); + if (current === undefined) { + return res.status(404).json({ error: 'not_found', message: 'Ticket neexistuje.' }); + } + + const ticket = getTicket(req.params.id, tenantIds); + if (!ticket) return res.status(404).json({ error: 'not_found', message: 'Ticket neexistuje.' }); + + if (current !== null && current !== access.personId) { + // Uz to nekdo resi. Prehodit praci nekomu jinemu je jine rozhodnuti. + if (!hasPermission(req.user!, 'ticket.assign.others')) { + return res.status(409).json({ + error: 'already_assigned', + message: 'Ticket už někdo řeší. Přehodit ho může jen ten, kdo na to má právo.', + }); + } + } + + const mine = groupsOfPerson(access.personId, tenantIds).map((group) => group.id); + const inMyGroup = ticket.assigneeGroupId !== null && mine.includes(ticket.assigneeGroupId); + + if (!inMyGroup && current !== null && current !== access.personId) { + return res.status(403).json({ + error: 'forbidden', + message: 'Tenhle ticket není ve vaší skupině.', + }); + } + + const updated = claimTicket(req.params.id, access.personId, tenantIds); + if (!updated) { + return res.status(400).json({ error: 'failed', message: 'Převzetí se nepodařilo.' }); + } + + recordAudit({ + userId: req.user!.id, + userEmail: req.user!.email, + tenantId: updated.tenantId, + action: 'ticket.claim', + target: updated.id, + }); + + return res.json(updated); +}); + /** Kdo co ma u sebe. MUSI byt pred /tickets/:id, jinak by to spadlo na detail. */ dashboardRouter.get('/tickets/workload', (req, res) => { const scope = scopeOrDeny(req, res); @@ -665,6 +736,9 @@ dashboardRouter.use('/connectors', connectorsRouter); // Nastaveni vsech entit. Cele stoji na fabrice crudRouter. dashboardRouter.use('/settings', settingsRouter); +// Pozvanky do firmy. +dashboardRouter.use('/invites', inviteRouter); + // Data pro vlastni widgety. Jeden request na cely prehled. dashboardRouter.use('/widget-data', widgetDataRouter); diff --git a/src/routes/invites.ts b/src/routes/invites.ts new file mode 100644 index 0000000..ac00e9e --- /dev/null +++ b/src/routes/invites.ts @@ -0,0 +1,334 @@ +/** + * Pozvanky do firmy. + * + * Dva druhy cest v jednom souboru, protoze patri k sobe: + * - **za prihlasenim**: spravce firmy pozvanku vytvori, vidi seznam a muze ji + * zrusit, + * - **verejne**: kdo dostal odkaz, si prohledne, kam ho zve, a prijme ji. + * + * Verejna cast je zamerne uzka: rekne jen nazev firmy a jestli je pozvanka + * platna. Nic o tom, kdo ve firme je, ani co obsahuje - kod v adrese je jedina + * ochrana a odkaz se snadno dostane, kam nema. + */ + +import { Router } from 'express'; +import { z } from 'zod'; +import { recordAudit } from '../data/audit.js'; +import { publicBaseUrl } from '../config.js'; +import { + findByCode, + inviteProblem, + inviteStore, + listInvites, + markUsed, + newInvite, + refreshInvites, +} from '../data/invites.js'; +import { rolesFor, type Role } from '../data/permissions.js'; +import { personStore, refreshPeople, listPeople } from '../data/people.js'; +import { findTenant } from '../data/tenants.js'; +import bcrypt from 'bcryptjs'; +import { + findUserByEmail, + hashPassword, + listAllUsers, + refreshUsers, + userStore, +} from '../data/users.js'; +import { nowIso } from '../data/store/index.js'; +import { requireAuth } from '../middleware/auth.js'; +import { currentTenant } from './crud.js'; + +/** Verejne cesty. Bez prihlaseni, autorizuje kod v adrese. */ +export const publicInviteRouter = Router(); + +/** Cesty za prihlasenim: sprava pozvanek. */ +export const inviteRouter = Router(); + +// ------------------------------------------------------------------ verejne + +/** + * Co je za timhle odkazem. + * + * Vraci minimum: nazev firmy a jestli to jde prijmout. Kdyz uz e-mail znamy, + * rekne ho, aby ho clovek nemusel psat - ale nic navic. + */ +publicInviteRouter.get('/:code', (req, res) => { + const invite = findByCode(req.params.code); + if (!invite) { + return res.status(404).json({ error: 'not_found', message: 'Pozvánka neexistuje.' }); + } + + const problem = inviteProblem(invite); + const tenant = findTenant(invite.tenantId); + + return res.json({ + tenant: tenant?.name ?? 'Neznámá firma', + email: invite.email || null, + /** Uz ucet ma? Pak staci heslo, ne registrace. */ + knownUser: invite.email ? findUserByEmail(invite.email) !== undefined : false, + valid: problem === null, + problem, + }); +}); + +const acceptSchema = z.object({ + name: z.string().trim().min(2).max(80), + email: z.string().trim().email(), + password: z.string().min(8, 'Heslo musí mít aspoň 8 znaků.'), +}); + +/** + * Prijeti pozvanky. + * + * Tri pripady: + * - ucet neexistuje: zalozi se a pripoji k firme, + * - ucet existuje a heslo sedi: jen se pripoji k firme, + * - ucet existuje a heslo nesedi: odmitnuto. + * + * Heslo se **nikdy neposila** - clovek si ho nastavi sam prave tady. + */ +publicInviteRouter.post('/:code/accept', (req, res) => { + const invite = findByCode(req.params.code); + if (!invite) { + return res.status(404).json({ error: 'not_found', message: 'Pozvánka neexistuje.' }); + } + + const problem = inviteProblem(invite); + if (problem) return res.status(409).json({ error: 'invalid_invite', message: problem }); + + const parsed = acceptSchema.safeParse(req.body ?? {}); + if (!parsed.success) { + return res.status(400).json({ + error: 'validation_error', + message: parsed.error.issues[0]?.message ?? 'Neplatné údaje.', + }); + } + + const email = parsed.data.email.toLowerCase(); + + /* + * Kdyz je pozvanka na konkretni adresu, musi souhlasit. Bez toho by odkaz + * poslany dal fungoval komukoliv, coz u pristupu do firmy neni v poradku. + */ + if (invite.email && invite.email !== email) { + return res.status(403).json({ + error: 'wrong_email', + message: 'Tahle pozvánka je pro jinou e-mailovou adresu.', + }); + } + + const existing = findUserByEmail(email); + + return void (async () => { + let userId: string; + + if (existing) { + // Ucet uz je. Pripojime ho k firme, ale jen kdyz prokaze, ze je jeho. + /* + * Ucet uz existuje, takze musi prokazat, ze je jeho. Bez toho by kdokoliv + * s odkazem pripojil cizi adresu ke sve firme a videl by jeji data. + */ + const stored = listAllUsers().find((item) => item.id === existing.id); + const passwordOk = + stored !== undefined && (await bcrypt.compare(parsed.data.password, stored.passwordHash)); + + if (!passwordOk) { + return res.status(401).json({ + error: 'wrong_password', + message: 'S touhle adresou už účet existuje. Zadejte k němu svoje heslo.', + }); + } + + const already = existing.memberships.some( + (membership) => membership.tenantId === invite.tenantId, + ); + if (!already) { + await userStore.update( + existing.id, + { + memberships: [ + ...existing.memberships, + { tenantId: invite.tenantId, roleIds: invite.roleIds }, + ], + }, + { tenantIds: [], includeGlobal: true }, + ); + } + userId = existing.id; + } else { + const timestamp = nowIso(); + const created = await userStore.create({ + id: `usr_${Math.random().toString(36).slice(2, 10)}`, + tenantId: null, + email, + name: parsed.data.name, + passwordHash: hashPassword(parsed.data.password), + platformAdmin: false, + enabled: true, + memberships: [{ tenantId: invite.tenantId, roleIds: invite.roleIds }], + createdAt: timestamp, + updatedAt: timestamp, + }); + userId = created.id; + } + + await refreshUsers(); + + /* + * Resitel je nekdo, komu jde prehodit ticket. Neni to totez co ucet: + * ucetni muze mit pristup do portalu, aniz by kdy resila ticket. + */ + if (invite.asPerson) { + const alreadyPerson = listPeople([invite.tenantId]).some( + (person) => person.email.toLowerCase() === email, + ); + if (!alreadyPerson) { + const timestamp = nowIso(); + await personStore.create({ + id: `ppl_${Math.random().toString(36).slice(2, 10)}`, + tenantId: invite.tenantId, + name: parsed.data.name, + email, + role: '', + capacity: 8, + enabled: true, + externalIds: [], + createdAt: timestamp, + updatedAt: timestamp, + }); + await refreshPeople(); + } + } + + await markUsed(invite, userId); + + recordAudit({ + userId, + userEmail: email, + tenantId: invite.tenantId, + action: 'invite.accept', + target: invite.id, + detail: { pozval: invite.createdByEmail }, + }); + + console.info(`[pozvanky] ${email} prijal pozvanku do ${invite.tenantId}`); + return res.status(201).json({ ok: true, email }); + })().catch((err: unknown) => { + console.error('[pozvanky] prijeti selhalo:', err); + return res + .status(500) + .json({ error: 'internal_error', message: 'Pozvánku se nepodařilo přijmout.' }); + }); +}); + +// ----------------------------------------------------------------- za prihlasenim + +inviteRouter.use(requireAuth); + +/** Kdo smi zvat. Zvat lidi do firmy je totez co spravovat uzivatele. */ +const PERMISSION = 'user.manage'; + +inviteRouter.get('/', (req, res) => { + const tenantId = currentTenant(req, res); + if (!tenantId) return; + + const base = `${publicBaseUrl()}/pozvanka/`; + return res.json({ + items: listInvites([tenantId]).map((invite) => ({ + ...invite, + /** Cela adresa, at ji jde rovnou zkopirovat a poslat. */ + url: `${base}${invite.code}`, + problem: inviteProblem(invite), + })), + roles: rolesFor(tenantId).map((role: Role) => ({ + id: role.id, + key: role.key, + name: role.name, + })), + }); +}); + +const createSchema = z.object({ + email: z.string().trim().email().optional().or(z.literal('')), + note: z.string().trim().max(120).optional(), + roleIds: z.array(z.string().min(1)).min(1, 'Vyberte aspoň jednu roli.'), + asPerson: z.boolean().default(true), +}); + +inviteRouter.post('/', (req, res) => { + const tenantId = currentTenant(req, res); + if (!tenantId) return; + + const parsed = createSchema.safeParse(req.body ?? {}); + if (!parsed.success) { + return res.status(400).json({ + error: 'validation_error', + message: parsed.error.issues[0]?.message ?? 'Neplatné údaje.', + }); + } + + const invite = newInvite({ + tenantId, + email: parsed.data.email || undefined, + note: parsed.data.note, + roleIds: parsed.data.roleIds, + asPerson: parsed.data.asPerson, + createdByEmail: req.user!.email, + }); + + return void inviteStore + .create(invite) + .then(async () => { + await refreshInvites(); + recordAudit({ + userId: req.user!.id, + userEmail: req.user!.email, + tenantId, + action: 'invite.create', + target: invite.id, + detail: { email: invite.email || '(komukoliv)' }, + }); + return res.status(201).json({ + ...invite, + url: `${publicBaseUrl()}/pozvanka/${invite.code}`, + }); + }) + .catch((err: unknown) => { + console.error('[pozvanky] zalozeni selhalo:', err); + return res + .status(500) + .json({ error: 'internal_error', message: 'Pozvánku se nepodařilo vytvořit.' }); + }); +}); + +/** Zruseni pozvanky. Odkaz okamzite prestane platit. */ +inviteRouter.delete('/:id', (req, res) => { + const tenantId = currentTenant(req, res); + if (!tenantId) return; + + return void inviteStore + .remove(req.params.id, { tenantIds: [tenantId], includeGlobal: false }) + .then(async (removed) => { + if (!removed) { + return res.status(404).json({ error: 'not_found', message: 'Pozvánka neexistuje.' }); + } + await refreshInvites(); + recordAudit({ + userId: req.user!.id, + userEmail: req.user!.email, + tenantId, + action: 'invite.delete', + target: req.params.id, + }); + return res.status(204).end(); + }) + .catch((err: unknown) => { + console.error('[pozvanky] zruseni selhalo:', err); + return res + .status(500) + .json({ error: 'internal_error', message: 'Pozvánku se nepodařilo zrušit.' }); + }); +}); + +export { PERMISSION as invitePermission }; diff --git a/src/runtime/builtinSteps.ts b/src/runtime/builtinSteps.ts index 14a6273..b955a7c 100644 --- a/src/runtime/builtinSteps.ts +++ b/src/runtime/builtinSteps.ts @@ -30,6 +30,13 @@ export interface StepContext { tenantId: string; /** Ticket, ke kteremu beh patri. Nekdy vznikne az behem nej. */ ticketId: string | null; + /** + * Data, kterymi beh zacal - u webhooku cele prijate telo. + * + * Ulozi se do udalosti ticketu, kdyz krok nema vlastni. Bez toho by v historii + * bylo prazdno a na otazku "co nam vlastne prislo" by neslo odpovedet. + */ + trigger?: Record; } export interface StepOutcome { @@ -129,8 +136,21 @@ const handlers: Record = { externalId: inputs.externalId?.trim() || null, externalSource: inputs.source?.trim() || 'automatizace', type: inputs.event?.trim() || 'automation', - label: inputs.label?.trim() || subject || 'Událost', - payload: inputs.payload ? safeJson(inputs.payload) : {}, + /* + * Popisek do casove osy. "Udalost" je posledni moznost: kdyz v nem stoji + * u vsech radku totez, casova osa nerika nic. + */ + label: + inputs.label?.trim() || + subject || + (inputs.status?.trim() ? `stav ${inputs.status.trim()}` : '') || + inputs.externalId?.trim() || + 'Událost', + /* + * Bez vlastnich dat se ulozi to, cim beh zacal. Prazdna udalost je horsi + * nez zadna: v historii se tvari, ze se neco stalo, a nerekne co. + */ + payload: inputs.payload ? safeJson(inputs.payload) : (context.trigger ?? {}), fields, create: { subject: subject || inputs.externalId || 'Bez předmětu', @@ -180,6 +200,26 @@ const handlers: Record = { if (closed !== undefined) note += closed ? ', vyřízený' : ', nevyřízený'; } + /* + * Odesilatel poslal totez co posledne. Neni to chyba, ale musi to byt videt, + * jinak nikdo nepozna, ze proti nam neco tluce ve smycce. + */ + if (result.repeated) { + return { + ok: true, + summary: `bez změny, ${result.ticket.id} už tuhle zprávu dostal (${result.event.repeats}x)`, + detail: null, + ticketId: result.ticket.id, + outputs: { + ticketId: result.ticket.id, + created: false, + externalId: result.ticket.externalId, + repeated: true, + repeats: result.event.repeats, + }, + }; + } + return { ok: true, summary: @@ -194,6 +234,49 @@ const handlers: Record = { }; }, + /** + * Preda ticket skupine. + * + * Zaskrtnute "rovnou prirad" znamena, ze ho system hned da nejvolnejsimu. + * Nezaskrtnute nechava ticket ve **fronte skupiny** a kdo ma cas, si ho + * vezme sam - to je pro vetsinu tymu spravne, protoze clovek sam vi, kdy + * na dalsi praci ma. Automat se hodi tam, kde se ceka na rychlou reakci + * a nikdo nema cas koukat do fronty. + */ + 'ticket/assign-group': (inputs, context) => { + const ticketId = inputs.ticketId?.trim() || context.ticketId; + if (!ticketId) return missing('ticket'); + + const groupId = inputs.groupId?.trim(); + if (!groupId) return missing('skupina'); + + const group = findGroup(groupId); + if (!group || group.tenantId !== context.tenantId) { + return { ok: false, summary: `skupina ${groupId} neexistuje`, outputs: {} }; + } + + const updated = assignTicketGroup(ticketId, groupId, [context.tenantId]); + if (!updated) return { ok: false, summary: 'přiřazení skupině se nepodařilo', outputs: {} }; + + if (boolInput(inputs.autoAssign) !== true) { + return { + ok: true, + summary: `${group.name}, čeká ve frontě skupiny`, + detail: null, + outputs: { groupId: group.id, groupName: group.name }, + }; + } + + // Zaskrtnuto: rovnou tomu, kdo ma nejmene prace. + const handler = handlers['ticket/assign-least-busy']; + const outcome = handler({ ticketId, groupId }, context) as StepOutcome; + return { + ...outcome, + summary: `${group.name}: ${outcome.summary}`, + outputs: { groupId: group.id, groupName: group.name, ...outcome.outputs }, + }; + }, + /** * Preda ticket **nejvolnejsimu** cloveku ze skupiny. * diff --git a/src/runtime/context.ts b/src/runtime/context.ts index 12d7aca..c908fc0 100644 --- a/src/runtime/context.ts +++ b/src/runtime/context.ts @@ -15,7 +15,17 @@ import { AsyncLocalStorage } from 'node:async_hooks'; export interface RunMarker { runId: string; automationId: string; + /** Jmeno, ne ID. V logu ticketu ho cte clovek, ne stroj. */ + automationName: string; tenantId: string; + /** + * Radek logu, pod ktery patri zmeny tohohle behu. + * + * Log ma byt citelny jako "prislo tohle a zpusobilo to tuhle zmenu". Bez + * spolecneho rodice je z nej seznam nesouvisejicich vet a nikdo nepozna, co + * zpusobilo co. Menitelna schranka proto, ze radek vznika az v prubehu behu. + */ + traceParent: { id: string | null }; } const storage = new AsyncLocalStorage(); diff --git a/src/runtime/executor.ts b/src/runtime/executor.ts index f2248c9..143c371 100644 --- a/src/runtime/executor.ts +++ b/src/runtime/executor.ts @@ -47,6 +47,13 @@ export interface RunOptions { idempotencyKey: string | null; /** Kolik kroku se nejvys vykona. Pojistka proti nechtene dlouhemu stromu. */ maxSteps?: number; + /** + * Data, kterymi beh zacal. Doplni si je `runFlow` samo. + * + * Kroky si je predavaji jako kontext, ale ten se cestou plni vystupy. Pro + * zapis do udalosti ticketu je potreba puvodni prijata zprava, ne smes. + */ + trigger?: Record; } export interface StepResult { @@ -100,6 +107,8 @@ export async function runFlow( const startedAt = Date.now(); const results: StepResult[] = []; const working: RunContext = { ...context }; + // Puvodni data se drzi zvlast: `working` se cestou plni vystupy kroku. + options.trigger = { ...context }; const limit = options.maxSteps ?? MAX_STEPS; let error: string | null = null; @@ -182,6 +191,7 @@ async function runAction( const outcome = await builtin(inputsForStep, { tenantId: options.tenantId, ticketId: options.ticketId, + trigger: options.trigger, }); if (outcome.ticketId && !options.ticketId) { diff --git a/src/runtime/worker.ts b/src/runtime/worker.ts index c9833e6..dbffd22 100644 --- a/src/runtime/worker.ts +++ b/src/runtime/worker.ts @@ -106,7 +106,13 @@ async function runOne(item: QueueItem): Promise { * ticketu ji spusti, a tak porad. */ const result = await withRun( - { runId: item.id, automationId: item.automationId, tenantId: item.tenantId }, + { + runId: item.id, + automationId: item.automationId, + automationName: automation.name, + tenantId: item.tenantId, + traceParent: { id: null }, + }, () => runFlow(automation.flow.steps, item.payload, { tenantId: item.tenantId, diff --git a/web/src/App.tsx b/web/src/App.tsx index 441dcc9..8f73498 100644 --- a/web/src/App.tsx +++ b/web/src/App.tsx @@ -9,6 +9,7 @@ import { Spinner } from '@/components/ui/Spinner'; import Home from '@/pages/Home'; import About from '@/pages/About'; import Contact from '@/pages/Contact'; +import Invite from '@/pages/Invite'; import Login from '@/pages/Login'; import NotFound from '@/pages/NotFound'; import Services from '@/pages/Services'; @@ -48,6 +49,9 @@ export default function App() { {/* Prihlaseni - vlastni layout bez hlavicky a paticky */} } /> + {/* Pozvanka do firmy. Verejna: kdo sem prijde, jeste ucet mit nemusi. */} + } /> + {/* Klientsky portal */} }> ; +} + +const inputClass = + 'w-full rounded-lg border border-ink-600/70 bg-ink-850/70 px-3 py-2 text-sm text-white placeholder:text-white/25 focus:border-brand-400/70 focus:outline-none'; + +export function InvitePanel() { + const invites = useApiQuery('/api/dashboard/invites'); + + const [email, setEmail] = useState(''); + const [note, setNote] = useState(''); + const [roleIds, setRoleIds] = useState([]); + const [asPerson, setAsPerson] = useState(true); + const [busy, setBusy] = useState(false); + const [error, setError] = useState(null); + const [copied, setCopied] = useState(null); + + const roles = invites.data?.roles ?? []; + const items = invites.data?.items ?? []; + + async function create() { + setBusy(true); + setError(null); + try { + await apiFetch('/api/dashboard/invites', { + method: 'POST', + body: { email, note, roleIds, asPerson }, + }); + setEmail(''); + setNote(''); + invites.reload(); + } catch (err: unknown) { + setError(err instanceof Error ? err.message : 'Pozvánku se nepodařilo vytvořit.'); + } finally { + setBusy(false); + } + } + + async function remove(id: string) { + try { + await apiFetch(`/api/dashboard/invites/${id}`, { method: 'DELETE' }); + invites.reload(); + } catch (err: unknown) { + setError(err instanceof Error ? err.message : 'Pozvánku se nepodařilo zrušit.'); + } + } + + async function copy(url: string, id: string) { + try { + await navigator.clipboard.writeText(url); + setCopied(id); + window.setTimeout(() => setCopied(null), 1600); + } catch (err) { + console.warn('[pozvanky] schranka neni dostupna:', err); + } + } + + return ( +
+
+

Pozvat do firmy

+

+ Vznikne odkaz s jednorázovým kódem. Pošlete ho, jak chcete. Kdo ho otevře, vyplní + si jméno a heslo a je uvnitř. Platí týden. +

+ +
+ + + +
+ +
+ Role ve firmě +
+ {roles.map((role) => { + const on = roleIds.includes(role.id); + return ( + + ); + })} +
+
+ + + + {error &&

{error}

} + + + {roleIds.length === 0 && ( + Vyberte aspoň jednu roli. + )} +
+ + +
+ + + + + + + + + + {items.map((invite) => ( + + + + + + + ))} + +
Pro kohoOdkazStav +
+ + {invite.email || 'kdokoliv s odkazem'} + + {invite.note && ( + {invite.note} + )} + + + + {invite.usedAt ? ( + použitá {formatDateTime(invite.usedAt)} + ) : invite.problem ? ( + {invite.problem} + ) : ( + + platí do {formatDateTime(invite.expiresAt)} + + )} + + +
+
+ + +

+ + Odkaz je jediná ochrana, proto má náhodný kód a jde kdykoliv zrušit. Po použití + přestane platit sám. +

+
+ ); +} diff --git a/web/src/components/dashboard/TicketEvents.tsx b/web/src/components/dashboard/TicketEvents.tsx index f18ffd4..8b58276 100644 --- a/web/src/components/dashboard/TicketEvents.tsx +++ b/web/src/components/dashboard/TicketEvents.tsx @@ -18,7 +18,7 @@ export function TicketEvents({ events }: { events: TicketEvent[] }) { if (events.length === 0) { return (

- Žádné příchozí události. Ticket vznikl v portálu nebo ze simulace. + Žádné příchozí události. Ticket vznikl v portálu nebo ručně.

); } @@ -49,11 +49,24 @@ function EventRow({ event }: { event: TicketEvent }) { - {event.label} + + {event.label} + {/* + Opakovani jako informace u radku, ne jako dalsi radek. Odesilatel + umi poslat totez osmdesatkrat a osmdesat radku znamena, ze + v historii nikdo nic nenajde. + */} + {event.repeats > 1 && ( + + {event.repeats}x beze změny + + )} + {event.type} {event.source} {formatDateTime(event.at)} + {event.repeats > 1 && naposledy {formatDateTime(event.lastAt)}} diff --git a/web/src/components/home/References.tsx b/web/src/components/home/References.tsx index 89cb467..47a250c 100644 --- a/web/src/components/home/References.tsx +++ b/web/src/components/home/References.tsx @@ -47,7 +47,7 @@ function ReferenceCard({ reference }: { reference: Reference }) {
- „{reference.quote}“ + "{reference.quote}"
diff --git a/web/src/data/products.ts b/web/src/data/products.ts index bb26f00..3edb15f 100644 --- a/web/src/data/products.ts +++ b/web/src/data/products.ts @@ -62,7 +62,7 @@ export const products: Product[] = [ name: 'Dashboardy a reporting', tagline: 'Jedna obrazovka místo pěti exportů', description: - 'Data z různých zdrojů na jednom místě — v reálném čase, s historií a s alerty, když se něco vymkne.', + 'Data z různých zdrojů na jednom místě, v reálném čase, s historií a s alerty, když se něco vymkne.', icon: BarChart3, features: ['Metriky v reálném čase', 'Automatické reporty e-mailem', 'Přístupy podle rolí'], metric: { value: '5 s', label: 'aktualizace dat' }, diff --git a/web/src/pages/Invite.tsx b/web/src/pages/Invite.tsx new file mode 100644 index 0000000..a32e6fe --- /dev/null +++ b/web/src/pages/Invite.tsx @@ -0,0 +1,179 @@ +import { AlertCircle, Building2, CheckCircle2 } from 'lucide-react'; +import { useEffect, useState } from 'react'; +import { Link, useNavigate, useParams } from 'react-router-dom'; +import { useAuth } from '@/auth/AuthContext'; +import { Button } from '@/components/ui/Button'; +import { Container } from '@/components/ui/Container'; +import { apiFetch } from '@/lib/api'; +import { usePageMeta } from '@/lib/usePageMeta'; + +/** + * Prijeti pozvanky do firmy. + * + * Verejna stranka: kdo sem prijde, jeste neni prihlaseny. Kod v adrese je + * jedina ochrana, proto se tu neukazuje nic o firme krome jmena. + * + * Kdyz uz clovek ucet ma, zada k nemu svoje heslo a jen se pripoji k dalsi + * firme. Heslo se nikdy neposila mailem - nastavuje si ho sam tady. + */ +interface InviteInfo { + tenant: string; + email: string | null; + knownUser: boolean; + valid: boolean; + problem: string | null; +} + +const inputClass = + 'w-full rounded-xl border border-ink-600/70 bg-ink-850/70 px-3.5 py-2.5 text-sm text-white placeholder:text-white/30 focus:border-brand-400/70 focus:outline-none'; + +export default function Invite() { + const { code = '' } = useParams(); + const navigate = useNavigate(); + const { login } = useAuth(); + + usePageMeta({ title: 'Pozvánka - portál Automia' }); + + const [info, setInfo] = useState(null); + const [loadError, setLoadError] = useState(null); + + const [name, setName] = useState(''); + const [email, setEmail] = useState(''); + const [password, setPassword] = useState(''); + const [busy, setBusy] = useState(false); + const [error, setError] = useState(null); + const [done, setDone] = useState(false); + + useEffect(() => { + apiFetch(`/api/invites/${code}`, { auth: false }) + .then((data) => { + setInfo(data); + if (data.email) setEmail(data.email); + }) + .catch((err: unknown) => { + setLoadError(err instanceof Error ? err.message : 'Pozvánku se nepodařilo načíst.'); + }); + }, [code]); + + async function accept() { + setBusy(true); + setError(null); + try { + await apiFetch(`/api/invites/${code}/accept`, { + method: 'POST', + auth: false, + body: { name, email, password }, + }); + setDone(true); + // Rovnou prihlasit: clovek uz heslo zadal, druhy formular by byl obtez. + await login(email, password); + navigate('/dashboard', { replace: true }); + } catch (err: unknown) { + setError(err instanceof Error ? err.message : 'Pozvánku se nepodařilo přijmout.'); + } finally { + setBusy(false); + } + } + + return ( +
+ +
+ {loadError || (info && !info.valid) ? ( + <> +

+ + {info?.problem ?? loadError} +

+

+ Požádejte o novou pozvánku toho, kdo vám poslal odkaz. +

+ + Zpět na úvod + + + ) : done ? ( +

+ + Hotovo, přesměrovávám do portálu. +

+ ) : ( + <> +

+ + Pozvánka do firmy +

+

+ {info?.tenant ?? 'Načítám...'} +

+

+ {info?.knownUser + ? 'S touhle adresou už účet máte. Zadejte k němu svoje heslo a připojíte se k firmě.' + : 'Vyplňte jméno a heslo. Účet vznikne hned a rovnou vás přihlásíme.'} +

+ +
+ + + + + + + {error && ( +

+ + {error} +

+ )} + + +
+ + )} +
+ +
+ ); +} diff --git a/web/src/pages/dashboard/AutomationDetail.tsx b/web/src/pages/dashboard/AutomationDetail.tsx index 20940f6..d31efc4 100644 --- a/web/src/pages/dashboard/AutomationDetail.tsx +++ b/web/src/pages/dashboard/AutomationDetail.tsx @@ -403,6 +403,8 @@ export default function AutomationDetail() { + +
diff --git a/web/src/pages/dashboard/Automations.tsx b/web/src/pages/dashboard/Automations.tsx index 47c35c9..6e777d0 100644 --- a/web/src/pages/dashboard/Automations.tsx +++ b/web/src/pages/dashboard/Automations.tsx @@ -132,7 +132,11 @@ function AutomationCard({ automation }: { automation: Automation }) {
- +
{label}
{value}
+ {/* Samo cislo za dnesek nic nerika, dokud neni s cim srovnat. */} + {note &&
{note}
} ); } diff --git a/web/src/pages/dashboard/People.tsx b/web/src/pages/dashboard/People.tsx index 7837c38..429fb91 100644 --- a/web/src/pages/dashboard/People.tsx +++ b/web/src/pages/dashboard/People.tsx @@ -1,13 +1,21 @@ import { Mail, Users } from 'lucide-react'; -import { useState } from 'react'; +import { useMemo, useState } from 'react'; import { Link } from 'react-router-dom'; import { DataState } from '@/components/dashboard/DataState'; +import { EntityAdmin } from '@/components/dashboard/EntityAdmin'; +import { InvitePanel } from '@/components/dashboard/InvitePanel'; import { ViewSwitch, type ListView } from '@/components/dashboard/ViewSwitch'; import { Badge } from '@/components/ui/Badge'; import { cn } from '@/lib/cn'; import { useApiQuery } from '@/lib/useApiQuery'; import { usePageMeta } from '@/lib/usePageMeta'; -import type { PeopleResponse, Workload } from '@/types/dashboard'; +import type { Access, PeopleResponse, Workload } from '@/types/dashboard'; + +interface NamedRecord { + id: string; + name?: string; + [key: string]: unknown; +} /** * Lide firmy: kdo resi tickety a jak na tom je. @@ -19,6 +27,8 @@ export default function People() { usePageMeta({ title: 'Lidé - portál Automia' }); const [view, setView] = useState('table'); + const [tab, setTab] = useState('prehled'); + const access = useApiQuery('/api/dashboard/access'); const people = useApiQuery('/api/dashboard/people'); const workload = useApiQuery('/api/dashboard/tickets/workload', { @@ -29,6 +39,29 @@ export default function People() { const byPerson = new Map(rows.map((row) => [row.person.id, row])); const items = people.data?.items ?? []; + const can = useMemo(() => { + const owned = new Set(access.data?.permissions ?? []); + return (permission: string) => owned.has(permission); + }, [access.data]); + + /** Cleny skupiny se vybiraji klikanim. Opisovat ID je zdroj preklepu. */ + const personOptions = useMemo( + () => items.map((person) => ({ value: person.id, label: person.name })), + [items], + ); + + /* + * Vsechno o lidech je na jedne strance, ne v nastaveni. Pozvat kolegu + * a zalozit skupinu neni nastaveni portalu, je to bezna denni prace - a dokud + * to bylo schovane v nastaveni, nikdo to nenasel. + */ + const tabs = [ + { key: 'prehled', label: 'Přehled', show: true }, + { key: 'resitele', label: 'Řešitelé', show: can('people.manage') }, + { key: 'skupiny', label: 'Skupiny', show: can('group.manage') }, + { key: 'pozvanky', label: 'Pozvánky', show: can('user.manage') }, + ].filter((item) => item.show); + return (
@@ -38,9 +71,31 @@ export default function People() { Kdo řeší tickety, co má u sebe a jak dlouho mu to leží.

- + {tab === 'prehled' && } + {tabs.length > 1 && ( +
+ {tabs.map((item) => ( + + ))} +
+ )} + + {tab === 'prehled' && ( + <> {workload.data && (
@@ -174,6 +229,92 @@ export default function People() {
)} + + )} + + {tab === 'resitele' && ( + String(person.name ?? '') }, + { label: 'E-mail', render: (person: NamedRecord) => String(person.email ?? '') }, + { label: 'Zaměření', render: (person: NamedRecord) => String(person.role ?? '') }, + { + label: 'Kapacita', + narrow: true, + render: (person: NamedRecord) => String(person.capacity ?? ''), + }, + { + label: 'Stav', + narrow: true, + render: (person: NamedRecord) => + person.enabled === false ? ( + vypnutý + ) : ( + aktivní + ), + }, + ]} + fields={[ + { name: 'name', label: 'Jméno', kind: 'text', required: true }, + { name: 'email', label: 'E-mail', kind: 'text', required: true }, + { + name: 'role', + label: 'Zaměření', + kind: 'text', + hint: 'Jen popisek, nic nerozhoduje.', + }, + { + name: 'capacity', + label: 'Kapacita', + kind: 'number', + initial: 8, + hint: 'Kolik nevyřízených ticketů je pro něj zdravá zátěž. Není to limit.', + }, + { + name: 'externalIds', + label: 'ID z cizích aplikací', + kind: 'tags', + hint: 'Oddělená čárkou. Podle nich umí automatizace předat ticket právě jemu, například voicebotId.', + }, + { name: 'enabled', label: 'Aktivní', kind: 'checkbox', initial: true }, + ]} + /> + )} + + {tab === 'skupiny' && ( + String(group.name ?? '') }, + { + label: 'Členů', + narrow: true, + render: (group: NamedRecord) => + String((group.personIds as string[] | undefined)?.length ?? 0), + }, + ]} + fields={[ + { name: 'name', label: 'Název', kind: 'text', required: true }, + { + name: 'personIds', + label: 'Členové', + kind: 'multiselect', + options: personOptions, + hint: 'Automatické přidělování se nastavuje v kroku automatizace, ne tady.', + }, + ]} + /> + )} + + {tab === 'pozvanky' && } ); } diff --git a/web/src/pages/dashboard/Settings.tsx b/web/src/pages/dashboard/Settings.tsx index 9fb4c48..4ac107e 100644 --- a/web/src/pages/dashboard/Settings.tsx +++ b/web/src/pages/dashboard/Settings.tsx @@ -17,7 +17,9 @@ import type { Access } from '@/types/dashboard'; * jeden zaznam v `tabs` nize. * * Akce a widgety tady **nejsou**: jsou to definice toho, co aplikace umi, - * stejna uroven jako automatizace, a maji vlastni zalozku. + * stejna uroven jako automatizace, a maji vlastni zalozku. Resitele, skupiny + * a pozvanky taky ne - patri k lidem, protoze pozvat kolegu je denni prace, + * ne nastaveni portalu. */ interface CatalogItem { @@ -74,8 +76,6 @@ export default function Settings() { const tabs = useMemo(() => { const all: Array<{ key: string; label: string; show: boolean }> = [ { key: 'ucet', label: 'Můj účet', show: true }, - { key: 'people', label: 'Řešitelé', show: can('people.manage') }, - { key: 'groups', label: 'Skupiny', show: can('group.manage') }, { key: 'roles', label: 'Role a práva', show: can('role.manage') }, { key: 'types', label: 'Typy ticketů', show: can('ticketType.manage') }, { key: 'users', label: 'Uživatelé', show: platformAdmin }, @@ -91,8 +91,8 @@ export default function Settings() {

Nastavení

- Firmy, lidé, role a typy ticketů. Akce a widgety mají vlastní záložku, - jsou to definice, ne nastavení. + Firmy, role a typy ticketů. Řešitele, skupiny a pozvánky najdete + v záložce Lidé, akce a widgety mají vlastní záložku.

@@ -116,71 +116,6 @@ export default function Settings() { {tab === 'ucet' && } - {tab === 'people' && ( - String(person.name ?? '') }, - { label: 'E-mail', render: (person: NamedRecord) => String(person.email ?? '') }, - { label: 'Zaměření', render: (person: NamedRecord) => String(person.role ?? '') }, - { label: 'Kapacita', narrow: true, render: (person: NamedRecord) => String(person.capacity ?? '') }, - { - label: 'Stav', - narrow: true, - render: (person: NamedRecord) => - person.enabled === false ? vypnutý : aktivní, - }, - ]} - fields={[ - { name: 'name', label: 'Jméno', kind: 'text', required: true }, - { name: 'email', label: 'E-mail', kind: 'text', required: true }, - { name: 'role', label: 'Zaměření', kind: 'text', hint: 'Jen popisek, nic nerozhoduje.' }, - { - name: 'capacity', - label: 'Kapacita', - kind: 'number', - initial: 8, - hint: 'Kolik nevyřízených ticketů je pro něj zdravá zátěž. Není to limit.', - }, - { - name: 'externalIds', - label: 'ID z cizích aplikací', - kind: 'tags', - hint: 'Oddělená čárkou. Podle nich umí automatizace předat ticket právě jemu, například voicebotId.', - }, - { name: 'enabled', label: 'Aktivní', kind: 'checkbox', initial: true }, - ]} - /> - )} - - {tab === 'groups' && ( - String(group.name ?? '') }, - { - label: 'Členů', - narrow: true, - render: (group: NamedRecord) => String((group.personIds as string[] | undefined)?.length ?? 0), - }, - ]} - fields={[ - { name: 'name', label: 'Název', kind: 'text', required: true }, - { - name: 'personIds', - label: 'Členové', - kind: 'tags', - hint: 'ID řešitelů oddělená čárkou. Najdete je v záložce Řešitelé.', - }, - ]} - /> - )} - {tab === 'roles' && ( + {/* + Prevzeti: kdyz ticket lezi ve fronte skupiny, vezme si ho + clovek sam. Je to jina vec nez prehazovani prace shora, proto + tlacitko a ne polozka v selectu. + */} + {people.data?.meId && ticket.data.assignee?.id !== people.data.meId && ( + + )} + diff --git a/web/src/types/dashboard.ts b/web/src/types/dashboard.ts index c749b79..9e55002 100644 --- a/web/src/types/dashboard.ts +++ b/web/src/types/dashboard.ts @@ -79,6 +79,10 @@ export interface TicketEvent { label: string; payload: Record; at: string; + /** Kolikrat prislo totez. Vic nez 1 znamena, ze odesilatel posila opakovane. */ + repeats: number; + /** Kdy naposledy. Same jako `at`, dokud se neopakuje. */ + lastAt: string; } export interface Ticket { @@ -208,7 +212,12 @@ export interface Automation { name: string; kind: AutomationKind; enabled: boolean; + /** Behy za dnesni den. Po pulnoci zacina od nuly. */ runsToday: number; + /** Behy za vcerejsek, at je s cim srovnat. */ + runsYesterday: number; + /** Behy od zalozeni automatizace. */ + runsTotal: number; successRate: number; avgDurationMs: number; lastRunAt: string;