janpeter/Ops-dashboard
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Harden Worker Insights triage verdict production #111

Closed
janpeter wants to merge 0 commits from codex/worker-insights-triage-hardening into main
pull from: codex/worker-insights-triage-hardening
merge into: janpeter:main
Conversation 1 Commits - Files changed -
janpeter commented 2026-07-18 01:17:46 +02:00
Owner
Copy link

Wat verandert

  • Valideert triageverdicts strikt vóór opslag.
  • Weigert malformed tekst en evidence zonder bestaande event-sequences.
  • Slaat guardrail-only kandidaten over, maar behoudt geldige token_expired runs.

Waarom

De workers-consumer groepeert persistente verdicts tot incidenten. Zonder een strikt producercontract konden malformed of semantisch ongeschikte verdicts downstream als actiegericht incident verschijnen.

Impact

Alleen valide, bewijsbare verdicts bereiken status done. Bestaande retry- en failurepaden blijven intact.

Cross-repo contract

Companion consumer-PR: janpeter/scrum4me-workers#70

Deze producer-PR moet vóór of atomair met de workers-PR worden geïntegreerd.

Verificatie

  • npm test -- --run: 32 bestanden, 163 tests geslaagd.
  • npm run typecheck: geslaagd.
  • Onafhankelijke review: Ready: yes.
## Wat verandert - Valideert triageverdicts strikt vóór opslag. - Weigert malformed tekst en evidence zonder bestaande event-sequences. - Slaat guardrail-only kandidaten over, maar behoudt geldige `token_expired` runs. ## Waarom De workers-consumer groepeert persistente verdicts tot incidenten. Zonder een strikt producercontract konden malformed of semantisch ongeschikte verdicts downstream als actiegericht incident verschijnen. ## Impact Alleen valide, bewijsbare verdicts bereiken status `done`. Bestaande retry- en failurepaden blijven intact. ## Cross-repo contract Companion consumer-PR: https://git.jp-visser.nl/janpeter/scrum4me-workers/pulls/70 Deze producer-PR moet vóór of atomair met de workers-PR worden geïntegreerd. ## Verificatie - `npm test -- --run`: 32 bestanden, 163 tests geslaagd. - `npm run typecheck`: geslaagd. - Onafhankelijke review: Ready: yes.
fix(insights): retain token-expired triage candidates
Some checks failed
CI / Root app checks (pull_request) Failing after 0s
Details
CI / Ops-agent checks (pull_request) Failing after 0s
Details
CI / Deploy artifact checks (pull_request) Failing after 0s
Details
CI / Docker image build (pull_request) Failing after 0s
Details
8f07221316
janpeter changed title from WIP: Harden Worker Insights triage verdict production to Harden Worker Insights triage verdict production 2026-07-18 01:22:22 +02:00
s4m-codex-reviewer reviewed 2026-07-18 01:25:37 +02:00
s4m-codex-reviewer left a comment
Copy link

Verdict: COMMENT

Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden.

Findings

  • warning — PR diff:1 — De aangeleverde pr_diff is leeg. Daardoor kan deze review niet verifiëren welke code is gewijzigd, of de wijziging voldoet aan productpatronen, of tests/docs passend zijn bijgewerkt. Safe-default: niet approven zonder beoordeelbare diff.

Plan-conformiteit

Niet van toepassing: er is geen gekoppeld plan gevonden.

Product-standaarden

Relevante productdocs zijn geraadpleegd, waaronder de MD3/shadcn stylingconventies. Door de lege diff is niet vast te stellen of de PR deze standaarden raakt of naleeft.

# Verdict: COMMENT Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. ## Findings - **warning** — `PR diff:1` — De aangeleverde `pr_diff` is leeg. Daardoor kan deze review niet verifiëren welke code is gewijzigd, of de wijziging voldoet aan productpatronen, of tests/docs passend zijn bijgewerkt. Safe-default: niet approven zonder beoordeelbare diff. ## Plan-conformiteit Niet van toepassing: er is geen gekoppeld plan gevonden. ## Product-standaarden Relevante productdocs zijn geraadpleegd, waaronder de MD3/shadcn stylingconventies. Door de lege diff is niet vast te stellen of de PR deze standaarden raakt of naleeft.
janpeter closed this pull request 2026-07-18 01:26:17 +02:00
Some checks failed
CI / Root app checks (pull_request) Failing after 0s
Required
Details
CI / Ops-agent checks (pull_request) Failing after 0s
Required
Details
CI / Deploy artifact checks (pull_request) Failing after 0s
Required
Details
CI / Docker image build (pull_request) Failing after 0s
Required
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
janpeter/Ops-dashboard!111
No description provided.